fix: mask signer row operand to low byte

[thedavidmeister]

Summary

• subParserSigners read the full 16-bit OperandV2 value as the context row index, so signer<256>() silently resolved to the same row as signer<0>() (256 mod 256 = 0 at the EVM word level) but only by accident — larger values could land on entirely wrong rows.
• Fix: apply & 0xFF to clamp the operand to the intended 0–255 range before passing it to LibSubParse.subParserContext.
• Regression test: testSubParserContextSignerRowMaskLowByte verifies signer<256>() resolves to row 0.
• Pre-pins 0.1.13 deploy constants: SubParser address changes (new bytecode), all other contracts (RaindexV6, RouteProcessor, arb contracts) are unchanged from 0.1.12.

REQUIRES redeploy at land — the SubParser bytecode changes, so it must be redeployed to its new CREATE2 address before testProdDeploy* goes green.

Closes #2653

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

Summary by CodeRabbit

• New Features

  • Added support for Raindex ecosystem version 0.1.13 with pinned deployment addresses and code hashes for core components.

• Bug Fixes

  • Updated operand interpretation in signer-grid row derivation to use low byte only instead of full value.

• Tests

  • Added test verifying signer-grid row masking behavior wraps correctly.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: bbced90e-4ac2-4c02-879f-5a228f6a8c0b

📥 Commits

Reviewing files that changed from the base of the PR and between 175e84a and 8ca5816.

⛔ Files ignored due to path filters (1)

• src/generated/RaindexV6SubParser.pointers.sol is excluded by !**/generated/**

📒 Files selected for processing (4)

• crates/test_fixtures/abis/RaindexV6SubParser.json
• src/lib/LibRaindexSubParser.sol
• src/lib/deploy/LibRaindexDeploy.sol
• test/concrete/parser/RaindexV6SubParser.signers.t.sol

---

📝 Walkthrough

Walkthrough

subParserSigners is updated to mask the OperandV2 operand to its low 8 bits (& 0xFF) before using it as the signer-grid row index, matching the behavior of the sibling subParserSignedContext word. A test verifying that signer<256>() maps to row 0 is added, the compiled SubParser bytecode fixture is refreshed, and twelve new pinned deployment address/codehash constants for the 0.1.13 release are appended to LibRaindexDeploy.

Changes

Signer operand masking fix

Layer / File(s)	Summary
Low-byte operand masking in subParserSigners and test src/lib/LibRaindexSubParser.sol, test/concrete/parser/RaindexV6SubParser.signers.t.sol, crates/test_fixtures/abis/RaindexV6SubParser.json	subParserSigners masks the unwrapped OperandV2 with 0xFF before computing the row index; a new test asserts signer<256>() resolves to the same row as signer<0>(); the compiled bytecode fixture is updated to match the recompiled contract.

0.1.13 deployment constants

Layer / File(s)	Summary
Pinned 0.1.13 address and codehash constants src/lib/deploy/LibRaindexDeploy.sol	Twelve new address and bytes32 constants covering all six RaindexV6 ecosystem contracts at tag 0.1.13 are appended after the existing 0.1.12 entries.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• rainlanguage/raindex#2635: Adds routing tests for subParserSigners row-from-operand behavior that directly anticipates the & 0xFF masking fix introduced in this PR.
• rainlanguage/raindex#2683: Follows the same pattern of appending pinned address/bytes32 codehash constants to LibRaindexDeploy, adding the 0.1.2 tag entries.

Poem

🐇 A byte too wide, the signer would roam,
Past row 255, far away from home.
One mask of 0xFF, and the number wraps clean,
signer<256>() now rows to row 0, serene.
New constants pinned at 0.1.13 with care —
The rabbit deployed them with flair! 🎉

🚥 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 "fix: mask signer row operand to low byte" directly describes the main change: applying & 0xFF masking to the operand in the subParserSigners function, which is the core fix across all modifications.
Linked Issues check	✅ Passed	The PR fully addresses #2653 by masking the signer row operand to & 0xFF in LibRaindexSubParser.sol, aligning signers with signed-context behavior, adding a regression test, and pinning deploy constants for the fixed bytecode.
Out of Scope Changes check	✅ Passed	All changes are directly related to issue #2653: the operand masking fix, its test, updated ABI fixture, and pre-pinned deploy constants for the new bytecode. No unrelated modifications detected.
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 2026-06-17-issue-2653-signer-row-mask

---

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