From 1e5db94f0e90f59551716daee15ae702dd817e07 Mon Sep 17 00:00:00 2001 From: Erik Date: Thu, 11 Jun 2026 11:03:37 +0200 Subject: [PATCH] docs: plan amendment - user directive: port everything, test once at the end 'I don't care if it is non-playable... I want everything ported, then we test.' Per-phase playability + per-phase visual gates DROPPED. BR-2..BR-6 execute as ONE continuous port with build+tests green per commit and a single comprehensive visual pass at the end (T5). Replaces the playability rule with: every installment must be a COMPLETE retail behavior, never half of one (the BR-2 punch-without-ordering lesson, 88be519). --- docs/plans/2026-06-11-building-render-port-plan.md | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/docs/plans/2026-06-11-building-render-port-plan.md b/docs/plans/2026-06-11-building-render-port-plan.md index 17a1de6f..cb249766 100644 --- a/docs/plans/2026-06-11-building-render-port-plan.md +++ b/docs/plans/2026-06-11-building-render-port-plan.md @@ -1,6 +1,17 @@ # The holistic building-render port plan (Phase B) — one drawing discipline -**Status: AWAITING USER APPROVAL — no implementation until approved.** +**Status: APPROVED + AMENDED (2026-06-11). EXECUTION DIRECTIVE CHANGED BY THE +USER: "I don't care if it is non-playable… I want everything ported, then we +test."** The per-phase playability constraint and per-phase user visual gates +are DROPPED. BR-2 through BR-6 execute as ONE continuous port (the fused +render discipline), with build + unit/conformance tests green at every commit +(engineering hygiene, not gates), and **ONE comprehensive visual test pass at +the end**. Rationale: the first BR-2 attempt failed precisely because the +phase slicing cut retail's frame order in half (the punch shipped without +entities-drawn-last and erased characters in apertures — reverted `88be519`); +the installment-must-be-a-complete-retail-behavior rule replaces the +playability rule. BR-7 (collision) runs as an independent track; BR-8b +(lighting) still wants the verification resume first. Companion to the Phase A comparison: [`docs/research/2026-06-11-building-render-acdream-vs-retail-comparison.md`](../research/2026-06-11-building-render-acdream-vs-retail-comparison.md) (evidence appendices in