@cspeach/cli 1.1.38 → 1.1.39

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.
@@ -394,8 +394,29 @@ registerTool({
394
394
  handler: async (args, ctx) => {
395
395
  // atcRun(type, name, variant?) — type first
396
396
  // Review fix round 1, M-3: Esc / Ctrl+C stops a long ATC run.
397
- const result = await ctx.adt.atcRun(args.type, args.name, args.variant ?? 'DEFAULT', ctx.signal ? { signal: ctx.signal } : {});
398
- return { content: JSON.stringify(result, null, 2) };
397
+ const variant = args.variant ?? 'DEFAULT';
398
+ const result = await ctx.adt.atcRun(args.type, args.name, variant, ctx.signal ? { signal: ctx.signal } : {});
399
+ // Fable review 2026-10-15, I-1: a finished run whose worklist never named
400
+ // the object did not check it (the variant has no check for that type).
401
+ // Say so plainly; "0 findings" there is not a measurement. Every other
402
+ // result is the same JSON as before (the coverage key is not added).
403
+ const { coverage, ...shown } = result;
404
+ if (coverage === 'absent' && result.totalFindings === 0) {
405
+ const objectName = `${String(args.type).toUpperCase()} ${String(args.name).toUpperCase()}`;
406
+ return {
407
+ content: [
408
+ `ATC did not check ${objectName}.`,
409
+ 'SAP finished the run, but its worklist never listed this object.',
410
+ 'This is not a measurement. Do not report it as 0 findings or as clean.',
411
+ 'Likely causes:',
412
+ `- the ${String(variant).toUpperCase()} check variant may have no check for this object type (seen on S4H for DDLS, DDLX, SRVD, DOMA);`,
413
+ '- the name may be wrong, or the object may not exist or not be active.',
414
+ 'Use sap_search_object to confirm the object exists.',
415
+ `(ATC worklist ${result.worklistId || 'unknown'})`,
416
+ ].join('\n'),
417
+ };
418
+ }
419
+ return { content: JSON.stringify(shown, null, 2) };
399
420
  },
400
421
  });
401
422
  // -----------------------------------------------------------------------
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@cspeach/cli",
3
- "version": "1.1.38",
3
+ "version": "1.1.39",
4
4
  "description": "CSPeach — AI-assisted ABAP development CLI. Direct SAP access, signed skills, safety gates.",
5
5
  "keywords": [
6
6
  "abap",
@@ -54,7 +54,7 @@
54
54
  "@anthropic-ai/sdk": "^0.90.0",
55
55
  "@azure/msal-node": "6.0.1",
56
56
  "@cspeach/register-core": "0.4.0",
57
- "@cspeach/sap-client": "0.1.23",
57
+ "@cspeach/sap-client": "0.1.24",
58
58
  "@iarna/toml": "^2.2.5",
59
59
  "@inkjs/ui": "^2.0.0",
60
60
  "@inquirer/prompts": "^7.0.0",