test(supportsInterface): assert both SubParser and Extern branches reachable via C3 linearization

[thedavidmeister]

Closes #57
Closes #58
Closes #59

What

ERC4626Words.supportsInterface resolves the IERC165 diamond ambiguity between BaseRainlangSubParser and BaseRainlangExtern by delegating to super.supportsInterface with override(BaseRainlangSubParser, BaseRainlangExtern). No test previously called supportsInterface on a deployed instance, so a regression that silently dropped one branch would have gone undetected.

Adds ERC4626WordsSupportsInterfaceTest (9 tests) asserting:

• SubParser branch reachable: ISubParserV4, IDescribedByMetaV1, IParserToolingV1, ISubParserToolingV1
• Extern branch reachable: IInterpreterExternV4, IIntegrityToolingV1, IOpcodeToolingV1
• IERC165 itself is reported
• An unknown interface id returns false

Mutation-validated: return false causes 8/9 tests to fail.

Co-Authored-By: Claude noreply@anthropic.com

Summary by CodeRabbit

• Tests
  • Added comprehensive test coverage to verify correct interface support detection, including validation that unknown interfaces are properly rejected.

[coderabbitai]

Warning

Review limit reached

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

More reviews will be available in 58 minutes and 5 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.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate.

For paid Pro and Pro+ PR reviews, CodeRabbit uses rolling per-developer review limits. Reviews become available again as older review attempts age out of the rolling limit window.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 5cda872b-7ffd-47fb-818a-6fde33240319

📥 Commits

Reviewing files that changed from the base of the PR and between f212cdb and 20571cf.

📒 Files selected for processing (1)

• test/src/concrete/ERC4626Words.supportsInterface.t.sol

Walkthrough

A new Forge test file ERC4626Words.supportsInterface.t.sol is added. It instantiates ERC4626Words once in setUp and contains nine test functions that assert supportsInterface returns true for all interface IDs from both the SubParser and Extern inheritance branches plus IERC165, and returns false for 0xdeadbeef.

Changes

ERC4626Words supportsInterface test coverage

Layer / File(s)	Summary
supportsInterface assertions for both inheritance branches test/src/concrete/ERC4626Words.supportsInterface.t.sol	Introduces ERC4626WordsSupportsInterfaceTest with setUp constructing ERC4626Words and nine test functions covering ISubParserV4, IDescribedByMetaV1, IParserToolingV1, ISubParserToolingV1 (SubParser branch), IInterpreterExternV4, IIntegrityToolingV1, IOpcodeToolingV1 (Extern branch), IERC165, and rejection of 0xdeadbeef.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the PR's main change: adding tests for the supportsInterface method to verify both inheritance branches are reachable via C3 linearization in ERC4626Words.
Linked Issues check	✅ Passed	The PR comprehensively addresses all three linked issues (#57, #58, #59) by implementing test coverage for supportsInterface with assertions for both SubParser and Extern inheritance branches, IERC165, and negative controls.
Out of Scope Changes check	✅ Passed	The PR contains only test additions directly addressing the linked issues; no unrelated or out-of-scope changes to implementation code are present.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch issue-57-58-59-supportsInterface

---

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.}