feat(pack): add --archive-format zip|tar.gz option (default zip)

Adds --archive-format [zip|tar.gz] to apm pack --archive so callers can
opt into .tar.gz output. Default remains zip. Threaded through
BuildOptions → pack_bundle → export_plugin_bundle (both apm and plugin
bundle formats).

Author: nadav-y

src/apm_cli/bundle/packer.py

@@ -1,6 +1,7 @@
 """Bundle packer  -- creates self-contained APM bundles from the resolved dependency tree."""
 
 import shutil
+import tarfile
 import zipfile
 from dataclasses import dataclass, field
 from pathlib import Path
@@ -28,6 +29,7 @@ def pack_bundle(
     fmt: str = "apm",
     target: str | list[str] | None = None,
     archive: bool = False,
+    archive_format: str = "zip",
     dry_run: bool = False,
     force: bool = False,
     logger=None,
@@ -41,7 +43,8 @@ def pack_bundle(
         target: Target filter  -- ``"copilot"``, ``"claude"``, ``"all"``, a list of
             target strings (e.g. ``["claude", "vscode"]``), or *None*
             (auto-detect from apm.yml / project structure).
-        archive: If *True*, produce a ``.zip`` and remove the directory.
+        archive: If *True*, produce a ``.zip`` (or ``.tar.gz`` when *archive_format* is ``"tar.gz"``) and remove the directory.
+        archive_format: Archive format when *archive* is True -- ``"zip"`` (default) or ``"tar.gz"``.
         dry_run: If *True*, resolve the file list but write nothing to disk.
         force: On collision (plugin format), last writer wins.
 
@@ -64,6 +67,7 @@ def pack_bundle(
             output_dir=output_dir,
             target=target,
             archive=archive,
+            archive_format=archive_format,
             dry_run=dry_run,
             force=force,
             logger=logger,
@@ -270,12 +274,22 @@ def pack_bundle(
 
     # 10. Archive if requested
     if archive:
-        archive_path = output_dir / f"{pkg_name}-{pkg_version}.zip"
-        with zipfile.ZipFile(archive_path, "w", compression=zipfile.ZIP_DEFLATED) as zf:
-            for fp in sorted(bundle_dir.rglob("*")):
-                if fp.is_symlink() or not fp.is_file():
-                    continue
-                zf.write(fp, arcname=f"{bundle_dir.name}/{fp.relative_to(bundle_dir).as_posix()}")
+        if archive_format == "tar.gz":
+            archive_path = output_dir / f"{pkg_name}-{pkg_version}.tar.gz"
+            with tarfile.open(archive_path, "w:gz") as tf:
+                for fp in sorted(bundle_dir.rglob("*")):
+                    if fp.is_symlink() or not fp.is_file():
+                        continue
+                    tf.add(fp, arcname=f"{bundle_dir.name}/{fp.relative_to(bundle_dir).as_posix()}")
+        else:
+            archive_path = output_dir / f"{pkg_name}-{pkg_version}.zip"
+            with zipfile.ZipFile(archive_path, "w", compression=zipfile.ZIP_DEFLATED) as zf:
+                for fp in sorted(bundle_dir.rglob("*")):
+                    if fp.is_symlink() or not fp.is_file():
+                        continue
+                    zf.write(
+                        fp, arcname=f"{bundle_dir.name}/{fp.relative_to(bundle_dir).as_posix()}"
+                    )
         shutil.rmtree(bundle_dir)
         result.bundle_path = archive_path
 

src/apm_cli/bundle/plugin_exporter.py

@@ -11,6 +11,7 @@
 import json
 import re
 import shutil
+import tarfile
 import zipfile
 from pathlib import Path, PurePosixPath
 
@@ -407,6 +408,7 @@ def export_plugin_bundle(
     output_dir: Path,
     target: str | None = None,
     archive: bool = False,
+    archive_format: str = "zip",
     dry_run: bool = False,
     force: bool = False,
     logger=None,
@@ -420,7 +422,8 @@ def export_plugin_bundle(
         project_root: Root of the project containing ``apm.yml``.
         output_dir: Parent directory for the generated bundle.
         target: Unused for plugin format (reserved for future use).
-        archive: If True, produce a ``.zip`` and remove the directory.
+        archive: If True, produce a ``.zip`` (or ``.tar.gz`` when *archive_format* is ``"tar.gz"``) and remove the directory.
+        archive_format: Archive format when *archive* is True -- ``"zip"`` (default) or ``"tar.gz"``.
         dry_run: If True, resolve the file list without writing to disk.
         force: On collision, last writer wins instead of first.
 
@@ -645,13 +648,24 @@ def export_plugin_bundle(
 
     # 15. Archive if requested
     if archive:
-        archive_path = output_dir / f"{bundle_dir.name}.zip"
-        ensure_path_within(archive_path, output_dir)
-        with zipfile.ZipFile(archive_path, "w", compression=zipfile.ZIP_DEFLATED) as zf:
-            for fp in sorted(bundle_dir.rglob("*")):
-                if fp.is_symlink() or not fp.is_file():
-                    continue  # reject symlinks injected after write
-                zf.write(fp, arcname=f"{bundle_dir.name}/{fp.relative_to(bundle_dir).as_posix()}")
+        if archive_format == "tar.gz":
+            archive_path = output_dir / f"{bundle_dir.name}.tar.gz"
+            ensure_path_within(archive_path, output_dir)
+            with tarfile.open(archive_path, "w:gz") as tf:
+                for fp in sorted(bundle_dir.rglob("*")):
+                    if fp.is_symlink() or not fp.is_file():
+                        continue
+                    tf.add(fp, arcname=f"{bundle_dir.name}/{fp.relative_to(bundle_dir).as_posix()}")
+        else:
+            archive_path = output_dir / f"{bundle_dir.name}.zip"
+            ensure_path_within(archive_path, output_dir)
+            with zipfile.ZipFile(archive_path, "w", compression=zipfile.ZIP_DEFLATED) as zf:
+                for fp in sorted(bundle_dir.rglob("*")):
+                    if fp.is_symlink() or not fp.is_file():
+                        continue  # reject symlinks injected after write
+                    zf.write(
+                        fp, arcname=f"{bundle_dir.name}/{fp.relative_to(bundle_dir).as_posix()}"
+                    )
         shutil.rmtree(bundle_dir)
         result.bundle_path = archive_path
 

src/apm_cli/commands/pack.py

@@ -168,7 +168,15 @@ def _parse_marketplace_filter(
     "--archive",
     is_flag=True,
     default=False,
-    help="Produce a .zip archive instead of a directory.",
+    help="Produce an archive instead of a directory (see --archive-format).",
+)
+@click.option(
+    "--archive-format",
+    "archive_format",
+    type=click.Choice(["zip", "tar.gz"]),
+    default="zip",
+    show_default=True,
+    help="Archive format when --archive is set.",
 )
 @click.option(
     "-o",
@@ -267,6 +275,7 @@ def pack_cmd(  # noqa: PLR0913 -- Click handler, one param per CLI option
     fmt,
     target,
     archive,
+    archive_format,
     output,
     dry_run,
     force,
@@ -324,6 +333,7 @@ def pack_cmd(  # noqa: PLR0913 -- Click handler, one param per CLI option
         bundle_format=fmt,
         bundle_target=effective_target,
         bundle_archive=archive,
+        bundle_archive_format=archive_format,
         bundle_output=Path(output),
         bundle_force=force,
         marketplace_offline=offline,

src/apm_cli/core/build_orchestrator.py

@@ -41,6 +41,7 @@ class BuildOptions:
     bundle_format: str = "plugin"
     bundle_target: Any = None
     bundle_archive: bool = False
+    bundle_archive_format: str = "zip"
     bundle_output: Path | None = None
     bundle_force: bool = False
     # Marketplace-only options
@@ -105,6 +106,7 @@ def produce(self, options: BuildOptions, logger: Any) -> ProducerResult:
                 fmt=options.bundle_format,
                 target=options.bundle_target,
                 archive=options.bundle_archive,
+                archive_format=options.bundle_archive_format,
                 dry_run=options.dry_run,
                 force=options.bundle_force,
                 logger=logger,