Skip to content

Conversation

@marklawlor
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings October 11, 2025 04:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a warning mechanism to detect and prevent the use of lightningcss version 1.30.2, which contains a critical bug that breaks compilation. The change implements version checking logic that throws an error when the problematic version is detected.

Key changes:

  • Added version detection for lightningcss package to prevent usage of buggy version 1.30.2
  • Moved ESLint disable comment to file level for better code organization

) as typeof import("lightningcss");

try {
const lightningcssPackageJSONPath = require.resolve("../../package.json", {
Copy link

Copilot AI Oct 11, 2025

Choose a reason for hiding this comment

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

The require.resolve path '../../package.json' is incorrect for resolving the lightningcss package.json. It should be 'lightningcss/package.json' to resolve the actual lightningcss package's package.json file.

Suggested change
const lightningcssPackageJSONPath = require.resolve("../../package.json", {
const lightningcssPackageJSONPath = require.resolve("lightningcss/package.json", {

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@marklawlor marklawlor enabled auto-merge (squash) October 11, 2025 04:47
@marklawlor marklawlor merged commit 53249fe into main Oct 11, 2025
6 checks passed
@marklawlor marklawlor deleted the 10-11-fix_add_warning_for_lightningcss_1.30.2-recreation branch October 11, 2025 04:48
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