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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions