Reconcile — W84-W87 done statuses, onStart resolution, work mirror sync - #373
Merged
Conversation
Marks W84-W87 done with their merged PRs (#360-363), rewrites the 90-decisions.md onStart open item now that W84 (not W47) resolved it, and refreshes the design/state/work/ tracker mirror against HEAD.
build/ConvertTo-HumanDocumentation.ps1 was skipped after the prior commit's edits to design/30-slices.md and design/90-decisions.md; this regenerates engine/TODO.md and engine/OPEN-QUESTIONS.md so build/Test-Documentation.ps1 passes.
… not caused by tracker mirror
The-Running-Dev
added a commit
that referenced
this pull request
Aug 24, 2026
**What changed, and why.** Trailing sync from the prior reconcile commit (#373, `7c86a22`): bumps the `MirroredAt` stamp on all 48 `design/state/work/*.md` files to that SHA. No content changes — the reconcile itself was correct, this commit just wasn't pushed before the session that made it ended. ### Verified Ran and passed: - Typecheck — npm --prefix src/engine run typecheck — tsc --noEmit && tsc --noEmit -p tsconfig.scripts.json — exit 0 - Lint — npm --prefix src/engine run lint — eslint src scripts — exit 0 - Test — npm --prefix src/engine test (vitest run) — Test Files 79 passed (79), Tests 1174 passed (1174), Duration 5.10s - Pack package — cd src/engine; npm pack --silent — produced the-running-dev-game-engine-0.10.0.tgz, exit 0 - Inspect tarball — tar -tzf the-running-dev-game-engine-0.10.0.tgz: grep for ^package/src/|tsconfig.*\.json|\.test\. found NONE; package/dist/ present — PASS: tarball inspection clean - Consumer smoke — consumer-smoke: npm run install:engine && npm run build && npm run smoke — all three exited 0 (install:engine up to date, tsc build clean, node dist/smoke.js exit 0) - Parse-check PowerShell scripts — Parsed every *.ps1 repo-wide (excluding node_modules) with [System.Management.Automation.Language.Parser]::ParseFile — Checked 32 files, PASS: all PowerShell scripts parse cleanly - Validate Markdown links, terminology, and generated files — ./build/Test-Documentation.ps1 — "Human-documentation drift check passed across 18 generated engine page(s), compatibility pointers, and the guide. Documentation checks passed across 173 Markdown file(s)." - Build and verify landing page — npm --prefix src/engine run build (tsc -p tsconfig.build.json) exit 0, then npm --prefix site run check: unit tests (Test Files 2 passed, Tests 15 passed), test:browser (Test Files 1 passed, Tests 11 passed), test:build (vite build succeeded, "Both built HTML entry points contain their required static metadata, and 3 bundle(s) are free of Node-only runtime references"), test:merge ("Package-backed merge proven: /, /roadmap/ and the protected docs/ subtree all verified, and the top-level docs/ guard rejects a bad landing build") — all exited 0 - Test the host — dotnet test src/host/SubZeroDev.GameEngine.Host.Tests — all projects up-to-date for restore — Passed: 12, Failed: 0, Skipped: 0, Total: 12, Duration 351 ms Ran and failed: - Run Pester tests — Invoke-Pester -Path tools -Output Detailed -PassThru — Tests Passed: 243, Failed: 10, Skipped: 0, NotRun: 0; plus 1 BeforeAll/AfterAll block failure ("CI workflow: the Run Pester tests step is authenticated (#79)" — ParameterBindingValidationException: Cannot bind argument to parameter 'LiteralPath' because it is null, at Test-CIWorkflow.Tests.ps1:26). Identical count and identical failure set to the run recorded against this same branch's predecessor state earlier today (verify-report.json history, commit 8128a0f), which already traced these as pre-existing design-state drift, not caused by this branch's design/state/work/ MirroredAt-only edits: (1) "S4.6: unit/command/track and unit/document/agents-md exist and their closures are complete" — Expected $true, got $false. (2) "the Run Pester tests step carries a GH_TOKEN env, the same as Check the design state against the tree" — pre-existing, also fails on main. (3) "this repository's own table and its own enumeration agree" — Expected $null or empty, got ContractListUnreadable/GlobTableNotFound: design/20-contract.md. (4) "S12.5: the check exits 0 against this repository, and names the largest closure and its size" — Expected 0, got 2; dozens of [UnrecordedArtifact]/[LogEntryUnrecorded] entries, [RegionMalformed] in five design/ files, [ContractListUnreadable], [ProjectorFailed] exited 1, [TrackerUnavailable] for all 48 mirrored issues. (5) "S7.9: the projector runs against this repository and ProjectionStale does not fire" — ProjectorFailed/exited 1. (6) "S16.1/S16.2: one Contract record per Public-surface entry" — Expected 9, got 0. (7) "S16.5: design/state-index.md's consumers region lists real consumers" — Cannot find path design/state-index.md. (8) "S17.2: every invariant row sits inside the single invariants region" — Expected actual value greater than -1, got -1. (9) "S18.6: EnforcementUnevidenced rejects this repository's own superseded decision" — Cannot find path design/state/decisions/2026-08-03-ticking-checkbox-is-the-users.md. (10) "S7.2: -DryRun against the real repository writes nothing" — Expected 0, got 7 refusals (same set as #4). This branch does not touch design/30-slices.md, design/90-decisions.md, design/20-contract.md, or any tools/*.ps1 — only design/state/work/*.md MirroredAt stamps — so it cannot be the cause of this pre-existing drift; this command does not fix it. Filed as #375. Did not run: - Build documentation — Runs inside the ghcr.io/the-running-dev/docs-template container image via docs.ps1. docs.ps1 (the local wrapper) is absent from this checkout — CLAUDE.md states it is installer-generated (Invoke-SetupDocs), not committed. Docker itself is running locally, but the wrapper script needed to invoke the containerized build is missing, so this was not attempted; the corresponding CI check ("Verify Documentation Build") on the pull request is where the answer will come from. - Merge landing page into documentation build — npm --prefix site run merge needs a completed docs build in artifacts/docs, which the "Build documentation" gate above did not produce (docs-template container image not available locally without the docs.ps1 wrapper). - Positive route and probe smoke — Requires building the host Docker image, which restores SubZeroDev.Platform.Hosting from a private GitHub Packages NuGet feed inside the build via a docker/build-push-action secret (nuget_github_token). REGISTRY_TOKEN is not present in this local environment, so the image build step itself cannot be reproduced locally even though Docker is running; the corresponding CI check ("Build, run, and smoke the image") on the pull request is where the answer will come from. - Negative fixture -- corrupted artifact must fail to start — Same dependency as "Positive route and probe smoke": requires the built host image, blocked on the missing REGISTRY_TOKEN for the private NuGet feed restore. --- <details><summary><b>Agent detail</b></summary> <!-- agent:start --> - **Origin:** mechanical follow-up found uncommitted while starting `/slice`; not tied to a slice or issue. <!-- agent:end --> </details>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed, and why. Marks W84-W87 in
design/30-slices.mdas done against theirmerged PRs (#360-363), and rewrites the
onStartbuilding-meter open item indesign/90-decisions.md— it named W47 as the unit that would decide the question, butW84's incidents family is what actually resolved it, calling
applyWorldEffectsfrom asingle system-16/17 site and extending the §9.2 wear-delta rejection to
onStart. Alsorefreshes the
design/state/work/WorkReftracker mirror againstHEAD, per/track'sconvention as the mirror's sole writer. A follow-up commit regenerates
engine/TODO.mdand
engine/OPEN-QUESTIONS.md, which the first commit's edits left stale.Verified
Ran and passed:
npm --prefix src/engine run typecheck, exit 0npm --prefix src/engine run lint, exit 0npm --prefix src/engine test(vitest), 79 files / 1174 tests passednpm pack --silent, produced the tarball cleanlysrc/, notsconfig*.json, no.test.*;package/dist/present./build/Test-Documentation.ps1passes after the second commit's regeneration: "Human-documentation drift check passed across 18 generated engine page(s), compatibility pointers, and the guide. Documentation checks passed across 173 Markdown file(s)."dotnet testagainstsrc/host, 12/12 passedRan and failed:
main). The other 9 do not reproduce onmain(confirmed via a disposable worktree:mainshows only the 1 pre-existing failure) and persist even withdesign/state/work/removed from the tree, so the mirror files are ruled out as the cause — the remaining candidate is the content edited indesign/30-slices.md/design/90-decisions.md, not yet isolated further. Failures:S4.6(a command/document closure check returns false),S16.1/S16.2(expects 9 Contract records againstdesign/20-contract.md's Public-surface entries, finds 0),S16.5andS18.6(referencedesign/state-index.mdanddesign/state/decisions/2026-08-03-...md, neither of which exists in this tree),S7.9's projector exits 1,S12.5andS7.2both surface the same large[UnrecordedArtifact]/[LogEntryUnrecorded]/[RegionMalformed]finding set (the last of which flags ahuman-docregion closing-marker mismatch in five design files, three of which this PR does not touch), a table/enumeration disagreement (ContractListUnreadable/GlobTableNotFoundagainstdesign/20-contract.md), and[TrackerUnavailable]for all 48 mirrored issues despitegh auth statusshowing an authenticated session andgh issue view 142succeeding standalone. Full detail in.claude/verify-report.json. This looks like pre-existing design-state drift the branch's content edits expose rather than something the two-line status/decision changes themselves broke, but that is not confirmed — isolating which specific edit trips which check needs more investigation than fits this pass.Did not run:
docs.ps1(the installer-generated local wrapper) is absent from this checkout; the containerized build was not attempted. See CI's "Verify Documentation Build" check.artifacts/docs.REGISTRY_TOKENto restore a private NuGet feed; not set locally. See CI's "Build, run, and smoke the image" check.REGISTRY_TOKENdependency as above.Agent detail
against merged PRs W84 — Incidents That Happen On Their Own #360-363; design/state/work/ mirror refresh; human-doc regeneration