Skip to content

docs: correct CaptureKit limits and summary comments - #2428

Merged
r3dbars merged 1 commit into
mainfrom
codex/review-tools-cli-11
Nov 12, 2026
Merged

r3dbars merged 1 commit into
mainfrom
codex/review-tools-cli-11

Conversation

@r3dbars

@r3dbars r3dbars commented Nov 10, 2026

Copy link
Copy Markdown
Owner

Correct CaptureKit's classification-prefix comment to 512 KB, matching the existing constant and Core frontmatter limit. Remove an obsolete app-parser reference and the incorrect two-shape wording from the summary parser documentation.

Only documentation comments changed; non-comment lines are byte-identical to the base. Independent factual/source review: GREEN. The same two comment hunks also appear in pending #2336; that broader PR remains untouched.

Validation: nine mapped checks PASS, including the 79 CaptureKit, 149 CLI and 272 MCP package tests. Preflight passes; verify-change reports no applicable app check. E2E is deliberately BLOCKED with exit 3 on this account pending the reviewed isolation union, so the canonical proof records FAIL. No native app, model, SDK or real capture-library proof is claimed. Draft retained for review.

@r3dbars
r3dbars merged commit 171791b into main Nov 12, 2026
8 of 11 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