Skip to content

Enable OpenCode skill discovery in composer flow#18

Merged
Jay1 merged 1 commit into
mainfrom
jcode/surface-skills-first-class
May 28, 2026
Merged

Enable OpenCode skill discovery in composer flow#18
Jay1 merged 1 commit into
mainfrom
jcode/surface-skills-first-class

Conversation

@Jay1
Copy link
Copy Markdown
Owner

@Jay1 Jay1 commented May 28, 2026

Summary

  • Enables OpenCode composer capabilities to advertise supportsSkillDiscovery: true and supportsSkillMentions: true so the web composer can request skill discovery for OpenCode.
  • Adds listSkills implementation to apps/server/src/provider/Layers/OpenCodeAdapter.ts with defensive normalization for OpenCode runtime skill metadata (consoleState.skills) from array/object/string forms and optional legacy/legacy-disabled fields.
  • Normalizes skill descriptors with stable name, fallback opencode://skill/<name> path when missing, description, interface fields (displayName, shortDescription), enabled/scope, and robust handling of missing/invalid metadata.
  • Adds focused OpenCode adapter tests in apps/server/src/provider/Layers/OpenCodeAdapter.test.ts covering:
    • capability advertisement
    • array-form skills metadata
    • keyed/object-form metadata including short-form string aliases
    • missing metadata fallback to empty list
  • Adds planning/design documentation for implementation in docs/superpowers/plans/2026-05-28-opencode-skill-discovery.md and docs/superpowers/specs/2026-05-28-opencode-skill-discovery.md.
  • Branches: base main, head jcode/surface-skills-first-class.

Testing

  • Not run (no test commands executed in this step).
  • Suggested checks:
    • bun run --cwd apps/server test src/provider/Layers/OpenCodeAdapter.test.ts
    • Focused web composer checks (if needed): validate OpenCode skill menu trigger/selection behavior after capability flags are enabled.
    • LSP diagnostics on touched TypeScript files
    • Any relevant security scan for changed files (per repo process).

- add OpenCode `listSkills` support using runtime inventory metadata
- normalize skill descriptors from array/object/string metadata with safe fallback handling
- report `supportsSkillDiscovery`/`supportsSkillMentions` as true for OpenCode
- add focused adapter tests for capabilities and skills edge cases
- add implementation plan and design docs for OpenCode skill discovery
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 28, 2026

Warning

Review limit reached

@Jay1, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 15 minutes and 59 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7b76bef3-c030-4fdc-8d45-9c8005d8c313

📥 Commits

Reviewing files that changed from the base of the PR and between fe226ff and 508785d.

📒 Files selected for processing (4)
  • apps/server/src/provider/Layers/OpenCodeAdapter.test.ts
  • apps/server/src/provider/Layers/OpenCodeAdapter.ts
  • docs/superpowers/plans/2026-05-28-opencode-skill-discovery.md
  • docs/superpowers/specs/2026-05-28-opencode-skill-discovery-design.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jcode/surface-skills-first-class

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels May 28, 2026
@Jay1 Jay1 merged commit 274d0ec into main May 28, 2026
8 checks passed
@Jay1 Jay1 deleted the jcode/surface-skills-first-class branch May 28, 2026 15:46
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