Harden design skill routing and token contracts - #9
Open
mosobande wants to merge 1 commit into
Open
Conversation
This was referenced Aug 16, 2026
mosobande
force-pushed
the
fix/design-skill-system-cleanup
branch
from
August 17, 2026 14:45
4bc6448 to
5d9a02b
Compare
mosobande
force-pushed
the
feature/audit-usage
branch
from
August 17, 2026 14:52
842d810 to
b194d4a
Compare
mosobande
force-pushed
the
fix/design-skill-system-cleanup
branch
from
August 17, 2026 14:53
5d9a02b to
7fae05c
Compare
mosobande
force-pushed
the
feature/audit-usage
branch
from
August 17, 2026 15:14
b194d4a to
b67219b
Compare
mosobande
force-pushed
the
fix/design-skill-system-cleanup
branch
from
August 17, 2026 15:15
7fae05c to
0d4cd7b
Compare
mosobande
marked this pull request as draft
August 17, 2026 22:10
mosobande
marked this pull request as ready for review
August 17, 2026 22:13
There was a problem hiding this comment.
🤖 Alátùńwò AI review failed — I couldn't complete this review.
Please re-request a review to try again.
Error detail
{"type":"error","status":400,"error":{"type":"invalid_request_error","message":"The 'gpt-5.6-sol' model requires a newer version of Codex. Please upgrade to the latest app or CLI and try again."}}
There was a problem hiding this comment.
🤖 Alátùńwò AI review · 📄 Diff-only
Reviewed from the pull request diff only (no surrounding files).
This PR clarifies design-skill ownership and hardens token/slide validation.
- No blocking issues found.
- The slide-token reference contains one canonical-name mismatch that would produce an undefined CSS variable if copied.
| --slide-bg-gradient: var(--gradient-primary); | ||
| --slide-foreground: var(--color-primary-foreground); | ||
| --slide-foreground-muted: var(--color-gray-200); | ||
| --slide-border: var(--color-gray-300); |
There was a problem hiding this comment.
WARN — --spacing-section is not a canonical generated variable: the new architecture maps primitive.spacing.* to --space-*. Copying this slide contract will leave --slide-padding aliased to an undefined property. Change this to --slide-padding: var(--space-section); (assuming semantic.spacing.section resolves to that primitive).
mosobande
changed the base branch from
ori
to
feature/lightweight-skill-contracts
August 22, 2026 17:07
mosobande
force-pushed
the
feature/lightweight-skill-contracts
branch
from
August 22, 2026 17:28
92c5573 to
4bf4d6e
Compare
mosobande
force-pushed
the
fix/design-skill-system-cleanup
branch
from
August 22, 2026 17:28
6112755 to
d1f485a
Compare
mosobande
force-pushed
the
feature/lightweight-skill-contracts
branch
from
August 22, 2026 17:29
4bf4d6e to
ff421f3
Compare
mosobande
force-pushed
the
fix/design-skill-system-cleanup
branch
3 times, most recently
from
August 22, 2026 17:30
7bcbb0d to
1ec61c3
Compare
mosobande
force-pushed
the
feature/lightweight-skill-contracts
branch
from
August 22, 2026 18:03
5e9c26e to
688810c
Compare
mosobande
force-pushed
the
fix/design-skill-system-cleanup
branch
from
August 22, 2026 18:04
1ec61c3 to
6267ff2
Compare
mosobande
force-pushed
the
fix/design-skill-system-cleanup
branch
from
August 22, 2026 18:05
db38497 to
e3aa520
Compare
mosobande
changed the base branch from
feature/lightweight-skill-contracts
to
ori
August 23, 2026 13:04
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The design-skill system had overlapping outcome ownership, duplicated dependency procedures, stale helper claims, and token contracts that did not match slide generation. This stacked child keeps each specialist independently useful while separating lightweight design routing from artifact production and preserving PR #22's non-design contracts.
Stack
feature/lightweight-skill-contracts, exact base688810c1dfa8cb40b9af647a493d667edf7e3522.fix/design-skill-system-cleanup, exact heade3aa5203d6f589cfa0929ddd84d04a5e258ac59a.orifirst. Then retarget this PR tooriand reverify the unchanged design candidate.What changed
Lightweight design routing
aperea lightweight design router for broad, ambiguous, or multi-deliverable visual work.alagaafter routing.Design ownership
aperetobrand.asa-oju-ibanisoro.social-graphicsas a lightweight public skill for feed posts, carousels, stories, templates, and campaign variants.banner-designfocused on banner-like display surfaces.amoye-ui-uxas owner of evidence-backed visual/UX direction and optional persisted MASTER/page records.eto-apereas owner of canonical design tokens, component specifications, generated configuration, validation, and token migrations.Existing design hardening retained
slides.Release ownership inherited from parent
The child inherits PR #22's simplified release model: Changesets owns package versioning, npm regenerates
package-lock.json, README uses the stable latest-release link, and there are no custom release/catalog synchronizer scripts or duplicateVERSIONfile.Verification
The design implementation is unchanged from the previously accepted design candidate apart from inheriting the parent's release-tool simplification and stable README release link. The prior design validation passed Python compilation, JavaScript syntax, 39/39 Amoye tests, 7/7 Slides tests, and the Brand pytest suite.
Àtúnwò review remains
RECOMMEND_ACCEPTfor the design ownership and routing candidate; the release simplification is owned and reviewed in the parent.Review focus
apereas a lightweight router rather than an end-to-end producer;brandand UI-icon implementation underasa-oju-ibanisoro;social-graphicsversusbanner-design;Contributor: @mosobande