Add machine-readable (JSON) log output to all CLI tools

[Copilot]

All CLI binaries only supported human-readable log output, with no way to consume logs programmatically.

Changes

• {aptlyctl,apt2aptly,obs2aptly,aptly-latest-snapshots}/Cargo.toml: enable the json feature for tracing-subscriber
• Each binary's main.rs:
  • Add LogFormat enum (pretty | json) with clap::ValueEnum
  • Add --log-format to Opts
  • Extract subscriber initialization into a local fn init_tracing(LogFormat) helper
  • Fix initialization order to Opts::parse() → init_tracing() → color_eyre::install() so ErrorLayer is registered before color-eyre's hook runs
• aptlyctl/src/repo.rs: fix pre-existing clippy::useless_conversion warning

Usage

aptlyctl --log-format json snapshot list
obs2aptly --log-format json my-repo ./obs-packages

JSON output uses tracing-subscriber's built-in JSON formatter; pretty (the default) preserves existing behavior.

[sjoerdsimons]

@andrewshadura tagging you in as this is a fixed for an issue you filed ;)

[Copilot]

Pull request overview

Adds a new --log-format {pretty,json} CLI option across all binaries so logs can be consumed programmatically (JSON) while preserving existing human-readable output (pretty). This fits the codebase by standardizing tracing initialization across the CLI tools and enabling tracing-subscriber’s JSON formatter consistently.

Changes:

• Enable tracing-subscriber’s json feature in each binary crate.
• Add LogFormat + --log-format and refactor tracing subscriber setup into init_tracing(...) in each main.rs.
• Adjust startup order to parse CLI args before initializing tracing, then install color-eyre (plus a small clippy warning fix in aptlyctl/src/repo.rs).

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
obs2aptly/src/main.rs	Adds --log-format and initializes tracing via init_tracing(LogFormat).
obs2aptly/Cargo.toml	Enables tracing-subscriber JSON support for machine-readable logs.
aptlyctl/src/main.rs	Adds --log-format and refactors subscriber initialization into init_tracing.
aptlyctl/Cargo.toml	Enables tracing-subscriber JSON support.
aptlyctl/src/repo.rs	Fixes a pre-existing clippy warning (clippy::useless_conversion).
aptly-latest-snapshots/src/main.rs	Adds --log-format and moves tracing init after Opts::parse().
aptly-latest-snapshots/Cargo.toml	Enables tracing-subscriber JSON support.
apt2aptly/src/main.rs	Adds --log-format and refactors subscriber initialization into init_tracing.
apt2aptly/Cargo.toml	Enables tracing-subscriber JSON support.
Cargo.lock	Adds dependencies required by tracing-subscriber JSON feature (serde_json, tracing-serde, etc.).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[sjoerdsimons]

@copilot please fix the review comments and rebase on the latest main if needed

[Copilot]

@copilot please fix the review comments and rebase on the latest main if needed

Fixed in b86dab4. I addressed the review feedback, verified the branch is not behind main, and reran cargo fmt --all --check, cargo clippy --all-targets -- -D warnings, and cargo test --all-targets --all-features.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated no new comments.