Cordy/Cairn

Fork 0

API + frontend: capability feature-detect for spaces; controls reflect availability; surface real errors #216

New issue

Closed

opened 2026-08-13 15:17:37 +00:00 by Cordy · 1 comment

Cordy commented

2026-08-13 15:17:37 +00:00

Owner

Parent: #211. Resolves the UI half of #209 and #210.

API: expose what's available so the UI stops showing dead controls — a capabilities signal (extend GET /api/v1/me or a small /api/v1/capabilities) reporting e.g. spaces: { create, manageMembers } (true when app-owned spaces are on and/or the IdP power mode is configured).

Frontend:

Feature-detect (like the audit panel #173): hide/disable "New folder" (create-space) on the Spaces root and the "Members" control when the capability is absent, instead of firing requests that 404.
Surface real errors — the api() raw-Response must be parsed on the error branch so failures show the server message, not empty Create failed: / silent add-member (#136 class). Sweep the space create + member calls (and any sibling that has the same swallow).

Acceptance: on an instance with app-owned spaces enabled, create-space + members controls appear and work; on one with nothing configured they're hidden; every failing call shows a real message.

Parent: #211. Resolves the UI half of #209 and #210. **API:** expose what's available so the UI stops showing dead controls — a capabilities signal (extend `GET /api/v1/me` or a small `/api/v1/capabilities`) reporting e.g. `spaces: { create, manageMembers }` (true when app-owned spaces are on and/or the IdP power mode is configured). **Frontend:** - Feature-detect (like the audit panel #173): hide/disable "New folder" (create-space) on the Spaces root and the "Members" control when the capability is absent, instead of firing requests that 404. - **Surface real errors** — the `api()` raw-`Response` must be parsed on the error branch so failures show the server message, not empty `Create failed:` / silent add-member (#136 class). Sweep the space create + member calls (and any sibling that has the same swallow). **Acceptance:** on an instance with app-owned spaces enabled, create-space + members controls appear and work; on one with nothing configured they're hidden; every failing call shows a real message.

Cordy added this to the v0.7 — Closed beta (dogfood + hardening) milestone

2026-08-13 15:17:37 +00:00

Cordy referenced this issue

2026-08-13 16:13:06 +00:00

feat(spaces): app-owned space membership store (#213) #220

Cordy referenced this issue

2026-08-13 16:32:29 +00:00

feat(spaces): app-owned membership grants scope access (#214) #221

Cordy referenced this issue

2026-08-13 16:42:40 +00:00

feat(spaces): decouple IdP group client; IdP-write as an off-by-default power mode (#215) #222

Cordy referenced this issue from a commit

2026-08-13 16:54:34 +00:00

feat(spaces): app-owned SpacesAPI backend (#216)

Cordy referenced this issue from a commit

2026-08-13 16:55:06 +00:00

test(spaces): app-owned SpacesAPI handlers (#216)

Cordy referenced this issue from a commit

2026-08-13 16:55:34 +00:00

scaffold: wire app-owned SpacesAPI + /me capability (#216)

Cordy referenced this issue from a commit

2026-08-13 16:55:49 +00:00

scaffold: run #216 API wiring

forgejo-actions referenced this issue from a commit

2026-08-13 16:55:57 +00:00

feat(spaces): wire app-owned SpacesAPI + /me capability (#216)

Cordy referenced this issue

2026-08-13 16:56:10 +00:00

feat(spaces): app-owned SpacesAPI + /me capability (#216 API) #223

Cordy referenced this issue from a commit

2026-08-13 16:57:12 +00:00

feat(spaces): app-owned SpacesAPI + /me capability (#216 API) (#223)

Cordy commented

2026-08-13 16:57:12 +00:00

Author

Owner

API half merged (PR #223). SpacesAPI now dispatches to the app-owned spacestore when the IdP power mode is off, so create-space + member management work with no IdP client — apiH.Spaces is non-nil on every instance now, which removes the silent-404 backend cause of #209/#210. GET /me reports spaces: {create, manageMembers} for feature-detect. httptest coverage green.

Remaining for this issue: the frontend — read the capability from /me and hide/disable the create-space ("New folder" on the Spaces root) + Members controls when it's false, and parse the api() response on the error branch so failures show the real message instead of an empty toast. That's the next PR.

**API half merged** (PR #223). `SpacesAPI` now dispatches to the app-owned `spacestore` when the IdP power mode is off, so create-space + member management work with no IdP client — `apiH.Spaces` is non-nil on every instance now, which removes the silent-404 backend cause of #209/#210. `GET /me` reports `spaces: {create, manageMembers}` for feature-detect. httptest coverage green. Remaining for this issue: the **frontend** — read the capability from `/me` and hide/disable the create-space ("New folder" on the Spaces root) + Members controls when it's false, and parse the `api()` response on the error branch so failures show the real message instead of an empty toast. That's the next PR.

Cordy referenced this issue from a commit

2026-08-13 17:01:53 +00:00

scaffold: spaces frontend feature-detect + error surfacing (#216)

Cordy referenced this issue from a commit

2026-08-13 17:02:07 +00:00

scaffold: run #216 frontend

forgejo-actions referenced this issue from a commit

2026-08-13 17:02:17 +00:00

feat(spaces): frontend feature-detect + robust error surfacing (#216)

Cordy referenced this issue

2026-08-13 17:02:30 +00:00

feat(spaces): frontend feature-detect + robust error surfacing (#216 frontend) #224