fix(core): register task timings correctly in task profiling life cycle

[leosvelperez]

Current Behavior

In the TaskProfilingLifeCycle class the task timings are registered with a manually crafted key instead of using the task id. When processing the task results, timings are set with the task id which throws because of the mismatch.

Expected Behavior

In the TaskProfilingLifeCycle class the task timings should be registered with the task id.

Related Issue(s)

Fixes #16792

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 5, 2023 10:03am

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 8cb8d63. 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 -t e2e-macos --parallel=1 --base= --head=
• nx affected --target=e2e --base=575c6a152f4d167343bc30e83730cea9ff311c69 --head=8cb8d6362b7d4e2df22f879944c63287c353a106 --parallel=1
• nx affected --target=test --base=575c6a152f4d167343bc30e83730cea9ff311c69 --head=8cb8d6362b7d4e2df22f879944c63287c353a106 --parallel=1
• nx affected --target=build --base=575c6a152f4d167343bc30e83730cea9ff311c69 --head=8cb8d6362b7d4e2df22f879944c63287c353a106 --parallel=3
• nx run-many -t check-imports check-commit check-lock-files depcheck --parallel=1 --no-dte
• nx affected --target=lint --base=575c6a152f4d167343bc30e83730cea9ff311c69 --head=8cb8d6362b7d4e2df22f879944c63287c353a106 --parallel=3
• nx-cloud record -- nx format:check --base=575c6a152f4d167343bc30e83730cea9ff311c69 --head=8cb8d6362b7d4e2df22f879944c63287c353a106
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

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