@cspeach/cli 1.1.12 → 1.1.13

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/dist/one-shot.js CHANGED
@@ -17,7 +17,7 @@ import { setRenderingOverride, setHeadlessOverride } from './renderer/tty.js';
17
17
  import { alwaysDeclinePreviewHook } from './repl/update-method-preview-hook.js';
18
18
  import { createClassicOutputEmitter } from './renderer/notices.js';
19
19
  import { todoEmitter } from './ui/todo-emitter.js';
20
- import { applyLocalBuildConfig } from './tools/local-build.js';
20
+ import { applyLocalBuildConfig, applyStandaloneFileTools } from './tools/local-build.js';
21
21
  import { getEffectiveWriteMode } from './repl/mode-cycle.js';
22
22
  import { initCredits } from './cost/credits-wire.js';
23
23
  // Side-effect imports for tool registration.
@@ -92,6 +92,10 @@ export async function runOneShot(prompt, opts) {
92
92
  const announce = applyLocalBuildConfig(resolveLocalBuild(cfg), process.cwd());
93
93
  if (announce)
94
94
  console.log(chalk.dim(announce));
95
+ // 2026-09-17 — standalone writes files by contract; enable the file tools
96
+ // without the separate local_build step (see repl.tsx for the live case).
97
+ else if (standalone)
98
+ console.log(chalk.dim(applyStandaloneFileTools(process.cwd())));
95
99
  }
96
100
  const store = await getStore();
97
101
  if (!(await store.get())) {
@@ -1,4 +1,8 @@
1
- const TITLE_RE = /^## ABAP Design — (.+)$/m;
1
+ // 2026-09-17 — tolerant on heading level and dash: the skill prescribes
2
+ // "## ABAP Design — …" but a live standalone run wrote "# ABAP Design — …"
3
+ // and the whole design was silently lost (no file, so /abap-rap --from had
4
+ // nothing to start from). H1..H3 and em/en/plain dash all count.
5
+ const TITLE_RE = /^#{1,3} ABAP Design [—–-] (.+)$/m;
2
6
  const OBJECTS_HEADING_RE = /^### Objects to Create\s*$/m;
3
7
  const DEPENDENCY_HEADING_RE = /^### Dependency Sequence\s*$/m;
4
8
  const NEXT_HEADING_RE = /^###\s/m;
@@ -14,25 +18,49 @@ function sliceSection(markdown, headingRe) {
14
18
  function parseObjectsTable(section) {
15
19
  const lines = section.split(/\r?\n/);
16
20
  const rows = [];
21
+ // 2026-09-17 — header-aware, tolerant of a missing 5th column. The skill's
22
+ // template is "# | Name | Type | Purpose | Transport", but a standalone
23
+ // run (no transports) wrote four columns and every row was skipped by the
24
+ // old "at least 5 cells" rule — "Objects to Create table is empty", design
25
+ // lost. Columns are located by header name when a header row exists;
26
+ // otherwise the positional default applies. Package/transport is optional.
27
+ let nameIdx = 1;
28
+ let typeIdx = 2;
29
+ let pkgIdx = 4;
30
+ let headerSeen = false;
17
31
  for (const raw of lines) {
18
32
  const line = raw.trim();
19
33
  if (!line.startsWith('|'))
20
34
  continue;
21
- if (/^\|\s*-+\s*\|/.test(line))
22
- continue;
23
- if (/^\|\s*#\s*\|/.test(line))
35
+ if (/^\|\s*:?-+:?\s*\|/.test(line))
24
36
  continue;
25
37
  const cells = line.split('|').slice(1, -1).map((c) => c.trim());
26
- if (cells.length < 5)
38
+ if (!headerSeen && /^#$|^no\.?$|^nr\.?$/i.test(cells[0] ?? '')) {
39
+ headerSeen = true;
40
+ const find = (re) => cells.findIndex((c) => re.test(c));
41
+ const n = find(/^(object\s+)?name$/i);
42
+ const t = find(/^(object\s+)?type$/i);
43
+ const p = find(/^(package|transport|package\s*\/\s*transport|transport\s*\/\s*package)$/i);
44
+ if (n >= 0)
45
+ nameIdx = n;
46
+ if (t >= 0)
47
+ typeIdx = t;
48
+ pkgIdx = p >= 0 ? p : null;
49
+ continue;
50
+ }
51
+ if (cells.length < 3)
27
52
  continue;
28
53
  const numStr = cells[0];
29
54
  if (!/^\d+$/.test(numStr))
30
55
  continue;
56
+ const name = cells[nameIdx] ?? '';
57
+ if (!/^[A-Za-z0-9_/]+$/.test(name))
58
+ continue; // a continuation row, not an object
31
59
  rows.push({
32
60
  num: Number(numStr),
33
- name: cells[1],
34
- type: cells[2],
35
- package: cells[4],
61
+ name,
62
+ type: cells[typeIdx] ?? '',
63
+ package: pkgIdx !== null ? (cells[pkgIdx] ?? '') : '',
36
64
  });
37
65
  }
38
66
  return rows;
@@ -1,4 +1,6 @@
1
- const TITLE_RE = /^## Effort Estimate — (.+)$/m;
1
+ // 2026-09-17 — tolerant on heading level (H1..H3) and dash, same reason as
2
+ // extract-design.ts.
3
+ const TITLE_RE = /^#{1,3} Effort Estimate [—–-] (.+)$/m;
2
4
  const COMPONENTS_HEADING_RE = /^### Component Breakdown\s*$/m;
3
5
  const NEXT_HEADING_RE = /^###\s/m;
4
6
  function sliceSection(markdown, headingRe) {
@@ -1,4 +1,6 @@
1
- const TITLE_RE = /^## Spec Gap Analysis — (.+)$/m;
1
+ // 2026-09-17 — tolerant on heading level (H1..H3) and dash, same reason as
2
+ // extract-design.ts: a title written one level off must not lose the file.
3
+ const TITLE_RE = /^#{1,3} Spec Gap Analysis [—–-] (.+)$/m;
2
4
  const DOMAIN_RE = /^\*\*Inferred Domain:\*\*\s*(.+?)(?:\s*\[INFERRED.*?\])?$/m;
3
5
  const INPUT_TYPE_RE = /^\*\*Input Type:\*\*\s*(.+)$/m;
4
6
  const SOURCE_DOC_RE = /^\*\*Source Document:\*\*\s*(.+)$/m;
@@ -292,6 +292,16 @@ export async function runSaveCommand(args) {
292
292
  args.log(`[save] ${e instanceof Error ? e.message : String(e)}`);
293
293
  args.log(`[save] The output above is otherwise complete — re-run the skill or report this as a manifest bug.`);
294
294
  }
295
+ else {
296
+ // 2026-09-17 — marker-less skills (design, estimate, spec-gap) used to
297
+ // fail here in silence: no file, no line, and the next --from step had
298
+ // nothing to start from (seen live: a design titled "# ABAP Design"
299
+ // instead of "## ABAP Design"). Say what happened and what to do.
300
+ args.log('');
301
+ args.log(`[save] ${args.skillName}: could not read the output as a ${spec.artefactType} — nothing was saved.`);
302
+ args.log(`[save] ${e instanceof Error ? e.message : String(e)}`);
303
+ args.log(`[save] Ask CSPeach to "re-issue the ${spec.artefactType} in the standard structure" and it will be saved then.`);
304
+ }
295
305
  return null;
296
306
  }
297
307
  // C1 review fix — name the revision target BEFORE the confirm so the user
package/dist/repl.js CHANGED
@@ -95,7 +95,7 @@ import { maybeExpandNumberSelection, lastAssistantText } from './repl/numbered-m
95
95
  import { attachEarlyLineBuffer, noopEarlyLineBuffer } from './repl/early-line-buffer.js';
96
96
  import { snapshotTurnStart, pushPostTurnStatus } from './repl/post-turn-status.js';
97
97
  import { looksLikeCspeachShellCommand, CSPEACH_SHELL_NUDGE } from './repl/cspeach-shell-detect.js';
98
- import { applyLocalBuildConfig } from './tools/local-build.js';
98
+ import { applyLocalBuildConfig, applyStandaloneFileTools } from './tools/local-build.js';
99
99
  // Side-effect imports so tool registrations happen before listTools() is called.
100
100
  import './tools/sap-read.js';
101
101
  import './tools/sap-write.js';
@@ -464,6 +464,11 @@ export async function runRepl(opts) {
464
464
  const announce = applyLocalBuildConfig(resolveLocalBuild(cfg), process.cwd());
465
465
  if (announce)
466
466
  console.log(chalk.dim(announce + '\n'));
467
+ // 2026-09-17 — standalone writes its outputs to files by contract, so the
468
+ // file tools must be on without a separate local_build step (the RAP
469
+ // stack landed in the transcript with "There are no files" otherwise).
470
+ else if (standalone)
471
+ console.log(chalk.dim(applyStandaloneFileTools(process.cwd()) + '\n'));
467
472
  }
468
473
  const aliases = Object.keys(cfg.sap);
469
474
  // C.1 — --sap-alias flag override skips the interactive picker
@@ -1,6 +1,6 @@
1
1
  // GENERATED FILE — bundled skills for local mode.
2
2
  // Source: https://manifest.cspeach.dev/v1.json
3
- // Generated: 2026-09-17T20:19:57.515Z
3
+ // Generated: 2026-09-17T21:24:04.334Z
4
4
  // Manifest signature verified at build time.
5
5
  export const BUNDLED_SKILLS = {
6
6
  "abap-atc-fix": {
@@ -76,6 +76,27 @@ export const LOCAL_BUILD_TOOLS = [
76
76
  * which directory tree read/write/edit operate under; the shell safelist is
77
77
  * surfaced so the user sees which commands shell_exec may run.
78
78
  */
79
+ /**
80
+ * Standalone mode (2026-09-17). The standalone note tells the model to write
81
+ * every generated object plus GUIDE.md under ./cspeach-out — but the file
82
+ * tools were only switched on by `local_build`, a separate manual step. On
83
+ * Laeeq's other-laptop run the whole RAP stack ended up in the transcript
84
+ * with "There are no files". Standalone therefore enables the FILE tools
85
+ * itself. Only the file tools: no shell_exec, no Fiori/agent tools — those
86
+ * stay behind the explicit local_build consent.
87
+ */
88
+ export const STANDALONE_FILE_TOOLS = [
89
+ 'file_read',
90
+ 'read_document',
91
+ 'file_write',
92
+ 'file_edit',
93
+ 'glob',
94
+ 'grep',
95
+ ];
96
+ export function applyStandaloneFileTools(cwd) {
97
+ enableToolFlags([...STANDALONE_FILE_TOOLS]);
98
+ return `standalone: generated objects and GUIDE.md are written under ${cwd}\\cspeach-out`;
99
+ }
79
100
  export function applyLocalBuildConfig(localBuild, cwd) {
80
101
  if (localBuild !== true)
81
102
  return undefined;
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@cspeach/cli",
3
- "version": "1.1.12",
3
+ "version": "1.1.13",
4
4
  "description": "CSPeach — AI-assisted ABAP development CLI. Direct SAP access, 34 skills, safety gates.",
5
5
  "keywords": [
6
6
  "abap",