Skip to content

fix(provider): resolve skills per project workspace - #7

Closed
demget wants to merge 1 commit into
mainfrom
coda/fix-skill-resolution-in-coda
Closed

demget wants to merge 1 commit into
mainfrom
coda/fix-skill-resolution-in-coda

Conversation

@demget

@demget demget commented Aug 6, 2026

Copy link
Copy Markdown
Owner

What Changed

Resolve provider skills against the active project workspace or thread worktree instead of the server's current working directory. Add workspace-scoped skill discovery for Claude and Codex, with caching, RPC authorization, and provider-registry fallbacks for unsupported providers.

Update web and mobile composers and thread timelines to use the workspace-specific skills list. Add coverage for symlinked Claude skill directories and per-workspace provider resolution.

Why

Provider skill snapshots are shared per instance and discovered once from the server's working directory, so project-local .claude/skills and .agents/skills were missing from the skill picker and timeline for other projects. Resolving skills at the workspace boundary ensures each thread sees the skills available to the provider in the directory where it runs, while preserving snapshot fallbacks and degrading safely when discovery fails.

UI Changes

The skill picker and displayed skill chips now reflect skills available in the active project workspace or worktree. No visual layout changes.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

- Discover Claude and Codex skills against each thread's workspace
- Expose workspace-scoped skill lookup across web and mobile clients
@github-actions github-actions Bot added size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Aug 6, 2026
@demget

demget commented Sep 25, 2026

Copy link
Copy Markdown
Owner Author

Superseded by upstream pingdotgg#9180 (commit 15fea6c, "discover workspace skills everywhere"), which is now in main. Per-workspace skill resolution flows through workspaceSnapshots + resolveProviderSkillsForCwd (packages/client-runtime/src/providerSkills.ts), consumed by web and mobile, and fed by snapshotForCwd/refreshWorkspaceSnapshot in ProviderRegistry. PR7's providersListSkills RPC and useProviderSkills hooks no longer exist in main, so the change is redundant.

@demget demget closed this Sep 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant