fix(package): remove config and rules exports pointing to empty files #406

marcalexiei · 2025-07-02T06:41:24Z

closes config and rules package exports point to non-existing files #405

Important

Remove config and rules exports from package.json as they pointed to empty files.

Exports Removal:
- Removed ./config and ./rules exports from package.json as they pointed to empty files.

^{This description was created by}^{for dd7103d. You can customize this summary. It will automatically update as commits are pushed.}

Summary by CodeRabbit

Chores
- Cleaned up package exports by removing unused configuration and rules exports, streamlining the available export paths.

changeset-bot · 2025-07-02T06:41:27Z

🦋 Changeset detected

Latest commit: dd7103d

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

This PR includes changesets to release 1 package

Name	Type
eslint-plugin-import-x	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 · 2025-07-02T06:41:30Z

Walkthrough

This change removes the ./config and ./rules subpath exports from the package.json file, as these paths pointed to non-existent files. A corresponding changeset is added to document this patch update for the eslint-plugin-import-x package.

Changes

Files / Grouped Files	Change Summary
package.json	Removed `./config` and `./rules` subpath exports from the "exports" field.
.changeset/swift-friends-remain.md	Added a changeset documenting the removal of the two exports and the resulting patch update.

Assessment against linked issues

Objective	Addressed	Explanation
Remove or fix `config` and `rules` package `exports` that point to non-existing files (#405)	✅

Suggested labels

bug, internal

Suggested reviewers

JounQin

Poem

In the warren of code, a patch hops in,
Two exports removed, let clarity begin!
No more paths to nowhere, the garden is neat,
With swift little changes, the package is sweet.
🐇✨

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e50c5a6 and dd7103d.

📒 Files selected for processing (2)

.changeset/swift-friends-remain.md (1 hunks)
package.json (0 hunks)

💤 Files with no reviewable changes (1)

package.json

🧰 Additional context used

🧠 Learnings (2)

📓 Common learnings

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#378
File: src/rules/imports-first.ts:10-19
Timestamp: 2025-06-08T12:09:38.535Z
Learning: The current implementation in eslint-plugin-import-x uses the correct ESLint core DeprecatedInfo structure: deprecatedSince field and replacedBy array with objects containing rule properties that match the ExternalSpecifier type with name and url fields.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#378
File: src/rules/imports-first.ts:10-19
Timestamp: 2025-06-08T12:09:38.535Z
Learning: ESLint core ExternalSpecifier interface has optional name and url string properties. The current eslint-plugin-import-x implementation correctly uses this structure for the rule property within replacedBy arrays in DeprecatedInfo objects.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#357
File: package.json:160-160
Timestamp: 2025-05-30T15:39:53.248Z
Learning: The `typescript-eslint` package (without the `@typescript-eslint/` namespace) is the official unified entry package for ESLint flat config, not an incorrect package name.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#362
File: src/utils/create-rule.ts:0-0
Timestamp: 2025-05-31T03:10:38.972Z
Learning: When `rewriteRelativeImportExtensions` is enabled in TypeScript configuration, using `.ts` extensions in import paths is correct and necessary for Node.js ESM compatibility. TypeScript will rewrite these to `.js` during compilation.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#272
File: test/rules/no-unresolved.spec.ts:293-325
Timestamp: 2025-05-30T13:24:29.760Z
Learning: Node.js ES6 import statements do not support plain Windows absolute paths like `C:\path\to\file.js` because import specifiers are resolved as URLs, not plain file paths. Plain Windows absolute paths conflict with URL schemes and create ambiguity. For absolute imports on Windows, you must use `file://` URLs instead.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#385
File: src/rules/prefer-namespace-import.ts:0-0
Timestamp: 2025-06-18T12:49:41.795Z
Learning: The eslint-plugin-import-x project requires Node versions `^18.18.0 || ^20.9.0 || >=21.1.0` as specified in package.json, so modern JavaScript features like String.prototype.at() are safe to use.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#386
File: src/rules/prefer-namespace-import.ts:41-46
Timestamp: 2025-06-18T15:22:38.532Z
Learning: In eslint-plugin-import-x, JounQin prefers to throw on invalid rule options rather than handling them gracefully with try/catch blocks and reporting configuration errors.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#271
File: test/rules/no-unused-modules.spec.ts:1528-1532
Timestamp: 2025-03-30T09:06:59.006Z
Learning: The import from 'eslint8.56/use-at-your-own-risk' has incorrect TypeScript types but works correctly at runtime, which is properly handled with a `@ts-expect-error` comment.

.changeset/swift-friends-remain.md (6)

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#378
File: src/rules/imports-first.ts:10-19
Timestamp: 2025-06-08T12:09:38.535Z
Learning: The current implementation in eslint-plugin-import-x uses the correct ESLint core DeprecatedInfo structure: deprecatedSince field and replacedBy array with objects containing rule properties that match the ExternalSpecifier type with name and url fields.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#378
File: src/rules/imports-first.ts:10-19
Timestamp: 2025-06-08T12:09:38.535Z
Learning: ESLint core ExternalSpecifier interface has optional name and url string properties. The current eslint-plugin-import-x implementation correctly uses this structure for the rule property within replacedBy arrays in DeprecatedInfo objects.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#386
File: src/rules/prefer-namespace-import.ts:41-46
Timestamp: 2025-06-18T15:22:38.532Z
Learning: In eslint-plugin-import-x, JounQin prefers to throw on invalid rule options rather than handling them gracefully with try/catch blocks and reporting configuration errors.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#357
File: package.json:160-160
Timestamp: 2025-05-30T15:39:53.248Z
Learning: The `typescript-eslint` package (without the `@typescript-eslint/` namespace) is the official unified entry package for ESLint flat config, not an incorrect package name.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#385
File: src/rules/prefer-namespace-import.ts:0-0
Timestamp: 2025-06-18T12:49:41.795Z
Learning: The eslint-plugin-import-x project requires Node versions `^18.18.0 || ^20.9.0 || >=21.1.0` as specified in package.json, so modern JavaScript features like String.prototype.at() are safe to use.

Learnt from: JounQin
PR: un-ts/eslint-plugin-import-x#271
File: test/rules/no-unused-modules.spec.ts:1528-1532
Timestamp: 2025-03-30T09:06:59.006Z
Learning: The import from 'eslint8.56/use-at-your-own-risk' has incorrect TypeScript types but works correctly at runtime, which is properly handled with a `@ts-expect-error` comment.

⏰ Context from checks skipped due to timeout of 90000ms (21)

GitHub Check: Codacy Static Code Analysis
GitHub Check: Lint and Test with Node.js 22 and ESLint 8.56 on windows-latest
GitHub Check: Lint and Test with Node.js 22 and ESLint 9 on windows-latest
GitHub Check: Lint and Test with Node.js 18 and ESLint 8 on windows-latest
GitHub Check: Lint and Test with Node.js 20 and ESLint 8.56 on windows-latest
GitHub Check: Lint and Test with Node.js 22 and ESLint 8 on windows-latest
GitHub Check: Lint and Test with Node.js 20 and ESLint 9 on ubuntu-latest
GitHub Check: Lint and Test with Node.js 22 and ESLint 8 on ubuntu-latest
GitHub Check: Lint and Test with Node.js 24 and ESLint 8.56 on ubuntu-latest
GitHub Check: Lint and Test with Node.js 24 and ESLint 9 on ubuntu-latest
GitHub Check: Lint and Test with Node.js 20 and ESLint 8 on ubuntu-latest
GitHub Check: Lint and Test with Node.js 22 and ESLint 9 on ubuntu-latest
GitHub Check: Lint and Test with Node.js 18 and ESLint 9 on ubuntu-latest
GitHub Check: Lint and Test with Node.js 20 and ESLint 8.56 on ubuntu-latest
GitHub Check: Lint and Test with Node.js 18 and ESLint 8.56 on ubuntu-latest
GitHub Check: Lint and Test with Node.js 18 and ESLint 8 on ubuntu-latest
GitHub Check: Lint and Test with Node.js 24 and ESLint 8.56 on macos-latest
GitHub Check: Lint and Test with Node.js 18 and ESLint 8 on macos-latest
GitHub Check: Lint and Test with Node.js 20 and ESLint 9 on macos-latest
GitHub Check: Lint and Test with Node.js 18 and ESLint 9 on macos-latest
GitHub Check: Lint and Test with Node.js 20 and ESLint 8.56 on macos-latest

🔇 Additional comments (1)

.changeset/swift-friends-remain.md (1)

1-6: Changeset correctly documents patch release

The front-matter syntax and summary line follow Changesets conventions. Looks good.

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

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Explain this complex logic.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai explain this code block.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and explain its main purpose.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR.
@coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

codesandbox-ci · 2025-07-02T06:41:52Z

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

pkg-pr-new · 2025-07-02T06:42:11Z

Open in StackBlitz

npm i https://pkg.pr.new/eslint-plugin-import-x@406

commit: dd7103d

ellipsis-dev

Important

Looks good to me! 👍

Reviewed everything up to dd7103d in 51 seconds. Click for details.

Reviewed 24 lines of code in 2 files
Skipped 0 files when reviewing.
Skipped posting 2 draft comments. View those below.
Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. .changeset/swift-friends-remain.md:1

Draft comment:
Changeset file format looks correct.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

2. package.json:29

Draft comment:
Removed './config' and './rules' export entries correctly to resolve issue #405. Ensure consumers don't depend on these exports.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_SmmCgX9www4FdFYE

^{You can customize}^{by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

fix(package): remove config and rules exports pointing to empty files

dd7103d

ellipsis-dev bot reviewed Jul 2, 2025

View reviewed changes

JounQin approved these changes Jul 2, 2025

View reviewed changes

JounQin merged commit d0a7816 into un-ts:master Jul 2, 2025
46 checks passed

github-actions bot mentioned this pull request Jul 2, 2025

chore: release eslint-plugin-import-x #407

Open

marcalexiei deleted the remove-invalid-exports branch July 2, 2025 13:03

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

fix(package): remove config and rules exports pointing to empty files #406

fix(package): remove config and rules exports pointing to empty files #406

marcalexiei commented Jul 2, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

changeset-bot bot commented Jul 2, 2025

Uh oh!

coderabbitai bot commented Jul 2, 2025 •

edited

Loading

Chat

Support

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (`.coderabbit.yaml`)

Documentation and Community

Uh oh!

codesandbox-ci bot commented Jul 2, 2025

Uh oh!

pkg-pr-new bot commented Jul 2, 2025

Uh oh!

ellipsis-dev bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

fix(package): remove config and rules exports pointing to empty files #406

fix(package): remove config and rules exports pointing to empty files #406

Conversation

marcalexiei commented Jul 2, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

changeset-bot bot commented Jul 2, 2025

🦋 Changeset detected

Uh oh!

coderabbitai bot commented Jul 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Assessment against linked issues

Suggested labels

Suggested reviewers

Poem

Chat

Support

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

Uh oh!

codesandbox-ci bot commented Jul 2, 2025

Uh oh!

pkg-pr-new bot commented Jul 2, 2025

Uh oh!

ellipsis-dev bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

marcalexiei commented Jul 2, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Jul 2, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)