@ai-content-space/loopx 0.2.3 → 0.2.7

package/src/install-discovery.mjs

@@ -28,6 +28,7 @@ const LOOPX_SKILLS = [
   'debug',
   'tdd',
   'verify',
+  'doc-readability',
   'go-style',
   'kratos',
 ];
@@ -824,10 +825,46 @@ async function mergeClaudeHookSettings(env = process.env, options = {}) {
   return { settingsPath, hookPath, command };
 }
 
+function assertInstallTargetOptions(requestedTargets, options = {}) {
+  const targets = new Set(requestedTargets);
+  if (options.dir && targets.has('codex') && targets.has('claude')) {
+    throw new Error('install_custom_dir_requires_single_target');
+  }
+}
+
+export async function inspectInstallTargets(env = process.env, options = {}) {
+  const requestedTargets = Array.isArray(options.targets) && options.targets.length > 0
+    ? options.targets
+    : ['codex', 'claude'];
+  assertInstallTargetOptions(requestedTargets, options);
+  const results = {};
+  for (const target of requestedTargets) {
+    if (target === 'codex') {
+      results.codex = await inspectInstallState(codexInstallEnv({
+        ...env,
+        LOOPX_INSTALL_CUSTOM_DIR: options.dir,
+      }));
+      continue;
+    }
+    if (target === 'claude') {
+      results.claude = await inspectInstallState(claudeInstallEnv(env, options));
+      continue;
+    }
+    throw new Error(`unknown_install_target:${target}`);
+  }
+  return {
+    ok: true,
+    dryRun: true,
+    targets: requestedTargets,
+    results,
+  };
+}
+
 export async function installSkillsForTargets(env = process.env, options = {}) {
   const requestedTargets = Array.isArray(options.targets) && options.targets.length > 0
     ? options.targets
     : ['codex', 'claude'];
+  assertInstallTargetOptions(requestedTargets, options);
   const results = {};
   for (const target of requestedTargets) {
     if (target === 'codex') {
@@ -866,6 +903,7 @@ export async function verifyInstallTargets(env = process.env, options = {}) {
   const requestedTargets = Array.isArray(options.targets) && options.targets.length > 0
     ? options.targets
     : ['codex', 'claude'];
+  assertInstallTargetOptions(requestedTargets, options);
   const results = {};
   for (const target of requestedTargets) {
     if (target === 'codex') {

package/src/next-skill.mjs

@@ -12,16 +12,8 @@ export function nextSkillCommand(state) {
     return `$plan ${state.slug}`;
   }
   if (state.current_stage === 'done'
-    && state.completion_confirmed === true
-    && state.archive_status !== 'archived') {
-    return `$archive ${state.slug}`;
-  }
-  if (state.current_stage === 'review'
-    && state.review_verdict === 'approve'
-    && state.pending_user_decision === 'review->done'
-    && ['requested', 'approved'].includes(state.approval?.complete)
-    && state.archive_status !== 'archived') {
-    return `$archive ${state.slug}`;
+    && state.completion_confirmed === true) {
+    return '$finish';
   }
   if (state.stage_status !== 'awaiting-approval') {
     return null;
@@ -80,6 +72,12 @@ export function nextCliCommand(state) {
     && state.plan_blockers.length === 0) {
     return `loopx build .loopx/plans/requirements-snapshot-${state.slug}.md`;
   }
+  if (state.current_stage === 'review'
+    && state.review_verdict === 'approve'
+    && state.pending_user_decision === 'review->done'
+    && ['requested', 'approved'].includes(state.approval?.complete)) {
+    return `loopx approve ${state.slug} --from review --to done`;
+  }
   if (state.current_stage === 'review'
     && state.review_verdict === 'request-changes'
     && state.rollback_target === 'build'

package/src/workflow.mjs

@@ -523,25 +523,22 @@ function buildWorkspaceReadme() {
     '',
     '## Default Flow',
     '',
-    '`clarify -> plan -> build -> review -> done`',
+    '`clarify -> spec? -> plan -> (subagent-exec | exec) -> final-review -> fix-review? -> finish`',
     '',
-    '## Runtime Commands',
+    '## User Commands',
     '',
     '- `loopx init [--slug <slug>]`',
-    '- `loopx clarify <slug>`',
-    '- `loopx approve <slug> --from <stage> --to <stage>`',
-    '- `loopx plan <slug>`',
-    '- `loopx build <slug>`',
-    '- `loopx review <slug> [--reviewer <name>]`',
-    '- `loopx archive <slug>`',
-    '- `loopx autopilot <slug> [--reviewer <name>]`',
+    '- `loopx clarify <slug> [--standard|--deep] [--json]`',
     '- `loopx render [slug|--all]`',
     '- `loopx status [slug] [--json]`',
+    '- `loopx next <slug> [--json]`',
     '- `loopx setup-context`',
     '- `loopx doctor`',
     '- `loopx migrate`',
     '- `loopx repair-install`',
     '',
+    'Advanced runtime commands are hidden from the normal path. Use `loopx help advanced` only when debugging runtime compatibility paths.',
+    '',
     '## Document Boundaries',
     '',
     'User-facing documents to watch:',
@@ -549,7 +546,7 @@ function buildWorkspaceReadme() {
     '- `workflows/<slug>/spec.md`',
     '- `workflows/<slug>/plan.md`, `architecture.md`, `development-plan.md`, and `test-plan.md`',
     '- `workflows/<slug>/execution-record.md` and `review-report.md`',
-    '- `views/index.html` and `workflows/<slug>/view/index.html` after `loopx plan` or `loopx render`',
+    '- `views/index.html` and `workflows/<slug>/view/index.html` after planning or `loopx render`',
     '',
     'Documents users may read and edit as workflow fact sources:',
     '',
@@ -3029,7 +3026,7 @@ function buildCurrentEvidenceChain(state, readiness = buildReadiness(state), aut
     evidence.push(evidenceEntry(
       'review_approved',
       'Review verdict is approve.',
-      authorization.done.authorized ? 'Archive can consume the approved review -> done transition before syncing specs.' : 'Completion still requires explicit review -> done authorization.',
+      authorization.done.authorized ? 'Approved review can proceed with explicit review -> done authorization before $finish records branch disposition and learning audit.' : 'Completion still requires explicit review -> done authorization before $finish.',
     ));
   }
   if (state.archive_status === 'archived' && state.spec_sync_status === 'synced') {
@@ -3134,8 +3131,8 @@ function recommendedAction(state, legacy = false) {
   switch (state.current_stage) {
     case STAGES.CLARIFY:
       return state.approval.plan === APPROVAL_STATES.APPROVED
-        ? 'Run loopx plan to consume the approved clarify -> plan transition.'
-        : `Resolve ambiguity in ${state.clarify_profile ?? 'standard'} clarify mode and approve clarify -> plan.`;
+        ? `Follow $plan ${state.slug}.`
+        : 'Finish clarification, then follow $plan when ready.';
     case STAGES.PLAN:
       if (Array.isArray(state.plan_blockers) && state.plan_blockers.length > 0) {
         return 'Run loopx plan to continue the planning review loop until architect, critic, and planning artifact blockers are cleared.';
@@ -3152,7 +3149,9 @@ function recommendedAction(state, legacy = false) {
         : 'Approve build -> review when execution-record.md is complete.';
     case STAGES.REVIEW:
       if (state.review_verdict === 'approve') {
-        return 'Run loopx archive; archive consumes the pending review -> done completion transition before syncing specs.';
+        return state.approval.complete === APPROVAL_STATES.APPROVED
+          ? 'Run $finish to complete branch disposition and learning audit.'
+          : 'Approve review -> done, then run $finish to complete branch disposition and learning audit.';
       }
       if (state.review_verdict === 'request-changes') {
         if (state.requested_transition === TRANSITIONS.REVIEW_TO_BUILD && state.approval.build === APPROVAL_STATES.APPROVED) {
@@ -3177,10 +3176,7 @@ function recommendedAction(state, legacy = false) {
       if (state.autopilot_current_phase && state.autopilot_current_phase !== 'none' && state.autopilot_completed) {
         return 'Autopilot run is complete.';
       }
-      if (state.archive_status !== 'archived') {
-        return 'Run loopx archive to sync the approved change delta into long-lived specs.';
-      }
-      return 'Workflow is complete.';
+      return 'Workflow is complete. Run $finish if branch disposition and learning audit have not been recorded.';
     default:
       return 'Run loopx clarify to start a workflow.';
   }
@@ -3353,11 +3349,8 @@ function nextCommandForRollbackTarget(slug, target) {
   if (target === 'none') {
     return [
       'Next:',
-      `$archive ${slug}`,
-      '',
-      'CLI-only equivalent:',
       `loopx approve ${slug} --from review --to done`,
-      `loopx archive ${slug}`,
+      '$finish',
     ].join('\n');
   }
   return [
@@ -3370,7 +3363,7 @@ function nextCommandForRollbackTarget(slug, target) {
 function reviewUserMessageZh({ slug, verdict, rollbackTarget, findings }) {
   const label = reviewVerdictLabel(verdict);
   const next = verdict === 'APPROVE'
-    ? `下一步：直接归档；archive 会先消费 pending 的 review -> done 完成态。\n${nextCommandForRollbackTarget(slug, 'none')}`
+    ? `下一步：批准 review -> done，然后执行 finish 完成分支处置和学习审计。\n${nextCommandForRollbackTarget(slug, 'none')}`
     : `下一步：按审查发现处理，并${rollbackTargetLabel(rollbackTarget)}。\n${nextCommandForRollbackTarget(slug, rollbackTarget)}`;
   const findingText = Array.isArray(findings) && findings.length > 0 ? findings.join('；') : '无额外发现。';
   return `Review 结果：${slug} ${label}。审查发现：${findingText} ${next}`;
@@ -3530,8 +3523,8 @@ export async function initWorkspace(cwd, { slug, agentDelegation = {} } = {}) {
     schema_version: WORKSPACE_SCHEMA_VERSION,
     tool: 'loopx',
     product_contract: 'skill-first-v1',
-    default_flow: ['clarify', 'plan', 'build', 'review', 'done', 'archive'],
-    preferred_surface: ['clarify', 'plan', 'build', 'review', 'archive', 'autopilot'],
+    default_flow: ['clarify', 'plan', 'subagent-exec', 'final-review', 'finish'],
+    preferred_surface: ['clarify', 'plan', 'subagent-exec', 'review', 'final-review', 'fix-review', 'finish'],
     source_of_truth_policy: projectConventions.source_of_truth_policy,
     project_conventions: {
       existing_ai_rules: projectConventions.existing_ai_rules,
@@ -4736,7 +4729,7 @@ export async function reviewStage(cwd, slug, { reviewer = 'independent-reviewer'
         verdict: reviewInput.verdict,
         reviewMessageZh: reviewMessage,
         evidenceManifest: reviewInput.evidenceManifest,
-        followUps: ['执行 $archive；archive 会消费 pending 的 review -> done 完成态。'],
+        followUps: ['批准 review -> done，然后执行 $finish 完成分支处置和学习审计。'],
       });
     } catch (error) {
       journalWarning = error instanceof Error ? error.message : String(error);

package/skills/deepsearch/SKILL.md

@@ -1,38 +0,0 @@
----
-name: deepsearch
-description: Thorough codebase search
----
-
-# Deep Search Mode
-
-[DEEPSEARCH MODE ACTIVATED]
-
-## Objective
-
-Perform thorough search of the codebase for the specified query, pattern, or concept.
-
-## Search Strategy
-
-1. **Broad Search**
-   - Search for exact matches
-   - Search for related terms and variations
-   - Check common locations (components, utils, services, hooks)
-
-2. **Deep Dive**
-   - Read files with matches
-   - Check imports/exports to find connections
-   - Follow the trail (what imports this? what does this import?)
-
-3. **Synthesize**
-   - Map out where the concept is used
-   - Identify the main implementation
-   - Note related functionality
-
-## Output Format
-
-- **Primary Locations** (main implementations)
-- **Related Files** (dependencies, consumers)
-- **Usage Patterns** (how it's used across the codebase)
-- **Key Insights** (patterns, conventions, gotchas)
-
-Focus on being comprehensive but concise. Cite file paths and line numbers.