Skip to content

docs: Size recovery cards to the backup length - #96

Open
BenWestgate wants to merge 1 commit into
reviewability-v1from
claude/new-issue-fixes-fvhbl3-88
Open

BenWestgate wants to merge 1 commit into
reviewability-v1from
claude/new-issue-fixes-fvhbl3-88

Conversation

@BenWestgate

Copy link
Copy Markdown
Owner

Requested by Ben · project thread

Before: one portrait card with 32 four-character boxes, four per row, sized for the 127-character long form.

After: recovery-card.html has exactly 12 boxes (48 characters, 128-bit) on one landscape line. New recovery-card-256.html has 19 boxes (74 characters, 256-bit), 12 on the first line and 7 on the second, with a half-width last box for the final two characters. Neither card has a length field.

How: boxes sit in blocks of four with a wider gap between blocks, as the CLI prints them. Each box has a small light number in its upper-left corner. The guide and README link both cards. Both print on one letter page in Chromium.

Rarer lengths (54, 61, 67, 127) get no card here; the issue leaves that open.

Closes #88

🤖 Generated with Claude Code

https://claude.ai/code/session_015CuLXqAvAovfoVcUmmogwa


Generated by Claude Code

The card had 32 blank boxes, enough for the 127-character long form,
while most backups are 48 characters. Make the standard card exactly
12 groups and add a 256-bit card with 19 groups (74 characters, the
last box holding two). Neither needs a length field.

Print in landscape so the 12 groups fit on one line, matching the
first entry prompt. Groups follow the CLI's blocks of four with a wider
gap between blocks, and each box carries a small light number.

Closes #88

Claude-Session: https://claude.ai/code/session_015CuLXqAvAovfoVcUmmogwa
@BenWestgate BenWestgate self-assigned this Oct 1, 2026

@BenWestgate BenWestgate left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AI-generated release-gate review, posted at the maintainer's request.

ACK a77f83b for the scoped #88 requirements. The standard card now has exactly 12 four-character boxes for the 48-character/128-bit form; the new 256-bit card has 18 four-character boxes plus one two-character box for the 74-character form. The landscape flex geometry is consistent with the requested CLI grouping: three four-box blocks fit on the first line (~9.96in inside the 10.3in body), while the remaining seven groups wrap to the second line. The guide/README distinguish the two templates, and #88 explicitly leaves 54/61/67/127-character templates open, so their absence is not a blocker.

No correctness blocker found in the documentation/HTML diff. Before human integration, visually print-preview both templates on letter landscape at the supported browser/printer scale, and rewrite/squash the Claude-authored commit under the repository authorship policy.

@BenWestgate BenWestgate added the documentation Improvements or additions to documentation label Oct 1, 2026
@BenWestgate

Copy link
Copy Markdown
Owner Author

Rendered both cards to landscape letter PDFs with Chromium 153 and inspected the pages. The 48-character card has 12 numbered boxes on one line; the 74-character card has 18 full boxes plus the half-width final box on the second line. Both occupy one page with no visible clipping or overlap. The print layout is ready for human review. AI-assisted visual review disclosed here.

@BenWestgate BenWestgate added the gate: adversarial review Resolve, merge, or explicitly defer before the next full adversarial review. label Oct 1, 2026 — with ChatGPT Codex Connector
@BenWestgate
BenWestgate marked this pull request as ready for review October 1, 2026 09:09
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation gate: adversarial review Resolve, merge, or explicitly defer before the next full adversarial review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants