cli: add table output to all get commands add --json flag

[juan-malbeclabs]

Summary of Changes

• Refactored all get commands in smartcontract/cli/ (13 files) to use a typed display struct with #[derive(Tabled, Serialize)] and render output as a formatted table by default
• Added --json flag to every get command for machine-readable JSON output, enabling scripting and integration use cases
• multicastgroup get JSON output now serializes a combined object with info, allowlist, and users arrays
• Updated all CLI unit tests from fragile assert_eq! on exact formatted strings to assert!(output_str.contains(...)), and added JSON output test cases
• Updated E2E tests that previously parsed text output from get commands to use --json and json.Unmarshal for reliable structured parsing

Diff Breakdown

Category	Files	Lines (+/-)	Net
Core logic	13	+857 / -414	+443
Tests	6	+62 / -49	+13
Docs	1	+1 / -0	+1

Predominantly CLI core logic refactoring; E2E test updates are a small but important follow-on to consume the new structured output.

Key files (click to expand)

• smartcontract/cli/src/multicastgroup/get.rs — most complex refactor: splits into MulticastGroupDisplay, MulticastAllowlistDisplay, MulticastUserDisplay; JSON output wraps all three in a combined object
• smartcontract/cli/src/user/get.rs — adds UserDisplay struct covering multi-lookup fields (tenant, device, multicast groups, accesspass); skips default pubkey for unset tenant
• smartcontract/cli/src/device/get.rs — adds DeviceDisplay with all device fields including interfaces serialized via {:?}
• smartcontract/cli/src/link/get.rs — adds LinkDisplay; delay/jitter/delay_override formatted as Xms strings
• smartcontract/cli/src/accesspass/get.rs — adds AccessPassDisplay; type field renamed accesspass_type with #[tabled(rename)] and #[serde(rename)]
• smartcontract/cli/src/contributor/get.rs — adds ContributorDisplay struct with json flag
• e2e/internal/devnet/devnet.go — GetDevicePubkeyOnchain switched from text-line scanning to --json + json.Unmarshal
• e2e/contributor_auth_test.go — replaced manual line-splitting loops for pubkey extraction with --json parsing

Testing Verification

• All 213 CLI unit tests pass (cargo test -p doublezero_cli)
• cargo clippy -- -D warnings passes clean
• make rust-fmt applied
• E2E builds verified clean with both e2e and qa build tags (go build -tags e2e ./e2e/...)

[martinsander00]

We can choose to leave it like this but the way it works rn is the JSON output will contain a debug string like "[]" rather than an actual JSON array.

"interfaces": "[]" instead of "interfaces": []

[martinsander00]

Should we remove this?

The table column header will change from "Allocated Resources" to "resource".

[elitegreg]

I don't have a preference

[martinsander00]

Not super critical but tests could pass for wrong reasons and miss regressions. Could we:

• For JSON mode tests: deserialize and check specific field values
• For table mode tests: at minimum check contains("connections") paired with the value