MPT-16888 complete devtools

.github/workflows/pr-build-merge.yaml

@@ -51,8 +51,17 @@ jobs:
         run: uv run bandit -c pyproject.toml -r . -f json -o bandit.json
 
       - name: Run tests
+        id: run-tests
         run: uv run pytest
 
+      - name: Upload snapshot report on test failure
+        if: ${{ failure() && steps.run-tests.outcome == 'failure' }}
+        uses: actions/upload-artifact@v4
+        with:
+          name: snapshot-report-${{ matrix.python-version }}
+          path: snapshot_report.html
+          if-no-files-found: warn
+
   build:
     runs-on: ubuntu-latest
     timeout-minutes: 10
@@ -81,6 +90,7 @@ jobs:
         run: uv sync --all-extras --dev
 
       - name: Run tests
+        id: run-tests
         run: uv run pytest
 
       - name: Compute added/removed lines for notification

.gitignore

@@ -190,3 +190,5 @@ taplo.toml
 
 # Ignore dynaconf secret files
 .secrets.*
+
+snapshot_report.html

.pre-commit-config.yaml

@@ -1,5 +1,6 @@
 default_language_version:
     python: python3.12
+exclude: (^|/)__snapshots__/.*\.svg$
 repos:
     - repo: https://github.com/pre-commit/pre-commit-hooks
       rev: v5.0.0

mrok/agent/devtools/inspector/__main__.py

@@ -1,25 +1,3 @@
-import argparse
-from typing import Any
+from mrok.agent.devtools.inspector.app import module_main
 
-from mrok.agent.devtools.inspector.app import InspectorApp
-
-
-def run(port: int) -> None:
-    app = InspectorApp(port)
-    app.run()
-
-
-def main(argv: Any = None) -> None:
-    parser = argparse.ArgumentParser(description="mrok devtools agent")
-    parser.add_argument(
-        "-p",
-        "--subscriber-port",
-        type=int,
-        default=50001,
-        help="Port for subscriber (default: 50001)",
-    )
-    args = parser.parse_args(argv)
-    run(args.subscriber_port)
-
-
-main()
+module_main()