Skip to content

chore(deps): scope Dependabot to root + packages/*, exclude vendored submodules#89

Merged
jhfnetboy merged 2 commits intomainfrom
chore/dependabot-scope-excludes
Apr 26, 2026
Merged

chore(deps): scope Dependabot to root + packages/*, exclude vendored submodules#89
jhfnetboy merged 2 commits intomainfrom
chore/dependabot-scope-excludes

Conversation

@jhfnetboy
Copy link
Copy Markdown
Member

Summary

Test plan

  • Config parses (Dependabot will report via the Insights → Dependency graph tab).
  • Next weekly run does not open PRs under singleton-paymaster/lib/**.
  • Next weekly run still opens PRs for real deps in / and /packages/* when updates are available.

…submodules

Dependabot kept opening PRs for `singleton-paymaster/lib/**` package.json
files (transitive deps of upstream tests/docs tooling we never install,
execute, or bundle). Closing each one is noise.

- Add explicit glob negations for `singleton-paymaster/**`, `contracts/lib/**`,
  and `lib/**` to keep scanning scoped to our code.
- Also add `/packages/*` so real workspace deps (e.g. x402-facilitator-node)
  are still monitored.
- Document in the top comment what to do if noise PRs do slip through.
@jhfnetboy jhfnetboy requested a review from fanhousanbu as a code owner April 16, 2026 12:57
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Expand directories scope to cover our actual js surface (subgraph indexer
and gasless e2e helpers) and exclude vendored standards/ submodules.

Vendored submodule package.json files belong to upstream tooling we
never install or run, so they should never trigger version-update PRs.
Note: this does not stop GitHub Security Alerts (CVE-driven) which
ignore directories scoping by platform design — those still need to be
closed manually.
Copy link
Copy Markdown
Contributor

@fanhousanbu fanhousanbu left a comment

Choose a reason for hiding this comment

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

几个小问题确认一下:

  1. /subgraph/script/gasless-tests 是新加的,PR 描述里没提——这两个路径有 package.json 吗?没有的话可以去掉,避免冗余配置。

  2. !/standards/** 也没在描述里说明,这个目录是 vendored 内容还是项目自有代码?

  3. open-pull-requests-limit: 1 现在监控了多个目录,一个 PR 没合就会阻塞其他目录的更新,是有意保守设定吗?

@jhfnetboy jhfnetboy merged commit f23bd43 into main Apr 26, 2026
3 checks passed
@jhfnetboy jhfnetboy deleted the chore/dependabot-scope-excludes branch April 26, 2026 04:49
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