Skip to content

feat(check): remove the claude -p executor and --executor flag#155

Merged
RobbieMcKinstry merged 2 commits into
trunkfrom
remove-claude-p-executor
Jul 3, 2026
Merged

feat(check): remove the claude -p executor and --executor flag#155
RobbieMcKinstry merged 2 commits into
trunkfrom
remove-claude-p-executor

Conversation

@RobbieMcKinstry

Copy link
Copy Markdown
Contributor

The cersei in-process executor is strictly better and has been the
default; the shell-out fallback (and the ability to select between
executors) is no longer needed.

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com

RobbieMcKinstry commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

@RobbieMcKinstry RobbieMcKinstry force-pushed the remove-claude-p-executor branch from e4ba795 to 7d04fdf Compare July 2, 2026 21:29
Base automatically changed from checks-fireworks-and-model-display to trunk July 2, 2026 21:40
@RobbieMcKinstry RobbieMcKinstry force-pushed the remove-claude-p-executor branch from 7d04fdf to 2cad142 Compare July 2, 2026 21:43
RobbieMcKinstry and others added 2 commits July 3, 2026 15:46
… TUI

Fireworks exposes an Anthropic-compatible Messages endpoint, so it slots
into the existing `anthropic` provider via a base-URL override rather than
needing its own ProviderKind; add its model ID to the allowlist and document
the MultiTool.toml setup. Also surface the model in the presenter's live
header (inline TUI and stderr heartbeat) so a run against a non-default
provider/model is visibly distinguishable from the default.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The cersei in-process executor is strictly better and has been the
default; the shell-out fallback (and the ability to select between
executors) is no longer needed.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@RobbieMcKinstry RobbieMcKinstry force-pushed the remove-claude-p-executor branch from 2cad142 to e1f54ef Compare July 3, 2026 19:46
@RobbieMcKinstry RobbieMcKinstry enabled auto-merge July 3, 2026 19:46
@RobbieMcKinstry RobbieMcKinstry added this pull request to the merge queue Jul 3, 2026
Merged via the queue into trunk with commit 02228a4 Jul 3, 2026
10 checks passed
@RobbieMcKinstry RobbieMcKinstry deleted the remove-claude-p-executor branch July 3, 2026 19:57
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