Skip to content

docs(changelog): expand composite policy specification - #211

Merged
rayyan224 merged 3 commits into
mainfrom
docs/expand-composite-policy-changelog
Aug 25, 2026
Merged

docs(changelog): expand composite policy specification#211
rayyan224 merged 3 commits into
mainfrom
docs/expand-composite-policy-changelog

Conversation

@rayyan224

Copy link
Copy Markdown
Collaborator

Summary

  • expand the composite-policy motivation, interface, and authorization behavior
  • document the ERC-7201 children mapping layout and shared policy ID counter
  • clarify compatibility and migration guidance for integrators

Test plan

  • Confirm the documented namespace location, offset, and packed array layout against MockPolicyRegistryStorage.sol
  • Confirm the working tree contains documentation changes only
  • CI

Made with Cursor

Clarify authorization behavior, storage layout, and migration guidance so integrators can evaluate composite-policy semantics and adoption requirements.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown

Interface Coverage

✅ All interface functions have test coverage.

@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown

📊 Forge Coverage (src/lib/)

🟡 ≥95% across all metrics — some metrics below 99%.

File Lines Stmts Branches Funcs
🟡 B20FactoryLib.sol 97.70% 98.00% 100.00% 95.00%
🔴 test/lib/ForceFeeder.sol 0.00% 0.00% 100.00% 0.00%
🔴 test/lib/PrecompileProbe.sol 0.00% 0.00% 0.00% 0.00%
🟢 MockActivationRegistry.sol 100.00% 100.00% 100.00% 100.00%
🟢 MockActivationRegistryStorage.sol 100.00% 100.00% 100.00% 100.00%
🟢 MockB20.sol 100.00% 100.00% 100.00% 100.00%
🟢 MockB20Asset.sol 100.00% 100.00% 100.00% 100.00%
🟡 MockB20Factory.sol 98.96% 99.10% 100.00% 100.00%
🟢 MockB20Stablecoin.sol 100.00% 100.00% 100.00% 100.00%
🟢 MockB20Storage.sol 100.00% 100.00% 100.00% 100.00%
🟡 MockPolicyRegistry.sol 100.00% 99.54% 97.67% 100.00%
🟢 MockPolicyRegistryStorage.sol 100.00% 100.00% 100.00% 100.00%
Total 97.07% 97.52% 98.16% 97.00%

Full report: download artifact. To browse locally: make coverage (runs forge coverage + genhtml + opens the HTML report).

Use consistent table, list, and section spacing so the specification renders uniformly.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown

✅ Fork tests: all 721 passed

base/base is fully in sync with the base-std spec.

Place the nested-policy constraint beside the authorization flow and use a descriptive heading for related behavior.

Co-authored-by: Cursor <cursoragent@cursor.com>
@rayyan224
rayyan224 requested a review from stephancill August 25, 2026 21:29
@rayyan224
rayyan224 merged commit 6bb10a4 into main Aug 25, 2026
10 checks passed
@rayyan224
rayyan224 deleted the docs/expand-composite-policy-changelog branch August 25, 2026 21:36
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