janpeter/scrum4me-shared

Fork 0

feat(idea-169): rebaseline shared PPE storage runtime #53

Merged

janpeter merged 7 commits from codex/idea-169-runtime-rebaseline-shared into main

2026-08-23 19:24:45 +02:00

janpeter commented

2026-08-23 18:35:11 +02:00

Owner

Samenvatting

Integreert de exact gereviewde Plan-A shared runtime via een echte mergecommit.
Behoudt de huidige M32 archivevelden en index naast alle negen PPE-markers.
Voegt het complete duurzame PPE-opslagcontract, relaties, constraints en schema-projectietests toe.

Identiteit en ancestry

Frozen main / merge-ouder 1: c63f0bd712
Reviewed target / merge-ouder 2: 36993b2613
Rebaseline mergecommit: ddbfd77d71
Exacte huidige PR-head: 6852386ef9

Deze PR moet met behoud van merge-ancestry worden gemerged. Niet squashen of rebasen: de volgende consumerstappen en immutable evidence vertrouwen op deze reviewed-target ancestry.

Reviewfix

De runtimeprojectie staat nu als pure TypeScript-constante binnen lib.
De canonieke JSON-fixture blijft byte-identiek en is uitsluitend test/evidence-input.
De dependency-gate weigert voortaan iedere parent-relative import vanuit lib.
De fixture, lib-constante en bestaande SHA-256 worden exact tegen elkaar getoetst.
De eerder falende NodeNext ESM-compile is nu groen.

Verificatie

npm run verify: PASS
dependency gate: PASS
TypeScript en NodeNext-compile: PASS
Vitest: 25 testbestanden, 269 tests, nul failures
git diff --check en clean-tree: PASS

Scope

Dit is uitsluitend shippingstap 1. Er is nog niets gedeployed, geen controllerdatabase geactiveerd en geen operationele PPE-uitvoering gestart.

## Samenvatting - Integreert de exact gereviewde Plan-A shared runtime via een echte mergecommit. - Behoudt de huidige M32 archivevelden en index naast alle negen PPE-markers. - Voegt het complete duurzame PPE-opslagcontract, relaties, constraints en schema-projectietests toe. ## Identiteit en ancestry - Frozen main / merge-ouder 1: c63f0bd7128d2b681b3cf7e28e3f38787e2537b1 - Reviewed target / merge-ouder 2: 36993b26138554afee18d0e7af03e9e1b77dd85b - Rebaseline mergecommit: ddbfd77d719c08f091e30e474f05e76bc519ae75 - Exacte huidige PR-head: 6852386ef94d348315ba4021ea7e53968814b06b Deze PR moet met behoud van merge-ancestry worden gemerged. Niet squashen of rebasen: de volgende consumerstappen en immutable evidence vertrouwen op deze reviewed-target ancestry. ## Reviewfix - De runtimeprojectie staat nu als pure TypeScript-constante binnen lib. - De canonieke JSON-fixture blijft byte-identiek en is uitsluitend test/evidence-input. - De dependency-gate weigert voortaan iedere parent-relative import vanuit lib. - De fixture, lib-constante en bestaande SHA-256 worden exact tegen elkaar getoetst. - De eerder falende NodeNext ESM-compile is nu groen. ## Verificatie - npm run verify: PASS - dependency gate: PASS - TypeScript en NodeNext-compile: PASS - Vitest: 25 testbestanden, 269 tests, nul failures - git diff --check en clean-tree: PASS ## Scope Dit is uitsluitend shippingstap 1. Er is nog niets gedeployed, geen controllerdatabase geactiveerd en geen operationele PPE-uitvoering gestart.

janpeter added 6 commits

2026-08-23 18:35:11 +02:00

test: freeze parallel execution storage projection 0b53b856a5

fix: enforce parallel execution storage projection 24ad0e217e

test: cover template runtime imports 3e2c14d16e

feat: add canonical marked execution storage schema f4aeb6c896

test: close marked storage import discovery 36993b2613

merge(idea-169): rebaseline shared PPE storage runtime

CI / Verify (pull_request) Successful in 18s

Details

ddbfd77d71

s4m-codex-reviewer requested changes

2026-08-23 18:41:13 +02:00

Dismissed

s4m-codex-reviewer left a comment

Verdict: REQUEST_CHANGES

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

Findings

error — lib/parallel-plan-execution-storage.ts:3 — De nieuwe runtime-module onder lib/ importeert ../fixtures/parallel-plan-execution/storage-projection-v1.json. Volgens de productarchitectuur is scrum4me-shared/lib de pure gedeelde TS-laag en mogen lib/*.ts alleen zod en andere modules binnen lib/ importeren. Deze export wordt door consumers via de shared submodule/path-alias geconsumeerd; een JSON fixture buiten lib/ maakt de runtime/API afhankelijk van testdata en van consumer TypeScript-configuratie zoals JSON module resolution. Verplaats de bevroren projectie naar een echte lib/-module/constante, of houd de fixture alleen in tests en laat de runtime-code niet uit fixtures/ importeren.

Conclusie

Niet approvebaar zolang de gedeelde runtime-laag een fixture importeert. De schema-uitbreiding en tests zijn verder gericht op het bevriezen van het contract, maar deze import doorbreekt de gedocumenteerde shared-architectuur.

# Verdict: REQUEST_CHANGES geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - **error** — `lib/parallel-plan-execution-storage.ts:3` — De nieuwe runtime-module onder `lib/` importeert `../fixtures/parallel-plan-execution/storage-projection-v1.json`. Volgens de productarchitectuur is `scrum4me-shared/lib` de pure gedeelde TS-laag en mogen `lib/*.ts` alleen `zod` en andere modules binnen `lib/` importeren. Deze export wordt door consumers via de shared submodule/path-alias geconsumeerd; een JSON fixture buiten `lib/` maakt de runtime/API afhankelijk van testdata en van consumer TypeScript-configuratie zoals JSON module resolution. Verplaats de bevroren projectie naar een echte `lib/`-module/constante, of houd de fixture alleen in tests en laat de runtime-code niet uit `fixtures/` importeren. ## Conclusie Niet approvebaar zolang de gedeelde runtime-laag een fixture importeert. De schema-uitbreiding en tests zijn verder gericht op het bevriezen van het contract, maar deze import doorbreekt de gedocumenteerde shared-architectuur.

janpeter added 1 commit

2026-08-23 18:54:01 +02:00

fix(idea-169): keep shared runtime inside lib boundary

CI / Verify (pull_request) Successful in 18s

Details

6852386ef9

s4m-codex-reviewer approved these changes

2026-08-23 18:56:53 +02:00

s4m-codex-reviewer left a comment

APPROVED

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

Findings

Geen blokkerende of error-severity findings gevonden.

Review-notities

De wijziging blijft binnen de scrum4me-shared architectuurafspraken: lib/*.ts gebruikt alleen zod en sibling lib imports, en de aangescherpte verify-no-deps.sh bewaakt dat expliciet. De Prisma-aanpassingen passen bij het shared-schema eigenaarschap waarbij consumers schema genereren en migraties via de designated migrator lopen. De diff voegt gerichte tests toe voor AgentMessage/PPE-schema-inventaris, de frozen projection, import-closure en de dependency gate.

# APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings Geen blokkerende of error-severity findings gevonden. ## Review-notities De wijziging blijft binnen de `scrum4me-shared` architectuurafspraken: `lib/*.ts` gebruikt alleen `zod` en sibling `lib` imports, en de aangescherpte `verify-no-deps.sh` bewaakt dat expliciet. De Prisma-aanpassingen passen bij het shared-schema eigenaarschap waarbij consumers schema genereren en migraties via de designated migrator lopen. De diff voegt gerichte tests toe voor AgentMessage/PPE-schema-inventaris, de frozen projection, import-closure en de dependency gate.

janpeter merged commit a6a664cb75 into main

2026-08-23 19:24:45 +02:00

janpeter referenced this pull request from a commit

2026-08-23 19:24:47 +02:00

Merge pull request 'feat(idea-169): rebaseline shared PPE storage runtime' (#53) from codex/idea-169-runtime-rebaseline-shared into main

janpeter referenced this pull request from janpeter/Scrum4Me

2026-08-23 19:32:09 +02:00

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

janpeter referenced this pull request from janpeter/scrum4me-mcp

2026-08-23 19:32:29 +02:00

feat(idea-169): rebaseline marked MCP runtime #124