Skip to content

Give no value for an output a step did not report - #168

Merged
tylercschneider merged 9 commits into
mainfrom
fix/unreported-output-input
Nov 5, 2026
Merged

tylercschneider merged 9 commits into
mainfrom
fix/unreported-output-input

Conversation

@tylercschneider

Copy link
Copy Markdown
Contributor

One Sentence Summary

An admin building a flow no longer gets a wrong value passed on when a step asks for an output that an earlier step never reported.

ACs Covered

  • In a saved run, an input asking for an output its earlier step did not report gives no value.
  • A step's plain recorded value, one saved without an output name, is still given when the input names the step's only declared output or names no output.
  • A step declaring no outputs keeps giving its plain value, so saved flows keep working.

Decision Log

Which plain recorded values should an input read under a named output?

A plain value is read only under a step's single declared output or from a step declaring nothing, rather than under any output name.

Should the fix for unreported outputs cover comparisons as well as inputs?

Only inputs change, since comparisons are not given a run or the flow's document and changing them would mean changing what every route receives.

Code Example

Not relevant.

Out of Scope

  • Comparisons still read a plain value under any output name, because they are not given a run.
  • Inputs in a flow that saves no runs still take a plain value under any output name, because they have no run to look up the step's declared outputs in.

Ticket Billed Against

Not specified.

Tokens Used

  • Input: 54
  • Output: 15,973
  • Cache read: 3,425,962
  • Cache write: 45,633
  • Total: 3,487,622

Effort

  • Sessions: 0
  • Prompts: 0
  • Words typed: 0
  • Pasted blocks: 0
  • Words pasted: 0
  • Questions answered: 0
  • Interruptions: 0
  • Rejected tool calls: 0
  • Claude working time: 0m
  • Your active time: 0m

Time Spent

  • Total: 0m
  • Waiting on the owner: 0m
  • Model time: 0m
  • Worktree setup: 0m
  • Reading: 0m
  • Test-first cycles: 0m
  • Commits: 0m
  • Ticking criteria and recording decisions: 0m
  • Full-suite checks: 0m
  • Other tool calls: 0m
  • pr-scribe: 0m

Generation Metadata

Scribe: pr-scribe 9d1ee89
Rules: pr-body 9d1ee89, draft-reading 0d470c5, writing-style c3c2f61
Model: claude-sonnet-5-5, cc 2.1.312

🤖 Generated with Claude Code

tylercschneider and others added 9 commits November 5, 2026 09:50
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@tylercschneider
tylercschneider merged commit 8428d1f into main Nov 5, 2026
4 checks passed
@tylercschneider
tylercschneider deleted the fix/unreported-output-input branch November 5, 2026 15:04
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.

1 participant