Add test-first-loop template and Forge execution stability improvements - #5
Merged
Conversation
…te `test-first-loop`
…, completion criteria, and AI review gate
…ta loader interfaces
…Material icons in Forge UI
…ta loader interfaces
… tests for phase rendering
… for true multi-phase initialization
…ecution core regressions
…ion and validate matrix
…ection gate to Forge UI
…unning status single-source and deterministic
… loop against no-task-left session creation
…l Forge docs and run validations
- Update Forge.templates.test.tsx: the project-selection gate now disables the "Open templates" button when no workspace is active, so the test verifies the button is disabled rather than trying to click it. - Fix Rust phase check commands: replace `node -e "process.exit(N)"` with `node -e process.exitCode=N` to avoid shell parentheses interpretation on Unix and double-quote mangling on Windows cmd.exe. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.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
This PR introduces two feature sets across 13 commits:
1.
test-first-loopbundled templateA new multi-phase Forge template that enforces a test-first development workflow with an AI review gate.
template.json, schemas, hook scripts, and promptsforgeLoadPhaseViewinsrc/services/tauri.ts) to read templatephases.jsonand planstate.jsongetForgePhaseIconUrl2. Forge execution stability & project gate
Stability fixes to prevent ghost sessions and ensure deterministic UI state.
runningInfo.taskIdwins, then phase state, then plan state. Stalein_progressrows are demoted when another task is actively runningknownTaskIdsfrom the selected plan; stops execution cleanly when all tasks are exhausted instead of creating an extra sessionDocumentation & tests
docs/forge.mdwith phase metadata, icon resolution, execution guards, and project gate behaviortest-first-loophooksKey files
src-tauri/resources/forge/templates/test-first-loop/src/services/tauri.ts,src/utils/forgePhaseIcons.tssrc/features/forge/components/Forge.tsxsrc/features/forge/hooks/useForgeExecution.tssrc-tauri/src/shared/forge_execute_core.rssrc/features/app/components/Sidebar.test.tsxdocs/forge.md,docs/codebase-map.mdTest plan
test-first-looptemplate appears in bundled template listfile.svgvitestandcargo testsuites — no regressions🤖 Generated with Claude Code