Skip to content

Model UnlinkPool hashNote via PoseidonT4 boundary#65

Merged
Th0rgal merged 1 commit into
mainfrom
codex/unlink-pool-hashnote-poseidon-t4
May 14, 2026
Merged

Model UnlinkPool hashNote via PoseidonT4 boundary#65
Th0rgal merged 1 commit into
mainfrom
codex/unlink-pool-hashnote-poseidon-t4

Conversation

@Th0rgal
Copy link
Copy Markdown
Member

@Th0rgal Th0rgal commented May 14, 2026

Summary

  • add a linked PoseidonT4 boundary alongside the existing PoseidonT3 boundary
  • route hashNote through poseidonT4(npk, uint160(token), amount) instead of placeholder arithmetic
  • refresh the build_green task notes to reflect the remaining manifest blockers

Verification

  • git diff --check
  • lake build Benchmark.Cases.UnlinkXyz.Pool.Compile Benchmark.Generated.UnlinkXyz.Pool.Tasks.BuildGreen

Note

Medium Risk
Updates the UnlinkPool model’s note hashing from placeholder arithmetic to a new poseidonT4 external boundary, which can affect any logic/proofs depending on commitment values. Remaining changes are task/manifest metadata updates with low runtime impact.

Overview
Models UnlinkPool.hashNote as a real cryptographic boundary call by adding a linked external poseidonT4 and routing hashNote(npk, token, amount) through poseidonT4(npk, uint160(token), amount) instead of placeholder math.

Refreshes the build_green task template and manifest notes to reflect the updated state of the translation and to track remaining unsupported-feature blockers (including new entries for proxy storage rotation and assumed cryptographic/Permit2/Groth16 axioms).

Reviewed by Cursor Bugbot for commit 7859c8d. Bugbot is set up for automated code reviews on this repo. Configure here.

@Th0rgal Th0rgal merged commit ff4e90d into main May 14, 2026
3 checks passed
@Th0rgal Th0rgal deleted the codex/unlink-pool-hashnote-poseidon-t4 branch May 14, 2026 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant