fix(bundling): consider index/folder imports in manual file resolution

[JanPretzel]

Current Behavior

nxViteTsPaths manual fallback resolution does not take index/folder imports into account.

Expected Behavior

nxViteTsPaths manual fallback resolution should be able to resolve index/folder imports when importing via a configured TS path with placeholder.

Related Issue(s)

Fixes #19028

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
nx-dev	⬜️ Ignored (Inspect)	Visit Preview		Sep 18, 2023 8:28am

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 8f32bba. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 8 targets

• nx affected --target=e2e --base=fcf5b9cec39cd930f1307fc9da5b2e0bb0740ea3 --head=8f32bbaeec56c8bb28ec75a1008000a839d7ee66 --parallel=1
• nx affected --target=test --base=fcf5b9cec39cd930f1307fc9da5b2e0bb0740ea3 --head=8f32bbaeec56c8bb28ec75a1008000a839d7ee66 --parallel=1
• nx affected --target=build --base=fcf5b9cec39cd930f1307fc9da5b2e0bb0740ea3 --head=8f32bbaeec56c8bb28ec75a1008000a839d7ee66 --parallel=3
• nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• nx affected --target=lint --base=fcf5b9cec39cd930f1307fc9da5b2e0bb0740ea3 --head=8f32bbaeec56c8bb28ec75a1008000a839d7ee66 --parallel=3
• nx-cloud record -- nx format:check --base=fcf5b9cec39cd930f1307fc9da5b2e0bb0740ea3 --head=8f32bbaeec56c8bb28ec75a1008000a839d7ee66
• nx documentation --no-dte
• nx affected -t e2e-macos --parallel=1 --base=fcf5b9cec39cd930f1307fc9da5b2e0bb0740ea3 --head=8f32bbaeec56c8bb28ec75a1008000a839d7ee66

---

Sent with 💌 from NxCloud.

[senputra]

Bump! This is a good fix. I need this.

[christowiz]

I spent a couple days tracking down this issue as well.

[sgtsquiggs]

is there anything blocking this fix from being merged?

[mandarini]

thank you for this @JanPretzel

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.