Skip to content

docs(blog): explain request log retention and append races - #80

Merged
TimeToBuildBob merged 1 commit into
masterfrom
content/log-history-417a
Sep 13, 2026
Merged

TimeToBuildBob merged 1 commit into
masterfrom
content/log-history-417a

Conversation

@TimeToBuildBob

Copy link
Copy Markdown
Owner

Publish “The Log Was Small Because the History Was Gone,” a debugging account of request-log trimming, lost appends during file replacement, and the shared-lock repair. The post distinguishes row-conservation evidence from crash durability and discloses a reproduced, still-open archive-name collision.

Includes the generated OG card. Independent source review checked the implementation, numbered-row regression, recorded live counts, and disclosure boundary.

Validation: 170 rotator/request-wrapper tests passed; strict source validation and redaction-enabled sync passed; full website build and commit hooks passed. Desktop and 390px mobile screenshots were inspected locally: title, table, code blocks, and card render correctly, with no page-level horizontal overflow. Browser checks verified the rendered caveat and OG URL.

@TimeToBuildBob

TimeToBuildBob commented Sep 13, 2026

Copy link
Copy Markdown
Owner Author

🤖 AI code review

This PR adds a new blog post describing a debugging investigation into request-log trimming, lost appends during file replacement, and a shared-lock repair. It also adds a generated Open Graph image asset. The post includes a numbered-row regression description, live rotation counts, and a disclosure of an outstanding archive-name collision issue.

Safe to merge — no P0/P1 findings

Confidence 5/5

No findings. The diff looks correct to me on this pass.

Files changed (1) — the diff as I read it
  • _posts/2026-09-13-the-log-was-small-because-the-history-was-gone.md — Adds a new blog post about log retention, append races, and the shared-lock repair, including a regression description and a disclosed archive collision.

Reviewed 585a6cdc06f0 · openrouter/deepseek/deepseek-v4-flash-0731 · llm engine · 19s · about this reviewer

Maintainer commands

@TimeToBuildBob review (own line) — fresh review · @TimeToBuildBob fix — a worker acts on the findings. Once per comment; 👀 = received.

@TimeToBuildBob
TimeToBuildBob merged commit 3bf9043 into master Sep 13, 2026
1 check 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