Skip to content

Add a minimal run verb that drives the engine from a prompt#237

Merged
Antawari merged 2 commits into
mainfrom
cat0629/bonfire-run
Jun 29, 2026
Merged

Add a minimal run verb that drives the engine from a prompt#237
Antawari merged 2 commits into
mainfrom
cat0629/bonfire-run

Conversation

@Antawari

Copy link
Copy Markdown
Contributor

What

Adds a small bonfire run PROMPT verb: prompt → plan → engine.run, wired to the existing PipelineEngine.

  • Selects the default build workflow from the registry, stamps the prompt as the plan's task_description, runs the engine, and renders the typed PipelineResult (success + cost, or the typed failure as stage/gate/error).
  • The engine is constructed behind an injected factory, so the unit test drives the full path with a fake backend and no network.
  • --workflow / --budget options; exits 0 (success) / 1 (failure) / 2 (unknown workflow).

This is the minimal driver only. The fuller artifact-contract verb (cost/session log, a named working branch, a verdict document) is a deliberate later expansion.

Tests

tests/unit/test_run_command.py — happy path, failure path, prompt-reaches-task_description, unknown-workflow; all network-free via an injected fake backend.

Local sanity: ruff check + ruff format --check clean, protocol-doc citations 0 drift, full suite 5497 passed. (2 unrelated --version tests fail only in the local venv, which lacks bonfire dist-metadata so __version__ falls back to 0.1.0a2; CI's fresh install resolves 1.0.1 and they pass — independent of this change.)

🤖 Generated with Claude Code

Antawari and others added 2 commits June 29, 2026 12:30
A small driver: take a prompt, select the default build workflow, stamp
the prompt as the plan's task description, run the existing pipeline
engine, and render the typed result (success + cost, or the typed
failure). The engine is built behind an injected factory so the unit
test exercises the whole path with a fake backend and no network.

This is the minimal driver only. The fuller artifact-contract verb
(cost/session log, a named working branch, a verdict document) is a
deliberate later expansion, not part of this change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The new unit test draws against the frozen tests/unit package total;
declare it (the established pattern for new test coverage) so the shared
budget gate stays green without re-baselining the package.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Antawari Antawari merged commit b2e1416 into main Jun 29, 2026
4 checks passed
@Antawari Antawari deleted the cat0629/bonfire-run branch June 29, 2026 20:16
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