For: src/web/pages/settings.ts is too big to read or to change without a conflict; hooksPage, webhookCreate, webhookDelete and the helpers only they use (settings.ts:1920–2086) is one concept and gets its own module, with no change in behaviour.
Done:
[ ] src/web/pages/hooks.ts holds hooksPage, webhookCreate, webhookDelete and the helpers only they use (settings.ts:1920–2086); src/web/pages/settings.ts keeps none of it and imports or calls it from the new file.
[ ] Every route, page and form answers exactly as before: Settings › Webhooks draws the same HTML, a hook's secret is still never echoed, and the webhook tests pass unchanged.
[ ] A helper both files need is exported from where it lives now; nothing is copied.
[ ] Before pushing: bun x tsc --noEmit, the tests that cover the moved code, and bun scripts/design-lint.ts when src/web/ changes. A red test elsewhere that main also has is named in the pull.
Files: src/web/pages/settings.ts, src/web/pages/hooks.ts, the files that import what moved.
Risk: the Settings tab bar is shared; leave it where it is and import it. Runs in parallel with the labels move: keep each to its own lines of settings.ts.
For: src/web/pages/settings.ts is too big to read or to change without a conflict;
hooksPage,webhookCreate,webhookDeleteand the helpers only they use (settings.ts:1920–2086) is one concept and gets its own module, with no change in behaviour.Done:
hooksPage,webhookCreate,webhookDeleteand the helpers only they use (settings.ts:1920–2086); src/web/pages/settings.ts keeps none of it and imports or calls it from the new file.bun x tsc --noEmit, the tests that cover the moved code, andbun scripts/design-lint.tswhen src/web/ changes. A red test elsewhere that main also has is named in the pull.Files: src/web/pages/settings.ts, src/web/pages/hooks.ts, the files that import what moved.
Risk: the Settings tab bar is shared; leave it where it is and import it. Runs in parallel with the labels move: keep each to its own lines of settings.ts.