`GET /v1/stats` and the `simlock stats --csv` ledger
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 65/100
- Issue type
- Feature
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- typescript
- Domain
- api, backend, cli, documentation
Research direction
Start with the shared window rules and duration parser, then read src/core/usage/, src/contract/operations.ts, and the relevant handler in src/http/app.ts. Implement the ledger and CSV behavior in src/cli/index.ts, following the listed test cases; update docs/HTTP-API.md and docs/CLI.md. Done means the route matches CLI figures, CSV has one row per request, and the authorization and history-window cases behave as specified.
Written by the indexing model from the issue text.
Description
Part of #329.
Scope
After this PR, GET /v1/stats answers the same figures as simlock stats --json for the same window, and simlock stats --csv writes one row per lease request in the window, a file a spreadsheet opens.
Technical spec
Modules touched
src/http/app.ts—GET /v1/stats?since=<duration>or?from=<ISO>&to=<ISO>, with&ledger=1to include the ledger;todefaults to now; both forms together, or neither, follow the CLI's rules (neither means the last 24 hours). The handler parses the query with the shared duration parser, dispatchesusage.get, and returns its output as JSON.HISTORY_NOT_KEPTtakes its status from the contract's error table, as every error does; a bad window is 400.src/contract/operations.ts—usage.getinput gainsincludeLedger: boolean, defaultfalse; the output carriesrequests[]only when it istrue, so a console or a plainsimlock statscall never receives the ledger.src/core/usage/—computeUsageoutput gainsrequests[]when asked, the ledger:{ requestId, requester, label?, platform, model, workerId?, source?, requestedAt, grantedAt?, endedAt?, waitMs?, heldMs?, turnaroundMs?, outcome: "granted" | "rejected" | "open", reason?, endReason? }, one entry perlease.requestedin the window.src/cli/index.ts—--csvonsimlock statsasks for the ledger and writes it as RFC 4180 CSV to stdout, one header row, a column per ledger field in that order, timestamps as ISO 8601, empty cell for an absent value.--csvwith--jsonis a usage error.docs/HTTP-API.md(route, query, errors, gateway behaviour),docs/CLI.md(--csv),docs/CLIENT.mdif the ledger shows in the client type.
Contract and event changes
usage.getinput gainsincludeLedger(defaultfalse); output gainsrequests[]when it is set. Additive.- New route
GET /v1/stats. Operator token; an agent token gets 403 as every operator route does.
Rules in play
safety.mdrule 10:since,from,toare wire input, parsed and bounded before dispatch.architecture.mdrule 10: the window rules are the CLI's and the route's through one shared function, not two copies.
Tests
computeUsagelists one ledger entry per request with the timestamps and durations of its grant and end, andopenfor a request still waiting at the window's end- the CSV writer quotes a field containing a comma, a quote or a newline, and writes an empty cell for an absent value
simlock stats --csvprints a header row and one row per request, and rejects--csv --jsonGET /v1/stats?since=1hreturns the same body asusage.getfor that window, withoutrequests[];&ledger=1adds itGET /v1/stats?from=&to=withfromlater thantoreturns 400 naming the parameterGET /v1/statswith no window returns the last 24 hoursGET /v1/statswith a window that ends before the oldest held event returnsHISTORY_NOT_KEPTwith the status the error table gives itGET /v1/statswith an agent token returns 403- (e2e) after a scripted run,
curl GET /v1/stats?since=1handsimlock stats --since 1h --jsonprint equal figures - (e2e)
simlock stats --csv --since 1hhas as many data rows aslease.requestedevents in the window
Done when
- After a scripted run against the fake driver,
GET /v1/stats?since=1hwith an operator token returns the same figures assimlock stats --since 1h --json. simlock stats --csv --since 1h > run.csvwrites a file with one header row and one row per lease request, which a spreadsheet opens with the columns separated.- An agent token gets 403 on the route; a window older than the kept history gets 422 with the oldest kept time in the body.
docs/HTTP-API.mddescribes the route under Operator routes, anddocs/CLI.mddescribes--csv.
Out of scope
- The console view (next task).
- A Prometheus or OpenMetrics endpoint.
Depends on
- #347
Approval
- Approved for delivery
Written by an agent.
- Dominant language
- TypeScript
- Stars
- 15
- Forks
- 1
- Avg merge
- 8h 38m
- Merged PRs (30d)
- 135
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from callstackincubator/simlock
-
A request queued behind a failed boot whose destroy also fails is never wokenPossibly taken A pull request linked to this issue is open or already merged. Openbug:triage
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
callstackincubator/simlock#400 · 1 comment ·
Maintainers usually reply within 1 day
-
bug:new
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
callstackincubator/simlock#350 · 1 comment ·
Maintainers usually reply within 1 day
-
feature:spec
Difficulty 4/5 3-5 days Newbie friendliness 35/100
callstackincubator/simlock#412 ·
Maintainers usually reply within 1 day
-
Let requesters choose the lease IDPossibly taken A pull request linked to this issue is open or already merged. Openfeature:spec
Difficulty 5/5 Over a week Newbie friendliness 45/100
callstackincubator/simlock#410 · 2 comments ·
Maintainers usually reply within 1 day
-
bug:new flaky-test
Difficulty 3/5 1-2 days Newbie friendliness 56/100
callstackincubator/simlock#407 ·
Maintainers usually reply within 1 day
All issues in callstackincubator/simlock
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Maintainers usually reply within 1 day
-
area:ui enhancement issue-form:feature platform:cross-platform review: high
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
1lck/Lithe-IDEA#1092 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
developmentseed/deck.gl-raster#693 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
Marker-Inc-Korea/AutoRAG#1801 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day