CLAUDE.md: state both routes the tests take into the library

[thedavidmeister]

Closes #117

"Tests exercise the library through TestCloneFactory, a pure-delegation
concrete" names one of two routes as the route.
test/concrete/TestCloneFactory.sol exposes only the four
ICloneableFactoryV4 entry points, so the library's internal pure/view
units — effectiveSalt, effectiveOpenSalt, cloneCreationCode,
predictCloneAddress, checkImplementationCode — are unreachable through it.
Seven of the eight test files call LibICloneableFactoryV4.<fn> directly for
that reason.

A session adding a test for one of those units was told to route it through a
concrete that cannot reach it, and would either add an entry point the library's
own surface does not have or conclude the unit is untestable.

The replacement states both routes and what picks between them: the concrete
exists because msg.sender namespacing and the NewClone event are observable
only across an external call. "Pure-delegation concrete" is dropped from this
sentence — TestCloneFactory's own NatSpec says it at length, and CLAUDE.md is
under a byte cap.

CLAUDE.md is 3836 bytes on this branch against the 4096 agent-context-cap
(3807 on main). With all seven CLAUDE.md fixes from this audit batch landed it
comes to 4048.

Textual overlap with the PR closing #112. deno fmt reflows the whole Build
& Test paragraph as one unit, and that PR rewrites the sentences before this
one, so the two hunks overlap however the wording is arranged. Both branch off
main and each stands alone; I merge main into whichever lands second and
resolve.

QA

• Discriminating tests: n/a — CLAUDE.md is process documentation, and tests
  never assert doc text.
• Mutations applied: n/a — no src/ or test/ code changed.
• Oracle: test/concrete/TestCloneFactory.sol's declared surface against
  src/lib/LibICloneableFactoryV4.sol's function list, and the call sites in
  test/src/**.
• Category check: README.md describes the interfaces and the library but
  makes no claim about how the tests reach them, so CLAUDE.md held the only copy
  of this claim.

🤖 Generated with Claude Code

[coderabbitai]

Warning

Review limit reached

Next included review available in 41 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: fce2aced-28f6-423f-b967-fb286c30fd68

📥 Commits

Reviewing files that changed from the base of the PR and between 710dff5 and fad37bb.

📒 Files selected for processing (1)

• CLAUDE.md

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}