Skip to content

fix(deps): hold typescript below 7 until typescript-eslint supports it - #13

Closed
vpetersson-bot wants to merge 1 commit into
mainfrom
fix/dependabot-hold-typescript-7
Closed

vpetersson-bot wants to merge 1 commit into
mainfrom
fix/dependabot-hold-typescript-7

Conversation

@vpetersson-bot

Copy link
Copy Markdown

Problem

The monthly bun group Dependabot PR in this repo is red, and has been since the group started including typescript 7. The checks job fails at the Lint step with:

ESLint: 10.2.0
TypeError: Cannot read properties of undefined (reading 'Cjs')
    at node_modules/@typescript-eslint/typescript-estree/dist/create-program/shared.js:59
    ...
    at node_modules/@screenly/edge-apps/eslint.config.ts:4:54
error: script "lint" exited with code 1

Cause

@typescript-eslint/typescript-estree builds its default extension set from ts.Extension.Cjs. TypeScript 7 no longer exposes Extension on the public API surface, so ts.Extension is undefined and the module throws at import time — before any file is linted.

This is not a version-skew problem that a newer typescript-eslint fixes. Every published version, up to and including the current 8.70.0, declares:

"peerDependencies": { "typescript": ">=4.8.4 <6.1.0" }

No released typescript-eslint supports TypeScript 7 yet. typescript-eslint reaches this repo transitively through the shared ESLint config in @screenly/edge-apps, so there is nothing to bump here to make TypeScript 7 work.

Fix

Have Dependabot hold typescript below 7 until typescript-eslint supports it. Everything else in the group keeps flowing.

Verification

Reproduced and verified locally on Screenly/clock-app against the open group PR. With typescript at ^7.0.2, bun run lint fails as above. Pinning typescript back to ^6.0.3 while keeping all eight other updates in the group:

bun run lint        ✓
bun run type-check  ✓
bun run test        ✓  7 pass, 0 fail
bun run build       ✓

After merging

Comment @dependabot recreate on the open group PR — it will come back without the typescript bump and should go green.

The same breakage affects every Screenly edge app plus edge-apps-library; an identical change is going up in each.

🤖 Generated with Claude Code

typescript-eslint has no released version that supports TypeScript 7 --
even 8.70.0 declares a peer range of `typescript >=4.8.4 <6.1.0`. Linting
under TS 7 throws `Cannot read properties of undefined (reading 'Cjs')`
from @typescript-eslint/typescript-estree, which is what keeps the monthly
bun group PR red.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 13, 2026 08:10

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vpetersson-bot

Copy link
Copy Markdown
Author

Duplicate of #12, which already proposes the same fix. Closing this one in favour of the earlier PR.

@vpetersson-bot
vpetersson-bot deleted the fix/dependabot-hold-typescript-7 branch September 13, 2026 08:15
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