Fix CI: black formatting + exclude changelog from per-package check discovery

Two CI failures on PR #46016:

1. static-analysis (black): changelog.py and test_changelog_commands.py failed 'black --check'. Applied black with the repo's eng/black-pyproject.toml config.

2. verify-azpysdk-checks: the workflow discovers checks by scraping 'azpysdk -h' and invokes each one as 'azpysdk <check> --isolate <package>'. The changelog command group rejects --isolate (it's a developer tool wrapping Chronus, not a per-package check) and exits 2. Filter 'changelog' out of the discovery pipeline, same way 'next-*' checks are already filtered.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: Copilot

.github/workflows/azure-sdk-tools.yml

@@ -72,13 +72,16 @@ jobs:
         run: |
           uv pip install --system eng/tools/azure-sdk-tools[ghtools,conda,systemperf]
 
-          # Discover available azpysdk commands from the {command1,command2,...} line in help output
+          # Discover available azpysdk commands from the {command1,command2,...} line in help output.
+          # `changelog` is filtered out because it is a developer command group (add/verify/create/status)
+          # that wraps Chronus, not a per-package check, so the dispatcher cannot invoke it positionally.
           CHECKS=$(azpysdk -h 2>&1 | \
             grep -oP '\{[^}]+\}' | \
             tail -1 | \
             tr -d '{}' | \
             tr ',' '\n' | \
             grep -v '^next-' | \
+            grep -v '^changelog$' | \
             sort | \
             paste -sd,)
 

eng/tools/azure-sdk-tools/azpysdk/changelog.py

@@ -80,21 +80,17 @@ def register(
             ),
         )
         add_p.add_argument(
-            "--kind", "-k",
+            "--kind",
+            "-k",
             choices=_CHANGE_KINDS,
             default=None,
-            help=(
-                "Kind of change (e.g. breaking, feature, fix). "
-                "If omitted, chronus will prompt interactively."
-            ),
+            help=("Kind of change (e.g. breaking, feature, fix). " "If omitted, chronus will prompt interactively."),
         )
         add_p.add_argument(
-            "--message", "-m",
+            "--message",
+            "-m",
             default=None,
-            help=(
-                "Short description of the change. "
-                "If omitted, chronus will prompt interactively."
-            ),
+            help=("Short description of the change. " "If omitted, chronus will prompt interactively."),
         )
         add_p.set_defaults(func=self._run_add)
 
@@ -103,9 +99,7 @@ def register(
         verify_p.set_defaults(func=self._run_verify)
 
         # changelog create
-        create_p = changelog_sub.add_parser(
-            "create", help="Generate CHANGELOG.md from pending chronus entries"
-        )
+        create_p = changelog_sub.add_parser("create", help="Generate CHANGELOG.md from pending chronus entries")
         create_p.set_defaults(func=self._run_create)
 
         # changelog status

eng/tools/azure-sdk-tools/tests/test_changelog_commands.py

@@ -21,6 +21,7 @@
 # Helper – build a minimal parser that includes the changelog subcommands
 # ---------------------------------------------------------------------------
 
+
 def _build_parser():
     parser = argparse.ArgumentParser(prog="azpysdk")
     subparsers = parser.add_subparsers(title="commands", dest="command")
@@ -256,16 +257,30 @@ def test_add_with_message_passes_flag(self, mock_which, mock_call, _mock_install
     def test_add_with_kind_message_and_package(self, mock_which, mock_call, _mock_installed):
         """All flags (package, --kind, --message) should be forwarded together."""
         parser = _build_parser()
-        args = parser.parse_args([
-            "changelog", "add", "sdk/core/azure-core",
-            "--kind", "breaking", "-m", "Removed deprecated API",
-        ])
+        args = parser.parse_args(
+            [
+                "changelog",
+                "add",
+                "sdk/core/azure-core",
+                "--kind",
+                "breaking",
+                "-m",
+                "Removed deprecated API",
+            ]
+        )
         result = args.func(args)
         assert result == 0
         cmd = mock_call.call_args[0][0]
         assert cmd == [
-            "/usr/bin/npx", "--no", "chronus", "add", "azure-core",
-            "--kind", "breaking", "--message", "Removed deprecated API",
+            "/usr/bin/npx",
+            "--no",
+            "chronus",
+            "add",
+            "azure-core",
+            "--kind",
+            "breaking",
+            "--message",
+            "Removed deprecated API",
         ]
 
     @patch("azpysdk.changelog.changelog._is_chronus_installed", return_value=True)