Repository navigation
Hide the unreleased inbox command from help and the agent skill - #1750
Merged
Merged
Conversation
The Inbox back end (#1745) ships in the binary, but its window is still a stub, so 0.28.7 must not advertise it. `plannotator inbox` keeps working (and keeps `inbox --help`) for developers and the coming connections; it is only taken out of discovery. - cli.ts: HIDDEN_SUBCOMMANDS (today `inbox`). Top-level help no longer lists the inbox lines. - unknown-subcommand.ts: hidden subcommands stay known to the dispatcher but are never offered as a "Did you mean" suggestion. - The plannotator knowledge skill drops its Inbox section and table row. - plannotator-skill-reference.test.ts: every non-hidden subcommand must still be documented; a hidden one must be ABSENT from the skill and the top-level help, and must still be a real subcommand. - AGENTS.md: one line on HIDDEN_SUBCOMMANDS and how to un-hide. Co-Authored-By: Claude <noreply@anthropic.com>
1 task
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Inbox back end (#1745) is in the binary, but its window is still a stub page, so the next release (0.28.7) must not advertise it (owner: "don't show inbox until we release it").
plannotator inbox,inbox --background,inbox --no-open,inbox mcpandinbox --helpall keep working. They are only removed from the places that advertise the CLI.Hidden from:
plannotator --help: the two inbox usage lines are gone.plannotator inbono longer suggestsinbox.inboxis still a known subcommand, so it never reports "Unknown command".apps/skills/core/plannotator/SKILL.md): the## plannotator inboxsection and the table row are removed. The Pi and OpenCode copies are generated from this file at build time (both gitignored), and install.sh copies the same source, so nothing else needed editing.Mechanism:
HIDDEN_SUBCOMMANDSinapps/hook/server/cli.ts. The freshness guard (plannotator-skill-reference.test.ts) still requires every non-hidden subcommand to be documented in the skill. A hidden subcommand must be absent from the skill and from the top-level help, and it must still have aSUBCOMMAND_HELPentry, so a stale name in the set cannot make the test pass on nothing. I checked both directions by hand:inboxadded back to the skill, the test fails: "is unreleased … but SKILL.md documents it".plannotator inboxis not documented".To un-hide: delete it from the set, add the top-level usage lines back, and document it in the skill. AGENTS.md has a one-line note about this beside the skill's entry in the project structure. I kept that note out of the Inbox section to avoid conflicts with the inbox thread-routing work.
Checked and left as is: the marketing docs, the README, the plugin command stubs, the OpenCode/Pi commands and the
plannotatortool contract never mention the Inbox. Thesession.inbox.*names in the OpenCode plugin are OpenCode's own events and have nothing to do with this. The Inbox's own MCP instructions are seen only by an agent that has already registeredplannotator inbox mcp.uninstall --dry-runlists the Inbox only when its data exists.Proof (temp
PLANNOTATOR_DATA_DIR):bun apps/hook/server/index.ts --help: no inbox lines.… inbo: prints "Unknown command" with no suggestion.… inbox --help: still prints its usage.… inbox --background: printed its URL and exited 0./api/inbox/healthanswered, and the token-guarded stop route then stopped it.bun run typecheckpasses.bun test apps/hook packages/server: 1729 pass, 2 skip, 0 fail.🤖 Generated with Claude Code