Skip to content

simulate: full per-scenario dump only in CI, report file elsewhere#905

Open
theomonnom wants to merge 9 commits into
mainfrom
simulate/ci-dump-only-in-ci
Open

simulate: full per-scenario dump only in CI, report file elsewhere#905
theomonnom wants to merge 9 commits into
mainfrom
simulate/ci-dump-only-in-ci

Conversation

@theomonnom

@theomonnom theomonnom commented Jul 12, 2026

Copy link
Copy Markdown
Member

No description provided.

@u9g u9g force-pushed the simulate/ci-dump-only-in-ci branch from c8cf6f8 to f9d81d2 Compare July 13, 2026 00:21
@u9g

u9g commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Claude pushed a commit to this branch, I had claude remove that commit and make it a PR instead, hence the forcepush.

@u9g u9g left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly LGTM. We could reuse an existing codepath so I made a PR here: #906 . Otherwise, the changes look good.

The interactive branch hand-rolled the temp-file report and omitted the
broken-agent note, agent log path, and dashboard URL that the TUI's
report file includes via runReporter.Finish.
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.

3 participants