Skip to content

Housekeeping: branch cleanup, kit sync, tracker sync - #359

Merged
The-Running-Dev merged 6 commits into
mainfrom
sync-kit-2026-08-21
Aug 21, 2026
Merged

Housekeeping: branch cleanup, kit sync, tracker sync#359
The-Running-Dev merged 6 commits into
mainfrom
sync-kit-2026-08-21

Conversation

@The-Running-Dev

@The-Running-Dev The-Running-Dev commented Aug 21, 2026

Copy link
Copy Markdown
Owner

Summary

Verified

Ran and passed:

  • Typecheck — npm --prefix src/engine run typecheck, exit 0
  • Lint — npm --prefix src/engine run lint, exit 0
  • Test — npm --prefix src/engine test (vitest) — 79/79 test files, 1127/1127 tests
  • Pack package — npm pack --silent produced the-running-dev-game-engine-0.10.0.tgz
  • Inspect tarball — no src/, no tsconfig*.json, no .test.* entries; package/dist/ present
  • Consumer smoke — clean install/build/smoke against the packed tarball
  • Parse-check PowerShell scripts — every tools/*.ps1 parses clean
  • Validate Markdown links, terminology, and generated files — ./build/Test-Documentation.ps1: drift check passed across 18 generated engine pages, compatibility pointers and the guide; 123 Markdown files checked
  • Build and verify landing page — npm --prefix src/engine run build + npm --prefix site run check (format/lint/typecheck/test/browser/build/merge) all green

Ran and failed:

  • Run Pester tests — Invoke-Pester -Path tools: 241/253 passed, 12 failed. All 12 are in the newly kit-synced Read-DesignState.Tests.ps1/Test-DesignState.Tests.ps1/Test-CIWorkflow.Tests.ps1/Update-DesignProjection.Tests.ps1 — self-referential tests (Split-Path $PSScriptRoot -Parent) that validate the agent-kit's own dogfooded design/state/ per-unit-record mechanism and its verify.yml shape against whatever repo they're copied into. This repo's design/ uses 00-brief/10-design/20-contract/30-slices/90-decisions, not the kit's design/state/ scheme, so these fail structurally (StateSetAbsent) rather than from a regression — the same gap Test-DesignState.ps1 itself reports as exit 2 when run directly here. Full failing-test list and detail in .claude/verify-report.json.

Did not run:

  • Build documentation — docs.ps1 is absent from this checkout (CLAUDE.md: installer-generated via Invoke-SetupDocs, not committed); needs Docker and an installed docs.ps1, neither set up this session
  • Merge landing page into documentation build — depends on the artifacts/docs output of "Build documentation" above, which didn't run
  • Test the host — dotnet test needs NUGET_GITHUB_TOKEN for the sibling-repo NuGet feed, not set locally
  • Positive route and probe smoke — needs a built host image, blocked on the same missing token
  • Negative fixture (corrupted artifact must fail to start) — same

Agent detail
  • Not a slice — housekeeping (branch cleanup, /kit-sync, tracker sync) plus the doc-drift fix it surfaced
  • .claude/verify-report.json: validated by tools/Test-VerifyReport.ps1 (Valid, 15 gates, 0 findings, 0 could-not-evaluate) before this section was rendered from it
  • Left undone: the kit's self-referential Pester tests (12 failures) — whether they should be scoped out of non-dogfooding targets is an open question, not resolved here

Slices W78-W83 all merged (issues #326-#331, PRs #340-#348) but the
tracker still showed them not started.
Qodo is retired; automated PR review now runs via the
anthropics/claude-code-action GitHub Action.
Synced from af610a6 to e9e21bf (9 commits): adds
/install-code-review-agent, the codex profile launcher and its tests,
the design-state read/check/projection scripts and their tests, and
updates the existing command cores and Sync-Kit/Test-Companion tooling.

verify.md's core was Unmigrated-Blocked (this repo's gate table had no
companion) — moved the table into verify-local.md under the
gate-commands category the core declares, so the sync could take the
core outright. Recorded in design/90-decisions.md.
…cal gate table

engine/TODO.md and its compatibility pointer were stale against
design/30-slices.md after the tracker sync — build/Test-Documentation.ps1
caught it. Also updated verify-local.md's gate-commands table: verify.yml
no longer flags Test-Companion.ps1 or Test-DesignState.ps1 as CI gates.
15 discovered gates: 9 passed, 1 failed (Pester — the kit's
self-referential design/state/ tests, not applicable to this repo's
design/ shape), 5 did not run (docs.ps1 absent, NUGET_GITHUB_TOKEN
unavailable locally). See PR #359's Verified section for the rendered
report.
Cores taken outright: install-all.md, install-code-review-agent.md,
kit-sync.md pick up the phase-4-step-8 wording sync from kit commits
1eca3dc/adcc76c/1bcfe36.

The kit's own #109/#110 fix for the design-state self-tests bundles an
S12.7 regression-guard block that assumes design/state exists in
whatever repo the file runs in - this repo doesn't have it, so taking
that file verbatim would fail S12.7's own assertion here. Kept this
repo's existing equivalent -Skip: fix (already recorded in
design/90-decisions.md, 2026-08-21) instead of overwriting it with the
kit's file.

kit.json commit bumped to 811bc96; syncedCommit already advanced by
Sync-Kit.ps1.
@The-Running-Dev
The-Running-Dev merged commit 36e7bcc into main Aug 21, 2026
8 checks passed
@The-Running-Dev
The-Running-Dev deleted the sync-kit-2026-08-21 branch August 21, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant