@cspeach/cli 1.1.27 → 1.1.29

package/dist/router/router-core.js

@@ -36,7 +36,7 @@ export const ROUTER_SCHEMA_VERSION = 1;
  * its own bundled copy — so a stale published manifest never rolls a newer
  * CLI back to older routing. router-revision-pin.test.ts fails until bumped.
  */
-export const ROUTER_REVISION = 1;
+export const ROUTER_REVISION = 2;
 // 2026-06-12: keep in lockstep with .claude/skills (35 abap skills).
 // abap-dump replaced abap-dump-analyze + abap-dump-triage (portfolio merge);
 // added previously-missing abap-plan, abap-fiori-build, abap-cca-merge, abap-upgrade-merge.
@@ -74,7 +74,7 @@ export const SKILL_DESCRIPTIONS = {
     'abap-incident': 'End-to-end incident pipeline — from short dump to fix deployed and documented.',
     'abap-migrate': 'ECC→S/4HANA migration assessment for custom objects — API analysis, successor lookup, per-object effort.',
     'abap-modernize': 'Transform a CLASSIC program into a modern architecture (e.g. report → Fiori Elements / RAP) for the release level.',
-    'abap-plan': 'Plan and run a multi-session SAP project — phase envelope carries state across sessions, one phase per session.',
+    'abap-plan': 'Plan and run a multi-step SAP project — one approval of the plan and its object list, then the steps run back to back; the plan file carries state across sessions.',
     'abap-preflight': 'Pre-release checklist for a change set — dependencies, tests, rollback, readiness.',
     'abap-rap': 'Scaffold a complete RAP stack from scratch — table, CDS interface+projection, BDEF, behavior class, service definition+binding.',
     'abap-refactor': 'Modify / improve / add a feature to EXISTING custom ABAP code in place — restructure, clean up, or change a Z/Y program/class (incl. small feature changes like adding output coloring).',

package/dist/sap/reachability-state.js

@@ -0,0 +1,25 @@
+// cspeach-cli/src/sap/reachability-state.ts
+//
+// 2026-10-04 (continuous run, spec §3.5) — set when a SAP tool fails because
+// the SAP host is not reachable from this machine; read (and cleared) by the
+// REPL after a plan turn so the build pauses instead of failing.
+let unreachable = null;
+const CODE_RE = /\b(ETIMEDOUT|ECONNREFUSED|ENOTFOUND|EHOSTUNREACH|ENETUNREACH|ECONNRESET|EAI_AGAIN)\b/;
+export const SAP_UNREACHABLE_DIRECTIVE = 'SAP is not reachable from this machine right now. Stop here: do not retry SAP tools and do not write the plan manifest. '
+    + 'End the turn with one short sentence. CSPeach pauses the build and runs this step again when the network is back.';
+export function noteSapUnreachable(code) {
+    unreachable = code;
+}
+/** True while a SAP-unreachable failure is noted for this turn (does not clear it). */
+export function isSapUnreachableNoted() {
+    return unreachable !== null;
+}
+export function getAndResetSapUnreachable() {
+    const c = unreachable;
+    unreachable = null;
+    return c;
+}
+export function unreachableCodeInText(text) {
+    const m = CODE_RE.exec(String(text ?? ''));
+    return m ? m[1] : null;
+}

package/dist/session/audit-export.js

@@ -45,6 +45,7 @@ const WRITE_TOOLS = new Set([
     'sap_delete_object',
     'sap_create_object',
     'sap_set_class_include',
+    'sap_set_text_elements',
     'sap_create_domain',
     'sap_create_data_element',
     'sap_activate',
@@ -277,18 +278,11 @@ function outcomeCell(c) {
 }
 // ── Operator identity ─────────────────────────────────────────────────────
 //
-// SessionState carries NO SAP logon user (only the sap_system alias). The one
-// identity available is the local CSPeach operator, resolved the same way as
-// agent/loop.ts getAuthorIdentity (CSPEACH_AUTHOR_NAME → USER → USERNAME). It
-// is the person who RAN CSPeach, not the SAP account writes landed under — the
-// header labels it "Operator" and shows the SAP system separately so the two
-// are never conflated. (Inlined rather than importing agent/loop.js to keep
-// this a dependency-light pure builder.)
-function resolveOperator() {
-    return (process.env.CSPEACH_AUTHOR_NAME ||
-        process.env.USER ||
-        process.env.USERNAME ||
-        'unknown');
+// 2026-10-04 — the person who RAN CSPeach, as CSPeach knows them: the signed-in
+// account (from /v1/me), else an explicit CSPEACH_AUTHOR_NAME, else "not
+// signed in". NEVER the OS account name (USER/USERNAME printed "Admin").
+function resolveOperator(operator) {
+    return operator?.trim() || process.env.CSPEACH_AUTHOR_NAME?.trim() || 'not signed in (local run)';
 }
 // ── Transports seen ───────────────────────────────────────────────────────
 function transportsSeen(toolCalls, sessionTransport) {
@@ -320,7 +314,7 @@ function transportsSeen(toolCalls, sessionTransport) {
 }
 // ── Main builder ──────────────────────────────────────────────────────────
 export function buildAuditReport(input) {
-    const { session, costEntries, transport, credits } = input;
+    const { session, costEntries, transport, credits, operator } = input;
     const calls = session.toolCalls;
     const completed = calls.filter((c) => c.completed === true);
     const approvals = parseApprovals(calls);
@@ -344,14 +338,14 @@ export function buildAuditReport(input) {
     L();
     L(`- **Session:** \`${session.id}\``);
     L(`- **SAP system:** ${session.sap_system ?? '—'}`);
-    L(`- **Operator (local):** ${resolveOperator()}`);
+    L(`- **Operator (CSPeach sign-in):** ${resolveOperator(operator)}`);
     L(`- **Period:** ${dateFrom} → ${dateTo}`);
     const trs = transportsSeen(calls, transport);
     L(`- **Transport(s):** ${trs.length ? trs.map((t) => `\`${t}\``).join(', ') : '—'}`);
     L();
     L('> Scope: this record covers SAP-system mutations only — local file and');
-    L('> shell operations are excluded by design. Operator is the local CSPeach');
-    L('> user (who ran the session); the SAP logon user is not recorded in');
+    L('> shell operations are excluded by design. Operator is the CSPeach sign-in');
+    L('> of the person who ran the session; the SAP logon user is not recorded in');
     L('> session state. Plan audit states, if any, live in the plan handover,');
     L('> not this report.');
     L();

package/dist/session/recap.js

@@ -90,25 +90,18 @@ export function buildRecap(session, cwd, opts) {
     return lines;
 }
 /**
- * Scan `workspaceDir` for the newest plan family and, when it still has
- * unsatisfied phases, produce the plan recap fact. Returns null when there
- * is no plan, the newest plan family is fully satisfied, or its envelope is
- * invalid (silent skip by design).
- *
- * "Newest family": envelope files group into families by filename base
- * (`<base>-v<N>[-<sub>].cspeach.json` — same convention as
- * pickNewestPlanVersion / findNewestVersion in commands/plan-resume.ts);
- * within a family the highest version wins; families rank by their newest
- * file's mtime. Non-plan envelopes (design, spec-gap, …) are skipped so the
- * newest PLAN family is found even when a newer non-plan artefact exists.
+ * Newest file per envelope family, families newest first (by that file's
+ * mtime). [] when the folder is missing. Within a family the highest version
+ * wins, then the collision sub, then mtime. A version-less file is its own
+ * single-file family. Shared by the recap and the startup Continue offer.
  */
-function buildPlanPart(workspaceDir) {
+export function newestPlanFamilyFiles(workspaceDir) {
     let names;
     try {
         names = readdirSync(workspaceDir);
     }
     catch {
-        return null; // workspace missing — nothing to recap
+        return []; // workspace missing
     }
     const re = /^(.+)-v(\d+)(?:-(\d+))?\.cspeach\.json$/;
     const cands = [];
@@ -124,17 +117,10 @@ function buildPlanPart(workspaceDir) {
             continue;
         }
         const m = re.exec(name);
-        if (m) {
-            cands.push({ path, family: m[1], version: Number(m[2]), sub: m[3] ? Number(m[3]) : 0, mtimeMs });
-        }
-        else {
-            // Hand-renamed / version-less envelope: its own single-file family.
-            cands.push({ path, family: name.slice(0, -'.cspeach.json'.length), version: 0, sub: 0, mtimeMs });
-        }
+        cands.push(m
+            ? { path, family: m[1], version: Number(m[2]), sub: m[3] ? Number(m[3]) : 0, mtimeMs }
+            : { path, family: name.slice(0, -'.cspeach.json'.length), version: 0, sub: 0, mtimeMs });
     }
-    if (cands.length === 0)
-        return null;
-    // Newest version per family (version, then collision sub, then mtime).
     const byFamily = new Map();
     for (const c of cands) {
         const cur = byFamily.get(c.family);
@@ -144,9 +130,13 @@ function buildPlanPart(workspaceDir) {
             byFamily.set(c.family, c);
         }
     }
-    // Families newest-first by their representative file's mtime; the first
-    // family whose envelope IS a plan is "the newest plan family".
-    const familiesNewestFirst = [...byFamily.values()].sort((a, b) => b.mtimeMs - a.mtimeMs);
+    return [...byFamily.values()].sort((a, b) => b.mtimeMs - a.mtimeMs);
+}
+function buildPlanPart(workspaceDir) {
+    // The first family whose envelope IS a plan is "the newest plan family".
+    const familiesNewestFirst = newestPlanFamilyFiles(workspaceDir);
+    if (familiesNewestFirst.length === 0)
+        return null;
     for (const cand of familiesNewestFirst) {
         let env;
         try {