Skip to content

Split raw API command internals#97

Closed
odyslam wants to merge 1 commit into
codex/pcl-human-output-polishfrom
codex/pcl-api-command-extraction
Closed

Split raw API command internals#97
odyslam wants to merge 1 commit into
codex/pcl-human-output-polishfrom
codex/pcl-api-command-extraction

Conversation

@odyslam
Copy link
Copy Markdown
Contributor

@odyslam odyslam commented May 23, 2026

Supersedes the closed oversized PR #90. This is stack slice 3/8.

Base: codex/pcl-human-output-polish
Head: codex/pcl-api-command-extraction

What changed:

  • Extracts raw pcl api command internals by concern.
  • Separates manifest/list/inspect/call/coverage handling instead of growing one large command path.
  • Preserves the existing raw API debugging surface while making it easier to maintain.

Review focus:

  • Mechanical module extraction and unchanged behavior.
  • Raw API command parity, especially list/inspect/call/coverage paths.

Validation:

  • make ci passed on the stack tip codex/pcl-slim-generated-client.
  • Local ./target/debug/pcl api manifest --toon smoke passed.

@odyslam
Copy link
Copy Markdown
Contributor Author

odyslam commented May 23, 2026

Superseded by #98, which now combines the raw API command extraction and workflow output foundation as review step 2/5.

@odyslam odyslam closed this May 23, 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