Skip to content

docs: extract agent-neutral install requirements from the Bob doc - #492

Merged
randomparity merged 3 commits into
mainfrom
feat/agent-install-requirements-483
Oct 30, 2026
Merged

randomparity merged 3 commits into
mainfrom
feat/agent-install-requirements-483

Conversation

@randomparity

@randomparity randomparity commented Oct 30, 2026 •

Copy link
Copy Markdown
Owner

Summary

Splits docs/bob-install-requirements.md into an agent-neutral core and the Bob-specific remainder, so the per-agent docs planned under #482 (#484–#487) can link one copy of the shared facts.

  • New docs/agent-install-requirements.md holds what was Bob doc §1 (minimum file tree), §2 (<plugin root> resolution under copy vs symlink installs), and the adept-side half of §6 (the Claude Code/Codex-only feature table with its adept location column and the agents/openai.yaml description). ~/.bob/ is replaced by <install root> / ~/.<agent>/ placeholders, and the verification-marker legend now lives here with its environment line unchanged.
  • docs/bob-install-requirements.md keeps §3, §4, §5, §7 and the Bob column of §6. A short "1–2. Install root" section states Bob's install roots (~/.bob/, <project>/.bob/), keeps the Bob-specific project-precedence sentence, and links the core. Section numbers 3–7 are unchanged; §4's reference to §2 now links the core. §7 is byte-identical.
  • .claude-plugin/plugin.json 7.12.4 → 7.12.5 (PATCH, ADR 0022).

Claims are moved, not rewritten. The only wording changes are placeholder substitutions, "Bob install" → "install" in moved sentences, "the issue body" → "Issue #478's body" (it no longer sits beside the Bob doc's #478 reference), and the marker legend's "Bob's published documentation"/"IBM" → "the agent's"/"the agent's vendor". The Bob §6 agents/openai.yaml cell drops "these files are Codex CLI-specific", which the core now states.

Verification

  • BASE_SHA=<base> ./scripts/check-plugin-version.sh: exit 1 before the bump, exit 0 after.
  • just commit-check, just shape-check, just plugin-check: exit 0.
  • Managed pre-push hook (just ci) on the pushed head: exit 0.
  • Single-pass gauntlet review returned one high finding (leftover old §6 table in the Bob doc). That escalated the review to $trial-loop, which fixed the finding in 24cb65c and approved on its confirming pass.
  • Security pass: not triggered (docs and version field only).

Closes #483

Review exit payloads

Escalation: the single-pass gauntlet review (lens consumer-installation) returned one high finding, which escalated the review to $trial-loop (lens public-safety, budget 2). Pass 1 raised the same finding: leftover pre-change §6 table rows and detail paragraph in docs/bob-install-requirements.md. It was accepted and fixed in 24cb65c. Pass 2 approved. Review rounds: 2 this run, 3/1 cumulative.

Deferrals: none.

Follow-up candidates:

Title File evidence Trigger Recommendation Source pass
Moved citation check-public-safety.sh:4,11 points past the exec line docs/agent-install-requirements.md §1 note cites scripts/check-public-safety.sh:4,11; the script is 9 lines with exec at line 9 (pre-existing in the Bob doc) reproduced Correct the citation to :4,9 in a follow-up; this PR moves claims without rewriting them trial-loop pass 2

Rejected with evidence:

Outstanding notes: none.

Move the minimum file tree, <plugin root> resolution, and the adept-side
half of the Claude Code/Codex-only feature table into
docs/agent-install-requirements.md, written against <install root>.
The Bob doc keeps its install root, sections 3-5 and 7, and the Bob
column of section 6, and links the core for the rest.

Refs #483
@randomparity

Copy link
Copy Markdown
Owner Author

verdict: approve
exit: none
findings: 2
iterations: 2
security: not triggered
delivered-head-sha: 24cb65c

Forge whole-branch review

forge review: not required (Cast governed-small-change with no whole-branch reviewer)

Review exit payloads

Escalation: the single-pass gauntlet review (lens consumer-installation) returned one high finding, which escalated the review to $trial-loop (lens public-safety, budget 2). Pass 1 raised the same finding: leftover pre-change §6 table rows and detail paragraph in docs/bob-install-requirements.md. It was accepted and fixed in 24cb65c. Pass 2 approved. Review rounds: 2 this run, 3/1 cumulative.

Deferrals: none.

Follow-up candidates:

Title File evidence Trigger Recommendation Source pass
Moved citation check-public-safety.sh:4,11 points past the exec line docs/agent-install-requirements.md §1 note cites scripts/check-public-safety.sh:4,11; the script is 9 lines with exec at line 9 (pre-existing in the Bob doc) reproduced Correct the citation to :4,9 in a follow-up; this PR moves claims without rewriting them trial-loop pass 2

Rejected with evidence:

Outstanding notes: none.

@randomparity
randomparity merged commit 73e6ffb into main Oct 30, 2026
5 checks passed
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.

Extract agent-neutral install requirements from the Bob doc

1 participant