ventilatepro-cli 0.4.0__tar.gz → 0.5.1__tar.gz

{ventilatepro_cli-0.4.0 → ventilatepro_cli-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ventilatepro-cli
-Version: 0.4.0
+Version: 0.5.1
 Summary: VentilatePro CLI
 Project-URL: Homepage, https://ventilatepro.com/exam/cli/
 Project-URL: Documentation, https://ventilatepro.com/exam/cli/
@@ -47,6 +47,8 @@ ventilatepro projects show 1
 ventilatepro rooms list --project 1
 ventilatepro room-equipment summary --project 1
 ventilatepro room-equipment create --data '{"room": 501, "name": "Sterilizer", "heat_gain_value": 1.8, "heat_gain_unit": "KW"}'
+ventilatepro revit-imports review --project 1 --json
+ventilatepro revit-imports confirm 41 --project 1 --review-token <token-from-review> --yes
 ventilatepro categorization review --project 1 --output room-categorization.json
 ventilatepro categorization apply --project 1 --input room-categorization.json --yes
 ventilatepro ahus list --project 1
@@ -59,6 +61,8 @@ ventilatepro notes sync
 ventilatepro meetings context --project 1 --json
 ventilatepro meetings record --project 1 --input meeting.json --json
 ventilatepro meetings list --project 1
+ventilatepro tasks context --project 1 --json
+ventilatepro tasks create --project 1 --description "Issue updated duct plan" --assignee-id 12 --priority high
 ventilatepro mcp doctor
 ventilatepro-mcp
 ```
@@ -72,11 +76,21 @@ stable project-member IDs, interprets the PM's prose into a structured JSON
 payload, and calls `meetings record`. The server atomically creates the meeting,
 decisions, and assigned project tasks and returns a direct web-app link.
 
+For standalone work, Codex reads `tasks context` to resolve an assignee to a
+stable project-member user ID, then calls `tasks create`. The same workflow is
+available through the typed `vp_get_task_context` and
+`vp_create_project_task` MCP tools.
+
 For room classification, Codex can read the project-specific categorization
 review resource or use the dedicated MCP tools. Review manifests default every
 room to rejected so ambiguous spaces remain visible until a category is
 explicitly accepted with a reason.
 
+For staged Revit imports, agents use `revit-imports review` or
+`vp_review_revit_import` first. The returned review token identifies the exact
+payload and diff that was inspected. Confirmation requires that unchanged token
+plus explicit `--yes` or MCP `confirm=true`; stale reviews are rejected.
+
 ## MCP setup
 
 1. In the VentilatePro web app, open Account Settings and create a CLI token with

{ventilatepro_cli-0.4.0 → ventilatepro_cli-0.5.1}/README.md

@@ -32,6 +32,8 @@ ventilatepro projects show 1
 ventilatepro rooms list --project 1
 ventilatepro room-equipment summary --project 1
 ventilatepro room-equipment create --data '{"room": 501, "name": "Sterilizer", "heat_gain_value": 1.8, "heat_gain_unit": "KW"}'
+ventilatepro revit-imports review --project 1 --json
+ventilatepro revit-imports confirm 41 --project 1 --review-token <token-from-review> --yes
 ventilatepro categorization review --project 1 --output room-categorization.json
 ventilatepro categorization apply --project 1 --input room-categorization.json --yes
 ventilatepro ahus list --project 1
@@ -44,6 +46,8 @@ ventilatepro notes sync
 ventilatepro meetings context --project 1 --json
 ventilatepro meetings record --project 1 --input meeting.json --json
 ventilatepro meetings list --project 1
+ventilatepro tasks context --project 1 --json
+ventilatepro tasks create --project 1 --description "Issue updated duct plan" --assignee-id 12 --priority high
 ventilatepro mcp doctor
 ventilatepro-mcp
 ```
@@ -57,11 +61,21 @@ stable project-member IDs, interprets the PM's prose into a structured JSON
 payload, and calls `meetings record`. The server atomically creates the meeting,
 decisions, and assigned project tasks and returns a direct web-app link.
 
+For standalone work, Codex reads `tasks context` to resolve an assignee to a
+stable project-member user ID, then calls `tasks create`. The same workflow is
+available through the typed `vp_get_task_context` and
+`vp_create_project_task` MCP tools.
+
 For room classification, Codex can read the project-specific categorization
 review resource or use the dedicated MCP tools. Review manifests default every
 room to rejected so ambiguous spaces remain visible until a category is
 explicitly accepted with a reason.
 
+For staged Revit imports, agents use `revit-imports review` or
+`vp_review_revit_import` first. The returned review token identifies the exact
+payload and diff that was inspected. Confirmation requires that unchanged token
+plus explicit `--yes` or MCP `confirm=true`; stale reviews are rejected.
+
 ## MCP setup
 
 1. In the VentilatePro web app, open Account Settings and create a CLI token with

{ventilatepro_cli-0.4.0 → ventilatepro_cli-0.5.1}/tests/test_cli.py

@@ -7,10 +7,11 @@ from unittest import mock
 from typer.testing import CliRunner
 
 from ventilatepro import auth, config
-from ventilatepro.api_client import ApiError, NetworkError
+from ventilatepro.api_client import ApiClient, ApiError, NetworkError
 from ventilatepro.cli import app
 from ventilatepro.categorization import _classification_group_key
 from ventilatepro.queue import QueueStore
+from ventilatepro.revit_imports import build_revit_import_review
 
 
 class CliTests(unittest.TestCase):
@@ -361,6 +362,84 @@ class CliTests(unittest.TestCase):
         self.assertEqual(payload["members"][0]["user_id"], 12)
         fake_client.meeting_context.assert_called_once_with(1)
 
+    def test_tasks_context_outputs_assignable_directory(self):
+        fake_client = mock.Mock()
+        fake_client.task_context.return_value = {
+            "project": {"id": 1, "name": "Hospital"},
+            "members": [
+                {
+                    "user_id": 12,
+                    "name": "Alex Engineer",
+                    "email": "alex@example.com",
+                    "role": "engineer",
+                },
+            ],
+        }
+        with mock.patch.object(
+            auth,
+            "get_authenticated_client",
+            return_value=(fake_client, "http://localhost:8000"),
+        ):
+            result = self.runner.invoke(
+                app,
+                ["tasks", "context", "--project", "1", "--json"],
+            )
+
+        self.assertEqual(result.exit_code, 0)
+        payload = json.loads(result.stdout)
+        self.assertEqual(payload["members"][0]["user_id"], 12)
+        fake_client.task_context.assert_called_once_with(1)
+
+    def test_tasks_create_assigns_project_member(self):
+        fake_client = mock.Mock()
+        fake_client.base_url = "https://ventilatepro.com"
+        fake_client.create_project_task.return_value = {
+            "id": 46,
+            "project": 1,
+            "description": "Issue diffuser update",
+            "assigned_user": 12,
+            "priority": "high",
+            "include_in_minutes": False,
+        }
+        with mock.patch.object(
+            auth,
+            "get_authenticated_client",
+            return_value=(fake_client, "https://ventilatepro.com"),
+        ):
+            result = self.runner.invoke(
+                app,
+                [
+                    "tasks",
+                    "create",
+                    "--project",
+                    "1",
+                    "--description",
+                    "Issue diffuser update",
+                    "--assignee-id",
+                    "12",
+                    "--due-at",
+                    "2026-08-05T17:00:00-07:00",
+                    "--priority",
+                    "high",
+                    "--notes",
+                    "Coordinate with architecture.",
+                    "--estimated-hours",
+                    "2.5",
+                    "--json",
+                ],
+            )
+
+        self.assertEqual(result.exit_code, 0)
+        payload = json.loads(result.stdout)
+        self.assertEqual(payload["task"]["id"], 46)
+        self.assertIn("view=project-tasks", payload["deep_link"])
+        sent_payload = fake_client.create_project_task.call_args.args[0]
+        self.assertEqual(sent_payload["project"], 1)
+        self.assertEqual(sent_payload["assigned_user"], 12)
+        self.assertEqual(sent_payload["priority"], "high")
+        self.assertFalse(sent_payload["include_in_minutes"])
+        self.assertEqual(sent_payload["source"], "cli")
+
     def test_meetings_record_reads_structured_stdin_and_saves_immediately(self):
         fake_client = mock.Mock()
         fake_client.record_meeting.return_value = {
@@ -1181,3 +1260,161 @@ class CliTests(unittest.TestCase):
 
         self.assertEqual(result.exit_code, 0)
         fake_client.design_day_run.assert_called_once_with(1, {"zones": [{"zone_id": 1}]})
+
+    def test_revit_import_review_returns_diff_and_stable_review_token(self):
+        fake_client = mock.Mock()
+        session = {
+            "id": 41,
+            "project": 7,
+            "status": "pending",
+            "updated_at": "2026-07-29T10:00:00Z",
+            "payload": {
+                "rooms": [{"number": "101", "name": "Exam", "revit_space_unique_id": "uid-101"}],
+                "zones": [{"name": "Clinic"}],
+            },
+            "options": {"mark_missing_rooms": True},
+            "diff_summary": {
+                "summary": {
+                    "new_rooms": 1,
+                    "updated_rooms": 0,
+                    "missing_rooms": 2,
+                    "new_zones": 1,
+                    "missing_zones": 0,
+                    "ambiguous_matches": 0,
+                },
+                "new_rooms": [{"key": "revit:uid-101"}],
+                "updated_rooms": [],
+                "missing_rooms": [{"id": 9, "number": "099"}],
+                "new_zones": [{"name": "Clinic"}],
+                "missing_zones": [],
+                "conflicts": {},
+                "errors": [],
+            },
+        }
+        fake_client.revit_import_pending.return_value = session
+
+        with mock.patch.object(auth, "get_authenticated_client", return_value=(fake_client, "http://localhost:8000")):
+            result = self.runner.invoke(
+                app,
+                ["revit-imports", "review", "--project", "7", "--json"],
+            )
+
+        self.assertEqual(result.exit_code, 0)
+        payload = json.loads(result.stdout)
+        self.assertEqual(payload["summary"]["new_rooms"], 1)
+        self.assertEqual(payload["changes"]["missing_rooms"][0]["id"], 9)
+        self.assertEqual(payload["payload_counts"], {"rooms": 1, "zones": 1})
+        self.assertTrue(payload["review_token"].startswith("revit-import:41:"))
+        self.assertNotIn("payload", payload)
+        fake_client.revit_import_pending.assert_called_once_with(7)
+
+    def test_revit_import_confirm_requires_yes_before_any_api_call(self):
+        fake_client = mock.Mock()
+        with mock.patch.object(auth, "get_authenticated_client", return_value=(fake_client, "http://localhost:8000")):
+            result = self.runner.invoke(
+                app,
+                [
+                    "revit-imports",
+                    "confirm",
+                    "41",
+                    "--project",
+                    "7",
+                    "--review-token",
+                    "revit-import:41:test",
+                ],
+            )
+
+        self.assertNotEqual(result.exit_code, 0)
+        self.assertIn("requires --yes", result.output)
+        fake_client.revit_import_show.assert_not_called()
+        fake_client.revit_import_confirm.assert_not_called()
+
+    def test_revit_import_confirm_rechecks_project_status_and_review_token(self):
+        fake_client = mock.Mock()
+        session = {
+            "id": 41,
+            "project": 7,
+            "status": "pending",
+            "updated_at": "2026-07-29T10:00:00Z",
+            "payload": {
+                "rooms": [{"number": "101", "name": "Exam", "revit_space_unique_id": "uid-101"}],
+                "zones": [],
+            },
+            "options": {},
+            "diff_summary": {"summary": {"new_rooms": 1}},
+        }
+        review_token = build_revit_import_review(session)["review_token"]
+        fake_client.revit_import_show.return_value = session
+        fake_client.revit_import_confirm.return_value = {
+            "session": {"id": 41, "project": 7, "status": "committed"},
+            "result": {"created": 1, "updated": 0, "marked_missing": 0},
+        }
+
+        with mock.patch.object(auth, "get_authenticated_client", return_value=(fake_client, "http://localhost:8000")):
+            result = self.runner.invoke(
+                app,
+                [
+                    "revit-imports",
+                    "confirm",
+                    "41",
+                    "--project",
+                    "7",
+                    "--review-token",
+                    review_token,
+                    "--yes",
+                    "--json",
+                ],
+            )
+
+        self.assertEqual(result.exit_code, 0)
+        self.assertEqual(json.loads(result.stdout)["session"]["status"], "committed")
+        fake_client.revit_import_show.assert_called_once_with(41)
+        fake_client.revit_import_confirm.assert_called_once_with(41)
+
+    def test_revit_import_confirm_rejects_stale_review_token(self):
+        fake_client = mock.Mock()
+        fake_client.revit_import_show.return_value = {
+            "id": 41,
+            "project": 7,
+            "status": "pending",
+            "updated_at": "2026-07-29T10:05:00Z",
+            "payload": {
+                "rooms": [{"number": "102", "name": "Changed", "revit_space_unique_id": "uid-102"}],
+                "zones": [],
+            },
+            "options": {},
+            "diff_summary": {"summary": {"new_rooms": 1}},
+        }
+
+        with mock.patch.object(auth, "get_authenticated_client", return_value=(fake_client, "http://localhost:8000")):
+            result = self.runner.invoke(
+                app,
+                [
+                    "revit-imports",
+                    "confirm",
+                    "41",
+                    "--project",
+                    "7",
+                    "--review-token",
+                    "revit-import:41:stale",
+                    "--yes",
+                ],
+            )
+
+        self.assertNotEqual(result.exit_code, 0)
+        self.assertIn("changed after it was reviewed", result.output)
+        fake_client.revit_import_confirm.assert_not_called()
+
+    def test_revit_import_api_client_uses_scoped_proxy_and_long_commit_timeout(self):
+        client = ApiClient("https://ventilatepro.example", "token")
+        with mock.patch.object(client, "request", return_value={"status": "ok"}) as request:
+            result = client.revit_import_confirm(41)
+
+        self.assertEqual(result, {"status": "ok"})
+        request.assert_called_once_with(
+            "POST",
+            "/api/cli/proxy/revit-imports/41/commit/",
+            json_payload={},
+            params=None,
+            timeout=300,
+        )

ventilatepro_cli-0.5.1/tests/test_mcp_server.py

@@ -0,0 +1,331 @@
+import asyncio
+import unittest
+from unittest import mock
+
+from ventilatepro.mcp_server import _requires_confirmation, build_server
+
+
+class FakeClient:
+    def __init__(self):
+        self.calls = []
+
+    def schemas(self):
+        return {"schemas": {"api_request": {"description": "ok"}}}
+
+    def schema(self, schema_key):
+        return {"key": schema_key}
+
+    def auth_me(self):
+        return {"username": "agent", "scopes": ["projects:read"]}
+
+    def projects_list(self, *, limit=20, offset=0, query=None):
+        return {"count": 0, "limit": limit, "offset": offset, "query": query, "results": []}
+
+    def cli_proxy(self, method, api_path, *, json_payload=None, params=None):
+        self.calls.append((method, api_path, json_payload, params))
+        return {"method": method, "path": api_path, "payload": json_payload, "params": params}
+
+    def categorization_review(self, project_id):
+        self.calls.append(("categorization_review", project_id))
+        return {
+            "project": {"id": project_id},
+            "category_catalog": {"hcai": [], "occupancy": []},
+            "rooms": [],
+            "categorizations": [],
+        }
+
+    def record_meeting(self, payload):
+        self.calls.append(("record_meeting", payload))
+        return {
+            "meeting": {"id": 31, "title": payload["title"]},
+            "deep_link": "https://ventilatepro.com/?view=meeting-minutes&project=7&minute=31",
+        }
+
+    def task_context(self, project_id):
+        self.calls.append(("task_context", project_id))
+        return {
+            "project": {"id": project_id, "name": "Hospital"},
+            "members": [{"user_id": 12, "name": "Alex Engineer"}],
+        }
+
+    def create_project_task(self, payload):
+        self.calls.append(("create_project_task", payload))
+        return {"id": 46, **payload}
+
+    def revit_import_pending(self, project_id):
+        self.calls.append(("revit_import_pending", project_id))
+        return self.revit_session
+
+    def revit_import_show(self, import_id):
+        self.calls.append(("revit_import_show", import_id))
+        return self.revit_session
+
+    def revit_import_confirm(self, import_id):
+        self.calls.append(("revit_import_confirm", import_id))
+        return {
+            "session": {**self.revit_session, "status": "committed"},
+            "result": {"created": 1},
+        }
+
+    def revit_import_discard(self, import_id):
+        self.calls.append(("revit_import_discard", import_id))
+        return {**self.revit_session, "status": "discarded"}
+
+
+class McpServerTests(unittest.TestCase):
+    def test_tool_catalog_contains_core_tools(self):
+        async def run():
+            server = build_server()
+            tools = await server.list_tools()
+            names = {tool.name for tool in tools}
+            self.assertIn("vp_whoami", names)
+            self.assertIn("vp_api_request", names)
+            self.assertIn("vp_list_entities", names)
+            self.assertIn("vp_run_workflow", names)
+            self.assertIn("vp_get_categorization_review", names)
+            self.assertIn("vp_apply_categorization", names)
+            self.assertIn("vp_get_meeting_context", names)
+            self.assertIn("vp_get_task_context", names)
+            self.assertIn("vp_create_project_task", names)
+            self.assertIn("vp_record_meeting", names)
+            self.assertIn("vp_list_meetings", names)
+            self.assertIn("vp_get_meeting", names)
+            self.assertIn("vp_revise_decision", names)
+            self.assertIn("vp_review_revit_import", names)
+            self.assertIn("vp_confirm_revit_import", names)
+            self.assertIn("vp_discard_revit_import", names)
+
+        asyncio.run(run())
+
+    def test_record_meeting_tool_saves_without_confirmation(self):
+        async def run():
+            server = build_server()
+            fake_client = FakeClient()
+            with mock.patch(
+                "ventilatepro.mcp_server.auth.get_authenticated_client",
+                return_value=(fake_client, "http://localhost:8000"),
+            ), mock.patch("ventilatepro.mcp_server.QueueStore"):
+                _content, structured = await server.call_tool(
+                    "vp_record_meeting",
+                    {
+                        "project_id": 7,
+                        "idempotency_key": "mcp-meeting-31",
+                        "payload": {
+                            "title": "MCP Coordination",
+                            "started_at": "2026-07-24T09:00:00-07:00",
+                        },
+                    },
+                )
+
+            result = structured["result"]
+            self.assertEqual(result["status"], "created")
+            self.assertEqual(result["meeting"]["id"], 31)
+            sent_payload = fake_client.calls[0][1]
+            self.assertEqual(sent_payload["project_id"], 7)
+            self.assertEqual(sent_payload["idempotency_key"], "mcp-meeting-31")
+
+        asyncio.run(run())
+
+    def test_api_request_tool_uses_cli_proxy(self):
+        async def run():
+            server = build_server()
+            fake_client = FakeClient()
+            with mock.patch(
+                "ventilatepro.mcp_server.auth.get_authenticated_client",
+                return_value=(fake_client, "http://localhost:8000"),
+            ):
+                _content, structured = await server.call_tool(
+                    "vp_api_request",
+                    {
+                        "method": "GET",
+                        "path": "projects/1/",
+                        "params": {"include": "summary"},
+                    },
+                )
+
+            self.assertEqual(structured["result"]["path"], "projects/1/")
+            self.assertEqual(fake_client.calls[0], ("GET", "projects/1/", None, {"include": "summary"}))
+
+        asyncio.run(run())
+
+    def test_guarded_operations_require_confirmation(self):
+        self.assertTrue(_requires_confirmation("DELETE", "rooms/1/", None))
+        self.assertTrue(_requires_confirmation("POST", "projects/1/recalculate-stale/", {}))
+        self.assertTrue(_requires_confirmation("POST", "projects/1/design-day/run/", {}))
+        self.assertTrue(_requires_confirmation("POST", "projects/1/apply-categorization/", {}))
+        self.assertTrue(_requires_confirmation("PATCH", "projects/1/", {"force": True}))
+        self.assertFalse(_requires_confirmation("GET", "projects/1/", None))
+
+    def test_generic_api_tool_cannot_bypass_revit_review_token(self):
+        async def run():
+            server = build_server()
+            fake_client = FakeClient()
+            with mock.patch(
+                "ventilatepro.mcp_server.auth.get_authenticated_client",
+                return_value=(fake_client, "http://localhost:8000"),
+            ):
+                _content, structured = await server.call_tool(
+                    "vp_api_request",
+                    {
+                        "method": "POST",
+                        "path": "revit-imports/41/commit/",
+                        "payload": {},
+                        "confirm": True,
+                    },
+                )
+
+            result = structured["result"]
+            self.assertEqual(result["error"]["code"], "specialized_tool_required")
+            self.assertIn("vp_confirm_revit_import", result["error"]["message"])
+            self.assertEqual(fake_client.calls, [])
+
+        asyncio.run(run())
+
+    def test_create_project_task_tool_assigns_member_without_confirmation(self):
+        async def run():
+            server = build_server()
+            fake_client = FakeClient()
+            fake_client.base_url = "https://ventilatepro.com"
+            with mock.patch(
+                "ventilatepro.mcp_server.auth.get_authenticated_client",
+                return_value=(fake_client, "https://ventilatepro.com"),
+            ):
+                _content, structured = await server.call_tool(
+                    "vp_create_project_task",
+                    {
+                        "project_id": 7,
+                        "description": "Issue diffuser update",
+                        "assigned_user_id": 12,
+                        "priority": "high",
+                        "due_at": "2026-08-05T17:00:00-07:00",
+                    },
+                )
+
+            result = structured["result"]
+            self.assertEqual(result["status"], "created")
+            self.assertEqual(result["task"]["id"], 46)
+            self.assertIn("view=project-tasks", result["deep_link"])
+            call_name, sent_payload = fake_client.calls[0]
+            self.assertEqual(call_name, "create_project_task")
+            self.assertEqual(sent_payload["assigned_user"], 12)
+            self.assertFalse(sent_payload["include_in_minutes"])
+
+        asyncio.run(run())
+
+    def test_generic_api_tool_blocks_prefixed_revit_commit_path(self):
+        async def run():
+            server = build_server()
+            fake_client = FakeClient()
+            with mock.patch(
+                "ventilatepro.mcp_server.auth.get_authenticated_client",
+                return_value=(fake_client, "http://localhost:8000"),
+            ):
+                _content, structured = await server.call_tool(
+                    "vp_api_request",
+                    {
+                        "method": "POST",
+                        "path": "hvac/api/revit-imports/41/commit/",
+                        "payload": {},
+                        "confirm": True,
+                    },
+                )
+
+            self.assertEqual(structured["result"]["error"]["code"], "specialized_tool_required")
+            self.assertEqual(fake_client.calls, [])
+
+        asyncio.run(run())
+
+    def test_revit_import_review_and_confirm_require_matching_review_token(self):
+        async def run():
+            server = build_server()
+            fake_client = FakeClient()
+            fake_client.revit_session = {
+                "id": 41,
+                "project": 7,
+                "status": "pending",
+                "updated_at": "2026-07-29T10:00:00Z",
+                "payload": {
+                    "rooms": [{"number": "101", "name": "Exam", "revit_space_unique_id": "uid-101"}],
+                    "zones": [],
+                },
+                "options": {},
+                "diff_summary": {
+                    "summary": {"new_rooms": 1, "updated_rooms": 0, "missing_rooms": 0},
+                    "new_rooms": [{"key": "revit:uid-101"}],
+                },
+            }
+            with mock.patch(
+                "ventilatepro.mcp_server.auth.get_authenticated_client",
+                return_value=(fake_client, "http://localhost:8000"),
+            ):
+                _content, structured = await server.call_tool(
+                    "vp_review_revit_import",
+                    {"project_id": 7},
+                )
+                review = structured["result"]
+                self.assertEqual(review["summary"]["new_rooms"], 1)
+                self.assertTrue(review["review_token"].startswith("revit-import:41:"))
+
+                _content, blocked = await server.call_tool(
+                    "vp_confirm_revit_import",
+                    {
+                        "project_id": 7,
+                        "import_id": 41,
+                        "review_token": review["review_token"],
+                        "confirm": False,
+                    },
+                )
+                self.assertEqual(blocked["result"]["error"]["code"], "confirmation_required")
+
+                _content, confirmed = await server.call_tool(
+                    "vp_confirm_revit_import",
+                    {
+                        "project_id": 7,
+                        "import_id": 41,
+                        "review_token": review["review_token"],
+                        "confirm": True,
+                    },
+                )
+
+            self.assertEqual(confirmed["result"]["session"]["status"], "committed")
+            self.assertIn(("revit_import_confirm", 41), fake_client.calls)
+
+        asyncio.run(run())
+
+    def test_revit_import_confirm_rejects_stale_token(self):
+        async def run():
+            server = build_server()
+            fake_client = FakeClient()
+            fake_client.revit_session = {
+                "id": 41,
+                "project": 7,
+                "status": "pending",
+                "updated_at": "2026-07-29T10:05:00Z",
+                "payload": {"rooms": [{"number": "102", "name": "Changed"}], "zones": []},
+                "options": {},
+                "diff_summary": {"summary": {"new_rooms": 1}},
+            }
+            with mock.patch(
+                "ventilatepro.mcp_server.auth.get_authenticated_client",
+                return_value=(fake_client, "http://localhost:8000"),
+            ):
+                _content, structured = await server.call_tool(
+                    "vp_confirm_revit_import",
+                    {
+                        "project_id": 7,
+                        "import_id": 41,
+                        "review_token": "revit-import:41:stale",
+                        "confirm": True,
+                    },
+                )
+
+            result = structured["result"]
+            self.assertEqual(result["error"]["code"], "ValueError")
+            self.assertIn("changed after it was reviewed", result["error"]["message"])
+            self.assertNotIn(("revit_import_confirm", 41), fake_client.calls)
+
+        asyncio.run(run())
+
+
+if __name__ == "__main__":
+    unittest.main()

{ventilatepro_cli-0.4.0 → ventilatepro_cli-0.5.1}/ventilatepro/__init__.py

@@ -2,4 +2,4 @@
 
 __all__ = ["__version__"]
 
-__version__ = "0.4.0"
+__version__ = "0.5.1"