Skip to content

Log 'skipped, inputs unchanged' for toplevel tasks#14

Merged
zyla merged 1 commit intomainfrom
add-toplevel-skip-message
Mar 10, 2026
Merged

Log 'skipped, inputs unchanged' for toplevel tasks#14
zyla merged 1 commit intomainfrom
add-toplevel-skip-message

Conversation

@zyla
Copy link
Collaborator

@zyla zyla commented Mar 9, 2026

Motivation

Currently when all tasks are skipped, not output is emitted.
This frequently confuses agents, they "think" something is wrong with the caching system.

Summary

  • When a toplevel task has a local cache hit (no input changes), log skipped, inputs unchanged instead of exiting silently
  • Only toplevel tasks emit this message — subtasks remain quiet to avoid noise
  • Guarded by isNothing m_hashToSave so remote cache hits and prime-cache mode are unaffected

Test plan

  • New toplevel-skip-message test: toplevel task logs the skip message on second run
  • New subtask-skip-no-message test: subtask does NOT log the skip message
  • Updated 4 existing golden files that now include the new message
  • All 34 non-slow, non-S3 tests pass

🤖 Generated with Claude Code

When a toplevel task detects no input changes (local cache hit), it
previously exited silently. This adds an info log message so agents
and users can distinguish a cache hit from an empty/broken run.
The message only appears for toplevel tasks to avoid noise from subtasks.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@zyla zyla merged commit 068082c into main Mar 10, 2026
2 checks passed
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