Merge pull request #374 from docker/docker-model-run-short

ericcurtin · web-flow · commit c183daac6fa1 · 2025-11-06T09:56:14.000Z
Expand if possible
diff --git a/cmd/cli/desktop/desktop.go b/cmd/cli/desktop/desktop.go
@@ -266,10 +266,9 @@ func (c *Client) Inspect(model string, remote bool) (dmrm.Model, error) {
 			!strings.Contains(model, "@") {
 			// Do an extra API call to check if the model parameter isn't a model ID.
 			modelId, err := c.fullModelID(model)
-			if err != nil {
-				return dmrm.Model{}, fmt.Errorf("invalid model name: %s", model)
+			if err == nil {
+				model = modelId
 			}
-			model = modelId
 		}
 	}
 	rawResponse, err := c.listRawWithQuery(fmt.Sprintf("%s/%s", inference.ModelsPrefix, model), model, remote)

Original file line number	Diff line number	Diff line change
`@@ -266,10 +266,9 @@ func (c *Client) Inspect(model string, remote bool) (dmrm.Model, error) {`
`266`	`266`	`!strings.Contains(model, "@") {`
`267`	`267`	`// Do an extra API call to check if the model parameter isn't a model ID.`
`268`	`268`	`modelId, err := c.fullModelID(model)`
`269`		`- if err != nil {`
`270`		`- return dmrm.Model{}, fmt.Errorf("invalid model name: %s", model)`
	`269`	`+ if err == nil {`
	`270`	`+ model = modelId`
`271`	`271`	`}`
`272`		`- model = modelId`
`273`	`272`	`}`
`274`	`273`	`}`
`275`	`274`	`rawResponse, err := c.listRawWithQuery(fmt.Sprintf("%s/%s", inference.ModelsPrefix, model), model, remote)`