Skip to content

Commit e962be9

Browse files
JuliaEdomclaude
andcommitted
docs: retire hedges about work that has since been done
A caveat that has stopped being true does not read as harmless history; it reads as an open task. Three of them survived the 2026-09-08 push. The provider-lock step in terraform-validate carried "It has not been observed on a GitHub runner until the owner pushes." It has now, on ubuntu-latest: all three platforms regenerate to an empty diff, which is the whole point of the guard. ASSUMPTION-T-36-CI-LOCK in the OIDC runbook is marked discharged against that run and narrowed to what is still unobserved -- the darwin_arm64 and windows_amd64 entries are regenerated, never consumed, because CI has no runner on either. The sdk-ts job was annotated as still needing branch-protection wiring. It is one of the 15 required contexts, so the comment now says what that implies instead: renaming the job removes a gate silently, because a required context that never reports blocks merges rather than failing them. PROJECT_CLOSURE.md and release-readiness.md called cf247ba local-only and unpushed; it has been in origin/main since the push. Dated records under docs/evidence/ and docs/perf/ keep their wording -- they describe a moment, not the present. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017tNEcJFMjms4uMVwERcT5x
1 parent 3820a2f commit e962be9

5 files changed

Lines changed: 41 additions & 12 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -606,7 +606,9 @@ jobs:
606606
# same three-platform command the module README documents (re-run
607607
# 2026-09-05: no lock rewrite). Three provider zips (~0.7 GB) plus
608608
# terraform init is why this job's timeout-minutes is 20, not 10.
609-
# It has not been observed on a GitHub runner until the owner pushes.
609+
# Observed green on ubuntu-latest 2026-09-08 (run 34213482386): the step
610+
# regenerates all three platforms and the diff comes back empty, so the
611+
# tracked lock genuinely covers the runner it is used on.
610612
- name: Provider lock covers linux_amd64
611613
run: |
612614
cd infrastructure/terraform
@@ -619,8 +621,11 @@ jobs:
619621
# only in publish-npm.yml (tag pushes) and mutation.yml; every PR only got
620622
# `npm audit` (security.yml). This job gives PRs the same build-shaped
621623
# gate the publish workflow already trusts, so a broken SDK cannot merge.
622-
# Branch protection required-context wiring is a repo-settings change the
623-
# owner still has to make (gh api repos/{owner}/{repo}/branches/main/protection).
624+
# `sdk-ts` is one of the 15 required status checks on `main` (confirmed
625+
# 2026-09-08 via gh api repos/{owner}/{repo}/branches/main/protection), so
626+
# renaming this job silently removes a gate: the context stops reporting,
627+
# and a required check that never reports blocks merges instead of failing
628+
# them. Rename here and in branch protection together, or not at all.
624629
runs-on: ubuntu-latest
625630
timeout-minutes: 15
626631
defaults:

‎CHANGELOG.md‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,26 @@ All notable changes to AgentFlow are documented in this file.
44

55
## [Unreleased]
66

7+
### Docs — the pre-push hedges outlived the push
8+
9+
* **Several notes described work the owner "still has to do" that has since
10+
been done**, and a hedge that has stopped being true reads as a live task.
11+
The provider-lock guard in `terraform-validate` was annotated "has not been
12+
observed on a GitHub runner"; it has, on `ubuntu-latest`, regenerating all
13+
three platforms to an empty diff. `ASSUMPTION-T-36-CI-LOCK` in the OIDC
14+
runbook is marked discharged with that evidence, and narrowed to what is
15+
still genuinely unobserved: the macOS and Windows entries are regenerated
16+
identically, never actually consumed, because CI has no runner on either.
17+
* **`sdk-ts` is a required status check now**, not a wiring change waiting to
18+
happen, so the comment above the job says what that costs: renaming the job
19+
silently removes a gate, because a required context that never reports blocks
20+
merges rather than failing them. Rename it and branch protection together.
21+
* **Two closure documents called a pushed evidence commit local-only.**
22+
`cf247ba` has been in `origin/main` since the 2026-09-08 push;
23+
`PROJECT_CLOSURE.md` and `release-readiness.md` say so now. Dated evidence
24+
records under `docs/evidence/` and `docs/perf/` keep their original wording —
25+
they are chronology of a moment, not claims about the present.
26+
727
### Mutation gate — nine weeks red from a rename ripple and two untested one-liners
828

929
* **The weekly Mutation Testing workflow last passed on 2026-07-05** and failed

‎docs/PROJECT_CLOSURE.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ Push, PR mutations and release actions require explicit owner authorization.
126126
[perf/live-iceberg-materialization-2026-08-01.md](perf/live-iceberg-materialization-2026-08-01.md).
127127
Full one-event lake-to-serving smoke is also **PASS** — see
128128
[perf/full-lake-to-serving-e2e-2026-08-01.md](perf/full-lake-to-serving-e2e-2026-08-01.md)
129-
(local evidence commit `cf247ba`, unpushed).
129+
(evidence commit `cf247ba`, in `origin/main` since the 2026-09-08 push).
130130
Isolated checkpoint restore/replay is **PASS** with distinct J1/J2,
131131
savepoint restore linkage, exact-once E1/E2 counts across Kafka, Iceberg,
132132
ClickHouse, and API, DLQ `0`, and source lag `0` — see

‎docs/operations/aws-oidc-setup.md‎

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -197,13 +197,17 @@ contract is honest, the live delivery path is not claimed):
197197
`plan`, or `apply` ran against the real S3 backend. DynamoDB locking is
198198
retained on the tracked configuration; a live-backend migration to
199199
`use_lockfile` still needs AWS access.
200-
- **ASSUMPTION-T-36-CI-LOCK**: the provider-lock step in
201-
`.github/workflows/ci.yml` `terraform-validate` runs
200+
- **ASSUMPTION-T-36-CI-LOCK** — discharged 2026-09-08. The provider-lock
201+
step in `.github/workflows/ci.yml` `terraform-validate` runs
202202
`terraform providers lock -platform=linux_amd64 -platform=darwin_arm64
203-
-platform=windows_amd64` and `git diff --exit-code .terraform.lock.hcl`.
204-
It is a tracked guard for `linux_amd64` on `ubuntu-latest` (and the other
205-
two documented platforms), not observed evidence. It cannot have passed
206-
on a GitHub runner until the owner pushes.
203+
-platform=windows_amd64` followed by `git diff --exit-code
204+
.terraform.lock.hcl`. It was a tracked guard with no runner behind it until
205+
the work reached `origin`; it has since passed on `ubuntu-latest` (run
206+
34213482386, `terraform-validate` success, step "Provider lock covers
207+
linux_amd64"), which is the evidence that the tracked lock covers the
208+
platform CI actually runs on. What remains unobserved is only the
209+
`darwin_arm64` / `windows_amd64` halves being *used*, as opposed to being
210+
regenerated identically — CI has no runner on either.
207211

208212
## State-key and role scope
209213

‎docs/release-readiness.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,8 @@ Read-only GitHub Environment `npm` verification recorded at
9292
`2026-08-03T03:18:11Z` is **PASS**: the Environment exists and its non-empty
9393
`required_reviewers` rule names user `brownjuly2003-code` — see
9494
[operations/npm-environment-approval-2026-08-03.md](operations/npm-environment-approval-2026-08-03.md).
95-
Tracked full-smoke evidence is recorded in local evidence commit `cf247ba`
96-
(local-only, unpushed).
95+
Tracked full-smoke evidence is recorded in evidence commit `cf247ba`, which
96+
reached `origin/main` in the 2026-09-08 push.
9797

9898
## Summary
9999

0 commit comments

Comments
 (0)