feat: introduce unified sync command and grouped syncs

Motivation:
Each command execution in production has a high container startup overhead.
Combining the most frequent sync operations into a single command reduces this
overhead significantly.

Design Choices:
- Introduced a new "sync" command that runs both SMELT and Gitea syncs.
- Moved individual "smelt-sync" and "gitea-sync" commands into the "advanced"
  sub-command group.
- Added hidden, deprecated top-level aliases for "smelt-sync" and "gitea-sync"
  to maintain backward compatibility.
- Unified sync accepts all Gitea-related options.

Benefits:
- Reduced production runtime by approximately 30 seconds per execution.
- Improved CLI organization while maintaining compatibility with existing
  automation pipelines.

Author: okurz

openqabot/args.py

@@ -326,7 +326,7 @@ def updates_run(
     sys.exit(bot())
 
 
-@app.command("smelt-sync")
+@advanced_app.command("smelt-sync")
 def smelt_sync(ctx: typer.Context) -> None:
     """Sync data from SMELT into QEM Dashboard."""
     args = ctx.obj
@@ -336,7 +336,68 @@ def smelt_sync(ctx: typer.Context) -> None:
     sys.exit(syncer())
 
 
-@app.command("gitea-sync")
+@app.command("smelt-sync", hidden=True)
+def smelt_sync_deprecated(ctx: typer.Context) -> None:
+    """DEPRECATED: Sync data from SMELT into QEM Dashboard (use sync or advanced smelt-sync)."""  # noqa: D401
+    smelt_sync(ctx)
+
+
+@app.command("sync")
+def sync(  # noqa: PLR0913
+    ctx: typer.Context,
+    *,
+    gitea_repo: gitea_repo_arg = "products/SLFO",
+    allow_build_failures: Annotated[
+        bool,
+        typer.Option("--allow-build-failures", help="Sync data from PRs despite failing packages"),
+    ] = False,
+    consider_unrequested_prs: Annotated[
+        bool,
+        typer.Option(
+            "--consider-unrequested-prs",
+            help=f"Consider PRs where no review from team {config_module.settings.obs_group} was requested as well",
+        ),
+    ] = False,
+    pr_number: pr_number_arg = None,
+    amqp: Annotated[
+        bool,
+        typer.Option(
+            "--amqp",
+            help="After initial sync listen for new PRs via AMQP and submit them to QEM dashboard immediately",
+        ),
+    ] = False,
+    amqp_url: Annotated[str | None, typer.Option("--amqp-url", help="the URL of the AMQP server")] = None,
+    skip_initial_sync: Annotated[
+        bool,
+        typer.Option(
+            "--amqp-only",
+            help="Skip initial sync before handling AMQP events for new PRs",
+        ),
+    ] = False,
+) -> None:
+    """Sync data from both SMELT and Gitea into QEM Dashboard."""
+    args = ctx.obj
+    _require_token(args)
+
+    # SMELT sync
+    smelt_ret = SMELTSync(args)()
+
+    # Gitea sync
+    args.gitea_repo = gitea_repo
+    args.allow_build_failures = allow_build_failures
+    args.consider_unrequested_prs = consider_unrequested_prs
+    args.pr_number = pr_number
+    args.amqp = amqp
+    # Default from settings (which was already loaded in main callback)
+    args.amqp_url = amqp_url if amqp_url is not None else config_module.settings.amqp_url
+    args.skip_initial_sync = skip_initial_sync
+
+    gitea_ret = GiteaSync(args)()
+
+    sys.exit(smelt_ret or gitea_ret)
+
+
+@advanced_app.command("gitea-sync")
 def gitea_sync(  # noqa: PLR0913
     ctx: typer.Context,
     *,
@@ -385,6 +446,52 @@ def gitea_sync(  # noqa: PLR0913
     sys.exit(syncer())
 
 
+@app.command("gitea-sync", hidden=True)
+def gitea_sync_deprecated(  # noqa: PLR0913
+    ctx: typer.Context,
+    *,
+    gitea_repo: gitea_repo_arg = "products/SLFO",
+    allow_build_failures: Annotated[
+        bool,
+        typer.Option("--allow-build-failures", help="Sync data from PRs despite failing packages"),
+    ] = False,
+    consider_unrequested_prs: Annotated[
+        bool,
+        typer.Option(
+            "--consider-unrequested-prs",
+            help=f"Consider PRs where no review from team {config_module.settings.obs_group} was requested as well",
+        ),
+    ] = False,
+    pr_number: pr_number_arg = None,
+    amqp: Annotated[
+        bool,
+        typer.Option(
+            "--amqp",
+            help="After initial sync listen for new PRs via AMQP and submit them to QEM dashboard immediately",
+        ),
+    ] = False,
+    amqp_url: Annotated[str | None, typer.Option("--amqp-url", help="the URL of the AMQP server")] = None,
+    skip_initial_sync: Annotated[
+        bool,
+        typer.Option(
+            "--amqp-only",
+            help="Skip initial sync before handling AMQP events for new PRs",
+        ),
+    ] = False,
+) -> None:
+    """DEPRECATED: Sync data from Gitea into QEM Dashboard (use sync or advanced gitea-sync)."""  # noqa: D401
+    gitea_sync(
+        ctx,
+        gitea_repo=gitea_repo,
+        allow_build_failures=allow_build_failures,
+        consider_unrequested_prs=consider_unrequested_prs,
+        pr_number=pr_number,
+        amqp=amqp,
+        amqp_url=amqp_url,
+        skip_initial_sync=skip_initial_sync,
+    )
+
+
 @app.command("gitea-trigger")
 def gitea_trigger(  # noqa: PLR0913
     ctx: typer.Context,

tests/test_args.py

@@ -57,17 +57,40 @@ def test_updates_run(mocker: MockerFixture, tmp_path: Path) -> None:
 def test_sync_smelt(mocker: MockerFixture, tmp_path: Path) -> None:
     syncer = mocker.patch("openqabot.args.SMELTSync")
     syncer.return_value.return_value = 0
+    # Test top-level (deprecated)
     result = runner.invoke(app, ["--token", "foo", "--configs", str(tmp_path), "smelt-sync"])
     assert result.exit_code == 0
     syncer.assert_called_once()
+    # Test advanced group
+    syncer.reset_mock()
+    result = runner.invoke(app, ["--token", "foo", "--configs", str(tmp_path), "advanced", "smelt-sync"])
+    assert result.exit_code == 0
+    syncer.assert_called_once()
 
 
 def test_sync_gitea(mocker: MockerFixture, tmp_path: Path) -> None:
     syncer = mocker.patch("openqabot.args.GiteaSync")
     syncer.return_value.return_value = 0
+    # Test top-level (deprecated)
     result = runner.invoke(app, ["--token", "foo", "--configs", str(tmp_path), "gitea-sync"])
     assert result.exit_code == 0
     syncer.assert_called_once()
+    # Test advanced group
+    syncer.reset_mock()
+    result = runner.invoke(app, ["--token", "foo", "--configs", str(tmp_path), "advanced", "gitea-sync"])
+    assert result.exit_code == 0
+    syncer.assert_called_once()
+
+
+def test_sync(mocker: MockerFixture, tmp_path: Path) -> None:
+    smelt = mocker.patch("openqabot.args.SMELTSync")
+    smelt.return_value.return_value = 0
+    gitea = mocker.patch("openqabot.args.GiteaSync")
+    gitea.return_value.return_value = 0
+    result = runner.invoke(app, ["--token", "foo", "--configs", str(tmp_path), "sync"])
+    assert result.exit_code == 0
+    smelt.assert_called_once()
+    gitea.assert_called_once()
 
 
 def test_gitea_trigger(mocker: MockerFixture, tmp_path: Path) -> None: