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

M42 — Vis-thema voor iPhone en Watch (PR C) #294

Merged
janpeter merged 9 commits from feat/m42-vis-ios into main 2026-10-04 11:45:47 +02:00
Conversation 1 Commits 9 Files changed 96 +1582 -146
janpeter commented 2026-10-04 08:23:51 +02:00
Owner
Copy link

De iPhone- en Watch-app gebruiken HubTheme met 13 Vis-kleuren, vaste Watch-canvas, assets in de hoofd-appbundels, capsuleknoppen en serif-titels. Optievolgorde, beschikbaarheid en transportlogica blijven behouden. De bronchecklist en 14 iPhone-/11 Watch-simulatorbeelden staan in docs/plans/M42-bewijs/.

JP heeft op 2026-10-04 de app op zijn fysieke Ultra 2 beoordeeld en expliciet goedgekeurd. Het algemene oordeel is letterlijk vastgelegd in watch/oordeel.md. Er zijn geen nieuwe fysieke foto's of afzonderlijke sending/sent/failed-transportresultaten aangeleverd; de simulatorbeelden blijven duidelijk als simulatorbewijs gelabeld.

TestFlight 1.0 (27) is eerder op JP's opdracht geüpload en ontvangen. Main uit web-PR #293 is nu geïntegreerd, met opgeloste documentconflicten. De iOS-bronboom is exact gelijk aan de gepubliceerde/uploadbron op 1aceb804 (Git-tree fae2a374d7); het archive-executable is bytegelijk en buildnummer blijft 27. Geen nieuwe upload of versieophoging.

Nieuwe validatie: 804 iOS-tests geslaagd; app BUILD SUCCEEDED; lint zonder fouten (6 bestaande waarschuwingen), typecheck, 3927 Vitest-tests geslaagd/29 overgeslagen (maxWorkers2), productiebuild en documentlinks in 308 bestanden geslaagd.

Gepind plan · Vis-spec · Toestelacceptatie en integratie.

Reviewklaar. CI, PR-merge, serverdeployment en bestaande transportacceptatie buiten Vis blijven afzonderlijke stappen. Plan-PR #292 en web-PR #293 zijn gemerged. PBI-177 / ST-1626 / ST-1627 / T-1996.

De iPhone- en Watch-app gebruiken HubTheme met 13 Vis-kleuren, vaste Watch-canvas, assets in de hoofd-appbundels, capsuleknoppen en serif-titels. Optievolgorde, beschikbaarheid en transportlogica blijven behouden. De bronchecklist en 14 iPhone-/11 Watch-simulatorbeelden staan in docs/plans/M42-bewijs/. JP heeft op 2026-10-04 de app op zijn fysieke Ultra 2 beoordeeld en expliciet goedgekeurd. Het algemene oordeel is letterlijk vastgelegd in `watch/oordeel.md`. Er zijn geen nieuwe fysieke foto's of afzonderlijke sending/sent/failed-transportresultaten aangeleverd; de simulatorbeelden blijven duidelijk als simulatorbewijs gelabeld. TestFlight 1.0 (27) is eerder op JP's opdracht geüpload en ontvangen. Main uit web-PR #293 is nu geïntegreerd, met opgeloste documentconflicten. De iOS-bronboom is exact gelijk aan de gepubliceerde/uploadbron op 1aceb804 (Git-tree fae2a374d7fed0c322682658ef220229b285f417); het archive-executable is bytegelijk en buildnummer blijft 27. Geen nieuwe upload of versieophoging. Nieuwe validatie: 804 iOS-tests geslaagd; app BUILD SUCCEEDED; lint zonder fouten (6 bestaande waarschuwingen), typecheck, 3927 Vitest-tests geslaagd/29 overgeslagen (maxWorkers2), productiebuild en documentlinks in 308 bestanden geslaagd. [Gepind plan](https://git.jp-visser.nl/janpeter/Scrum4Me/src/commit/d228a1afa56b619ddabf8d8544d6789711a4726d/docs/plans/M42-vis-design-migratie.md) · [Vis-spec](https://git.jp-visser.nl/janpeter/Scrum4Me/src/commit/d228a1afa56b619ddabf8d8544d6789711a4726d/docs/specs/2026-10-03-vis-design-migratie-design.md) · [Toestelacceptatie en integratie](https://git.jp-visser.nl/janpeter/Scrum4Me/src/commit/d228a1afa56b619ddabf8d8544d6789711a4726d/docs/plans/M42-bewijs/pr294-toestelacceptatie.md). Reviewklaar. CI, PR-merge, serverdeployment en bestaande transportacceptatie buiten Vis blijven afzonderlijke stappen. Plan-PR #292 en web-PR #293 zijn gemerged. PBI-177 / ST-1626 / ST-1627 / T-1996. 
T-1990 blijft in uitvoering: XcodeGen, ios:build en 265 ios:test tests groen. Runtimekleurproef en screenshots op beide simulators ontbreken nog; T-1991 niet gestart.
T-1990: real simulator screenshots and sRGB pixel samples match ink on iPhone light/dark and Watch. Temporary isolated launch probe removed from source; restored ios:build succeeded. Existing 265 iOS tests passed. MCP status sync pending; no push.
T-1991 RED 40 failed/3 passed; GREEN 43. verify 3155 passed/29 skipped; ios:build succeeded. Preserve accent asset name; disable unused typed asset symbol generation to fix accent/AccentColor collision, warnings-as-errors unchanged. MCP sync pending.
T-1992: Ember(fill override), Ink, Outline and neutral Raised styles; 44pt minimum, pressed/disabled feedback, label foreground inherited. ios:test and ios:build passed. Visual assessment remains in T-1994/T-1996; MCP sync pending.
build(ST-1627): upload Vis-build 27 naar TestFlight
All checks were successful
CI / Lint, Typecheck, Test & Build (pull_request) Successful in 4m45s
Details
CI / Queue Dispatch Guard Gate (pull_request) Successful in 1m50s
Details
CI / DB Access Policy Gate (pull_request) Successful in 3m6s
Details
CI / Deploy Manual (workflow_dispatch) (pull_request) Has been skipped
Details
CI / DB Access Production Sentinel (pull_request) Successful in 1m11s
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
1aceb80491
merge(ST-1627): integreer Vis-web en leg Ultra 2-goedkeuring vast
All checks were successful
CI / DB Access Policy Gate (pull_request) Successful in 3m7s
Details
CI / Lint, Typecheck, Test & Build (pull_request) Successful in 4m41s
Details
CI / DB Access Production Sentinel (pull_request) Successful in 1m17s
Details
CI / Deploy Manual (workflow_dispatch) (pull_request) Has been skipped
Details
CI / Queue Dispatch Guard Gate (pull_request) Successful in 3m2s
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
d228a1afa5
janpeter changed title from WIP: M42 — Vis-thema voor iPhone en Watch (PR C) to M42 — Vis-thema voor iPhone en Watch (PR C) 2026-10-04 11:40:56 +02:00
janpeter scheduled this pull request to auto merge when all checks succeed 2026-10-04 11:44:07 +02:00
s4m-codex-reviewer approved these changes 2026-10-04 11:48:14 +02:00
s4m-codex-reviewer left a comment
Copy link

APPROVED

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

Findings

  • Geen blokkerende of overige concrete findings gevonden.

De diff is beoordeeld op codekwaliteit, architectuur, tests en documentatie tegen de productdocs en de repo-lokale Vis-specificatie §5. De aparte SwiftUI-only HubTheme-target, appbundelassets, knopvoorgronden en afhankelijkheden zijn consistent. De bestaande antwoord-, beschikbaarheids- en retrylogica blijft behouden. Simulatorbewijs en fysieke Ultra 2-acceptatie zijn duidelijk afgebakend.

Verificatie op commit d228a1afa5: PR-head bevestigd; 26 kleursets onafhankelijk vergeleken met alle 13 specificatietokens (iPhone licht/donker en Watch), geslaagd; documentlinks: 308 bestanden, geen fouten. Geselecteerde simulatorbeelden visueel gecontroleerd. De iOS-tree fae2a374d7 komt overeen met de gedocumenteerde build- en toestelbron.

De documentatie rapporteert 804 native tests en geslaagde iOS-builds. Die zijn op deze Linux-runtime niet opnieuw uitgevoerd. De nieuwe Vitest-assettest kon hier niet worden gestart door een npm-installatiefout; de onafhankelijke kleursetcontrole hierboven is wel uitgevoerd. Volledige Dynamic Type-, VoiceOver- en transportacceptatie wordt niet uit deze review afgeleid.

# APPROVED geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen blokkerende of overige concrete findings gevonden. De diff is beoordeeld op codekwaliteit, architectuur, tests en documentatie tegen de productdocs en de repo-lokale Vis-specificatie §5. De aparte SwiftUI-only HubTheme-target, appbundelassets, knopvoorgronden en afhankelijkheden zijn consistent. De bestaande antwoord-, beschikbaarheids- en retrylogica blijft behouden. Simulatorbewijs en fysieke Ultra 2-acceptatie zijn duidelijk afgebakend. Verificatie op commit d228a1afa56b619ddabf8d8544d6789711a4726d: PR-head bevestigd; 26 kleursets onafhankelijk vergeleken met alle 13 specificatietokens (iPhone licht/donker en Watch), geslaagd; documentlinks: 308 bestanden, geen fouten. Geselecteerde simulatorbeelden visueel gecontroleerd. De iOS-tree fae2a374d7fed0c322682658ef220229b285f417 komt overeen met de gedocumenteerde build- en toestelbron. De documentatie rapporteert 804 native tests en geslaagde iOS-builds. Die zijn op deze Linux-runtime niet opnieuw uitgevoerd. De nieuwe Vitest-assettest kon hier niet worden gestart door een npm-installatiefout; de onafhankelijke kleursetcontrole hierboven is wel uitgevoerd. Volledige Dynamic Type-, VoiceOver- en transportacceptatie wordt niet uit deze review afgeleid.
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!294
No description provided.