@stravica-ai/rcf-schemas 0.6.0 → 0.6.1

package/CHANGELOG.md

@@ -4,6 +4,31 @@ All notable changes to `@stravica-ai/rcf-schemas` are documented in this file.
 
 The format roughly follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and the project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). Pre-1.0 breaking changes are signalled by a minor bump per semver 0.x convention.
 
+## 0.6.1 - 2026-09-05
+
+Patch bump for the T-0 visual-round Playwright-first ship-gate seam (`projects/blueprint-library/specs/visual-round-spec-2026-09-04.md`, section 3.4). Adds an additive, optional `probePacks[]` carrier on `browserVerificationRecord` so the record shape that rcf-lite's `packages/rcf-lite/src/browser-verify/pack-runner.js` already emits (per pack `packName`, `packVersion`, `blueprintSlug`, `applicable`, `checks[]`, optional `preChecks[]`, optional pack-level `detail` string) can persist without tripping `additionalProperties: false`.
+
+Patch semver convention: URLs pinned at `v0.6.0`. No breaking change; every 0.6.0-valid manifest keeps validating.
+
+### Added
+
+- **`manifest.schema.json`**: new optional `probePacks` array on `$defs.browserVerificationRecord`. Items are the new `$defs.browserVerificationProbePack` def: required `packName` (min-length-1 string), `packVersion` (min-length-1 string), `blueprintSlug` (min-length-1 string), `applicable` (boolean), `checks` (array, empty allowed when `applicable` is false); optional `detail` (string, carries the appliesTo skip reason for a not-applicable pack), and optional `preChecks` (array of the same check-item shape). `additionalProperties: false` on the pack item.
+- **`manifest.schema.json`**: new `$defs.browserVerificationProbePackCheck` def used for both `checks[]` and `preChecks[]`. Required `id` (min-length-1 string, equals an AC id contributed by the same blueprint for browser checks per pack-loader), `verdict` (enum `pass | warn | fail | skipped`, where the `skipped` token is emitted by the pack-runner when a browser check's `dependsOn` pre-check failed), `severity` (enum `block | warn | advisory`); optional `detail` (string). `additionalProperties: false` on the check item.
+- **Fixtures (valid)**: `fixtures/valid/manifest/manifest-014-browser-verify-probe-packs.json` (two packs, one `applicable: false` with a skip detail and an empty `checks`, one `applicable: true` with a `preChecks` entry and three checks covering `pass`, `fail` with detail, and `skipped` with a `skipped-by-pre-check:<id>` detail).
+- **Fixtures (invalid)**: `fixtures/invalid/manifest/manifest-023-browser-verify-probe-pack-unknown-field.json` (a check item with an unknown field, refused by `additionalProperties: false`).
+- **Tests**: `test/manifest.test.js` gains 7 cases (two-pack positive shape, back-compat absent-probePacks, unknown-field-on-check rejection, `skipped` verdict acceptance, unknown-verdict rejection, unknown-severity rejection, missing-required-`checks` rejection). Total suite grows from 323 to 330.
+
+### Migration
+
+- **Existing chains on 0.6.0**: bump `@stravica-ai/rcf-schemas` to `0.6.1`. Every existing document keeps validating; `probePacks` is optional and unknown to pre-T-0 callers.
+- **rcf-lite consumer bump**: `rcf-lite` pins `@stravica-ai/rcf-schemas` exactly. Its own release bumps the pin (rides the admin-console train or a separate one-line PR); this repo does not touch rcf-lite.
+
+### Notes
+
+- Traces to work item `w-2026-09-05-dave-001` and the T-4 integration gate (`d-2026-09-05-001`) that found `rcf verify browser --probe-pack <name>` composing a `probePacks[]` record and exiting `3` with `validation /browserVerification/0 must NOT have additional properties` before this schema could accept it. Every earlier T-N gate ran with `--dry-run` and did not exercise the persist path.
+- Sub-defs sit alongside the existing `browserVerificationInvariantCheck` and `browserVerificationAuthSmokeCheck` peers. Reuses the file's per-check-item pattern.
+- Aggregate-verdict semantics live in `packages/rcf-lite/src/browser-verify/manifest-writer.js:aggregateVerdict` per spec section 3.4; the schema does not encode the cross-field rule.
+
 ## 0.6.0 - 2026-09-04
 
 Minor bump for commit track S1 of the ratified eval-node spec (Baz ratified 2026-09-04, "eval-node - agree spec is right shape, schema route is correct"). Adds a new `eval.schema.json` document type as the graded-output peer of a Test Suite, adds an optional `determinism` marker on the inline acceptance criterion, and adds an optional `tags` array on the User Story so a consumer authoring rule (rcf-lite blueprint-authoring checklist s6) becomes satisfiable. Every change is additive at the document layer: every 0.5.1-valid document remains valid.

package/fixtures/invalid/manifest/manifest-023-browser-verify-probe-pack-unknown-field.json

@@ -0,0 +1,35 @@
+{
+  "version": "2.0.0",
+  "projectName": "invalid: probePacks check with unknown field",
+  "prd": { "id": "PRD-001", "path": "rcf/prd.json" },
+  "tad": { "id": "TAD-001", "path": "rcf/tad.json" },
+  "bs":  { "id": "BS-001",  "path": "rcf/build-sequence.json" },
+  "browserVerification": [
+    {
+      "id": "bv-FBS-040-1",
+      "fbsId": "FBS-040",
+      "createdAt": "2026-09-05T09:20:00Z",
+      "mode": "agentScreenshotCritique",
+      "runtimeProfile": "local-dev",
+      "runtimeUrl": "http://127.0.0.1:3000",
+      "routesChecked": [
+        { "path": "/", "screenshotPath": ".rcf/artefacts/x.png", "themeApplied": "light" }
+      ],
+      "invariantChecks": [
+        { "invariant": "sharedNavPresent", "verdict": "pass" }
+      ],
+      "probePacks": [
+        {
+          "packName": "application-datatable-grid-shell",
+          "packVersion": "1.0.0",
+          "blueprintSlug": "application-datatable",
+          "applicable": true,
+          "checks": [
+            { "id": "AC-17101-1", "verdict": "pass", "severity": "block", "unexpectedField": "x" }
+          ]
+        }
+      ],
+      "verdict": "pass"
+    }
+  ]
+}

package/fixtures/valid/manifest/manifest-014-browser-verify-probe-packs.json

@@ -0,0 +1,48 @@
+{
+  "version": "2.0.0",
+  "projectName": "visual-round T-0 probe-pack fixture",
+  "prd": { "id": "PRD-001", "path": "rcf/prd.json" },
+  "tad": { "id": "TAD-001", "path": "rcf/tad.json" },
+  "bs":  { "id": "BS-001",  "path": "rcf/build-sequence.json" },
+  "browserVerification": [
+    {
+      "id": "bv-FBS-040-1",
+      "fbsId": "FBS-040",
+      "createdAt": "2026-09-05T09:00:00Z",
+      "mode": "agentScreenshotCritique",
+      "runtimeProfile": "local-dev",
+      "runtimeUrl": "http://127.0.0.1:3000",
+      "routesChecked": [
+        { "path": "/", "screenshotPath": ".rcf/artefacts/bv-FBS-040-1/root-light.png", "themeApplied": "light" }
+      ],
+      "invariantChecks": [
+        { "invariant": "sharedNavPresent", "verdict": "pass" }
+      ],
+      "probePacks": [
+        {
+          "packName": "application-dashboard-tile-states",
+          "packVersion": "1.0.0",
+          "blueprintSlug": "application-dashboard",
+          "applicable": false,
+          "detail": "appliesTo returned false for FBS-040",
+          "checks": []
+        },
+        {
+          "packName": "application-datatable-grid-shell",
+          "packVersion": "1.0.0",
+          "blueprintSlug": "application-datatable",
+          "applicable": true,
+          "preChecks": [
+            { "id": "AC-17101-0", "verdict": "pass", "severity": "warn" }
+          ],
+          "checks": [
+            { "id": "AC-17101-1", "verdict": "pass", "severity": "block" },
+            { "id": "AC-17101-2", "verdict": "fail", "severity": "block", "detail": "row order after sort did not match server order" },
+            { "id": "AC-17101-3", "verdict": "skipped", "severity": "warn", "detail": "skipped-by-pre-check:AC-17101-0" }
+          ]
+        }
+      ],
+      "verdict": "block"
+    }
+  ]
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stravica-ai/rcf-schemas",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "type": "module",
   "description": "JSON Schemas for the Requirements Confidence Framework (RCF). Language-neutral. The contract every RCF tool keys to.",
   "license": "Apache-2.0",

package/schemas/manifest.schema.json

@@ -451,9 +451,75 @@
           "type": "string",
           "minLength": 1,
           "description": "Operator's reason for shipping despite a block verdict. Written by --accept-block on --mark complete."
+        },
+        "probePacks": {
+          "type": "array",
+          "items": { "$ref": "#/$defs/browserVerificationProbePack" },
+          "description": "Playwright-first probe-pack results per the visual-round spec (2026-09-04) section 3.4. Additive and optional: pre-T-0 callers omit the field; a T-0 run emits one entry per pack composed by packages/rcf-lite/src/browser-verify/pack-runner.js. The aggregate verdict extends per spec section 3.4 (block on any severity:block pack check failing, warn on any severity:warn pack check failing)."
         }
       }
     },
+    "browserVerificationProbePack": {
+      "type": "object",
+      "additionalProperties": false,
+      "required": ["packName", "packVersion", "blueprintSlug", "applicable", "checks"],
+      "properties": {
+        "packName": {
+          "type": "string",
+          "minLength": 1,
+          "description": "Pack identifier, kebab-slug. Prefixed by the blueprint slug per pack-schema.js."
+        },
+        "packVersion": {
+          "type": "string",
+          "minLength": 1,
+          "description": "Pack version at emission time, copied from the pack's declared version field."
+        },
+        "blueprintSlug": {
+          "type": "string",
+          "minLength": 1,
+          "description": "Slug of the blueprint that contributes the pack."
+        },
+        "applicable": {
+          "type": "boolean",
+          "description": "Whether the pack's appliesTo predicate returned true for this FBS. When false, checks[] is emitted empty and a detail string explains the skip."
+        },
+        "detail": {
+          "type": "string",
+          "description": "Free-text explanation attached to the pack record, typically the appliesTo skip reason or an error message from the pack-runner."
+        },
+        "checks": {
+          "type": "array",
+          "items": { "$ref": "#/$defs/browserVerificationProbePackCheck" },
+          "description": "One entry per pack check; empty when applicable is false."
+        },
+        "preChecks": {
+          "type": "array",
+          "items": { "$ref": "#/$defs/browserVerificationProbePackCheck" },
+          "description": "Optional fast-fail source-scan pre-checks per spec section 3.5. Any pre-check that fails may skip browser checks that declare dependsOn on it, recorded as verdict:skipped on the browser check."
+        }
+      }
+    },
+    "browserVerificationProbePackCheck": {
+      "type": "object",
+      "additionalProperties": false,
+      "required": ["id", "verdict", "severity"],
+      "properties": {
+        "id": {
+          "type": "string",
+          "minLength": 1,
+          "description": "Check id. For a browser check the id equals an AC id contributed by the same blueprint (loader cross-checks this at pack load time)."
+        },
+        "verdict": {
+          "enum": ["pass", "warn", "fail", "skipped"],
+          "description": "Check verdict. skipped is emitted by the pack-runner when a browser check's dependsOn pre-check failed."
+        },
+        "severity": {
+          "enum": ["block", "warn", "advisory"],
+          "description": "Check severity, copied from the pack declaration. Feeds the aggregate verdict rule per spec section 3.4."
+        },
+        "detail": { "type": "string" }
+      }
+    },
     "browserVerificationRoute": {
       "type": "object",
       "additionalProperties": false,