Skip to content

fix: add --summary and --matrix as standalone flags to analyze command#300

Closed
Copilot wants to merge 2 commits intodevelopfrom
copilot/fix-broken-summary-matrix-flags
Closed

fix: add --summary and --matrix as standalone flags to analyze command#300
Copilot wants to merge 2 commits intodevelopfrom
copilot/fix-broken-summary-matrix-flags

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 13, 2026

sync-ctl analyze --summary . and sync-ctl analyze --matrix . were failing because only --detailed existed as a standalone flag — --summary and --matrix required the indirect --display <format> syntax.

Changes

  • src/cli.rs — Added --summary (-s) and --matrix (-M) boolean flags to the Analyze command with conflicts_with_all guards against --display, --detailed, and each other. Updated --detailed to also conflict with the two new flags.
  • src/handlers/analyze.rs — Extended handle_analyze signature with summary: bool and matrix: bool; updated mode resolution to check standalone flags before falling back to --display.
  • src/main.rs — Updated Commands::Analyze destructuring (telemetry + dispatch) and the handle_analyze wrapper to thread the new flags through.
  • src/lib.rs — Updated run_command's Commands::Analyze arm to match.

Usage

# These now work as expected (previously failed)
sync-ctl analyze --summary .
sync-ctl analyze --matrix .

# Existing behaviour unchanged
sync-ctl analyze --detailed .
sync-ctl analyze --display summary .
sync-ctl analyze --display matrix .

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • eu.i.posthog.com
    • Triggering command: ./target/debug/sync-ctl ./target/debug/sync-ctl analyze --summary . /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a9/home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/sync_ctl-085c79826d587148.1ib2terw9kdktccr0lv88dhiz.1s0cilt.rcgu.o /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a9/home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/sync_ctl-085c79826d587148.1mprg5ppqf1scri0avtxdl3tt.1s0cilt.rcgu.o /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a9/home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/sync_ctl-085c79826d587148.1op72gmkoz9kqg1w461cyyvq0.1s0cilt.rcgu.o c/lib.rs /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a9/home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/sync_ctl-085c79826d587148.1ysnxg5reels0q44zkmibh731.1s0cilt.rcgu.o /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a90ced�� /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a90cede8101d.async_trait.5bec850b8ca3244c-cgu.13.rcgu.o /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a90cede8101d.async_trait.5bec850b8ca3244c-cgu.14.rcgu.o lib/�� lake3-488ca39814ad3356/out/flag_check lib/rustlib/x86_64-REDACTED-linux-gnu/lib/libminiz_oxide-ec844042f1e20091.rlib lake3-488ca39814ad3356/out/flag_check.c et/debug/deps/crcc et/debug/deps/cr-Wl,--version-script=/tmp/rustcC44Q1Z/list et/debug/deps/cr-Wl,--no-undefined-version 0.rcgu.o (dns block)
    • Triggering command: ./target/debug/sync-ctl ./target/debug/sync-ctl analyze --matrix . opt-level=2 -C embed-bitcode=no src/lib.rs codegen-units=256 -C debuginfo=2 -C et/d�� et/debug/build/o-g et/debug/build/o-gdwarf-4 bin/rustc et/debug/build/o/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/rustc -incompat et/debug/build/oencode_unicode bin/rustc (dns block)
    • Triggering command: /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/syncable_cli-836e341553921ab4 /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/syncable_cli-836e341553921ab4 --hash-style=gnu --as-needed -dynamic-linker /lib64/ld-linux-x86-64.so.2 -pie rk_detection-c4bc392450324547.1euzbko1kfi3zlf75ny465uop.09qur4a.rcgu.o rk_detection-c4bc392450324547.1m4e3kxa02rs4mhf5dvt34nnv.09qur4a.rcgu.o rk_detection-c4bc392450324547.2h161878vmqe1suwsr7tffpre.09qur4a.rcgu.o rk_detection-c4bc392450324547.33vui6xlhkmszpt6ri5khdep6.09qur4a.rcgu.o rk_detection-c4bc392450324547.3ax2eo0fei9v7uh9dej0uz7n2.09qur4a.rcgu.o rk_detection-c4bc392450324547.5n41880dmlqecg8sl68rfyk9x.09qur4a.rcgu.o rk_detection-c4bc392450324547.7jes940ucpocg3byqnqzl7h5i.09qur4a.rcgu.o rk_detection-c4bc392450324547.7k8pko0hd0lmsqjsxqskvsipa.09qur4a.rcgu.o rk_detection-c4bc392450324547.7l7gbmvcprzjh08rn6l3jkcgy.09qur4a.rcgu.o rk_detection-c4bc392450324547.8d2jr6p0wu1ln5u0zs873csf9.09qur4a.rcgu.o rk_detection-c4bc392450324547.8ylg33rvvbu0104mk82453coq.09qur4a.rcgu.o rk_detection-c4bc392450324547.904kgdae1w8sjjyf128ohnmj8.09qur4a.rcgu.o rk_detection-c4bc392450324547.aiu28d9wqb5jsf2seb92m45f9.09qur4a.rcgu.o rk_detection-c4bc392450324547.bzw3b9ojdjko9g7vb1e3ab3ga.09qur4a.rcgu.o (dns block)
  • https://api.github.com/repos/syncable-dev/syncable-cli/releases/latest
    • Triggering command: ./target/debug/sync-ctl ./target/debug/sync-ctl analyze --summary . /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a9/home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/sync_ctl-085c79826d587148.1ib2terw9kdktccr0lv88dhiz.1s0cilt.rcgu.o /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a9/home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/sync_ctl-085c79826d587148.1mprg5ppqf1scri0avtxdl3tt.1s0cilt.rcgu.o /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a9/home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/sync_ctl-085c79826d587148.1op72gmkoz9kqg1w461cyyvq0.1s0cilt.rcgu.o c/lib.rs /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a9/home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/sync_ctl-085c79826d587148.1ysnxg5reels0q44zkmibh731.1s0cilt.rcgu.o /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a90ced�� /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a90cede8101d.async_trait.5bec850b8ca3244c-cgu.13.rcgu.o /home/REDACTED/work/syncable-cli/syncable-cli/target/debug/deps/async_trait-65c0a90cede8101d.async_trait.5bec850b8ca3244c-cgu.14.rcgu.o lib/�� lake3-488ca39814ad3356/out/flag_check lib/rustlib/x86_64-REDACTED-linux-gnu/lib/libminiz_oxide-ec844042f1e20091.rlib lake3-488ca39814ad3356/out/flag_check.c et/debug/deps/crcc et/debug/deps/cr-Wl,--version-script=/tmp/rustcC44Q1Z/list et/debug/deps/cr-Wl,--no-undefined-version 0.rcgu.o (http block)
    • Triggering command: ./target/debug/sync-ctl ./target/debug/sync-ctl analyze --matrix . opt-level=2 -C embed-bitcode=no src/lib.rs codegen-units=256 -C debuginfo=2 -C et/d�� et/debug/build/o-g et/debug/build/o-gdwarf-4 bin/rustc et/debug/build/o/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/rustc -incompat et/debug/build/oencode_unicode bin/rustc (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>flag --summary & --matrix is broken</issue_title>
<issue_description>calling sync-ctl analyze --summary / --matrix . isn't currently registered as valid flag inputs. </issue_description>

<agent_instructions>For some reason the when calling "sync-ctl analyze --summary . " or "sync-ctl analyze --matrix ." the cli keeps failing, so something broke in terms of when these flags are called out.

Calling out "sync-ctl analyze --detailed ." is working fine</agent_instructions>

Comments on the Issue (you are @copilot in this section)


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

Co-authored-by: Alex793x <113964069+Alex793x@users.noreply.github.com>
Copilot AI changed the title [WIP] [ISSUE-42] Fix --summary and --matrix flags in sync-ctl analyze fix: add --summary and --matrix as standalone flags to analyze command Mar 13, 2026
Copilot AI requested a review from Alex793x March 13, 2026 15:12
@Alex793x Alex793x closed this Mar 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants