forge update

[thedavidmeister]

Motivation

Solution

Checks

By submitting this for review, I'm confirming I've done the following:

• made this PR as small as possible
• unit-tested any new functionality
• linked any relevant issues or PRs
• included screenshots (if this involves a front-end change)

Summary by CodeRabbit

• Chores
  • Updated an internal subproject reference and bumped compiler version; no functional impact.
• Documentation
  • Added/clarified internal documentation for hashing behavior.
• Tests
  • Updated tests and benchmark snapshots (numeric benchmark values adjusted); no behavioral changes.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Updated a forge-std submodule pointer, bumped Solidity pragmas, renamed internal parameters and local variables with added documentation in hashing libraries, adjusted tests to match renamed parameters, and updated a gas benchmark snapshot; no public API signatures were changed.

Changes

Cohort / File(s)	Summary
Submodule lib/forge-std	Advance submodule pointer from b8f065f... to 1801b054... (single-line pointer change).
Benchmark Snapshot .gas-snapshot	Updated two benchmark metric values for LibHashNoAllocTest (numeric snapshot changes only).
Hash Library (optimized) src/LibHashNoAlloc.sol	Bumped pragma to ^0.8.25; renamed internal parameters/locals (e.g., data_→data, words_→words, a_/b_→a/b), added docs for HASH_NIL and internal funcs, minor assembly variable adjustments; no signature or algorithm changes.
Hash Library (slow reference) test/LibHashSlow.sol	Pragma bump to ^0.8.25; renamed internal params/locals in hash*Slow functions and updated keccak256 usages to new names; no public API changes.
Tests test/LibHashNoAlloc.t.sol	Pragma bump to ^0.8.25; test function parameter names updated (e.g., bytes_→bs, words_→words, a_→a); adjusted local variable name for HASH_NIL check.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• 2025 10 21 audit #4: Updates the lib/forge-std submodule pointer and touches the same benchmark snapshot area.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'forge update' is vague and generic, using non-descriptive terms that don't convey meaningful information about the specific changes in the changeset.	Provide a more descriptive title that highlights the main purpose of the update, such as 'Update forge-std subproject and refactor parameter naming conventions' or 'Bump Solidity compiler version and standardize parameter names'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
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.}

---

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