Skip to content

refactor(cli): extract startup and dispatch boundaries - #83

Draft
Maximooch wants to merge 2 commits into
fix/multi-agent-runtimefrom
codex/cli-acbra-startup
Draft

Maximooch wants to merge 2 commits into
fix/multi-agent-runtimefrom
codex/cli-acbra-startup

Conversation

@Maximooch

@Maximooch Maximooch commented Aug 8, 2026 •

Copy link
Copy Markdown
Owner

What changed

  • extracts model/runtime projection into penguin/cli/model_runtime.py
  • extracts environment normalization and atomic dependency composition
  • introduces structured dispatch/output services and deterministic tests
  • preserves explicit reasoning enablement, effort, and token-budget provenance

Why

cli.py reconstructed model intent and published startup globals while dependencies were still being built. Model overrides could inherit source-model capability metadata, and startup behavior was difficult to test without real providers.

Impact

CLI startup now constructs a complete result before publishing compatibility globals. Target-model capabilities are resolved from the target, and metadata-only help paths can remain provider-free.

Validation

  • included unit/contract tests for bootstrap, environment, model projection, dispatch, command services, and output policy
  • final stacked CLI suite: 134 passed
  • installed wheel help smoke passed

Stack base: #82 (fix/multi-agent-runtime).

CI note

The two Link-provider jobs and Python 3.9 goal job also fail on base PR #82 with the same unrelated RUF043 test-pattern and Python 3.9 union-annotation errors. Python 3.12 goal, TUI, docs, and deployment checks pass.

@vercel

vercel Bot commented Aug 8, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
penguin Ready Ready Preview Aug 8, 2026 12:09am

@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 3b07318a-2ece-4eea-b30b-1322d6056119

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

This branch was successfully deployed

1 active deployment
Preview — 1b9ad97a Deployed Aug 8, 2026 by vercel[bot]
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