Skip to content

docs: align §1 thesis sentence + README with the pillar-#2 reframe - #204

Merged
jensholdgaard merged 1 commit into
mainfrom
docs/align-thesis-sentence
Jun 14, 2026
Merged

docs: align §1 thesis sentence + README with the pillar-#2 reframe#204
jensholdgaard merged 1 commit into
mainfrom
docs/align-thesis-sentence

Conversation

@jensholdgaard

Copy link
Copy Markdown
Owner

What

Resolves RFC 0012 §7's open question (maintainer-directed): aligns CLAUDE.md §1's thesis sentence and README's parallel with the pillar-#2 reframe.

Why

Both thesis sentences read "collapses … the compression layer … into one stack." That's accurate as architecture, but per RFC 0012 §7 it can be misread as template-mining-is-the-byte-compressor. A one-clause clarification removes the ambiguity without touching the load-bearing sentence.

Changes

  • CLAUDE.md §1 — adds: "Here 'compression' is Parquet's byte codec plus the miner's logical template reduction (§2 pillar docs: apply RFC maturity-model amendments #2) — the value of which is query pruning, not on-disk bytes that beat a codec."
  • README.md — parallel parenthetical clause.
  • CLAUDE.md footer — the 2026-06-14 changelog line now notes §1 too.
  • RFC 0012 §7 — open question marked resolved.

Scope / verification

Within RFC 0012's approved meta scope (§7 explicitly: "in scope for the same enactment" at the maintainer's discretion). The framing-phrase grep (before any byte-level codec / over a competent byte codec) still returns only RFC 0011 + RFC 0012 — the new clauses use "bytes that beat a codec", not the framing phrases. mdbook build clean.

🤖 Generated with Claude Code

Resolves RFC 0012 §7's open question (maintainer-directed). The §1 thesis
sentence (CLAUDE.md) and README's parallel both list "collapses … the
compression layer …"; add a one-clause clarification to each that the
compression collapsed is Parquet's byte codec plus the miner's *logical*
reduction (query pruning), not on-disk bytes that beat a codec. The
load-bearing thesis sentence itself is unchanged; only the clarifying
clause is added. CLAUDE.md footer changelog extended to note §1; RFC 0012
§7 open-question marked resolved. Within RFC 0012's approved meta scope.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jensholdgaard
jensholdgaard requested a review from Copilot June 14, 2026 19:42
@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@jensholdgaard, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 24 minutes and 35 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: bbe3245d-0f55-42ff-b239-d7b291bf9818

📥 Commits

Reviewing files that changed from the base of the PR and between 7f9e57d and 89ea88d.

📒 Files selected for processing (3)
  • CLAUDE.md
  • README.md
  • docs/rfcs/0012-claude-md-pillar-2-wording.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/align-thesis-sentence

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI 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.

Pull request overview

This PR clarifies the meaning of “compression” in the project’s thesis sentences so readers don’t misinterpret template mining as a byte-level compressor; it aligns CLAUDE.md §1 and README.md with the pillar-#2 reframing documented in RFC 0012.

Changes:

  • Add a one-clause clarification in README.md’s thesis paragraph that “compression” refers to Parquet’s byte codec plus the miner’s logical reduction (query pruning), not better-than-codec byte savings.
  • Add the parallel clarification to CLAUDE.md §1 and update the footer changelog entry to reflect both §2 and §1 updates.
  • Mark RFC 0012 §7’s open question about updating the §1 thesis sentence as resolved, documenting the enactment.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
README.md Adds an explicit parenthetical clarifying “compression” in the thesis sentence to avoid bytes-vs-codec misreadings.
CLAUDE.md Adds the same clarification to §1’s thesis sentence and updates the 2026-06-14 footer entry to mention §1.
docs/rfcs/0012-claude-md-pillar-2-wording.md Updates §7 to mark the §1-thesis parallel touch as resolved and records the rationale.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jensholdgaard
jensholdgaard merged commit 0d9aa4f into main Jun 14, 2026
12 checks passed
@jensholdgaard
jensholdgaard deleted the docs/align-thesis-sentence branch June 14, 2026 20:31
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