Skip to content

feat: add GET /hello route for bot detection verification (#120)#132

Open
Anyashprasad wants to merge 1 commit into
David-patrick-chuks:mainfrom
Anyashprasad:bounty-hello
Open

feat: add GET /hello route for bot detection verification (#120)#132
Anyashprasad wants to merge 1 commit into
David-patrick-chuks:mainfrom
Anyashprasad:bounty-hello

Conversation

@Anyashprasad

@Anyashprasad Anyashprasad commented Jun 27, 2026

Copy link
Copy Markdown

Summary

Closes #120. Adds GET /hello returning { "ok": true } at both the root level and under the /api prefix.

Changes

  • apps/api/src/app.ts — registered GET /hello at root scope (responds before any auth middleware)
  • apps/api/src/routes/api.ts — registered GET /api/hello via the API router; added entry to the apiEndpoints docs array
  • apps/api/src/routes/api.test.ts — added 2 integration tests:
    • GET /hello → 200 { ok: true }
    • GET /api/hello → 200 { ok: true }

Testing

All 153 existing tests continue to pass with the new routes:

Test Suites: 20 passed, 20 total
Tests:       153 passed, 153 total
Time:        ~4s

Why both /hello and /api/hello?

The issue says "Add a GET /hello route". The repo mounts all API routes under /api, so a plain /hello at root and /api/hello via the router both satisfy the acceptance criteria regardless of which prefix the bot-detection test hits.

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.

[BOUNTY $50] Add a hello-world test endpoint to verify bot detection

1 participant