Skip to content

Return a read operation's output from operations propose - #27

Merged
shipmill-romamo[bot] merged 1 commit into
mainfrom
feat/read-operation-output
Oct 8, 2026
Merged

shipmill-romamo[bot] merged 1 commit into
mainfrom
feat/read-operation-output

Conversation

@romamo

@romamo romamo commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

Closes #24

What changes

  • operations propose returns a read operation's play output in data.output; the .diff file stays the record
  • The output is the hosts' text, so it is declared external and data carries _trusted: false
  • A proposed change answers output: null
  • The read operation's MCP tool no longer suggests decisions approve --yes for a run that needs no approval

Why

Found in a spike where Nemotron 3 Super diagnosed a real disk-full incident through a declared catalog: read operations were the agent's only evidence, and it could not read them without opening files.

Checks

  • uv run pytest: 365 passed, two new CLI tests (read output returned and untrusted; no output for a proposed change)
  • uv run ruff check ., uv run mypy: clean

🤖 Generated with Claude Code

https://claude.ai/code/session_01APTdZZw8Jw9PQF4TD55wMv

A read operation runs for real when it is proposed, but its play output
lived only in the decision's .diff file, so an agent on the CLI or MCP
could not read what it had just run. data.output now carries it, marked
untrusted, and the read tool stops suggesting an approval.

Closes #24

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01APTdZZw8Jw9PQF4TD55wMv
@shipmill-romamo
shipmill-romamo Bot merged commit 59abe10 into main Oct 8, 2026
1 check passed
@shipmill-romamo
shipmill-romamo Bot deleted the feat/read-operation-output branch October 8, 2026 16:11
@shipmill-romamo shipmill-romamo Bot added this to the 0.6.0 milestone Oct 8, 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.

operations propose: a read operation's output is not returned, only the path to its .diff

1 participant