Rewrite the setup wizard #127

Closed

jcoffey-dev wants to merge 0 commits from phase4/wizard into main

pull from: phase4/wizard

jcoffey-dev commented

2026-10-19 04:14:30 +00:00

Owner

Phase 4, part 2: the setup wizard. Stacked on #126, so merge that first.

The wizard's logic moves to wizard.ts with tests: which fields sit on which page, what's still missing, jumping to the first missing field, matching a server refusal to its field, reading the result and the sign-in details shown at the end. The component is rewritten around it. No change in behavior.

After this PR: 83.3%, 12,517 upstream lines left in 48 files, 36 dual headers. Tests: 756 → 766.

Visual check: see #126. It covered all six phase 4 PRs and found no change in the UI.

Typecheck, lint, i18n:check, tests, build and notice check pass locally. No new strings.

Phase 4, part 2: the setup wizard. Stacked on #126, so merge that first. The wizard's logic moves to `wizard.ts` with tests: which fields sit on which page, what's still missing, jumping to the first missing field, matching a server refusal to its field, reading the result and the sign-in details shown at the end. The component is rewritten around it. No change in behavior. After this PR: 83.3%, 12,517 upstream lines left in 48 files, 36 dual headers. Tests: 756 → 766. **Visual check:** see #126. It covered all six phase 4 PRs and found no change in the UI. Typecheck, lint, i18n:check, tests, build and notice check pass locally. No new strings.

jcoffey-dev added 2 commits 2026-10-19 04:14:30 +00:00

Rewrite the delivery trace page

ci / build (pull_request) Skipped

Details

github/ci (branch) GitHub Actions

Details

ci / github (pull_request) Successful in 2m14s

Details

ci / publish (pull_request) Skipped

Details

ci / notices (pull_request) Successful in 2m48s

Details

b1d833bbe4

The trace's logic is now plain, tested code: trace.ts pairs each step's
start with its outcome, groups steps by delivery attempt, names and
describes them, and holds the trace's states as a reducer; stream.ts runs
the token and the event stream. The page only draws them, unchanged.

Two fixes on the way: "Started Ns ago" counts from the trace's own start
(it carried on from the previous trace before), and cancelling while the
token is still on its way no longer opens the stream afterwards.

Rewrite the setup wizard

ci / build (pull_request) Skipped

Details

github/ci (branch) GitHub Actions

Details

ci / github (pull_request) Successful in 1m10s

Details

ci / publish (pull_request) Skipped

Details

ci / notices (pull_request) Successful in 2m47s

Details

737eac2279

Its rules are now plain, tested code in wizard.ts: which fields each step
shows, which are required, where the server's refusal lands, and what the
finish screen lists. The wizard itself is rewritten on top of them, with
the same steps, look and update.

jcoffey-dev referenced this pull request

2026-10-19 04:14:43 +00:00

Rewrite the actions page #128

jcoffey-dev closed this pull request

2026-10-19 04:18:29 +00:00

jcoffey-dev deleted branch phase4/wizard

2026-10-19 04:18:29 +00:00

jcoffey-dev commented

2026-10-19 04:18:38 +00:00

Author

Owner

Landed in main with #128, which was stacked on this one.