@ai-content-space/loopx 0.2.2 → 0.2.3

package/README.md

@@ -15,7 +15,7 @@
 Recommended v1 flow:
 
 ```text
-clarify -> spec? -> plan -> subagent-exec | exec -> review -> fix-review? -> finish
+clarify -> spec? -> plan -> (subagent-exec | exec) -> final-review -> fix-review? -> finish
 ```
 
 `spec` is conditional. Use it when API, data, state, permission, migration, compatibility, product behavior, or architecture decisions must be fixed before planning. Skip it when the remaining work is local implementation choice.
@@ -32,10 +32,13 @@ Core workflow skills:
 - `subagent-exec`: execute an approved plan with fresh subagents and staged review.
 - `exec`: execute an approved plan inline when subagents are unavailable or not desired.
 - `review`: request independent code review against a git range and plan or requirements.
+- `final-review`: review the completed feature for runtime, integration, and test-gap risk before finishing.
 - `fix-review`: evaluate and implement code review feedback rigorously.
 - `finish`: verify and choose merge, PR, keep, or discard.
 - `refactor-plan`: interview and write a behavior-preserving refactor plan with tiny commits.
 
+`review` and its matching `fix-review` run inside `subagent-exec` or `exec` as task/checkpoint review loops. `final-review` is the top-level whole-feature review before `finish`, and its feedback also goes through `fix-review`.
+
 Support skills:
 
 - `tdd`
@@ -56,6 +59,8 @@ For the v1 skill-suite workflow, human-maintained artifacts live under `docs/loo
 
 `finish` may generate spec candidates in `docs/loopx/specs/` when completed work produces stable team rules. These candidates are repo-tracked and must remain visible in the git diff.
 
+`finish` is the terminal completion step for one implementation decision. Rerun it only after keep-as-is, PR iteration, interruption before executing a choice, or new changes after review feedback. Do not rerun it after merge or discard.
+
 Generated support state, hook diagnostics, installer metadata, HTML views, manifests, and runtime JSON remain under `.loopx/`.
 
 Local agent memory lives under `.loopx/memory/`:

package/README.zh-CN.md

@@ -15,7 +15,7 @@
 推荐 v1 流程：
 
 ```text
-clarify -> spec? -> plan -> subagent-exec | exec -> review -> fix-review? -> finish
+clarify -> spec? -> plan -> (subagent-exec | exec) -> final-review -> fix-review? -> finish
 ```
 
 `spec` 是条件设计门。涉及 API、数据、状态、权限、迁移、兼容、产品行为或架构决策时使用；只剩局部实现选择时可以跳过，直接进入 `plan`。
@@ -32,10 +32,13 @@ clarify -> spec? -> plan -> subagent-exec | exec -> review -> fix-review? -> fin
 - `subagent-exec`：用 fresh subagents 和 staged review 执行已批准计划。
 - `exec`：没有 subagent 或用户选择 inline 时顺序执行计划。
 - `review`：基于 git range 和计划/需求发起独立代码评审。
+- `final-review`：在收尾前对完整 feature 做运行时、集成和测试缺口风险评审。
 - `fix-review`：严谨评估并处理 code review feedback。
 - `finish`：验证完成后选择 merge、PR、保留或丢弃。
 - `refactor-plan`：访谈并写行为保持的 tiny-commit 重构计划。
 
+`review` 和对应的 `fix-review` 在 `subagent-exec` 或 `exec` 内部作为 task/checkpoint review loop 运行。`final-review` 是 `finish` 前的顶层 whole-feature review，它的反馈也通过 `fix-review` 处理。
+
 辅助 skills：
 
 - `tdd`
@@ -56,6 +59,8 @@ v1 skill-suite 工作流的人工维护长期产物放在 `docs/loopx/`：
 
 当完成的工作产生稳定团队规则时，`finish` 可以在 `docs/loopx/specs/` 生成 spec candidates。这些候选是 repo-tracked，必须保留在 git diff 中供审阅。
 
+`finish` 是一次 implementation decision 的终端完成步骤。只有在上次选择保留、PR 迭代、执行选择前中断，或 review feedback 后出现新变更时才重新执行；merge 或 discard 后不要重复执行。
+
 生成的支撑状态、hook 诊断、安装元数据、HTML views、manifests 和 runtime JSON 仍放在 `.loopx/` 下。
 
 本地 agent memory 放在 `.loopx/memory/`：

package/docs/loopx/design/loopx-skill-suite-v1-design.md

@@ -14,6 +14,7 @@ The v1 product surface is the installed and governed bundled skill suite:
 - `subagent-exec`
 - `exec`
 - `review`
+- `final-review`
 - `fix-review`
 - `finish`
 - `refactor-plan`
@@ -32,16 +33,16 @@ The repository may retain auxiliary or compatibility skill source directories fo
 Recommended flow:
 
 ```text
-clarify -> spec? -> plan -> subagent-exec | exec -> review -> fix-review? -> finish
+clarify -> spec? -> plan -> (subagent-exec | exec) -> final-review -> fix-review? -> finish
 ```
 
 `spec` is a conditional design gate. `clarify` may route directly to `plan` when the remaining questions are local implementation choices. It routes to `spec` when product behavior, APIs, state, data, permissions, migration, compatibility, or architecture decisions need to be fixed before implementation planning.
 
 `plan` is the superpowers `writing-plans` workflow under the loopx name. It writes executable plans and offers two execution options: `subagent-exec` recommended, or `exec` inline.
 
-`review` is the superpowers `requesting-code-review` workflow under the loopx name. `fix-review` is the superpowers `receiving-code-review` workflow under the loopx name.
+`review` is the task or checkpoint code review workflow inside `subagent-exec` or `exec`. `final-review` is the top-level whole-feature runtime, integration, and test-gap review before completion. `fix-review` handles feedback from either review layer.
 
-`finish` verifies completion, extracts local memory, proposes repo-tracked spec candidates when stable team rules emerged, then presents merge, PR, keep, or discard options.
+`finish` verifies completion, extracts local memory, proposes repo-tracked spec candidates when stable team rules emerged, then presents merge, PR, keep, or discard options. It is the terminal completion step for one implementation decision; rerun it only after keep-as-is, PR iteration, interruption before choice execution, or new changes after review feedback. Do not rerun it after merge or discard.
 
 ## Artifacts
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@ai-content-space/loopx",
   "type": "module",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "Skill-first workflow suite for agentic coding assistants",
   "repository": {
     "type": "git",

package/plugins/loopx/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "loopx",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "Skill-first workflow suite for agentic coding assistants",
   "skills": "./skills/",
   "interface": {

package/plugins/loopx/skills/clarify/SKILL.md

@@ -3,7 +3,7 @@ name: clarify
 description: "Grills ambiguous loopx work until material questions are answered, then routes to spec or plan using a design gate. Not for clear implementation tasks, approved specs, or code changes."
 when_to_use: "clarify, requirements, ambiguous request, unclear scope, non-goals, decision boundaries, acceptance criteria, 需求澄清, 范围不清"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # loopx Clarify

package/plugins/loopx/skills/debug/SKILL.md

@@ -3,7 +3,7 @@ name: debug
 description: "Finds root cause for bugs, failing tests, build failures, regressions, and unexpected behavior before fixes. Not for new feature planning or routine code review."
 when_to_use: "debug, bug, test failure, build failure, regression, unexpected behavior, root cause, 报错, 失败, 回归, 排查"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Systematic Debugging

package/plugins/loopx/skills/exec/SKILL.md

@@ -3,7 +3,7 @@ name: exec
 description: "Executes a written loopx implementation plan sequentially with review checkpoints. Not for unclear plans, missing requirements, or subagent-first execution."
 when_to_use: "written implementation plan, inline execution, sequential plan execution, review checkpoints, no subagent lane"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Exec
@@ -35,9 +35,12 @@ For each task:
 ### Step 3: Complete Development
 
 After all tasks complete and verified:
+- Announce: "I'm using the final-review skill to review the completed feature."
+- **REQUIRED SUB-SKILL:** Use loopx:final-review
+- If final-review finds Critical or Important issues, use loopx:fix-review to handle feedback before proceeding
 - Announce: "I'm using the finish skill to complete this work."
 - **REQUIRED SUB-SKILL:** Use loopx:finish
-- Follow that skill to verify tests, present options, execute choice
+- Follow finish to verify tests, present options, execute choice
 
 ## When to Stop and Ask for Help
 
@@ -68,4 +71,6 @@ After all tasks complete and verified:
 
 **Required workflow skills:**
 - **loopx:plan** - Creates the plan this skill executes
+- **loopx:final-review** - Final whole-feature runtime and integration risk review
+- **loopx:fix-review** - Handles final-review feedback before finish
 - **loopx:finish** - Complete development after all tasks

package/plugins/loopx/skills/final-review/SKILL.md

@@ -3,7 +3,7 @@ name: final-review
 description: "Performs whole-feature review after implementation and staged task review. Not for per-task review, unresolved scope, implementation, or pure documentation polish."
 when_to_use: "final-review, final code review, whole feature review, integration review, pre-finish review, after subagent-exec, runtime risk review, 最终评审"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Final Review

package/plugins/loopx/skills/finish/SKILL.md

@@ -3,7 +3,7 @@ name: finish
 description: "Finishes completed loopx development work after tests pass by presenting merge, PR, keep, or discard options. Not for unfinished work or failing verification."
 when_to_use: "implementation complete, tests pass, finish branch, create pull request, merge locally, keep branch, discard work"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Finish

package/plugins/loopx/skills/fix-review/SKILL.md

@@ -3,7 +3,7 @@ name: fix-review
 description: "Handles received code review feedback with verification, technical evaluation, pushback, and one-item-at-a-time fixes. Not for requesting a new review or implementing unrelated changes."
 when_to_use: "fix-review, received code review feedback, review comments, reviewer suggestions, requested changes, 处理评审意见"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Fix Review

package/plugins/loopx/skills/go-style/SKILL.md

@@ -3,7 +3,7 @@ name: go-style
 description: "Applies loopx Go coding style for .go edits, tests, errors, context, naming, and interface boundaries. Not for non-Go code or Kratos-specific architecture by itself."
 when_to_use: "go-style, Go, golang, .go files, go tests, gofmt, idiomatic Go, Go style, Go 代码"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Go Style
@@ -59,7 +59,13 @@ Choose the one already used in the codebase.
 
 - Exported symbols should have Go doc comments that start with the symbol name.
 - Short local comments are acceptable when they explain why, not what.
-- Prefer complete sentences for package, exported type, exported function, and non-obvious behavior comments.
+- Prefer complete sentences for package, exported type, exported function, exported method, and non-obvious behavior comments.
+- Match new comments to the user's requested language. If the user asks in Chinese or explicitly requests Chinese comments, write new comments in Chinese while preserving Go doc naming conventions such as `// UserService ...` and `// CreateUser ...`.
+- Do not translate existing comments unless the user explicitly asks for translation; preserve the surrounding file's established comment language when no user preference is stated.
+- Remove comments that only restate syntax, names, or immediately obvious control flow.
+- Add comments for non-obvious business rules, ordering constraints, compatibility behavior, concurrency assumptions, performance tradeoffs, and external API quirks.
+- Check nearby existing comments when behavior changes; stale comments are worse than missing comments.
+- Prefer clearer names, smaller functions, or stronger types over comments that explain avoidable confusion.
 
 ## Verification
 

package/plugins/loopx/skills/kratos/SKILL.md

@@ -3,7 +3,7 @@ name: kratos
 description: "Supports Go-Kratos microservices, proto/buf APIs, service/biz/data layers, middleware, auth, config, and troubleshooting. Not for generic Go style alone."
 when_to_use: "kratos, Go-Kratos, proto, buf, service layer, biz layer, data layer, middleware, auth, config, Kratos 微服务"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Kratos

package/plugins/loopx/skills/plan/SKILL.md

@@ -3,7 +3,7 @@ name: plan
 description: "Creates bite-sized implementation plans from approved requirements, clarify output, or design specs with exact files, tests, commands, expected output, and execution handoff. Not for unresolved requirements, design decisions, PRD generation, or code changes."
 when_to_use: "plan, implementation plan, execution plan, task breakdown, approved requirements, approved design spec, docs/loopx/design, 实施计划, 执行计划, 任务拆分"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 argument-hint: "<design spec path or feature name>"
 ---
 
@@ -156,13 +156,13 @@ Plan complete and saved to `docs/loopx/plans/<filename>.md`.
 
 Two execution options:
 
-1. Subagent-Driven (recommended) - dispatch a fresh subagent per task, review between tasks, fast iteration
+1. Subagent Exec (recommended) - dispatch a fresh subagent per task, review between tasks, fast iteration
 2. Inline Execution - execute tasks in this session using exec, batch execution with checkpoints
 
 Which approach?
 ```
 
-If Subagent-Driven is chosen:
+If Subagent Exec is chosen:
 
 - REQUIRED SUB-SKILL: Use `loopx:subagent-exec`
 - Fresh subagent per task plus two-stage review

package/plugins/loopx/skills/refactor-plan/SKILL.md

@@ -3,7 +3,7 @@ name: refactor-plan
 description: "Creates a behavior-preserving refactor plan with user interview, repo evidence, tiny commits, scope boundaries, and testing decisions. Not for feature changes or immediate implementation."
 when_to_use: "refactor-plan, refactor request, refactoring RFC, tiny commits, behavior-preserving cleanup, architecture cleanup, 重构计划"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 This skill will be invoked when the user wants to create a refactor request. You should go through the steps below. You may skip steps if you don't consider them necessary.

package/plugins/loopx/skills/review/SKILL.md

@@ -3,7 +3,7 @@ name: review
 description: "Dispatches a loopx code reviewer subagent against a concrete git range and requirements. Not for implementation, planning, or unresolved review scope."
 when_to_use: "request code review, completed task review, major feature review, pre-merge review, subagent code quality check"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Review

package/plugins/loopx/skills/spec/SKILL.md

@@ -3,7 +3,7 @@ name: spec
 description: "Writes software design specs from already-clarified requirements, including solution approach, architecture outline, detailed design, tradeoffs, verification design, and handoff context. Not for unresolved requirements, PRD generation, implementation task planning, or code changes."
 when_to_use: "spec, design spec, technical design, design proposal, detailed design, architecture design, 设计方案, 概要设计, 详细设计, 技术方案"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # loopx Spec

package/plugins/loopx/skills/subagent-exec/SKILL.md

@@ -3,7 +3,7 @@ name: subagent-exec
 description: "Executes approved loopx implementation plans with fresh subagents per independent task and staged review. Not for planning, unclear requirements, or tightly coupled edits."
 when_to_use: "approved implementation plan, independent tasks, subagent execution, staged spec review, code quality review, parallel-capable execution"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Subagent Exec

package/plugins/loopx/skills/subagent-exec/agents/openai.yaml

@@ -1,3 +1,3 @@
 interface:
-  display_name: "Subagent-Driven Development"
-  short_description: "Execute implementation plans with staged subagent reviews"
+  display_name: "Subagent Exec"
+  short_description: "Execute loopx plans with staged subagent reviews"

package/plugins/loopx/skills/subagent-exec/codex-subagents.md

@@ -17,7 +17,7 @@ Use this reference before executing this skill in Codex.
 
 Subagent dispatch requires Codex multi-agent support. If `spawn_agent`,
 `wait_agent`, or `close_agent` are unavailable, do not pretend this skill ran
-as subagent-driven development. Use `loopx:exec` instead.
+as subagent-exec. Use `loopx:exec` instead.
 
 Codex environments that require explicit feature flags should enable:
 

package/plugins/loopx/skills/tdd/SKILL.md

@@ -3,7 +3,7 @@ name: tdd
 description: "Guides feature and bugfix implementation through a failing test before production code and red-green-refactor discipline. Not for generated files or throwaway prototypes."
 when_to_use: "tdd, failing test first, feature implementation, bugfix, regression test, red green refactor, 测试先行"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Test-Driven Development (TDD)

package/plugins/loopx/skills/verify/SKILL.md

@@ -3,7 +3,7 @@ name: verify
 description: "Requires fresh verification evidence before claiming work is complete, fixed, passing, review-ready, or ready to commit. Not for speculative confidence or stale results."
 when_to_use: "verify, completion claim, fixed claim, tests pass, review-ready, commit, fresh evidence, 验证, 完成前检查"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Verification Before Completion

package/scripts/codex-workflow-hook.mjs

@@ -31,7 +31,6 @@ function nextSkill(state) {
   if (!state || !state.slug) {
     return null;
   }
-  const reviewBuildCommand = `$build --from-review .loopx/workflows/${state.slug}/review-report.md`;
   if (isClarifyReadyForPlan(state)) {
     return `$plan ${state.slug}`;
   }
@@ -51,7 +50,7 @@ function nextSkill(state) {
     && state.current_stage === 'plan'
     && Array.isArray(state.plan_blockers)
     && state.plan_blockers.length === 0) {
-    return `$build .loopx/plans/requirements-snapshot-${state.slug}.md`;
+    return `$subagent-exec .loopx/plans/requirements-snapshot-${state.slug}.md`;
   }
   if (state.current_stage === 'build'
     && state.stage_status === 'awaiting-approval'
@@ -71,13 +70,13 @@ function nextSkill(state) {
       || state.approval?.build === 'requested'
       || state.approval?.build === 'approved'
     )) {
-    return reviewBuildCommand;
+    return null;
   }
   if (state.current_stage === 'review'
     && state.review_verdict === 'request-changes'
     && state.requested_transition === 'review->build'
     && state.approval?.build === 'approved') {
-    return reviewBuildCommand;
+    return null;
   }
   if (state.current_stage === 'review'
     && state.review_verdict === 'request-changes'
@@ -94,6 +93,36 @@ function nextSkill(state) {
   return null;
 }
 
+function nextCli(state) {
+  if (!state || !state.slug) {
+    return null;
+  }
+  if (state.stage_status === 'awaiting-approval'
+    && state.current_stage === 'plan'
+    && Array.isArray(state.plan_blockers)
+    && state.plan_blockers.length === 0) {
+    return `loopx build .loopx/plans/requirements-snapshot-${state.slug}.md`;
+  }
+  if (state.current_stage === 'review'
+    && state.review_verdict === 'request-changes'
+    && state.rollback_target === 'build'
+    && (
+      state.pending_user_decision === 'review->build'
+      || state.requested_transition === 'review->build'
+      || state.approval?.build === 'requested'
+      || state.approval?.build === 'approved'
+    )) {
+    return `loopx build --from-review .loopx/workflows/${state.slug}/review-report.md`;
+  }
+  if (state.current_stage === 'review'
+    && state.review_verdict === 'request-changes'
+    && state.requested_transition === 'review->build'
+    && state.approval?.build === 'approved') {
+    return `loopx build --from-review .loopx/workflows/${state.slug}/review-report.md`;
+  }
+  return null;
+}
+
 function blockers(state) {
   const values = [
     ...(Array.isArray(state.plan_blockers) ? state.plan_blockers : []),
@@ -134,7 +163,7 @@ function nextActionLine(state, workflow) {
   if (isClarifyReadyForPlan(state) && state.approval?.plan !== 'approved') {
     return `approve clarify -> plan, then $plan ${state.slug || workflow}`;
   }
-  return nextSkill(state) || state.recommended_next_action || 'none';
+  return nextSkill(state) || nextCli(state) || state.recommended_next_action || 'none';
 }
 
 function implementationGateLines(state) {
@@ -228,6 +257,8 @@ try {
     `loopx workflow: ${state.slug || workflow}`,
     `stage: ${stageText(state)}`,
     `next: ${nextActionLine(state, workflow)}`,
+    `next skill: ${nextSkill(state) || '(none)'}`,
+    `next cli: ${nextCli(state) || '(none)'}`,
     `blockers: ${blockers(state)}`,
     ...implementationGateLines(state),
     `approval: ${JSON.stringify(state.approval || {})}`,

package/skills/clarify/SKILL.md

@@ -3,7 +3,7 @@ name: clarify
 description: "Grills ambiguous loopx work until material questions are answered, then routes to spec or plan using a design gate. Not for clear implementation tasks, approved specs, or code changes."
 when_to_use: "clarify, requirements, ambiguous request, unclear scope, non-goals, decision boundaries, acceptance criteria, 需求澄清, 范围不清"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # loopx Clarify

package/skills/debug/SKILL.md

@@ -3,7 +3,7 @@ name: debug
 description: "Finds root cause for bugs, failing tests, build failures, regressions, and unexpected behavior before fixes. Not for new feature planning or routine code review."
 when_to_use: "debug, bug, test failure, build failure, regression, unexpected behavior, root cause, 报错, 失败, 回归, 排查"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Systematic Debugging

package/skills/exec/SKILL.md

@@ -3,7 +3,7 @@ name: exec
 description: "Executes a written loopx implementation plan sequentially with review checkpoints. Not for unclear plans, missing requirements, or subagent-first execution."
 when_to_use: "written implementation plan, inline execution, sequential plan execution, review checkpoints, no subagent lane"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Exec
@@ -35,9 +35,12 @@ For each task:
 ### Step 3: Complete Development
 
 After all tasks complete and verified:
+- Announce: "I'm using the final-review skill to review the completed feature."
+- **REQUIRED SUB-SKILL:** Use loopx:final-review
+- If final-review finds Critical or Important issues, use loopx:fix-review to handle feedback before proceeding
 - Announce: "I'm using the finish skill to complete this work."
 - **REQUIRED SUB-SKILL:** Use loopx:finish
-- Follow that skill to verify tests, present options, execute choice
+- Follow finish to verify tests, present options, execute choice
 
 ## When to Stop and Ask for Help
 
@@ -68,4 +71,6 @@ After all tasks complete and verified:
 
 **Required workflow skills:**
 - **loopx:plan** - Creates the plan this skill executes
+- **loopx:final-review** - Final whole-feature runtime and integration risk review
+- **loopx:fix-review** - Handles final-review feedback before finish
 - **loopx:finish** - Complete development after all tasks

package/skills/final-review/SKILL.md

@@ -3,7 +3,7 @@ name: final-review
 description: "Performs whole-feature review after implementation and staged task review. Not for per-task review, unresolved scope, implementation, or pure documentation polish."
 when_to_use: "final-review, final code review, whole feature review, integration review, pre-finish review, after subagent-exec, runtime risk review, 最终评审"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Final Review

package/skills/finish/SKILL.md

@@ -3,7 +3,7 @@ name: finish
 description: "Finishes completed loopx development work after tests pass by presenting merge, PR, keep, or discard options. Not for unfinished work or failing verification."
 when_to_use: "implementation complete, tests pass, finish branch, create pull request, merge locally, keep branch, discard work"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Finish

package/skills/fix-review/SKILL.md

@@ -3,7 +3,7 @@ name: fix-review
 description: "Handles received code review feedback with verification, technical evaluation, pushback, and one-item-at-a-time fixes. Not for requesting a new review or implementing unrelated changes."
 when_to_use: "fix-review, received code review feedback, review comments, reviewer suggestions, requested changes, 处理评审意见"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Fix Review

package/skills/go-style/SKILL.md

@@ -3,7 +3,7 @@ name: go-style
 description: "Applies loopx Go coding style for .go edits, tests, errors, context, naming, and interface boundaries. Not for non-Go code or Kratos-specific architecture by itself."
 when_to_use: "go-style, Go, golang, .go files, go tests, gofmt, idiomatic Go, Go style, Go 代码"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Go Style
@@ -59,7 +59,13 @@ Choose the one already used in the codebase.
 
 - Exported symbols should have Go doc comments that start with the symbol name.
 - Short local comments are acceptable when they explain why, not what.
-- Prefer complete sentences for package, exported type, exported function, and non-obvious behavior comments.
+- Prefer complete sentences for package, exported type, exported function, exported method, and non-obvious behavior comments.
+- Match new comments to the user's requested language. If the user asks in Chinese or explicitly requests Chinese comments, write new comments in Chinese while preserving Go doc naming conventions such as `// UserService ...` and `// CreateUser ...`.
+- Do not translate existing comments unless the user explicitly asks for translation; preserve the surrounding file's established comment language when no user preference is stated.
+- Remove comments that only restate syntax, names, or immediately obvious control flow.
+- Add comments for non-obvious business rules, ordering constraints, compatibility behavior, concurrency assumptions, performance tradeoffs, and external API quirks.
+- Check nearby existing comments when behavior changes; stale comments are worse than missing comments.
+- Prefer clearer names, smaller functions, or stronger types over comments that explain avoidable confusion.
 
 ## Verification
 

package/skills/kratos/SKILL.md

@@ -3,7 +3,7 @@ name: kratos
 description: "Supports Go-Kratos microservices, proto/buf APIs, service/biz/data layers, middleware, auth, config, and troubleshooting. Not for generic Go style alone."
 when_to_use: "kratos, Go-Kratos, proto, buf, service layer, biz layer, data layer, middleware, auth, config, Kratos 微服务"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Kratos

package/skills/plan/SKILL.md

@@ -3,7 +3,7 @@ name: plan
 description: "Creates bite-sized implementation plans from approved requirements, clarify output, or design specs with exact files, tests, commands, expected output, and execution handoff. Not for unresolved requirements, design decisions, PRD generation, or code changes."
 when_to_use: "plan, implementation plan, execution plan, task breakdown, approved requirements, approved design spec, docs/loopx/design, 实施计划, 执行计划, 任务拆分"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 argument-hint: "<design spec path or feature name>"
 ---
 
@@ -156,13 +156,13 @@ Plan complete and saved to `docs/loopx/plans/<filename>.md`.
 
 Two execution options:
 
-1. Subagent-Driven (recommended) - dispatch a fresh subagent per task, review between tasks, fast iteration
+1. Subagent Exec (recommended) - dispatch a fresh subagent per task, review between tasks, fast iteration
 2. Inline Execution - execute tasks in this session using exec, batch execution with checkpoints
 
 Which approach?
 ```
 
-If Subagent-Driven is chosen:
+If Subagent Exec is chosen:
 
 - REQUIRED SUB-SKILL: Use `loopx:subagent-exec`
 - Fresh subagent per task plus two-stage review

package/skills/refactor-plan/SKILL.md

@@ -3,7 +3,7 @@ name: refactor-plan
 description: "Creates a behavior-preserving refactor plan with user interview, repo evidence, tiny commits, scope boundaries, and testing decisions. Not for feature changes or immediate implementation."
 when_to_use: "refactor-plan, refactor request, refactoring RFC, tiny commits, behavior-preserving cleanup, architecture cleanup, 重构计划"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 This skill will be invoked when the user wants to create a refactor request. You should go through the steps below. You may skip steps if you don't consider them necessary.

package/skills/review/SKILL.md

@@ -3,7 +3,7 @@ name: review
 description: "Dispatches a loopx code reviewer subagent against a concrete git range and requirements. Not for implementation, planning, or unresolved review scope."
 when_to_use: "request code review, completed task review, major feature review, pre-merge review, subagent code quality check"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Review

package/skills/spec/SKILL.md

@@ -3,7 +3,7 @@ name: spec
 description: "Writes software design specs from already-clarified requirements, including solution approach, architecture outline, detailed design, tradeoffs, verification design, and handoff context. Not for unresolved requirements, PRD generation, implementation task planning, or code changes."
 when_to_use: "spec, design spec, technical design, design proposal, detailed design, architecture design, 设计方案, 概要设计, 详细设计, 技术方案"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # loopx Spec

package/skills/subagent-exec/SKILL.md

@@ -3,7 +3,7 @@ name: subagent-exec
 description: "Executes approved loopx implementation plans with fresh subagents per independent task and staged review. Not for planning, unclear requirements, or tightly coupled edits."
 when_to_use: "approved implementation plan, independent tasks, subagent execution, staged spec review, code quality review, parallel-capable execution"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Subagent Exec

package/skills/subagent-exec/agents/openai.yaml

@@ -1,3 +1,3 @@
 interface:
-  display_name: "Subagent-Driven Development"
-  short_description: "Execute implementation plans with staged subagent reviews"
+  display_name: "Subagent Exec"
+  short_description: "Execute loopx plans with staged subagent reviews"

package/skills/subagent-exec/codex-subagents.md

@@ -17,7 +17,7 @@ Use this reference before executing this skill in Codex.
 
 Subagent dispatch requires Codex multi-agent support. If `spawn_agent`,
 `wait_agent`, or `close_agent` are unavailable, do not pretend this skill ran
-as subagent-driven development. Use `loopx:exec` instead.
+as subagent-exec. Use `loopx:exec` instead.
 
 Codex environments that require explicit feature flags should enable:
 

package/skills/tdd/SKILL.md

@@ -3,7 +3,7 @@ name: tdd
 description: "Guides feature and bugfix implementation through a failing test before production code and red-green-refactor discipline. Not for generated files or throwaway prototypes."
 when_to_use: "tdd, failing test first, feature implementation, bugfix, regression test, red green refactor, 测试先行"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Test-Driven Development (TDD)

package/skills/verify/SKILL.md

@@ -3,7 +3,7 @@ name: verify
 description: "Requires fresh verification evidence before claiming work is complete, fixed, passing, review-ready, or ready to commit. Not for speculative confidence or stale results."
 when_to_use: "verify, completion claim, fixed claim, tests pass, review-ready, commit, fresh evidence, 验证, 完成前检查"
 metadata:
-  version: "0.2.2"
+  version: "0.2.3"
 ---
 
 # Verification Before Completion

package/src/build-stop-gate.mjs

@@ -73,7 +73,7 @@ export function evaluateBuildStopGate(state) {
     const delegationCount = Number.isFinite(Number(state.active_delegation_count)) ? Number(state.active_delegation_count) : null;
     const delegationText = delegationCount !== null ? ` active delegations=${delegationCount}.` : '';
     const auditText = state.completion_audit_status ? ` completion audit: ${state.completion_audit_status}.` : '';
-    const nextAction = state.next_action ? ` next action: ${state.next_action}` : ' next action: continue the contract-covered next step in $build.';
+    const nextAction = state.next_action ? ` next action: ${state.next_action}` : ' next action: continue the contract-covered loopx build step.';
     const completionSignal = state.completion_signal ? ` completion signal: ${state.completion_signal}` : ' completion signal: review handoff readiness, a real blocker, user stop, or a return to plan/clarify.';
     return {
       allow: false,

package/src/cli.mjs

@@ -6,7 +6,7 @@ import { createInterface } from 'node:readline/promises';
 import { archiveStage, autopilotStage, approveStage, buildStage, clarifyStage, initWorkspace, planStage, reviewStage, statusSummary } from './workflow.mjs';
 import { renderHtmlViews } from './html-views.mjs';
 import { installBundledSkills, installSkillsForTargets } from './install-discovery.mjs';
-import { nextSkillCommand, withNextSkill } from './next-skill.mjs';
+import { nextCliCommand, nextSkillCommand, withNextSkill } from './next-skill.mjs';
 import { doctorRuntime, migrateLegacyRuntime } from './runtime-maintenance.mjs';
 import { setupWorkspaceContext } from './workspace-context.mjs';
 
@@ -178,6 +178,10 @@ function printHumanStatus(status) {
   if (nextSkill) {
     console.log(`next skill: ${nextSkill}`);
   }
+  const nextCli = nextCliCommand(status.state);
+  if (nextCli) {
+    console.log(`next cli: ${nextCli}`);
+  }
   console.log(`next: ${status.next_action}`);
 }
 

package/src/install-discovery.mjs

@@ -31,18 +31,6 @@ const LOOPX_SKILLS = [
   'go-style',
   'kratos',
 ];
-const LOOPX_LEGACY_SKILLS = [
-  'build',
-  'autopilot',
-  'archive',
-  'writing-plans',
-  'executing-plans',
-  'subagent-driven-development',
-  'requesting-code-review',
-  'receiving-code-review',
-  'finishing-a-development-branch',
-  'request-refactor-plan',
-];
 const LOOPX_INSTALLATION_IDENTITY = 'loopx';
 const LOOPX_MANAGED_SCRIPT_ITEMS = [
   {
@@ -453,20 +441,6 @@ async function removeStaleOwnedInstall(currentRow) {
   await removeInstalledSkill(currentRow.installedPath);
 }
 
-async function pruneLegacyLoopxOwnedSkills(nextData, env = process.env) {
-  const pruned = [];
-  for (const skillName of LOOPX_LEGACY_SKILLS) {
-    const row = nextData.skills?.[skillName];
-    if (!isLoopxOwnedIdentity(skillName, row, env)) {
-      continue;
-    }
-    await removeStaleOwnedInstall(row);
-    delete nextData.skills[skillName];
-    pruned.push({ skillName, installedPath: row.installedPath });
-  }
-  return pruned;
-}
-
 async function removeInstalledFile(path) {
   if (!existsSync(path)) {
     return;
@@ -668,7 +642,6 @@ export async function installBundledSkills(env = process.env, options = {}) {
   const nextData = jsonClone(data);
   nextData.version = nextData.version || 3;
   nextData.skills = nextData.skills || {};
-  const pruned = await pruneLegacyLoopxOwnedSkills(nextData, env);
   const baselinePath = getTemplateBaselinePath(env);
   const existingBaseline = await readTemplateBaseline(baselinePath);
   const baselineItemsByPath = new Map((existingBaseline?.items || []).map((item) => [templateItemKey(item), item]));
@@ -767,7 +740,6 @@ export async function installBundledSkills(env = process.env, options = {}) {
     installed,
     conflicts,
     skipped,
-    pruned,
     templateGovernance,
     inspection: await inspectInstallState(env),
   };

package/src/next-skill.mjs

@@ -2,7 +2,6 @@ export function nextSkillCommand(state) {
   if (!state || !state.slug) {
     return null;
   }
-  const reviewBuildCommand = `$build --from-review .loopx/workflows/${state.slug}/review-report.md`;
   if (state.current_stage === 'clarify'
     && state.clarify_current_round > 0
     && state.unresolved_ambiguity_count === 0
@@ -28,7 +27,7 @@ export function nextSkillCommand(state) {
     return null;
   }
   if (state.current_stage === 'plan' && Array.isArray(state.plan_blockers) && state.plan_blockers.length === 0) {
-    return `$build .loopx/plans/requirements-snapshot-${state.slug}.md`;
+    return `$subagent-exec .loopx/plans/requirements-snapshot-${state.slug}.md`;
   }
   if (state.current_stage === 'build'
     && state.stage_status === 'awaiting-approval'
@@ -48,13 +47,13 @@ export function nextSkillCommand(state) {
       || state.approval?.build === 'requested'
       || state.approval?.build === 'approved'
     )) {
-    return reviewBuildCommand;
+    return null;
   }
   if (state.current_stage === 'review'
     && state.review_verdict === 'request-changes'
     && state.requested_transition === 'review->build'
     && state.approval?.build === 'approved') {
-    return reviewBuildCommand;
+    return null;
   }
   if (state.current_stage === 'review'
     && state.review_verdict === 'request-changes'
@@ -71,20 +70,62 @@ export function nextSkillCommand(state) {
   return null;
 }
 
+export function nextCliCommand(state) {
+  if (!state || !state.slug) {
+    return null;
+  }
+  if (state.stage_status === 'awaiting-approval'
+    && state.current_stage === 'plan'
+    && Array.isArray(state.plan_blockers)
+    && state.plan_blockers.length === 0) {
+    return `loopx build .loopx/plans/requirements-snapshot-${state.slug}.md`;
+  }
+  if (state.current_stage === 'review'
+    && state.review_verdict === 'request-changes'
+    && state.rollback_target === 'build'
+    && (
+      state.pending_user_decision === 'review->build'
+      || state.requested_transition === 'review->build'
+      || state.approval?.build === 'requested'
+      || state.approval?.build === 'approved'
+    )) {
+    return `loopx build --from-review .loopx/workflows/${state.slug}/review-report.md`;
+  }
+  if (state.current_stage === 'review'
+    && state.review_verdict === 'request-changes'
+    && state.requested_transition === 'review->build'
+    && state.approval?.build === 'approved') {
+    return `loopx build --from-review .loopx/workflows/${state.slug}/review-report.md`;
+  }
+  return null;
+}
+
 export function nextSkillHint(state) {
   const command = nextSkillCommand(state);
   if (!command) {
     return null;
   }
-  return `Next: ${command}`;
+  return `Next skill: ${command}`;
+}
+
+export function nextCliHint(state) {
+  const command = nextCliCommand(state);
+  if (!command) {
+    return null;
+  }
+  return `Next CLI: ${command}`;
 }
 
 export function withNextSkill(payload, state) {
   const nextCommand = nextSkillCommand(state);
   const nextHint = nextSkillHint(state);
+  const cliCommand = nextCliCommand(state);
+  const cliHint = nextCliHint(state);
   return {
     ...payload,
     next_skill_command: nextCommand,
     next_skill_hint: nextHint,
+    next_cli_command: cliCommand,
+    next_cli_hint: cliHint,
   };
 }

package/src/workflow.mjs

@@ -118,7 +118,7 @@ function reviewReportArtifactPath(slug) {
 }
 
 function reviewReworkBuildCommand(slug) {
-  return `$build --from-review ${reviewReportArtifactPath(slug)}`;
+  return `loopx build --from-review ${reviewReportArtifactPath(slug)}`;
 }
 
 function nowIso() {
@@ -4203,7 +4203,7 @@ export async function buildStage(cwd, slug, options = {}) {
       active_delegation_count: delegationLedger?.active_blocking_count || 0,
       completion_audit_path: displayPath(cwd, completionAuditPath),
       completion_audit_status: completionAudit?.status || 'pending',
-      next_action: 'Continue $build execution and gather fresh implementation evidence.',
+      next_action: 'Continue loopx build execution and gather fresh implementation evidence.',
       completion_signal: 'Build may stop only after execution-record.md is complete and build -> review handoff readiness is reached, or after a real blocker is recorded.',
     });
     current = await adapter.executeLanes({
@@ -4271,7 +4271,7 @@ export async function buildStage(cwd, slug, options = {}) {
       completion_audit_status: completionAudit.status,
       next_action: blockers.length === 0
         ? 'Verify execution evidence and prepare build -> review handoff.'
-        : 'Continue $build to resolve blockers before review handoff.',
+        : 'Continue loopx build to resolve blockers before review handoff.',
       completion_signal: 'Build may stop only after execution-record.md is complete and build -> review handoff readiness is reached, or after a real blocker is recorded.',
     });
     const writtenSupportPaths = await writeBuildSupportArtifacts(root, current, noDeslop, {
@@ -4373,7 +4373,7 @@ export async function buildStage(cwd, slug, options = {}) {
     active_delegation_count: delegationLedger?.active_blocking_count || 0,
     completion_audit_path: displayPath(cwd, completionAuditPath),
     completion_audit_status: completionAudit?.status || (finalBlocked ? 'blocked' : 'passed'),
-    next_action: finalBlocked ? 'Run $build again after resolving recorded blockers.' : 'Approve build -> review and run $review.',
+    next_action: finalBlocked ? 'Run loopx build again after resolving recorded blockers.' : 'Approve build -> review and run $review.',
     completion_signal: finalBlocked ? 'Build is stopped because real blockers remain recorded.' : 'execution-record.md is complete and build -> review handoff is ready.',
     execution_record_status: next.execution_record_status,
     execution_record_path: artifactPath(root, 'execution-record.md'),