Skip to content

feat(wallet+commons): add canonical digest library, signature guardrails, and docs#7

Merged
hooftly merged 2 commits intomainfrom
develop
Sep 9, 2025
Merged

feat(wallet+commons): add canonical digest library, signature guardrails, and docs#7
hooftly merged 2 commits intomainfrom
develop

Conversation

@hooftly
Copy link
Member

@hooftly hooftly commented Sep 9, 2025

Summary

This PR introduces an authoritative commons library for digest semantics and
attestation structure, hardens wallet signature handling with explicit constants
and renamed events, and adds guardrail tests to lock in the canonical 4417-byte
signature layout. Documentation has been updated to clearly define digest
semantics and signature packing order.

Changes

  • Introduced commons library clarifying canonical digest choice, aggregator
    attestation structure, and atomic key rotation with nonce tied to WOTS index.
  • Hardened wallet implementation with:
    • Explicit signature-length constants and documented byte offsets.
    • Renamed on-chain verification event to ForceOnChainVerifySet.
  • Added dedicated signature-layout guardrail test to enforce 4417-byte format
    and offset calculations.
  • Documented canonical digest semantics and signature packing order in README
    for developer clarity.

Motivation

These updates provide a single source of truth for digest and attestation logic,
make wallet signature handling more robust, and ensure future changes cannot
break the canonical hybrid signature format. Developer guidance is now clearer
and easier to follow.

Testing

  • forge build --sizes
  • forge test -vv
  • grep -R "ForceOnChainSet" -n .

Checklist

  • Tests added or updated
  • Docs updated
  • Lint checks pass
  • Ready for review.

… and docs

- Introduced authoritative commons library clarifying canonical digest choice,
  aggregator attestation structure, and atomic key rotation with nonce tied
  to WOTS index.
- Hardened wallet with explicit signature-length constants, documented byte
  offsets, and renamed on-chain verification event to ForceOnChainVerifySet.
- Added signature-layout guardrail test to enforce 4417-byte format and
  offset calculations.
- Documented canonical digest semantics and signature packing order in README
  for clearer developer guidance.
- Replaced the deprecated withOpacity call with withValues(alpha: 0.1) in the activity feed status badge to align with updated Flutter APIs and prevent analyzer failures
@hooftly hooftly merged commit 4c4982e into main Sep 9, 2025
2 checks passed
@hooftly hooftly deleted the develop branch September 9, 2025 18:18
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