feat(skills): live refresh from the filesystem

[k11kirky]

Problem

Skills loaded in the UI do not reflect external changes made on disk (e.g. edits from a terminal, agent sessions, or another editor) until the page is manually refreshed.

Changes

A watchSkills method was added to SkillsService that watches the writable skill roots (~/.claude/skills and each repo's .claude/skills directory) using WatcherService. File system events are debounced (300 ms) and collapsed into a single { changed: true } event to avoid flooding the UI with updates.

A skills.watch tRPC subscription endpoint was added to the skills router, exposing this stream to the frontend.

A useSkillsWatcher hook was introduced that subscribes to skills.watch and calls queryClient.invalidateQueries on all skills queries whenever a change event arrives. This hook is mounted inside SkillsView so the skills list and file contents stay in sync with disk automatically.

The hardcoded staleTime: 30_000 overrides were removed from useSkills, useSkillContents, and useSkillFile since live invalidation via the watcher makes manual staleness tuning unnecessary.

WatcherService is now bound in skillsModule so that hosts loading only the skills module (e.g. Electron main) have the dependency available without requiring a separate binding.

How did you test this?

• Added a watchSkillDirs integration test that creates a skill directory on disk after the watcher attaches and asserts that a debounced change event is emitted within 10 seconds.
• Added a unit test confirming watchSkillDirs finishes immediately when given an empty directory list.

Automatic notifications

• Publish to changelog?
• Alert Sales and Marketing teams?

[k11kirky]

• refactor(skills): apply simplify-pass cleanups across the stack #2612
• feat(skills): Codex unification — bring your skills, use them anywhere #2611
• feat(skills): publish skills to the team and install team skills locally #2610
• feat(skills): PostHog cloud team skills — read path #2609
• feat(skills): skills.sh marketplace browse and install #2608
• feat(skills): validation and shadowing signals #2607
• feat(skills): live refresh from the filesystem #2606 👈 (View in Graphite)
• feat(skills): create, edit, and delete user and repo skills #2605
• feat(skills): render full skill directories #2604
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

React Doctor could not complete this scan.

No React dependency found in /tmp/react-doctor-baseline-qF8HM6/package.json. Add "react" to dependencies (or peerDependencies) and re-run.

_{Reviewed by React Doctor for commit 5b8c1af.}

[greptile-apps]

Prompt To Fix All With AI

Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
packages/workspace-server/src/services/skills/schemas.ts:82-84
**Exported schema is never used**

`skillsChangedEvent` is defined and exported here but not referenced anywhere else — it doesn't appear in `skills.router.ts` as `.output(skillsChangedEvent)` and no other file imports it. Every other procedure in the router uses an explicit output schema; the `watch` subscription currently carries no runtime validation. Either wire it up with `.output(skillsChangedEvent)` on the procedure, or remove the schema.

_{Reviews (1): Last reviewed commit: "feat(skills): live refresh from the file..." | Re-trigger Greptile}

[k11kirky]

Merge activity

• Jun 12, 11:31 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Jun 12, 11:32 AM UTC: Graphite couldn't merge this pull request because a downstack PR feat(skills): create, edit, and delete user and repo skills #2605 failed to merge.
• Jun 12, 11:54 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Jun 12, 11:56 AM UTC: Graphite rebased this pull request as part of a merge.
• Jun 12, 12:05 PM UTC: @k11kirky merged this pull request with Graphite.