fix: process all documents from stdin, not just the first

[MPV]

What

Route the stdin path through the same document-splitting the file path already uses, so a multi-document stream piped into kir - yields images from every document, not just the first.

Before → After

Given a two-document manifest on stdin (a Deployment, then a CronJob):

$ kir - < deploy-and-cronjob.yaml   # or: kubectl get deploy,cronjob -o yaml | kir -

# before
nginx:1.27
# ^ only the first document is read; backup:2.1 from the CronJob is silently dropped

# after
nginx:1.27
backup:2.1

Exit code is 0 in both cases — the failure was silent, which is what made it dangerous.

Why

ProcessStdin handed the whole stdin buffer to yamlparser.ProcessData, whose decoder reads only the first YAML document. Files already went through a splitting path, so kir file.yaml handled multiple documents correctly while … | kir - did not — the same input via two routes gave different results. Stdin now goes through the shared processDocuments path, so both routes behave identically.

Tests

Adds a TestCLI/StdinMultiDoc golden that pipes a two-document stream through cmd.Run — the seam where the bug lived. The pre-existing TestCLI/Stdin case pipes a single document, so it passed with or without this fix; the new golden emits both images only with the fix (verified: it fails on master, printing just nginx:1.27).

Release

fix: → patch. This is silent data loss in an existing path (stdin already worked; it just truncated), not a new capability — contrast #54, which makes the tool tolerate input it previously rejected outright.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Pc6NAURAqjU4LYJx93tgSC