[WIP] Remove needs: build from CI jobs

[Copilot]

Plan: Remove "needs: build" from ci.yml jobs

Analysis

The .github/workflows/ci.yml file currently has two jobs with unnecessary needs: build dependencies:

• Line 11: test job has needs: build
• Line 70: js job has needs: build

These dependencies force the jobs to wait for the build job to complete, but analysis shows:

• No artifacts are shared between jobs (only test uploads a coverage report)
• No job outputs are used by dependent jobs
• Each job independently checks out code and sets up its environment
• The dependencies create unnecessary sequential execution instead of parallel execution

Changes Required

• Remove needs: build from the test job (line 11)
• Remove needs: build from the js job (line 70)
• Verify the CI workflow still passes with parallel job execution
• Run local tests to ensure no regressions

Expected Impact

• Jobs will run in parallel instead of sequentially
• Faster CI execution time
• No functional changes to what each job does

Changes Made

✅ Removed needs: build from the test job
✅ Removed needs: build from the js job
✅ Validated YAML syntax with actionlint - passed
✅ Ran unit tests - all passed
✅ Verified build still works correctly

Original prompt

Remove "needs: build" in jobs of ci.yml

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Copilot]

Pull Request Overview

This PR optimizes the CI workflow by removing unnecessary job dependencies, allowing the test and js jobs to run in parallel with the build job instead of waiting for it to complete.

Key changes:

• Removed needs: build dependency from the test job, enabling parallel execution
• Removed needs: build dependency from the js job, enabling parallel execution

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.