feat(commands): finish #59 remainder — unify /rewind, remove dead ide#71
Merged
Conversation
/rewind with arguments now forwards to the shadow-git revert paths (list, diff, last, <n>, <uuid>) so file rollback stays discoverable through the one visible rewind command. Bare /rewind keeps the interactive conversation-rewind overlay. Completes the rewind bullet of #59 that #68 only hid. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Issue #59 asked to verify whether the ide integration actually works for coven-code. It cannot: the command suggests installing a coven-code.coven-code editor extension that is not published anywhere (VS Code marketplace, Open VSX, OpenCoven org), and the ~/.coven-code/ide/*.lock files it reads have no writer without that extension. Real IDE support remains the ACP surface (coven-code acp). Removes the named command, its slash adapter, and the now-orphaned core::ide detection module. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
This PR completes the remaining work from issue #59 by (1) making /rewind the single discoverable entry point for both conversation rewind and shadow-git file rollback, and (2) removing the non-functional ide command surface plus the now-orphaned core IDE-detection module.
Changes:
- Route
/rewind <args>to the existing shadow-git rollback implementation (former/undo+/revertpaths), while keeping bare/rewindas the interactive rewind overlay. - Remove the
idenamed command, its slash adapter, andclaurst-core’sidemodule exports; update command lists/docs accordingly. - Add/adjust command tests to cover the new
/rewindrouting behavior.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src-rust/crates/tui/src/app.rs | Removes /ide from autocomplete and updates /rewind description to reflect file rollback routing. |
| src-rust/crates/core/src/lib.rs | Drops core::ide module + re-exports (detect_ide, IdeKind). |
| src-rust/crates/core/src/ide.rs | Deletes IDE environment detection implementation (now unused). |
| src-rust/crates/commands/src/named_commands.rs | Removes IdeCommand and updates the named-command registry/tests accordingly. |
| src-rust/crates/commands/src/lib.rs | Implements /rewind arg routing to RevertCommand, removes /ide adapter, updates help text and tests. |
| docs/superpowers/plans/2026-06-10-issue-59-remainder.md | Adds an implementation plan document capturing the steps/evidence for the changes. |
| docs/commands.md | Updates /rewind docs and removes /ide from adapters and named-command listings. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+5189
to
+5195
| /rewind list — list assistant turns with recorded file changes\n\ | ||
| /rewind diff [n] — preview a turn's diff without reverting\n\ | ||
| /rewind last — revert the most recent assistant turn\n\ | ||
| /rewind <n> — revert the n-th most recent assistant turn\n\ | ||
| /rewind <uuid> — revert the turn whose message id starts with <uuid>\n\n\ | ||
| The legacy /undo and /revert commands remain hidden one-release\n\ | ||
| compatibility aliases for the argument forms." |
Comment on lines
+134
to
+136
| /rewind last — revert the most recent assistant turn | ||
| /rewind <n> — revert the n-th most recent assistant turn | ||
| /rewind <uuid> — revert the turn whose message id starts with <uuid> |
This was referenced Jun 11, 2026
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
Follow-up to #68, which closed #59 with two bullets incomplete:
1. Actually unify
/undo//revert//rewind(#68 only hid the first two).After #68, file rollback was reachable only through hidden commands — once the one-release compatibility aliases are removed, the capability would have disappeared from the surface entirely. Now
/rewindis the single visible entry point:/rewind— interactive conversation-rewind overlay (unchanged)/rewind list//rewind diff [n]//rewind last//rewind <n>//rewind <uuid>— forward to the shadow-git revert paths absorbed from/undoand/revert/undoand/revertstay hidden one-release aliases exactly as #68 left them.2. Verify the
idecommand's integration; remove if dead (issue bullet #68 skipped).Verified dead:
IdeCommandsuggestscode --install-extension coven-code.coven-code— no such extension is published on the VS Code Marketplace or Open VSX, and none of the 45 OpenCoven org repos is an editor extension.~/.coven-code/ide/*.lockfiles it reads are written only by that nonexistent extension, so the connection-status surface can never show anything.coven-code acp), which is untouched.Removed: the
idenamed command + slash adapter, the now-orphanedclaurst_core::idedetection module (detect_ide/IdeKindhad no other consumers), autocomplete/help/category entries, and docs rows.Verification
cargo fmt -p claurst-commands -p claurst-tui --checkcargo test -p claurst-commands(78 passed, incl. 2 new rewind-routing tests)cargo test -p claurst-core(472 passed)cargo test -p claurst-tuicargo check -p claurstgrep -rn "detect_ide|IdeKind|IdeCommand|extension_install_command" crates→ no matchesRefs #59
🤖 Generated with Claude Code