From 14a07dd879d525217e3c336722331b4666c99e53 Mon Sep 17 00:00:00 2001 From: Lilleman auf Larv Date: Sun, 8 Nov 2026 15:10:53 +0100 Subject: [PATCH] Record the comprehension check: PR #216 paired ruling better --- docs/comprehension-history.md | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/docs/comprehension-history.md b/docs/comprehension-history.md index 18b245c..d8ad7c5 100644 --- a/docs/comprehension-history.md +++ b/docs/comprehension-history.md @@ -340,3 +340,22 @@ Inherited architect, decided by: - Locality, Shape: `src/http/app.ts:271` `shared`: the two buildContext calls differ only in the translator and params - Self-sufficiency: `src/http/context.ts:28` `RequestContext.assetHref`: the contract field says on the spot what it does and how it differs from the view local - Self-sufficiency: `README.md:1050` pagination sketch: URL to partial in five lines, with the param-renaming rule + +## 2026-11-08T14:10:47Z, PR #216 at bb134a3, against 6ca63c6 + +Ruling: better + +| Seat | Navigation | Locality | Shape | Self-sufficiency | Overall | +|---|---|---|---|---|---| +| Mid A | better | same | better | same | better | +| Inherited architect | same | better | better | same | better | + +Mid A, decided by: + +- Navigation, Shape: `src/http/request-tracking.ts:9` `trackRequest`: the per-request logger, span and access line now live in a named module instead of the tail of the 318-line `createApp` +- Navigation, Shape: `src/http/request-fields.ts:33` `clientAddress`: the caller-IP and X-Forwarded-For rule has its own pure function in a file named for it + +Inherited architect, decided by: + +- Locality, Shape: `src/http/request-tracking.ts:9` `trackRequest`: the exactly-once finalize and close listener are one named unit, changeable without reading `createApp`'s other state +- Locality, Shape: `src/http/request-fields.ts:33` `clientAddress`: pure function of peer, header and list, with the reasons for reading from the right beside it