Skip to content

[deps] Triage stale and overlapping dependency PRs after CI recovery #234

@jamiesun

Description

@jamiesun

Context Boundary

Expectation vs. Reality

Expected:

After main CI is stable again, dependency PRs should be triaged into one clear path: merge the safe non-overlapping updates, close superseded duplicates, and create a replacement PR for any major update that needs manual migration.

Actual:

Multiple dependency PRs are open at the same time, including overlapping Go security-library updates and overlapping npm/esbuild/Vite update paths. This makes it unclear which PR should be reviewed first and which are already superseded.

Reproduction Path

  1. Run:
gh pr list --state open --limit 30 --json number,title,headRefName,isDraft,url,updatedAt
  1. Observe the open Dependabot PR set listed above.
  2. Run web dependency installation/build validation:
npm --prefix web ci
npm --prefix web run build
  1. Observe that install/build can complete, while npm still reports moderate/high vulnerability audit output during install.
  2. Run Go vulnerability validation:
go run golang.org/x/vuln/cmd/govulncheck@latest ./...
  1. Current local validation reported no called Go vulnerabilities, so the Go PRs should be triaged by dependency freshness and CI compatibility rather than assumed runtime exploitability.

Blast Radius

  • Affected users/paths: maintainers reviewing dependency and security PRs.
  • Frequency: every CI run and dependency review until stale/superseded PRs are resolved.
  • Severity rationale: not a runtime crash; this is release hygiene and security-maintenance debt. Leaving overlapping PRs open increases merge conflicts and hides which dependency path is authoritative.
  • Workaround: manually inspect each PR and close superseded ones before reviewing the next dependency update.

Evidence

  • PR fix: restore strict security scan on main #231 restores the security scan path so dependency PRs can be judged on stable CI.
  • npm --prefix web ci && npm --prefix web run build completed locally, with npm audit output reporting 13 vulnerabilities (8 moderate, 5 high) during install.
  • go run golang.org/x/vuln/cmd/govulncheck@latest ./... completed locally with no called vulnerabilities found.
  • Open PR list shows duplicate/overlapping dependency streams, especially Go crypto updates and npm grouped updates.

Metadata

Metadata

Assignees

No one assigned

    Labels

    dependenciesPull requests that update a dependency file

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions