Skip to content

chore(deps): ignore typescript majors in dependabot - #13

Merged
ashwinsingh2007 merged 1 commit into
mainfrom
chore/dependabot-ignore-typescript-7
Jul 14, 2026
Merged

chore(deps): ignore typescript majors in dependabot#13
ashwinsingh2007 merged 1 commit into
mainfrom
chore/dependabot-ignore-typescript-7

Conversation

@artiom

@artiom artiom commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

TypeScript 7.0 (the native Go compiler, tsgo) is now latest on npm (7.0.2, published 2026-07-08), but it ships without the JS compiler API. This repo lints with a typescript-eslint-based toolchain, which needs that API, so a Dependabot bump to 7.x would break linting.

This adds a Dependabot ignore rule for typescript major updates, mirroring browserless/browserless#5496. Minor and patch updates keep flowing; majors can still be applied manually once the ecosystem catches up.

Note: typescript is currently pinned at 5.9.2, so this also holds back the 5→6 major — both jumps are best done manually.

References

🤖 Generated with Claude Code


Open in Devin Review

TypeScript 7 (native/Go compiler) has no JS API yet, which breaks
the typescript-eslint toolchain. Skip major bumps until the
ecosystem supports it.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@ashwinsingh2007
ashwinsingh2007 merged commit 72fba73 into main Jul 14, 2026
3 checks passed
ashwinsingh2007 pushed a commit that referenced this pull request Jul 14, 2026
## Summary

Bumps typescript from 5.9.2 to 6.0.3 — the last JS-based release before
the native TS 7 (`tsgo`), and the recommended stepping stone toward it.
Majors are excluded from Dependabot (#13), so this is the manual bump.

One accompanying fix TS 6 requires: `"ignoreDeprecations": "6.0"` in
`tsconfig.json`, because TS 6 hard-errors (TS5107) on the deprecated
`moduleResolution: node` (node10). That option stops working entirely in
TS 7, so migrating to `node16`/`nodenext` resolution is the proper
longer-term fix — note that @n8n/node-cli's own scaffolding templates
still emit node10, so upstream hasn't migrated yet either.

## Verification

- `n8n-node build`: pass
- `n8n-node lint`: pass, no unsupported-TypeScript-version warnings
(typescript-eslint 8.58 allows `<6.1.0`)
- `tsc --noEmit`: clean
- `npm install`: single deduped typescript@6.0.3 across the tree, zero
peer warnings (@n8n/node-cli declares no typescript peer)

🤖 Generated with [Claude Code](https://claude.com/claude-code)
<!-- devin-review-badge-begin -->

---

<a
href="https://app.devin.ai/review/browserless/n8n-nodes-browserless-api/pull/14"
target="_blank">
  <picture>
<source media="(prefers-color-scheme: dark)"
srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1">
<img
src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1"
alt="Open in Devin Review">
  </picture>
</a>
<!-- devin-review-badge-end -->

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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