Skip to content

CLI polish: make restore path handling explicit #168

@jonathanpopham

Description

@jonathanpopham

Summary

During v0.6.14 production surface verification, supermodel restore --local --dir /definitely/missing correctly failed with a missing-directory error.

One polish issue surfaced: supermodel restore --local /some/path silently ignores the positional argument because the command only reads the --dir flag and does not validate args. The command docs/examples use --dir, so this is not blocking, but it is easy to misread from a CLI-user perspective.

Proposed fix

Pick one behavior and test it:

  • Prefer: accept an optional positional path (restore [path]) and map it to the same behavior as --dir.
  • Alternative: set Args: cobra.NoArgs so unexpected positional args fail fast.

Verification context

  • Release: v0.6.14
  • Happy path verified: restore . --local --max-tokens 1000
  • Reverse path verified: restore --local --dir /definitely/missing exits non-zero

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions