Add provider-neutral shared skill roots - #1271
Merged
Merged
Conversation
SawyerHood
changed the title
This was referenced Aug 10, 2026
SawyerHood
force-pushed
the
shared-skill-roots
branch
from
August 10, 2026 19:07
6840d4e to
b221bf4
Compare
SawyerHood
marked this pull request as ready for review
August 10, 2026 19:09
SawyerHood
force-pushed
the
shared-skill-roots
branch
from
August 10, 2026 19:17
b221bf4 to
5d16589
Compare
SawyerHood
force-pushed
the
shared-skill-roots
branch
from
August 10, 2026 19:34
5d16589 to
3bc19a1
Compare
SawyerHood
added a commit
that referenced
this pull request
Aug 10, 2026
Stack layer 1 of 3. ## Summary - Add validated user and project nativeSkillRoots to custom ACP agent configuration. - Resolve each root on the target host for the composer. - Keep the ACP launch specification and host protocol synchronized. - Document the configuration and add boundary, daemon, and route tests. ## Stack - This pull request targets main. - Next: #1271 ## Validation - Configuration, host contract, host discovery, and public route tests pass. - Affected package typechecks pass. Closes #1176
SawyerHood
added a commit
that referenced
this pull request
Aug 10, 2026
Stack layer 3 of 3. ## Summary - Discover acp-cursor project skills from .cursor/skills. - Follow a .cursor/skills root link to a canonical root such as .agents/skills. - Add Cursor scopes to the Skills view, SDK, CLI, and delete contract. - Mark skills reached through a linked root as read-only. ## Stack - Depends on #1271. - This is the top layer. ## Validation - The exact .agents/skills plus .cursor/skills link reproduction passes. - Host discovery, server listing, contract, and Skills view tests pass. - Affected package typechecks pass. Closes #1251
ymichael
added a commit
that referenced
this pull request
Aug 11, 2026
## Summary - preserve GitHub check timestamps through the host-daemon contract - collapse check runs by name using the newest timestamp before calculating the PR rollup - bump the host-daemon protocol for the wire-contract change - cover non-chronological GitHub rollup ordering with a regression test ## Testing - pnpm exec turbo run test --filter=@bb/host-workspace --filter=@bb/host-daemon-contract --filter=@bb/server --force - pnpm exec turbo run typecheck --filter=@bb/domain --filter=@bb/host-workspace --filter=@bb/host-daemon-contract --filter=@bb/host-daemon --filter=@bb/server - live sanity check against PR #1271: 19 raw runs collapse to 9 passing checks with 0 failures - git diff --check Closes #1256
amadad
pushed a commit
to amadad/bb
that referenced
this pull request
Aug 14, 2026
Stack layer 1 of 3. ## Summary - Add validated user and project nativeSkillRoots to custom ACP agent configuration. - Resolve each root on the target host for the composer. - Keep the ACP launch specification and host protocol synchronized. - Document the configuration and add boundary, daemon, and route tests. ## Stack - This pull request targets main. - Next: get-bb#1271 ## Validation - Configuration, host contract, host discovery, and public route tests pass. - Affected package typechecks pass. Closes get-bb#1176
amadad
pushed a commit
to amadad/bb
that referenced
this pull request
Aug 14, 2026
Stack layer 2 of 3. ## Summary - Add top-level sharedSkillRoots for one physical user or project skill collection. - Index shared roots as read-only sources in the composer, Skills view, SDK, and bb CLI. - Stage shared skills for Codex, Claude, Pi, and ACP runtimes. - Apply deterministic precedence: bb project, shared project, bb user, shared user, plugins, then built-ins. - Support target-host paths without copying the canonical source. ## Stack - Depends on get-bb#1270. - Next: get-bb#1272 ## Validation - Shared-root configuration, discovery, precedence, remote transport, runtime staging, CLI, and UI tests pass. - Affected package typechecks pass. Closes get-bb#1188
amadad
pushed a commit
to amadad/bb
that referenced
this pull request
Aug 14, 2026
Stack layer 3 of 3. ## Summary - Discover acp-cursor project skills from .cursor/skills. - Follow a .cursor/skills root link to a canonical root such as .agents/skills. - Add Cursor scopes to the Skills view, SDK, CLI, and delete contract. - Mark skills reached through a linked root as read-only. ## Stack - Depends on get-bb#1271. - This is the top layer. ## Validation - The exact .agents/skills plus .cursor/skills link reproduction passes. - Host discovery, server listing, contract, and Skills view tests pass. - Affected package typechecks pass. Closes get-bb#1251
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
1 participant
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.
Stack layer 2 of 3.
Summary
Stack
Validation
Closes #1188