Skip to content

docs: backfill the paged-lenses TIB and docs/ conventions - #53

Merged
haydenshively merged 2 commits into
mainfrom
docs/paged-lenses-backfill
Aug 28, 2026
Merged

docs: backfill the paged-lenses TIB and docs/ conventions#53
haydenshively merged 2 commits into
mainfrom
docs/paged-lenses-backfill

Conversation

@haydenshively

@haydenshively haydenshively commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Docs-only. Extracted from #52, whose code half was closed — so the doc backfill never landed.

Adds two files:

  • docs/000012-tib-paged-lenses-partial-results.md — the design document for the paged-lens work that shipped in 07e49df and 0df02a9: RETURN mode deleted, the (U[] results, uint256[] skipped) lens contract and its four clauses, the client fill loop, and the decision to respond in the shape the policy's abi declares. Written against the 0.0.12 tree, so its src/foo.ts:42 references resolve there, not against main.
  • docs/README.md — the conventions for this directory: the NNNNNN-<kind>-<slug>.md filename, the version-as-baseline rule, TIB frontmatter (landed present only once work ships), the fixed section spine, and the Notes/Derivation split.

The index table lists only the document included here; the two 000016 rows from the original branch would have been broken links. The illustrative 000016-* filenames inside the format examples are left as-is — they're format illustrations, not links.

No source or test changes.

🤖 Generated with Claude Code


Devin Review

Backfills the design document for the paged-lens work that shipped in
07e49df and 0df02a9, plus the docs/README.md describing the naming
scheme, TIB frontmatter, and section spine those documents follow.

Docs only; no source or test changes.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@haydenshively haydenshively self-assigned this Aug 28, 2026

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@haydenshively
haydenshively requested review from cashd and jonator August 28, 2026 03:12

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 887be369a9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/000012-tib-paged-lenses-partial-results.md Outdated
Ten `src/foo.ts:42` references in the paged-lenses TIB were written against a
different tree than the `0.0.12` baseline its filename declares, so following
them landed on unrelated code. Recalculated against `2cd71df`.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

@jonator jonator left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Has a pretty claude-ish tone but LGTM

@haydenshively
haydenshively merged commit 7ea8e70 into main Aug 28, 2026
3 checks passed
@haydenshively
haydenshively deleted the docs/paged-lenses-backfill branch August 28, 2026 13:44
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.

2 participants