Files
plainpages/todo.md
T
lilleman 76995008eb
CI / full-gate (push) Successful in 3m17s
Mirror / github-mirror (push) Successful in 4s
Remove the shipped item 20 from the plan
2026-10-31 09:15:12 +01:00

12 KiB

todo

Scoring

Score = 100 * V / (2**S + 8)

Bar = 1

Next ID = 40

Items

ID Release Blocked by Exempt Item L E I R S Moves Score
4 0.4.21 Refuse a permission save whose baseline no longer matches Keto, so concurrent edits are not silently lost. 2 2 2 4 4 2 by 4 2.8
7 0.4.21 Confirm a successful permission save, and report a partial failure instead of hiding it. 2 2 2 4 4 2 by 4 2.8
39 0.4.21 Pass an absolute return_to on to Kratos even when the URL also carries a chosen locale. 1 2 2 3 2 2 by 2 2.8
22 0.4.21 Offer a users:read holder a View row action on the users list, not Edit. 2 2 1 3 3 2 by 2 2.1
34 0.4.21 Serve every stylesheet and script under a content-versioned URL with a year-long immutable Cache-Control, so a page view downloads none of them again. 4 2 3 7 6 8 by 6, 2 by 2 1.8
3 0.4.21 Guard the group paths against self-lockout, or accept the gap explicitly. 3 2 2 5 6 2 by 6 1.4
1 0.4.21 owner Publish every release to Docker Hub, and give the Hub page its short description. 3 3 2 6 5 1 by 3 1.3
32 0.4.21 Fail the gate on a stylesheet feature the browser floor lacks, unless it is guarded. 4 2 2 6 7 code health by 2 1.1
13 0.4.21 Accept a verification code pasted with surrounding whitespace. 4 2 3 7 7 2 by 8 1.0
19 0.5.0 question Run the documented Docker commands on macOS and fix whatever misbehaves. 2 2 1 3 3 1 by 3 3.3
16 0.5.0 question Decide whether the single Keto Resource namespace splits into per-domain namespaces. 3 2 3 6 6 7 by 2 0.5
6 0.5.0 Warn at discovery when a route or nav node gates on a permission no plugin declares. 2 1 1 2 2 7 by 3, 9 by 3 8.0
23 0.5.0 Give list pages one view-model helper from query to pagination, instead of ~40 hand-written lines each. 3 1 2 4 5 1 by 4, 9 by 2, code health by 3 7.9
28 0.5.0 Let a plugin add entries to the profile menu through the plugin API. 2 2 2 4 4 7 by 4, 1 by 2 4.2
2 0.5.0 Rename the admin example plugin to a name that says what it administers. 3 1 2 4 6 9 by 3, code health by 2 2.6
5 0.5.0 List grants that no installed plugin declares, with a remove action. 2 2 2 4 4 2 by 3 2.1
14 0.5.0 Drop a double-clicked duplicate submit, with no client-side JavaScript. 4 2 3 7 7 2 by 7, 1 by 4 1.4

Details

4. Refuse a permission save whose baseline no longer matches Keto, so concurrent edits are not silently lost.

Sketch: post the rendered set as a hidden baseline; when it no longer matches Keto, re-render with "this changed while you had the page open" rather than applying.

7. Confirm a successful permission save, and report a partial failure instead of hiding it.

applyGrants loops writes then deletes with no transaction, so a Keto error midway leaves a half-applied set behind the generic error page. The alert alert-pos pattern of the recovery-code banner is available.

39. Pass an absolute return_to on to Kratos even when the URL also carries a chosen locale.

flowReturnTo (src/auth/routes.ts) returns the absolute target as-is only when no locale is chosen. With /login?return_to=<absolute OAuth2 login URL>&locale=sv-SE and no ?flow=, it builds <origin>/auth/complete?locale=sv-SE instead and drops the absolute target, so after sign-in the visitor lands on the dashboard and the OAuth2 login challenge never continues. Normal paths don't hit it, because Hydra's redirect carries no locale and the language picker acts on a page that already holds a ?flow=; a hand-built URL does. Fix by keeping the absolute target when localPath rejects it, losing the language across that round-trip instead, and add the case to the login return_to test in src/http/app.test.ts.

22. Offer a users:read holder a View row action on the users list, not Edit.

The pencil opens the read-only detail page, so it is harmless, but the label contradicts what the reader can do. Needs canWrite threaded into listTable, a common.view core catalog key and an i-eye icon.

34. Serve every stylesheet and script under a content-versioned URL with a year-long immutable Cache-Control, so a page view downloads none of them again.

serveStatic (src/http/static.ts) sends only content-length and content-type, so every page view downloads styles.css, auth.css and each plugin's stylesheet again: about 50 kB on a plugin's issue page, seconds per page over GPRS. Firefox can paint the page unstyled meanwhile, when an extension forces layout early.

  • A versioned URL carries a hash of the file's content, as ?v=<hash> or in the path, and answers Cache-Control: public, max-age=31536000, immutable. A release changes the hash, so it still reaches every browser at once.
  • An unversioned /public/ URL answers a short max-age with revalidation: ETag or Last-Modified, and 304 on If-None-Match or If-Modified-Since, for GET and HEAD.
  • Every /public/ href the host renders comes out versioned: the shell's <link>s and favicon, the error pages (views/403.ejs, 404, 500, 503, error), views/oauth-consent.ejs, and the styles hrefs a plugin passes to views/partials/shell.ejs.
  • Recommended: the host versions a plugin's hrefs itself. A plugin names its asset once, as today, and the shell adds the version to any /public/ href. A helper would make every plugin author remember to call it, and adds to the barrel.
  • Contract: views/partials/*.ejs are contract surface (AGENTS.md → Rules). Host-side versioning keeps the styles option and its meaning, and an unversioned href keeps working, so no minor bump of HOST_API_VERSION is needed and 0.4.21 holds. A helper exported through @plainpages/plugin-api would add to the contract and need 0.5.0.
  • Pages stay private, no-store (src/http/app.ts); the static branch returns before that header is set.

Tests first, with expected values from this item: src/http unit tests for both header sets and the 304s, and an E2E test that a repeat page view fetches no stylesheet again. State the choice once in a comment in static.ts, and in a sentence where the README tells plugin authors and operators about assets. Requested by plainpages-plugin-fastjira, whose own item waits on the release that ships this.

3. Guard the group paths against self-lockout, or accept the gap explicitly.

The self-revoke guard covers only your own direct grants on the Users screen. Unticking a permission on a group you belong to, leaving that group, or deleting it can still strip your own access with no warning. AGENTS.md records this as a known gap. The robust fix is a "last effective holder" check, which needs a reverse Keto query.

1. Publish every release to Docker Hub, and give the Hub page its short description.

Hub's newest tag is 0.4.0, while git has v0.4.1 onwards, and the overview still says 0.4.0. No release.yml run exists for the tags auto-release pushes as renovate-bot; find out why Gitea doesn't fire push: tags for them, fix it, and backfill the missing tags. fejkdata did this (../fejkdata, commits 51d924a and 34c8234), and two parts port directly: a one-line description riding the existing PATCH as its description field, and a unit test that the description opens a README sentence and every command the overview shows appears in the README. fejkdata also syncs the page on any push to main, with its own token in an environment only main may use; decide whether that fits here, where {{VERSION}} must render a released tag.

32. Fail the gate on a stylesheet feature the browser floor lacks, unless it is guarded.

Nothing checks README → "Two browser floors", and no E2E engine is old enough to show a break. Lint every stylesheet, the examples' included, against Chromium 87, Safari 15 and Firefox 115, and allow a feature only inside an @supports that guards it.

13. Accept a verification code pasted with surrounding whitespace.

A pasted code fails the browser's pattern check. No zero-JS fix exists: pattern cannot transform, so loosening it only lets the untrimmed value through to Kratos. Check first: kratos-admin.ts returns a recovery_link alongside the code; if the stock courier mail can carry one, a template change fixes this with no host code. Otherwise proxy the flow POST through the host: submitFlow (src/auth/kratos-public.ts) already relays cookies and normalises a 422 redirect_browser_to, and has no callers outside tests. CSRF improves, since the POST becomes same-origin and form-action 'self' can join the CSP. Risks: one flow.ui.action serves all five flows, so this rewrites the sign-in path; and Go's nosurf checks Referer only over https, which no test covers. Code entry has no E2E coverage today.

19. Run the documented Docker commands on macOS and fix whatever misbehaves.

Needs a Mac. Two suspects, both from --user "$(id -u):$(id -g)": a macOS id -g is 20, which is dialout inside the noble image, and Docker Desktop remaps bind-mount ownership in its VM.

16. Decide whether the single Keto Resource namespace splits into per-domain namespaces.

Ory's own examples model Shift, Document and so on. One global Resource bucket is the project's "no catch-all names" rule applied to namespaces.

6. Warn at discovery when a route or nav node gates on a permission no plugin declares.

Declaring stays optional on purpose, but the picker is built from declarations only, so such a route is ungrantable from the GUI and answers a permanent 403. A warning, not an error.

23. Give list pages one view-model helper from query to pagination, instead of ~40 hand-written lines each.

Every list screen bridges parseListQuery/paginate to the partials by hand. At minimum a buildPaginationModel(page, hrefFor). Exported through the barrel, so it is contract surface.

28. Let a plugin add entries to the profile menu through the plugin API.

The profile dropdown stays a dropdown: Sign out is its only entry today, and it will gain more. views/partials/shell.ejs hand-rolls it (AGENTS.md → UI, the dropdown rule), so the entries need a manifest field that discovery checks, gated like a nav node.

2. Rename the admin example plugin to a name that says what it administers.

Candidates: auth-admin, users-groups-admin. The id is also the mount path, so the compose files, E2E stacks and README move with it.

5. List grants that no installed plugin declares, with a remove action.

grantDiff never revokes an undeclared name, and nothing shows one either, so it can't be audited or cleaned, and reinstalling its plugin silently reactivates access nobody remembers granting.

14. Drop a double-clicked duplicate submit, with no client-side JavaScript.

A second identical POST is expected from a non-technical user, and today creates two users, mints two recovery codes, or registers two OAuth2 clients. It must not break a legitimately repeatable action. Sketch: a CSS-only affordance so the second click has nothing to hit, plus the host recognising a duplicate (same session, route and payload within a short window), logging and dropping it. Open: what identifies "the same submission" (a one-time token minted into each form beats hashing the payload), the window, where the record lives given the host is stateless, and how a plugin declares a route repeatable.