Skip to content

Fix missing marketplace hook CLI payloads#112

Closed
YoungsPlace wants to merge 1 commit into
code-yeongyu:mainfrom
YoungsPlace:fix-marketplace-hook-cli-payloads
Closed

Fix missing marketplace hook CLI payloads#112
YoungsPlace wants to merge 1 commit into
code-yeongyu:mainfrom
YoungsPlace:fix-marketplace-hook-cli-payloads

Conversation

@YoungsPlace

@YoungsPlace YoungsPlace commented Jul 5, 2026

Copy link
Copy Markdown

Problem

The LazyCodex marketplace payload declared hook commands for components/start-work-continuation/dist/cli.js and components/ulw-loop/dist/cli.js, but those generated CLI files were absent from the checked-in payload. This matches the missing-file failure reported in #108 and can leave installs with dangling hook/bin targets after marketplace update or repair.

Fixes #108.

Changes

  • Add the generated dist/cli.js payloads for start-work-continuation and ulw-loop.
  • Stop ignoring those component dist/ directories so the generated payload files can be tracked like the other bundled component CLIs.
  • Add an aggregate hook regression test that fails when any ${PLUGIN_ROOT}/... command target is missing.

Verification

  • node --test plugins/omo/test/aggregate-hooks.test.mjs
  • node --check components/start-work-continuation/dist/cli.js && node --check components/ulw-loop/dist/cli.js from plugins/omo
  • CLI smoke checks for components/ulw-loop/dist/cli.js help and expected start-work-continuation usage failure
  • npm test

Summary by cubic

Include the missing CLI payloads for marketplace hooks so installs no longer have dangling hook/bin targets. Adds a guardrail test to catch missing command targets; fixes #108.

  • Bug Fixes
    • Add generated dist/cli.js for start-work-continuation and ulw-loop.
    • Track these dist/ directories (remove .gitignore ignores) so CLIs are published.
    • Add aggregate hook test that fails when any ${PLUGIN_ROOT}/... target is missing.

Written for commit 3e02011. Summary will update on new commits.

Review in cubic

@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Thanks for the PR! Changes to LazyCodex land through oh-my-openagent — could you open this against omo-codex over there instead? PRs in this repository can't be merged.

If you're working with a coding agent, prompt it like this:

I opened #112 ("Fix missing marketplace hook CLI payloads") against code-yeongyu/lazycodex, but changes there land through code-yeongyu/oh-my-openagent, where the Codex implementation lives under packages/omo-codex. Read my original PR, then open an equivalent PR against code-yeongyu/oh-my-openagent. In the new PR body, link the original PR (#112), explain that it was first opened against lazycodex, and describe in detail what that PR changed and why.

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.

Marketplace omo@4.15.1 still omits hook CLIs after lazycodex#92 / oh-my-openagent#5768

1 participant