Skip to content

chore(renovate): hold typescript below v7 until astro check supports it - #12778

Closed
jd wants to merge 1 commit into
mainfrom
devs/jd/jd/typescript-7/hold-typescript-below-v7-until-astro-check--beb3452b
Closed

chore(renovate): hold typescript below v7 until astro check supports it#12778
jd wants to merge 1 commit into
mainfrom
devs/jd/jd/typescript-7/hold-typescript-below-v7-until-astro-check--beb3452b

Conversation

@jd

@jd jd commented Sep 11, 2026

Copy link
Copy Markdown
Member

TypeScript 7 (the native/Corsa compiler) does not ship the programmatic
API astro check relies on. astro check fails outright once typescript
resolves to 7.x:

The TypeScript module loaded (found 7.0.2) does not expose the
programmatic API that astro check relies on. TypeScript's native
compiler (7.0 and later) does not ship this API yet.

@astrojs/language-server is already at its latest published release
(2.16.16) with no newer version to pick up, so there is no fix to
upgrade into yet. astro check is the only thing type-checking .astro
files in this repo, so it stays in the lint script rather than being
dropped to make a version number fit.

Add a packageRule constraining typescript to <7.0.0 so renovate stops
proposing v7 bumps, with a description pointing at the missing API and
the upstream tracking issue so the hold reads as a decision rather than
an unexplained pin. 6.x patch releases still flow normally.

Supersedes #12585, which stays open for renovate to retire once this
lands (closing it directly would only suppress 7.0.2 specifically —
7.0.3 would come back next week, red again, with no record of why).

Upstream tracking: withastro/roadmap#1321

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com

TypeScript 7 (the native/Corsa compiler) does not ship the programmatic
API astro check relies on. astro check fails outright once typescript
resolves to 7.x:

  The TypeScript module loaded (found 7.0.2) does not expose the
  programmatic API that `astro check` relies on. TypeScript's native
  compiler (7.0 and later) does not ship this API yet.

@astrojs/language-server is already at its latest published release
(2.16.16) with no newer version to pick up, so there is no fix to
upgrade into yet. astro check is the only thing type-checking .astro
files in this repo, so it stays in the lint script rather than being
dropped to make a version number fit.

Add a packageRule constraining typescript to `<7.0.0` so renovate stops
proposing v7 bumps, with a description pointing at the missing API and
the upstream tracking issue so the hold reads as a decision rather than
an unexplained pin. 6.x patch releases still flow normally.

Supersedes #12585, which stays open for renovate to retire once this
lands (closing it directly would only suppress 7.0.2 specifically —
7.0.3 would come back next week, red again, with no record of why).

Upstream tracking: withastro/roadmap#1321

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Change-Id: Ibeb3452b2cb5e6258624031fb5e8e06022779a23
Copilot AI lite review requested due to automatic review settings September 11, 2026 04:15
@mergify
mergify Bot had a problem deploying to Mergify Merge Protections September 11, 2026 04:15 Failure
@mergify

mergify Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🔴 2 of 6 protections blocking · waiting on 👀 reviews

Protection Waiting on
🔴 👀 Review Requirements 👀 reviews
🔴 🔎 Reviews 👀 reviews
🟢 🤖 Continuous Integration
🟢 Enforce conventional commit
🟢 📕 PR description
🟢 🚦 Auto-queue

🔴 👀 Review Requirements

Waiting for

  • #approved-reviews-by >= 2
This rule is failing.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = renovate[bot]
    • all of:
      • author = mergify-ci-bot
      • -head ~= ^docs-agent/

🔴 🔎 Reviews

Waiting for

  • #review-requested = 0
This rule is failing.
  • #review-requested = 0
  • #changes-requested-reviews-by = 0
  • #review-threads-unresolved = 0

Show 4 satisfied protections

🟢 🤖 Continuous Integration

  • all of:
    • check-success = build
    • check-success = diagram-tokens
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|internal|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?!?:

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

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.

🟢 Approval recommended

The scoped configuration change has no unresolved blocking issues.

Pull request overview

This PR updates Renovate configuration to hold TypeScript below 7.0.0 until Astro supports its compiler API.

Changes:

  • Restricts TypeScript to versions below 7.0.0.
  • Documents the compatibility rationale and upstream tracking issue.
File summaries
File Summary
renovate.json Adds the TypeScript version constraint and rationale.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mergify
mergify Bot requested a review from a team September 11, 2026 04:19
@jd

jd commented Sep 11, 2026

Copy link
Copy Markdown
Member Author

ok not needed i guess

@jd jd closed this Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants