Improve non-JS identifier error message: Phase 2

[mizdra]

follow-up: #220

[changeset-bot]

🦋 Changeset detected

Latest commit: 1000693

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages

Name	Type
@css-modules-kit/core	Minor
@css-modules-kit/codegen	Patch
@css-modules-kit/ts-plugin	Patch
@css-modules-kit/stylelint-plugin	Patch
@css-modules-kit/eslint-plugin	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[Copilot]

Pull Request Overview

This PR improves error handling for non-JavaScript identifiers by moving validation from individual parsers to a centralized location. The validation now happens during semantic checking and type definition generation, providing better error messages and preventing invalid identifiers from being processed.

• Moves non-JS identifier validation from individual parsers (rule, keyframe, at-value) to the checker and DTS creator
• Updates error message from "non-JavaScript identifier" to "invalid names as JavaScript identifiers"
• Consolidates validation logic using a new isValidAsJSIdentifier utility function

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
packages/core/src/util.ts	Adds isValidAsJSIdentifier utility function to centralize validation logic
packages/core/src/checker.ts	Implements validation checks for local tokens and imported values with improved error messages
packages/core/src/dts-creator.ts	Filters out invalid identifiers during type definition generation
packages/core/src/parser/*.ts	Removes individual validation logic from parsers
packages/ts-plugin/e2e/feature/*.test.ts	Updates tests to reflect validation moved from syntactic to semantic phase
packages/core/src/parser/*.test.ts	Removes test cases that are no longer applicable