janpeter/scrum4me-docker
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat(skills): add reviewed ui-ux-pro-max hardening pin #66

Merged
janpeter merged 1 commit from codex/ui-ux-pro-max-hardening-pin into master 2026-07-17 15:12:27 +02:00
Conversation 1 Commits 1 Files changed 49 +7351
janpeter commented 2026-07-17 15:04:25 +02:00
Owner
Copy link

Samenvatting

  • pin upstream commit f8ac5e1266dba8354ea96e19994d9f4345e7ec31 met geverifieerde tree- en contenthash
  • vendor alleen de skill-subtree en root-MIT-licentie
  • pas runtime-neutrale paden en worktree/symlink-confinement toe als leesbare overlays
  • promoveer 2.11.0+f8ac5e1.s4m.1 als derde APPROVED skill

Verificatie

  • upstream tests: 16/16 groen onder verplichte SCRUM4ME_JOB_WORKTREE
  • datavalidatie: OK (12 domeinen, 22 stacks)
  • adaptertests: 6/6 groen
  • volledige suite: 44/44 groen
  • skills:verify: 3 approved
  • reconstructiehash: 28ae96d4ed673db217d3af21b330c7aaa0f57a573ee39671da76a668c70eb22c

Rolloutpoort

Claude tweede review is aangevraagd. Deze PR rolt niets uit; fleet-rollout blijft geblokkeerd tot beide runtimecanaries en expliciet menselijk akkoord.

## Samenvatting - pin upstream commit f8ac5e1266dba8354ea96e19994d9f4345e7ec31 met geverifieerde tree- en contenthash - vendor alleen de skill-subtree en root-MIT-licentie - pas runtime-neutrale paden en worktree/symlink-confinement toe als leesbare overlays - promoveer 2.11.0+f8ac5e1.s4m.1 als derde APPROVED skill ## Verificatie - upstream tests: 16/16 groen onder verplichte SCRUM4ME_JOB_WORKTREE - datavalidatie: OK (12 domeinen, 22 stacks) - adaptertests: 6/6 groen - volledige suite: 44/44 groen - skills:verify: 3 approved - reconstructiehash: 28ae96d4ed673db217d3af21b330c7aaa0f57a573ee39671da76a668c70eb22c ## Rolloutpoort Claude tweede review is aangevraagd. Deze PR rolt niets uit; fleet-rollout blijft geblokkeerd tot beide runtimecanaries en expliciet menselijk akkoord.
feat(skills): add reviewed ui-ux-pro-max hardening pin
All checks were successful
CI / Compose config (pull_request) Successful in 4s
Details
CI / Docker build (pull_request) Successful in 1m3s
Details
0df7b2bdff
s4m-codex-reviewer approved these changes 2026-07-17 15:06:09 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

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

Findings

  • Geen findings.

Beoordeling

De diff voegt een expliciet aangeroepen, gereviewde ui-ux-pro-max skill-pin toe met MIT-licentie, provenance/hash-velden, runtime-lijst voor Claude en Codex, en een reviewdocument dat resterende rollout-poorten expliciet blokkeert. De hardening-patches sluiten aan op de worker-standaarden: runtime-neutrale skillpaden via $SCRUM4ME_SKILLS_ROOT en persistente output begrensd tot SCRUM4ME_JOB_WORKTREE, inclusief negatieve tests voor ontbrekende env, path escape en symlink escape. De catalogus houdt invocationPolicy op EXPLICIT, wat past bij de productdocs voor headless worker safety en beperkt agentgedrag door omvangrijke skill-instructies.

# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen findings. ## Beoordeling De diff voegt een expliciet aangeroepen, gereviewde `ui-ux-pro-max` skill-pin toe met MIT-licentie, provenance/hash-velden, runtime-lijst voor Claude en Codex, en een reviewdocument dat resterende rollout-poorten expliciet blokkeert. De hardening-patches sluiten aan op de worker-standaarden: runtime-neutrale skillpaden via `$SCRUM4ME_SKILLS_ROOT` en persistente output begrensd tot `SCRUM4ME_JOB_WORKTREE`, inclusief negatieve tests voor ontbrekende env, path escape en symlink escape. De catalogus houdt `invocationPolicy` op `EXPLICIT`, wat past bij de productdocs voor headless worker safety en beperkt agentgedrag door omvangrijke skill-instructies.
janpeter merged commit 27293b388f into master 2026-07-17 15:12:27 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels
No items
No labels
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-docker!66
No description provided.