Skip to content

ci: add CI workflow for typecheck, build, and test#1

Merged
kud merged 2 commits into
mainfrom
ci/add-test-workflow
Jul 5, 2026
Merged

ci: add CI workflow for typecheck, build, and test#1
kud merged 2 commits into
mainfrom
ci/add-test-workflow

Conversation

@kud

@kud kud commented Jul 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add .github/workflows/ci.yml running the project's real quality gates (typecheck, build, test) on every push to main and on pull requests.
  • No lint script exists in package.json, so that gate is intentionally omitted — nothing invented.
  • Install uses npm ci with cache: npm since package-lock.json is committed (verified via git ls-files).
  • Matches the existing release.yml conventions: actions/checkout@v5, actions/setup-node@v5, Node 24, ubuntu-latest.

Test plan

  • npm ci — clean install from the committed lockfile
  • npm run typecheck — passes clean
  • npm run build — passes clean (tsup, ESM output)
  • npm run test — 9/9 tests pass; both test files (src/lib.test.ts, src/tui/lib/client.test.ts) are fully mocked, so the suite runs credential-free with no Qobuz auth/Keychain/network dependency

kud added 2 commits July 5, 2026 02:47
The typecheck step requires @kud/macos-media-keys types, which are macOS-only and fail to install on Linux runners. Also adds a disclaimer to the README clarifying that this is an unofficial project unaffiliated with Qobuz.
@kud
kud marked this pull request as ready for review July 5, 2026 02:21
@kud
kud merged commit e540f26 into main Jul 5, 2026
1 check passed
@kud
kud deleted the ci/add-test-workflow branch July 5, 2026 02:21
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