@reticlehq/test 2.4.0 → 2.5.0

package/dist/flow-spec.js

@@ -60,7 +60,7 @@ function errorSpec(name, code) {
  * ERROR spec; enumeration of its siblings is never aborted.
  */
 export async function flowsAsSpecs(source, opts) {
-    const store = typeof source === 'string'
+    const store = 'string' === typeof source
         ? new FlowStore(opts?.fs ?? createNodeFileSystem(), source, opts?.clock ?? { now: () => 0 })
         : source;
     const names = await store.list();

package/dist/index.d.ts

@@ -17,6 +17,6 @@ export type { SpecContext, SpecFn, ReticleSpec, SpecResult, RunSummary, ContextF
 export { flowToSpec, flowsAsSpecs } from './flow-spec.js';
 export type { FlowSpec, FlowSpecOptions, FlowsAsSpecsOptions, SpecRunResult } from './flow-spec.js';
 export { assertSuccess, successToPredicate } from './success-assert.js';
-export { registerFlowSpecs, reticleFlowsAsSpecs, FlowMalformedError, SpecFailure, } from './register.js';
+export { registerFlowSpecs, FlowMalformedError, SpecFailure } from './register.js';
 export type { RegisterFn, RegisterFlowSpecsOptions } from './register.js';
 export { SpecKind, SpecOutcome, SpecMessage, FLOW_LOAD_ERROR_PREFIX } from './constants.js';

package/dist/index.js

@@ -12,5 +12,5 @@ export { TestStatus, STATUS_GLYPH, SUMMARY_FOOTER_PREFIX, JUnit, DEFAULT_JUNIT_S
 // flows under .reticle/flows become the runnable suite.
 export { flowToSpec, flowsAsSpecs } from './flow-spec.js';
 export { assertSuccess, successToPredicate } from './success-assert.js';
-export { registerFlowSpecs, reticleFlowsAsSpecs, FlowMalformedError, SpecFailure, } from './register.js';
+export { registerFlowSpecs, FlowMalformedError, SpecFailure } from './register.js';
 export { SpecKind, SpecOutcome, SpecMessage, FLOW_LOAD_ERROR_PREFIX } from './constants.js';

package/dist/input-mode.js

@@ -3,11 +3,11 @@ import { InputMode } from '@reticlehq/core';
 import { ReticleSkip } from './skip.js';
 import { SKIP_REASON_REAL_INPUT } from './constants.js';
 function isRecord(value) {
-    return typeof value === 'object' && value !== null;
+    return 'object' === typeof value && value !== null;
 }
 /** The act envelope's inputMode field, narrowed from the raw tool result. */
 export function readInputMode(actResult) {
-    if (typeof actResult !== 'object' || actResult === null)
+    if (typeof actResult !== 'object' || null === actResult)
         return undefined;
     const mode = actResult['inputMode'];
     if (mode === InputMode.REAL || mode === InputMode.SYNTHETIC)
@@ -56,7 +56,7 @@ async function probeInputMode(invoke, sessionId) {
         if (!isRecord(session))
             continue;
         if (sessionId === undefined || session['sessionId'] === sessionId) {
-            return session['realInputAvailable'] === true ? InputMode.REAL : InputMode.SYNTHETIC;
+            return true === session['realInputAvailable'] ? InputMode.REAL : InputMode.SYNTHETIC;
         }
     }
     return InputMode.SYNTHETIC;

package/dist/junit.js

@@ -1,19 +1,36 @@
 import { DEFAULT_JUNIT_SUITE_NAME, JUnit, TestStatus } from './constants.js';
 import { summarize } from './summary.js';
 /**
- * Characters illegal in XML 1.0 per section 2.2: U+0000-U+0008, U+000B, U+000C, U+000E-U+001F.
- * Tab (U+0009), newline (U+000A), and carriage return (U+000D) are the only control chars allowed.
+ * XML 1.0 section 2.2: every code point below U+0020 is illegal EXCEPT tab, newline and carriage
+ * return. Stated as codes rather than a regex character class, which cannot express control
+ * characters without tripping `no-control-regex` and needing the rule turned off to read it.
+ *
+ * Iterating code points also means an astral character (an emoji in a test name) survives intact;
+ * the regex worked on UTF-16 units.
  */
-// eslint-disable-next-line no-control-regex -- intentional: these are the exact chars to strip
-const XML_ILLEGAL_CHARS = /[\x00-\x08\x0B\x0C\x0E-\x1F]/g;
+const XML_MIN_LEGAL_CODE = 0x20;
+const XML_LEGAL_CONTROL_CODES = new Set([
+    0x09, // tab
+    0x0a, // newline
+    0x0d, // carriage return
+]);
+/** Drop the control characters no XML parser will accept. */
+function stripXmlIllegal(value) {
+    let out = '';
+    for (const ch of value) {
+        const code = ch.codePointAt(0) ?? 0;
+        if (code >= XML_MIN_LEGAL_CODE || XML_LEGAL_CONTROL_CODES.has(code))
+            out += ch;
+    }
+    return out;
+}
 /**
  * Strip characters illegal in XML 1.0 then escape the five XML-significant characters. Without the
  * strip, ANSI colour codes in error output produce a document no parser will read — CI shows
  * nothing instead of showing the failure.
  */
 function escapeXml(value) {
-    return value
-        .replace(XML_ILLEGAL_CHARS, '')
+    return stripXmlIllegal(value)
         .replace(/&/g, '&')
         .replace(/</g, '&lt;')
         .replace(/>/g, '&gt;')
@@ -23,7 +40,7 @@ function escapeXml(value) {
 /** First line of text (for the attribute summary). Handles LF, CRLF, and bare CR. */
 function firstLine(text) {
     const match = /\r?\n|\r/.exec(text);
-    return match === null ? text : text.slice(0, match.index);
+    return null === match ? text : text.slice(0, match.index);
 }
 function seconds(ms) {
     return (ms / 1000).toFixed(3);

package/dist/matchers.js

@@ -2,12 +2,12 @@ import { ReticleTool } from '@reticlehq/server';
 import { ReticleAssertionError } from './skip.js';
 import { CONSOLE_LEVEL_ERROR, DEFAULT_ASSERT_TIMEOUT_MS, PredicateKind } from './constants.js';
 function asVerdict(value) {
-    if (typeof value !== 'object' || value === null) {
+    if (typeof value !== 'object' || null === value) {
         return { pass: false, failureReason: 'assert returned a non-object result' };
     }
     const record = value;
-    const pass = record['pass'] === true;
-    const failureReason = typeof record['failureReason'] === 'string' ? record['failureReason'] : undefined;
+    const pass = true === record['pass'];
+    const failureReason = 'string' === typeof record['failureReason'] ? record['failureReason'] : undefined;
     return {
         pass,
         evidence: record['evidence'],

package/dist/register.d.ts

@@ -26,5 +26,3 @@ export interface RegisterFlowSpecsOptions extends FlowsAsSpecsOptions {
  * vitest collects the registered cases.
  */
 export declare function registerFlowSpecs(source: Parameters<typeof flowsAsSpecs>[0], getSession: () => Promise<FlowReplaySession> | FlowReplaySession, opts?: RegisterFlowSpecsOptions): Promise<void>;
-/** Public alias matching the design's stated entrypoint name. */
-export declare const reticleFlowsAsSpecs: typeof registerFlowSpecs;

package/dist/register.js

@@ -24,7 +24,7 @@ function renderEvidence(result) {
         .filter((s) => s.drift !== undefined)
         .map((s) => {
         const nearest = s.drift?.nearest;
-        const fix = nearest === null || nearest === undefined ? '' : ` (nearest: ${nearest})`;
+        const fix = null === nearest || nearest === undefined ? '' : ` (nearest: ${nearest})`;
         return `  step ${String(s.step)}: ${s.drift?.reason ?? 'drift'}${fix}`;
     });
     const successLine = result.successResult.pass || result.successResult.failureReason === undefined
@@ -68,5 +68,3 @@ export async function registerFlowSpecs(source, getSession, opts) {
         register(spec.name, specToCase(spec, getSession));
     }
 }
-/** Public alias matching the design's stated entrypoint name. */
-export const reticleFlowsAsSpecs = registerFlowSpecs;

package/dist/resolve.js

@@ -3,13 +3,13 @@ import { QueryBy } from '@reticlehq/core';
 import { ReticleQueryEmptyError } from './skip.js';
 import { NO_ELEMENT_FOR_TESTID } from './constants.js';
 function asQueryEnvelope(value) {
-    if (typeof value !== 'object' || value === null)
+    if (typeof value !== 'object' || null === value)
         return {};
     const record = value;
     const elements = Array.isArray(record['elements'])
         ? record['elements']
         : undefined;
-    const hint = typeof record['hint'] === 'object' && record['hint'] !== null
+    const hint = 'object' === typeof record['hint'] && record['hint'] !== null
         ? record['hint']
         : undefined;
     return {
@@ -30,7 +30,7 @@ export async function resolveTestid(invoke, testid, sessionId) {
     };
     const result = asQueryEnvelope(await invoke(ReticleTool.QUERY, args));
     const first = result.elements?.[0]?.ref;
-    if (typeof first === 'string' && first.length > 0)
+    if ('string' === typeof first && first.length > 0)
         return first;
     const present = result.hint?.presentTestids;
     const detail = present !== undefined ? { evidence: { presentTestids: present } } : undefined;

package/dist/summary.js

@@ -11,7 +11,12 @@ export function summarize(results) {
         else
             skipped += 1;
     }
-    return { total: results.length, passed, failed, skipped, ok: failed === 0 };
+    // `ok` is NOT merely "nothing failed". A suite whose every spec skipped — no browser, no real
+    // input, no session — has verified nothing, and reporting that as ok is the false green this
+    // product exists to prevent, in its own CI gate. An EMPTY suite stays ok: having recorded no
+    // flows yet is a different statement from having recorded flows and run none of them.
+    const total = results.length;
+    return { total, passed, failed, skipped, ok: 0 === failed && (0 === total || passed > 0) };
 }
 function resultLine(r) {
     const head = `${STATUS_GLYPH[r.status]} ${r.name} (${String(r.durationMs)}ms)`;

package/dist/test-context.js

@@ -7,15 +7,15 @@ import { failFromVerdict } from './matchers.js';
 import { expectAbsent, expectElement, expectNet, expectNoConsoleErrors, expectSignal, expectText, } from './matchers.js';
 import { DEFAULT_ASSERT_TIMEOUT_MS } from './constants.js';
 function asVerdict(value) {
-    const record = typeof value === 'object' && value !== null ? value : {};
+    const record = 'object' === typeof value && value !== null ? value : {};
     const verdict = record['verdict'];
-    if (typeof verdict !== 'object' || verdict === null) {
+    if (typeof verdict !== 'object' || null === verdict) {
         return { pass: false, failureReason: 'act_and_wait returned no verdict' };
     }
     const v = verdict;
-    const failureReason = typeof v['failureReason'] === 'string' ? v['failureReason'] : undefined;
+    const failureReason = 'string' === typeof v['failureReason'] ? v['failureReason'] : undefined;
     return {
-        pass: v['pass'] === true,
+        pass: true === v['pass'],
         evidence: v['evidence'],
         ...(failureReason !== undefined ? { failureReason } : {}),
     };

package/dist/types.d.ts

@@ -26,6 +26,13 @@ export interface RunSummary {
     passed: number;
     failed: number;
     skipped: number;
+    /**
+     * The one field a CI gate should branch on: nothing failed AND something actually ran.
+     *
+     * Deliberately not `failed === 0` — a suite where every spec skipped has verified nothing, and
+     * calling that a pass is exactly the false green this product exists to catch. An empty suite is
+     * still ok, because recording no flows yet is not the same as running none of the ones you have.
+     */
     ok: boolean;
 }
 /** Builds the per-spec `t` from the invoker — supplied by MATCHERS; the runner only calls it. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reticlehq/test",
-  "version": "2.4.0",
+  "version": "2.5.0",
   "description": "Reticle test runner: reticleTest spec registry + a runner that drives Reticle tools without MCP/stdio.",
   "type": "module",
   "main": "./dist/index.js",
@@ -12,8 +12,8 @@
     }
   },
   "dependencies": {
-    "@reticlehq/server": "2.4.0",
-    "@reticlehq/core": "2.4.0"
+    "@reticlehq/core": "2.5.0",
+    "@reticlehq/server": "2.5.0"
   },
   "peerDependencies": {
     "vitest": "^3.2.6"
@@ -52,6 +52,7 @@
     "build": "tsc -b",
     "typecheck": "tsc -b",
     "lint": "eslint src",
-    "test:unit": "vitest run src --passWithNoTests"
+    "test:unit": "vitest run src --passWithNoTests",
+    "test:coverage": "vitest run src --passWithNoTests --coverage.enabled --coverage.provider=v8 --coverage.reporter=text-summary --coverage.include='src/**/*.ts' --coverage.exclude='src/**/*.test.ts'"
   }
 }