Fix #39: settings Danger Zone + delete repository #56

Merged

crueber merged 1 commit from fix/issue-39 into main

2026-09-04 21:03:21 +00:00

crueber commented

2026-09-04 20:58:18 +00:00

Owner

No new backend: DELETE /{o}/{r}/api (both lanes, admin-only, 204) + SDK repo.delete() already exist and are httptest-covered — this wires them. Docs: 01 §5.1 (delete + fork/GC semantics: children survive, stale forks.json entries tolerated as miss=skip) + Decisions entries in 01 and 08.

Tests: node --test web/test/unit/*.test.js 238/238 green (incl. 3 new danger matcher tests); vite build clean; real headless-Chrome drive of /demo/walgit/settings in dark + light, zero console errors, full flow verified (arm/disarm states, real DELETE, navigate to /, repo gone server-side). No new deps. No Go changes (coverage gate N/A).

Adds a Danger Zone section at the bottom of the repo settings page with Delete Repository as the first entry (typed exact owner/name confirm + disabled-until-match button + double-submit guard; success navigates to /; failures surface plain-text). No new backend: DELETE /{o}/{r}/api (both lanes, admin-only, 204) + SDK repo.delete() already exist and are httptest-covered — this wires them. Docs: 01 §5.1 (delete + fork/GC semantics: children survive, stale forks.json entries tolerated as miss=skip) + Decisions entries in 01 and 08. Tests: node --test web/test/unit/*.test.js 238/238 green (incl. 3 new danger matcher tests); vite build clean; real headless-Chrome drive of /demo/walgit/settings in dark + light, zero console errors, full flow verified (arm/disarm states, real DELETE, navigate to /, repo gone server-side). No new deps. No Go changes (coverage gate N/A).

crueber added 1 commit

2026-09-04 20:58:18 +00:00

Fix #39 : settings Danger Zone + delete repository (01 §5.1, 08 §2) a9fde1d40a

Backend already provides DELETE /{o}/{r}/api on both lanes (admin-only,
204, manifest-first delete) with SDK mirror repo.delete() — this change
wires the UI: Danger Zone section at the bottom of Settings, reusable
DangerConfirm typed-confirm (exact owner/name, busy guard), success
navigates to / since the repo is gone. Docs: 01 §5.1 delete + fork/GC
semantics, 08 §2 DangerConfirm contract.

crueber referenced this pull request

2026-09-04 20:58:21 +00:00

At the bottom of the "settings" tab, add a "Danger Zone" #39

crueber referenced this pull request

2026-09-04 21:01:27 +00:00

At the bottom of the "settings" tab, add a "Danger Zone" #39

crueber merged commit 7274f6d90f into main

2026-09-04 21:03:21 +00:00

crueber referenced this pull request from a commit

2026-09-04 21:03:21 +00:00

Merge pull request 'Fix #39: settings Danger Zone + delete repository' (#56) from fix/issue-39 into main

crueber referenced this pull request