chore(deps): bump astro-eslint-parser 1.4.0 → 2.1.0 - #12093
Merged
mergify[bot] merged 1 commit intoJul 15, 2026
Merged
Conversation
Contributor
Merge Protections🟢 All 6 merge protections satisfied — ready to merge. Show 6 satisfied protections🟢 🤖 Continuous Integration
🟢 👀 Review Requirements
🟢 Enforce conventional commitMake sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 Reviews
🟢 📕 PR description
🟢 🚦 Auto-queueWhen all merge protections are satisfied, this pull request will be queued automatically. |
There was a problem hiding this comment.
Pull request overview
Upgrades the astro-eslint-parser dev dependency to v2.1.0 and updates the ESLint flat config to account for the package becoming ESM-only with no default export, unblocking lint failures seen in the earlier Dependabot attempt.
Changes:
- Bump
astro-eslint-parserfrom^1.4.0to^2.1.0. - Update
eslint.config.jsto import the parser via a namespace import (import * as astroParser) so ESLint receives the expected parser object. - Refresh
pnpm-lock.yamlto reflect the new dependency graph (including the reduced transitive deps mentioned in the release notes).
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
package.json |
Updates astro-eslint-parser version range to ^2.1.0. |
eslint.config.js |
Switches from default import to namespace import to match v2’s ESM named exports. |
pnpm-lock.yaml |
Lockfile update for astro-eslint-parser@2.1.0 and related transitive dependency changes. |
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jd
marked this pull request as ready for review
July 13, 2026 10:17
kozlek
previously approved these changes
Jul 13, 2026
Contributor
|
@jd this pull request is now in conflict 😩 |
Major upgrade of the `astro-eslint-parser` dev dependency. Supersedes the failing Dependabot PR #12007, whose `lint` check was red because v2 is a breaking change. Breaking changes handled: - **v2 is now a pure named-export ESM module** — it no longer provides a `default` export. The old `import astroParser from 'astro-eslint-parser'` threw `SyntaxError: The requested module 'astro-eslint-parser' does not provide an export named 'default'` when ESLint loaded the flat config, which is exactly the failing `lint` check. Switched to a namespace import (`import * as astroParser`), which exposes `parseForESLint`/`meta` — the shape ESLint's flat-config `parser` expects. - **v2 drops old Node** (now requires `^22.22.3 || ^24.16.0 || >=26.3.0`). The repo's `.node-version` pins `24`, which CI resolves to the latest 24.x (well past 24.16), so no bump is needed. - v2.1.0 swaps fast-glob/is-glob for tinyglobby (transitive only); the lockfile shrinks accordingly, no source changes required. `pnpm check` (astro check + eslint + biome) passes clean locally. Supersedes #12007. Change-Id: Ieb581ee0f89a9f22c78f705ede03bfec1b479275
Member
Author
Revision history
|
jd
force-pushed
the
devs/jd/jd/astro-eslint-parser-2/bump-astro-eslint-parser-1-4-0-2-1-0--eb581ee0
branch
from
July 15, 2026 08:54
343a847 to
f0d6fe6
Compare
kozlek
approved these changes
Jul 15, 2026
sileht
approved these changes
Jul 15, 2026
Contributor
Merge Queue Status
This pull request spent 2 minutes 51 seconds in the queue, including 2 minutes 17 seconds running CI. Required conditions to merge
|
53 tasks
mergify
Bot
deleted the
devs/jd/jd/astro-eslint-parser-2/bump-astro-eslint-parser-1-4-0-2-1-0--eb581ee0
branch
July 15, 2026 09:21
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Major upgrade of the
astro-eslint-parserdev dependency. Supersedes thefailing Dependabot PR #12007, whose
lintcheck was red because v2 is abreaking change.
Breaking changes handled:
defaultexport. The oldimport astroParser from 'astro-eslint-parser'threw
SyntaxError: The requested module 'astro-eslint-parser' does not provide an export named 'default'when ESLint loaded the flat config,which is exactly the failing
lintcheck. Switched to a namespace import(
import * as astroParser), which exposesparseForESLint/meta— theshape ESLint's flat-config
parserexpects.^22.22.3 || ^24.16.0 || >=26.3.0).The repo's
.node-versionpins24, which CI resolves to the latest24.x (well past 24.16), so no bump is needed.
lockfile shrinks accordingly, no source changes required.
pnpm check(astro check + eslint + biome) passes clean locally.Supersedes #12007.