-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add Windsurf IDE support with slash commands #113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Windsurf IDE support with slash commands #113
Conversation
…command-support-for-windsurf Merge remote-tracking branch 'origin/main' into codex/research-slash-command-support-for-windsurf
TabishB
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey thanks for giving this a try! It's a great effort for someone from a non-dev background!
There's a few things that need to be tidied up, but I can handle this. Namely this needs the following extra stuff:
- Adding/updating unit tests
- Updates to documentation
But great effort! I'll get this merged in soon and you get to be a contributor and part of this projects history!
| import { SlashCommandId } from '../../templates/index.js'; | ||
|
|
||
| const FILE_PATHS: Record<SlashCommandId, string> = { | ||
| proposal: '.windsurf/commands/openspec-proposal.md', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this is .windsurf/workflows/ not .windsurf/commands/ according to the docs: https://docs.windsurf.com/windsurf/cascade/workflows
let me know if I got something wrong here.
| archive: '.windsurf/commands/openspec-archive.md' | ||
| }; | ||
|
|
||
| const FRONTMATTER: Record<SlashCommandId, string> = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe we don't need a frontmatter for Windsurf. Seems like a plain markdown works, but could be wrong. I'll test this out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Turns out you do need a formatter, but just description and auto_execution_mode
…ows and simplify templates\n\n- Write workflows to .windsurf/workflows instead of .windsurf/commands\n- Remove YAML frontmatter; add concise intro before managed markers\n- Add init/update tests for Windsurf and marker preservation\n- List Windsurf in README native tools table\n- Normalize registry indentation
…n\n- Drop intro hook and headings for Windsurf workflows\n- Keep OPENSPEC markers-only body for safe updates\n- Adjust tests to assert marker-managed content
…de description and auto_execution_mode: 3 for proposal/apply/archive\n- Keep content minimal; body remains marker-managed
* docs(windsurf): propose workflow support * restore missing opencode spec * Add Windsurf IDE support with slash commands * feat(windsurf): add Windsurf workflows support under .windsurf/workflows and simplify templates\n\n- Write workflows to .windsurf/workflows instead of .windsurf/commands\n- Remove YAML frontmatter; add concise intro before managed markers\n- Add init/update tests for Windsurf and marker preservation\n- List Windsurf in README native tools table\n- Normalize registry indentation * chore(windsurf): remove optional intro content to simplify workflows\n\n- Drop intro hook and headings for Windsurf workflows\n- Keep OPENSPEC markers-only body for safe updates\n- Adjust tests to assert marker-managed content * feat(windsurf): add required YAML frontmatter to workflows\n\n- Include description and auto_execution_mode: 3 for proposal/apply/archive\n- Keep content minimal; body remains marker-managed --------- Co-authored-by: Tabish Bidiwale <[email protected]> Co-authored-by: Tabish Bidiwale <[email protected]>
* docs(windsurf): propose workflow support * restore missing opencode spec * Add Windsurf IDE support with slash commands * feat(windsurf): add Windsurf workflows support under .windsurf/workflows and simplify templates\n\n- Write workflows to .windsurf/workflows instead of .windsurf/commands\n- Remove YAML frontmatter; add concise intro before managed markers\n- Add init/update tests for Windsurf and marker preservation\n- List Windsurf in README native tools table\n- Normalize registry indentation * chore(windsurf): remove optional intro content to simplify workflows\n\n- Drop intro hook and headings for Windsurf workflows\n- Keep OPENSPEC markers-only body for safe updates\n- Adjust tests to assert marker-managed content * feat(windsurf): add required YAML frontmatter to workflows\n\n- Include description and auto_execution_mode: 3 for proposal/apply/archive\n- Keep content minimal; body remains marker-managed --------- Co-authored-by: Tabish Bidiwale <[email protected]> Co-authored-by: Tabish Bidiwale <[email protected]>
Add Windsurf IDE support with slash commands
Testing:
openspec init.windsurf/commands/Did not run a full application test on v7.0.0 due to time constraints, but the implementation follows the exact same pattern as the existing Kilocode integration that was added in v7.0.0.