larvit/plainpages
3
0
Fork 0
Code Issues 1 Pull Requests 5 Actions 5 Packages Projects Releases Wiki Activity

Split handleRequest into named stages, so the stage order is written once #222

Merged
lilleman merged 5 commits from item-50 into main 2026-11-08 20:37:12 +01:00
Conversation 0 Commits 5 Files Changed 7
+384 -182
lilleman commented 2026-11-08 19:56:12 +01:00
Owner
Copy Link
Copy Source

Splits handleRequest in src/http/app.ts into module-level stages. The order is written once, in a short orchestrator, and every stage fits the 50-line limit. Behaviour doesn't change. Also files item 59, because reading cost is rising.

Splits `handleRequest` in `src/http/app.ts` into module-level stages. The order is written once, in a short orchestrator, and every stage fits the 50-line limit. Behaviour doesn't change. Also files item 59, because reading cost is rising.
lilleman added 2 commits 2026-11-08 19:56:12 +01:00
Split handleRequest into named stages, so the stage order is written once b4579131a2
File item 59: lower the reading cost, which is rising
CI / full-gate (push) Successful in 3m25s
Details
5e00bba85e
lilleman added 3 commits 2026-11-08 20:33:23 +01:00
Answer security round 1: 0 🔴, 0 🟡 (0 misread); route on the method and path read before the hooks c4eb9dde1f
Answer prose round 1: 0 🔴, 5 🟡 (0 misread); comments match the stages, item 59's detail reads plainly ea6c32a7db
Answer stability round 1: 0 🔴, 4 🟡 (0 misread); test the guards the split rewrote
CI / full-gate (push) Successful in 3m26s
Details
Mirror / github-mirror (push) Successful in 3s
Details
9ded502be3
lilleman marked the pull request as ready for review 2026-11-08 20:37:11 +01:00
lilleman merged commit 9ded502be3 into main 2026-11-08 20:37:12 +01:00
lilleman deleted branch item-50 2026-11-08 20:37:12 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claes.eismar.se lilleman (Lilleman auf Larv) read-only renovate-bot
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: larvit/plainpages#222
Delete Branch "item-50"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?