declapract-typescript-ehmpathy 0.49.11 → 0.49.12

package/dist/practices/cicd-app-react-native-expo/best-practice/.github/workflows/deploy.yml

@@ -9,8 +9,8 @@ on:
       - 'master'
   workflow_dispatch:
     inputs:
-      stage:
-        description: 'which stage do you want to deploy to?'
+      access:
+        description: 'which access do you want to deploy with?'
         type: choice
         options:
           - prep
@@ -40,7 +40,7 @@ jobs:
     if: github.event_name != 'workflow_dispatch' || github.event.inputs.thoroughly == 'true'
     with:
       creds-aws-region: us-east-1
-      creds-aws-role-arn: arn:aws:iam::@declapract{variable.awsAccountId.prep}:role/@declapract{variable.projectName}-github-actions-prep
+      creds-aws-role-arn: ${{ vars.CREDS_CICD_AWS_PREP_OIDC_ROLE_ARN }} # use aws auth via oidc, if this repo supplies it
     secrets: inherit # keyrack firewall in .test.yml filters to declared keys
 
   prep:
@@ -48,14 +48,14 @@ jobs:
     needs: [test]
     # dont deploy on merge to main by default, since builds are expensive
     if: |
-      ((github.ref == 'refs/heads/main' && github.event_name != 'workflow_dispatch' && false) || github.event.inputs.stage == 'prep') &&
+      ((github.ref == 'refs/heads/main' && github.event_name != 'workflow_dispatch' && false) || github.event.inputs.access == 'prep') &&
       always() && (needs.test.result == 'success' || needs.test.result == 'skipped')
     with:
-      stage: prep
+      access: prep
+      github-environment: preparation # deployment:false keeps the gate and suppresses the PR-timeline spam
       build: development
-      github-environment: prep
       creds-aws-region: us-east-1
-      creds-aws-role-arn: arn:aws:iam::@declapract{variable.awsAccountId.prep}:role/@declapract{variable.projectName}-github-actions-prep
+      creds-aws-role-arn: ${{ vars.CREDS_CICD_AWS_PREP_OIDC_ROLE_ARN }}
     secrets:
       expo-token: ${{ secrets.EXPO_TOKEN }}
 
@@ -63,14 +63,14 @@ jobs:
     uses: ./.github/workflows/.deploy-expo.yml
     needs: [test]
     if: |
-      (startsWith(github.ref, 'refs/tags/') && github.event_name != 'workflow_dispatch') || (github.event.inputs.stage == 'prod') &&
+      ((startsWith(github.ref, 'refs/tags/') && github.event_name != 'workflow_dispatch') || github.event.inputs.access == 'prod') &&
       always() && (needs.test.result == 'success' || needs.test.result == 'skipped')
     with:
-      stage: prod
+      access: prod
       build: production
-      github-environment: prod
+      github-environment: ${{ (github.ref == 'refs/heads/main' || startsWith(github.ref, 'refs/tags/')) && 'production-on-main' || 'production-on-else-apply' }} # main/tag => on-main; else => on-else-apply (actor_id-gated writer)
       creds-aws-region: us-east-1
-      creds-aws-role-arn: arn:aws:iam::@declapract{variable.awsAccountId.prod}:role/@declapract{variable.projectName}-github-actions-prod
+      creds-aws-role-arn: ${{ vars.CREDS_CICD_AWS_PROD_OIDC_APPLY_ROLE_ARN }}
       app-web-deployment-s3-bucket: s3://@declapract{variable.projectName}-@declapract{variable.infrastructureNamespaceId}-prod
     secrets:
       expo-token: ${{ secrets.EXPO_TOKEN }}

package/dist/practices/cicd-app-react-native-expo/best-practice/.github/workflows/test.yml

@@ -20,8 +20,6 @@ jobs:
   suite:
     uses: ./.github/workflows/.test.yml
     with:
-      aws-region: us-east-1
-      aws-account-id: '@declapract{variable.awsAccountId.prep}'
-    secrets:
-      aws-access-key-id: ${{ secrets.PREP_AWS_ACCESS_KEY_ID }}
-      aws-secret-access-key: ${{ secrets.PREP_AWS_SECRET_ACCESS_KEY }}
+      creds-aws-region: us-east-1
+      creds-aws-role-arn: ${{ vars.CREDS_CICD_AWS_PREP_OIDC_ROLE_ARN }} # use aws auth via oidc, if this repo supplies it
+    secrets: inherit # keyrack firewall in .test.yml filters to declared keys

package/dist/practices/cicd-app-react-native-expo/best-practice/package.json

@@ -6,10 +6,10 @@
   },
   "scripts": {
     "start:hot:proxy": "npx local-ssl-proxy --source 3447 --target 8081",
-    "start:hot:prep": "STAGE=prep npx expo start --clear --dev-client",
+    "start:hot:prep": "ACCESS=prep npx expo start --clear --dev-client",
     "start:hot:prep:link": "echo '🚀 https://localhost.@declapract{variable.app.web.domain.prep}:@declapract{variable.app.web.port.hot}'",
     "start:hot:prep:proxy": "npm run start:hot:prep:link && npm run start:hot:proxy",
-    "start:hot:prod": "STAGE=prod npx expo start --clear --dev-client",
+    "start:hot:prod": "ACCESS=prod npx expo start --clear --dev-client",
     "start:hot:prod:link": "echo '🚀 https://localhost.@declapract{variable.app.web.domain.prod}:@declapract{variable.app.web.port.hot}'",
     "start:hot:prod:proxy": "npm run start:hot:prod:link && npm run start:hot:proxy",
     "start:cold:proxy": "npx local-ssl-proxy --source 3449 --target 3000",

package/dist/practices/cicd-common/.declapract.integration.test.ts

@@ -331,6 +331,31 @@ describe('cicd-common workflow templates', () => {
         expect(fileAfter.contents).not.toContain('@declapract{');
       });
 
+      /**
+       * .what = #595 — the PR acceptance shard defaults to IN-PROCESS (LOCALLY=true), so a
+       *         fleetless service (one that publishes only dev/prep, no `test` fleet) inherits a
+       *         green PR gate rather than 404ing against a `svc-<name>-test-*` slug that does not
+       *         exist. this is the delivered template, through the real pipeline — so it proves a
+       *         consumer receives the in-process default, not that the source file merely holds it.
+       * .teeth = both acceptance shard steps (explicit + dynamic) carry `LOCALLY=true`, and NO
+       *          acceptance step runs a bare `THOROUGH=true npm run test:acceptance` without it —
+       *          the cloud-invoke default that regressed a fleetless service to all-404. revert
+       *          either step to the bare command and the count assertions redden.
+       */
+      then('the acceptance shard defaults to in-process (LOCALLY=true) -- #595 fleetless gate', () => {
+        // every `test:acceptance` shard command carries LOCALLY=true (the explicit + both dynamic branches)
+        const acceptanceRuns = (
+          fileAfter.contents.match(/npm run test:acceptance/g) || []
+        ).length;
+        const localAcceptanceRuns = (
+          fileAfter.contents.match(/LOCALLY=true THOROUGH=true npm run test:acceptance/g) || []
+        ).length;
+        expect(acceptanceRuns).toBeGreaterThan(0);
+        // teeth: EVERY acceptance run carries LOCALLY=true — a bare cloud-invoke step (the all-404
+        // regression) would make localAcceptanceRuns < acceptanceRuns.
+        expect(localAcceptanceRuns).toEqual(acceptanceRuns);
+      });
+
       /**
        * .why = the wish's third acceptance criterion — "the pin is LEGIBLE: a reader can tell
        *        WHICH version each SHA corresponds to" — is the one criterion no assertion can

package/dist/practices/cicd-common/__snapshots__/.declapract.integration.test.ts.snap

@@ -338,16 +338,7 @@ jobs:
 
       - name: report slow tests
         if: always() && hashFiles('jest-results.json') != ''
-        run: |
-          set -eu
-          jq -r '.testResults[] | select(.perfStats.runtime != null) | "\\(.name):\\(.perfStats.runtime)"' jest-results.json | while IFS=: read -r name runtime; do
-            seconds=$((runtime / 1000))
-            if [[ $seconds -ge 30 ]]; then
-              echo "::warning file=\${name}::slow test: \${seconds}s"
-            elif [[ $seconds -ge 10 ]]; then
-              echo "::notice file=\${name}::test duration: \${seconds}s"
-            fi
-          done
+        run: bash .github/workflows/report-slow-tests.sh jest-results.json
 
   test-shards-acceptance:
     runs-on: ubuntu-24.04
@@ -402,12 +393,18 @@ jobs:
       - name: build
         run: npm run build
 
+      # .why = the PR acceptance gate runs IN-PROCESS (LOCALLY=true), not against a deployed fleet.
+      #        a PR has no deployed lambda to invoke, and a service that publishes only dev/prep (no
+      #        \`test\` fleet) would target \`svc-<name>-test-*\` — a slug that does not exist — and 404
+      #        on every call. in-process runs the handler directly, so a fleetless service inherits a
+      #        green PR gate out of the box rather than hand-patching LOCALLY=true into this shard.
+      #        the deployed-fleet acceptance run is a separate post-deploy grain (.deploy-sls.yml).
       - name: test:acceptance (explicit \${{ matrix.shard.index }})
         if: matrix.shard.type == 'explicit'
         run: |
           set -eu
           patterns='\${{ join(matrix.shard.patterns, '|') }}'
-          THOROUGH=true npm run test:acceptance -- --testPathPatterns="$patterns" --json --outputFile=jest-results.json
+          LOCALLY=true THOROUGH=true npm run test:acceptance -- --testPathPatterns="$patterns" --json --outputFile=jest-results.json
 
       - name: test:acceptance (dynamic \${{ matrix.shard.shard }}/\${{ matrix.shard.total }})
         if: matrix.shard.type == 'dynamic'
@@ -415,23 +412,14 @@ jobs:
           set -eu
           exclude='\${{ needs.enshard.outputs.acceptance-patterns }}'
           if [[ -n "$exclude" ]]; then
-            THOROUGH=true npm run test:acceptance -- --testPathIgnorePatterns="$exclude" --shard=\${{ matrix.shard.shard }}/\${{ matrix.shard.total }} --json --outputFile=jest-results.json
+            LOCALLY=true THOROUGH=true npm run test:acceptance -- --testPathIgnorePatterns="$exclude" --shard=\${{ matrix.shard.shard }}/\${{ matrix.shard.total }} --json --outputFile=jest-results.json
           else
-            THOROUGH=true npm run test:acceptance -- --shard=\${{ matrix.shard.shard }}/\${{ matrix.shard.total }} --json --outputFile=jest-results.json
+            LOCALLY=true THOROUGH=true npm run test:acceptance -- --shard=\${{ matrix.shard.shard }}/\${{ matrix.shard.total }} --json --outputFile=jest-results.json
           fi
 
       - name: report slow tests
         if: always() && hashFiles('jest-results.json') != ''
-        run: |
-          set -eu
-          jq -r '.testResults[] | select(.perfStats.runtime != null) | "\\(.name):\\(.perfStats.runtime)"' jest-results.json | while IFS=: read -r name runtime; do
-            seconds=$((runtime / 1000))
-            if [[ $seconds -ge 30 ]]; then
-              echo "::warning file=\${name}::slow test: \${seconds}s"
-            elif [[ $seconds -ge 10 ]]; then
-              echo "::notice file=\${name}::test duration: \${seconds}s"
-            fi
-          done
+        run: bash .github/workflows/report-slow-tests.sh jest-results.json
 
   # union job: aggregates shard results into single status for pr merge constraints
   test-integration:

package/dist/practices/cicd-common/__snapshots__/declastruct-insync-grep.declapract.integration.test.ts.snap

@@ -0,0 +1,3 @@
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
+
+exports[`cicd-common .declastruct.yml in-sync grep given: the shipped .declastruct.yml when: the apply-gate grep is read then: the emitted apply-gate grep line matches snapshot: declastruct apply-gate grep 1`] = `"if grep -qF "everything is in sync" ./plan.log"`;

package/dist/practices/cicd-common/best-practice/.github/workflows/.declastruct.yml

@@ -18,18 +18,18 @@ on:
         required: true
       scope:
         type: string
-        description: "the provider backend this wish targets (github | aws); scopes the plan artifact name"
+        description: "the provider backend this wish targets (github | aws)"
         required: true
       access:
         type: string
-        description: "the access tier this wish targets (test | prep | prod); read by the wish via sdk-environment as the ACCESS envar (aws wishes), and scopes the plan artifact name so parallel calls do not collide"
+        description: "the access to provision with (test | prep | prod)"
         required: true
       github-environment-plan:
         type: string
-        description: "github environment for the plan job (empty = no env gate)"
+        description: "github environment for the plan job"
       github-environment-apply:
         type: string
-        description: "github environment for the apply job (empty = no env gate)"
+        description: "github environment for the apply job"
       allow-apply:
         type: boolean
         description: "whether the apply step is enabled. defaults to true on main"
@@ -69,7 +69,12 @@ jobs:
 
   plan:
     runs-on: ubuntu-24.04
-    environment: ${{ inputs.github-environment-plan }}
+    environment:
+      # deployment: false keeps the built-in gate but suppresses the auto-Deployment, so a plan
+      # run posts no "deployed to <env>" status to the PR timeline. the environment is ALWAYS
+      # declared — for prep and prod alike.
+      name: ${{ inputs.github-environment-plan }}
+      deployment: false
     needs: [install]
     env:
       ACCESS: ${{ inputs.access }} # aws wishes: sdk-environment getEnvironment reads ACCESS
@@ -119,8 +124,12 @@ jobs:
       - name: evaluate plan
         id: evaluate-plan
         run: |
-          # declastruct prints this exact phrase when there is no diff
-          if grep "in sync" ./plan.log
+          # declastruct prints `🎉 everything is in sync!` (LOWERCASE) when there is no diff.
+          # grep -qF: fixed-string (no char read as regex), quiet (the exit code is the whole
+          # signal), and the full phrase so an unrelated line that holds "in sync" cannot
+          # false-match. the marker is lowercase — a capital-lead grep never matches it, so the
+          # fast path would be dead code (#561).
+          if grep -qF "everything is in sync" ./plan.log
           then
             echo "has-changes-planned=false" >> "$GITHUB_OUTPUT"
           else
@@ -142,7 +151,12 @@ jobs:
 
   apply:
     runs-on: ubuntu-24.04
-    environment: ${{ inputs.github-environment-apply }}
+    environment:
+      # deployment: false keeps the built-in gate but suppresses the auto-Deployment, so the apply
+      # run posts no "deployed to <env>" status to the PR timeline. the environment is ALWAYS
+      # declared — for prep and prod alike.
+      name: ${{ inputs.github-environment-apply }}
+      deployment: false
     needs: [install, plan]
     if: ${{ inputs.allow-apply == true && needs.plan.outputs.has-changes-planned == 'true' }}
     env:

package/dist/practices/cicd-common/best-practice/.github/workflows/.test.yml

@@ -252,16 +252,7 @@ jobs:
 
       - name: report slow tests
         if: always() && hashFiles('jest-results.json') != ''
-        run: |
-          set -eu
-          jq -r '.testResults[] | select(.perfStats.runtime != null) | "\(.name):\(.perfStats.runtime)"' jest-results.json | while IFS=: read -r name runtime; do
-            seconds=$((runtime / 1000))
-            if [[ $seconds -ge 30 ]]; then
-              echo "::warning file=${name}::slow test: ${seconds}s"
-            elif [[ $seconds -ge 10 ]]; then
-              echo "::notice file=${name}::test duration: ${seconds}s"
-            fi
-          done
+        run: bash .github/workflows/report-slow-tests.sh jest-results.json
 
   test-shards-acceptance:
     runs-on: ubuntu-24.04
@@ -316,12 +307,18 @@ jobs:
       - name: build
         run: npm run build
 
+      # .why = the PR acceptance gate runs IN-PROCESS (LOCALLY=true), not against a deployed fleet.
+      #        a PR has no deployed lambda to invoke, and a service that publishes only dev/prep (no
+      #        `test` fleet) would target `svc-<name>-test-*` — a slug that does not exist — and 404
+      #        on every call. in-process runs the handler directly, so a fleetless service inherits a
+      #        green PR gate out of the box rather than hand-patching LOCALLY=true into this shard.
+      #        the deployed-fleet acceptance run is a separate post-deploy grain (.deploy-sls.yml).
       - name: test:acceptance (explicit ${{ matrix.shard.index }})
         if: matrix.shard.type == 'explicit'
         run: |
           set -eu
           patterns='${{ join(matrix.shard.patterns, '|') }}'
-          THOROUGH=true npm run test:acceptance -- --testPathPatterns="$patterns" --json --outputFile=jest-results.json
+          LOCALLY=true THOROUGH=true npm run test:acceptance -- --testPathPatterns="$patterns" --json --outputFile=jest-results.json
 
       - name: test:acceptance (dynamic ${{ matrix.shard.shard }}/${{ matrix.shard.total }})
         if: matrix.shard.type == 'dynamic'
@@ -329,23 +326,14 @@ jobs:
           set -eu
           exclude='${{ needs.enshard.outputs.acceptance-patterns }}'
           if [[ -n "$exclude" ]]; then
-            THOROUGH=true npm run test:acceptance -- --testPathIgnorePatterns="$exclude" --shard=${{ matrix.shard.shard }}/${{ matrix.shard.total }} --json --outputFile=jest-results.json
+            LOCALLY=true THOROUGH=true npm run test:acceptance -- --testPathIgnorePatterns="$exclude" --shard=${{ matrix.shard.shard }}/${{ matrix.shard.total }} --json --outputFile=jest-results.json
           else
-            THOROUGH=true npm run test:acceptance -- --shard=${{ matrix.shard.shard }}/${{ matrix.shard.total }} --json --outputFile=jest-results.json
+            LOCALLY=true THOROUGH=true npm run test:acceptance -- --shard=${{ matrix.shard.shard }}/${{ matrix.shard.total }} --json --outputFile=jest-results.json
           fi
 
       - name: report slow tests
         if: always() && hashFiles('jest-results.json') != ''
-        run: |
-          set -eu
-          jq -r '.testResults[] | select(.perfStats.runtime != null) | "\(.name):\(.perfStats.runtime)"' jest-results.json | while IFS=: read -r name runtime; do
-            seconds=$((runtime / 1000))
-            if [[ $seconds -ge 30 ]]; then
-              echo "::warning file=${name}::slow test: ${seconds}s"
-            elif [[ $seconds -ge 10 ]]; then
-              echo "::notice file=${name}::test duration: ${seconds}s"
-            fi
-          done
+        run: bash .github/workflows/report-slow-tests.sh jest-results.json
 
   # union job: aggregates shard results into single status for pr merge constraints
   test-integration:

package/dist/practices/cicd-common/best-practice/.github/workflows/.test.yml.declapract.ts

@@ -1,4 +1,5 @@
 import { FileCheckType, type FileFixFunction } from 'declapract';
+import { MalfunctionError } from 'helpful-errors';
 
 /**
  * .what = validates .test.yml matches the template exactly
@@ -9,7 +10,17 @@ export const check = FileCheckType.EQUALS;
 /**
  * .what = replaces .test.yml with the template content
  * .why = ensures workflow has firewall step for keyrack secrets
+ * .note = fail-fast on absent declared contents. a declared file ALWAYS has content — declapract
+ *         reads the template beside this declaration. so a null/undefined `declaredFileContents`
+ *         is an internal invariant violation (a build wired wrong), never a valid state; a `?? ''`
+ *         would swallow it and write a ZERO-BYTE workflow to a consumer — a silent break the wish
+ *         forbids. throw loud so the misconfigured build surfaces, rather than ship an empty file.
  */
 export const fix: FileFixFunction = async (_contents, context) => {
-  return { contents: context.declaredFileContents ?? '' };
+  if (context.declaredFileContents === null || context.declaredFileContents === undefined)
+    throw new MalfunctionError(
+      'no declaredFileContents for .test.yml — declapract could not read the template beside its declaration',
+      { relativeFilePath: '.github/workflows/.test.yml' },
+    );
+  return { contents: context.declaredFileContents };
 };

package/dist/practices/cicd-common/best-practice/.github/workflows/report-slow-tests.sh

@@ -0,0 +1,65 @@
+#!/usr/bin/env bash
+######################################################################
+# .what = rank test files by wall-clock duration from a jest --json report
+#
+# .why  = jest 30 dropped the perfStats key that the prior report filtered on
+#         (`perfStats.runtime`), so the filter matched zero rows and the report
+#         went silent-green for its whole life (radio #602). endTime-startTime
+#         (epoch ms) is the only per-file duration jest 30 still emits, so read
+#         the wall-clock span instead — and fail LOUD if a future schema change
+#         zeroes it again, never silent-green.
+#
+# usage:
+#   report-slow-tests.sh [jest-results.json]
+#
+# guarantee:
+#   - exit 0 = report written to $GITHUB_STEP_SUMMARY (or /dev/null locally)
+#   - exit 0 = empty shard (0 test files) — a legitimate state, stated explicitly
+#   - exit 1 = non-empty report yielded zero real durations (schema drift)
+######################################################################
+set -eu
+
+results="${1:-jest-results.json}"
+
+total=$(jq '.testResults | length' "$results")
+durations=$(jq -r '.testResults[] | "\((.endTime // 0) - (.startTime // 0)):\(.name)"' "$results" | sort -t: -k1 -rn)
+
+summary="${GITHUB_STEP_SUMMARY:-/dev/null}"
+
+# an empty shard (no matched files) is legitimate — say so explicitly, never fail.
+if [[ "$total" -eq 0 ]]; then
+  echo "slow-test report: 0 test files in $results (empty shard)" >> "$summary"
+  exit 0
+fi
+
+# a non-empty testResults that yields zero real durations means the jest --json duration
+# schema changed again — the exact silent-death #602 fixed. fail loud, never silent-green.
+nonzero=$(printf '%s\n' "$durations" | awk -F: '$1 > 0' | wc -l | tr -d ' ')
+if [[ "$nonzero" -eq 0 ]]; then
+  echo "::error::slow-test report parsed 0 durations from $total test files — the jest --json duration schema likely changed (see radio #602)"
+  exit 1
+fi
+
+# a ranked table into the step summary — annotations cannot be sorted, and a diagnosis
+# needs the order (the single slowest file sets the suite's wall-clock floor).
+{
+  echo "### slowest test files"
+  echo ""
+  echo "| duration | file |"
+  echo "| --- | --- |"
+  printf '%s\n' "$durations" | while IFS=: read -r ms name; do
+    [ -z "$name" ] && continue
+    printf '| %ds | %s |\n' "$((ms / 1000))" "$name"
+  done
+} >> "$summary"
+
+# inline annotations flag the outliers at their file (the 30s/10s thresholds are inherited).
+printf '%s\n' "$durations" | while IFS=: read -r ms name; do
+  [ -z "$name" ] && continue
+  seconds=$((ms / 1000))
+  if [[ $seconds -ge 30 ]]; then
+    echo "::warning file=${name}::slow test: ${seconds}s"
+  elif [[ $seconds -ge 10 ]]; then
+    echo "::notice file=${name}::test duration: ${seconds}s"
+  fi
+done

package/dist/practices/cicd-common/declastruct-insync-grep.declapract.integration.test.ts

@@ -0,0 +1,49 @@
+import { readFileSync } from 'node:fs';
+import { join } from 'node:path';
+
+import { given, then, when } from 'test-fns';
+
+/**
+ * .what = clamps #561: the .declastruct.yml apply-gate greps the LOWERCASE marker declastruct
+ *         actually emits (`🎉 everything is in sync!`), so the in-sync fast path is live, not dead.
+ * .why  = declastruct's marker is lowercase (planChanges.ts: `context.log.info('🎉 everything is in
+ *         sync!')`). a capital-E `Everything is in sync` grep is case-sensitive and never matches, so
+ *         every plan falls to the has-changes-planned=true arm — a fast path dead its whole life. it
+ *         fails SAFE (over-gates), so no alarm ever fired; a clamp is the only thing that keeps the
+ *         grep tied to the tool's real output.
+ * .teeth = revert the grep to `grep "Everything is in sync"` (capital E) and the "greps the
+ *          lowercase marker" assertion reddens.
+ * .note = the subject is the shipped TEMPLATE, read from __dirname. this file sits at the practice
+ *         ROOT, so declapract never emits it (per the #583 relocation convention).
+ */
+describe('cicd-common .declastruct.yml in-sync grep', () => {
+  given('the shipped .declastruct.yml', () => {
+    const yml = readFileSync(
+      join(__dirname, 'best-practice/.github/workflows/.declastruct.yml'),
+      'utf8',
+    );
+
+    when('the apply-gate grep is read', () => {
+      // declastruct emits this literal phrase, lowercase; the workflow must match it verbatim
+      const markerLower = 'everything is in sync';
+      const markerCapital = 'Everything is in sync';
+
+      then('it greps the lowercase marker declastruct actually emits', () => {
+        expect(yml).toContain(`grep -qF "${markerLower}"`);
+      });
+
+      then('it does NOT grep the capital-E form that never matches (#561)', () => {
+        expect(yml).not.toContain(markerCapital);
+      });
+
+      then('the emitted apply-gate grep line matches snapshot', () => {
+        // snapshot the caller-faced contract itself, so a reword that keeps the
+        // marker token green (a re-ordered -qF, a re-phrased fallback) reddens a vibecheck
+        const grepLine = yml
+          .split('\n')
+          .find((line) => line.includes('grep -qF "'));
+        expect(grepLine?.trim()).toMatchSnapshot('declastruct apply-gate grep');
+      });
+    });
+  });
+});

package/dist/practices/cicd-common/report-slow-tests.play.declapract.integration.test.ts

@@ -0,0 +1,144 @@
+import { spawnSync } from 'node:child_process';
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+
+import { given, then, useBeforeAll, when } from 'test-fns';
+
+/**
+ * .what = integration test for the report-slow-tests shell the .test.yml workflow shells out to.
+ * .why  = the prior inline report filtered on `perfStats.runtime`, a key jest 30 dropped — so it
+ *         matched zero rows and went silent-green for its whole life (radio #602). this clamps the
+ *         go-forward: it reads endTime-startTime, ranks by wall-clock, and — the whole point of the
+ *         fix — fails LOUD if a future schema change zeroes the durations again, never silent-green.
+ * .how  = spawn the shipped best-practice/.github/workflows/report-slow-tests.sh against fixture
+ *         jest-results.json files, with GITHUB_STEP_SUMMARY pointed at a temp file, and assert the
+ *         exit code, the summary table, and the stdout annotations.
+ * .note = subject is the SHIPPED shell template. this file sits at the PRACTICE ROOT (not under
+ *         best-practice/), so declapract's loader — which walks only best-practice/ for templates —
+ *         never reads it as a declaration and never copies it to a consumer. it is the play-test
+ *         counterpart to husky/check.timestamps.play.declapract.integration.test.ts.
+ */
+
+// the shell under test (the distributed best-practice template)
+const scriptPath = path.join(
+  __dirname,
+  'best-practice/.github/workflows/report-slow-tests.sh',
+);
+
+// write a fixture jest-results.json + a fresh summary sink into a temp dir; return their paths
+const genFixture = (
+  jestResults: unknown,
+): { resultsPath: string; summaryPath: string } => {
+  const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'report-slow-tests-'));
+  const resultsPath = path.join(dir, 'jest-results.json');
+  const summaryPath = path.join(dir, 'step-summary.md');
+  fs.writeFileSync(resultsPath, JSON.stringify(jestResults));
+  fs.writeFileSync(summaryPath, '');
+  return { resultsPath, summaryPath };
+};
+
+// spawn the report against a fixture; return exit code + stdout + the summary file contents
+const runReport = (
+  jestResults: unknown,
+): { code: number | null; stdout: string; summary: string } => {
+  const { resultsPath, summaryPath } = genFixture(jestResults);
+  const res = spawnSync('bash', [scriptPath, resultsPath], {
+    encoding: 'utf-8',
+    env: { ...process.env, GITHUB_STEP_SUMMARY: summaryPath },
+  });
+  return {
+    code: res.status,
+    stdout: res.stdout,
+    summary: fs.readFileSync(summaryPath, 'utf-8'),
+  };
+};
+
+describe('report-slow-tests', () => {
+  given('[case1] a report with a slow (>30s), a medium (>10s), and a fast file', () => {
+    when('[t0] the report runs', () => {
+      const result = useBeforeAll(() =>
+        runReport({
+          testResults: [
+            { name: '/repo/src/medium.test.ts', startTime: 1000, endTime: 16000 }, // 15s
+            { name: '/repo/src/fast.test.ts', startTime: 1000, endTime: 3000 }, // 2s
+            { name: '/repo/src/slow.test.ts', startTime: 1000, endTime: 41000 }, // 40s
+          ],
+        }),
+      );
+
+      then('it exits 0', () => {
+        expect(result.code).toEqual(0);
+      });
+
+      then('the summary carries the ranked table', () => {
+        expect(result.summary).toContain('### slowest test files');
+        expect(result.summary).toContain('| duration | file |');
+      });
+
+      then('the slowest file is ranked first in the table', () => {
+        const slowIdx = result.summary.indexOf('/repo/src/slow.test.ts');
+        const mediumIdx = result.summary.indexOf('/repo/src/medium.test.ts');
+        const fastIdx = result.summary.indexOf('/repo/src/fast.test.ts');
+        expect(slowIdx).toBeGreaterThan(-1);
+        expect(slowIdx).toBeLessThan(mediumIdx);
+        expect(mediumIdx).toBeLessThan(fastIdx);
+      });
+
+      then('it emits a warn annotation for the >30s file', () => {
+        expect(result.stdout).toContain(
+          '::warning file=/repo/src/slow.test.ts::slow test: 40s',
+        );
+      });
+
+      then('it emits a notice annotation for the >10s file', () => {
+        expect(result.stdout).toContain(
+          '::notice file=/repo/src/medium.test.ts::test duration: 15s',
+        );
+      });
+
+      then('it emits no annotation for the fast file', () => {
+        expect(result.stdout).not.toContain('/repo/src/fast.test.ts::');
+      });
+    });
+  });
+
+  given('[case2] a non-empty report whose durations are all zero (schema drift)', () => {
+    // the exact silent-death #602 fixed: testResults present, but no real duration to be read.
+    when('[t0] the report runs', () => {
+      const result = useBeforeAll(() =>
+        runReport({
+          testResults: [
+            { name: '/repo/src/a.test.ts', startTime: 1000, endTime: 1000 },
+            { name: '/repo/src/b.test.ts', startTime: 2000, endTime: 2000 },
+          ],
+        }),
+      );
+
+      then('it fails loud with exit 1 (never silent-green)', () => {
+        expect(result.code).toEqual(1);
+      });
+
+      then('it prints an error annotation that names the schema drift', () => {
+        expect(result.stdout).toContain('::error::');
+        expect(result.stdout).toContain('jest --json duration schema');
+        expect(result.stdout).toContain('#602');
+      });
+    });
+  });
+
+  given('[case3] an empty shard (0 test files)', () => {
+    when('[t0] the report runs', () => {
+      const result = useBeforeAll(() => runReport({ testResults: [] }));
+
+      then('it exits 0 (an empty shard is legitimate)', () => {
+        expect(result.code).toEqual(0);
+      });
+
+      then('the summary states the empty-shard state explicitly', () => {
+        expect(result.summary).toContain('0 test files');
+        expect(result.summary).toContain('empty shard');
+      });
+    });
+  });
+});

package/dist/practices/cicd-package/best-practice/.github/workflows/provision.yml

@@ -25,7 +25,8 @@ jobs:
       wish-path: provision/github.repo/resources.ts
       scope: github
       access: prod # github repo settings are inherently a prod resource
-      github-environment-apply: prod
+      github-environment-plan: production-on-else-plan
+      github-environment-apply: ${{ (github.ref == 'refs/heads/main' || startsWith(github.ref, 'refs/tags/')) && 'production-on-main' || 'production-on-else-apply' }} # main/tag => on-main; else => on-else-apply
       creds-github-app-owner: ${{ github.repository_owner }}
       creds-github-app-id: ${{ vars.DECLASTRUCT_GITHUB_CONFORMER_APP_ID }}
     secrets: