fix(angular): return error when name is missing from package.json

[Pascalmh]

Current Behavior

Situation: You have a angular-cli app with a package.json that has no "name"-property

ng add @nrwl/angular
Skipping installation: Package already installed
Cannot read properties of undefined (reading 'startsWith')

Traced the error back to packages/angular/src/generators/ng-add/utilities/normalize-options.ts:38

Expected Behavior

Migration "just works" - somekind of fallback is used, or I am presented with an Error message so that I can fix the error.

[nx-cloud]

☁️ Nx Cloud Report

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

• nx affected --target=e2e --base=f25e94a5dc8e27e3c1f2ec5bcd614d797f8d6008 --head=11cf09fb0c4fe33ef40dad8b49fa8fbdc572cd44 --exclude=e2e-storybook,e2e-react-native,e2e-detox --parallel=1
• nx affected --target=test --base=f25e94a5dc8e27e3c1f2ec5bcd614d797f8d6008 --head=11cf09fb0c4fe33ef40dad8b49fa8fbdc572cd44 --parallel=1
• nx affected --target=build --base=f25e94a5dc8e27e3c1f2ec5bcd614d797f8d6008 --head=11cf09fb0c4fe33ef40dad8b49fa8fbdc572cd44 --parallel=3
• nx affected --target=lint --base=f25e94a5dc8e27e3c1f2ec5bcd614d797f8d6008 --head=11cf09fb0c4fe33ef40dad8b49fa8fbdc572cd44 --parallel=3
• nx-cloud record -- yarn nx workspace-lint
• nx-cloud record -- yarn documentation
• nx build typedoc-theme
• nx-cloud record -- yarn depcheck
• nx-cloud record -- yarn check-commit
• nx-cloud record -- yarn nx format:check --base=f25e94a5dc8e27e3c1f2ec5bcd614d797f8d6008 --head=11cf09fb0c4fe33ef40dad8b49fa8fbdc572cd44
• nx-cloud record -- yarn check-imports

---

Sent with 💌 from NxCloud.

[vercel]

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

1 Ignored Deployment

Name	Status	Preview	Updated
nx-dev	⬜️ Ignored (Inspect)		May 18, 2022 at 9:13AM (UTC)

[leosvelperez]

Thanks for the contribution!

I left a comment to make a change to the solution provided. The npmScope is not required so we shouldn't error and block the migration when we can't find the name in the package.json.

[leosvelperez]

LGTM! 🎉

I forced pushed the branch to kick-off CI.

Thanks again for contributing!

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