@ai-content-space/loopx 0.1.1 → 0.1.2

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@ai-content-space/loopx",
   "type": "module",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "Skill-first loopx workflow product for Codex",
   "repository": {
     "type": "git",

package/skills/plan/SKILL.md

@@ -130,6 +130,7 @@ Critic evaluates:
 - clarity of risk mitigation
 - testable acceptance criteria
 - concrete verification steps
+- execution-input completeness for each new or changed ingress / workflow entrypoint
 - explicit non-goals and decision boundaries
 - in deliberate mode: pre-mortem and expanded test plan quality
 
@@ -166,6 +167,7 @@ The final plan must include:
 
 - ADR: Decision, Drivers, Alternatives considered, Why chosen, Consequences, Follow-ups
 - concrete implementation steps sized to the actual task
+- execution inputs mapped to concrete sources before build starts
 - available execution lanes and recommended lane
 - test and verification commands
 - residual risks and assumptions
@@ -199,6 +201,7 @@ Without `--interactive`, report the approved plan and recommended next command,
 - `plan_package_status`: `missing|partial|complete`
 - `plan_acceptance_criteria_testable`: `true|false`
 - `plan_verification_steps_resolved`: `true|false`
+- `plan_execution_inputs_resolved`: `true|false`
 - `requested_transition`: remains explicit before build/autopilot
 
 The plan gate is blocked until:
@@ -208,6 +211,7 @@ The plan gate is blocked until:
 - Critic verdict is `approve`
 - acceptance criteria are testable
 - verification steps are concrete
+- execution inputs are fully mapped to concrete sources
 - user approval exists for any execution transition
 </Runtime_State_Machine>
 

package/src/cli.mjs

@@ -2,6 +2,7 @@
 
 import { autopilotStage, approveStage, buildStage, clarifyStage, initWorkspace, planStage, reviewStage, statusSummary } from './workflow.mjs';
 import { installBundledSkills } from './install-discovery.mjs';
+import { withNextSkill } from './next-skill.mjs';
 import { doctorRuntime, migrateLegacyRuntime } from './runtime-maintenance.mjs';
 
 function usage() {
@@ -118,7 +119,7 @@ async function main() {
       case 'clarify': {
         const profile = options.get('--deep') ? 'deep' : 'standard';
         const result = await clarifyStage(process.cwd(), positionals[0], { profile });
-        console.log(JSON.stringify({ ok: true, command, root: result.root, state: result.state }, null, 2));
+        console.log(JSON.stringify(withNextSkill({ ok: true, command, root: result.root, state: result.state }, result.state), null, 2));
         return;
       }
       case 'approve': {
@@ -126,7 +127,7 @@ async function main() {
           from: options.get('--from'),
           to: options.get('--to'),
         });
-        console.log(JSON.stringify({ ok: true, command, root: result.root, state: result.state }, null, 2));
+        console.log(JSON.stringify(withNextSkill({ ok: true, command, root: result.root, state: result.state }, result.state), null, 2));
         return;
       }
       case 'plan': {
@@ -135,21 +136,21 @@ async function main() {
           interactive: Boolean(options.get('--interactive')),
           deliberate: Boolean(options.get('--deliberate')),
         });
-        console.log(JSON.stringify({ ok: true, command, root: result.root, state: result.state }, null, 2));
+        console.log(JSON.stringify(withNextSkill({ ok: true, command, root: result.root, state: result.state }, result.state), null, 2));
         return;
       }
       case 'build': {
         const result = await buildStage(process.cwd(), positionals[0], {
           noDeslop: Boolean(options.get('--no-deslop')),
         });
-        console.log(JSON.stringify({ ok: true, command, root: result.root, state: result.state }, null, 2));
+        console.log(JSON.stringify(withNextSkill({ ok: true, command, root: result.root, state: result.state }, result.state), null, 2));
         return;
       }
       case 'review': {
         const result = await reviewStage(process.cwd(), positionals[0], {
           reviewer: options.get('--reviewer') || 'independent-reviewer',
         });
-        console.log(JSON.stringify({ ok: true, command, root: result.root, state: result.state, verdict: result.verdict }, null, 2));
+        console.log(JSON.stringify(withNextSkill({ ok: true, command, root: result.root, state: result.state, verdict: result.verdict }, result.state), null, 2));
         return;
       }
       case 'autopilot': {

package/src/next-skill.mjs

@@ -0,0 +1,33 @@
+export function nextSkillCommand(state) {
+  if (!state || state.stage_status !== 'awaiting-approval' || !state.slug) {
+    return null;
+  }
+  if (state.current_stage === 'clarify') {
+    return `$plan ${state.slug}`;
+  }
+  if (state.current_stage === 'plan' && Array.isArray(state.plan_blockers) && state.plan_blockers.length === 0) {
+    return `$build ${state.slug}`;
+  }
+  if (state.current_stage === 'build' && Array.isArray(state.build_blockers) && state.build_blockers.length === 0) {
+    return `$review ${state.slug}`;
+  }
+  return null;
+}
+
+export function nextSkillHint(state) {
+  const command = nextSkillCommand(state);
+  if (!command) {
+    return null;
+  }
+  return `Next: ${command}`;
+}
+
+export function withNextSkill(payload, state) {
+  const nextCommand = nextSkillCommand(state);
+  const nextHint = nextSkillHint(state);
+  return {
+    ...payload,
+    next_skill_command: nextCommand,
+    next_skill_hint: nextHint,
+  };
+}

package/src/plan-runtime.mjs

@@ -65,6 +65,8 @@ function chineseBulletList(items) {
 
 function plannerDraftFromSource({ slug, sourceText, deliberateMode }) {
   const summary = buildSourceSummary(sourceText);
+  const executionInputs = bulletsFromSection(extractSection(sourceText, 'Execution Inputs'), []);
+  const executionInputsResolved = executionInputs.length > 0 && executionInputs.every((item) => !/\b(TBD|待定|unknown|later)\b/i.test(item));
   const preMortem = deliberateMode
     ? [
         '如果运行时没有独立的规划适配层，真实编排与测试替身会相互污染。',
@@ -113,10 +115,14 @@ function plannerDraftFromSource({ slug, sourceText, deliberateMode }) {
       '## Implementation Steps',
       '',
       '1. Add a plan orchestration adapter for planner, architect, and critic.',
-      '2. Record plan iteration, review verdicts, and docs blockers in workflow state.',
+      '2. Record plan iteration, review verdicts, execution-input blockers, and docs blockers in workflow state.',
       '3. Generate canonical plan artifacts and Chinese docs outputs from the approved planning source.',
       '4. Expose plan-stage progress in CLI status.',
-      '5. Add deterministic regression coverage for happy path, iterate path, and docs blockers.',
+      '5. Add deterministic regression coverage for happy path, iterate path, docs blockers, and unresolved execution inputs.',
+      '',
+      '## Execution Inputs',
+      '',
+      ...(executionInputs.length > 0 ? executionInputs.map((item) => `- ${item}`) : ['- TBD: execution inputs not yet mapped to concrete sources.']),
       '',
       '## Risks',
       '',
@@ -187,10 +193,11 @@ function plannerDraftFromSource({ slug, sourceText, deliberateMode }) {
       '- clarify -> plan happy path',
       '- critic iterate then approve path',
       '- docs missing or non-Chinese blocking path',
+      '- execution inputs missing or marked TBD blocking path',
       '',
       '## Observability',
       '',
-      '- status exposes iteration, architect review status, critic verdict, and docs blockers',
+      '- status exposes iteration, architect review status, critic verdict, execution input blockers, and docs blockers',
     ].join('\n'),
     docs: {
       architecture: [
@@ -241,6 +248,7 @@ function plannerDraftFromSource({ slug, sourceText, deliberateMode }) {
     optionsReviewed: true,
     acceptanceCriteriaTestable: true,
     verificationStepsResolved: true,
+    executionInputsResolved,
   };
 }
 
@@ -284,12 +292,16 @@ function defaultCriticReview({ plannerDraft, iteration }) {
   if (!plannerDraft.verificationStepsResolved) {
     findings.push('Verification steps are not concrete.');
   }
+  if (!plannerDraft.executionInputsResolved) {
+    findings.push('Execution inputs are not fully mapped to concrete sources.');
+  }
   if (!containsChinese(plannerDraft.docs.architecture) || !containsChinese(plannerDraft.docs.design) || !containsChinese(plannerDraft.docs.testPlan)) {
     findings.push('Required docs outputs are not Chinese.');
   }
   return reviewArtifact('critic', iteration, findings.length > 0 ? 'iterate' : 'approve', findings, {
     acceptanceCriteriaTestable: plannerDraft.acceptanceCriteriaTestable,
     verificationStepsResolved: plannerDraft.verificationStepsResolved,
+    executionInputsResolved: plannerDraft.executionInputsResolved,
   });
 }
 
@@ -302,6 +314,9 @@ function scriptedVerdict(script, index, fallback) {
 }
 
 function scriptedCriticReview({ plannerDraft, iteration }, script, index) {
+  if (!Array.isArray(script) || script.length === 0) {
+    return defaultCriticReview({ plannerDraft, iteration });
+  }
   const verdict = scriptedVerdict(script, index, 'approve');
   const findings = verdict === 'approve'
     ? ['Structured planning outputs satisfy the scripted approval path.']
@@ -309,6 +324,7 @@ function scriptedCriticReview({ plannerDraft, iteration }, script, index) {
   return reviewArtifact('critic', iteration, verdict, findings, {
     acceptanceCriteriaTestable: plannerDraft.acceptanceCriteriaTestable,
     verificationStepsResolved: plannerDraft.verificationStepsResolved,
+    executionInputsResolved: plannerDraft.executionInputsResolved,
   });
 }
 
@@ -321,6 +337,9 @@ export function createScriptedPlanAdapter(script = {}) {
     },
     async architect(context) {
       const base = defaultArchitectReview(context);
+      if (!Array.isArray(script.architect) || script.architect.length === 0) {
+        return base;
+      }
       const mode = scriptedVerdict(script.architect, architectIndex, 'approve');
       architectIndex += 1;
       return {
@@ -363,7 +382,8 @@ export function createRealPlanAdapter({ model } = {}) {
         '  "principlesResolved": boolean,',
         '  "optionsReviewed": boolean,',
         '  "acceptanceCriteriaTestable": boolean,',
-        '  "verificationStepsResolved": boolean',
+        '  "verificationStepsResolved": boolean,',
+        '  "executionInputsResolved": boolean',
         '}',
         `Deliberate mode: ${Boolean(context.deliberateMode)}`,
         '',
@@ -433,7 +453,8 @@ export function createRealPlanAdapter({ model } = {}) {
         '  "verdict": "approve" | "iterate" | "reject",',
         '  "findings": string[],',
         '  "acceptanceCriteriaTestable": boolean,',
-        '  "verificationStepsResolved": boolean',
+        '  "verificationStepsResolved": boolean,',
+        '  "executionInputsResolved": boolean',
         '}',
         'Do not ask questions. Do not wrap JSON in markdown.',
         '',

package/src/workflow.mjs

@@ -329,6 +329,7 @@ function createInitialState(slug, profile) {
     plan_critic_verdict: 'none',
     plan_acceptance_criteria_testable: false,
     plan_verification_steps_resolved: false,
+    plan_execution_inputs_resolved: false,
     plan_docs_status: 'missing',
     plan_docs_artifact_paths: null,
     plan_review_artifact_paths: [],
@@ -588,6 +589,9 @@ async function readPlanCompletion(cwd, root, slug, state) {
   if (!state.plan_verification_steps_resolved) {
     blockers.push('verification_steps_unresolved');
   }
+  if (!state.plan_execution_inputs_resolved) {
+    blockers.push('execution_inputs_unresolved');
+  }
   if (!state.plan_artifact_path || !existsSync(state.plan_artifact_path)) {
     blockers.push('missing_prd');
   }
@@ -1215,6 +1219,7 @@ export async function planStage(cwd, slug, options = {}) {
       plan_critic_verdict: criticReview.verdict,
       plan_acceptance_criteria_testable: criticReview.acceptanceCriteriaTestable,
       plan_verification_steps_resolved: criticReview.verificationStepsResolved,
+      plan_execution_inputs_resolved: criticReview.executionInputsResolved,
       plan_package_status: 'complete',
       plan_docs_artifact_paths: docPaths,
       plan_review_artifact_paths: reviewArtifactPaths,
@@ -1322,7 +1327,7 @@ export async function buildStage(cwd, slug, options = {}) {
     build_support_evidence_paths: supportArtifacts,
     build_no_deslop: noDeslop,
     active_run_id: current?.runId || null,
-    pending_user_decision: TRANSITIONS.NONE,
+    pending_user_decision: finalBlocked ? TRANSITIONS.NONE : TRANSITIONS.BUILD_TO_REVIEW,
     requested_transition: TRANSITIONS.NONE,
     last_confirmed_transition: TRANSITIONS.PLAN_TO_BUILD,
     approval: {

package/templates/plan.md

@@ -24,6 +24,12 @@ stage_owner: plan
 2. keep the loop bounded and deterministic
 3. verify outputs before review
 
+## Execution Inputs
+
+- list every required build-time input for new or changed entrypoints
+- map each input to a concrete source: config, filename, payload, upstream API, user input, or default
+- no item may remain TBD before plan approval
+
 ## Risks
 
 - skipping approval or verification would break the loopx contract