Skip to content

docs: propose safe public FIXED response cache ADR - #85

Merged
DPS0340 merged 2 commits into
devfrom
docs/25-safe-fixed-cache-adr
Sep 14, 2026
Merged

docs: propose safe public FIXED response cache ADR#85
DPS0340 merged 2 commits into
devfrom
docs/25-safe-fixed-cache-adr

Conversation

@DPS0340

@DPS0340 DPS0340 commented Sep 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

Related to #25; does not close it.

Document a proposed, restricted public FIXED representation cache before restoring the currently broken automatic population path. A learning-order-only fix would activate routing/middleware bypass, wire-framing and lifetime hazards.

  • Introduce a provisional explicit public-invariance opt-in rather than silently strengthening existing FIXED declarations.
  • Define common parsed eligibility, owned bounded representation storage, normal send paths, and deliberate retirement of legacy automatic wire replay.
  • State the compatibility change, alternatives, rollback policy, correctness/security gates and separate performance acceptance requirement.

Documentation only. Status is Proposed for maintainer review. No runtime implementation, cache activation, defaults, automatic merge or performance improvement is included. The compatibility/API decision requires maintainer acceptance before release. PR #84 remains an independent serializer prerequisite, not cache restoration.

Verification

  • Independent source-grounded review found no decisive design contradiction or missing safety invariant.
  • Exact reviewed document blob: ac195e19c5e94d050ea039c162d27ced10742f75.
  • git diff --cached --check passed; exactly one documentation file is staged.
  • Source anchors are pinned to 7e19b461c5772f4e46e7d58fb96827bbb954cf97; cache source/option anchors were also checked unchanged on dev 374e3873c46da1c18aaa7a4d16461fb1f1b19bb4.
  • No runtime tests or benchmarks were rerun for this prose-only change. Historical experimental evidence and limitations are linked in the ADR.

gg582 added a commit that referenced this pull request Sep 13, 2026
Public TCP observation (issue #25) shows FIXED routes invoke the
handler on every request in both classic and C1M modes: the documented
RAM-cache contract is not what the runtime does. Correct the promise in
endpoint_opts.h and docs/api/app.md, add docs/fixed-cache-status.md
describing the evidence, why the learning block cannot simply be moved,
and the ADR-0001 (draft PR #85) restoration path.

(cherry picked from commit 3c80056)
@DPS0340
DPS0340 marked this pull request as ready for review September 14, 2026 05:48
@DPS0340
DPS0340 merged commit c70ee77 into dev Sep 14, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant