fix(core): fix dependent projects for watch

[Cammisuli]

Current Behavior

Expected Behavior

Related Issue(s)

Fixes #

[vercel]

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

Name	Status	Preview	Updated
nx-dev	✅ Ready (Inspect)	Visit Preview	Jan 11, 2023 at 3:19PM (UTC)

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit fdb7dba. 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

---

⌛ The following targets are in progress

• nx affected --target=e2e --base=b0c6ca92679798f84a34b6bf52bae081e1b28440 --head=e32d25bce0565f3e7713caaf3687e44d1e4f7787 --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox --parallel=1
• nx affected -t test lint --base=b0c6ca92679798f84a34b6bf52bae081e1b28440 --head=e32d25bce0565f3e7713caaf3687e44d1e4f7787 --parallel=1

✅ Successfully ran 8 targets

• nx affected --target=e2e --base=b0c6ca92679798f84a34b6bf52bae081e1b28440 --head=fdb7dba0567bdc790829e610c7207d0f120c9716 --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox --parallel=1
• nx affected -t test lint --base=b0c6ca92679798f84a34b6bf52bae081e1b28440 --head=fdb7dba0567bdc790829e610c7207d0f120c9716 --parallel=1
• nx affected --target=build --base=b0c6ca92679798f84a34b6bf52bae081e1b28440 --head=fdb7dba0567bdc790829e610c7207d0f120c9716 --parallel=3
• nx run-many -t check-imports check-commit check-lock-files root-lint depcheck documentation --parallel=3 --no-dte
• nx-cloud record -- nx format:check --base=b0c6ca92679798f84a34b6bf52bae081e1b28440 --head=fdb7dba0567bdc790829e610c7207d0f120c9716
• nx affected --target=build --base=b0c6ca92679798f84a34b6bf52bae081e1b28440 --head=e32d25bce0565f3e7713caaf3687e44d1e4f7787 --parallel=3
• nx run-many -t check-imports check-commit check-lock-files root-lint depcheck documentation --parallel=3 --no-dte
• nx-cloud record -- nx format:check --base=b0c6ca92679798f84a34b6bf52bae081e1b28440 --head=e32d25bce0565f3e7713caaf3687e44d1e4f7787

---

Sent with 💌 from NxCloud.

[AgentEnder]

Looks good, but we can avoid some extra iterations pretty easily if we use a set from the get go.

[AgentEnder]

Suggested change

	const watchedProjects = [...config.watchProjects];
	const watchedProjects = new Set()

[AgentEnder]

Suggested change

	for (const project of config.watchProjects) {
	watchedProjects.push(
	...findAllProjectNodeDependencies(
	project,
	for (const project of config.watchProjects) {
	watchedProjects.add(project)
	for (const dep of findAllProjectNodeDependencies(...)) {
	watchedProjects.add(dep)
	}
	}

[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.