Skip to content

fix: an unknown --mount is refused, not read as an empty store - #107

Merged
eyelock merged 1 commit into
developfrom
fix/unknown-mount
Oct 30, 2026
Merged

eyelock merged 1 commit into
developfrom
fix/unknown-mount

Conversation

@eyelock

@eyelock eyelock commented Oct 30, 2026

Copy link
Copy Markdown
Owner

--mount with a mount that doesn't exist used to match nothing and exit 0. So ynm recall --mount projet, or --mount project outside an initialised repository, looked exactly like an empty store. For the Wheelhouse doctor, that reads as "no seat has an identity".

Now it's refused with exit 2, and the error names the mounts there are:

$ ynm recall --mount bogus
 ›   Error: no mount "bogus" (have: personal, project, team)
  • The check: Ynm.requireMount() accepts both local mounts and remote (hosted) ones. Every read that filters by mount calls it: recall, context, list, export, review, reindex, sync and wiki build.
  • The exit code: it throws an UnknownMountError, which the CLI maps to exit 2, as it does for other input it refuses. docs/reference/exit-codes-and-json.md now lists the case.
  • Tests:
    • A service test covers each read path.
    • A CLI test runs recall, list, context, export and wiki build outside a repository with --mount project, and checks they exit 2 with the message.
    • Live in ynw: --mount bogus exits 2, while --mount project and the remote --mount team still work.
  • Checks:
    • make check, make typecheck, make test-tutorials (12/12) and make docs-links pass.
    • All test tasks pass except the service doctor test that fails on any machine with opencode on PATH. It fails the same way on develop.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@eyelock
eyelock merged commit bc22cd3 into develop Oct 30, 2026
5 checks passed
@eyelock
eyelock deleted the fix/unknown-mount branch October 30, 2026 14:39
@eyelock eyelock mentioned this pull request Oct 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant