Skip to content

fix(cli): correct Action discovery and bundled skill workflows - #57

Merged
Glacier-Luo merged 2 commits into
mainfrom
feature/action-root-id
Oct 16, 2026
Merged

Glacier-Luo merged 2 commits into
mainfrom
feature/action-root-id

Conversation

@Glacier-Luo

Copy link
Copy Markdown
Collaborator

Single Action schema reads can hit fixed collection routes when a service ID is search, categories or services. Use the existing read-only batch route, preserve exact endpoint UUIDs and all HTTP methods, and return 404 for missing/empty results without falling back to an alias.

Align the bundled consumer/provider skills, README and help with the released backend: typed Decisions and DYNAMIC billing, HTTP/2 contract preservation, non-EVM BlockPI/Aptos JSON and BCS, stable endpoint keys and working-revision publication. Explicitly document the remaining receipt/scaffold limitations.

Validation:

  • bun install --frozen-lockfile, bun run typecheck, bun run build passed.
  • bun run test: 1430 passed, 0 failed across 94 files with Node 24.5.0/Bun 1.3.14.
  • npm pack --dry-run --ignore-scripts passed; updated skills are included and package version remains unchanged.
  • Skill validation, JSON/shell/reference checks, backend Decisions/DYNAMIC example validation and independent local review passed.
  • Node26's existing proxy fixture fails even on unmodified main because it only implements CONNECT; Node24 runs all 31 proxy modes without skips. No Node26 acceptance is claimed.

CLI publication remains paused. This PR does not change a version, tag, Release Please PR or npm publication. Runtime receipt linkage, HTTP/2 editor test calls and scaffold implementation are separate follow-ups.

Kanby-Task: KANBY-98

@Glacier-Luo

Copy link
Copy Markdown
Collaborator Author

@codex review

Please review the current head 3ab01af04493004117fd966c39206acc0716743e for this scoped Action/skills/documentation follow-up. Check remaining findings against the actual changed contract and preserve the documented publication boundaries. CLI publication remains paused.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-16T06:45:26.080522Z 3ab01af Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Chef's kiss.

Reviewed commit: 3ab01af044

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Glacier-Luo
Glacier-Luo merged commit d257169 into main Oct 16, 2026
1 check passed
This was referenced Oct 16, 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