Skip to content

Harden design skill routing and token contracts - #9

Open
mosobande wants to merge 1 commit into
orifrom
fix/design-skill-system-cleanup
Open

Harden design skill routing and token contracts#9
mosobande wants to merge 1 commit into
orifrom
fix/design-skill-system-cleanup

Conversation

@mosobande

@mosobande mosobande commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

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

  • Parent: PR Harden skill contracts and portfolio integrity #22, feature/lightweight-skill-contracts, exact base 688810c1dfa8cb40b9af647a493d667edf7e3522.
  • Child: this PR, fix/design-skill-system-cleanup, exact head e3aa5203d6f589cfa0929ddd84d04a5e258ac59a.
  • Both branches remain one-commit stacked candidates and the audit-plan artifact is absent from their reachable history.
  • Merge PR Harden skill contracts and portfolio integrity #22 to ori first. Then retarget this PR to ori and reverify the unchanged design candidate.

What changed

Lightweight design routing

  • Make apere a lightweight design router for broad, ambiguous, or multi-deliverable visual work.
  • Route focused requests directly to their specialist.
  • Hand integrated multi-artifact production to alaga after routing.

Design ownership

  • Move logo exploration, corporate-identity resources, custom-icon visual language, and bundled search resources from apere to brand.
  • Keep product/UI icon implementation with asa-oju-ibanisoro.
  • Add social-graphics as a lightweight public skill for feed posts, carousels, stories, templates, and campaign variants.
  • Keep banner-design focused on banner-like display surfaces.
  • Keep amoye-ui-ux as owner of evidence-backed visual/UX direction and optional persisted MASTER/page records.
  • Keep eto-apere as owner of canonical design tokens, component specifications, generated configuration, validation, and token migrations.

Existing design hardening retained

  • Preserve token-alias, token-validation, slide-contract, Tailwind, shadcn, and Brand token-writer cleanup.
  • Preserve shell-free Brand color extraction and regression coverage.
  • Keep slide-production internals with slides.
  • Reconcile Alarina, README, plugin catalog, and the design changeset without removing parent routes.

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 duplicate VERSION file.

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_ACCEPT for the design ownership and routing candidate; the release simplification is owned and reviewed in the parent.

Review focus

  1. apere as a lightweight router rather than an end-to-end producer;
  2. identity-resource ownership under brand and UI-icon implementation under asa-oju-ibanisoro;
  3. social-graphics versus banner-design;
  4. Amoye persisted visual direction versus Eto Apere implementation-token ownership;
  5. preservation of existing token and slide hardening while inheriting the parent release model.

Contributor: @mosobande

@mosobande mosobande added bug Something isn't working documentation Improvements or additions to documentation labels Aug 16, 2026
@mosobande
mosobande force-pushed the fix/design-skill-system-cleanup branch from 4bc6448 to 5d9a02b Compare August 17, 2026 14:45
@mosobande mosobande changed the title Harden design skill routing and token contracts Narrow Alarina routing after the portfolio audit Aug 17, 2026
@mosobande
mosobande changed the base branch from ori to feature/audit-usage August 17, 2026 14:46
@mosobande
mosobande force-pushed the feature/audit-usage branch from 842d810 to b194d4a Compare August 17, 2026 14:52
@mosobande
mosobande force-pushed the fix/design-skill-system-cleanup branch from 5d9a02b to 7fae05c Compare August 17, 2026 14:53
@mosobande mosobande changed the title Narrow Alarina routing after the portfolio audit Harden design skill routing and token contracts Aug 17, 2026
@mosobande
mosobande force-pushed the feature/audit-usage branch from b194d4a to b67219b Compare August 17, 2026 15:14
@mosobande
mosobande force-pushed the fix/design-skill-system-cleanup branch from 7fae05c to 0d4cd7b Compare August 17, 2026 15:15
@mosobande
mosobande marked this pull request as draft August 17, 2026 22:10
@mosobande
mosobande changed the base branch from feature/audit-usage to ori August 17, 2026 22:12
@mosobande
mosobande marked this pull request as ready for review August 17, 2026 22:13
@mosobande
mosobande requested review from atunwo and removed request for atunwo August 22, 2026 11:38

@atunwo atunwo Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 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."}}

@mosobande
mosobande requested review from atunwo and removed request for atunwo August 22, 2026 11:54

@atunwo atunwo Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Alátùńwò AI review failed — I couldn't complete this review.

Please re-request a review to try again.

Error detail
The operation was aborted

@mosobande
mosobande requested review from atunwo and removed request for atunwo August 22, 2026 12:04

@atunwo atunwo Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Alátùńwò AI review failed — I couldn't complete this review.

Please re-request a review to try again.

Error detail
The operation was aborted

@mosobande
mosobande requested review from atunwo and removed request for atunwo August 22, 2026 12:19

@atunwo atunwo Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 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);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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).

@atunwo
atunwo Bot removed the request for review from atunwo August 22, 2026 12:21
@mosobande
mosobande changed the base branch from ori to feature/lightweight-skill-contracts August 22, 2026 17:07
@mosobande
mosobande force-pushed the feature/lightweight-skill-contracts branch from 92c5573 to 4bf4d6e Compare August 22, 2026 17:28
@mosobande
mosobande force-pushed the fix/design-skill-system-cleanup branch from 6112755 to d1f485a Compare August 22, 2026 17:28
@mosobande
mosobande force-pushed the feature/lightweight-skill-contracts branch from 4bf4d6e to ff421f3 Compare August 22, 2026 17:29
@mosobande
mosobande force-pushed the fix/design-skill-system-cleanup branch 3 times, most recently from 7bcbb0d to 1ec61c3 Compare August 22, 2026 17:30
@mosobande
mosobande force-pushed the feature/lightweight-skill-contracts branch from 5e9c26e to 688810c Compare August 22, 2026 18:03
@mosobande
mosobande force-pushed the fix/design-skill-system-cleanup branch from 1ec61c3 to 6267ff2 Compare August 22, 2026 18:04
@mosobande
mosobande force-pushed the fix/design-skill-system-cleanup branch from db38497 to e3aa520 Compare August 22, 2026 18:05
@mosobande
mosobande changed the base branch from feature/lightweight-skill-contracts to ori August 23, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant