Repository navigation
chore(deps): minimatch (patch → 10.2.6) [integration-tests/package.json] - #1487
Conversation
❌ ErrorsYour PR has failed checks. Please review the issues below and take necessary action before merging. 🚦 2 Pipeline jobs failed
Useful? React with 👍 / 👎 This comment will be updated automatically if new data arrives.🔗 Commit SHA: 034b763 | Docs | View more details | Give us feedback! |
Auto-rebase completeBranch is up to date with Auto-Rebase · Add |
b2e90ce to
89cc865
Compare
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
PRApprover will approve and merge this PR, FAQ, #dx-source-code-management 🛠️ PRApproval Status
➡️ Current phase: waiting to merge... |
There was a problem hiding this comment.
🟡 Changes recommended
The lockfile still records the bundled minimatch dependency at 10.2.5 with no 10.2.6 entry, leaving the intended dependency update unapplied while unrelated packages change.
1 open issue
1 file not reviewed
integration-tests/package-lock.json: Generated file
What changed in this PR
This PR refreshes dependencies for the Lambda extension’s integration tests. Although titled as a minimatch patch update, the diff changes dependency overrides and other locked packages.
Changes:
- Removes several OpenTelemetry, AWS SDK, and CDK overrides and adds an XML parser override under
@aws-sdk/xml-builder. - Updates XML-related dependencies and several other transitive packages.
- Consolidates OpenTelemetry semantic conventions entries at version 1.28.0.
| File | Description |
|---|---|
integration-tests/package.json |
Revises transitive dependency overrides. |
integration-tests/package-lock.json |
Refreshes dependency versions and consolidates OpenTelemetry entries. |
Review effort: Balanced
| "@aws-sdk/xml-builder": { | ||
| "fast-xml-parser": "^5.8.0" |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 89cc865d1d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "@aws-sdk/xml-builder": { | ||
| "fast-xml-parser": "^5.8.0" | ||
| } |
There was a problem hiding this comment.
Resolve the advertised minimatch 10.2.6 update
This manifest rewrite does not perform the stated dependency update: the resulting integration-tests/package-lock.json still records the bundled aws-cdk-lib/node_modules/minimatch as 10.2.5 at lines 2587–2592, exactly as in the parent, so npm ci continues installing 10.2.5. Regenerate the dependency tree so it actually resolves 10.2.6—likely through an aws-cdk-lib release that bundles it—and update the summary to describe the unrelated override and transitive-package changes accurately.
AGENTS.md reference: AGENTS.md:L30-L31
Useful? React with 👍 / 👎.
Co-authored-by: dd-octo-sts-019303[bot] <256648753+dd-octo-sts-019303[bot]@users.noreply.github.com>
Co-authored-by: dd-octo-sts-019303[bot] <256648753+dd-octo-sts-019303[bot]@users.noreply.github.com>
Co-authored-by: gh-worker-campaigns-3e9aa4[bot] <244854796+gh-worker-campaigns-3e9aa4[bot]@users.noreply.github.com>
Auto-rebase completeBranch is up to date with Auto-Rebase · Add |
89cc865 to
034b763
Compare
new commit added, dismiss all previous approvals

Warning
Shotclock auto-merge countdown
This PR may be automatically approved and merged on or after November 12, 2026 (UTC) if nobody acts and all required Shotclock eligibility checks pass.
To stop Shotclock, comment on this PR, request changes, push a commit, or close the PR. Converting it to draft pauses automation.
Summary: Dependency update — 1 package upgraded (patch changes only)
Manifests changed:
integration-tests/package.json(npm)✅ Action Required: Please review the changes below. If they look good, approve and merge this PR.
Updates
Review Checklist
Standard review:
Update Mode: all_vulns
🤖 Generated by DataDog Automated Dependency Management System