Add next-step hints to guide users through command chains - #241
Merged
Conversation
- Add `↳ view a session: mcpc @sessionname` hint after listing sessions in `mcpc` (no args), so users and agents see what to do next. - Move `pass --stdio to include` out of the `mcpc connect` summary line into a dedicated `↳ run: mcpc connect --stdio` hint, matching the unauthorized/expired-session hint style. - Document the convention as a new "Next-step hints" principle in CLAUDE.md.
Adds a one-liner before the install snippet pointing newbies to nodejs.org and bun.sh so they don't have to figure out where to get the runtime first.
Previously the hint was placed directly after the sessions list, where its 2-space indent collided visually with the 4-space recovery hints under each session (e.g. `↳ run: mcpc @test restart`), making it look like a per-session hint rather than a global one. Now `listSessionsAndAuthProfiles` returns `hasSessions`, and the caller prints the view-session hint immediately above the existing `Run "mcpc --help"...` line — visually grouped with the global footer instead of nested inside the session list.
- Rephrase the view-session hint to match the existing `Run "..."` footer line: `Run "mcpc @sessionname" to view session details.` sits naturally next to `Run "mcpc --help" for usage information.` - Fix `formatTimeAgo` to use singular form when the count is 1: `1 week ago` / `1 month ago` instead of `1 weeks ago` / `1 months ago`.
Aligns the two footer lines with the existing `To X, run: ...` / `For X, run: ...` pattern used in tasks.ts, helpers.ts, bridge-manager.ts, and errors.ts: To view server info and capabilities, run: mcpc @session For usage information, run: mcpc --help
jancurn
commented
May 29, 2026
…install - New `test/unit/cli/sessions.test.ts` covers all branches of `formatTimeAgo`, including the singular/plural fix for `1 week ago` and `1 month ago`. Uses fake timers for deterministic results. - README install section: drop the `(≥ 18)` / `(≥ 1)` version qualifiers and say "install the latest" instead, per review comment on #241.
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
3 participants
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.
Summary
mcpc(list sessions) andmcpc connect(skipped stdio) so the next command is discoverable without--help. Documented as a new principle inCLAUDE.md.formatTimeAgopluralization:1 week ago/1 month agoinstead of1 weeks ago/1 months ago.Test plan
mcpcwith at least one session printsTo view server info and capabilities, run: mcpc @sessionaboveFor usage information, run: mcpc --help; with zero sessions the first line is omitted.mcpc connect(auto-discovery) prints↳ run: mcpc connect --stdioafter the skipped-stdio summary.created 1 week ago(singular).https://claude.ai/code/session_015wCgMKyoa4kMuvHiqdo5JY