Skip to content

feat(omo-senpi): mirror ulw mutation skills - #610

Open
realsigridjin wants to merge 1 commit into
code-yeongyu:mainfrom
realsigridjin:feat/omo-senpi-ulw-mutation-mirror-clean
Open

feat(omo-senpi): mirror ulw mutation skills#610
realsigridjin wants to merge 1 commit into
code-yeongyu:mainfrom
realsigridjin:feat/omo-senpi-ulw-mutation-mirror-clean

Conversation

@realsigridjin

@realsigridjin realsigridjin commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Mirror exactly the eight requested packages/omo-senpi paths into native Senpi.
  • Adapt the two copied test entrypoints to Senpi's existing Vitest and Node URL-path conventions.
  • Supersedes feat(omo-senpi): mirror ulw mutation skills #608, whose branch unintentionally inherited two unrelated scripts/check-upstream-release* changes.

Validation

  • npm run check passes.
  • vitest run packages/coding-agent/test/mcp/catalog-cache.test.ts passes (6 tests).
  • The copied OMO fixture tests still require the unmirrored OMO plugin/shared-skill tree (6 pass, 7 fixture-dependent failures); native CI does not register this source mirror as a workspace package.

Scope

Exactly eight files under packages/omo-senpi/; no runtime registration is introduced.


Summary by cubic

Mirrors the ULW loop and mutation skills into packages/omo-senpi, adds a sync and QA harness, and enforces a contract to keep Senpi’s skills in lockstep with OMO, with no runtime registration changes.

  • New Features
    • Added skills/ulw-loop and skills/ulw-mutation-test (with workflow docs and references).
    • Added plugin/scripts/sync-skills.mjs to mirror and stage skills for Senpi.
    • Added scripts/qa/ulw-prompts-e2e.mjs for ULW directive prompt E2E checks.
    • Added tests to verify skill frontmatter and sync integrity: src/components/ulw-loop/mutation-skill-contract.test.ts, src/skills-sync.test.ts.
    • Added AGENTS.md with adapter and component overview.

Written for commit 8609ba3. Summary will update on new commits.

Review in cubic

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