Lowercase model names in ps output

[tmchow]

The ps command displayed model names in whatever case was used during loading, while ls always showed lowercase names from the registry. This made the output inconsistent between the two commands.

Now ps lowercases model names after stripping defaults, matching ls behavior. The e2e test no longer needs the strings.ToLower workaround that was compensating for this.

Resolves the TODO at e2e/cli_test.go:48.

Note: This PR was authored by a human with AI coding assistance.

[sourcery-ai]

Hey - I've left some high level feedback:

• To avoid future divergence between ps and ls, consider extracting a shared model-name normalization helper (including default stripping and lowercasing) and reusing it in both commands instead of inlining the logic here.
• If SHA-based model identifiers also need consistent treatment, consider whether strings.ToLower should apply in both branches of the if strings.HasPrefix(modelName, "sha256:") check, or explicitly document why hashes are exempt.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- To avoid future divergence between `ps` and `ls`, consider extracting a shared model-name normalization helper (including default stripping and lowercasing) and reusing it in both commands instead of inlining the logic here.
- If SHA-based model identifiers also need consistent treatment, consider whether `strings.ToLower` should apply in both branches of the `if strings.HasPrefix(modelName, "sha256:")` check, or explicitly document why hashes are exempt.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[gemini-code-assist]

Code Review

This pull request updates the ps command to lowercase model names for consistency with the ls command and updates the corresponding E2E tests. A correctness issue was identified in cmd/cli/commands/ps.go where strings.ToLower is applied after stripDefaultsFromModelName. Since the stripping logic is case-sensitive, it should be applied to the lowercased string to ensure defaults like :latest are correctly removed from mixed-case inputs.

[ericcurtin]

I think the bot comment is worth addressing

[tmchow]

Good catch, fixed in 249bb42. Moved strings.ToLower inside the stripDefaultsFromModelName call so case-sensitive prefix/suffix matching works correctly on mixed-case input.