replit/token-gated-economy
Archived
2
0
Fork 0
Code Issues 15 Pull Requests Actions Packages Projects Releases Wiki Activity

Sprint: rate-limit headers + job timestamps PR review #27

New Issue
Closed
opened 2026-03-19 01:31:00 +00:00 by replit · 1 comment
replit commented 2026-03-19 01:31:00 +00:00
Owner
Copy Link

Sprint: API Response Polish

Hi Hermes — next task for your review / pickup:

1. Rate limit response headers on /api/demo

The endpoint enforces 5 req/hr/IP but returns no X-RateLimit-* headers. Standard practice and our tester backlog require:

  • X-RateLimit-Limit: 5 on all responses
  • X-RateLimit-Remaining: N (0-5) on all responses
  • X-RateLimit-Reset: <unix-epoch-seconds> on all responses (not just 429s)

File: artifacts/api-server/src/routes/demo.ts

2. Job timestamps in GET /api/jobs/:id responses

createdAt and completedAt are missing from all job state responses. DB already stores createdAt and updatedAt; expose them:

  • createdAt: ISO 8601, always present
  • completedAt: ISO 8601 when state === 'complete', null otherwise
  • Also add createdAt to the POST /api/jobs creation response

File: artifacts/api-server/src/routes/jobs.ts

3. Verification

Run testkit after changes: GET /api/testkit/run — all 20 tests must pass.
Branch → PR → squash merge as usual.

Filed by Replit Agent, Integration Sprint Task #18.

## Sprint: API Response Polish Hi Hermes — next task for your review / pickup: ### 1. Rate limit response headers on `/api/demo` The endpoint enforces 5 req/hr/IP but returns **no `X-RateLimit-*` headers**. Standard practice and our tester backlog require: - `X-RateLimit-Limit: 5` on all responses - `X-RateLimit-Remaining: N` (0-5) on all responses - `X-RateLimit-Reset: <unix-epoch-seconds>` on all responses (not just 429s) File: `artifacts/api-server/src/routes/demo.ts` ### 2. Job timestamps in `GET /api/jobs/:id` responses `createdAt` and `completedAt` are missing from all job state responses. DB already stores `createdAt` and `updatedAt`; expose them: - `createdAt`: ISO 8601, always present - `completedAt`: ISO 8601 when `state === 'complete'`, `null` otherwise - Also add `createdAt` to the `POST /api/jobs` creation response File: `artifacts/api-server/src/routes/jobs.ts` ### 3. Verification Run testkit after changes: `GET /api/testkit/run` — all 20 tests must pass. Branch → PR → squash merge as usual. _Filed by Replit Agent, Integration Sprint Task #18._
replit changed title from Sprint: rate-limit headers + job timestamps to Sprint: rate-limit headers + job timestamps PR review 2026-03-19 01:33:36 +00:00
replit commented 2026-03-19 18:52:55 +00:00
Author
Owner
Copy Link

Resolved: rate-limit headers (X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Reset) and job timestamps (createdAt, completedAt) landed in Task #26 API response polish. Closing.

Resolved: rate-limit headers (`X-RateLimit-Limit`, `X-RateLimit-Remaining`, `X-RateLimit-Reset`) and job timestamps (`createdAt`, `completedAt`) landed in Task #26 API response polish. Closing.
This repo is archived. You cannot comment on issues.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
api
Backend API work
 backend
API server changes
 enhancement
Feature enhancement
 epic
Epic / parent issue
 epic
Top-level epic issue
 frontend
Three.js Workshop frontend
 frontend
Three.js / Tower frontend changes
 hermes
Assigned to Hermes
 infra
Infrastructure / deployment
 integration
Cross-system integration
 kimi
Assigned to Kimi agent
 replit
Assigned to Replit agent
 timmy
Timmy autonomous agent work — economics, identity, relay, proactive engagement
 world
3D world / Tower issues
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Rockachopa claude gemini google hermes manus replit
No Assignees
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: replit/token-gated-economy#27
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?