janpeter/Scrum4Me
0
0
Fork 0
Code Issues 3 Pull requests 1 Projects Releases Packages Wiki Activity Actions

feat(idea-169): rebaseline PPE storage consumer #193

Merged
janpeter merged 7 commits from codex/idea-169-runtime-rebaseline-scrum4me into main 2026-08-23 19:44:17 +02:00
Conversation 1 Commits 7 Files changed 12 +4334 -6
janpeter commented 2026-08-23 19:32:09 +02:00
Owner
Copy link

Gekoppeld plan en evidence

Samenvatting

  • Integreert de exact gereviewde IDEA-169 Plan-A opslagconsumer via een echte mergecommit.
  • Pint vendor/scrum4me-shared exact op reviewfix 6852386ef94d348315ba4021ea7e53968814b06b, die via shared mergecommit a6a664cb75cb07a3e514d6dd986920bf664f5260 op main staat.
  • Behoudt de reviewed PPE-migratie byte-identiek en regenereert het consumerschema uit shared.
  • Houdt historische evidence aan immutable ancestor bb9e805 en maakt de geïsoleerde PPE-migratiegate uitvoerbaar in CI.

Identiteit en ancestry

Deze PR moet met behoud van merge-ancestry worden gemerged. Niet squashen of rebasen: de immutable evidence valideert de reviewed-target ancestry.

Verse verificatie na shared merge

  • npm run verify: PASS
  • ESLint: 0 errors, 6 bekende warnings
  • Vitest: 296 testbestanden geslaagd, 1 overgeslagen; 2409 tests geslaagd, 4 overgeslagen
  • Productiebuild: PASS
  • Plan-A migratie-SHA-256: d9e919e5b5737783a2663db23c03e42eae1d5a7f76b85c22a030b5ba35bb29d2
  • Generated schema diff-clean en shared gitlink exact
  • Merge-tree tegen actuele origin/main: conflictvrij

Scope

Geen deployment, live-databasemigratie, controlleractivatie of operationele PPE-uitvoering.

## Gekoppeld plan en evidence - [Storage-foundation rolloutplan](https://git.jp-visser.nl/janpeter/Scrum4Me/src/branch/codex/idea-169-runtime-rebaseline-scrum4me/docs/runbooks/idea-169-storage-foundation-rollout.md) - [Shared predecessor PR #53](https://git.jp-visser.nl/janpeter/scrum4me-shared/pulls/53) ## Samenvatting - Integreert de exact gereviewde IDEA-169 Plan-A opslagconsumer via een echte mergecommit. - Pint vendor/scrum4me-shared exact op reviewfix 6852386ef94d348315ba4021ea7e53968814b06b, die via shared mergecommit a6a664cb75cb07a3e514d6dd986920bf664f5260 op main staat. - Behoudt de reviewed PPE-migratie byte-identiek en regenereert het consumerschema uit shared. - Houdt historische evidence aan immutable ancestor bb9e805 en maakt de geïsoleerde PPE-migratiegate uitvoerbaar in CI. ## Identiteit en ancestry - Frozen main / merge-ouder 1: ca6a97d4a83150d06158dbba917780bfc902acb9 - Reviewed target / merge-ouder 2: ac7294ceec4a8a67f066648983e2f8ac00798b42 - Rebaseline mergecommit: c68817472ac8c52cfc37c723050d7397525563f0 - Consumer reviewfix: fc974d9a219d33d5ab49d44135eac6cd48a9459e - Exacte PR-head: 64db5c591a8bf69317a43eae13d88938849878dd Deze PR moet met behoud van merge-ancestry worden gemerged. Niet squashen of rebasen: de immutable evidence valideert de reviewed-target ancestry. ## Verse verificatie na shared merge - npm run verify: PASS - ESLint: 0 errors, 6 bekende warnings - Vitest: 296 testbestanden geslaagd, 1 overgeslagen; 2409 tests geslaagd, 4 overgeslagen - Productiebuild: PASS - Plan-A migratie-SHA-256: d9e919e5b5737783a2663db23c03e42eae1d5a7f76b85c22a030b5ba35bb29d2 - Generated schema diff-clean en shared gitlink exact - Merge-tree tegen actuele origin/main: conflictvrij ## Scope Geen deployment, live-databasemigratie, controlleractivatie of operationele PPE-uitvoering.
chore(idea-169): advance shared reviewfix gitlink
Some checks failed
CI / Lint, Typecheck, Test & Build (pull_request) Failing after 3m48s
Details
CI / Deploy Manual (workflow_dispatch) (pull_request) Has been skipped
Details
CI / Detect deploy-relevant changes (pull_request) Has been skipped
Details
CI / Deploy Preview (PR) (pull_request) Has been skipped
Details
CI / Deploy Production (main) (pull_request) Has been skipped
Details
64db5c591a
s4m-codex-reviewer approved these changes 2026-08-23 19:36:27 +02:00
s4m-codex-reviewer left a comment
Copy link

APPROVED

geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

  • Geen blokkerende findings gevonden.

Reviewnotities

  • De PR voegt de PPE-storagefundering additief toe: nullable markeringskolommen, nieuwe PPE-tabellen, SQL-only checks/partial indexes en Prisma-projectie zijn onderling afgedekt door gerichte tests.
  • CI krijgt een expliciete disposable PostgreSQL-service voor de nieuwe live migratietests; dat past bij de test die PPE_TEST_DATABASE_URL vereist.
  • De submodule-update en evidence/runbook-bestanden zijn consistent met het karakter van een storage-foundation PR.
# APPROVED geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen blokkerende findings gevonden. ## Reviewnotities - De PR voegt de PPE-storagefundering additief toe: nullable markeringskolommen, nieuwe PPE-tabellen, SQL-only checks/partial indexes en Prisma-projectie zijn onderling afgedekt door gerichte tests. - CI krijgt een expliciete disposable PostgreSQL-service voor de nieuwe live migratietests; dat past bij de test die `PPE_TEST_DATABASE_URL` vereist. - De submodule-update en evidence/runbook-bestanden zijn consistent met het karakter van een storage-foundation PR.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels   
severity/s3

   
severity/s4
No labels
severity/s3
severity/s4
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/Scrum4Me!193
No description provided.