Skip to content

feat(skills): document live main-plan selection - #253

Closed
rayyan224 wants to merge 1 commit into
rayyanalam/skills-08-add-plan-based-adr-writerfrom
rayyanalam/skills-09-document-live-main-plan-selection
Closed

rayyan224 wants to merge 1 commit into
rayyanalam/skills-08-add-plan-based-adr-writerfrom
rayyanalam/skills-09-document-live-main-plan-selection

Conversation

@rayyan224

Copy link
Copy Markdown
Collaborator

Summary

Spell out how the architect sees council candidates, selects, overrides or rejects them, and how the confirmed main plan and alternatives are handed to the ADR writer.

Part 9 of a stack that adds design skills under .claude/skills/. Stacked on rayyanalam/skills-08-add-plan-based-adr-writer; review and merge bottom-up.

Changes

 .claude/skills/design-council/SKILL.md | 65 ++++++++++++++++++++++++++++------
 1 file changed, 55 insertions(+), 10 deletions(-)

Spell out how the architect sees council candidates, selects, overrides or rejects them, and how the confirmed main plan and alternatives are handed to the ADR writer.
@github-actions

Copy link
Copy Markdown

Interface Coverage

✅ All interface functions have test coverage.

@github-actions

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% 99.68% 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%
🟢 MockBaseTime.sol 100.00% 100.00% 100.00% 100.00%
🟡 MockPolicyRegistry.sol 100.00% 99.58% 97.78% 100.00%
🟢 MockPolicyRegistryStorage.sol 100.00% 100.00% 100.00% 100.00%
Total 97.14% 97.51% 98.21% 97.07%

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

@github-actions

Copy link
Copy Markdown

❌ Fork tests did not complete

No completed, non-empty suite was recorded. Check the workflow logs for build, setup, or execution errors.

@rayyan224
rayyan224 deleted the branch rayyanalam/skills-08-add-plan-based-adr-writer October 23, 2026 20:01
@rayyan224 rayyan224 closed this Oct 23, 2026
@rayyan224
rayyan224 deleted the rayyanalam/skills-09-document-live-main-plan-selection branch October 23, 2026 20:02
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