Fix evolution reset and repair workspace roots#113
Draft
cursor[bot] wants to merge 2 commits into
Draft
Conversation
Co-authored-by: EXboy <EXboys@users.noreply.github.com>
Co-authored-by: EXboy <EXboys@users.noreply.github.com>
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
skilllite evolution reset --force: reset now scopes prompt/log/snapshot state to the selected workspacechat/root and removes_evolvedunder the effective workspace skills root.skilllite evolution repair-skillsso desktop/CLI repair validates the selected workspace skills root (skills/with.skillsfallback) instead of silently scanning legacy.skills.--workspaceoverride behavior, and repair's modern-vs-legacy skills-root selection.Bug and Impact
evolution reset --forcecould reset a different chat store throughpaths::chat_root()while leaving current workspaceskills/_evolvedintact, causing data loss in the wrong store and stale evolved skills in the intended workspace.evolution repair-skillscould validate.skillsand report success while broken skills under modernskills/_evolvedwere never checked.Root Cause
chat/and effective skills roots, but reset/repair retained legacy/global root resolution.Fix
--workspace/-wtoevolution resetandevolution repair-skillswith default..chat_root_for_workspaceandresolve_run_skills_root.--workspaceexplicitly.Task Linkage
TASK-2026-070tasks/TASK-2026-070-evolution-reset-repair-workspace/Injected Specs
spec/architecture-boundaries.mdspec/security-nonnegotiables.md(not applicable; no sandbox/security policy changes)spec/testing-policy.mdspec/docs-sync.mdValidation Evidence
rustup update stable && rustup default stable && rustc --version && cargo --versioncargo test -p skilllite --test cli_evolution_workspacecargo fmt --checkpython3 scripts/validate_tasks.pycargo clippy --all-targets -- -D warningscargo testcargo test -p skillliterustc 1.96.1,cargo 1.96.1after initial Cargo 1.83 edition-2024 manifest failure.4 passed; 0 failed.Task validation passed (70 task directories checked).cargo clippy --all-targets -- -D warnings: passed.cargo test: passed.cargo test -p skilllite: passed.Regression Scope
skilllite evolution resetCLI behavior and destructive workspace root selection.skilllite evolution repair-skillsCLI/desktop subprocess behavior.skills/vs legacy.skills.disable,explain).Docs Sync (EN/ZH)
docs/en/ASSISTANT-SPLIT-ARCHITECTURE.mddocs/zh/ASSISTANT-SPLIT-ARCHITECTURE.mdReview Checklist
tasks/TASK-2026-070-evolution-reset-repair-workspace/TASK.mdsatisfied.tasks/TASK-2026-070-evolution-reset-repair-workspace/STATUS.mdupdated with latest progress.tasks/TASK-2026-070-evolution-reset-repair-workspace/REVIEW.mdincludes merge readiness decision.tasks/board.mdstatus is up to date.