Skip to content

Conversation

@avivkeller
Copy link
Member

This PR improves our Dependabot groups, and adds support for updating @node-core/* packages, since Dependabot can't. (Because those packages don't currently follow semver)

Copilot AI review requested due to automatic review settings July 26, 2025 15:26
@avivkeller avivkeller requested a review from a team as a code owner July 26, 2025 15:26
@vercel
Copy link

vercel bot commented Jul 26, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
api-docs-tooling ✅ Ready (Inspect) Visit Preview Jul 26, 2025 3:37pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances dependency management by updating the Dependabot configuration and adding a custom workflow to handle @node-core/* packages that don't follow semantic versioning.

  • Adds a new GitHub workflow to automatically upgrade @node-core/* dependencies weekly
  • Reorganizes and expands Dependabot groups for better dependency management
  • Excludes @node-core/* packages from Dependabot to avoid conflicts with the custom workflow

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/update-internal-dependencies.yml New workflow to upgrade @node-core/* dependencies using npm view and jq
.github/dependabot.yml Updated dependency groups and excluded @node-core/* packages
Comments suppressed due to low confidence (1)

.github/workflows/update-internal-dependencies.yml:22

  • The actions/setup-node@v3.9.1 version does not exist. The latest v3 version is v3.8.1. Consider using v4.x.x for the most recent version.
        uses: actions/setup-node@3235b876344d2a9aa001b8d1453c930bba69e610 # v3.9.1

@avivkeller
Copy link
Member Author

We might need to use a bot pat to run the CI, since GitHub actions doesn't by default

@ovflowd
Copy link
Member

ovflowd commented Jul 26, 2025

We might need to use a bot pat to run the CI, since GitHub actions doesn't by default

Can you elaborate? 👀

Copy link
Member

@ovflowd ovflowd left a comment

Choose a reason for hiding this comment

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

SGTM

Copy link
Member

@AugustinMauroy AugustinMauroy left a comment

Choose a reason for hiding this comment

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

LGMT !

@avivkeller
Copy link
Member Author

We might need to use a bot pat to run the CI, since GitHub actions doesn't by default

Can you elaborate? 👀

It's the same reason why @nodejs-github-bot performs dependency updates in https://github.com/nodejs/node. See https://docs.github.com/en/actions/concepts/security/github_token#when-github_token-triggers-workflow-runs.

@avivkeller
Copy link
Member Author

Should I request a bot token from admin?

@ovflowd
Copy link
Member

ovflowd commented Jul 28, 2025

Should I request a bot token from admin?

Im low capacity this week, so if the bot token is the path forward? I thought the default token would be enough (the one GHA injects), if not, feel free to request a token to admin, or better saying, ask them to configure it on this repo.

@avivkeller
Copy link
Member Author

I thought the default token would be enough (the one GHA injects)

Unfortunately not, I'll open an issue.

@avivkeller avivkeller merged commit fa2e7e2 into main Aug 4, 2025
17 checks passed
@avivkeller avivkeller deleted the update-dependabot-config-internal-deps branch August 4, 2025 23:15
branch: 'upgrade-node-core-deps'
path: '.'
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Copy link
Member

Choose a reason for hiding this comment

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

As requested by nodejs/admin#991, this should be

Suggested change
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
GITHUB_TOKEN: ${{ secrets.DOC_KIT_BOT_PAT }}

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.

CI: automatically update internal dependencies (PRs) i.e.: ui-components

7 participants