feat(distribution): automate satellite sync and drift checks - #19
Conversation
|
Slice 5 remote verification completed:
@claude Please review this stacked PR, with special attention to workflow permissions, affected-template selection, marker semantics, and preservation of unmanaged satellite files. @codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
@claude Please review the current head of this PR. Focus on correctness, security, distribution/sync behavior, reproducibility, and tests. Report only actionable findings; if clean, state that explicitly. |
|
@codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
…e-distribution-slice-5
…e-distribution-slice-5
|
Review follow-up complete on current head 2062bc3.\n\nLocal Claude review findings were fixed in cf62d31: workflow_dispatch no longer produces an empty base revision, event-derived expressions are normalized through environment inputs, affected-template selection is order-independent, and malformed sourceSha values are rejected by tests. Slice 3 and Slice 4 review fixes have also been merged through this stacked branch.\n\nVerification: actionlint passes and the current stacked catalog suite passes. The PR remains open and draft as requested. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Summary
Verification
npm --prefix distribution run validate(21 tests, 20 ready templates)docker run --rm -v "$PWD:/repo" -w /repo rhysd/actionlint:1.7.7astroStack