@ai-content-space/loopx 0.2.1 → 0.2.3

package/src/install-discovery.mjs

@@ -21,6 +21,7 @@ const LOOPX_SKILLS = [
   'subagent-exec',
   'exec',
   'review',
+  'final-review',
   'fix-review',
   'finish',
   'refactor-plan',
@@ -30,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 = [
   {
@@ -452,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;
@@ -667,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]));
@@ -766,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'),