Skip to content

Allow i report to accept piped natural-language input#25

Merged
CoreyRDean merged 1 commit into
mainfrom
ux/report-stdin-input
May 9, 2026
Merged

Allow i report to accept piped natural-language input#25
CoreyRDean merged 1 commit into
mainfrom
ux/report-stdin-input

Conversation

@CoreyRDean
Copy link
Copy Markdown
Owner

Non-technical summary

i report can now take natural-language input from stdin as well as argv, so report drafting works in Unix-style composed flows like i report < notes.txt and echo 'more detail' | i report 'first detail'.

This matters now because the repo explicitly treats composability as a core product promise, and i report was breaking that promise by silently ignoring piped text.

After this change, the command behaves more like the rest of the tool: it accepts natural-language input from the shell surface the user gives it instead of forcing everything onto the command line.

Technical summary

  • read non-TTY stdin in cmdReport using the same bounded pipe-reading helper already used by natural-language mode
  • merge argv text first, then append trimmed stdin content, so the final report prompt preserves the ordering requested in issue Add support for stdin input in i report #21
  • keep the existing no-input usage path intact when both argv and stdin are empty
  • add focused unit coverage for args-only, stdin-only, args-plus-stdin, and blank-stdin cases
  • update the v1 spec and README to document the new i report stdin behavior
  • no breaking changes

Additional notes

Trade-off: this change only fixes i report input assembly; it does not change the broader interactive confirmation or GitHub-auth behavior of the subcommand.

Intentionally deferred: broader i report UX policy cleanup and any stdin support changes for other subcommands.

Remaining gap: i report is now composable at the input boundary, but the rest of the subcommand still has separate follow-up opportunities around dry-run semantics and non-interactive confirmation behavior.

Closes #21

@CoreyRDean CoreyRDean marked this pull request as ready for review May 9, 2026 17:16
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@CoreyRDean CoreyRDean merged commit 5d2858d into main May 9, 2026
8 checks passed
@CoreyRDean CoreyRDean deleted the ux/report-stdin-input branch May 9, 2026 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for stdin input in i report

1 participant