feat(ipc): runner ↔ tool IPC channel with disableCache by wan9chi · Pull Request #441 · voidzero-dev/vite-task · GitHub

Motivation

Runner-aware tools need a small, reliable way to tell the task runner when a run must not be cached. Dev-server style workflows are the clearest example: they may run successfully but intentionally do not produce a deterministic cacheable result. This PR lands the IPC channel with the one cache-affecting verb that is safe on its own: disableCache().

The IPC internals are deliberately minimal: protocol, sync client, and server ship only the disableCache verb. The Node addon, however, exposes the full client surface from day one — ignoreInput/ignoreOutput/getEnv/getEnvs exist as documented no-ops — because the npm-published JS wrapper calls these methods unconditionally, and a stable addon surface lets real tools (vite) run end-to-end at every PR in this stack. Each no-op becomes a real IPC request in the PR that adds the runner logic consuming it: env verbs in #430, ignore verbs in #431.

Review focus

disableCache() is the only IPC verb and the only semantic cache behavior in this PR.
The channel (server lifecycle, addon handoff via VP_RUN_NODE_CLIENT_PATH, drain semantics) is the durable infrastructure the follow-ups build on.
E2E: the direct-client disable_cache fixture, plus real vite dev disabling caching end-to-end (vite's env/ignore calls hit the no-ops and fall back gracefully).

Verification

cargo fmt --check
cargo shear --deny-warnings
cargo check --all-features
cargo clippy --all-features -- -D warnings
cargo test
cargo test -p vite_task_bin --test e2e_snapshots -- --ignored

Draft while the stack is under review.

🤖 Generated with Claude Code

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	cargo/napi@3.9.0
	cargo/napi-build@2.3.2
	cargo/napi-derive@3.5.6

View full report

Add the runner-aware IPC protocol, embedded Node client addon, execution-time server lifecycle, and disableCache handling. The protocol ships with the single disableCache verb; getEnv/getEnvs land with env tracking (#430) and ignoreInput/ignoreOutput with auto output tracking (#431), each alongside the runner logic that consumes them. Co-Authored-By: OpenAI <codex@openai.com>

Merge activity

Jun 12, 2:09 PM UTC: A user started a stack merge that includes this pull request via Graphite.
Jun 12, 2:09 PM UTC: @wan9chi merged this pull request with Graphite.

Serve real env values over the IPC channel (#441) and make them part of cache correctness: - The server resolves `getEnv`/`getEnvs` against an env map and records the served values and glob match-sets. - The post-run fingerprint stores tracked envs and glob match-sets; cache lookup re-validates them and reports the changed env by name (wording matches manual `env` config misses). - `CACHE_SCHEMA_VERSION` bumps to 14: `PostRunFingerprint` gains fields, so the serialized cache value layout changes and must not share a schema directory with binaries built before this change. - `ignoreInput`/`ignoreOutput` stay accepted-but-no-op — they land with auto output tracking. - Auto output restoration stays disabled: `output: None` resolves to a disabled glob config (matching prior behavior), so a cache hit replays recorded output without restoring written files. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3277d1cfb1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Enable IPC for explicit-input cached tasks

When a cached task configures input without { "auto": true } (for example to avoid fspy and use explicit globs), this gate never starts the IPC server, and injected_envs() therefore also omits VP_RUN_NODE_CLIENT_PATH. The JS client treats that as outside the runner and disableCache() becomes a no-op, so dev/watch tools that explicitly opt out are still cached and replayed on the next run. The IPC channel needs to be available for all cacheable executions, not only auto-input tracking.

Useful? React with 👍 / 👎.

wan9chi mentioned this pull request Jun 11, 2026

fix(plan): extend the command's env context with prefix envs before interpreting it #440

Merged

wan9chi force-pushed the codex/ipc-disable-cache-core branch from c248af8 to 0c0e509 Compare June 11, 2026 16:09

This was referenced Jun 11, 2026

fix(cache): provide runner IPC for explicit-input tasks #430

Merged

feat(cache): honor runner ignored inputs #431

Merged

wan9chi force-pushed the codex/ipc-disable-cache-core branch from 0c0e509 to ba56768 Compare June 11, 2026 16:18

wan9chi force-pushed the codex/plan-prefix-env-context branch from 5a198fe to 8771d84 Compare June 11, 2026 16:46

wan9chi force-pushed the codex/ipc-disable-cache-core branch from ba56768 to 2ed4f28 Compare June 11, 2026 16:46

Base automatically changed from codex/plan-prefix-env-context to main June 11, 2026 17:02

wan9chi force-pushed the codex/ipc-disable-cache-core branch from 2ed4f28 to e04c5b9 Compare June 11, 2026 17:06

wan9chi mentioned this pull request Jun 11, 2026

refactor(execute): single exit paths for pipe/wait and cache update #442

Merged

wan9chi changed the base branch from main to graphite-base/441 June 11, 2026 17:44

wan9chi force-pushed the codex/ipc-disable-cache-core branch from e04c5b9 to 8cbc27b Compare June 11, 2026 17:44

wan9chi changed the base branch from graphite-base/441 to execute-spawn-exit-paths June 11, 2026 17:44

wan9chi force-pushed the codex/ipc-disable-cache-core branch 2 times, most recently from b11487a to c32ed5a Compare June 11, 2026 22:46

wan9chi force-pushed the execute-spawn-exit-paths branch 2 times, most recently from b886a13 to c9a46fd Compare June 12, 2026 01:12

wan9chi force-pushed the codex/ipc-disable-cache-core branch 2 times, most recently from 466f08c to 3c75926 Compare June 12, 2026 01:35

wan9chi force-pushed the execute-spawn-exit-paths branch from c9a46fd to a100974 Compare June 12, 2026 01:35

Base automatically changed from execute-spawn-exit-paths to main June 12, 2026 01:40

wan9chi force-pushed the codex/ipc-disable-cache-core branch 4 times, most recently from 87978be to 5babe6d Compare June 12, 2026 10:34

wan9chi force-pushed the codex/ipc-disable-cache-core branch from 5babe6d to 3277d1c Compare June 12, 2026 10:54

wan9chi marked this pull request as ready for review June 12, 2026 14:09

wan9chi merged commit 5f05678 into main Jun 12, 2026
18 checks passed

wan9chi deleted the codex/ipc-disable-cache-core branch June 12, 2026 14:09

chatgpt-codex-connector Bot reviewed Jun 12, 2026

View reviewed changes

-                      // Resolve input negative globs for fspy path filtering (already
-                      // workspace-root-relative).
-                      let fspy_negatives = if metadata.input_config.includes_auto {
+                      let tracking = if metadata.input_config.includes_auto {

feat(ipc): runner ↔ tool IPC channel with disableCache by wan9chi · Pull Request #441 · voidzero-dev/vite-task · GitHub

Uh oh!

feat(ipc): runner ↔ tool IPC channel with disableCache - #441

feat(ipc): runner ↔ tool IPC channel with disableCache#441
wan9chi merged 1 commit into
mainfrom
codex/ipc-disable-cache-core

wan9chi commented Jun 11, 2026 •
edited
Loading

Uh oh!

Uh oh!

wan9chi commented Jun 11, 2026 •
edited
Loading

Uh oh!

Uh oh!

socket-security Bot commented Jun 11, 2026 •
edited
Loading

Uh oh!

Uh oh!

wan9chi commented Jun 12, 2026 •
edited
Loading

Uh oh!

Uh oh!

Uh oh!

chatgpt-codex-connector Bot left a comment

Uh oh!

chatgpt-codex-connector Bot Jun 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

feat(ipc): runner ↔ tool IPC channel with disableCache by wan9chi · Pull Request #441 · voidzero-dev/vite-task · GitHub

Uh oh!

feat(ipc): runner ↔ tool IPC channel with disableCache - #441

feat(ipc): runner ↔ tool IPC channel with disableCache#441wan9chi merged 1 commit intomainvoidzero-dev/vite-task:mainfrom codex/ipc-disable-cache-corevoidzero-dev/vite-task:codex/ipc-disable-cache-coreCopy head branch name to clipboard

Conversation

wan9chi commented Jun 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Motivation

Review focus

Verification

Uh oh!

wan9chi commented Jun 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

socket-security Bot commented Jun 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

wan9chi commented Jun 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector Bot Jun 12, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

feat(ipc): runner ↔ tool IPC channel with disableCache#441
wan9chi merged 1 commit into
mainfrom
codex/ipc-disable-cache-core

wan9chi commented Jun 11, 2026 •
edited
Loading

wan9chi commented Jun 11, 2026 •
edited
Loading

socket-security Bot commented Jun 11, 2026 •
edited
Loading

wan9chi commented Jun 12, 2026 •
edited
Loading