Skip to content

feat(skills): path parity, fail-closed workers, plugin director overlay - #1322

Closed
TheGreatAxios wants to merge 4 commits into
mainfrom
cl-9917-skill-loading-path-parity-director-scoped-attach
Closed

TheGreatAxios wants to merge 4 commits into
mainfrom
cl-9917-skill-loading-path-parity-director-scoped-attach

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Skill discovery includes project and user .corbits/skills plus .agents / .claude / .codex (plugin catalog still first).
  • Workers without attachedSkills / optionalSkills cannot load the catalog. Coder may load first-party typescript; designer ships no skills (fail-closed; a plugin/local overlay may declare its own).
  • A trusted plugin (or local) profile may replace a shipped director id on spawn_agent(agent=…). intent= still uses the closed package.
  • use_skill appends the skill directory + sibling-file hint; spawn_agent accepts skills[] unioned with the package allowlist.
  • Dispatch prompt still has no full skill dump; slash playbooks stay the parent surface.

Verification

  • bun test src/extensions/skills.test.ts src/agent/use-skill.test.ts src/subagent/agent-fleet.test.ts src/subagent/run-skill-scope.test.ts src/agent/directors/identity.test.ts src/plugins/agent-plugins.test.ts src/agent/profiles.test.ts src/agent/directors/registry.test.ts src/prompts.test.ts — 182 pass, 0 fail
  • bunx oxfmt --check clean on touched files; tsc --noEmit clean via pre-commit

Fixes CL-9917

Discover project and user .corbits/.agents/.claude/.codex skills.
Workers without attached or optional skills cannot load the catalog.
A trusted plugin profile may replace a shipped director id on spawn.
@linear-code

linear-code Bot commented Oct 5, 2026

Copy link
Copy Markdown

CL-9917

Designer stays fail-closed. Plugin overlays may still declare their own
optionalSkills.
use_skill returns skill dir so workers can read brand-guidelines.md next to SKILL.md; user-global skill dirs become read-only trusted roots; spawn_agent accepts skills[] unioned with package allowlist.
Bare-name resolution now contains symlinks to the winning base dir.
Docs: .corbits clause, homedir paths, skill-dir footer, spawn skills[].
@TheGreatAxios

Copy link
Copy Markdown
Collaborator Author

Closed per operator triage decision — Linear issue being canceled; content superseded or deprioritized.

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.

1 participant