Repository navigation
docs: add Bob install requirements document - #479
Merged
Merged
Conversation
Records the minimum file tree, plugin-root resolution behaviour, symlink-following caveats, skill naming, collision/idempotency rules, Bob-unsupported features, and open blocking decisions for a future Bob installer. Each claim is marked verified or unverified with the environment used. Closes #478
…n bump - Add 'documented' verification marker for Bob-docs-sourced claims - Fix file tree ASCII diagram to show references/ and scripts/ as siblings of skills/ inside ~/.bob/ - Correct ADR 0001 quote attribution (text is from CLAUDE.md, not ADR) - Fix version-field table row to cite plugin.json, not marketplace.json - Fix plan citation from line 979 to line 969 for the SKIPPED header - Attribute symlink measurement to issue body, not a re-run - Bump plugin version to 7.12.3 (required per-PR edit, ADR 0022)
…refs - Fix §2 symlink table: logical-cd scripts resolve to ~/.bob/, not checkout; physical-cd scripts resolve to checkout (split, not both fail) - Fix §4 harness injection claim: mark as unverified under Bob - Fix §6 table: use 'documented' for Bob-docs claims; correct marketplace description to 'skills marketplace'; fix convention attribution - Fix §1 cross-reference from §4 to §5 - Fix plan line citation from 979 to 969 for SKIPPED header - Update spec scope note to account for mandatory plugin.json bump
Collaborator
Author
|
verdict: approve Forge whole-branch review |
This was referenced Oct 29, 2026
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
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.
Add
docs/bob-install-requirements.md, a document that records what a Bob usermust supply to make adept skills work. Each claim is marked verified, documented, or
unverified with the environment used.
The document covers all seven items from the issue:
~/.bob) and project (.bob) scope, includingreferences/andscripts/check-public-safety.sh<plugin root>resolution for copy vs symlink install, per script, with supported path statednamefield, not directory name)$nameshorthand,agents/openai.yaml, plugin marketplace)Also includes the mandatory
plugin.jsonversion bump (7.12.2 → 7.12.3, ADR 0022).CI note
Ubuntu CI failed twice on
tests/fixtures/quest-log/claim-test.sh— the concurrent-mkdir CAStest fires a race on ubuntu azure runners. This change does not touch any claim code. Filed as
#480. macOS CI and all other ubuntu checks pass.
Closes #478