[Bobowski]

Intent

Remove the duplicate code of the status report and of the compose runner. Source: the user, in the Yard audit. This pull request stacks on #49.

Found

• status.report and status.app_report each ran ps -a, stats, and the crash inspect, and each built the rows of an app.
• Each status call sent one Caddy /adapt request for each app, through sites.list_hosts. yard status asked Caddy about every site file every time.
• _row walked the data folder of an app once for each container of that app.
• compose.py built its own Podman argv, its own --remote flag, and its own copy of os.environ (_podman, _process_env, _run). Podman holds the same logic. _process_env set CONTAINER_HOST only when it was missing. The yard container and the apply image always set it (ops/compose.yml, ops/Containerfile), so the function changed nothing in production.

Changed

• status._look asks Podman once: the containers, the samples, and the crashes. report and app_report both use it and both use _app_rows. _app_rows reads the tip, the deploy record, and the data size of the app once.
• _row takes the data size and no Config.
• sites.list_hosts keeps the hosts of each site file in Yard.site_hosts, keyed by the (mtime_ns, size) of the file. Caddy adapts a file again only when the file changes.
• Podman.stream(*args, on_line) runs a command with no timeout and streams each line. compose runs rm, run, logs -f, and build through it. _process_env and _run are gone.

Not done

• I did not pass yard.podman into compose. compose makes a Podman from the config, so the signatures of compose.apply and of the test fakes stay the same.

Proof

$ just check
...
500 passed in 61.46s

test_list_hosts_adapts_each_file_once shows one /adapt for two reads, then a new adapt after the file changes. test_stream_calls_on_line_in_order checks the line stream and the exit code.

Questions

• A missing podman binary in a compose call now raises the code podman in place of compose. Nothing reads that code.

🤖 Generated with Claude Code