fix(js): identify correct circular dependecies

[xiongemi]

Current Behavior

currently, when detecting circular dep in release, it tests dependentProject.target === projectName, which adds all projects as circular deps. because filter allDependentProjects is localPackageDependency.target === projectName, so dependentProject.target === projectName is always true

Expected Behavior

correct the logic of detecting circular deps, it should test

• current project.target -> a (source: current project, target: a)
• a.target -> current project (source: a, target: current project)

Related Issue(s)

Fixes #28336

[vercel]

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

1 Skipped Deployment

Name	Status	Preview	Updated (UTC)
nx-dev	⬜️ Ignored (Inspect)		Jan 27, 2025 5:48am

[nx-cloud]

View your CI Pipeline Execution ↗ for commit 19136bf.

Command	Status	Duration	Result
nx affected --targets=lint,test,build,e2e,e2e-c...	❌ Failed	40m 25s	View ↗
nx run-many -t check-imports check-commit check...	✅ Succeeded	59s	View ↗
nx-cloud record -- nx-cloud conformance:check	✅ Succeeded	1s	View ↗
nx-cloud record -- nx format:check --base=a8aba...	✅ Succeeded	27s	View ↗
nx-cloud record -- nx sync:check	✅ Succeeded	25s	View ↗
nx documentation --no-dte	✅ Succeeded	43s	View ↗

---

☁️ Nx Cloud last updated this comment at 2025-01-27 06:27:13 UTC