Skip to content

docs(install): make the no-terminal Claude Code path unmistakable - #154

Merged
sajor2000 merged 2 commits into
mainfrom
docs/no-terminal-path
Oct 21, 2026
Merged

sajor2000 merged 2 commits into
mainfrom
docs/no-terminal-path

Conversation

@sajor2000

Copy link
Copy Markdown
Owner

What

The docs-only fallback from #142: a "No terminal at all" path in docs/setup.md (section 1) and a matching note in the README's "Claude Code, easiest path".

Once Claude Code itself runs on any surface (desktop app or terminal — plugin install works on any surface per the official plugin docs), the marketplace install, /methodsmith:method-setup, and auto-update enablement are all in-session; only the source-checkout and offline-artifact routes need a terminal or a packaged file.

Decision recorded

The full packaged classroom installer (signed binaries, update surface) stays out of scope for now, per the docs-only fallback named in the issue. Reopen #142 if the installer becomes wanted.

Closes #142

Validation

  • bun run release:validate — in sync
  • bun test tests/displayed-counts.test.ts — pass
  • Markdown-only change

MedSci audit gap #3 docs-only fallback (#142): the packaged classroom
installer stays out of scope for now, but the audience that avoids
terminals gets an explicit route - once Claude Code runs on any surface
(desktop app or terminal), the marketplace install, method-setup, and
auto-update are all in-session. Per code.claude.com/docs/en/discover-
plugins, plugin install works on any Claude Code surface.

Closes #142

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

Next included review available in 56 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 68 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 5ce1b181-9c8d-422b-9274-39bb5c3d347f
📥 Commits

Reviewing files that changed from the base of the PR and between 7a61c2c and ffe2227.

📒 Files selected for processing: 2
  • README.md
  • docs/setup.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@sajor2000
sajor2000 merged commit 31d0292 into main Oct 21, 2026
3 checks passed
@sajor2000
sajor2000 deleted the docs/no-terminal-path branch October 21, 2026 03:35
sajor2000 added a commit that referenced this pull request Oct 21, 2026
Gaps #3, #5, #7, #10 closed (PRs #150, #151, #153, #154); #1, #6, #8
tracked as issues #141, #144, #146 with scoping; #9 wontfix as #147.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
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.

Decide whether a no-terminal install path is in scope (MedSci audit gap #3)

1 participant