@onlineapps/conn-orch-validator 12.0.0 → 12.1.0

package/CHANGELOG.md

@@ -4,6 +4,73 @@ All notable changes to this package. Follows [Keep a Changelog](https://keepacha
 
 ## [Unreleased]
 
+## [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)
+
+Řádky dokumentace (`docs-lint`, `docs-lint-clean` → `D-PORT`, `D-NPM`, `D-SCRIPT`, `D-RETIRED`, `D-HEADER`, `D-LINT`): lint
+`lint-biz-docs`, který skončil bez verdiktu (exit 2, pád spawnu, nerozparsovatelný JSON), je NOT RUN na každém řádku, který
+z něj čte, a signál nese `complete: false`; dosud z toho most dělal šest nálezů `deploy` s radami o obsahu dokumentace a
+`complete: true` (změřeno 2026-09-17, CI joby 16572194080 a 16571115198: GitLab Runner zakládá `api_biz/<svc>.tmp` a lint ho
+bral za službu). Důvod NOT RUN nese CELÝ stderr lintu, první řádek první (poslední řádek byla závorka gitu „Stopping at
+filesystem boundary“); `Fix:` se doplní jen když ho hláška lintu nenese. Chybějící `config/biz-docs-lint.tree.json` zůstává
+nálezem (`C-LINT`). Zároveň v api (`6f9d2f2b`): lint i `sync-biz-facts` berou sourozence ze SSOT `config/services.json`
+(`directory` u `enabled`), ne z výpisu `api_biz/` — nedeklarovaný adresář se nečte a řekne se to jednou větou.
+
+### Changed (testy) — třetí živá DB sada přes `liveDatabase.js`; sondy skládají dočasný adresář z `os.tmpdir()` (d.620b, W620b)
+
+`bizCiGateCli.setupDbAccount.integration` (blok „how far the production grant reaches“) bere spojení z
+`tests/helpers/liveDatabase.js`, práce je v `tests/helpers/grantReachProbe.js` a sada je třetí v jobu `test-validator-db`;
+`setupDatabaseProbe.js` a `dbAccountProbe.js` už nepíší do doslovného `/tmp`.
+
+
+### Changed (testy) — živé DB sady databázi deklarují, nehledají; „unwritable root“ se pod rootem předvede (d.628, d.623c, W623)
+
+`setupDatabaseLive.integration` a `dbCiAccount.integration` čtou spojení z `DB_HOST`, `DB_PORT`, `CI_DB_ROOT_USER`,
+`CI_DB_ROOT_PASSWORD` — jména, která čte `biz-ci-gate setup-db-account` — přes jednoho vlastníka `tests/helpers/liveDatabase.js`;
+dosud každá nesla vlastní kopii s vývojovým kontejnerem `gen_mariadb10.5`, sítí `gendb-network` a heslem z gitignorovaného
+`config/env-active/gen-db.env`, takže v CI checkoutu padaly. Bez deklarace se případy registrují jako přeskočené s vytištěným
+důvodem (runner hlásí „27 skipped“, ne „27 passed“); s deklarací se měří a nedostupný server, chybějící klient i chybějící
+`DB_DOCKER_NETWORK` jsou pád. V CI je pouští nový job `test-validator-db` se službou `mariadb:10.5.29` (vlastník 2026-09-18,
+conf `ci-validator-db-job` 001). Lokálně: README § Running the live-database suites. `oaValidateCli.integration` — případ
+„an unwritable service root fails fast“ se pod uid 0 předvádí pod neprivilegovaným účtem stroje místo NOT RUN.
+
+
+### Changed — šablona `init.sh` staví strom z `package-lock.json` (`npm ci`), ne `npm install` (d.629, W629)
+
+`templates/business-service/init.sh` `oa_npm_install()` = `npm ci --no-audit --no-fund`. `npm install --package-lock-only`
+(re-pin v živém bind-mount stromu) přepíše lock i skrytý `node_modules/.package-lock.json`, složky balíčků nechá staré,
+a `npm install` pak hlásí „up to date“ — služba tiše běží na jiné verzi, než pinuje lock a R6 (nález INFRA-monitoring,
+INFRA B137 `28fef3dc` totéž v sedmi infra službách). Podmínka instalace (blok `oa-deps-guard v1`, marker
+`.oa_drive_deps_hash`) ani znění bloku se nemění. Změřeno bez sítě nad lokálním tarballem: složka 1.0.0 × lock 2.0.0 →
+`npm install` „up to date“ (1.0.0), `npm ci` „added 1 package“ (2.0.0). Řádek uniformy `F-INIT` roznáší jen blok, servisní
+`oa_npm_install()` je řádek každé služby — běžících osm se nemění, dostanou ho pokynem. Služba oskládaná ze šablony
+potřebuje `package-lock.json` už při prvním startu: šablona ho neveze, první boot bez něj skončí hlášeným `EUSAGE`.
+
+
+### Fixed — hlavní grant databázového účtu escapuje `_` (d.620, W620)
+
+`databaseAccountSql` escapuje `_` i v hlavním grantu (`` ON `oagen\_x`.* ``), ne jen v CI wildcardu: databázová část
+`GRANT`u je LIKE vzor i bez `%`, takže grant, který dostane každá nasazená služba, otevíral i sousední schéma lišící se
+v tom jednom znaku (`oagen_emailer` → `oagen5emailer`; nález INFRA-DOCS z falzifikace `INSTALL.md`). Escape je jedna
+funkce `escapeSchemaPattern()` použitá na obě věty; `%` ve jméně schématu se odmítá s `Fix:` (žádné platformní jméno ho
+nenese, `\` odmítal už `FORBIDDEN_IN_VALUE`). Změřeno proti živé MariaDB 10.5: před opravou účet přečetl sousedovy řádky,
+po opravě `ERROR 1142 (42000) … SELECT command denied`; vlastní schéma dál čitelné, CI throwaway grant (`oagen\_x\_%`) drží.
+
+
+### Fixed — pozice balíčku ve workspace nese konvenční prefix `api/`, ne jméno adresáře checkoutu (d.625, W625)
+
+`sync/readmeLocation.js` odvozovalo `PACKAGE_IN_WORKSPACE` (a s ním `requiresSiblings` řádku `L-README-REGION`) z adresáře
+NAD checkoutem. GitLab CI klonuje repozitář pod jménem projektu `infra-mono`, takže vznikalo
+`infra-mono/shared/connector/conn-orch-validator`; `resolveWorkspacePath()` rozřeší přes marker jen hlavu `api` a zbytek
+připojí doslova → řetězec neukazoval nikam a běh končil větou `Fix: run with --workspace pointing at a checkout that carries
+infra-mono/…`, kterou nikdo nemůže splnit (`automation-gates.md` §1/4). Pravidlo stojí v hlavičce `manifest/workspaceRoot.js`:
+`api/` na začátku cesty je konvence deklarujícího textu, nikdy jméno adresáře na disku — prefix se proto čte z
+`WORKSPACE_MARKER` (jediný vlastník), ne píše podruhé. Změřeno v obrazu jobu nad jedním stromem pod dvěma jmény: 27 unit
+testů a 6 bats červených pod `infra-mono`, zelené pod `api`; po opravě zelené pod oběma. Štítek souboru na stdout
+`oa-sync-template` dál nese cestu na disku (tam soubor je) — srovnán test, ne výstup.
+
+
 ## [12.0.0] — 2026-09-17
 
 ### Fixed — BREAKING: R2 tvrdí pravidlo sekvence nasazení, ne včerejší text (d.614, W614)

package/README.md

@@ -269,6 +269,53 @@ namespace an integration tier builds into (`src/utils/throwawaySchema.js`). The
 `_` is escaped, so the grant stays inside one service — unescaped it is a LIKE
 wildcard, and `oagen_meta_%` would also match `oagen_metadata`.
 
+**The escape belongs to BOTH grants.** The database-name position of a `GRANT`
+is a LIKE pattern whether or not a `%` follows it, so the grant every installed
+service gets is written `` `oagen\_meta`.* `` too: unescaped, it also grants
+`oagen5meta` and every other schema differing in that one character. `%` in a
+schema name is refused rather than escaped — no platform name carries one.
+
+### Running the live-database suites
+
+Two suites of this package measure what a **real** MariaDB does with the SQL the
+package builds — `tests/unit/setupDatabaseLive.integration.test.js` (what
+`ci:gate:setup` leaves in the schema) and `tests/unit/dbCiAccount.integration.test.js`
+(whether the grants above are enough, and still not too much). They are part of
+`npm run test:unit`, and in CI they have a job with a database of its own,
+`test-validator-db` (owner decision
+[`ci-validator-db-job` 001](../../../docs/governance/confirmations/ci-validator-db-job.md)).
+
+Neither suite looks for a database. It is **declared**, under the names this
+package's own CLI reads:
+
+| Variable | What it is |
+|---|---|
+| `DB_HOST` / `DB_PORT` | the server the run measures |
+| `CI_DB_ROOT_USER` / `CI_DB_ROOT_PASSWORD` | its administrative login — the suites create and drop schemas of their own |
+| `DB_DOCKER_NETWORK` | only where no `mariadb` client is on PATH: the docker network the probe is carried in on |
+
+Declare nothing and the suites report **NOT RUN** with that sentence and a `Fix:`
+— they never stand down quietly. Declare a database and it is measured: an
+unreachable server, a missing client and a missing network are failures, so a run
+cannot go green having looked at nothing.
+
+On a developer machine the dev server publishes no port and macOS carries no
+client, so both the network and the credential come from the local environment —
+the password is read out of the gitignored file that owns it, never written into
+the repository:
+
+```bash
+cd api/shared/connector/conn-orch-validator
+DB_HOST=gen_mariadb10.5 DB_PORT=3306 \
+CI_DB_ROOT_USER=root \
+CI_DB_ROOT_PASSWORD="$(grep '^MARIADB_ROOT_PASSWORD=' ../../../config/env-active/gen-db.env | cut -d= -f2-)" \
+DB_DOCKER_NETWORK=gendb-network \
+npx jest tests/unit/setupDatabaseLive.integration.test.js tests/unit/dbCiAccount.integration.test.js
+```
+
+The run prints which server it measured, or why it measured none — that line is
+what tells a green suite that looked from a green suite that did not.
+
 ---
 
 ## Environment contract (`env` block)

package/jest.config.js

@@ -13,6 +13,17 @@ module.exports = {
   testMatch: [
     '**/tests/**/*.test.js'
   ],
+
+  // tests/fixtures/manifest/** holds SIMULATED CHECKOUTS that the manifest
+  // suites read from disk as data, never require. Jest's haste map indexes every
+  // package.json it crawls, so the three fixture workspaces that each carry the
+  // service `biz-alpha` read as three modules of one name and the crawl prints
+  // "Haste module naming collision" on stdout. The reason the fixtures keep
+  // their names, and why the pattern stops at `manifest/` rather than covering
+  // tests/fixtures/ whole, is written once — in the repository's jest.config.js,
+  // beside the same string. This config owns the same exclusion for its own map,
+  // because rootDir here is the package.
+  modulePathIgnorePatterns: ['/tests/fixtures/manifest/'],
   coverageThreshold: {
     global: {
       branches: 80,

package/package.json

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

package/src/manifest/checks/docsLintBridge.js

@@ -107,6 +107,50 @@ const LINT_TIMEOUT_MS = 120000;
 /** Room for the JSON of a large tree; the biggest measured today is 54 findings. */
 const LINT_MAX_BUFFER = 32 * 1024 * 1024;
 
+/**
+ * How much of the linter's own output the refusal sentence carries, in
+ * characters. A budget, not a line count: what makes a refusal actionable is the
+ * FIRST line, which is the linter's `[tool] ERROR:` message with its own `Fix:`
+ * in it, and the lines after it are context of falling value. So the first line
+ * is never cut and the rest fills what is left.
+ */
+const REFUSAL_BUDGET = 700;
+
+/** The remedy for a refusal whose own message named none. */
+const REFUSAL_FIX = `Fix: run ${LINT_SCRIPT} over this tree by hand and repair what it reports.`;
+
+/**
+ * What a run that produced no verdict says, from everything the linter printed.
+ *
+ * Until d.632 this read `…split('\n').filter(Boolean).pop()` — the LAST line —
+ * and the last line of the measured failure was git's own aside, `Stopping at
+ * filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set)`. The sentence
+ * that says WHAT the linter refused, and how to repair it, is the first one, so
+ * the reader was handed the footnote and not the message (CI jobs 16572194080,
+ * 16571115198; `.claude/rules/automation-gates.md` §1 requirement 4).
+ *
+ * @param {{status: number|null, stderr: string, stdout: string}} run the finished process
+ * @returns {string}
+ */
+function describeRefusal({ status, stderr, stdout }) {
+  const lines = String(stderr || stdout || '').split('\n').map((line) => line.trim()).filter(Boolean);
+  const exited = status === null || status === undefined ? 'on a signal' : String(status);
+  if (lines.length === 0) return `the lint exited ${exited} without a verdict and printed nothing. ${REFUSAL_FIX}`;
+
+  let said = lines[0];
+  for (const line of lines.slice(1)) {
+    if (said.length + line.length + 3 > REFUSAL_BUDGET) {
+      said += ' …';
+      break;
+    }
+    said += ` | ${line}`;
+  }
+  // The linter writes its own `Fix:` into the message; a second one beside it
+  // would be this module restating the tool (docblock at the top of this file).
+  // Only a message that named none gets the command.
+  return `the lint exited ${exited} without a verdict — ${said}${said.includes('Fix:') ? '' : `. ${REFUSAL_FIX}`}`;
+}
+
 /**
  * The lint's answers, keyed by what the run was about. One service is linted
  * ONCE however many rows cite it: five rows spawning five processes over the
@@ -296,7 +340,7 @@ function claimantOf(ruleId, rows) {
  * Run the lint over one tree, once.
  *
  * @param {{ lintScript: string, docsDir: string, treeConfig: string }} params absolute paths
- * @returns {{findings: Array<object>, skipped: Array<object>}|{problem: string}}
+ * @returns {{findings: Array<object>, skipped: Array<object>}|{refusal: string}}
  */
 function lintOnce({ lintScript, docsDir, treeConfig }) {
   // `\0` as the escape, never the raw byte: written literally it makes the
@@ -313,7 +357,7 @@ function lintOnce({ lintScript, docsDir, treeConfig }) {
 
 /**
  * @param {{ lintScript: string, docsDir: string, treeConfig: string }} params
- * @returns {{findings: Array<object>, skipped: Array<object>}|{problem: string}}
+ * @returns {{findings: Array<object>, skipped: Array<object>}|{refusal: string}}
  */
 function runLint({ lintScript, docsDir, treeConfig }) {
   const run = spawnSync(process.execPath, [lintScript, ...lintArgumentsFor({ docsDir, treeConfig })], {
@@ -326,23 +370,20 @@ function runLint({ lintScript, docsDir, treeConfig }) {
     maxBuffer: LINT_MAX_BUFFER
   });
 
-  if (run.error) return { problem: `${run.error.message} (${lintScript})` };
+  if (run.error) return { refusal: `the lint could not be started: ${run.error.message} (${lintScript}). ${REFUSAL_FIX}` };
 
   // 0 = clean, 1 = findings; anything else is the lint refusing to run, and its
   // reason is on stderr. A run that ended there is NOT an empty finding list.
-  if (run.status !== 0 && run.status !== 1) {
-    const said = String(run.stderr || run.stdout || '').trim().split('\n').filter(Boolean).pop();
-    return { problem: `the lint exited ${run.status === null ? 'on a signal' : run.status}: ${said || 'no output'}` };
-  }
+  if (run.status !== 0 && run.status !== 1) return { refusal: describeRefusal(run) };
 
   let parsed;
   try {
     parsed = JSON.parse(run.stdout);
   } catch (error) {
-    return { problem: `its JSON output could not be read: ${error.message}` };
+    return { refusal: `the lint exited ${run.status} and its JSON output could not be read: ${error.message}. ${REFUSAL_FIX}` };
   }
   if (!Array.isArray(parsed.findings) || !Array.isArray(parsed.skipped)) {
-    return { problem: 'its JSON output carries no "findings"/"skipped" arrays' };
+    return { refusal: `the lint exited ${run.status} and its JSON output carries no "findings"/"skipped" arrays. ${REFUSAL_FIX}` };
   }
   return { findings: parsed.findings, skipped: parsed.skipped };
 }
@@ -408,13 +449,23 @@ function treeConfigOf(block) {
 /**
  * Ask the lint about one service tree, once, for whichever row is asking.
  *
- * Three answers, and every caller words them in its own sentence: the tree has
- * nothing to lint, the question could not be decided (with the reason), or the
- * lint's own payload.
+ * Four answers, and every caller words them in its own sentence: the tree has
+ * nothing to lint, the question could not be decided because a file with a fix
+ * of its own is missing, the lint REFUSED to run at all, or the lint's own
+ * payload.
+ *
+ * The last two were one answer until d.632, and both became findings. They are
+ * different facts. An absent tree config is a defect of this repository with a
+ * row that owns it (`C-LINT`) and a repair somebody can carry out here, so it
+ * stays a finding. A lint that refused to run measured NOTHING: it decided no
+ * rule, so a row reporting a `deploy` finding about documentation is asserting
+ * something nobody looked at — six of them, in the run this split comes from.
+ * That is NOT RUN (`.claude/rules/automation-gates.md` §5).
  *
  * @param {{ block: object, serviceRoot: string, workspaceRoot: string }} params
  * @returns {{nothingToLint: true}
  *           |{undecidable: {relative: string, reason: string}}
+ *           |{refused: string}
  *           |{answer: {findings: Array<object>, skipped: Array<object>}}}
  */
 function askLint({ block, serviceRoot, workspaceRoot }) {
@@ -442,7 +493,7 @@ function askLint({ block, serviceRoot, workspaceRoot }) {
     docsDir,
     treeConfig
   });
-  if (answer.problem !== undefined) return { undecidable: { relative: DOCS_DIR, reason: answer.problem } };
+  if (answer.refusal !== undefined) return { refused: answer.refusal };
 
   return { answer };
 }
@@ -475,7 +526,7 @@ const docsLint = Object.freeze({
 
   /**
    * @param {{ row: object, block: object, serviceRoot: string, workspaceRoot: string }} params
-   * @returns {Array<{where: string, what: string}>}
+   * @returns {Array<{where: string, what: string}>|{findings: Array<object>, notRun: string}}
    */
   run({ row, block, serviceRoot, workspaceRoot }) {
     const rows = rowsOf(block);
@@ -490,6 +541,9 @@ const docsLint = Object.freeze({
         what: `${row.rule} could not be decided — ${asked.undecidable.reason}`
       }];
     }
+    if (asked.refused !== undefined) {
+      return { findings: [], notRun: `${row.rule} could not be decided — ${asked.refused}` };
+    }
     const answer = asked.answer;
 
     const found = answer.findings
@@ -511,9 +565,10 @@ const docsLint = Object.freeze({
     // The mapping is the linter's own two channels onto this uniform's two: its
     // `findings` are findings, its `skipped` list is the NOT RUN one (that is
     // what `--allow-missing-siblings` fills, and what its own text output prints
-    // as NOT RUN lines). A tree the linter REFUSES outright is neither — that
-    // is `answer.problem` above, and the absent tree config before it, both of
-    // which stay findings with a fix somebody can carry out.
+    // as NOT RUN lines). A tree whose lint REFUSED to run is NOT RUN too, for
+    // the same reason and one branch above (`askLint` § refused); the absent
+    // tree config before it stays a finding, because `C-LINT` owns that file
+    // and the repair is here.
     //
     // The sentence itself is `describeUndecided` above — shared with `D-LINT`,
     // which reports the same absence about the rules no row claims.
@@ -579,6 +634,9 @@ const docsLintClean = Object.freeze({
         what: `no finding could be decided — ${asked.undecidable.reason}`
       }];
     }
+    if (asked.refused !== undefined) {
+      return { findings: [], notRun: `no finding could be decided — ${asked.refused}` };
+    }
     const answer = asked.answer;
 
     const found = errorGraded(answer.findings)
@@ -605,6 +663,7 @@ module.exports = {
   claimantOf,
   citationsOf,
   covers,
+  describeRefusal,
   errorGraded,
   LINT_SCRIPT,
   LINT_DIR

package/src/sync/readmeLocation.js

@@ -29,7 +29,7 @@ const path = require('path');
 
 const { loadManifest, DEFAULT_MANIFEST_PATH, LIBRARY_MANIFEST_PATH } = require('../manifest/loadManifest');
 const {
-  API_CHECKOUT_ROOT, PACKAGE_ROOT: OWN_ROOT, resolveWorkspacePath
+  API_CHECKOUT_ROOT, PACKAGE_ROOT: OWN_ROOT, WORKSPACE_MARKER, resolveWorkspacePath
 } = require('../manifest/workspaceRoot');
 const { KINDS, renderUniformRegion } = require('./readmePointer');
 
@@ -74,6 +74,13 @@ function serviceManifestPath(serviceRoot) {
   return path.join(serviceRoot, 'node_modules', ...selfName().split('/'), SERVICE_MANIFEST_IN_PACKAGE);
 }
 
+/**
+ * The conventional prefix a workspace-relative path uses for the api checkout,
+ * read off the marker that declares it rather than typed a second time here
+ * (`../manifest/workspaceRoot.js`, which owns both the prefix and the marker).
+ */
+const [API_PREFIX] = WORKSPACE_MARKER.split('/');
+
 /**
  * Where this package lies inside its own workspace, workspace-relative — or null
  * when it lies in no checkout at all (an installed copy, a service container).
@@ -84,10 +91,25 @@ function serviceManifestPath(serviceRoot) {
  * at the manifest AS IT LIES IN THAT CHECKOUT, and where there is no copy there
  * is no link — which is a question the run could not answer, never a package
  * that passed (`.claude/rules/automation-gates.md` §5).
+ *
+ * The head of that path is the CONVENTION `api/`, never the name the checkout
+ * happens to wear on disk — the rule `workspaceRoot.js` states in its header and
+ * implements in `resolveWorkspacePath()`, which is the function this string is
+ * handed to. Deriving it as "the directory above the checkout, then the checkout"
+ * broke that rule wherever the checkout is named something else: GitLab CI checks
+ * this repository out as `infra-mono`, so the string became
+ * `infra-mono/shared/connector/conn-orch-validator`, which `resolveWorkspacePath`
+ * joins literally — pointing at nothing in every workspace, including the fixture
+ * and sandbox trees the suites build. Measured 2026-09-17 in the job's own image,
+ * one tree under two names: under `infra-mono` the run died with
+ * `Fix: run with --workspace pointing at a checkout that carries
+ * infra-mono/shared/connector/conn-orch-validator` — an instruction nobody can
+ * carry out (`.claude/rules/automation-gates.md` §1 requirement 4) — while the
+ * same tree named `api` was green.
  */
 const PACKAGE_IN_WORKSPACE = API_CHECKOUT_ROOT === null
   ? null
-  : path.relative(path.dirname(API_CHECKOUT_ROOT), OWN_ROOT).split(path.sep).join('/');
+  : [API_PREFIX, ...path.relative(API_CHECKOUT_ROOT, OWN_ROOT).split(path.sep)].join('/');
 
 /**
  * The manifest as it lies in the workspace being read, whichever of the two it

package/src/utils/dbAccountGrants.js

@@ -33,6 +33,24 @@
  * what keeps the grant inside one service, and it is the difference
  * `tests/unit/dbAccountGrants.test.js` measures.
  *
+ * ## The escape belongs to BOTH grants, not only the wildcard one
+ *
+ * The database-name position of a `GRANT` is read as a LIKE pattern whether or
+ * not a `%` follows it, so `` ON `oagen_emailer`.* `` — the grant EVERY
+ * installed service gets — also grants `oagen5emailer` and every other schema
+ * differing in that one character. One escape function therefore runs on the
+ * schema name before either statement is built; the production runbook
+ * (`api/docs/setup/INSTALL.md` § the account and its grants, `schema_grant`)
+ * already escaped it, and a definition laxer than the runbook installing from it
+ * is the wrong way round. Measured against the dev server in
+ * `tests/unit/bizCiGateCli.setupDbAccount.integration.test.js`: with the
+ * unescaped grant the account reads the neighbour's rows; with it, `ERROR 1142`.
+ *
+ * `%` — the other wildcard of that position — is REFUSED rather than escaped,
+ * for the reason the comment above `FORBIDDEN_IN_VALUE` gives: no platform
+ * schema name carries one, so a value that does came from somewhere it should
+ * not have. `_` cannot be refused the same way: every platform name has one.
+ *
  * Production never gets that grant: it installs one schema, and a wildcard there
  * would be a standing privilege nobody asked for.
  *
@@ -84,6 +102,43 @@ function requireValue(value, name, fix) {
   return value;
 }
 
+/**
+ * The one escape of this file, used by every grant it builds.
+ *
+ * MariaDB reads the database-name position of a `GRANT` as a LIKE pattern, so
+ * `_` there matches any single character. Every platform schema is
+ * `oagen_<shortname>`, which means every unescaped grant this package could
+ * build would reach a schema nobody named.
+ *
+ * `%` is not escaped here — `requireGrantableSchema` refuses it before this
+ * runs, so the only wildcard left to neutralise is `_`.
+ *
+ * @param {string} schema a schema name already checked by `requireGrantableSchema`
+ * @returns {string} the same name as a LIKE pattern matching only itself
+ */
+function escapeSchemaPattern(schema) {
+  return schema.replace(/_/g, '\\_');
+}
+
+/**
+ * A schema name this file can turn into a grant that reaches that schema ALONE.
+ *
+ * `requireValue` already refuses the backslash (an escape nobody wrote here) and
+ * the quoting characters. This adds the one character that is neither a quoting
+ * problem nor escapable without changing what the name means: `%`.
+ */
+function requireGrantableSchema(schema, fix) {
+  requireValue(schema, 'schema', fix);
+  if (schema.includes('%')) {
+    throw new Error('[DbAccountGrants] schema carries `%`, which the database-name position of a GRANT '
+      + `reads as a wildcard: ${JSON.stringify(schema)}.\n`
+      + '       A grant built from it would reach schemas nobody named, and escaping it would grant a '
+      + 'schema whose name really contains `%` — neither is what a platform schema means.\n'
+      + `       Fix: ${fix}`);
+  }
+  return schema;
+}
+
 /**
  * Every statement that creates this service's account and grants it what it
  * needs, in the order they must run.
@@ -95,26 +150,30 @@ function requireValue(value, name, fix) {
  *   (`DB_USER` of `config/env-templates/<service>.env`)
  * @param {string} options.password the password the account is created with
  * @param {boolean} [options.ciThrowaway] also grant the throwaway namespace
- *   `<schema>\_%`, which only an integration tier builds into
+ *   `<escaped schema>\_%`, which only an integration tier builds into
  * @returns {string[]} the statements, in order
  */
 function databaseAccountSql({ schema, account, password, ciThrowaway = false } = {}) {
-  requireValue(schema, 'schema', 'pass database.schema from config/service/integration-contract.json.');
+  requireGrantableSchema(schema, 'pass database.schema from config/service/integration-contract.json.');
   requireValue(account, 'account', 'pass DB_USER from this service\'s config/env-templates/<service>.env.');
   requireValue(password, 'password', 'pass the password the account is created with; an account created '
     + 'with an empty password is one anybody on the network can use.');
 
   const identity = `'${account}'@'${ACCOUNT_HOST}'`;
+  // Both grants below are built from this one value: the database-name position
+  // is a LIKE pattern in either of them (see the head of this file).
+  const pattern = escapeSchemaPattern(schema);
 
   const statements = [
     `CREATE USER IF NOT EXISTS ${identity} IDENTIFIED BY '${password}';`,
-    `GRANT ALL PRIVILEGES ON \`${schema}\`.* TO ${identity};`
+    `GRANT ALL PRIVILEGES ON \`${pattern}\`.* TO ${identity};`
   ];
 
   if (ciThrowaway === true) {
-    // The backslash escapes `_` so the pattern stays inside this service; see
-    // the head of this file for the neighbour it would otherwise reach.
-    statements.push(`GRANT ALL PRIVILEGES ON \`${schema}\\_%\`.* TO ${identity};`);
+    // `\_%` is the throwaway SUFFIX — the one wildcard meant to be one — added
+    // to the escaped name; see the head of this file for the neighbour an
+    // unescaped name would otherwise reach.
+    statements.push(`GRANT ALL PRIVILEGES ON \`${pattern}\\_%\`.* TO ${identity};`);
   }
 
   // Last, always: a grant is not live for a connection opened before it.

package/templates/business-service/init.sh

@@ -1,8 +1,14 @@
 #!/bin/sh
 set -e
 
+# `npm ci`, not `npm install`: the tree is built from package-lock.json alone,
+# never reconciled against npm's hidden node_modules/.package-lock.json, which
+# a `--package-lock-only` re-pin rewrites while the package directories keep
+# the old versions. A lock that disagrees with package.json stops the boot here
+# instead of installing something nobody declared.
+# @see api/tests/scripts/infra-init-scripts.bats
 oa_npm_install() {
-  npm install --no-audit --no-fund
+  npm ci --no-audit --no-fund
 }
 
 # --- oa-deps-guard v1 (identical in every OA Drive init.sh) ---
@@ -42,7 +48,7 @@ if [ "$CURRENT_HASH" != "$PREVIOUS_HASH" ] || [ "$NEEDS_INSTALL" = "1" ]; then
     echo "$CURRENT_HASH" > "$MARKER_FILE"
     echo "[init.sh] Dependencies installed, marker $MARKER_FILE updated."
   else
-    echo "[init.sh] Dependency install failed - marker $MARKER_FILE left untouched, so the next start retries instead of trusting a half-installed tree. Fix: read the npm error above, then run 'npm install' in this service directory."
+    echo "[init.sh] Dependency install failed - marker $MARKER_FILE left untouched, so the next start retries instead of trusting a half-installed tree. Fix: read the npm error above; a service that has a package-lock.json installs with 'npm ci --no-audit --no-fund' in this directory - 'npm install' is what let the folders and the lock drift apart; a service that has none yet needs 'npm install' once, to create it."
     exit 1
   fi
 else