Skip to content

Deprecate shell command and hide from help output#243

Merged
jancurn merged 1 commit into
mainfrom
claude/let-xowdx
May 20, 2026
Merged

Deprecate shell command and hide from help output#243
jancurn merged 1 commit into
mainfrom
claude/let-xowdx

Conversation

@jancurn
Copy link
Copy Markdown
Member

@jancurn jancurn commented May 20, 2026

Summary

Deprecates the shell command (mcpc shell @<session> and mcpc @<session> shell) by hiding it from help output and adding a deprecation warning when invoked. The command will be removed in a future release.

Changes

  • Hidden from help: Both shell command variants are now marked as hidden in the CLI command definitions, removing them from --help output
  • Deprecation warning: Added a deprecation warning message that prints to stderr when the shell command is invoked
  • Updated documentation:
    • Added deprecation notice to CHANGELOG.md
    • Updated README.md help output to reflect the current command set (without shell)
    • Removed shell from the "Available commands" section in server details output
  • Test updates: Updated unit tests to reflect that shell is no longer listed in available commands and is not shown when a server has no capabilities

Implementation Details

  • The deprecation warning uses the existing formatWarning() utility for consistent styling
  • The "Available commands" section in server details is now only shown if there are actual commands to display (previously always shown even if only shell was available)
  • Both top-level (mcpc shell @<session>) and session-level (mcpc @<session> shell) command variants are deprecated consistently

https://claude.ai/code/session_01UvWPJHftasQatWBX7Uisbh

Hide both `mcpc shell @<session>` and `mcpc @<session> shell` from
`--help` output, and print a deprecation warning to stderr when the
command is invoked. The command itself still works.

Also drop the `mcpc <target> shell` hint from the "Available commands"
list shown by `mcpc @<session>` so we stop advertising it.
@jancurn jancurn merged commit 4c5e9dc into main May 20, 2026
6 checks passed
@jancurn jancurn deleted the claude/let-xowdx branch May 20, 2026 22:36
jancurn pushed a commit that referenced this pull request May 20, 2026
Resolves a README.md conflict from #243 (shell command deprecation) by
adopting main's help block — the `shell <@session>` row is gone from
`mcpc --help`. Also removes docs/vhs/shell.tape and its references
from docs/vhs/README.md and docs/vhs/mcpc-demo.tape so the templates
don't showcase a now-deprecated command.

Refs #242, #243
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.

3 participants