nexarch 0.12.9 → 0.12.10

package/dist/commands/init-project-infra.js

@@ -25,6 +25,36 @@ async function confirm(question) {
         rl.close();
     }
 }
+/**
+ * The ingest runs that still have to happen, one per root module.
+ *
+ * Registration records the repository; it does not record the estate. Only
+ * state does that, and state is per root module — seven of them here, each
+ * needing its own Terraform run. Naming this as a list of commands rather than
+ * a paragraph of advice is the difference between an agent finishing the job
+ * and an agent reporting "skeleton_only" and stopping, which is exactly what
+ * happened when the handoff lived only in prose at the end of a real run.
+ */
+function nextStepsFor(projectRef, detection) {
+    return detection.rootModules.map((root) => ({
+        rootModule: root.relative,
+        environment: root.environmentHint,
+        initialised: root.initialised,
+        command: `nexarch ingest-infra --dir ${root.relative} --project-ref ${projectRef}`,
+    }));
+}
+/** Prints the per-root ingest commands, the part that actually populates the graph. */
+function printNextSteps(steps, registered) {
+    if (steps.length === 0)
+        return;
+    console.log(`\n${registered ? "Registered" : "Would register"} the repository — the estate itself is not in the graph yet.`);
+    console.log(`Each of the ${steps.length} root modules holds its own state and needs its own ingest:\n`);
+    for (const step of steps) {
+        const notes = [step.environment ? `${step.environment} environment` : "shared stack", step.initialised ? "initialised" : "run `terraform init` here first"].join(", ");
+        console.log(`    # ${step.rootModule} — ${notes}`);
+        console.log(`    ${step.command}`);
+    }
+}
 /**
  * Shows what a real run would write.
  *
@@ -35,8 +65,19 @@ async function confirm(question) {
  */
 function reportDryRun(params) {
     const { asJson, projectRef, detection, entities, relationships } = params;
+    const nextSteps = nextStepsFor(projectRef, detection);
     if (asJson) {
-        process.stdout.write(`${JSON.stringify({ ok: true, dryRun: true, wrote: false, projectRef, detection, plan: { entities, relationships } }, null, 2)}\n`);
+        process.stdout.write(`${JSON.stringify({
+            ok: true,
+            dryRun: true,
+            wrote: false,
+            projectRef,
+            registrationStatus: "skeleton_only",
+            enrichmentCompleted: false,
+            detection,
+            plan: { entities, relationships },
+            nextSteps,
+        }, null, 2)}\n`);
         return;
     }
     console.log(`\nWould write ${entities.length} entit${entities.length === 1 ? "y" : "ies"}:`);
@@ -47,7 +88,11 @@ function reportDryRun(params) {
         for (const rel of relationships)
             console.log(`    ${rel.fromEntityRef} --${rel.relationshipTypeCode}--> ${rel.toEntityRef}`);
     }
-    console.log(`\nNothing was written. Re-run without --dry-run to register.`);
+    // The dry run used to stop here, which made registration look like the whole
+    // job: a reader saw four entities and no hint that seven ingest runs were the
+    // actual work. A preview has to preview the handoff too.
+    printNextSteps(nextSteps, false);
+    console.log(`\nNothing was written. Re-run without --dry-run to register, then run the commands above.`);
 }
 export async function runInfrastructureOnboarding(options, detection) {
     const { dir, nameOverride, asJson, nonInteractive, skipIngest, dryRun } = options;
@@ -132,15 +177,9 @@ export async function runInfrastructureOnboarding(options, detection) {
     // the map up front — rather than letting the caller discover it through a
     // failure — is what lets an agent finish the job unaided.
     const roots = detection.rootModules;
-    if (!asJson && roots.length > 1) {
-        console.log(`\nThis repository has ${roots.length} root modules, each with its own state.`);
-        console.log("Ingest them one at a time — every root module is a separate estate:\n");
-        for (const root of roots) {
-            const notes = [root.environmentHint ? `${root.environmentHint} environment` : "shared stack", root.initialised ? "initialised" : "run `terraform init` here first"].join(", ");
-            console.log(`    # ${root.relative} — ${notes}`);
-            console.log(`    nexarch ingest-infra --dir ${root.relative} --project-ref ${projectRef}`);
-        }
-    }
+    const nextSteps = nextStepsFor(projectRef, detection);
+    if (!asJson && roots.length > 1)
+        printNextSteps(nextSteps, true);
     let ingested = false;
     if (!skipIngest && roots.length <= 1) {
         const shouldIngest = nonInteractive ? false : await confirm("\nRead the current state and populate the graph now?");
@@ -156,9 +195,23 @@ export async function runInfrastructureOnboarding(options, detection) {
             }
         }
     }
+    // Registration alone is a skeleton, and saying so in the payload keeps the
+    // caller's report honest: the numbers it needs to decide "am I finished?"
+    // come from the command that did the work, not from a convention it half
+    // remembers.
+    const outstanding = ingested ? nextSteps.filter((step) => step.rootModule !== "." && roots.length > 1) : nextSteps;
     const { file, snippet } = ciSnippetFor(detection.ciSystem, detection.ciFile);
     if (asJson) {
-        process.stdout.write(`${JSON.stringify({ ok: true, projectRef, detection, ingested, ci: { file, snippet } }, null, 2)}\n`);
+        process.stdout.write(`${JSON.stringify({
+            ok: true,
+            projectRef,
+            registrationStatus: outstanding.length > 0 ? "skeleton_only" : "enriched",
+            enrichmentCompleted: outstanding.length === 0,
+            detection,
+            ingested,
+            nextSteps: outstanding,
+            ci: { file, snippet },
+        }, null, 2)}\n`);
         return;
     }
     console.log(`\nInfrastructure changes continuously, so ingestion belongs in the pipeline, not in someone's memory.`);
@@ -166,5 +219,9 @@ export async function runInfrastructureOnboarding(options, detection) {
     console.log(snippet);
     console.log(`\nNEXARCH_TOKEN is a service credential — create one in the workspace under`);
     console.log(`Discovery → Agents, then store it as a masked CI variable.`);
+    if (outstanding.length > 0) {
+        console.log(`\nThis run registered the repository only. Until the ${outstanding.length} ingest command${outstanding.length === 1 ? "" : "s"} above`);
+        console.log(`have run, the graph knows the repo exists but nothing it provisions.`);
+    }
 }
 export { detectInfrastructureProject };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nexarch",
-  "version": "0.12.9",
+  "version": "0.12.10",
   "description": "Your architecture workspace for AI delivery.",
   "keywords": [
     "nexarch",