fix(core): change name schema order in project.json

[Roozenboom]

Current Behavior

The addProjectToWorkspaceJson generator utility is creating a project.json with first the schema property and afterwards the name and rest of the projectConfiguration. This works ok, but has unexpected behavior when you use another generator (for example @nrwl/angular:application) that does a formatting on the project configuration files. The 'wrongly' sorted project.json files will show as updated in the changed files list. Which is something we like to prevent in a large monorepo with multiple codeowners.

Expected Behavior

Expected behavior is that every schematic/generator and utility is generating the project.json in the same way so that generated files are formatted correctly and not show up as unrelated changes when you create a new app or lib.

Steps to Reproduce

I created a small reproduction path on nx-examples: nrwl/nx-examples#250
If you checkout this code and run nx g @nrwl/angular:app test-app --dry-run you will see that the output shows:
UPDATE libs/cart/cart-page/project.json
which isn't expected when you generate a new app called test-app.

Solution

The solution to this issue is as simple as changing the order of the properties in the project configuration in the addProjectToWorkspaceJson utility

[nx-cloud-staging]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit d5dd4ce. 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=2d860eaf882f7d6389f387f06b59e34d66361b72 --head=d5dd4ce9223ddfc6b2f0a8aabed648d3d39c50f8 --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox --parallel=1
• nx affected --target=test --base=2d860eaf882f7d6389f387f06b59e34d66361b72 --head=d5dd4ce9223ddfc6b2f0a8aabed648d3d39c50f8 --parallel=1
• nx affected --target=build --base=2d860eaf882f7d6389f387f06b59e34d66361b72 --head=d5dd4ce9223ddfc6b2f0a8aabed648d3d39c50f8 --parallel=3
• nx depcheck --no-dte
• nx affected --target=lint --base=2d860eaf882f7d6389f387f06b59e34d66361b72 --head=d5dd4ce9223ddfc6b2f0a8aabed648d3d39c50f8 --parallel=3
• nx lint --no-dte
• nx check-lock-files --no-dte
• nx check-commit --no-dte
• nx-cloud record -- nx format:check --base=2d860eaf882f7d6389f387f06b59e34d66361b72 --head=d5dd4ce9223ddfc6b2f0a8aabed648d3d39c50f8
• nx check-imports --no-dte
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

[vercel]

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

Name	Status	Preview	Updated
nx-dev	✅ Ready (Inspect)	Visit Preview	Nov 25, 2022 at 2:43PM (UTC)

[FrozenPandaz]

Thank you for your contribution! LGTM 🎉

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