Skip to content

feat(conventions+gate): api-client collection paths carry a TRAILING SLASH (build14 endgame wall)#155

Merged
agjs merged 1 commit into
mainfrom
fix/api-client-trailing-slash
Jul 20, 2026
Merged

feat(conventions+gate): api-client collection paths carry a TRAILING SLASH (build14 endgame wall)#155
agjs merged 1 commit into
mainfrom
fix/api-client-trailing-slash

Conversation

@agjs

@agjs agjs commented Jul 20, 2026

Copy link
Copy Markdown
Owner

Stacked on #154. build14 reached best=1 error (all 4 prior walls cleared) then oscillated ~40 turns on PathsWithMethod POST /api/v1/supplier — the generated key is /api/v1/supplier/ (trailing slash: Elysia .group + .post('/')). Verified general (scaffold uses /api/v1/notifications/, schema has /api/v1/{capabilities,notifications,supplier}/). data-fetching guide + the PathsWithMethod per-error steer now both teach: collection root = /api/v1// (list+create), by-id = /api/v1//{id}. Panel PASS; full suite green (2832).

… (build14 endgame wall)

build14 reached best=1 error (closest to green ever, all 4 prior walls cleared) then
oscillated ~40 turns on `PathsWithMethod<paths,"post">` for POST /api/v1/supplier — the
generated key is /api/v1/supplier/ (WITH trailing slash): Elysia mounts the group at
/api/v1/<resource> and the create/list handler at '/', so the collection root key carries
the slash. The model discovered it itself at turn 157 but ran out of runway.

data-fetching guide now teaches: collection root = /api/v1/<resource>/ (list GET + create
POST, TRAILING SLASH); by-id = /api/v1/<resource>/{id} (no trailing slash). Examples +
regression test updated.
Base automatically changed from fix/wire-resource-idempotent-remount to main July 20, 2026 21:02
@agjs
agjs merged commit 49ae234 into main Jul 20, 2026
1 check passed
@agjs
agjs deleted the fix/api-client-trailing-slash branch July 20, 2026 21:02
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.

1 participant