skills/fission-ai__openspec
1
0
Fork 0
mirror of https://github.com/Fission-AI/OpenSpec.git synced 2026-09-14 20:16:53 +08:00
Code Issues Packages Projects Releases Wiki Activity
Files
feat/grok-cli-support
fission-ai__openspec/openspec/changes/add-tool-command-surface-capabilities/tasks.md
T
shin 3f0ca3f6ce feat: add Trae command adapter (#1090) 
* feat(tools): add Trae command adapter

- Added Trae command adapter for generating `.trae/commands/opsx-<id>.md` files
- Complete unit tests (9 test cases) and integration tests
- Updated documentation and .gitignore
- Fixed YAML escaping for carriage returns (\r)

Co-Authored-By: Claude Code <noreply@anthropic.com>

* fix: handle empty string in YAML escaping

- Add explicit check for empty string in escapeYamlValue
- Return quoted empty string '""' instead of unquoted empty scalar
- Update test to verify empty string is properly quoted

Co-Authored-By: Claude Code <noreply@anthropic.com>

* fix: address PR review feedback for Trae adapter

- Update docs/commands.md Trae entry to reflect generated opsx-* commands
- Export traeAdapter from adapters/index.ts

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* docs: align Trae command adapter docs

---------

Co-authored-by: jjxyxsjr <jjxyxsjr@users.noreply.github.com>
Co-authored-by: Claude Code <noreply@anthropic.com>
Co-authored-by: Tabish Bidiwale <30385142+TabishB@users.noreply.github.com>
Co-authored-by: TabishB <tabishbidiwale@gmail.com>
2026-07-07 17:51:42 +00:00

4.1 KiB
Raw Permalink Blame History

0. Stacking Coordination

  • 0.1 Rebase this change on latest main before implementation
  • 0.2 If simplify-skill-installation is merged first, preserve its profile/delivery model and apply this change as a capability-aware refinement
  • 0.3 If this change merges first, ensure follow-up rebases do not reintroduce a blanket "commands = remove all skills" rule
  • 0.4 If add-global-install-scope is merged, verify combined scope × delivery × command-surface behavior remains deterministic

1. Tool Command-Surface Capability Model

  • 1.1 Extend tool metadata in src/core/config.ts with an optional command-surface capability field
  • 1.2 Define supported capability values: adapter, skills-invocable, none
  • 1.3 Mark known skills-invocable tools such as ForgeCode, Kimi CLI, and Mistral Vibe as skills-invocable
  • 1.4 Add a shared capability resolver (explicit metadata override first, inferred fallback from adapter presence second)
  • 1.5 Add focused unit tests for capability resolution (explicit override, inferred adapter, inferred none)

2. Init: Capability-Aware Delivery Planning

  • 2.1 Refactor init generation logic to compute per-tool effective actions (generate/remove skills and commands) instead of using only global booleans
  • 2.2 In delivery=commands, keep/generate skills for skills-invocable tools and do not remove those managed skill directories
  • 2.3 In delivery=commands, fail fast before writes when any selected tool resolves to none
  • 2.4 Update init output to clearly report effective behavior for skills-invocable tools (skills used as command surface)
  • 2.5 Ensure init no longer reports "no adapter" for tools intentionally using skills-invocable
  • 2.6 Add/adjust init tests for delivery=commands + kimi (skills retained/generated, no adapter error), mixed tools (claude,kimi) with per-tool expected outputs, and deterministic failure path for unsupported command surface (none)

3. Update: Capability-Aware Sync and Drift Detection

  • 3.1 Refactor update sync logic to apply delivery behavior per tool capability (not globally per run)
  • 3.2 In delivery=commands, keep/generate managed skills for skills-invocable tools
  • 3.3 In delivery=commands, fail before partial updates when configured tools include a none command surface
  • 3.4 Update profile/delivery drift detection to avoid perpetual drift for skills-invocable tools under commands delivery
  • 3.5 Ensure configured-tool detection still includes skills-invocable tools under commands delivery when managed skills exist
  • 3.6 Update summary output so skills-invocable behavior is reported as expected behavior (not implicit skip/error)
  • 3.7 Add/adjust update tests for delivery=commands + configured Kimi CLI (skills retained/generated), idempotent second update (no false drift loop), mixed configured tools (claude + kimi), and deterministic preflight failure for unsupported command surface (none)

4. UX and Error Messaging

  • 4.1 Add interactive init compatibility note for delivery=commands when selected tools include skills-invocable
  • 4.2 Add deterministic non-interactive error text with incompatible tool IDs and suggested alternatives (both or skills)
  • 4.3 Align init and update wording so capability-related behavior/messages are consistent

5. Documentation Updates

  • 5.1 Update docs/supported-tools.md to document command-surface semantics for skills-invocable tools and clarify delivery interactions
  • 5.2 Update docs/cli.md delivery guidance to explain capability-aware behavior for delivery=commands
  • 5.3 Add a short troubleshooting note for "commands-only + unsupported tool" failures

6. Verification

  • 6.1 Run targeted tests: test/core/init.test.ts and test/core/update.test.ts
  • 6.2 Run any new capability/unit test files added in this change
  • 6.3 Run full test suite (pnpm test) and resolve regressions
  • 6.4 Manual smoke check: openspec init --tools kimi with delivery=commands
  • 6.5 Manual smoke check: mixed tools (claude,kimi) with delivery=commands
Reference in New Issue View Git Blame Copy Permalink