feat: skip non-workload documents instead of aborting the stream

[MPV]

What

Skip non-workload documents (Service, ConfigMap, CRDs, …) instead of erroring out the whole file.

Before → After

$ kir deployment-and-service.yaml

# before
error: error processing document: unsupported kind Service
# exit 0, and the Deployment's image is lost

# after
nginx:1.27
# the Service is skipped; surrounding workloads survive

Why

Any document whose kind has no PodSpec made ProcessData return an error that aborted the file and discarded images already collected. Non-workload kinds — and kinds not registered in the scheme (CRDs) — are now skippable: no images, no error.

Golden suite: the former TestError becomes TestSkipsNonWorkloads (Service → empty stderr, exit 0), and TestMixed covers a workload + Service + workload file.

Docs

Records the resulting document-classification contract so the stderr/exit behaviour is explicit rather than implied — docs/adr/0007-document-classification.md (added to the ADR log from #76): the three tiers (workload → images, known image-less → skipped silently, unprocessable → stderr + non-zero), plus the planned warning: on stderr / exit 0 ("seen but not detected") for unrecognized CRDs (#75). The root README carries a short user-facing table, the docs/adr/README.md index gains its 0007 row, and the yamlparser.go comments point back to the ADR.

The key decision: a valid-but-image-less document is not an error and not a warning — it's expected input with nothing to report, so stdout stays images-only and the exit code stays trustworthy (also what keeps a directory full of Services from tripping the non-zero exit in #55).

Stack / Release

Based on master (rebased onto the #76 ADR backfill). feat: → minor — tolerating mixed manifests is a new capability.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Pc6NAURAqjU4LYJx93tgSC