@cspeach/cli 1.1.11 → 1.1.13

package/dist/agent/loop.js

@@ -905,19 +905,31 @@ export async function runTurn(params) {
             // carry the manifest. See ./turn-assistant-text.ts for details.
             turnStatusEmitter.activity('finishing up');
             const assistantText = collectTurnAssistantText(session.messages, turnStartMessageCount);
-            if (!params.suppressSaveHook) {
-                await maybeOfferSave({
-                    skill: params.skill,
-                    assistantText,
-                    userMessage: userMessageForSave,
-                    tokensUsed: Math.max(0, (session.usage?.output_tokens ?? 0) - turnStartOutputTokens),
-                    // A4 — record the actual turn model if an override ran the stream
-                    // (unreachable today: plan-resume turns suppress this hook, and
-                    // they are the only modelOverride caller — kept honest anyway).
-                    model: params.modelOverride ?? resolveModelRole('session_default', cfg) ?? session.model ?? 'unknown',
-                    emit,
-                    promotedFrom: promotedFromForSave,
-                });
+            // 2026-09-17 — the save hook below asks the user things (save y/N,
+            // blocker pickers). Those are not tool dispatches, so nothing paused
+            // the status row: it kept showing "finishing up" with a running timer
+            // while the user was the one being waited on (Laeeq's dry run). Pause
+            // for the whole hook — the row hides while paused — and resume after,
+            // so any real work that follows (auto-compact) shows again.
+            turnStatusEmitter.pause();
+            try {
+                if (!params.suppressSaveHook) {
+                    await maybeOfferSave({
+                        skill: params.skill,
+                        assistantText,
+                        userMessage: userMessageForSave,
+                        tokensUsed: Math.max(0, (session.usage?.output_tokens ?? 0) - turnStartOutputTokens),
+                        // A4 — record the actual turn model if an override ran the stream
+                        // (unreachable today: plan-resume turns suppress this hook, and
+                        // they are the only modelOverride caller — kept honest anyway).
+                        model: params.modelOverride ?? resolveModelRole('session_default', cfg) ?? session.model ?? 'unknown',
+                        emit,
+                        promotedFrom: promotedFromForSave,
+                    });
+                }
+            }
+            finally {
+                turnStatusEmitter.resume();
             }
             // Phase 2 #9 (2026-05-16) — auto-compact end-of-turn hook.
             // Runs only on the success path (after maybeOfferSave) so an interrupted

package/dist/agent/providers/license-gate.js

@@ -9,7 +9,7 @@ export class CspeachLicenseError extends Error {
         super(`CSPeach requires a license to run in '${mode}' mode.\n` +
             `In this mode your prompts go straight to your own LLM — but the CSPeach skills\n` +
             `and CLI are licensed software. Run \`cspeach login\` to activate your license.\n` +
-            `Need an access key? Email laeeq.siddique@cremencing.com.`);
+            `Need an access key? Email info@cremencing.com.`);
         this.name = 'CspeachLicenseError';
         this.mode = mode;
     }

package/dist/cost/display-mode.js

@@ -21,9 +21,13 @@
 export function costDisplayMode(cfg, balanceInfo) {
     if (cfg.llm?.mode !== 'managed')
         return 'usd';
-    if (balanceInfo?.metered !== true)
-        return 'usd';
     if (cfg.show_cogs === true)
         return 'usd';
+    // 2026-09-17 — a managed account NEVER falls back to dollars. Before, a
+    // balance lookup that failed or took over 2.5s at start (slow network on a
+    // customer laptop) dropped the display to USD — our cost on their screen,
+    // exactly what credits mode exists to prevent. Unknown or unmetered
+    // balances render "credits: updating…" until the next good sample.
+    void balanceInfo;
     return 'credits';
 }

package/dist/one-shot.js

@@ -17,7 +17,7 @@ import { setRenderingOverride, setHeadlessOverride } from './renderer/tty.js';
 import { alwaysDeclinePreviewHook } from './repl/update-method-preview-hook.js';
 import { createClassicOutputEmitter } from './renderer/notices.js';
 import { todoEmitter } from './ui/todo-emitter.js';
-import { applyLocalBuildConfig } from './tools/local-build.js';
+import { applyLocalBuildConfig, applyStandaloneFileTools } from './tools/local-build.js';
 import { getEffectiveWriteMode } from './repl/mode-cycle.js';
 import { initCredits } from './cost/credits-wire.js';
 // Side-effect imports for tool registration.
@@ -92,6 +92,10 @@ export async function runOneShot(prompt, opts) {
         const announce = applyLocalBuildConfig(resolveLocalBuild(cfg), process.cwd());
         if (announce)
             console.log(chalk.dim(announce));
+        // 2026-09-17 — standalone writes files by contract; enable the file tools
+        // without the separate local_build step (see repl.tsx for the live case).
+        else if (standalone)
+            console.log(chalk.dim(applyStandaloneFileTools(process.cwd())));
     }
     const store = await getStore();
     if (!(await store.get())) {

package/dist/projects/answer-blockers.js

@@ -87,8 +87,9 @@ export async function offerAnswerBlockers(args) {
     const answers = new Map();
     for (let i = 0; i < blockers.length; i++) {
         const b = blockers[i];
-        args.log('');
-        args.log(`Blocker ${i + 1}/${blockers.length}: ${b.text}`);
+        // 2026-09-17 — the "Blocker i/n: …" line is printed only on the free-text
+        // path below. When a picker handles the question it shows the same text
+        // itself, and printing it here too showed every blocker twice.
         // Choices first, where the question has them and a picker is available.
         // 'write' falls through to the text box; 'skip' leaves the blocker open.
         let askText = true;
@@ -111,6 +112,8 @@ export async function offerAnswerBlockers(args) {
             // c.kind === 'write' → askText stays true
         }
         if (askText) {
+            args.log('');
+            args.log(`Blocker ${i + 1}/${blockers.length}: ${b.text}`);
             const a = (await args.prompt('  Your answer (Enter to skip): ')).trim();
             if (a.length > 0)
                 answers.set(b.id, a);

package/dist/projects/extract-design.js

@@ -1,4 +1,8 @@
-const TITLE_RE = /^## ABAP Design — (.+)$/m;
+// 2026-09-17 — tolerant on heading level and dash: the skill prescribes
+// "## ABAP Design — …" but a live standalone run wrote "# ABAP Design — …"
+// and the whole design was silently lost (no file, so /abap-rap --from had
+// nothing to start from). H1..H3 and em/en/plain dash all count.
+const TITLE_RE = /^#{1,3} ABAP Design [—–-] (.+)$/m;
 const OBJECTS_HEADING_RE = /^### Objects to Create\s*$/m;
 const DEPENDENCY_HEADING_RE = /^### Dependency Sequence\s*$/m;
 const NEXT_HEADING_RE = /^###\s/m;
@@ -14,25 +18,49 @@ function sliceSection(markdown, headingRe) {
 function parseObjectsTable(section) {
     const lines = section.split(/\r?\n/);
     const rows = [];
+    // 2026-09-17 — header-aware, tolerant of a missing 5th column. The skill's
+    // template is "# | Name | Type | Purpose | Transport", but a standalone
+    // run (no transports) wrote four columns and every row was skipped by the
+    // old "at least 5 cells" rule — "Objects to Create table is empty", design
+    // lost. Columns are located by header name when a header row exists;
+    // otherwise the positional default applies. Package/transport is optional.
+    let nameIdx = 1;
+    let typeIdx = 2;
+    let pkgIdx = 4;
+    let headerSeen = false;
     for (const raw of lines) {
         const line = raw.trim();
         if (!line.startsWith('|'))
             continue;
-        if (/^\|\s*-+\s*\|/.test(line))
-            continue;
-        if (/^\|\s*#\s*\|/.test(line))
+        if (/^\|\s*:?-+:?\s*\|/.test(line))
             continue;
         const cells = line.split('|').slice(1, -1).map((c) => c.trim());
-        if (cells.length < 5)
+        if (!headerSeen && /^#$|^no\.?$|^nr\.?$/i.test(cells[0] ?? '')) {
+            headerSeen = true;
+            const find = (re) => cells.findIndex((c) => re.test(c));
+            const n = find(/^(object\s+)?name$/i);
+            const t = find(/^(object\s+)?type$/i);
+            const p = find(/^(package|transport|package\s*\/\s*transport|transport\s*\/\s*package)$/i);
+            if (n >= 0)
+                nameIdx = n;
+            if (t >= 0)
+                typeIdx = t;
+            pkgIdx = p >= 0 ? p : null;
+            continue;
+        }
+        if (cells.length < 3)
             continue;
         const numStr = cells[0];
         if (!/^\d+$/.test(numStr))
             continue;
+        const name = cells[nameIdx] ?? '';
+        if (!/^[A-Za-z0-9_/]+$/.test(name))
+            continue; // a continuation row, not an object
         rows.push({
             num: Number(numStr),
-            name: cells[1],
-            type: cells[2],
-            package: cells[4],
+            name,
+            type: cells[typeIdx] ?? '',
+            package: pkgIdx !== null ? (cells[pkgIdx] ?? '') : '',
         });
     }
     return rows;

package/dist/projects/extract-estimate.js

@@ -1,4 +1,6 @@
-const TITLE_RE = /^## Effort Estimate — (.+)$/m;
+// 2026-09-17 — tolerant on heading level (H1..H3) and dash, same reason as
+// extract-design.ts.
+const TITLE_RE = /^#{1,3} Effort Estimate [—–-] (.+)$/m;
 const COMPONENTS_HEADING_RE = /^### Component Breakdown\s*$/m;
 const NEXT_HEADING_RE = /^###\s/m;
 function sliceSection(markdown, headingRe) {

package/dist/projects/extract-spec-gap.js

@@ -1,4 +1,6 @@
-const TITLE_RE = /^## Spec Gap Analysis — (.+)$/m;
+// 2026-09-17 — tolerant on heading level (H1..H3) and dash, same reason as
+// extract-design.ts: a title written one level off must not lose the file.
+const TITLE_RE = /^#{1,3} Spec Gap Analysis [—–-] (.+)$/m;
 const DOMAIN_RE = /^\*\*Inferred Domain:\*\*\s*(.+?)(?:\s*\[INFERRED.*?\])?$/m;
 const INPUT_TYPE_RE = /^\*\*Input Type:\*\*\s*(.+)$/m;
 const SOURCE_DOC_RE = /^\*\*Source Document:\*\*\s*(.+)$/m;

package/dist/projects/save-command.js

@@ -292,6 +292,16 @@ export async function runSaveCommand(args) {
             args.log(`[save] ${e instanceof Error ? e.message : String(e)}`);
             args.log(`[save] The output above is otherwise complete — re-run the skill or report this as a manifest bug.`);
         }
+        else {
+            // 2026-09-17 — marker-less skills (design, estimate, spec-gap) used to
+            // fail here in silence: no file, no line, and the next --from step had
+            // nothing to start from (seen live: a design titled "# ABAP Design"
+            // instead of "## ABAP Design"). Say what happened and what to do.
+            args.log('');
+            args.log(`[save] ${args.skillName}: could not read the output as a ${spec.artefactType} — nothing was saved.`);
+            args.log(`[save] ${e instanceof Error ? e.message : String(e)}`);
+            args.log(`[save] Ask CSPeach to "re-issue the ${spec.artefactType} in the standard structure" and it will be saved then.`);
+        }
         return null;
     }
     // C1 review fix — name the revision target BEFORE the confirm so the user

package/dist/repl.js

@@ -95,7 +95,7 @@ import { maybeExpandNumberSelection, lastAssistantText } from './repl/numbered-m
 import { attachEarlyLineBuffer, noopEarlyLineBuffer } from './repl/early-line-buffer.js';
 import { snapshotTurnStart, pushPostTurnStatus } from './repl/post-turn-status.js';
 import { looksLikeCspeachShellCommand, CSPEACH_SHELL_NUDGE } from './repl/cspeach-shell-detect.js';
-import { applyLocalBuildConfig } from './tools/local-build.js';
+import { applyLocalBuildConfig, applyStandaloneFileTools } from './tools/local-build.js';
 // Side-effect imports so tool registrations happen before listTools() is called.
 import './tools/sap-read.js';
 import './tools/sap-write.js';
@@ -464,6 +464,11 @@ export async function runRepl(opts) {
         const announce = applyLocalBuildConfig(resolveLocalBuild(cfg), process.cwd());
         if (announce)
             console.log(chalk.dim(announce + '\n'));
+        // 2026-09-17 — standalone writes its outputs to files by contract, so the
+        // file tools must be on without a separate local_build step (the RAP
+        // stack landed in the transcript with "There are no files" otherwise).
+        else if (standalone)
+            console.log(chalk.dim(applyStandaloneFileTools(process.cwd()) + '\n'));
     }
     const aliases = Object.keys(cfg.sap);
     // C.1 — --sap-alias flag override skips the interactive picker

package/dist/sap/onboarding.js

@@ -71,7 +71,7 @@ export async function runAddSapWizard() {
         default: 'basic',
     });
     const username = await input({
-        message: 'SAP username (e.g. S.LAEEQ):',
+        message: 'SAP username (e.g. JSMITH):',
         validate: (v) => v.length > 0 || 'Required',
     });
     let certPath;