Repository navigation
docs: document Muse Code, Node.js 22 and shipped MSIX widgets - #327
Conversation
|
@sfl review |
There was a problem hiding this comment.
✅ Verdict: PASS • Round 1
Confidence: 9/10
Review done by claude-opus-5-5 (same family: same-family: codex gated: codex Weekly 161% SUS 62 🔴 pause, defer)
Docs-only change: it adds Muse Code to the README, ACCOUNT_CONFIGURATION, PRIVACY and SUBSCRIPTION_PLANS, raises the README Node.js prerequisite to 22 and updates the iOS-vs-Windows widget and distribution rows for the shipped MSIX package. I checked every new claim against the exact-SHA implementation and docs and found no defects in the security, accuracy or quality passes.
🔨 Full-Spectrum Review Summary
Effort: high
Review time: 1m 31s
Token count: Unavailable
| Pass | 🔴 Critical | 🟠 High | 🟡 Medium | ⚪ Low |
|---|---|---|---|---|
| 🔒 Security | 0 | 0 | 0 | 0 |
| ✅ Accuracy | 0 | 0 | 0 | 0 |
| 🧰 Quality & Maintainability | 0 | 0 | 0 | 0 |
| Total | 0 | 0 | 0 | 0 |
Key Findings
No material defects found.
Coverage
- Files reviewed:
README.mddocs/ACCOUNT_CONFIGURATION.mddocs/IOS_DIFFERENCES.mddocs/PRIVACY.mddocs/SUBSCRIPTION_PLANS.md
- Context read:
docs/MUSE_CODE.mdcrates/codexbar-providers/src/muse.rscrates/codexbar-providers/fixtures/muse/measured.jsoncrates/codexbar-app/src/catalog.rscrates/codexbar-app/src/settings_view.rscrates/codexbar-store/src/settings.rscrates/codexbar-core/src/account.rspackage.jsondocs/PACKAGING.mddocs/NPM_ADVISORIES.md.github/workflows/ci.yml
- Verified:
- Compare
base_commit.shais78e91a4fc7245b407a6bd32af47186f3d59321c9, the head commit is2eaf8038c2bf2e42187614f548c279fecff7c4e9, the merge base equals the base, and the head is ahead by 1 and behind by 0 names::MUSE_CODEis"MuseCode"insettings.rs, which matches the newproviderIdvaluecatalog.rssetsmulti_account: falseforOPENCODE_ZEN,X_SUBSCRIPTIONandMUSE_CODE, andsettings_view.rsgates the add button oninfo.multi_account; this matches the revised several-accounts sentenceSettings::is_enabledreturns false forMUSE_CODEuntil an account exists, and Moonshot defaults to off, which matches the PRIVACY first-start listMuseProvider::fetchrunsmuse usage --json, removesMETA_API_KEY(and the other MUSE_* variables) from the child environment, and caches a success for 5 minutes or until the earliest reset, which matches PRIVACYparsereadsallowance.tierthroughsafe_text(64 chars, charset check) intowith_plan, and the measured fixture tier is the Muse Code Power Usage example, which matches the SUBSCRIPTION_PLANS rowpackage.jsonengines.nodeis>=22and NPM_ADVISORIES says Node.js 22 or newer, which matches the README- The
docs/PACKAGING.md## Widgetsheading exists for the#widgetsanchor and describes the Windows 11 Widgets board, Developer Mode for the self-signed package and the App Installer update channel - The added table rows keep each table's column count, and README provider-setup numbering continues at 13
- Compare
- Not verified:
- Tests run: None. Reviewer execution is disabled.
HemSoft full-spectrum review • 3 evidence-based passes • Reviewed head 2eaf8038c2bf2e42187614f548c279fecff7c4e9 • SFL v0
Summary
Docs-only fixes for three findings from the 2026-11-07 docs audit, each checked against
origin/mainat78e91a4before editing.Closes #324
Closes #325
Closes #326
#324: Muse Code in the user-facing docs
README.md: adds Muse Code to the Providers table and as step 13 of Provider setup, and removes the two stray Muse lines that sat under License.docs/ACCOUNT_CONFIGURATION.md: addsMuseCodeto theproviderIdlist (names::MUSE_CODE = "MuseCode") and a Muse Code row (Command line, one account) to Sign-in methods.docs/SUBSCRIPTION_PLANS.md: adds a Muse Code row to Sources and limits, givingallowance.tieras the plan source (muse.rspasses it towith_planafter the text check).docs/PRIVACY.md: lists Muse Code among the providers that are off until an account exists (settings.rsis_enabled), and adds a paragraph on what themuse usage --jsoncall does, how its result is cached and what CodexBar keeps, followingdocs/MUSE_CODE.md.Decision:
ACCOUNT_CONFIGURATION.mdsaid every provider except OpenCode Zen takes several accounts. Incatalog.rs, X Subscription and Muse Code aremulti_account: falsetoo, andsettings_view.rsshows the add button only for multi-account providers. That sentence now names all three.#325: Node.js prerequisite
README.mdnow says Node.js 22 or later, matchingpackage.jsonengines.node >=22anddocs/NPM_ADVISORIES.md. CI uses Node 24 (ci.ymlnode-version: '24').#326: Shipped widgets and MSIX in
docs/IOS_DIFFERENCES.md#93, #94 and #95 are closed, and
package.ps1andcrates/codexbar-app/src/widgets/exist. The widgets row now names the Windows 11 Widgets board, from the packaged app with Developer Mode. The distribution row now says a self-signed MSIX package with local-channel updates, or a build from source. Both rows link todocs/PACKAGING.md.Validation
npm run lint:md: exit 0git diff --check: clean🤖 Generated with Claude Code