Skip to content

Separate architecture planning and simplify skill ownership - #27

Merged
mosobande merged 5 commits into
orifrom
feature/separate-solution-architecture
Aug 24, 2026
Merged

Separate architecture planning and simplify skill ownership#27
mosobande merged 5 commits into
orifrom
feature/separate-solution-architecture

Conversation

@mosobande

@mosobande mosobande commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add solution-architect as the implementation-ready technical architecture owner and refocus Atọna on one exact-current initiative lifecycle plan.
  • Consolidate specialist contributions into receipts and one human-readable HTML plan instead of parallel plan state and reports.
  • Simplify the script portfolio by removing local shadcn and Tailwind wrappers, Amoye private persistence, Brand cross-owner helpers, the legacy Slides wrapper, and inert flags while retaining safety-critical provider and lifecycle mechanisms.
  • Support explicitly requested draft pull requests and merge requests in Seda while keeping new items ready by default.
  • Reconcile the simplification with the current design-skill ownership model, direct routes, package metadata, and Experimental isolation.

Compatibility

This carries a major changeset. Retired commands, flags, wrappers, and private state paths are intentionally unsupported. The source repository also stops tracking machine-local skills-lock state.

Verification

  • npm ci: passed; 0 vulnerabilities
  • Amoye unit tests: 36 passed
  • Brand and Eto tests: 4 passed
  • Slides unit tests: 7 passed
  • Seda provider tests: 20 passed
  • Wo provider tests: 35 passed
  • Amoye data validation and structured search smoke: passed
  • Brand logo and CIP search smokes: passed
  • Token generation and token-backed Slides deck smoke: passed
  • Frontmatter, agent YAML, manifest inventory, direct references, changeset status, syntax, and stale-route checks: passed
  • Changeset result: major, 3.0.0 to 4.0.0
  • Broad review: RECOMMEND_ACCEPT

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Olúwaṣèyí Ṣóbandé added 5 commits August 24, 2026 00:16
- Add the `solution-architect` Engineering skill and refocus `atona` on initiative lifecycle planning
- Support explicitly requested draft PRs/MRs while keeping new items ready by default
- Reconcile portfolio metadata, receipts, ADRs, and documentation
- Remove redundant wrappers, persistence, and cross-skill mutation
- Move deterministic behavior and safety contracts to owning skills
- Document script ownership and compatibility breaks
Preserve the accepted skill-script simplification boundaries while integrating the current design-skill ownership and token-validation hardening. Remove machine-local installer state from the source candidate.
@mosobande
mosobande requested a review from atunwo August 24, 2026 00:24

@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 separates technical architecture from lifecycle planning, simplifies design-skill tooling, and adds explicit draft PR/MR support.

  • Blocker: whole-repository candidate snapshots no longer match any Git paths.
  • Review was limited to the supplied diff because the workspace contained no repository checkout, so tests and caller behavior could not be verified directly.

Comment thread skills/engineering/alaga/scripts/snapshot-candidate.py
@mosobande
mosobande requested review from atunwo and removed request for atunwo August 24, 2026 00:34
@mosobande
mosobande merged commit 8b0f786 into ori Aug 24, 2026
4 checks passed
@github-actions github-actions Bot mentioned this pull request Aug 24, 2026
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