Skip to content

Merge pull request #1 from TevaLabs/main#366

Merged
josephchimebuka merged 1 commit into
TevaLabs:mainfrom
tecmalik:fix-/320-consolidate-round-listing
Jun 30, 2026
Merged

Merge pull request #1 from TevaLabs/main#366
josephchimebuka merged 1 commit into
TevaLabs:mainfrom
tecmalik:fix-/320-consolidate-round-listing

Conversation

@tecmalik

Copy link
Copy Markdown
Contributor

done

closes #320

Summary

Closes #

Dual-entrypoint architecture

This repo ships two Express applications. npm run dev (src/index.ts) is the
default production path and is the one reviewers and CI care about. src/app.ts
(npm run dev:hackathon) is a separate mock/demo entrypoint.

Several past issues were closed while the functionality existed only in one
entrypoint. Before requesting review, confirm your change behaves correctly on the
default npm run dev path. See docs/architecture.md for
the entrypoint map and the checklist for adding new routes.

Affected endpoints

Checklist

  • Verified the change works on the default npm run dev (src/index.ts) entrypoint.
  • If the change affects shared functionality, verified it on the hackathon src/app.ts entrypoint too (or explained why it does not apply).
  • Listed all affected endpoints above (or marked "None").
  • Added or updated tests covering the change.
  • npm run lint and npm test pass locally.
  • Updated docs (README, OpenAPI annotations, docs/) where behavior changed.

Notes for reviewers

@josephchimebuka josephchimebuka merged commit d67361f into TevaLabs:main Jun 30, 2026
1 of 6 checks passed
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.

Consolidate hackathon rounds.ts and production rounds.routes.ts behind shared service

2 participants