@sensigo/realm-cli 0.39.0 → 0.40.0

package/dist/index.js

@@ -4,7 +4,7 @@ import 'dotenv/config';
 import { Command } from 'commander';
 import { workflowCommands, runCommands, topLevelCommands } from './commands-registry.js';
 const program = new Command();
-program.name('realm').description('Realm workflow engine CLI').version('0.39.0');
+program.name('realm').description('Realm workflow engine CLI').version('0.40.0');
 // realm workflow — operations on workflow definitions
 const workflowCmd = new Command('workflow').description('Manage workflow definitions');
 for (const cmd of workflowCommands)

package/dist/lib/loader-warnings.d.ts

@@ -1,5 +1,13 @@
 import { type LoaderWarning, type WarningCode } from '@sensigo/realm';
-/** Prints every warning via the single renderLoaderWarning format source. */
+/**
+ * Prints every warning via the single renderLoaderWarning format source, correcting the
+ * "— ignored" claim for any warning this boundary will actually refuse.
+ *
+ * Resolved against DEFAULT_POLICY, which is the policy the boundary-reject itself uses. A warning
+ * that only fails under `--strict` still renders "— ignored" — see the report's design note; the
+ * `--strict` path prints its own "failing due to --strict" line, so the author is not left
+ * without an explanation.
+ */
 export declare function printLoaderWarnings(warnings: LoaderWarning[]): void;
 /**
  * The dormant issue #170 boundary-reject hook, factored ONCE so validate/register/watch can't
@@ -12,4 +20,17 @@ export declare function printLoaderWarnings(warnings: LoaderWarning[]): void;
 export declare function rejectOnErrorSeverity(warnings: LoaderWarning[], policy?: Record<WarningCode, 'warn' | 'error'>): boolean;
 /** True if `--strict` should fail the command given these accumulated warnings. */
 export declare function failsStrict(warnings: LoaderWarning[]): boolean;
+/**
+ * Renders a load failure without saying "invalid" twice (issue #417).
+ *
+ * The loader's own message already begins `Invalid workflow: …`, so wrapping it in `Invalid: `
+ * produced `Invalid: Invalid workflow: Step 'x': …` — an author's first three words are the same
+ * word twice, before anything they can act on. Where the message announces itself, it is printed
+ * verbatim; where it does not (a non-loader error surfacing on the same path), the prefix still
+ * earns its place.
+ *
+ * Applies only to the message-echoing renders. The standalone strict-escalation lines have their
+ * own text and are untouched.
+ */
+export declare function renderLoadFailure(message: string): string;
 //# sourceMappingURL=loader-warnings.d.ts.map

package/dist/lib/loader-warnings.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"loader-warnings.d.ts","sourceRoot":"","sources":["../../src/lib/loader-warnings.ts"],"names":[],"mappings":"AAIA,OAAO,EAIL,KAAK,aAAa,EAClB,KAAK,WAAW,EACjB,MAAM,gBAAgB,CAAC;AAExB,6EAA6E;AAC7E,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,aAAa,EAAE,GAAG,IAAI,CAInE;AAED;;;;;;;GAOG;AACH,wBAAgB,qBAAqB,CACnC,QAAQ,EAAE,aAAa,EAAE,EACzB,MAAM,GAAE,MAAM,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAkB,GAC7D,OAAO,CAET;AAWD,mFAAmF;AACnF,wBAAgB,WAAW,CAAC,QAAQ,EAAE,aAAa,EAAE,GAAG,OAAO,CAE9D"}
+{"version":3,"file":"loader-warnings.d.ts","sourceRoot":"","sources":["../../src/lib/loader-warnings.ts"],"names":[],"mappings":"AAIA,OAAO,EAIL,KAAK,aAAa,EAClB,KAAK,WAAW,EACjB,MAAM,gBAAgB,CAAC;AAiBxB;;;;;;;;GAQG;AACH,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,aAAa,EAAE,GAAG,IAAI,CASnE;AAED;;;;;;;GAOG;AACH,wBAAgB,qBAAqB,CACnC,QAAQ,EAAE,aAAa,EAAE,EACzB,MAAM,GAAE,MAAM,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAkB,GAC7D,OAAO,CAET;AAWD,mFAAmF;AACnF,wBAAgB,WAAW,CAAC,QAAQ,EAAE,aAAa,EAAE,GAAG,OAAO,CAE9D;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CAEzD"}

package/dist/lib/loader-warnings.js

@@ -3,10 +3,35 @@
 // only caller of renderLoaderWarning — no command hand-rolls a `⚠ ` string), and the dormant
 // #170 boundary-reject is factored into ONE shared helper so validate/register/watch can't drift.
 import { renderLoaderWarning, resolveSeverity, DEFAULT_POLICY, } from '@sensigo/realm';
-/** Prints every warning via the single renderLoaderWarning format source. */
+/**
+ * The unknown-key warning is minted with "— ignored", which is true of the LENIENT loader
+ * (run/agent/listen really do ignore the key and carry on) and false here: the only callers of
+ * this function are validate/register/watch, and post-#170 those three REFUSE a workflow whose
+ * warning resolves to 'error'. Printing "— ignored" one line above "escalated to an error" tells
+ * the author the opposite of what is about to happen to them.
+ *
+ * Substituted at the RENDER, not the mint, precisely because the mint is shared with the lenient
+ * path — where the original wording is true and worth keeping. The did-you-mean fragment is
+ * untouched: it is the most useful half of the line and the reason the author can fix this in one
+ * edit.
+ */
+const IGNORED_CLAUSE = '— ignored';
+const REFUSED_CLAUSE = '— REFUSED below';
+/**
+ * Prints every warning via the single renderLoaderWarning format source, correcting the
+ * "— ignored" claim for any warning this boundary will actually refuse.
+ *
+ * Resolved against DEFAULT_POLICY, which is the policy the boundary-reject itself uses. A warning
+ * that only fails under `--strict` still renders "— ignored" — see the report's design note; the
+ * `--strict` path prints its own "failing due to --strict" line, so the author is not left
+ * without an explanation.
+ */
 export function printLoaderWarnings(warnings) {
     for (const w of warnings) {
-        console.warn(renderLoaderWarning(w));
+        const line = renderLoaderWarning(w);
+        console.warn(resolveSeverity(w.code, DEFAULT_POLICY) === 'error'
+            ? line.replace(IGNORED_CLAUSE, REFUSED_CLAUSE)
+            : line);
     }
 }
 /**
@@ -30,4 +55,19 @@ const ALL_ERROR_POLICY = Object.fromEntries(Object.keys(DEFAULT_POLICY).map((cod
 export function failsStrict(warnings) {
     return rejectOnErrorSeverity(warnings, ALL_ERROR_POLICY);
 }
+/**
+ * Renders a load failure without saying "invalid" twice (issue #417).
+ *
+ * The loader's own message already begins `Invalid workflow: …`, so wrapping it in `Invalid: `
+ * produced `Invalid: Invalid workflow: Step 'x': …` — an author's first three words are the same
+ * word twice, before anything they can act on. Where the message announces itself, it is printed
+ * verbatim; where it does not (a non-loader error surfacing on the same path), the prefix still
+ * earns its place.
+ *
+ * Applies only to the message-echoing renders. The standalone strict-escalation lines have their
+ * own text and are untouched.
+ */
+export function renderLoadFailure(message) {
+    return message.startsWith('Invalid workflow:') ? message : `Invalid: ${message}`;
+}
 //# sourceMappingURL=loader-warnings.js.map

package/dist/lib/loader-warnings.js.map

@@ -1 +1 @@
-{"version":3,"file":"loader-warnings.js","sourceRoot":"","sources":["../../src/lib/loader-warnings.ts"],"names":[],"mappings":"AAAA,6FAA6F;AAC7F,gGAAgG;AAChG,6FAA6F;AAC7F,kGAAkG;AAClG,OAAO,EACL,mBAAmB,EACnB,eAAe,EACf,cAAc,GAGf,MAAM,gBAAgB,CAAC;AAExB,6EAA6E;AAC7E,MAAM,UAAU,mBAAmB,CAAC,QAAyB;IAC3D,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;QACzB,OAAO,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC,CAAC;IACvC,CAAC;AACH,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,qBAAqB,CACnC,QAAyB,EACzB,SAAgD,cAAc;IAE9D,OAAO,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,eAAe,CAAC,CAAC,CAAC,IAAI,EAAE,MAAM,CAAC,KAAK,OAAO,CAAC,CAAC;AAC3E,CAAC;AAED;;;;GAIG;AACH,MAAM,gBAAgB,GAA0C,MAAM,CAAC,WAAW,CAC/E,MAAM,CAAC,IAAI,CAAC,cAAc,CAAmB,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC,CACrC,CAAC;AAE3C,mFAAmF;AACnF,MAAM,UAAU,WAAW,CAAC,QAAyB;IACnD,OAAO,qBAAqB,CAAC,QAAQ,EAAE,gBAAgB,CAAC,CAAC;AAC3D,CAAC"}
+{"version":3,"file":"loader-warnings.js","sourceRoot":"","sources":["../../src/lib/loader-warnings.ts"],"names":[],"mappings":"AAAA,6FAA6F;AAC7F,gGAAgG;AAChG,6FAA6F;AAC7F,kGAAkG;AAClG,OAAO,EACL,mBAAmB,EACnB,eAAe,EACf,cAAc,GAGf,MAAM,gBAAgB,CAAC;AAExB;;;;;;;;;;;GAWG;AACH,MAAM,cAAc,GAAG,WAAW,CAAC;AACnC,MAAM,cAAc,GAAG,iBAAiB,CAAC;AAEzC;;;;;;;;GAQG;AACH,MAAM,UAAU,mBAAmB,CAAC,QAAyB;IAC3D,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;QACzB,MAAM,IAAI,GAAG,mBAAmB,CAAC,CAAC,CAAC,CAAC;QACpC,OAAO,CAAC,IAAI,CACV,eAAe,CAAC,CAAC,CAAC,IAAI,EAAE,cAAc,CAAC,KAAK,OAAO;YACjD,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,cAAc,EAAE,cAAc,CAAC;YAC9C,CAAC,CAAC,IAAI,CACT,CAAC;IACJ,CAAC;AACH,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,qBAAqB,CACnC,QAAyB,EACzB,SAAgD,cAAc;IAE9D,OAAO,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,eAAe,CAAC,CAAC,CAAC,IAAI,EAAE,MAAM,CAAC,KAAK,OAAO,CAAC,CAAC;AAC3E,CAAC;AAED;;;;GAIG;AACH,MAAM,gBAAgB,GAA0C,MAAM,CAAC,WAAW,CAC/E,MAAM,CAAC,IAAI,CAAC,cAAc,CAAmB,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC,CACrC,CAAC;AAE3C,mFAAmF;AACnF,MAAM,UAAU,WAAW,CAAC,QAAyB;IACnD,OAAO,qBAAqB,CAAC,QAAQ,EAAE,gBAAgB,CAAC,CAAC;AAC3D,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,iBAAiB,CAAC,OAAe;IAC/C,OAAO,OAAO,CAAC,UAAU,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,YAAY,OAAO,EAAE,CAAC;AACnF,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sensigo/realm-cli",
-  "version": "0.39.0",
+  "version": "0.40.0",
   "publishConfig": {
     "access": "public"
   },
@@ -55,6 +55,7 @@
     }
   },
   "devDependencies": {
+    "@anthropic-ai/sdk": "^0.120.0",
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^26.1.1",
     "openai": "^6.34.0",
@@ -63,9 +64,9 @@
     "vitest": "^4.1.0"
   },
   "dependencies": {
-    "@sensigo/realm": "^0.39.0",
-    "@sensigo/realm-mcp": "^0.39.0",
-    "@sensigo/realm-testing": "^0.39.0",
+    "@sensigo/realm": "^0.40.0",
+    "@sensigo/realm-mcp": "^0.40.0",
+    "@sensigo/realm-testing": "^0.40.0",
     "@modelcontextprotocol/sdk": "1.30.0",
     "chalk": "^5.0.0",
     "commander": "^15.0.0",