Skip to content

Fix Dependabot workflow: add --skip-install flag and error logging#2140

Merged
TalZaccai merged 1 commit intomainfrom
talzacc/security_script
Apr 7, 2026
Merged

Fix Dependabot workflow: add --skip-install flag and error logging#2140
TalZaccai merged 1 commit intomainfrom
talzacc/security_script

Conversation

@TalZaccai
Copy link
Copy Markdown
Contributor

@TalZaccai TalZaccai commented Apr 7, 2026

The \ ix-dependabot-alerts\ workflow completed successfully but found zero packages. The stderr log (now printed thanks to the error logging fix) revealed:

\
Command failed (exit 139): pnpm install --frozen-lockfile
\\

Root cause: The script runs \pnpm install --frozen-lockfile\ at startup, but the workflow already installs dependencies. Running it again OOM-kills the runner (exit 139 = signal kill).

Changes

  1. --skip-install\ flag: Skips the redundant \pnpm install\ when deps are already available (e.g. in CI)
  2. Error logging: When the script produces no valid JSON, stderr is now printed to the job log
  3. Both workflow invocations (dry-run + per-package fix) now pass --skip-install\

How to verify

Merge and re-run the workflow — it should now get past the install step and actually discover fixable packages.

The default GITHUB_TOKEN returns 403 on the /dependabot/alerts
endpoint regardless of workflow permissions. The workflow now:

- Uses DEPENDABOT_PAT secret (falls back to GITHUB_TOKEN)
- Prints stderr log when the script produces no JSON output
- Documents the PAT requirement at the top of the workflow

A PAT with security_events scope must be added as a repo secret
named DEPENDABOT_PAT for the workflow to function.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@TalZaccai TalZaccai force-pushed the talzacc/security_script branch from 6d28438 to 1b2344e Compare April 7, 2026 02:19
@TalZaccai TalZaccai temporarily deployed to development-fork April 7, 2026 02:19 — with GitHub Actions Inactive
@TalZaccai TalZaccai temporarily deployed to development-fork April 7, 2026 02:19 — with GitHub Actions Inactive
@TalZaccai TalZaccai changed the title Fix Dependabot workflow: add error logging and GitHub App auth Fix Dependabot workflow: add error logging Apr 7, 2026
@TalZaccai TalZaccai added this pull request to the merge queue Apr 7, 2026
Merged via the queue into main with commit 8cc83f0 Apr 7, 2026
21 checks passed
@TalZaccai TalZaccai changed the title Fix Dependabot workflow: add error logging Fix Dependabot workflow: add --skip-install flag and error logging Apr 7, 2026
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.

1 participant