Skip to content

ci(dependabot): ignore the typescript, @types/node, nx and dotenv majors - #54

Merged
DealAppSeo merged 1 commit into
mainfrom
claude/cc1-2026-10-14-dependabot-ignore-majors
Oct 14, 2026
Merged

DealAppSeo merged 1 commit into
mainfrom
claude/cc1-2026-10-14-dependabot-ignore-majors

Conversation

@DealAppSeo

Copy link
Copy Markdown
Owner

Config only. Minor and patch bumps were already grouped; this adds the ignore block the other repos carry (trustshell #513, trinity-symphony-shared #71, repid-engine #1311), so the reviewed majors are not re-raised once their PRs are closed.

Ignored major Open PRs it covers Why
typescript 5 → 7 #42, #46, #48, #49 TypeScript 7 is the native-compiler rewrite. Not in a published-SDK repo ahead of a release.
@types/node 22 → 26 #43, #47 CI runs Node 22.
nx 22 → 23 #44 Major toolchain bump, no driver.
dotenv 17 → 18 #45, #50 Major, no driver.

#46, #47 and #50 also fail clean install from lockfile and contracts tests. They bump packages/contracts/package.json and its nested lockfile but not the root package-lock.json, so npm ci at the root fails. Those are real failures, but they are moot once the majors are closed.

Delete an ignore entry when the repo is ready to take that major. github-actions has no ignores.

🤖 Generated with Claude Code

Minor+patch were already grouped. This adds the ignore block the other repos
carry (trustshell #513, trinity-symphony-shared #71, repid-engine #1311) so the
reviewed majors (#42-#50) are not re-raised after they are closed. TypeScript 7
is the native-compiler rewrite and must not land in a published-SDK repo ahead
of a release.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@strix-security

strix-security Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Strix Security Review

No security issues found.

Review summary

Reviewed the single changed file, .github/dependabot.yml. The PR adds a Dependabot ignore block that suppresses version-update:semver-major PRs for four npm packages (typescript, @types/node, nx, dotenv), each justified in the PR description (native-compiler rewrite, CI/runtime version alignment, and un-drivered major bumps). This is a build/CI configuration change with no effect on application runtime behavior. It neither introduces nor removes a security control: the ignore rules apply only to Dependabot's version updates, not to its separate security/advisory update mechanism, and github-actions remains unignored. No injection, auth, secret, deserialization, path traversal, SSRF, XSS, crypto, race-condition, data-exposure, mass-assignment, IDOR, open-redirect, ReDoS, or prototype-pollution surface is touched. No security issues found.

Updated for 946a0f9.


Reviewed by Strix
Re-run review · Configure security review settings

@DealAppSeo
DealAppSeo merged commit 70fe74e into main Oct 14, 2026
9 checks passed
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.

1 participant