Skip to content

ci: add test + coverage workflow on main and PR - #15

Merged
elecnix merged 1 commit into
mainfrom
ci/test-coverage
Jun 23, 2026
Merged

ci: add test + coverage workflow on main and PR#15
elecnix merged 1 commit into
mainfrom
ci/test-coverage

Conversation

@elecnix

@elecnix elecnix commented Jun 23, 2026

Copy link
Copy Markdown
Owner

What

Adds a GitHub Actions workflow (.github/workflows/test.yml) that runs on every push to main and every PR targeting main.

How

  • Node.js 22 built-in test runner: node --experimental-strip-types --test
  • Coverage via --experimental-test-coverage flag (also built-in, no external deps)
  • Uses actions/checkout@v4 and actions/setup-node@v4

Also removes the redundant parse/syntax smoke test from test/agent-session.test.ts — the CI workflow now catches syntax errors without the subprocess-spawn hack.

Coverage output (example)

# file                   | line % | branch % | funcs % | uncovered lines
# agent-identity         |        |          |         |
#  daemon-client.ts      |  93.33 |    66.67 |   75.00 | 31-32 65-67
# test                   |        |          |         |
#  agent-session.test.ts |  98.13 |    73.91 |  100.00 | 73-74
# all files              |  96.15 |    70.73 |   90.91 |

The syntax-validity smoke test (spawning node --experimental-strip-types
on index.ts) is now redundant — the CI workflow added in this PR runs the
same check as part of its test step. Syntax errors in index.ts will cause
the CI job to fail.
@elecnix
elecnix force-pushed the ci/test-coverage branch from e1f458e to e10fef5 Compare June 23, 2026 03:07
@elecnix
elecnix marked this pull request as ready for review June 23, 2026 03:09
@elecnix
elecnix merged commit b53a69f into main Jun 23, 2026
@elecnix
elecnix deleted the ci/test-coverage branch June 23, 2026 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant