fix: preserve pandoc executable permissions on macOS during extraction

This fixes an issue where pandoc becomes non-executable after installation
because Zipfile.extract() doesn't maintain file permissions.

README.md

@@ -72,6 +72,10 @@ The `tidocs merge` command provides a web interface for combining multiple relea
 
 ## Changelog
 
+### v1.0.1
+
+- Fix the issue that Pandoc becomes non-executable after installation on macOS because `Zipfile.extract()` doesn't maintain file permissions.
+
 ### v1.0.0
 
 - Support merging multiple TiDB release notes Markdown files with HTML tables into one well-formatted Word document.

src/tidocs/cli.py

@@ -7,6 +7,8 @@
 from hypercorn.asyncio import serve
 from hypercorn.config import Config
 
+from tidocs.pandoc_wrapper import Pandoc
+
 APPS = {
     "merge": "Merge Release Notes"
 }
@@ -36,7 +38,7 @@ def launch_marimo_app(appname: str, host: str, port: int) -> None:
 
 
 @click.command(no_args_is_help=True)
-@click.version_option(version='1.0.0')
+@click.version_option(version='1.0.1')
 @click.argument(
     "appname",
     type=click.Choice(list(APPS.keys())),
@@ -79,6 +81,9 @@ def cli(appname: str, host: str, port: int) -> None:
 
             $ tidocs merge --host 0.0.0.0 --port 9000
     """
+    if appname == "merge":
+        pandoc = Pandoc()
+        pandoc.install()
     launch_marimo_app(appname, host, port)
 
 

src/tidocs/pandoc_wrapper.py

@@ -140,6 +140,7 @@ def _progress_callback(count: int, block_size: int, total_size: int) -> None:
                         if basename == self.binary_name:
                             file.extract(info, tmp_dir_path)
                             extracted = tmp_dir_path / info.filename
+                            extracted.chmod(info.external_attr >> 16)
                             break
             elif archive_name.endswith(".tar.gz"):
                 with tarfile.open(archive_path, "r") as file:
@@ -163,7 +164,7 @@ def _progress_callback(count: int, block_size: int, total_size: int) -> None:
         # Save version information after successful installation
         self.save_version_info()
         if not self.is_version_matched():
-            raise AssertionError("Pandoc installation failed.")
+            raise AssertionError("Pandoc version does not match requirements.")
 
         return self.pandoc_binary