Skip to content

feat(run): support task-reported unchanged outputs - #729

Open
kdy1 wants to merge 3 commits into
voidzero-dev:mainfrom
kdy1:codex/report-unchanged
Open

kdy1 wants to merge 3 commits into
voidzero-dev:mainfrom
kdy1:codex/report-unchanged

Conversation

@kdy1

@kdy1 kdy1 commented Sep 18, 2026

Copy link
Copy Markdown

Motivation

Tools with their own incremental checks can finish successfully without changing any outputs, but the runner cannot currently distinguish that result from an ordinary execution. Add an explicit unchanged report while preserving the existing input-based cache policy: dependent and directly requested tasks still validate their own inputs and cache entries.

Related to delinoio/oss#894 and https://x.com/cnakazawa/status/2100838949073965395.

Changes

  • Add acknowledged, command-scoped reporting through Rust Client::report_unchanged(), JavaScript reportUnchanged(), and vp run --report-unchanged, including uncached commands. Reports take effect only after successful execution and remain isolated across compound and nested commands.
  • Refresh eligible cache entries with the current input fingerprint, logs, and duration while preserving the previous output archive for the same cache key. Reject incomplete IPC and damaged archives instead of treating them as unchanged or empty output.
  • Display unchanged executions separately from cache hits in normal output, detailed summaries, and --last-details, without counting their runtime as saved time. Document usage and retain compatibility with older JavaScript addons and saved summaries.

kdy1 and others added 2 commits September 18, 2026 16:29
Add acknowledged per-command reports through the Rust and JavaScript clients
and vp run --report-unchanged. Accept reports only after successful execution,
preserve existing output archives, and keep downstream input-based cache checks.

Show unchanged executions separately from cache hits and cover reporting,
command isolation, failure handling, and artifact restoration.

Co-authored-by: GPT-6 <codex@openai.com>
Co-authored-by: GPT-6 <codex@openai.com>

@kdy1 kdy1 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I used AI for this, and feel free to request any changes.

cc @cpojer

Read CARGO_BIN_EXE_vt at runtime so Windows tests use the executable
remapped from the nextest archive instead of the Linux build path.

Co-authored-by: GPT-6 <codex@openai.com>
@kdy1
kdy1 marked this pull request as ready for review September 18, 2026 08:07
@wan9chi

wan9chi commented Sep 18, 2026

Copy link
Copy Markdown
Member

@kdy1 Thanks for working on this! I'm focusing on something else right now and will look into this (and #730) once I have some availability.

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.

2 participants