Uh oh!
There was an error while loading. Please reload this page.
Add Cloudflare Workers Vitest test setup - #44
Merged
Conversation
The project is deployed as a Cloudflare Worker, so the test harness now uses Cloudflare's Vitest pool instead of Bun's built-in runner. Existing tests were migrated to Vitest, utility crypto/CORS coverage was added for issue #2, and a small Worker runtime smoke test proves wrangler vars and request handling load through the Cloudflare test environment. Constraint: Cloudflare Workers runtime APIs and wrangler bindings should be exercised through @cloudflare/vitest-pool-workers Rejected: Keep Bun's built-in test runner as primary | it does not exercise Workers runtime bindings and had incompatible module mock behavior locally Rejected: Maintain parallel Bun and Vitest suites | unnecessary duplicate maintenance for this small Worker Confidence: high Scope-risk: moderate Directive: Keep docs/superpowers artifacts out of product commits unless the user explicitly asks to version workflow docs Tested: bun run test (10 files, 106 tests); git diff --check; no bun:test/mock.module/vi.module leftovers Not-tested: GitHub Actions execution on remote runner
Deploying with |
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs | github | 3ae4b15 | Commit Preview URL Branch Preview URL | May 02 2026, 12:57 AM |
sounmind
marked this pull request as ready for review
May 2, 2026 00:58
soobing
self-requested a review
May 2, 2026 01:32
SamTheKorean
approved these changes
May 2, 2026
SamTheKorean
left a comment
There was a problem hiding this comment.
이제는 태스트를 레퍼런스삼아서 더 원할하게 기여할 수 있겠네요! 고생하셨습니다!
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
package.json,bun.lock,vitest.config.js를 추가했습니다.bun:test에서vitest로 마이그레이션했습니다.utils/github.js,utils/cors.js단위 테스트를 추가했습니다.cloudflare:test/cloudflare:workers기반 Worker runtime smoke test를 추가했습니다.bun install후bun run test실행으로 변경했습니다.Closes#2
Verification
Notes
utils/github.js의 PKCS1 private key 처리도 실제 지원되도록 보강하고 회귀 테스트를 추가했습니다..claude/,.omc/,docs/)은 커밋에 포함하지 않았습니다.