Skip to content

fix(ci): harden weekly-update — allowedTools, two-phase update, diff validation#1159

Merged
jdalton merged 5 commits intomainfrom
fix/ci-hardening
Apr 4, 2026
Merged

fix(ci): harden weekly-update — allowedTools, two-phase update, diff validation#1159
jdalton merged 5 commits intomainfrom
fix/ci-hardening

Conversation

@jdalton
Copy link
Copy Markdown
Contributor

@jdalton jdalton commented Apr 4, 2026

Replace --dangerously-skip-permissions with --allowedTools whitelist, two-phase haiku/sonnet, SFW_BIN PATH wrapper, post-agent diff validation.


Note

Medium Risk
Medium risk because it restructures the automated dependency-update workflow (new AI agent permissions, signing, gating, and test/fix phases), which could prevent updates/PRs from being created or allow unexpected file changes if misconfigured.

Overview
Hardens and restructures the weekly-update.yml dependency-update job to run in two phases: a fast Claude pass to update dependencies, then pnpm build/pnpm test, and an escalated Claude pass to fix failures.

Replaces --dangerously-skip-permissions with an explicit --allowedTools whitelist, adds GPG commit signing setup/cleanup, and wraps pnpm execution via a SFW_BIN shim for consistent tooling.

Adds a final status gate plus a post-run diff validation step that blocks push/PR creation unless only expected dependency/config or code/test files changed, and expands uploaded artifacts to include separate update/fix logs and build/test logs.

Reviewed by Cursor Bugbot for commit 692f2ec. Configure here.

@jdalton jdalton force-pushed the fix/ci-hardening branch from 41cd21b to e0ae67e Compare April 4, 2026 13:29
@jdalton jdalton force-pushed the fix/ci-hardening branch from 7f0806f to a08855c Compare April 4, 2026 13:37
@jdalton jdalton force-pushed the fix/ci-hardening branch from a08855c to d2cb368 Compare April 4, 2026 13:42
@jdalton jdalton merged commit 90b8620 into main Apr 4, 2026
12 of 13 checks passed
@jdalton jdalton deleted the fix/ci-hardening branch April 4, 2026 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants