fix: @eslint/js not found issue

[JounQin]

Summary by CodeRabbit

Release Notes

• Bug Fixes
  • Resolved "@eslint/js not found" error affecting code linting functionality.

[changeset-bot]

🦋 Changeset detected

Latest commit: 070c8ca

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

This PR includes changesets to release 1 package

Name	Type
@alauda/doom	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

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e852b3b1-3e87-4979-9b4f-6b4b4e766eb2

📥 Commits

Reviewing files that changed from the base of the PR and between a70be23 and 070c8ca.

⛔ Files ignored due to path filters (1)

• yarn.lock is excluded by !**/yarn.lock, !**/*.lock

📒 Files selected for processing (3)

• .changeset/goofy-monkeys-train.md
• package.json
• packages/doom/package.json

---

Walkthrough

This PR relocates the @eslint/js dependency from the root package's devDependencies to the packages/doom package's dependencies, fixing a "not found" issue. A changeset file documenting the patch release is also added.

Changes

Cohort / File(s)	Summary
Changeset .changeset/goofy-monkeys-train.md	New changeset file added documenting a patch release for @alauda/doom with fix message regarding @eslint/js not found issue.
Dependency Migration package.json, packages/doom/package.json	Removed @eslint/js from root devDependencies and added it to packages/doom dependencies to resolve resolution issues.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• fix: stick on eslint 9 temporarily #257: Modifies ESLint-related dependency declarations across package.json files, directly paralleling this dependency reorganization.
• chore: cleanup remark lint rules #125: Updates ESLint setup configuration and related dependencies, addressing broader ESLint infrastructure changes.
• chore: bump all dependencies #278: Modifies @eslint/js dependency entries across root and packages/doom package.json files with similar scope.

Suggested labels

dependencies

Poem

🐰 A rabbit hops with glee,
Moving eslint from root to tree,
packages/doom now claims its own,
No more "not found" in the zone! 🎉
Dependencies dance, at last set free! ✨

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/eslint

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

This PR addresses an @eslint/js resolution error by ensuring the package that actually imports it (@alauda/doom) declares it as a dependency, and updates the lockfile and changeset accordingly.

Changes:

• Add @eslint/js to packages/doom dependencies.
• Remove @eslint/js from the root package.json devDependencies.
• Add a changeset to publish a patch release of @alauda/doom.

Reviewed changes

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

File	Description
yarn.lock	Reflects dependency graph changes for @eslint/js across workspaces.
packages/doom/package.json	Declares @eslint/js as a dependency for the @alauda/doom package.
package.json	Removes @eslint/js from root devDependencies (may create a phantom dependency for root tooling).
.changeset/goofy-monkeys-train.md	Adds a patch changeset for @alauda/doom describing the fix.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

[Copilot]

@eslint/js is still imported by the repo-level eslint.config.js, but it’s no longer declared in the root package.json. With Yarn’s node-modules linker this can become a phantom dependency (working only via hoisting) and may break yarn lint/CI if hoisting changes. Please either re-add @eslint/js to the root devDependencies, or refactor the root ESLint config to avoid importing it directly.