test(core): cover directory barrel reachability - #1703
Merged
Conversation
- Add tests for tsconfig path alias resolution to index.ts - Add tests for baseUrl resolution to index.ts - Add tests for relative directory imports to index.ts - Add Next.js-specific test with auto-detected entries - Add direct resolver tests to verify directory resolution Tests confirm that barrel files imported via directory paths (e.g., @/features/foo -> foo/index.ts) are correctly resolved and not flagged as unused. Related to issue #1699 Co-authored-by: Skosh <skoshx@users.noreply.github.com>
Test confirms that directory imports work correctly with moduleResolution: bundler setting (used by Next.js). Co-authored-by: Skosh <skoshx@users.noreply.github.com>
commit: |
Contributor
Interactive terminal E2ETerminal Control verified the built CLI at
|
aidenybai
changed the title
aidenybai
marked this pull request as ready for review
August 30, 2026 05:13
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
3 participants
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.
Summary
The reported false positive does not reproduce on current
main. This PR adds a narrow regression to keep that behavior stable. If the current release still reports the barrel, a minimal repository is needed to identify the missing configuration edge.Validation
One unrelated timing assertion failed once during the first full run. The exact test passed three consecutive reruns, and the full suite then passed.
Closes #1699