Refactor

Author: mbv06

CHANGELOG.md

@@ -7,6 +7,7 @@ All notable changes to this project are documented here.
 ### Added
 
 - Add optional 3MF post-processing script checks. By default, the bridge notifies the user, writes the script text to the log, and then continues opening the file.
+- Add `post_process_action` config support with `warn`, `block`, and `ignore` modes.
 
 ### Fixed
 

README.md

@@ -5,6 +5,8 @@
 [![License: MIT](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT)
 [![Platform: Windows | macOS | Linux](https://img.shields.io/badge/platform-Windows%20%7C%20macOS%20%7C%20Linux-lightgrey.svg)]()
 
+[Installation](#installation) · [Security Model](#security-model) · [Changelog](CHANGELOG.md)
+
 Slicer URI Bridge helps open 3D model links from websites in Bambu Studio, including sites that do not provide a native Bambu Studio button or where that integration is not available.
 
 https://github.com/user-attachments/assets/32b1fd48-4498-42de-81d6-629b452712b9

src/slicer_uri_bridge/cli.py

@@ -6,13 +6,14 @@
 import subprocess
 import sys
 import tomllib
-from importlib.metadata import version as package_version
+from importlib.metadata import PackageNotFoundError, version as package_version
 from pathlib import Path
 
 from .config import config_matches_default, init_user_config, user_config_path
 from .manager import main as manager_main
 
 
+PACKAGE_NAME = "slicer-uri-bridge"
 YES_VALUES = {"y", "yes"}
 NO_VALUES = {"n", "no"}
 IS_WINDOWS = sys.platform == "win32"
@@ -27,6 +28,13 @@ def eprint(message: str) -> None:
     print(message, file=sys.stderr)
 
 
+def metadata_version() -> str:
+    try:
+        return package_version(PACKAGE_NAME)
+    except PackageNotFoundError:
+        return "unknown"
+
+
 def build_parser() -> argparse.ArgumentParser:
     parser = argparse.ArgumentParser(
         prog="slicer-uri-bridge",
@@ -35,7 +43,7 @@ def build_parser() -> argparse.ArgumentParser:
     parser.add_argument(
         "--version",
         action="version",
-        version=f"%(prog)s {package_version('slicer-uri-bridge')}",
+        version=f"%(prog)s {metadata_version()}",
     )
 
     subparsers = parser.add_subparsers(dest="command", required=True)

tests/test_cli.py

@@ -33,6 +33,30 @@ def isatty(self) -> bool:
         return True
 
 
+class CliVersionTests(unittest.TestCase):
+    def test_version_action_uses_fallback_when_package_metadata_is_absent(self) -> None:
+        with (
+            patch("slicer_uri_bridge.cli.package_version", side_effect=cli.PackageNotFoundError),
+            patch("sys.stdout", new_callable=StringIO) as stdout,
+        ):
+            parser = cli.build_parser()
+            with self.assertRaises(SystemExit) as exit_context:
+                parser.parse_args(["--version"])
+
+        self.assertEqual(exit_context.exception.code, 0)
+        self.assertIn("slicer-uri-bridge unknown", stdout.getvalue())
+
+    def test_subcommands_do_not_require_package_metadata(self) -> None:
+        with (
+            patch("slicer_uri_bridge.cli.package_version", side_effect=cli.PackageNotFoundError),
+            patch("slicer_uri_bridge.cli.user_config_path", return_value=Path("config.toml")),
+            patch("sys.stdout", new_callable=StringIO) as stdout,
+        ):
+            self.assertEqual(cli.main(["config-path"]), 0)
+
+        self.assertIn("config.toml", stdout.getvalue())
+
+
 class InteractiveOnboardingTests(unittest.TestCase):
     def test_package_can_run_as_python_module(self) -> None:
         completed = subprocess.run(