ci(temporal-conformance): give the non-SQL step the job's live PostgreSQL URL (#21577)

Fixes #21564
Clause-②: no

## What this changes

The `Temporal Conformance (live PG + MySQL)` job's step "Run the non-SQL
temporal backends under the skewed process zone" now sets
`OS_TEST_POSTGRES_URL` in its `env:`, with the same literal the job's
three SQL steps use
(`postgres://postgres:postgres@127.0.0.1:5432/postgres`), for the
postgres service the job already provisions. Until now that step set
only `TZ`, so every `service-analytics` live-PostgreSQL cell
(`describe.skipIf(!config)`) was a named skip there while the job read
green.

The header clause "no CI step provisions that variable for this package"
in 15 `service-analytics` test files becomes false with this wiring. It
is corrected, comment-only, to name the step that now sets it. No other
line of those files changes: every changed line in them is a ` * `
comment line. The 15th file, `objectql-echo-date-bucket.test.ts`, landed
on `main` after this PR was opened, and the takeover added it (claim
amendment `5973803595`).

No other job, no other step, and no timeout changes. No cell is
re-skipped.

## What the step needed, measured (A2)

Only the one `env:` line:

- **Build closure.** `turbo run build
--filter=@objectstack/service-analytics... --dry=json` selects 16
packages, including `@objectstack/driver-sql`, `@objectstack/objectql`
and `@objectstack/driver-sqlite-wasm` (dev dependencies of
`service-analytics`). The step's existing build already builds what the
cells import.
- **PostgreSQL client.** `pg` is an optional peer of `driver-sql`,
resolved by pnpm into `driver-sql`'s `knex` instance;
`require.resolve('pg')` from that `knex` answers `pg@8.22.0`. The cells
reach `pg` through `SqlDriver`, so nothing is added to
`service-analytics`.
- **Database and tables.** Every cell connects to the default `postgres`
database and creates and drops its own tables. The table names are
distinct across the 17 files that read the URL (`os21316_deal`,
`os21505_coercion`, `os21441_bucket_deal`, ...), so parallel vitest
workers do not collide.
- **Server zone.** The job's existing step "Point both servers at a
non-UTC timezone" sets the server to `Asia/Shanghai` before any test
step.

## Readings

- **A3.** No `service-analytics` file reads
`OS_EXPECT_LIVE_DIALECT_MATRIX` (`git grep` exit 1), and no code in the
step's other four packages reads it. So the flag is not added to the
step.
- **A4.** No `service-analytics` file reads `OS_TEST_MYSQL_URL` or any
other server variable the job provides. Its only `process.env` reads are
`OS_TEST_POSTGRES_URL` (34, in 17 files), `NODE_ENV` (10) and `TZ` (4).
Nothing is listed for MySQL.
- **A5.** `core`, `formula`, `driver-memory` and `driver-mongodb` have
zero `OS_TEST_POSTGRES_URL` hits (`git grep` exit 1 for each). Adding
the variable to the step starts no test in them.

## Acceptance

- **CI, on this head `6e12d6934a`:** `Temporal Conformance (live PG +
MySQL)` (job `111301243023`) is success, and step 14, the step this PR
wires, is success (3m38s). All 34 check runs are complete: 32 success, 2
skipped (`Check Changeset` under `skip-changeset`, and the opt-in
packed-tarball smoke).
- **The cells run in CI, by contrast with the head before the fix:**
- At `a3fb6c0334`, with this same wiring, step 14 failed. Four
`objectql-face-order-limit` live-PostgreSQL cells went red on #21485's
defect: a `date` month bucket landed a day early on a non-UTC server.
- #21485 was fixed by PR #21611 (`440cd329a9`), which this branch now
carries. With the same wiring, step 14 passes at `25fb56f1b7` and at
`6e12d6934a`.
- That red-then-green on the same step shows the cells run there and now
pass.
- **NOT MEASURED: a quoted step-14 log line.** The job-log host answers
Forbidden from the seat containers, and the API's log tail does not
reach step 14. A seat whose egress reaches the log host can read step
14's per-file vitest lines for `objectql-face-order-limit` and
`objectql-echo-date-bucket`.
- **Local reproduction of the job's two zones** (PostgreSQL 16.14 with
`timezone=Asia/Shanghai`, process `TZ=America/New_York`, the full
`service-analytics` suite on the merged tree `25fb56f1b7`, from vitest's
JSON reporter):
  - With the URL: 4419 passed, 0 failed, 2 skipped of 4421.
  - Without it: 4160 passed, 261 skipped.
  - So 259 cells move from skipped to run.
- `objectql-face-order-limit`: 26 / 0 / 0, including the four formerly
red cells.
  - `objectql-echo-date-bucket`: 16 / 0 / 0.
- The two remaining skips are cell-scoped by design: the SQLite cell's
"the server is not on UTC" check in `read-scope-temporal-coercion`, and
a SQLite-only case in `analytics-faces-one-lowering`. Their PostgreSQL
twin, "(pg) the server is not on UTC", passed, so the run was not
vacuous.

## Acceptance notes

- The ruling says each of these cells "asserts a non-UTC server". At
this tree, 1 of the 17 files does
(`read-scope-temporal-coercion.test.ts`, `show timezone`). The other 16
rely on the job's provisioning step for the server's zone. Noted, not
changed.
-
`packages/plugins/plugin-security/src/rls-boolean-comparand-door.test.ts:40`
carries the same "no CI step provisions" clause about its own package.
This PR's step does not run that package, so the clause there stays true
and is not touched.
- **Stall guard.** `check:stall-guard-budget` reads this step's static
budget as window 10m, cap 20m, job budget 30m, slack 10m. The step ran
3m38s on this head.
- **Release.** Nothing publishes: `.github/workflows/ci.yml`, plus
comments in test files outside `service-analytics`'s `files[]`. No
changeset; `skip-changeset`.
- **Governed.** `check-governed-merges.mjs`: 0 of 16 paths hit the
register, NOT governed; 118 changed lines (+80 / −38).

## Gates

Run on head `6e12d6934a`: the merge `1866c0185c` of `origin/main`
`5b5e83f446`, plus the header commit. Each exit code was recorded before
any pipe.

- `dispatch-gates.mjs --commands` derived 87 commands. All 87 ran, and
all exited 0. `--ran` reconciled them as 87 derived, 87 run, 0
NOT-MEASURED.
- `check:changeset-gate-self-tests` was re-run once, alone, after the
container's commit-signing hop answered 503 inside its scratch
repository on the first pass. The re-run exited 0.
- `service-analytics`: its build closure built, and
`objectql-echo-date-bucket.test.ts` passed without the URL (10 passed, 8
PostgreSQL cells skipped locally). Its full suite with the URL is in
Acceptance above.

Opened by `domain:spec` seat 2 (`session_01YDt3PzwfrkuFzUBF89WPmM`);
taken over by `domain:spec` seat 1 (`session_01T9u38rswFp5Rw8DswRUReJ`,
claim `5973375306`).

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: objectstack-fleet[bot]

.github/workflows/ci.yml

@@ -1526,6 +1526,14 @@ jobs:
       - name: Run the non-SQL temporal backends under the skewed process zone
         env:
           TZ: America/New_York
+          # [#21564] The same URL the SQL legs of this job receive, for the
+          # postgres service this job already runs. `service-analytics` keys
+          # its live-PostgreSQL cells on it (`describe.skipIf(!config)`), so
+          # without this line they were named skips here and the job read as
+          # though ADR-0053 D-A3's live non-UTC PostgreSQL row had run. Core,
+          # formula, driver-memory and driver-mongodb do not read it. No
+          # `OS_EXPECT_LIVE_DIALECT_MATRIX` here: no test in this step reads it.
+          OS_TEST_POSTGRES_URL: postgres://postgres:postgres@127.0.0.1:5432/postgres
         run: |
           # The axis is only real if the zone actually took. Without this, a
           # dropped `TZ:` line silently returns the job to UTC coverage and

packages/services/service-analytics/src/__tests__/cube-measure-field-type-door.test.ts

@@ -39,9 +39,11 @@
  * ## The dialect axis of THIS file
  *
  * The SQLite cell always runs. The PostgreSQL cell runs where
- * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise; no CI step
- * provisions that variable for this package, so the live cell is red-capable
- * and un-run in CI, and the PR that landed this file carries its local
+ * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise; CI provisions
+ * that variable for this package in the Temporal Conformance job's step
+ * "Run the non-SQL temporal backends under the skewed process zone"
+ * (`.github/workflows/ci.yml`), so the live cell is red-capable and runs in
+ * CI, and the PR that landed this file carries its local
  * PostgreSQL 16 run. The live cell owns its table, dropped before and after.
  */
 

packages/services/service-analytics/src/__tests__/cube-measure-relationship-path-type.test.ts

@@ -48,9 +48,11 @@
  * ## The dialect axis of THIS file
  *
  * The SQLite cell always runs. The PostgreSQL cell runs where
- * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise; no CI step
- * provisions that variable for this package, so the live cell is red-capable
- * and un-run in CI, and the PR that landed this file carries its local
+ * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise; CI provisions
+ * that variable for this package in the Temporal Conformance job's step
+ * "Run the non-SQL temporal backends under the skewed process zone"
+ * (`.github/workflows/ci.yml`), so the live cell is red-capable and runs in
+ * CI, and the PR that landed this file carries its local
  * PostgreSQL 16 run. The live cell owns its tables, dropped before and after.
  */
 

packages/services/service-analytics/src/__tests__/json-stored-door-live-drivers.test.ts

@@ -28,9 +28,11 @@
  * ## The dialect axis
  *
  * The SQLite cell always runs. The PostgreSQL cell runs where
- * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise; no CI step
- * provisions that variable for this package, so the live cell is red-capable
- * and un-run in CI. The live cell owns its table, dropped before and after.
+ * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise; CI provisions
+ * that variable for this package in the Temporal Conformance job's step
+ * "Run the non-SQL temporal backends under the skewed process zone"
+ * (`.github/workflows/ci.yml`), so the live cell is red-capable and runs in
+ * CI. The live cell owns its table, dropped before and after.
  */
 
 import { describe, it, expect, beforeAll, afterAll } from 'vitest';

packages/services/service-analytics/src/__tests__/json-stored-door-undeclared-join.test.ts

@@ -45,9 +45,11 @@
  * ## The dialect axis of THIS file
  *
  * The SQLite cell always runs. The PostgreSQL cell runs where
- * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise; no CI step
- * provisions that variable for this package, so the live cell is red-capable
- * and un-run in CI, and the PR that landed this file carries its local
+ * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise; CI provisions
+ * that variable for this package in the Temporal Conformance job's step
+ * "Run the non-SQL temporal backends under the skewed process zone"
+ * (`.github/workflows/ci.yml`), so the live cell is red-capable and runs in
+ * CI, and the PR that landed this file carries its local
  * PostgreSQL 16 run. The live cell owns its tables, dropped before and after.
  */
 

packages/services/service-analytics/src/__tests__/list-at-scalar-operator-both-faces.test.ts

@@ -38,7 +38,10 @@
  * before any leaf exists, so the arm reaches it with no change here.
  *
  * The PostgreSQL cells run where `OS_TEST_POSTGRES_URL` is set and are a named
- * skip otherwise; no CI step provisions that variable for this package. The
+ * skip otherwise; CI provisions that variable for this package in the
+ * Temporal Conformance job's step
+ * "Run the non-SQL temporal backends under the skewed process zone"
+ * (`.github/workflows/ci.yml`). The
  * live cell owns its table, dropped before and after.
  */
 

packages/services/service-analytics/src/__tests__/native-sql-aggregate-policies.test.ts

@@ -44,8 +44,11 @@
  *
  * The SQLite cell always runs; there, the fold is the policy that diverged. The
  * PostgreSQL cell runs where `OS_TEST_POSTGRES_URL` is set and is a named skip
- * otherwise; no CI step provisions that variable for this package, so the live
- * cell is red-capable and un-run in CI, and the PR that landed this file carries
+ * otherwise; CI provisions that variable for this package in the
+ * Temporal Conformance job's step
+ * "Run the non-SQL temporal backends under the skewed process zone"
+ * (`.github/workflows/ci.yml`), so the live cell is red-capable and runs in
+ * CI, and the PR that landed this file carries
  * its local PostgreSQL 16 run. The operand the PostgreSQL / MySQL cells rely on
  * is pinned per dialect in CI by `@objectstack/core`'s `aggregate-answer.test.ts`
  * and by `driver-sql`'s move proof. MySQL is not a cell here. The live cell owns

packages/services/service-analytics/src/__tests__/native-sql-base-column-qualify.test.ts

@@ -41,9 +41,11 @@
  * ## The dialect axis of THIS file
  *
  * The SQLite cell always runs. The PostgreSQL cell runs where
- * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise; no CI step
- * provisions that variable for this package, so the live cell is red-capable
- * and un-run in CI, and the PR that landed this file carries its local
+ * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise; CI provisions
+ * that variable for this package in the Temporal Conformance job's step
+ * "Run the non-SQL temporal backends under the skewed process zone"
+ * (`.github/workflows/ci.yml`), so the live cell is red-capable and runs in
+ * CI, and the PR that landed this file carries its local
  * PostgreSQL 16 run. The live cell owns its tables, dropped before and after.
  */
 

packages/services/service-analytics/src/__tests__/native-sql-boolean-comparand-door.test.ts

@@ -38,7 +38,10 @@
  * edit in place would throw.
  *
  * The PostgreSQL cell runs where `OS_TEST_POSTGRES_URL` is set and is a named
- * skip otherwise; no CI step provisions that variable for this package. The
+ * skip otherwise; CI provisions that variable for this package in the
+ * Temporal Conformance job's step
+ * "Run the non-SQL temporal backends under the skewed process zone"
+ * (`.github/workflows/ci.yml`). The
  * live cell owns its table, dropped before and after.
  */
 

packages/services/service-analytics/src/__tests__/native-sql-measure-number-presentation.test.ts

@@ -56,9 +56,11 @@
  * ## The dialect axis of THIS file
  *
  * The SQLite cell always runs. The PostgreSQL cell runs where
- * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise. No CI step
- * provisions that variable for this package, so the live cell is red-capable
- * and un-run in CI; the PR that landed this file carries its local PostgreSQL
+ * `OS_TEST_POSTGRES_URL` is set and is a named skip otherwise. CI provisions
+ * that variable for this package in the Temporal Conformance job's step
+ * "Run the non-SQL temporal backends under the skewed process zone"
+ * (`.github/workflows/ci.yml`), so the live cell is red-capable and runs in
+ * CI; the PR that landed this file carries its local PostgreSQL
  * 16 run. `@objectstack/core`'s `aggregate-answer.test.ts` pins the presenter
  * itself in CI. The live cell owns its table, dropped before and after.
  */