[vikram-ecobee]

HS-4407: Migrate CI to GitHub Actions

Jira Ticket: HS-4407

Summary

• replace the CircleCI pipeline with GitHub Actions
• run cached npm installs, builds, unit tests, coverage uploads, and Codecov reporting
• publish to npm from successful pushes to master using repository secrets
• update the README CI badge

Validation

• npm ci
• npm run build
• npx run test:ci

---

Thanks ❤️

[gemini-code-assist]

Code Review

This pull request removes the CircleCI configuration, updates the CI badge in the README to point to GitHub Actions, adds a new test:ci script in package.json, and refactors magic numbers into named constants in src/helpers/constants.ts. However, the GitHub Actions workflow file (.github/workflows/ci.yml) referenced in the README is missing from the pull request and needs to be added.

[Copilot]

🟡 Changes recommended

Publishing has an authentication-variable mismatch and allows releases from main despite the master-only requirement.

2 open findings

• Export NPM_TOKEN for npm authentication · New
• Restrict npm publishing to the master branch · New

What changed in this PR

Migrates the library’s CI and npm publishing from CircleCI to GitHub Actions.

Changes:

• Adds cached installs, builds, unit tests, coverage reporting, and publishing.
• Replaces the CI badge and adds a dedicated CI test script.
• Names existing retry codes without changing their values.

File	Description
src/​helpers/​constants.ts	Replaces numeric retry codes with named constants.
README.md	Switches the CI badge to GitHub Actions.
package.json	Adds the test:ci script.
.github/​workflows/​ci.yml	Adds build, test, reporting, and publishing jobs.
.circleci/​config.yml	Removes the previous pipeline.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (6cadfa3) to head (4e17bcb).

Additional details and impacted files

@@            Coverage Diff            @@
##            master       #50   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         4           
  Lines           73        82    +9     
  Branches         9         9           
=========================================
+ Hits            73        82    +9     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

🔵 Needs a closer look

Node 20 dependency installation and authenticated npm publishing still need end-to-end runner validation.

0 open findings

2 resolved since last review

• Export NPM_TOKEN for npm authentication
• Restrict npm publishing to the master branch

🧠 Review effort: Balanced

[ddamico-ecobee]

LGTM, if the short-lived token is the approach we are going with for now!