Skip to content

feat(subagent): attach style and philosophy at worker spawn - #1156

Merged
TheGreatAxios merged 7 commits into
mainfrom
cl-8791-sub-agents-cannot-load-skills-available-to-the-primary
Sep 22, 2026
Merged

feat(subagent): attach style and philosophy at worker spawn#1156
TheGreatAxios merged 7 commits into
mainfrom
cl-8791-sub-agents-cannot-load-skills-available-to-the-primary

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Summary

  • Closed directors that listed both style and philosophy attach those skill bodies at spawn from plugin skill dirs; intern and Skywalker stay unattached, and brand directors are unchanged.
  • Every worker mounts skill_search and use_skill (including grok/kimi leaves), scoped to the union of attached and optional names, so bundled corbits-skills resolve the same way they do on the primary.
  • use_skill refuses attached and already-loaded skill names without returning the body again; the first load still returns the body.

Verification

  • bun run typecheck, bun run build, and bun run test pass
  • bun run check passes (lint, typecheck, dead-exports, build, 7840 tests)

Fixes CL-8791

@linear-code

linear-code Bot commented Sep 22, 2026

Copy link
Copy Markdown

CL-8791

@TheGreatAxios
TheGreatAxios force-pushed the cl-8791-sub-agents-cannot-load-skills-available-to-the-primary branch 2 times, most recently from cde9c4c to 1124964 Compare September 22, 2026 15:18
@TheGreatAxios
TheGreatAxios enabled auto-merge (squash) September 22, 2026 15:19
@TheGreatAxios
feat(subagent): attach style and philosophy at worker spawn
9297d31
@TheGreatAxios
fix(skills): split skill tool copy and plugin-only attach resolve
dde3dc6 
Shared skill_search/use_skill copy told the primary it had attached
skills. Primary stays an on-demand catalog; workers keep the
attach-aware descriptions.

Attached style/philosophy resolve from plugin skillDirs only so a
project-local SKILL.md cannot jailbreak the worker prompt.
@TheGreatAxios
feat(skills): refuse reloading attached and already-loaded skills
55fdcce
@TheGreatAxios
docs(skills): note use_skill refuses already-in-context bodies
7cf1628
@TheGreatAxios
fix(skills): claim use_skill names before resolve so parallel loads d…
34adbb3 
…o not double-dump
@TheGreatAxios
style(skills): oxfmt use_skill parallel-load test
a64a666
@TheGreatAxios
fix(intern): declare unset attachedSkills on the workspace package
f5495d3
@TheGreatAxios
TheGreatAxios force-pushed the cl-8791-sub-agents-cannot-load-skills-available-to-the-primary branch from 1124964 to f5495d3 Compare September 22, 2026 15:23
@TheGreatAxios
TheGreatAxios enabled auto-merge (squash) September 22, 2026 15:25
@TheGreatAxios
TheGreatAxios merged commit b095818 into main Sep 22, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@TheGreatAxios