Skip to content

Fix factual errors found in topic brief review - #4

Merged
pinebit merged 1 commit into
mainfrom
fix-review-findings
Sep 26, 2026
Merged

pinebit merged 1 commit into
mainfrom
fix-review-findings

Conversation

@pinebit

@pinebit pinebit commented Sep 26, 2026

Copy link
Copy Markdown
Owner

Corrects or qualifies claims in all 19 topic briefs that an external review flagged as wrong, outdated, or overstated. Each finding was re-checked before editing, against primary docs or by running code (Go, Python, Node, tsc).

Notable changes:

  • devops: Terraform removed blocks destroy the resource by default; only lifecycle { destroy = false } keeps it. preStop runs before SIGTERM. kube-proxy modes now list nftables (GA 1.33) and IPVS as deprecated (1.35). PDBs apply only to evictions.
  • distributed: Raft majority-commit applies to current-term entries only; Kafka ISR includes the leader, and min.insync.replicas is a floor, not an ack count; 3f + 1 scoped to partial synchrony; state-based vs op-based CRDTs.
  • database: optimistic locking increments version; skip scan (PostgreSQL 18+, MySQL 8.0.13+); WAL durability wording; PostgreSQL recovery is redo-only; 3NF vs BCNF split; PgBouncer prepared statements need max_prepared_statements > 0.
  • solidity / ethereum: SSTORE cost described by original/current/new value; override not needed for interface functions since 0.8.8; EIP-7702 nonce increments; EIP-712 and permit replay caveats.
  • golang: closed buffered channels drain before returning zero values; interface comparison panics only for the same non-comparable type; loop-variable scoping applies to := variables only.
  • python, rust, javascript, typescript, frontend, backend, networking, os, aws, security, system, algorithms, ai: narrower wording where the old claim held only in some cases (e.g. QUIC loss recovery is per connection, MSS 1440 on IPv6, bcrypt 72-byte limit, Node 20 timer phase change, const enum erasure, greedy set cover H(n) ≤ ln n + 1).

python3 scripts/lint_briefs.py and zensical build --clean --strict pass.

🤖 Generated with Claude Code

Correct or qualify claims across all 19 briefs flagged by an external
review, after re-verifying each finding against primary docs or by
running code. Includes the follow-up verification pass (SSTORE cost
rules, 403 semantics, allSettled, bcrypt limit, PgBouncer default,
set-cover bound, correlated availability).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@pinebit
pinebit merged commit 0eaca2b into main Sep 26, 2026
1 check passed
@pinebit
pinebit deleted the fix-review-findings branch September 26, 2026 08:11
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