feat(tools): add Zed Agent support - #1659
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (3)
📝 WalkthroughWalkthroughAdded Zed Agent as a skills-only tool using the shared ChangesZed Agent support
CLI tool documentation
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to This PR adds built-in Zed Agent skills support while preserving existing Codex behavior and documents the shared-root setup. Targeted tests, build, and lint pass, so no actionable merge-blocking risk remains beyond normal checks. Sequence Diagram(s)sequenceDiagram
participant User
participant InitCLI as openspec init
participant ValidateTools as validateTools
participant SharedSkills as .agents/skills
User->>InitCLI: select zed, Codex, or agents
InitCLI->>ValidateTools: reconcile shared target
ValidateTools->>SharedSkills: write skills and target marker
SharedSkills-->>User: provide Zed-compatible skills
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Deploying openspec-docs with
|
| Latest commit: |
0fea1c0
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://35788436.openspec-docs.pages.dev |
| Branch Preview URL: | https://codex-add-zed-support.openspec-docs.pages.dev |
|
Scope-check clarification: the Windsurf alias text is unchanged from |
Resolves the docs conflict with Zed Agent support (Fission-AI#1659), which landed on main while this PR was open. Both tools add a row to the same five tool tables, so each side dropped the other; the resolution keeps both in every list, in alphabetical order. Also normalizes this branch's `dsh`(shorthand for DeepSeek Harness) to a plain `dsh` so it matches how every other tool id is written, and drops a trailing space it introduced in docs/cli.md. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Risk: low. One new entry in the supported-tools table. Purely additive — no existing tool's behavior changes.
What was wrong
openspec init --tools zedwas rejected, even though Zed's built-in Agent reads project skills from.agents/skills, which OpenSpec already generates.What changes
Adds
zedas a skills-only target that reuses the existing.agents/skillsgeneration and ownership marker. Detection uses the standard.zedproject directory.Why it's safe
No new generation path — it opts into the one that already exists. Adding Zed to a project that already uses Codex preserves Codex's dual-syntax tree; the shared-root ownership marker is unchanged.
Proof
Before:
Invalid tool(s): zed. After: init succeeds andopenspec updaterediscovers it. 412 focused shared-root, update and CLI tests pass. CI green on all three platforms. Verified on this release stack thatinit --tools zedwrites 6 skills under.agents/skills.Closes #202