Skip to content

chore: fix renovate age-gate gaps and add missing dep-type rules - #630

Merged
akudev merged 6 commits into
mainfrom
main_2609b
Sep 21, 2026
Merged

akudev merged 6 commits into
mainfrom
main_2609b

Conversation

@akudev

@akudev akudev commented Sep 21, 2026

Copy link
Copy Markdown
Member
  • lockFileMaintenance: add minimumReleaseAge "0 days" (Renovate cannot evaluate transitive dep ages → permanent "pending" status without it; pnpm's install-time gate enforces the real cooldown)
  • lockFileMaintenance: enable automerge
  • vulnerabilityAlerts: enable automerge (3-day delay still buffers)
  • Group rule: add explicit minimumReleaseAge "7 days" (does not inherit)
  • Add peerDependencies widen rule (ts-interface-generator)
  • Add engines exclusion rule (dts-generator)

- lockFileMaintenance: add minimumReleaseAge "0 days" (Renovate cannot
  evaluate transitive dep ages → permanent "pending" status without it;
  pnpm's install-time gate enforces the real cooldown)
- lockFileMaintenance: enable automerge
- vulnerabilityAlerts: enable automerge (3-day delay still buffers)
- Group rule: add explicit minimumReleaseAge "7 days" (does not inherit)
- Add peerDependencies widen rule (ts-interface-generator)
- Add engines exclusion rule (dts-generator)
- Bump pnpm minimumReleaseAge from 3 days to 7 days

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The required empty changeset is missing, so the Changeset workflow will fail.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 High severity

Open (1)
What changed in this PR

Updates dependency automation and supply-chain cooldown policies.

Changes:

  • Enables automerge for vulnerability and lockfile updates.
  • Adds dependency-type handling and explicit age gates.
  • Raises pnpm’s release-age gate to seven days.
File Description
renovate.json Refines Renovate rules, age gates, and automerge behavior.
pnpm-workspace.yaml Raises the installation cooldown to seven days.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread renovate.json
@akudev
akudev requested review from codeworrior and petermuessig and a balanced review from Copilot September 21, 2026 11:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The seven-day pnpm gate conflicts with Renovate’s three-day vulnerability-update window and can block security automerge.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
Resolved since last review (1)
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

Comment thread pnpm-workspace.yaml Outdated
"not-pending" waits for CI status checks on the bare branch before
opening a PR, but CI only triggers on pull_request events. Result:
8 updates stuck in "Pending Status Checks" on the dashboard, including
a security fix for sanitize-html that was autoclosed (#616).
The 7-day install-time gate conflicts with Renovate's 3-day vulnerability
alert window: security fix PRs would fail CI until the package is 7 days
old, defeating the fast-track. Keep 3 days to align both gates.
The lockfile was regenerated for the 7-day age gate which has been
reverted to 3 days. Restore the original versions.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The configuration changes are consistent with pnpm’s install-time release-age enforcement.

Review effort: Balanced
Findings: None

Resolved since last review (1)

@akudev
akudev merged commit c337648 into main Sep 21, 2026
8 checks passed
@akudev
akudev deleted the main_2609b branch September 21, 2026 14:56
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.

3 participants