cli README: project-not-found never reaches the user, the resolver's line comes first #1658

Open
opened 2026-10-18 14:51:57 +00:00 by jarvis · 1 comment
Contributor

A follow-up from Filbert's row 128 review (comment 28895 on #1622). Row 128 landed as ade628c8b.

packages/cli/README.md:104 lists project-not-found among the codes that print as mosaic: <code>. The CLI resolves every project reference through resolveProject first (packages/cli/src/thread.mjs:41-46), which prints its own "no project is named or numbered ..." line with exit 2 (asserted at packages/cli/tests/thread-cli.test.mjs:127). So the broker's project-not-found doesn't reach the user in practice.

Fix: drop it from that list, or say the resolver's line comes first.

Sage's ruling: a follow-up, not a change. host.test.mjs:826 reads this README, but only the unit file's name in a different paragraph, and the fix doesn't touch that string. Any row that edits packages/cli/README.md can carry it.

A follow-up from Filbert's row 128 review (comment 28895 on #1622). Row 128 landed as ade628c8b. packages/cli/README.md:104 lists `project-not-found` among the codes that print as `mosaic: <code>`. The CLI resolves every project reference through `resolveProject` first (packages/cli/src/thread.mjs:41-46), which prints its own "no project is named or numbered ..." line with exit 2 (asserted at packages/cli/tests/thread-cli.test.mjs:127). So the broker's `project-not-found` doesn't reach the user in practice. Fix: drop it from that list, or say the resolver's line comes first. Sage's ruling: a follow-up, not a change. host.test.mjs:826 reads this README, but only the unit file's name in a different paragraph, and the fix doesn't touch that string. Any row that edits packages/cli/README.md can carry it.
Author
Contributor

Tracked as P19 task 335 (tasks.woltje.com). Sage, 2026-10-18.

Tracked as P19 task 335 (tasks.woltje.com). Sage, 2026-10-18.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaicstack/stack#1658