Apply Sample A server containerization artifacts and finalize Track U checklist states without engine-core changes.

DavidQ · DavidQ · commit a49176f17100 · 2026-04-06T15:42:42.000-04:00
diff --git a/docs/dev/CODEX_COMMANDS.md b/docs/dev/CODEX_COMMANDS.md
@@ -2,22 +2,15 @@ MODEL: GPT-5.4-codex
 REASONING: high
 
 COMMAND:
-Create BUILD_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION
+Create APPLY_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION
 
 Requirements:
-- Follow PLAN_PR + BUILD_PR + APPLY_PR
-- This is code/runtime work, not documentation-only work
-- Use the BUILD doc as the exact blueprint
-- One PR per purpose
+- This is acceptance/finalization work after the BUILD step already wrote code
+- Do not broaden scope
+- Keep Sample A server containerization only
 - No engine core changes
-- Keep scope to Sample A server containerization only
-- Create/update:
-  - games/network_sample_a/server/Dockerfile
-  - games/network_sample_a/.dockerignore
-  - games/network_sample_a/server/docker-compose.yml
-  - games/network_sample_a/server/README.md
-  - games/network_sample_a/server/networkSampleADashboardServer.mjs only if required for env-safe host/access compatibility
+- Confirm code artifacts exist and validate them
 - Update docs/roadmaps/NETWORK_SAMPLES_PLAN.md with bracket-only edits only
 - Do not overwrite BIG_PICTURE_ROADMAP.md
 - Do not overwrite PRODUCTIZATION_ROADMAP.md
-- Package repo-structured delta zip to <project folder>/tmp/BUILD_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION_delta.zip
+- Package repo-structured delta zip to <project folder>/tmp/APPLY_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION_delta.zip
diff --git a/docs/dev/COMMIT_COMMENT.txt b/docs/dev/COMMIT_COMMENT.txt
@@ -1 +1 @@
-Build Sample A server containerization artifacts and update Track U roadmap states without engine-core changes.
+Apply Sample A server containerization artifacts and finalize Track U checklist states without engine-core changes.
diff --git a/docs/dev/NEXT_COMMAND.txt b/docs/dev/NEXT_COMMAND.txt
@@ -1 +1 @@
-APPLY_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION
+PLAN_PR_DEBUG_SURFACES_SERVER_RUNTIME_ISOLATION
diff --git a/docs/dev/reports/change_summary.txt b/docs/dev/reports/change_summary.txt
@@ -1,35 +1,25 @@
 # Change Summary
 
 ## Bundle
-BUILD_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION_delta
+APPLY_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION_delta
 
-## Runtime Work Applied
-- Updated `games/network_sample_a/.dockerignore` to a strict allowlist pattern aligned with Dockerfile copy paths.
-- Updated `games/network_sample_a/server/README.md` with explicit stdout/stderr-only logging expectation.
-- Kept existing Sample A server runtime/container files aligned to the BUILD blueprint without broad refactors.
+## Intent
+Docs-only APPLY bundle for accepting and finalizing the already-completed Codex server containerization build.
 
-## Roadmap Update (Bracket-Only)
-- `docs/roadmaps/NETWORK_SAMPLES_PLAN.md` Track U items set to `[x]`:
-  - Dockerfile for server
-  - .dockerignore
-  - Environment variable contract
-  - Local run command
-  - Compose-ready service definition
-  - Port mapping rules
-  - Health/readiness check
-  - Logging/output expectations
-  - Container debug notes
+## Acceptance Validation
+- Confirmed artifact presence:
+  - `games/network_sample_a/server/Dockerfile`
+  - `games/network_sample_a/.dockerignore`
+  - `games/network_sample_a/server/docker-compose.yml`
+  - `games/network_sample_a/server/README.md`
+  - `games/network_sample_a/server/networkSampleADashboardServer.mjs`
+- Required check passed:
+  - `node --check games/network_sample_a/server/networkSampleADashboardServer.mjs`
+- Docker CLI is unavailable in this environment, so image build/compose runtime checks could not be executed here.
 
-## Guardrails Confirmed
-- No engine core changes
-- Scope limited to Sample A server containerization
-- No overwrite of `BIG_PICTURE_ROADMAP.md`
-- No overwrite of `PRODUCTIZATION_ROADMAP.md`
-
-## Validation Highlights
-- `node --check games/network_sample_a/server/networkSampleADashboardServer.mjs` passed.
-- Local smoke run passed:
-  - `/admin/network-sample-a/health` returned 200
-  - `/admin/network-sample-a/api/metrics?key=sample-a-admin` returned 200
-  - unauthenticated `/admin/network-sample-a/api/metrics` returned 403
-- Docker CLI is unavailable in this environment, so image build validation could not be executed here.
+## Scope And Roadmap
+- Scope stayed on Sample A server containerization acceptance/finalization only.
+- No engine/core files were modified in this APPLY working delta.
+- `docs/roadmaps/NETWORK_SAMPLES_PLAN.md` Track U checklist states are complete (`[x]`) and remain bracket-only.
+- `docs/roadmaps/BIG_PICTURE_ROADMAP.md` was not overwritten.
+- `docs/roadmaps/PRODUCTIZATION_ROADMAP.md` was not overwritten.
diff --git a/docs/dev/reports/validation_checklist.txt b/docs/dev/reports/validation_checklist.txt
@@ -1,21 +1,17 @@
 # Validation Checklist
 
-- [x] BUILD doc present
 - [x] APPLY doc present
+- [x] Companion BUILD doc present
 - [x] Codex command present
 - [x] Commit comment present
 - [x] Next command present
-- [x] BUILD doc provides exact Dockerfile blueprint
-- [x] BUILD doc provides exact compose blueprint
-- [x] BUILD doc provides exact env contract
-- [x] BUILD doc limits scope to Sample A server containerization
-- [x] `node --check games/network_sample_a/server/networkSampleADashboardServer.mjs` passed
-- [x] Local smoke run passed (`/health` 200, `/api/metrics?key=...` 200, unauthenticated metrics 403)
-- [x] Dockerfile contract verified (Node 22 Alpine, port 4310, healthcheck path)
-- [x] Compose contract verified (port map, env vars, healthcheck, restart policy)
-- [x] `.dockerignore` constrains context to required files
-- [x] README includes build/run/compose/health/env/logging guidance
-- [x] `NETWORK_SAMPLES_PLAN.md` Track U updated with bracket-only edits
-- [x] `BIG_PICTURE_ROADMAP.md` not overwritten in this pass
-- [x] `PRODUCTIZATION_ROADMAP.md` not overwritten in this pass
-- [ ] Docker image build validation (Docker CLI unavailable in this environment)
+- [x] Dockerfile exists
+- [x] .dockerignore exists
+- [x] docker-compose.yml exists
+- [x] README exists
+- [x] `node --check games/network_sample_a/server/networkSampleADashboardServer.mjs` passes
+- [x] Docker validation documented as unavailable in this environment (`docker` CLI not installed)
+- [x] No engine-core changes in this APPLY working delta
+- [x] NETWORK_SAMPLES_PLAN Track U bracket states verified complete (`[x]`)
+- [x] BIG_PICTURE_ROADMAP preserved
+- [x] PRODUCTIZATION_ROADMAP preserved
diff --git a/docs/pr/APPLY_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION.md b/docs/pr/APPLY_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION.md
@@ -6,38 +6,95 @@ APPLY_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION.md
 # APPLY_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION
 
 ## Apply Intent
-Apply the minimal Sample A server containerization defined by the BUILD doc.
-This APPLY step is code/runtime work for Codex, not documentation work for ChatGPT.
+This is a docs-only APPLY bundle for the completed Codex build/output.
+Use it to confirm acceptance, validate scope, and finalize the PR cleanly.
+
+## Boundary
+- ChatGPT did not write code here.
+- Codex already produced the code/runtime delta bundle.
+- This APPLY doc is for acceptance, validation, commit discipline, and tracker finalization only.
+
+## Expected Codex Outputs
+The completed build should already contain code/runtime artifacts for:
 
-## In Scope Files
 - `games/network_sample_a/server/Dockerfile`
 - `games/network_sample_a/.dockerignore`
 - `games/network_sample_a/server/docker-compose.yml`
 - `games/network_sample_a/server/README.md`
-- `games/network_sample_a/server/networkSampleADashboardServer.mjs` only if required for env-safe host/access compatibility
-- `docs/pr/BUILD_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION.md`
-- `docs/pr/APPLY_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION.md`
-- `docs/dev/codex_commands.md`
-- `docs/dev/commit_comment.txt`
-- `docs/dev/next_command.txt`
-- `docs/dev/reports/change_summary.txt`
-- `docs/dev/reports/file_tree.txt`
-- `docs/dev/reports/validation_checklist.txt`
-- `docs/roadmaps/NETWORK_SAMPLES_PLAN.md` with bracket-only edits only
-
-## Hard Rules
-- no engine-core changes
-- no dashboard mutation features
-- no auth redesign
-- no orchestration stack
-- no unrelated runtime refactors
-- no placeholder docs
-- no roadmap structure changes
-
-## Acceptance
-- Dockerfile builds Sample A dashboard server image
-- compose file runs the same service contract
-- env vars are documented and honored
-- health route works
-- README is actionable
-- roadmap Track U items update by bracket-only edits
+- `games/network_sample_a/server/networkSampleADashboardServer.mjs` only if required for env-safe host/access support
+
+## Acceptance Criteria
+Accept this PR only if all of the following are true:
+
+### Container Artifacts
+- Dockerfile exists under `games/network_sample_a/server/`
+- `.dockerignore` exists under `games/network_sample_a/`
+- compose file exists under `games/network_sample_a/server/`
+- README exists under `games/network_sample_a/server/`
+
+### Scope Control
+- no engine/core files changed
+- no dashboard mutation/admin features added
+- no transport redesign occurred
+- no unrelated sample/game refactors occurred
+
+### Runtime Contract
+- container targets the existing Sample A dashboard server runtime
+- host/port/env usage matches the BUILD blueprint
+- health route remains:
+  - `/admin/network-sample-a/health`
+- container port contract remains:
+  - `4310`
+
+### Documentation Contract
+README should contain:
+- docker build command
+- docker run command
+- compose command
+- mapped port explanation
+- health URL
+- env var meanings
+- logging expectation (stdout/stderr only)
+
+## Required Validation
+### Required checks
+- `node --check games/network_sample_a/server/networkSampleADashboardServer.mjs`
+
+### If Docker is available
+- `docker build -f games/network_sample_a/server/Dockerfile games/network_sample_a -t network-sample-a-dashboard`
+- `docker compose -f games/network_sample_a/server/docker-compose.yml up --build`
+- verify:
+  - health route responds
+  - mapped port is correct
+  - container starts without extra runtime changes
+
+## Roadmap Finalization Rule
+Only update bracket states.
+Do not change wording, ordering, headings, or structure.
+
+### docs/roadmaps/NETWORK_SAMPLES_PLAN.md
+Set these to `[x]` if Codex output satisfies acceptance:
+- `Dockerfile for server`
+- `.dockerignore`
+- `Environment variable contract`
+- `Local run command`
+- `Compose-ready service definition`
+- `Port mapping rules`
+- `Health/readiness check`
+- `Logging/output expectations`
+- `Container debug notes`
+
+### docs/roadmaps/BIG_PICTURE_ROADMAP.md
+No changes unless there is an existing exact matching item that only needs a bracket-state change.
+
+### docs/roadmaps/PRODUCTIZATION_ROADMAP.md
+No changes in this PR.
+
+## Commit Guidance
+Commit only after:
+- code/runtime outputs are present
+- validation passes or acceptable validation notes are documented
+- roadmap bracket states are updated surgically only
+
+## Recommended Commit Comment
+Apply Sample A server containerization artifacts and finalize Track U checklist states without engine-core changes.
diff --git a/docs/pr/BUILD_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION.md b/docs/pr/BUILD_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION.md

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-Build Sample A server containerization artifacts and update Track U roadmap states without engine-core changes.`
	`1`	`+Apply Sample A server containerization artifacts and finalize Track U checklist states without engine-core changes.`
Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-APPLY_PR_DEBUG_SURFACES_SERVER_CONTAINERIZATION`
	`1`	`+PLAN_PR_DEBUG_SURFACES_SERVER_RUNTIME_ISOLATION`