test V5 sentinel + min sentinels through IFlowV5 re-export

[thedavidmeister]

What

test/interface/IFlowV5.t.sol imported RAIN_FLOW_SENTINEL directly from src/interface/deprecated/v4/IFlowV4.sol — the V5 test never exercised the V5 interface at all. The file was byte-identical to IFlowV4.t.sol (modulo the relative import depth), so if IFlowV5.sol ever changed its sentinel re-export (intentionally or by mistake), the V5 test would still pass because it bypassed V5 entirely.

This PR changes the import to come from src/interface/IFlowV5.sol, exercising the re-export surface the test is named to pin. It also adds a pin for the MIN_FLOW_SENTINELS re-export (== 3) through V5, since that constant is re-exported on the same chain (V3 → V4 → V5) and was previously untested via V5.

Test-only. No source changes.

Mutation validation

Both assertions were proven to read through the V5 interface by mutating the V5 re-export and confirming the relevant test fails, then restoring:

Mutation in IFlowV5.sol	Test	Result
RAIN_FLOW_SENTINEL re-export replaced with a wrong local constant	testSentinelValue	FAIL (killed)
MIN_FLOW_SENTINELS re-export replaced with 99	testMinFlowSentinelsValue	FAIL (killed)

Before the import change (importing from V4 directly), neither mutation would have been caught — that is the gap this PR closes.

Verification

• forge build: clean.
• forge test: 56 passed, 0 failed (the V5 suite is now 2 tests, both green).
• forge fmt --check on the changed file: clean.

Note: main is currently red on rainix-sol-static due to a pre-existing slither redundant-statement finding fixed by #476 (not yet merged). This PR's CI will show that same pre-existing static red until #476 lands — it is unrelated to this change, which touches only a test file.

Closes #419

🤖 Generated with Claude Code

[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 48 minutes and 2 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: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 6d21cc17-2a7b-4b9f-9abd-733b0bdff925

📥 Commits

Reviewing files that changed from the base of the PR and between 28d5e67 and e48bd8f.

📒 Files selected for processing (1)

• test/interface/IFlowV5.t.sol

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch 2026-06-14-issue-419

---

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 and usage tips.}

[thedavidmeister]

Reviewed e48bd8f: tests V5 sentinel + MIN_FLOW_SENTINELS through the IFlowV5 re-export (mutation-validated); rebased onto green main, CI green. Closes #419. LGTM.