fix(cursor): discover skills for provider snapshots

[stephenjason89]

What changed

Cursor provider snapshots now discover global skills from the user-level directories supported by Cursor. The implementation reuses the existing best-effort skill frontmatter scanner instead of adding a new RPC or client state.

Discovery checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills. Later native roots win name collisions.

Why

The Cursor provider omitted skills when building its snapshot, so T3 Code always showed an empty skill picker even when Cursor loaded the same global skills from disk.

Addresses the global Cursor portion of #2736. Project-scoped Cursor discovery and OpenCode remain outside this PR.

Verification

• 34 focused provider and skill-discovery tests pass
• Server typecheck passes
• Formatting and targeted lint pass
• Local probe discovers all 13 skills currently installed under ~/.agents/skills

UI changes

The existing skill picker is populated for global Cursor skills. No client code or layout changed.

Note

Add skill discovery to Cursor provider snapshots via reusable discoverSkillsFromRoots

• Extracts a reusable discoverSkillsFromRoots routine from discoverClaudeSkills that scans ordered root directories, reads SKILL.md frontmatter, and returns a de-duplicated, name-sorted skill list with later-root precedence.
• Adds discoverCursorSkills covering ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills.
• Threads discovered skills through all branches of checkCursorProviderStatus so provider snapshots include user-scope skills regardless of CLI status.
• Behavioral Change: checkCursorProviderStatus now always attempts skill discovery from HOME/USERPROFILE (or OS homedir fallback) and includes skills in snapshot responses even when the CLI is disabled, missing, or times out.

^{Macroscope summarized 98214ad.}

---

Note

Low Risk
Best-effort filesystem reads of user skill dirs and snapshot fields only; no auth, execution, or client behavior changes.

Overview
Cursor provider snapshots now include user-level skills scanned from disk, so the skill picker is no longer empty when Cursor already loads those same global skills.

Discovery reuses the existing SKILL.md frontmatter scanner via a shared discoverSkillsFromRoots helper. Cursor checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills (later roots win on name collisions). Skills are attached on every status path, including disabled/CLI-missing, using HOME/USERPROFILE when provided.

Project-scoped Cursor skills are not included.

^{Reviewed by Cursor Bugbot for commit 98214ad. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: af9536f3-3c81-443f-8055-55cd00685eb9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the extracted discoverSkillsFromRoots helper, the new CursorSkills.ts module, and the checkCursorProviderStatus/CursorDriver call sites against the Effect service conventions.

Imports (effect/* subpath namespaces, named import of @t3tools/contracts), dependency acquisition (FileSystem/Path from the environment, cwd read from ServerConfig and passed as pure configuration), error handling, and test coverage for the new discovery behavior all look consistent with the conventions. One change-discipline issue: an explanatory invariant comment was dropped while the loop was moved into the shared helper.

^{Posted via Macroscope — Effect Service Conventions}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR adds production skill discovery for Cursor and makes those discovered skills appear in the existing user-facing slash-menu flow, including when the CLI is unavailable or disabled. The behavior is localized and best-effort, but it is a new capability with cross-state provider snapshot changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 886c7f8

[t3dotgg]

Note

🤖 GPT-5.6 Sol responding on behalf of Theo

We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together.

We selected #8336 for Cursor skill discovery. Its current diff scans the direct user and project roots and connects discovered skills to explicit invocation across providers. This branch adds global snapshot discovery by sharing the Claude scanner. Its smaller scanner and native-root precedence case remain useful reference.

If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed.

[stephenjason89]

What changed

Cursor provider snapshots now discover global skills from the user-level directories supported by Cursor. The implementation reuses the existing best-effort skill frontmatter scanner instead of adding a new RPC or client state.

Discovery checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills. Later native roots win name collisions.

Why

The Cursor provider omitted skills when building its snapshot, so T3 Code always showed an empty skill picker even when Cursor loaded the same global skills from disk.

Addresses the global Cursor portion of #2736. Project-scoped Cursor discovery and OpenCode remain outside this PR.

Verification

• 34 focused provider and skill-discovery tests pass
• Server typecheck passes
• Formatting and targeted lint pass
• Local probe discovers all 13 skills currently installed under ~/.agents/skills

UI changes

The existing skill picker is populated for global Cursor skills. No client code or layout changed.

Note

Add skill discovery to Cursor provider snapshots via reusable discoverSkillsFromRoots

• Extracts a reusable discoverSkillsFromRoots routine from discoverClaudeSkills that scans ordered root directories, reads SKILL.md frontmatter, and returns a de-duplicated, name-sorted skill list with later-root precedence.
• Adds discoverCursorSkills covering ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills.
• Threads discovered skills through all branches of checkCursorProviderStatus so provider snapshots include user-scope skills regardless of CLI status.
• Behavioral Change: checkCursorProviderStatus now always attempts skill discovery from HOME/USERPROFILE (or OS homedir fallback) and includes skills in snapshot responses even when the CLI is disabled, missing, or times out.

^{Macroscope summarized 98214ad.}

---

Note

Low Risk
Best-effort filesystem reads of user skill dirs and snapshot fields only; no auth, execution, or client behavior changes.

Overview
Cursor provider snapshots now include user-level skills scanned from disk, so the skill picker is no longer empty when Cursor already loads those same global skills.

Discovery reuses the existing SKILL.md frontmatter scanner via a shared discoverSkillsFromRoots helper. Cursor checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills (later roots win on name collisions). Skills are attached on every status path, including disabled/CLI-missing, using HOME/USERPROFILE when provided.

Project-scoped Cursor skills are not included.

^{Reviewed by Cursor Bugbot for commit 98214ad. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: af9536f3-3c81-443f-8055-55cd00685eb9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the extracted discoverSkillsFromRoots helper, the new CursorSkills.ts module, and the checkCursorProviderStatus/CursorDriver call sites against the Effect service conventions.

Imports (effect/* subpath namespaces, named import of @t3tools/contracts), dependency acquisition (FileSystem/Path from the environment, cwd read from ServerConfig and passed as pure configuration), error handling, and test coverage for the new discovery behavior all look consistent with the conventions. One change-discipline issue: an explanatory invariant comment was dropped while the loop was moved into the shared helper.

^{Posted via Macroscope — Effect Service Conventions}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR adds production skill discovery for Cursor and makes those discovered skills appear in the existing user-facing slash-menu flow, including when the CLI is unavailable or disabled. The behavior is localized and best-effort, but it is a new capability with cross-state provider snapshot changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 886c7f8

[t3dotgg]

Note

🤖 GPT-5.6 Sol responding on behalf of Theo

We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together.

We selected #8336 for Cursor skill discovery. Its current diff scans the direct user and project roots and connects discovered skills to explicit invocation across providers. This branch adds global snapshot discovery by sharing the Claude scanner. Its smaller scanner and native-root precedence case remain useful reference.

If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed.

[stephenjason89]

What changed

Cursor provider snapshots now discover global skills from the user-level directories supported by Cursor. The implementation reuses the existing best-effort skill frontmatter scanner instead of adding a new RPC or client state.

Discovery checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills. Later native roots win name collisions.

Why

The Cursor provider omitted skills when building its snapshot, so T3 Code always showed an empty skill picker even when Cursor loaded the same global skills from disk.

Addresses the global Cursor portion of #2736. Project-scoped Cursor discovery and OpenCode remain outside this PR.

Verification

• 34 focused provider and skill-discovery tests pass
• Server typecheck passes
• Formatting and targeted lint pass
• Local probe discovers all 13 skills currently installed under ~/.agents/skills

UI changes

The existing skill picker is populated for global Cursor skills. No client code or layout changed.

Note

Add skill discovery to Cursor provider snapshots via reusable discoverSkillsFromRoots

• Extracts a reusable discoverSkillsFromRoots routine from discoverClaudeSkills that scans ordered root directories, reads SKILL.md frontmatter, and returns a de-duplicated, name-sorted skill list with later-root precedence.
• Adds discoverCursorSkills covering ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills.
• Threads discovered skills through all branches of checkCursorProviderStatus so provider snapshots include user-scope skills regardless of CLI status.
• Behavioral Change: checkCursorProviderStatus now always attempts skill discovery from HOME/USERPROFILE (or OS homedir fallback) and includes skills in snapshot responses even when the CLI is disabled, missing, or times out.

^{Macroscope summarized 98214ad.}

---

Note

Low Risk
Best-effort filesystem reads of user skill dirs and snapshot fields only; no auth, execution, or client behavior changes.

Overview
Cursor provider snapshots now include user-level skills scanned from disk, so the skill picker is no longer empty when Cursor already loads those same global skills.

Discovery reuses the existing SKILL.md frontmatter scanner via a shared discoverSkillsFromRoots helper. Cursor checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills (later roots win on name collisions). Skills are attached on every status path, including disabled/CLI-missing, using HOME/USERPROFILE when provided.

Project-scoped Cursor skills are not included.

^{Reviewed by Cursor Bugbot for commit 98214ad. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: af9536f3-3c81-443f-8055-55cd00685eb9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the extracted discoverSkillsFromRoots helper, the new CursorSkills.ts module, and the checkCursorProviderStatus/CursorDriver call sites against the Effect service conventions.

Imports (effect/* subpath namespaces, named import of @t3tools/contracts), dependency acquisition (FileSystem/Path from the environment, cwd read from ServerConfig and passed as pure configuration), error handling, and test coverage for the new discovery behavior all look consistent with the conventions. One change-discipline issue: an explanatory invariant comment was dropped while the loop was moved into the shared helper.

^{Posted via Macroscope — Effect Service Conventions}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR adds production skill discovery for Cursor and makes those discovered skills appear in the existing user-facing slash-menu flow, including when the CLI is unavailable or disabled. The behavior is localized and best-effort, but it is a new capability with cross-state provider snapshot changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 886c7f8

[t3dotgg]

Note

🤖 GPT-5.6 Sol responding on behalf of Theo

We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together.

We selected #8336 for Cursor skill discovery. Its current diff scans the direct user and project roots and connects discovered skills to explicit invocation across providers. This branch adds global snapshot discovery by sharing the Claude scanner. Its smaller scanner and native-root precedence case remain useful reference.

If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed.

[stephenjason89]

What changed

Cursor provider snapshots now discover global skills from the user-level directories supported by Cursor. The implementation reuses the existing best-effort skill frontmatter scanner instead of adding a new RPC or client state.

Discovery checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills. Later native roots win name collisions.

Why

The Cursor provider omitted skills when building its snapshot, so T3 Code always showed an empty skill picker even when Cursor loaded the same global skills from disk.

Addresses the global Cursor portion of #2736. Project-scoped Cursor discovery and OpenCode remain outside this PR.

Verification

• 34 focused provider and skill-discovery tests pass
• Server typecheck passes
• Formatting and targeted lint pass
• Local probe discovers all 13 skills currently installed under ~/.agents/skills

UI changes

The existing skill picker is populated for global Cursor skills. No client code or layout changed.

Note

Add skill discovery to Cursor provider snapshots via reusable discoverSkillsFromRoots

• Extracts a reusable discoverSkillsFromRoots routine from discoverClaudeSkills that scans ordered root directories, reads SKILL.md frontmatter, and returns a de-duplicated, name-sorted skill list with later-root precedence.
• Adds discoverCursorSkills covering ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills.
• Threads discovered skills through all branches of checkCursorProviderStatus so provider snapshots include user-scope skills regardless of CLI status.
• Behavioral Change: checkCursorProviderStatus now always attempts skill discovery from HOME/USERPROFILE (or OS homedir fallback) and includes skills in snapshot responses even when the CLI is disabled, missing, or times out.

^{Macroscope summarized 98214ad.}

---

Note

Low Risk
Best-effort filesystem reads of user skill dirs and snapshot fields only; no auth, execution, or client behavior changes.

Overview
Cursor provider snapshots now include user-level skills scanned from disk, so the skill picker is no longer empty when Cursor already loads those same global skills.

Discovery reuses the existing SKILL.md frontmatter scanner via a shared discoverSkillsFromRoots helper. Cursor checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills (later roots win on name collisions). Skills are attached on every status path, including disabled/CLI-missing, using HOME/USERPROFILE when provided.

Project-scoped Cursor skills are not included.

^{Reviewed by Cursor Bugbot for commit 98214ad. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: af9536f3-3c81-443f-8055-55cd00685eb9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the extracted discoverSkillsFromRoots helper, the new CursorSkills.ts module, and the checkCursorProviderStatus/CursorDriver call sites against the Effect service conventions.

Imports (effect/* subpath namespaces, named import of @t3tools/contracts), dependency acquisition (FileSystem/Path from the environment, cwd read from ServerConfig and passed as pure configuration), error handling, and test coverage for the new discovery behavior all look consistent with the conventions. One change-discipline issue: an explanatory invariant comment was dropped while the loop was moved into the shared helper.

^{Posted via Macroscope — Effect Service Conventions}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR adds production skill discovery for Cursor and makes those discovered skills appear in the existing user-facing slash-menu flow, including when the CLI is unavailable or disabled. The behavior is localized and best-effort, but it is a new capability with cross-state provider snapshot changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 886c7f8

[t3dotgg]

Note

🤖 GPT-5.6 Sol responding on behalf of Theo

We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together.

We selected #8336 for Cursor skill discovery. Its current diff scans the direct user and project roots and connects discovered skills to explicit invocation across providers. This branch adds global snapshot discovery by sharing the Claude scanner. Its smaller scanner and native-root precedence case remain useful reference.

If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed.

[stephenjason89]

What changed

Cursor provider snapshots now discover global skills from the user-level directories supported by Cursor. The implementation reuses the existing best-effort skill frontmatter scanner instead of adding a new RPC or client state.

Discovery checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills. Later native roots win name collisions.

Why

The Cursor provider omitted skills when building its snapshot, so T3 Code always showed an empty skill picker even when Cursor loaded the same global skills from disk.

Addresses the global Cursor portion of #2736. Project-scoped Cursor discovery and OpenCode remain outside this PR.

Verification

• 34 focused provider and skill-discovery tests pass
• Server typecheck passes
• Formatting and targeted lint pass
• Local probe discovers all 13 skills currently installed under ~/.agents/skills

UI changes

The existing skill picker is populated for global Cursor skills. No client code or layout changed.

Note

Add skill discovery to Cursor provider snapshots via reusable discoverSkillsFromRoots

• Extracts a reusable discoverSkillsFromRoots routine from discoverClaudeSkills that scans ordered root directories, reads SKILL.md frontmatter, and returns a de-duplicated, name-sorted skill list with later-root precedence.
• Adds discoverCursorSkills covering ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills.
• Threads discovered skills through all branches of checkCursorProviderStatus so provider snapshots include user-scope skills regardless of CLI status.
• Behavioral Change: checkCursorProviderStatus now always attempts skill discovery from HOME/USERPROFILE (or OS homedir fallback) and includes skills in snapshot responses even when the CLI is disabled, missing, or times out.

^{Macroscope summarized 98214ad.}

---

Note

Low Risk
Best-effort filesystem reads of user skill dirs and snapshot fields only; no auth, execution, or client behavior changes.

Overview
Cursor provider snapshots now include user-level skills scanned from disk, so the skill picker is no longer empty when Cursor already loads those same global skills.

Discovery reuses the existing SKILL.md frontmatter scanner via a shared discoverSkillsFromRoots helper. Cursor checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills (later roots win on name collisions). Skills are attached on every status path, including disabled/CLI-missing, using HOME/USERPROFILE when provided.

Project-scoped Cursor skills are not included.

^{Reviewed by Cursor Bugbot for commit 98214ad. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: af9536f3-3c81-443f-8055-55cd00685eb9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the extracted discoverSkillsFromRoots helper, the new CursorSkills.ts module, and the checkCursorProviderStatus/CursorDriver call sites against the Effect service conventions.

Imports (effect/* subpath namespaces, named import of @t3tools/contracts), dependency acquisition (FileSystem/Path from the environment, cwd read from ServerConfig and passed as pure configuration), error handling, and test coverage for the new discovery behavior all look consistent with the conventions. One change-discipline issue: an explanatory invariant comment was dropped while the loop was moved into the shared helper.

^{Posted via Macroscope — Effect Service Conventions}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR adds production skill discovery for Cursor and makes those discovered skills appear in the existing user-facing slash-menu flow, including when the CLI is unavailable or disabled. The behavior is localized and best-effort, but it is a new capability with cross-state provider snapshot changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 886c7f8

[t3dotgg]

Note

🤖 GPT-5.6 Sol responding on behalf of Theo

We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together.

We selected #8336 for Cursor skill discovery. Its current diff scans the direct user and project roots and connects discovered skills to explicit invocation across providers. This branch adds global snapshot discovery by sharing the Claude scanner. Its smaller scanner and native-root precedence case remain useful reference.

If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed.

[stephenjason89]

What changed

Cursor provider snapshots now discover global skills from the user-level directories supported by Cursor. The implementation reuses the existing best-effort skill frontmatter scanner instead of adding a new RPC or client state.

Discovery checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills. Later native roots win name collisions.

Why

The Cursor provider omitted skills when building its snapshot, so T3 Code always showed an empty skill picker even when Cursor loaded the same global skills from disk.

Addresses the global Cursor portion of #2736. Project-scoped Cursor discovery and OpenCode remain outside this PR.

Verification

• 34 focused provider and skill-discovery tests pass
• Server typecheck passes
• Formatting and targeted lint pass
• Local probe discovers all 13 skills currently installed under ~/.agents/skills

UI changes

The existing skill picker is populated for global Cursor skills. No client code or layout changed.

Note

Add skill discovery to Cursor provider snapshots via reusable discoverSkillsFromRoots

• Extracts a reusable discoverSkillsFromRoots routine from discoverClaudeSkills that scans ordered root directories, reads SKILL.md frontmatter, and returns a de-duplicated, name-sorted skill list with later-root precedence.
• Adds discoverCursorSkills covering ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills.
• Threads discovered skills through all branches of checkCursorProviderStatus so provider snapshots include user-scope skills regardless of CLI status.
• Behavioral Change: checkCursorProviderStatus now always attempts skill discovery from HOME/USERPROFILE (or OS homedir fallback) and includes skills in snapshot responses even when the CLI is disabled, missing, or times out.

^{Macroscope summarized 98214ad.}

---

Note

Low Risk
Best-effort filesystem reads of user skill dirs and snapshot fields only; no auth, execution, or client behavior changes.

Overview
Cursor provider snapshots now include user-level skills scanned from disk, so the skill picker is no longer empty when Cursor already loads those same global skills.

Discovery reuses the existing SKILL.md frontmatter scanner via a shared discoverSkillsFromRoots helper. Cursor checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills (later roots win on name collisions). Skills are attached on every status path, including disabled/CLI-missing, using HOME/USERPROFILE when provided.

Project-scoped Cursor skills are not included.

^{Reviewed by Cursor Bugbot for commit 98214ad. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: af9536f3-3c81-443f-8055-55cd00685eb9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the extracted discoverSkillsFromRoots helper, the new CursorSkills.ts module, and the checkCursorProviderStatus/CursorDriver call sites against the Effect service conventions.

Imports (effect/* subpath namespaces, named import of @t3tools/contracts), dependency acquisition (FileSystem/Path from the environment, cwd read from ServerConfig and passed as pure configuration), error handling, and test coverage for the new discovery behavior all look consistent with the conventions. One change-discipline issue: an explanatory invariant comment was dropped while the loop was moved into the shared helper.

^{Posted via Macroscope — Effect Service Conventions}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR adds production skill discovery for Cursor and makes those discovered skills appear in the existing user-facing slash-menu flow, including when the CLI is unavailable or disabled. The behavior is localized and best-effort, but it is a new capability with cross-state provider snapshot changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 886c7f8

[t3dotgg]

Note

🤖 GPT-5.6 Sol responding on behalf of Theo

We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together.

We selected #8336 for Cursor skill discovery. Its current diff scans the direct user and project roots and connects discovered skills to explicit invocation across providers. This branch adds global snapshot discovery by sharing the Claude scanner. Its smaller scanner and native-root precedence case remain useful reference.

If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed.

[stephenjason89]

What changed

Cursor provider snapshots now discover global skills from the user-level directories supported by Cursor. The implementation reuses the existing best-effort skill frontmatter scanner instead of adding a new RPC or client state.

Discovery checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills. Later native roots win name collisions.

Why

The Cursor provider omitted skills when building its snapshot, so T3 Code always showed an empty skill picker even when Cursor loaded the same global skills from disk.

Addresses the global Cursor portion of #2736. Project-scoped Cursor discovery and OpenCode remain outside this PR.

Verification

• 34 focused provider and skill-discovery tests pass
• Server typecheck passes
• Formatting and targeted lint pass
• Local probe discovers all 13 skills currently installed under ~/.agents/skills

UI changes

The existing skill picker is populated for global Cursor skills. No client code or layout changed.

Note

Add skill discovery to Cursor provider snapshots via reusable discoverSkillsFromRoots

• Extracts a reusable discoverSkillsFromRoots routine from discoverClaudeSkills that scans ordered root directories, reads SKILL.md frontmatter, and returns a de-duplicated, name-sorted skill list with later-root precedence.
• Adds discoverCursorSkills covering ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills.
• Threads discovered skills through all branches of checkCursorProviderStatus so provider snapshots include user-scope skills regardless of CLI status.
• Behavioral Change: checkCursorProviderStatus now always attempts skill discovery from HOME/USERPROFILE (or OS homedir fallback) and includes skills in snapshot responses even when the CLI is disabled, missing, or times out.

^{Macroscope summarized 98214ad.}

---

Note

Low Risk
Best-effort filesystem reads of user skill dirs and snapshot fields only; no auth, execution, or client behavior changes.

Overview
Cursor provider snapshots now include user-level skills scanned from disk, so the skill picker is no longer empty when Cursor already loads those same global skills.

Discovery reuses the existing SKILL.md frontmatter scanner via a shared discoverSkillsFromRoots helper. Cursor checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills (later roots win on name collisions). Skills are attached on every status path, including disabled/CLI-missing, using HOME/USERPROFILE when provided.

Project-scoped Cursor skills are not included.

^{Reviewed by Cursor Bugbot for commit 98214ad. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: af9536f3-3c81-443f-8055-55cd00685eb9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the extracted discoverSkillsFromRoots helper, the new CursorSkills.ts module, and the checkCursorProviderStatus/CursorDriver call sites against the Effect service conventions.

Imports (effect/* subpath namespaces, named import of @t3tools/contracts), dependency acquisition (FileSystem/Path from the environment, cwd read from ServerConfig and passed as pure configuration), error handling, and test coverage for the new discovery behavior all look consistent with the conventions. One change-discipline issue: an explanatory invariant comment was dropped while the loop was moved into the shared helper.

^{Posted via Macroscope — Effect Service Conventions}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR adds production skill discovery for Cursor and makes those discovered skills appear in the existing user-facing slash-menu flow, including when the CLI is unavailable or disabled. The behavior is localized and best-effort, but it is a new capability with cross-state provider snapshot changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 886c7f8

[t3dotgg]

Note

🤖 GPT-5.6 Sol responding on behalf of Theo

We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together.

We selected #8336 for Cursor skill discovery. Its current diff scans the direct user and project roots and connects discovered skills to explicit invocation across providers. This branch adds global snapshot discovery by sharing the Claude scanner. Its smaller scanner and native-root precedence case remain useful reference.

If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed.

[stephenjason89]

What changed

Cursor provider snapshots now discover global skills from the user-level directories supported by Cursor. The implementation reuses the existing best-effort skill frontmatter scanner instead of adding a new RPC or client state.

Discovery checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills. Later native roots win name collisions.

Why

The Cursor provider omitted skills when building its snapshot, so T3 Code always showed an empty skill picker even when Cursor loaded the same global skills from disk.

Addresses the global Cursor portion of #2736. Project-scoped Cursor discovery and OpenCode remain outside this PR.

Verification

• 34 focused provider and skill-discovery tests pass
• Server typecheck passes
• Formatting and targeted lint pass
• Local probe discovers all 13 skills currently installed under ~/.agents/skills

UI changes

The existing skill picker is populated for global Cursor skills. No client code or layout changed.

Note

Add skill discovery to Cursor provider snapshots via reusable discoverSkillsFromRoots

• Extracts a reusable discoverSkillsFromRoots routine from discoverClaudeSkills that scans ordered root directories, reads SKILL.md frontmatter, and returns a de-duplicated, name-sorted skill list with later-root precedence.
• Adds discoverCursorSkills covering ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills.
• Threads discovered skills through all branches of checkCursorProviderStatus so provider snapshots include user-scope skills regardless of CLI status.
• Behavioral Change: checkCursorProviderStatus now always attempts skill discovery from HOME/USERPROFILE (or OS homedir fallback) and includes skills in snapshot responses even when the CLI is disabled, missing, or times out.

^{Macroscope summarized 98214ad.}

---

Note

Low Risk
Best-effort filesystem reads of user skill dirs and snapshot fields only; no auth, execution, or client behavior changes.

Overview
Cursor provider snapshots now include user-level skills scanned from disk, so the skill picker is no longer empty when Cursor already loads those same global skills.

Discovery reuses the existing SKILL.md frontmatter scanner via a shared discoverSkillsFromRoots helper. Cursor checks ~/.claude/skills, ~/.codex/skills, ~/.agents/skills, and ~/.cursor/skills (later roots win on name collisions). Skills are attached on every status path, including disabled/CLI-missing, using HOME/USERPROFILE when provided.

Project-scoped Cursor skills are not included.

^{Reviewed by Cursor Bugbot for commit 98214ad. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: af9536f3-3c81-443f-8055-55cd00685eb9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Reviewed the extracted discoverSkillsFromRoots helper, the new CursorSkills.ts module, and the checkCursorProviderStatus/CursorDriver call sites against the Effect service conventions.

Imports (effect/* subpath namespaces, named import of @t3tools/contracts), dependency acquisition (FileSystem/Path from the environment, cwd read from ServerConfig and passed as pure configuration), error handling, and test coverage for the new discovery behavior all look consistent with the conventions. One change-discipline issue: an explanatory invariant comment was dropped while the loop was moved into the shared helper.

^{Posted via Macroscope — Effect Service Conventions}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR adds production skill discovery for Cursor and makes those discovered skills appear in the existing user-facing slash-menu flow, including when the CLI is unavailable or disabled. The behavior is localized and best-effort, but it is a new capability with cross-state provider snapshot changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 886c7f8

[t3dotgg]

Note

🤖 GPT-5.6 Sol responding on behalf of Theo

We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together.

We selected #8336 for Cursor skill discovery. Its current diff scans the direct user and project roots and connects discovered skills to explicit invocation across providers. This branch adds global snapshot discovery by sharing the Claude scanner. Its smaller scanner and native-root precedence case remain useful reference.

If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed.