Merge pull request #336 from jwjohns/feature/context-size-in-list

ericcurtin · web-flow · commit 9983314c2885 · 2025-10-28T22:15:32.000Z
fix(cli): log context parsing errors
diff --git a/cmd/cli/commands/list.go b/cmd/cli/commands/list.go
@@ -158,13 +158,15 @@ func appendRow(table *tablewriter.Table, tag string, model dmrm.Model) {
 	}
 	// Strip default "ai/" prefix and ":latest" tag for display
 	displayTag := stripDefaultsFromModelName(tag)
-	contextSize := "-"
+	contextSize := ""
 	if model.Config.ContextSize != nil {
 		contextSize = fmt.Sprintf("%d", *model.Config.ContextSize)
 	} else if model.Config.GGUF != nil {
 		if v, ok := model.Config.GGUF["llama.context_length"]; ok {
 			if parsed, err := strconv.ParseUint(v, 10, 64); err == nil {
 				contextSize = fmt.Sprintf("%d", parsed)
+			} else {
+				fmt.Fprintf(os.Stderr, "invalid context length %q for model %s: %v\n", v, model.ID, err)
 			}
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -158,13 +158,15 @@ func appendRow(table *tablewriter.Table, tag string, model dmrm.Model) {`
`158`	`158`	`}`
`159`	`159`	`// Strip default "ai/" prefix and ":latest" tag for display`
`160`	`160`	`displayTag := stripDefaultsFromModelName(tag)`
`161`		`- contextSize := "-"`
	`161`	`+ contextSize := ""`
`162`	`162`	`if model.Config.ContextSize != nil {`
`163`	`163`	`contextSize = fmt.Sprintf("%d", *model.Config.ContextSize)`
`164`	`164`	`} else if model.Config.GGUF != nil {`
`165`	`165`	`if v, ok := model.Config.GGUF["llama.context_length"]; ok {`
`166`	`166`	`if parsed, err := strconv.ParseUint(v, 10, 64); err == nil {`
`167`	`167`	`contextSize = fmt.Sprintf("%d", parsed)`
	`168`	`+ } else {`
	`169`	`+ fmt.Fprintf(os.Stderr, "invalid context length %q for model %s: %v\n", v, model.ID, err)`
`168`	`170`	`}`
`169`	`171`	`}`
`170`	`172`	`}`