dkritarth / dkritarth/FreeFlow
No automated tests or CI
- Dominant language
- JavaScript
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
Repo has zero test files and no test script in either \`package.json\` (root) or \`cli/package.json\`. No \`.github/workflows\` — nothing runs on push/PR (already flagged in branch protection: no required status checks because none exist).
For a security-sensitive Worker (auth, PIN hashing, rate limiting, size limits) this is risky — the last hardening PR (#3) was reviewed by hand because there's no automated way to catch a regression like the CSP breaking the shipped template.
**Ask:**
- Add a test runner (Vitest fits Cloudflare Workers well, has official \`@cloudflare/vitest-pool-workers\` support) and cover: bearer auth, PIN hash verify, rate limiting math, draft ID validation regex, upload size rejection.
- Add a \`.github/workflows/ci.yml\` that runs on PR: \`npm install\`, tests, and at minimum \`node --check src/index.js cli/index.js\`.
- Once CI exists, wire it into branch protection as a required status check.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with the root package.json, cli/package.json, src/index.js, and cli/index.js to understand the existing scripts and entry points. Add the requested Vitest coverage for authentication, PIN verification, rate limiting, draft ID validation, and upload limits, then add .github/workflows/ci.yml to run installation, tests, and node --check. Done means the tests and checks pass in CI and the workflow is available as a required branch-protection status check.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, nodejs
- Domain
- ci-cd, cloud, testing
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100