feat: make skills platform-agnostic across coding agents#330
Merged
Conversation
…cklist sections
New skill compliance checklist sections:
- Cross-Platform Task Tracking: documents TaskCreate/TaskUpdate/TaskList
(Claude Code), update_plan (Codex), flags TodoWrite as legacy
- Script Path References: codifies relative paths over
${CLAUDE_PLUGIN_ROOT}, requires companion markdown links
- Rename skill to report-bug-ce to avoid name collisions with other plugins - Replace Claude Code-specific env detection with multi-platform guidance - Add Agent Platform/Agent Version fields to bug report template - List platform equivalents for user interaction tools - Fix nested code fence that broke markdown rendering
heal-skill overlaps with existing skill-creator and skill-development skills, uses non-standard XML tags, violates the tool selection checklist, and wraps behavior agents handle naturally.
- Replace ${CLAUDE_PLUGIN_ROOT} with relative script paths
- Replace TodoWrite with TaskCreate (Claude Code) / update_plan (Codex)
- Use fully-qualified agent name for pr-comment-resolver
- Add sequential fallback for platforms without parallel dispatch
- Use semantic wording for skill cross-references
- List platform equivalents for question tools (AskUserQuestion, request_user_input, ask_user) with plain-text fallback - Replace Claude Code-specific structured YAML question format with numbered lists that work on any platform - Keep efficient test -f stack detection script
- Remove all non-standard XML tags - List platform equivalents for user interaction tools - Make Chrome MCP warning platform-aware instead of Claude Code-specific - Generalize port detection to check AGENTS.md as primary instruction file - Remove duplicate agent-browser vs Chrome MCP warnings from CLI reference
- Replace mcp__xcodebuildmcp__* Claude Code syntax with semantic tool descriptions (e.g. "Call XcodeBuildMCP's list_simulators tool") - Add standard install instructions (Homebrew + npx) for XcodeBuildMCP - List platform equivalents for user interaction tools - Remove non-standard XML tags and emojis from output template - Fix incorrect agent reference in ce:review integration section - Fix skill name mismatch (/xcode-test -> /test-xcode) in examples
- Remove heal-skill entry (skill deleted) - Rename /report-bug to /report-bug-ce - Fix /xcode-test to /test-xcode to match actual skill name
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3c35ebec4d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
report-bug(avoid plugin name collisions), replace Claude Code-specific env detection with multi-platform guidance, fix nested code fence bug${CLAUDE_PLUGIN_ROOT}with relative paths, replaceTodoWritewithTaskCreate/update_plan, use fully-qualified agent names, add sequential fallbackmcp__xcodebuildmcp__*syntax with semantic tool descriptions, add standard install instructions (Homebrew + npx), remove XML tags and emojis, fix/xcode-test->/test-xcodename mismatch