Skip to content

Add Zed integration#2780

Open
arrrrny wants to merge 5 commits into
github:mainfrom
arrrrny:feat/2779-zed-integration
Open

Add Zed integration#2780
arrrrny wants to merge 5 commits into
github:mainfrom
arrrrny:feat/2779-zed-integration

Conversation

@arrrrny
Copy link
Copy Markdown

@arrrrny arrrrny commented May 31, 2026

Summary

  • add a built-in zed integration that installs Spec Kit commands as Zed skills under .agents/skills
  • render Zed skill invocations correctly in init next-steps output and hook messages
  • add integration coverage for Zed plus registry/list assertions and docs updates

Testing

  • uv run --extra test python3 -m pytest tests/integrations/test_registry.py tests/integrations/test_integration_subcommand.py::TestIntegrationList::test_list_shows_available_integrations tests/integrations/test_integration_zed.py -q

Closes #2779

@arrrrny arrrrny requested a review from mnriem as a code owner May 31, 2026 08:39
Copilot AI review requested due to automatic review settings May 31, 2026 08:39
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds Zed editor as a new skills-based integration that installs Spec Kit commands as .agents/skills/speckit-<name>/SKILL.md files invokable via Zed's slash-command menu.

Changes:

  • New ZedIntegration class registered in the integrations registry
  • Hook invocation rendering and init-flow handling for zed skill mode
  • Documentation, registry, and integration-test updates including a new test_integration_zed.py

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/specify_cli/integrations/zed/init.py New ZedIntegration (skills-based, .agents/skills, AGENTS.md)
src/specify_cli/integrations/init.py Imports and registers ZedIntegration
src/specify_cli/extensions.py Adds zed_skill_mode branch rendering /skill-name invocations
src/specify_cli/commands/init.py Adds Zed to skill-mode flags, post-init step, and display command logic
docs/reference/integrations.md Adds Zed row to integrations table
docs/index.md Bumps integration count from 30 to 31 and lists Zed
tests/integrations/test_registry.py Adds zed to expected-keys list
tests/integrations/test_integration_subcommand.py Asserts zed appears in list output
tests/integrations/test_integration_zed.py New tests for Zed integration and hook invocation rendering

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/specify_cli/integrations/zed/__init__.py Outdated
Copy link
Copy Markdown
Collaborator

@mnriem mnriem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address Copilot feedback

@arrrrny
Copy link
Copy Markdown
Author

arrrrny commented Jun 1, 2026

Side note, I have been using it last couple days and it works as expected

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot's findings

  • Files reviewed: 9/9 changed files
  • Comments generated: 1

Comment thread docs/index.md
@mnriem
Copy link
Copy Markdown
Collaborator

mnriem commented Jun 2, 2026

Please address Copilot feedback and the test & lint errors

Copilot AI review requested due to automatic review settings June 2, 2026 23:10
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Comment thread src/specify_cli/commands/init.py Outdated
Comment thread src/specify_cli/commands/init.py
Comment thread src/specify_cli/commands/init.py
Comment thread src/specify_cli/extensions.py
Comment thread src/specify_cli/extensions.py Outdated
Comment thread src/specify_cli/integrations/zed/__init__.py Outdated
- Remove non-actionable --skills flag from ZedIntegration (Zed is always
  skills-based, like Agy)
- Align zed_skill_mode predicate with ai_skills for consistency across
  init output and hook rendering
- Consolidate claude/cursor/zed slash-skill return blocks in
  _render_hook_invocation to reduce duplication
- Override test_options_include_skills_flag for Zed (no --skills flag)
@arrrrny arrrrny requested a review from mnriem June 2, 2026 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Agent]: Add support for Zed

3 participants