metrics trends --from/--to flags are ignored (empty query values)

[0xble]

Summary

metrics trends reads from/to through globally-colliding Viper keys, so provided flags may be dropped and sent as empty strings.

Repro

eightctl metrics trends --from 2026-02-20 --to 2026-02-21 -v

Actual

Request can be emitted with:

.../trends?from=&to=&...

Additionally, current app-api now requires tz, so missing timezone can trigger:

{ "error": "\"tz\" is required" }

Expected

CLI should:

1. Preserve provided --from/--to values.
2. Include tz consistently (e.g., from configured timezone or local timezone).

Suggested Fix

1. Read from/to directly from cmd.Flags().GetString(...) for this command.
2. Stop binding shared from/to keys globally via Viper for unrelated commands.
3. Add tz query parameter when calling trends endpoint.

I have implemented and use this fix on a local fork, feel free to reference my implementation diff: https://gist.github.com/0xble/3c658a06df85667034038d1177559d8c

Environment

• macOS (Apple Silicon)
• eightctl 0.1.0-dev

[omarshahine]

Fix in progress in PR #31.

Two problems here:

1. Flag collision: --from/--to were bound to global Viper keys, so the values got stomped by other commands. Fixed by reading directly from cmd.Flags() and removing the global viper.BindPFlag calls.
2. Missing tz param: The API now requires a tz query parameter. Added resolveTZ() that maps "local" to the system's IANA zone name (falls back to UTC).

[omarshahine]

Fixed in PR #31.