Skip to content

Commit ae5f9fe

Browse files
committed
chore(changeset): patch @objectstack/client — the docblock ships in dist/*.d.ts
Measured rather than assumed: `@objectstack/client`'s `files[]` is ["dist","README.md","CHANGELOG.md"], and after `pnpm --filter @objectstack/client build` the new docblock prose is present in both dist/index.d.ts and dist/index.d.mts (positive control: the pre-existing "the vendor's RECORD vocabulary" prose, same files, same grep; negative control: an absent marker, 0 hits). Published bytes move, so `patch`, not `skip-changeset`. Claude-Session: https://claude.ai/code/session_01TSf4DV7ziu4V5j73e46b7c Co-authored-by: Claude <noreply@anthropic.com>
1 parent 72c307a commit ae5f9fe

1 file changed

Lines changed: 28 additions & 0 deletions

File tree

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
---
2+
'@objectstack/client': patch
3+
---
4+
5+
`oauth.applications.register`'s docblock says where a plain `name` IS honoured, and that it is not this route
6+
7+
A caller who wants to name an OAuth client reaches for `name`. On the route this
8+
method posts — the provider's `/oauth2/create-client` — that member is not in
9+
the body schema and is stripped: driven on a real socket, the call answered
10+
**201** and the value was absent from the response, from `applications.get`,
11+
from `applications.list`, and `null` in the `sys_oauth_application` row's `name`
12+
column. Nothing in the answer says so.
13+
14+
The spelling is not wrong everywhere, which is what made it worth writing down:
15+
`POST /api/v1/auth/sys-oauth-application/register` — the session-required
16+
ObjectStack mount behind the Console's *Setup → OAuth Applications* form —
17+
answered **200** to the same body, mapped `name` onto `client_name`, and set
18+
that column. That mount is `disposition: 'server-only'` in the auth route ledger
19+
and objectstack#17210 ruled it stays that way, so no SDK method builds its URL.
20+
21+
The docblock now states both halves where the caller reads them: post
22+
`client_name` to name a client from here, and `redirect_uris` must arrive
23+
pre-split — the newline-separated-textarea split is the Console wrapper's, not
24+
this route's.
25+
26+
Docblock only. No method is added, no request or response type changes, and the
27+
ledger row is untouched — but the text ships inside `dist/*.d.ts` as editor
28+
hover, so it is a `patch` rather than a no-publish change.

0 commit comments

Comments
 (0)