docs: add code-audit improvement suggestions - #27
Draft
i11v wants to merge 1 commit into
Draft
Conversation
33 audited findings across worker, web, and tooling/CI, each written as a self-contained task (problem, quoted code, concrete fix, acceptance criteria) so any suggestion can be picked up and implemented independently. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WgpPdTSGKeydeG9Xqb8Sqv
i11v
force-pushed
the
claude/code-audit-suggestions-5ilnk8
branch
from
July 13, 2026 19:51
62eb710 to
6f22b97
Compare
|
🚀 Preview deployed: https://preview-27.tablo.run |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds
docs/IMPROVEMENTS.md— a full code audit of the repo with 33 verified findings, written so each suggestion is a self-contained task (problem statement, quoted code, concrete fix sketch, acceptance criteria, effort estimate) that can be picked up and implemented independently, including by a smaller model.Scope of the audit
packages/worker(8 findings: WKR-01…08) — Durable Objects, gateway caching, Golemio clientpackages/web(13 findings: WEB-01…13) — React app, WS hook, URL/state handling, accessibilityEvery claim was verified against the actual code (and where relevant, by executing the code in question); a closing section lists things that were examined and deliberately not flagged, to save future auditors time.
Highlights (fix-first list)
?s=URL bypasses the wire-protocol caps and wedges the WebSocket feedNaNcoordinate passes the build-time schema check but serializes tonull, shipping a client-breaking stop index through green CItest:integrationisn't wired into CIBaseline at audit time: typecheck, lint, and all 82 unit tests green.
🤖 Generated with Claude Code
https://claude.ai/code/session_01WgpPdTSGKeydeG9Xqb8Sqv
Generated by Claude Code