@onlineapps/conn-orch-validator 12.1.0 → 12.1.1

package/CHANGELOG.md

@@ -4,6 +4,88 @@ All notable changes to this package. Follows [Keep a Changelog](https://keepacha
 
 ## [Unreleased]
 
+## [12.1.1] — 2026-09-25
+
+### Fixed — `verify-deploy-uniform.sh`: rada při checkoutu mimo `api_biz/` vede na cestu, kterou šablona skutečně deklaruje (d.899)
+
+Hláška radila `GIT_CLONE_PATH: $CI_BUILDS_DIR/oa-uniform/api_biz/<service>` bez `$CI_CONCURRENT_ID`,
+zatímco `.oa-uniform` v šabloně `.gitlab-ci.yml` deklaruje `$CI_BUILDS_DIR/oa-uniform/$CI_CONCURRENT_ID/api_biz/<service>`.
+Nově radí „make this job extend .oa-uniform" a cituje cestu šablony; `biz-deploy-uniform-gate.bats` čte
+očekávanou cestu ze šablony, takže se hláška a šablona nemohou rozejít.
+
+### Fixed — `verify-deploy-uniform.sh` smaže jen klon, který sám vytvořil; cizí `api/` odmítne a nechá netknutý (d.885)
+
+Šablonový skript brány nasazení odvozuje `API_CHECKOUT="<workspace>/api"` a existující adresář před
+klonem maže, aby runner, který znovu používá svůj builds adresář, neměřil proti zastaralému SSOT. Za
+„svůj" ale uznal každý adresář s `.git` a `config/services.json`, a ty nese KAŽDÝ checkout api
+repozitáře. 2026-09-24 skript spustil vývojář z `api_biz/emailer` na pracovní stanici,
+`<workspace>/api` byl skutečný pracovní strom a skript ho smazal (639 sledovaných souborů; obnoveno).
+
+Skript teď po vlastním `git clone` zapíše do klonu značku `.verify-deploy-uniform.clone`
+(`created-by verify-deploy-uniform.sh <ISO čas> <pid>`). Mazat smí jen adresář, který nese `.git`,
+`config/services.json` A tu značku. Bez značky končí `die` (exit 2): „exists and carries no marker …
+Expected: a clone this script made. Fix: remove it yourself, or run this script only from a CI
+checkout where <workspace>/api is free" a na adresář nesáhne. Přepínač, který by kontrolu vypnul,
+neexistuje (`automation-gates.md` §1 požadavky 3 a 5). URL `file://` se nezakazuje: ochranu nese
+značka a druhé pravidlo pro tutéž věc by bylo druhou kolejí.
+
+RED (`api/tests/scripts/biz-deploy-uniform-gate.bats`, tři nové případy): `not ok 1 an api checkout
+the gate did not make is refused and left exactly as it was` (`[ "$status" -eq 2 ]` failed: skript
+cizí checkout smazal a naklonoval znovu), `not ok 2` a `not ok 3` (značka neexistuje). GREEN: celý
+soubor `1..20`, 20× `ok`, včetně případů se skutečným enginem, které čtou klon se značkou.
+
+### Fixed — konfigurační řádek se severitou `deploy` už neshodí boot služby; start blokuje jen `boot` (d.870)
+
+Krok 2 boot validace (`validateConfig`, řádky `C-SERVICE`, `C-OPS`, `C-CONTRACT`) soudil start podle
+`blocking_severities` uniformy. Ten seznam je ale **verdikt nasaditelnosti**: `runManifest` z něj počítá
+`ok`, tištěné jako DEPLOYABLE / NOT DEPLOYABLE, a pro službu zní `["boot", "deploy"]`. Krok 7 přitom
+soudil start jen podle `boot`. Konfigurační řádek se severitou `deploy` by tedy odmítl start služby, kterou
+konfirmace `biz-service-manifest` 001 §4 nechává běžet („`deploy` — the service **starts**") a jen ji označí
+za nenasaditelnou. Dnes to nikdo neviděl, protože všechny tři řádky kroku 2 mají `boot`. Chyba by se ukázala
+v den, kdy jednomu z nich manifest dá `deploy`.
+
+Otázku „blokuje tento nález start?" teď oba kroky kladou jedné funkci `blocksBoot` nad konstantou
+`BOOT_SEVERITY` (`src/manifest/manifestShape.js`, vedle `SEVERITIES`). `blocking_severities` zůstává tím,
+čím je: seznamem verdiktu uniformy. Tvar výsledku kroků (`valid`, `errors`, `findings`, `deployable`,
+`deployFindings`) se nemění.
+
+RED (`tests/unit/ValidationOrchestrator.manifestStep.test.js`): nový případ „a config row of severity deploy
+does not refuse the boot" `Expected: true / Received: false` na `step2.valid`, `1 failed, 15 passed`. Kontrolní
+případ (týž řádek s `boot` → `valid: false` a přesně jedna chybová zpráva) byl zelený před opravou i po ní.
+
+### Documentation — čísla kroků a citace zrušené hlášky v docblocích (d.870)
+
+- Docblock `runCookbookTests()` říkal „Step 4", ale běh ho loguje jako `Step 5/7`; docblock
+  `validateOperations()` říkal „Step 3" (tedy číslo kroku prostředí), ale běh ho loguje jako `Step 4/7`.
+- Odstavec o zrušeném kroku Readiness (a týž komentář v `tests/unit/ValidationOrchestrator.unit.test.js`)
+  citoval jako přítomnou hlášku kroku 2 `operations.json has no operations defined`, kterou žádný kód nevydává.
+  Teď cituje řádek uniformy `C-OPS`, který tu otázku zodpovídá. Odkaz na řádek nezestárne jako opsaný text.
+
+### Fixed — R6 porovná se SSOT KAŽDÝ pin `@onlineapps/*`, i u knihoven, které infra neinstaluje (d.654, d.654b)
+
+Zúžení na `infraConsumed` přeskakovalo i porovnání ROVNOSTI pinu se SSOT, ne jen otázku
+kompatibility s infra vydáním. Pravidlo R6 přitom žádnou infra klauzuli nemá: služba pinuje
+přesně to, co SSOT deklaruje (`.claude/rules/architecture-principles.md` § Version pinning).
+`notGated` proto **už není výjimka z porovnání, jen informace** o tom, kterou VĚTU nález
+dostane — infra balíček „biz pins X, infra ships Y", balíček bez infra konzumenta „service
+<jméno> pins X, the platform library SSOT declares Y … Fix: npm install <pkg>@Y --save-exact".
+Zelený řádek navíc říká, kolik pinů se doopravdy porovnalo, aby se coverage brány nedala
+odhadnout z názvu druhého seznamu.
+
+**Nahrazuje chování popsané u d.413** („`notGated` zůstává pro deklarovaný balíček, který
+infra neinstaluje" — tam byl `notGated` výjimka z porovnání; od d.654 už není). Položka
+vydané verze zůstává, jak byla: popisuje, co ta verze dělala.
+
+Změřeno nad `api_biz/ingest` (13 pinů `@onlineapps/*`, SSOT deklaruje 28 knihoven a
+`infraConsumed` 7) — PŘED (vydaný validátor 12.1.0): `not gated (no infra service installs
+these): …8 balíčků…` + `OK verify-lib-compat — 5 pin(s) match the platform library set`;
+PO: `no infra comparison (no infra service installs these; their SSOT pin is compared all
+the same): …8 balíčků…` + `OK verify-lib-compat — 13 pin(s) match the platform library
+SSOT, 5 of them also compared against the infra library set`. Falzifikace na kopii
+`package.json` s `@onlineapps/service-wrapper` 10.1.2 proti SSOT 10.1.1 (balíček, který
+žádná infra služba neinstaluje): 12.1.0 exit 0 „5 pin(s) match", nový validátor exit 1
+s jmenovaným nálezem a opravou. Nedotčená kopie dál exit 0.
+
 ## [12.1.0] — 2026-09-18
 
 ### Fixed — lint, který se odmítl spustit, je NOT RUN, ne šest nálezů o dokumentaci (d.632, W632)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/conn-orch-validator",
-  "version": "12.1.0",
+  "version": "12.1.1",
   "description": "Validation orchestrator for OA Drive microservices - coordinates validation across all layers (base, infra, orch, business)",
   "oa": {
     "category": "orchestration"

package/src/ValidationOrchestrator.js

@@ -24,6 +24,21 @@ const { loadManifest, DEFAULT_MANIFEST_PATH } = require('./manifest/loadManifest
 const { runManifest } = require('./manifest/runManifest');
 const { resolveWorkspaceRoot } = require('./manifest/workspaceRoot');
 const { renderBanner, describeFinding } = require('./manifest/report');
+const { BOOT_SEVERITY } = require('./manifest/manifestShape');
+
+/**
+ * Whether a finding refuses the service's start — the one question steps 2 and
+ * 7 both ask, so they ask it here (`.claude/rules/change-discipline.md` § One
+ * rail per concern). Until d.870 step 2 answered it with the uniform's
+ * `blocking_severities`, which is the deployability verdict and names `deploy`
+ * as well, while step 7 answered `boot` alone: a config row given `deploy`
+ * would have refused a start that confirmation `biz-service-manifest` 001 §4
+ * says goes ahead.
+ *
+ * @param {{severity: string}} finding
+ * @returns {boolean}
+ */
+const blocksBoot = (finding) => finding.severity === BOOT_SEVERITY;
 const { buildDeployabilitySignal, writeDeployabilitySignal } = require('./manifest/deployabilitySignal');
 const { isGitCheckout, NOT_A_CHECKOUT } = require('./manifest/gitCheckout');
 
@@ -392,8 +407,8 @@ class ValidationOrchestrator {
    *     (handler presence and shape, bundle_scope, input, output, the retired
    *     v2 fields) — the two loops were textual copies of one another.
    *   - Its only two extra guards, "operations must be an object" and "no
-   *     operations defined", are step 2's `operations.json has no operations
-   *     defined`.
+   *     operations defined", are answered by step 2 through the uniform row
+   *     `C-OPS`.
    *   - Run against a valid baseline plus 13 mutations covering every rule,
    *     its verdict equalled `step2 && step3` in all 14 cases. Against the
    *     eight live biz services it emitted one check, scored 80 out of a
@@ -651,9 +666,13 @@ class ValidationOrchestrator {
    * leaves them out, because they are answered, not because they stopped
    * mattering.
    *
-   * Which severity fails the boot is the uniform's own list, never a constant
-   * here — a service raises `boot`, a library `publish` (`api/shared/TODO.md`
-   * §0.2b-15).
+   * Only a `boot` finding fails the start (`blocksBoot`, the same question
+   * step 7 asks). The uniform's `blocking_severities` is a different list: the
+   * deployability verdict, `["boot", "deploy"]` for a service — a `deploy` row
+   * here is reported, and the service starts undeployable (confirmation
+   * `biz-service-manifest` 001 §4; d.870). This step only ever runs the service
+   * uniform (`manifestRun()` loads `DEFAULT_MANIFEST_PATH`), so no library
+   * severity reaches it.
    *
    * @returns {{valid: boolean, errors: string[], findings: object[]}}
    */
@@ -661,7 +680,7 @@ class ValidationOrchestrator {
     try {
       const run = this.manifestRun();
       const findings = run.findings.filter((finding) => CONFIG_STEP_ROWS.includes(finding.id));
-      const blocking = findings.filter((finding) => run.blockingSeverities.includes(finding.severity));
+      const blocking = findings.filter(blocksBoot);
 
       this.logger.info(`[ValidationOrchestrator] ✓ Config files: ${blocking.length === 0 ? 'PASS' : 'FAIL'}`);
       return {
@@ -701,7 +720,7 @@ class ValidationOrchestrator {
   }
 
   /**
-   * Step 3: Validate operations compliance (v3 — handler registry dispatch).
+   * Step 4: Validate operations compliance (v3 — handler registry dispatch).
    * Required per operation: handler ('handlers/<path>#<export>'), bundle_scope, input, output.
    * Forbidden (v2): endpoint, method, path.
    * Warned about: a missing `description` — inherited from the removed
@@ -767,7 +786,7 @@ class ValidationOrchestrator {
   }
 
   /**
-   * Step 4: Run cookbook tests via CookbookTestRunner (v3 handler dispatch).
+   * Step 5: Run cookbook tests via CookbookTestRunner (v3 handler dispatch).
    * Operations must declare a `handler` in operations.json; the runner
    * loads the module via `require()` and calls the exported function
    * in-process. Failures fail this validation step.
@@ -1003,7 +1022,7 @@ class ValidationOrchestrator {
       });
 
       const findings = result.findings.filter((finding) => !CONFIG_STEP_ROWS.includes(finding.id));
-      const bootFindings = findings.filter((finding) => finding.severity === 'boot');
+      const bootFindings = findings.filter(blocksBoot);
 
       return {
         valid: bootFindings.length === 0,

package/src/cli/biz-ci-gate.js

@@ -467,8 +467,13 @@ async function runVerifyLibCompat(options) {
   const librarySet = await loadLibrarySet(options.libraries);
   const result = checkLibCompat(pkg, librarySet);
 
+  // What is skipped for these is the INFRA comparison, and nothing else: since
+  // d.654 their pin is held against the SSOT like every other. The line used to
+  // read "not gated", which after d.654 told the reader the opposite of what
+  // the run did.
   if (result.notGated.length > 0) {
-    process.stdout.write(`[BizCiGate] not gated (no infra service installs these): ${result.notGated.join(', ')}\n`);
+    process.stdout.write('[BizCiGate] no infra comparison (no infra service installs these; their SSOT pin '
+      + `is compared all the same): ${result.notGated.join(', ')}\n`);
   }
   if (!result.ok) {
     for (const violation of result.violations) {
@@ -476,7 +481,14 @@ async function runVerifyLibCompat(options) {
     }
     process.exit(1);
   }
-  process.stdout.write(`[BizCiGate] OK verify-lib-compat — ${result.gated.length} pin(s) match the platform library set\n`);
+  // Two numbers, because the run answers two questions and a single count
+  // could only ever be true of one of them: how many pins were held against the
+  // SSOT (all of them, since d.654), and how many of those infra also installs,
+  // which is the compatibility half. Both come from the rule module's own
+  // lists, never counted again here (`.claude/rules/automation-gates.md` §5 —
+  // a coverage sentence smaller than the coverage is a false signal).
+  process.stdout.write(`[BizCiGate] OK verify-lib-compat — ${result.compared.length} pin(s) match the platform `
+    + `library SSOT, ${result.gated.length} of them also compared against the infra library set\n`);
 }
 
 async function runVerifyContract(options) {

package/src/manifest/manifestShape.js

@@ -21,16 +21,26 @@
 
 const { walkManifest, collectRows, isPlainObject } = require('./walk');
 
+/**
+ * The one severity that refuses a service's start (001 §4). Every other
+ * consequence leaves the service running: `deploy` makes it undeployable,
+ * `publish` belongs to a library, which has no start at all, and `warn` stops
+ * nothing. It is NOT the uniform's `blocking_severities` — that list is the
+ * VERDICT (DEPLOYABLE / PUBLISHABLE, `runManifest` turns it into `ok`), and for
+ * a service it names `deploy` too (d.870).
+ */
+const BOOT_SEVERITY = 'boot';
+
+/** The one severity that stops nothing: it is printed in the table and nowhere else. */
+const WARN_SEVERITY = 'warn';
+
 /**
  * The consequences a row may carry, and no others. `boot` and `deploy` are the
  * service uniform's (001 §4); `publish` is the library uniform's — a library has
  * no boot, and the choke point nothing reaches a service past is
  * `scripts/publish-library.sh` (002 §10).
  */
-const SEVERITIES = Object.freeze(['boot', 'deploy', 'publish', 'warn']);
-
-/** The one severity that stops nothing: it is printed in the table and nowhere else. */
-const WARN_SEVERITY = 'warn';
+const SEVERITIES = Object.freeze([BOOT_SEVERITY, 'deploy', 'publish', WARN_SEVERITY]);
 
 /** Which severities a uniform may declare as blocking — every consequence except `warn`. */
 const BLOCKING_CANDIDATES = Object.freeze(SEVERITIES.filter((severity) => severity !== WARN_SEVERITY));
@@ -435,6 +445,7 @@ module.exports = {
   rowNeedsWorkspace,
   describeFromProblem,
   SEVERITIES,
+  BOOT_SEVERITY,
   CHECK_SCOPES,
   WORKSPACE_DEPENDENT_SCOPES,
   BLOCKING_CANDIDATES,

package/src/utils/libCompat.js

@@ -7,26 +7,34 @@
  * matches the version the platform declares. Mismatch means the service was
  * built against a different platform contract than the one it will run on.
  *
- * Only packages at least one infra service actually installs are compared: the
- * SSOT lists biz-only packages too, and for those there is no infra version to
- * diverge from, so asking "does this match what infra ships" has no answer.
- * The SSOT carries that set as `infraConsumed`. A bare version map has no such
- * list — a platform-release entry records what was deployed, so everything in
- * it is by definition shipped and must match.
+ * EVERY package the SSOT declares and the service pins is compared against the
+ * declared version: R6's rule is that a service pins exactly what the SSOT
+ * declares (`.claude/rules/architecture-principles.md` § Version pinning), and
+ * that sentence carries no infra clause. What the `infraConsumed` list decides
+ * is which SENTENCE a stale pin gets, because the two say different things:
  *
- * That narrowing covers a package the SSOT DECLARES and infra does not install.
- * A package the SSOT declares in NEITHER list is a different thing: not
- * biz-only, but unknown to the platform. R6's rule is that a service pins
- * exactly what the SSOT declares (`.claude/rules/architecture-principles.md`
- * § Version pinning), and for an undeclared package there is no such version at
- * all — so it is refused, never narrowed away. Passing it would be a fallback
- * to "not gated" (principle 3), and it would let a typo or a retired package
- * name travel into an image with the gate reporting OK (measured 2026-09-14).
+ *   - infra installs the package  → biz and infra run different code against
+ *     one contract, and the fix may be to deploy the matching infra release;
+ *   - no infra service installs it → there is no infra version to diverge
+ *     from, so the only statement left is the SSOT one: this pin is not what
+ *     the platform declares, and the fix is to re-pin.
  *
- * The narrowing decides one thing only: which packages are COMPARED against an
- * infra version. It never relaxes the exact-pin rule, which `^`, `~` and
- * `latest` break in EVERY `@onlineapps/*` dependency, gated or not
- * (`.claude/rules/architecture-principles.md` § Version pinning) — a floating
+ * Until 2026-09-18 (W651) `infraConsumed` decided more than that: a package no
+ * infra service installs skipped the version comparison altogether. Measured
+ * then: 20 of the 28 SSOT packages are biz-only, so for most of a service's
+ * pins a biz pipeline compared nothing at all, and a service could ship months
+ * behind the platform with a green gate. A bare version map has no such list —
+ * a platform-release entry records what was deployed, so everything in it is by
+ * definition shipped, and every mismatch is a compatibility one.
+ *
+ * A package the SSOT declares in NEITHER list is a different thing again: not
+ * biz-only, but unknown to the platform — there is no version to pin against at
+ * all, so it is refused, never narrowed away. Passing it would be a fallback to
+ * "not gated" (principle 3), and it would let a typo or a retired package name
+ * travel into an image with the gate reporting OK (measured 2026-09-14).
+ *
+ * The narrowing never relaxes the exact-pin rule either, which `^`, `~` and
+ * `latest` break in EVERY `@onlineapps/*` dependency, gated or not — a floating
  * range makes the same commit install different code on different days whether
  * or not infra happens to ship that package too. Until 2026-09-15 (W413) the
  * exactness test sat inside the gated branch, so a biz-only pin could float
@@ -76,12 +84,39 @@ function normalizeLibrarySet(raw) {
   return { versions: raw, infraConsumed: null };
 }
 
+/**
+ * The service a finding belongs to, taken from its package.json.
+ *
+ * The SSOT-pin finding names the service, because the cross-repository audit
+ * path (`api/scripts/ci/verify-lib-compat.mjs`) reports several repositories in
+ * one run. A package.json without a name is invalid input — npm requires the
+ * field — so it is refused with an actionable message rather than rendered as
+ * "service undefined".
+ */
+function serviceNameOf(pkg) {
+  const name = pkg?.name;
+  if (typeof name !== 'string' || name.length === 0) {
+    throw new Error('[LibCompat] Service package.json declares no "name" - a pin finding must say which '
+      + 'service carries the pin. Fix: add a "name" field to the service package.json.');
+  }
+  return name;
+}
+
 /**
  * Compare a package.json against the platform library set.
  *
+ * `compared` is what the gate may claim as coverage: the packages whose
+ * version was actually held against the declared one. It is appended by the
+ * comparing loop itself, never recomputed from the other two lists by a
+ * renderer — a second derivation of one fact is the drift
+ * `.claude/rules/change-discipline.md` § One rail per concern forbids, and
+ * `gated ∪ notGated` is NOT the same set: a caret pin and a package the SSOT
+ * declares nowhere are classified and then refused before any comparison
+ * happens.
+ *
  * @param {object} pkg parsed package.json of the biz service
  * @param {object} librarySet committed SSOT shape or bare version map
- * @returns {{ok: boolean, gated: string[], notGated: string[], violations: Array<{package: string, message: string}>}}
+ * @returns {{ok: boolean, gated: string[], notGated: string[], compared: string[], violations: Array<{package: string, message: string}>}}
  */
 function checkLibCompat(pkg, librarySet) {
   const { versions, infraConsumed } = normalizeLibrarySet(librarySet);
@@ -91,6 +126,7 @@ function checkLibCompat(pkg, librarySet) {
 
   const gated = [];
   const notGated = [];
+  const compared = [];
   const violations = [];
 
   for (const [name, version] of ours) {
@@ -133,13 +169,23 @@ function checkLibCompat(pkg, librarySet) {
       continue;
     }
 
-    if (bizOnly) continue;
+    // The equality comparison is unconditional, for the same reason the
+    // exactness test above is: R6 proves a service pins exactly what the SSOT
+    // declares, and the SSOT declares a version for the biz-only packages too.
+    // Only the sentence differs — there is no infra release to deploy for a
+    // package no infra service installs, so naming one would send the reader
+    // after a fix that does not exist.
+    compared.push(name);
 
     if (declared !== version) {
       violations.push({
         package: name,
-        message: `${name}: biz pins ${version}, infra ships ${declared} — `
-          + 'rebuild the service against the current platform, or deploy the matching infra release first.'
+        message: bizOnly
+          ? `${name}: service ${serviceNameOf(pkg)} pins ${version}, the platform library SSOT declares `
+            + `${declared} — a service pins exactly what the SSOT declares, whether or not an infra service `
+            + `installs the package. Fix: npm install ${name}@${declared} --save-exact.`
+          : `${name}: biz pins ${version}, infra ships ${declared} — `
+            + 'rebuild the service against the current platform, or deploy the matching infra release first.'
       });
     }
   }
@@ -148,6 +194,7 @@ function checkLibCompat(pkg, librarySet) {
     ok: violations.length === 0,
     gated,
     notGated,
+    compared,
     violations
   };
 }

package/templates/business-service/scripts/verify-deploy-uniform.sh

@@ -79,7 +79,7 @@ WORKSPACE_ROOT=$(dirname "$CONTAINER_DIR")
 API_CHECKOUT="$WORKSPACE_ROOT/api"
 
 if [ "$(basename "$CONTAINER_DIR")" != "api_biz" ]; then
-  die "The checkout is not in the layout the uniform needs - $SERVICE_ROOT lies in '$(basename "$CONTAINER_DIR")', and the engine answers the rows that read the platform SSOT only for a service under <root>/api_biz/ beside <root>/api. Expected: the job checks this project out there. Fix: in .gitlab-ci.yml set the job variable GIT_CLONE_PATH: \$CI_BUILDS_DIR/oa-uniform/api_biz/<service>, and confirm the runner allows a custom build directory ([runners.custom_build_dir] enabled)." 2
+  die "The checkout is not in the layout the uniform needs - $SERVICE_ROOT lies in '$(basename "$CONTAINER_DIR")', and the engine answers the rows that read the platform SSOT only for a service under <root>/api_biz/ beside <root>/api. Expected: the job checks this project out there. Fix: make this job extend .oa-uniform in .gitlab-ci.yml (it sets GIT_CLONE_PATH: \$CI_BUILDS_DIR/oa-uniform/\$CI_CONCURRENT_ID/api_biz/<service>), and confirm the runner allows a custom build directory ([runners.custom_build_dir] enabled)." 2
 fi
 
 command -v git >/dev/null 2>&1 || die "Missing tool - 'git' does not run, and the api checkout this run reads cannot be fetched without it. Fix: install git in the job image (apk add --no-cache git)." 2
@@ -90,12 +90,24 @@ command -v node >/dev/null 2>&1 || die "Missing tool - 'node' does not run, and
 # Read-only and shallow: the uniform reads files, never history. A runner reuses
 # its builds directory between pipelines, so a checkout left by an earlier run
 # would silently measure this deploy against a stale SSOT — it is removed and
-# fetched again, and only when it is recognisably this script's own artefact.
+# fetched again, and ONLY when this script can prove it made it.
+#
+# The proof is the marker the clone step below writes. `.git` plus
+# `config/services.json` is no proof at all: every checkout of the api repository
+# carries both. On 2026-09-24 the script was run from api_biz/emailer on a
+# workstation, <workspace>/api was the developer's real working tree, it passed
+# that test and was deleted (639 tracked files; d.885). A directory without the
+# marker is somebody's work, so the script refuses and touches nothing
+# (`automation-gates.md` §1 requirement 3, Safe). There is no flag that skips
+# the check: the one systemic path is a free place or a clone this script made.
+CLONE_MARKER=".verify-deploy-uniform.clone"
 if [ -e "$API_CHECKOUT" ]; then
-  if [ -d "$API_CHECKOUT/.git" ] && [ -f "$API_CHECKOUT/config/services.json" ]; then
+  if [ -d "$API_CHECKOUT/.git" ] && [ -f "$API_CHECKOUT/config/services.json" ] \
+    && [ -f "$API_CHECKOUT/$CLONE_MARKER" ] \
+    && head -n 1 "$API_CHECKOUT/$CLONE_MARKER" | grep -q '^created-by verify-deploy-uniform.sh '; then
     rm -rf "$API_CHECKOUT"
   else
-    die "Cannot place the api checkout - $API_CHECKOUT already exists and is not an api clone this script made (no .git and config/services.json). Expected: that path free, or an earlier clone of the api repository. Fix: remove it, or point the job's GIT_CLONE_PATH at a build directory this pipeline owns." 2
+    die "Cannot place the api checkout - $API_CHECKOUT exists and carries no marker $CLONE_MARKER, so it is not a clone this script made, and deleting it could destroy somebody's work. Expected: a clone this script made. Fix: remove it yourself, or run this script only from a CI checkout where <workspace>/api is free (the job's GIT_CLONE_PATH points at a build directory this pipeline owns)." 2
   fi
 fi
 
@@ -106,6 +118,11 @@ if ! git clone --depth 1 --single-branch --branch "$API_REF" "$API_URL" "$API_CH
   die "The api checkout could not be cloned - git clone --branch $API_REF $SAFE_URL failed, so api/config/services.json cannot be read and the uniform would answer a fraction of its rows. A partial answer is not a deploy permit, so this refuses rather than continues. Expected: read access for this pipeline's job token. Fix: in GitLab open the api project, Settings > CI/CD > Job token permissions, and add this project to the allowlist (owner's step, confirmation biz-service-manifest 008 point 2); check also that the ref '$API_REF' exists there." 2
 fi
 
+# The marker is what lets the NEXT run delete this clone, so it is written the
+# moment the clone exists and before anything reads it.
+printf 'created-by verify-deploy-uniform.sh %s %s\n' "$(date -u +%Y-%m-%dT%H:%M:%SZ)" "$$" > "$API_CHECKOUT/$CLONE_MARKER" \
+  || die "Cannot mark the api checkout - writing $API_CHECKOUT/$CLONE_MARKER failed, and without it the next run would refuse to replace this clone. Expected: a writable build directory. Fix: check the permissions of $WORKSPACE_ROOT on the runner." 2
+
 # --- the engine -------------------------------------------------------------
 #
 # The pin decides which uniform this service is measured against, so the engine