Phase 3B baseline replacement plan — supersedes PR #248 (docs only)#250
Open
mdtazizulislam wants to merge 1 commit into
Open
Phase 3B baseline replacement plan — supersedes PR #248 (docs only)#250mdtazizulislam wants to merge 1 commit into
mdtazizulislam wants to merge 1 commit into
Conversation
B3 PG18 verification proved PR #248's initDatabase()-derived baseline is unfaithful to production. Apples-to-apples catalog counts (throwaway PG16): initDatabase() = 125 tables / 1595 cols / 400 idx / 267 fk; real prod (operator PG18/Gate 1 snapshot) = 125 / 1649 / 402 / 266. Applying all 24 hand-written migration .sql files on top of initDatabase() moved columns only 1595->1596 — so ~53 prod columns (+2 indexes, -1 FK) exist in NO source of truth = undocumented production drift. Consequence: PR #248 baseline superseded. The authoritative 0000_prod_baseline must be introspected from the REAL Gate 1 snapshot (operator, PG18), not initDatabase(). Deliverables: 01_DRIFT_FINDING.md (evidence), 02_REPLACEMENT_RUNBOOK.md (operator regenerate on PG18 + normalized diff / drift inventory + journal + seeded db:migrate zero-DDL proof + commit to replacement PR). Diagnostics ran on throwaway PG16 (torn down); production untouched; PR #248 stays unmerged. No production deploy/migration execution/schema application/backend wiring/ feature-flag activation. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MiPyhqoV9Pt7XN2tiFpS4A
✅ Deploy Preview for zonforge-customer-operations canceled.
|
✅ Deploy Preview for ai-powered-cyber ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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
2 participants
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.
Summary
B3 PG18 verification proved PR #248's
initDatabase()-derived baseline is not faithful to production. This package documents the confirmed drift and the plan to regenerate the authoritative0000_prod_baselinefrom the operator's real Gate 1 snapshot on PG18. Docs only — no production deploy, migration execution, schema application, backend wiring, or feature-flag activation. PR #248 stays unmerged and is superseded.Confirmed drift (apples-to-apples, same catalog queries, throwaway PG16)
initDatabase())Applying all 24 hand-written migration
.sqlfiles on top ofinitDatabase()moved columns only 1595 → 1596. So ~53 production columns (+2 indexes, −1 FK) exist in no source of truth — undocumented production drift (the core Phase-3 problem). The earlier "measurement artifact?" question is settled: measured identically, the gap is real.Consequence
Deliverables
01_DRIFT_FINDING.md— full evidence (apples-to-apples counts + the migration test).02_REPLACEMENT_RUNBOOK.md— operator steps on PG18: regenerate from the real snapshot → fix theint4_opsartifact → normalized structural diff (captures the ~54-column drift inventory) → build journal → prove seededdb:migrateskips with zero DDL (the step B3 blocked) → commit to this replacement PR.Next / decision
Operator runs the replacement runbook on PG18. On PASS, the snapshot-derived baseline supersedes PR #248 (close #248 in its favor), B3 clears, and B2 (restore/PITR) is next — all separately approved. The drift inventory from the diff also feeds Phase 3C convergence.
Guardrails honored
db:migratestill no-op; PR [SUPERSEDED by #250] Phase 3B promotion execution prep — 0000_prod_baseline (B1) #248 unmerged🤖 Generated with Claude Code
Generated by Claude Code