Add --kind and --message flags to changelog add for breaking changes

Add --kind (-k) and --message (-m) flags to `azpysdk changelog add`
that forward to chronus's native --kind and --message options. This
lets developers tag entries as breaking (or any other change kind)
non-interactively:

  azpysdk changelog add --kind breaking -m "Removed deprecated API"
  azpysdk changelog add -k feature -m "Added new endpoint"

Valid kinds: breaking, feature, deprecation, fix, dependencies, internal

Also fix _ensure_chronus_installed to skip the interactive prompt
when stdin is not a TTY (non-interactive / CI environments).

Agent-Logs-Url: https://github.com/Azure/azure-sdk-for-python/sessions/aa3dee26-9283-4594-a49f-64fc64d3d995

Co-authored-by: l0lawrence <100643745+l0lawrence@users.noreply.github.com>

Author: Copilot

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

@@ -12,6 +12,10 @@
 _CHRONUS_PACKAGE = "@chronus/chronus"
 _CHRONUS_MODULE_PATH = os.path.join("node_modules", "@chronus", "chronus")
 
+# Valid change kinds from .chronus/config.yaml.  Kept in sync manually
+# so the CLI can validate before calling out to chronus.
+_CHANGE_KINDS = ["breaking", "feature", "deprecation", "fix", "dependencies", "internal"]
+
 
 class changelog(Check):
     """Manage changelogs with Chronus.
@@ -53,6 +57,23 @@ def register(
                 "automatically. Otherwise chronus detects modified packages interactively."
             ),
         )
+        add_p.add_argument(
+            "--kind", "-k",
+            choices=_CHANGE_KINDS,
+            default=None,
+            help=(
+                "Kind of change (e.g. breaking, feature, fix). "
+                "If omitted, chronus will prompt interactively."
+            ),
+        )
+        add_p.add_argument(
+            "--message", "-m",
+            default=None,
+            help=(
+                "Short description of the change. "
+                "If omitted, chronus will prompt interactively."
+            ),
+        )
         add_p.set_defaults(func=self._run_add)
 
         # changelog verify
@@ -122,14 +143,17 @@ def _ensure_chronus_installed(self) -> None:
             raise SystemExit(1)
 
 
-        print(
-            "\nChronus is not installed locally. It is listed as a dev dependency\n"
-            f"in {os.path.join(REPO_ROOT, 'package.json')}.\n"
-        )
-        answer = input("Run 'npm install' in the repo root to install it? [Y/n] ").strip().lower()
-        if answer not in ("", "y", "yes"):
-            logger.info("Skipped Chronus installation.")
-            raise SystemExit(1)
+        if sys.stdin.isatty():
+            print(
+                "\nChronus is not installed locally. It is listed as a dev dependency\n"
+                f"in {os.path.join(REPO_ROOT, 'package.json')}.\n"
+            )
+            answer = input("Run 'npm install' in the repo root to install it? [Y/n] ").strip().lower()
+            if answer not in ("", "y", "yes"):
+                logger.info("Skipped Chronus installation.")
+                raise SystemExit(1)
+        else:
+            logger.info("Chronus not installed — running 'npm install' automatically (non-interactive).")
 
         logger.info(f"Running: npm install  (cwd: {REPO_ROOT})")
         rc = subprocess.call([npm, "install"], cwd=REPO_ROOT)
@@ -194,6 +218,10 @@ def _run_add(self, args: argparse.Namespace) -> int:
         When no *package* argument is given but CWD is inside a package
         directory (``sdk/<service>/<package>``), the package path is detected
         automatically so the developer doesn't have to specify it.
+
+        Optional ``--kind`` and ``--message`` flags are forwarded to chronus
+        so the developer can skip the interactive prompts (e.g.
+        ``azpysdk changelog add --kind breaking -m "Removed foo API"``).
         """
         chronus_args = ["add"]
         package = getattr(args, "package", None)
@@ -203,6 +231,15 @@ def _run_add(self, args: argparse.Namespace) -> int:
                 logger.info(f"Detected package from current directory: {package}")
         if package:
             chronus_args.append(package)
+
+        kind = getattr(args, "kind", None)
+        if kind:
+            chronus_args.extend(["--kind", kind])
+
+        message = getattr(args, "message", None)
+        if message:
+            chronus_args.extend(["--message", message])
+
         return self._run_chronus(chronus_args)
 
     def _run_verify(self, args: argparse.Namespace) -> int:

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

@@ -7,7 +7,7 @@
 
 import pytest
 
-from azpysdk.changelog import changelog, REPO_ROOT, _CHRONUS_MODULE_PATH
+from azpysdk.changelog import changelog, REPO_ROOT, _CHRONUS_MODULE_PATH, _CHANGE_KINDS
 
 
 # ---------------------------------------------------------------------------
@@ -47,6 +47,46 @@ def test_changelog_add_with_package(self):
         assert args.changelog_command == "add"
         assert args.package == "sdk/storage/azure-storage-blob"
 
+    def test_changelog_add_with_kind(self):
+        parser = _build_parser()
+        args = parser.parse_args(["changelog", "add", "--kind", "breaking"])
+        assert args.kind == "breaking"
+
+    def test_changelog_add_with_kind_short(self):
+        parser = _build_parser()
+        args = parser.parse_args(["changelog", "add", "-k", "feature"])
+        assert args.kind == "feature"
+
+    def test_changelog_add_with_message(self):
+        parser = _build_parser()
+        args = parser.parse_args(["changelog", "add", "--message", "Fixed the bug"])
+        assert args.message == "Fixed the bug"
+
+    def test_changelog_add_with_message_short(self):
+        parser = _build_parser()
+        args = parser.parse_args(["changelog", "add", "-m", "Added new API"])
+        assert args.message == "Added new API"
+
+    def test_changelog_add_with_kind_and_message(self):
+        parser = _build_parser()
+        args = parser.parse_args(
+            ["changelog", "add", "sdk/core/azure-core", "--kind", "breaking", "-m", "Removed old API"]
+        )
+        assert args.package == "sdk/core/azure-core"
+        assert args.kind == "breaking"
+        assert args.message == "Removed old API"
+
+    def test_changelog_add_invalid_kind_rejected(self):
+        parser = _build_parser()
+        with pytest.raises(SystemExit):
+            parser.parse_args(["changelog", "add", "--kind", "notakind"])
+
+    def test_changelog_add_all_valid_kinds_accepted(self):
+        parser = _build_parser()
+        for kind in _CHANGE_KINDS:
+            args = parser.parse_args(["changelog", "add", "--kind", kind])
+            assert args.kind == kind
+
     def test_changelog_create(self):
         parser = _build_parser()
         args = parser.parse_args(["changelog", "create"])
@@ -146,6 +186,50 @@ def test_add_explicit_package_overrides_cwd(self, mock_which, mock_call, _mock_i
         cmd = mock_call.call_args[0][0]
         assert cmd == ["/usr/bin/npx", "--no", "chronus", "add", "sdk/core/azure-core"]
 
+    @patch("azpysdk.changelog.changelog._is_chronus_installed", return_value=True)
+    @patch("azpysdk.changelog.subprocess.call", return_value=0)
+    @patch("azpysdk.changelog.shutil.which", return_value="/usr/bin/npx")
+    def test_add_with_kind_passes_flag(self, mock_which, mock_call, _mock_installed):
+        """The --kind flag should be forwarded to chronus."""
+        parser = _build_parser()
+        args = parser.parse_args(["changelog", "add", "--kind", "breaking"])
+        with patch("os.getcwd", return_value=REPO_ROOT):
+            result = args.func(args)
+        assert result == 0
+        cmd = mock_call.call_args[0][0]
+        assert cmd == ["/usr/bin/npx", "--no", "chronus", "add", "--kind", "breaking"]
+
+    @patch("azpysdk.changelog.changelog._is_chronus_installed", return_value=True)
+    @patch("azpysdk.changelog.subprocess.call", return_value=0)
+    @patch("azpysdk.changelog.shutil.which", return_value="/usr/bin/npx")
+    def test_add_with_message_passes_flag(self, mock_which, mock_call, _mock_installed):
+        """The --message flag should be forwarded to chronus."""
+        parser = _build_parser()
+        args = parser.parse_args(["changelog", "add", "-m", "Fixed upload bug"])
+        with patch("os.getcwd", return_value=REPO_ROOT):
+            result = args.func(args)
+        assert result == 0
+        cmd = mock_call.call_args[0][0]
+        assert cmd == ["/usr/bin/npx", "--no", "chronus", "add", "--message", "Fixed upload bug"]
+
+    @patch("azpysdk.changelog.changelog._is_chronus_installed", return_value=True)
+    @patch("azpysdk.changelog.subprocess.call", return_value=0)
+    @patch("azpysdk.changelog.shutil.which", return_value="/usr/bin/npx")
+    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",
+        ])
+        result = args.func(args)
+        assert result == 0
+        cmd = mock_call.call_args[0][0]
+        assert cmd == [
+            "/usr/bin/npx", "--no", "chronus", "add", "sdk/core/azure-core",
+            "--kind", "breaking", "--message", "Removed deprecated API",
+        ]
+
     @patch("azpysdk.changelog.changelog._is_chronus_installed", return_value=True)
     @patch("azpysdk.changelog.subprocess.call", return_value=0)
     @patch("azpysdk.changelog.shutil.which", return_value="/usr/bin/npx")