fix: discover project skills from workspace#4031
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Needs human review This PR introduces new user-facing capability (workspace-scoped skill discovery) with a new RPC endpoint and UI integration, which warrants human review despite the well-structured implementation with tests. You can customize Macroscope's approvability policy. Learn more. |
What Changed
Why
The composer previously used the provider startup snapshot, whose skill list was resolved from the server process directory. Project-specific skills therefore did not appear for the active workspace and could come from the wrong repository.
UI Changes
No layout or styling changes. Existing skill autocomplete now includes skills from the selected workspace.
Checklist
Validation
Note
Discover project skills from workspace via
server.listProviderSkillsRPCserver.listProviderSkillsWebSocket RPC that acceptsinstanceIdandcwd, returning skills scoped to that workspace directory.listCodexProviderSkillsinCodexProvider.tsby spawning an app-server probe, callingskills/listwith the providedcwd, and parsing the response; falls back to snapshot skills on failure (10s timeout).probeCodexAppServerProvider, so the snapshot only contains non-repo skills.ChatComposerskill menu now queries workspace-scoped skills viauseProviderSkills, falling back to non-repo snapshot skills when the query is unavailable.Macroscope summarized 787bb12.