Release v0.3.0-alpha.28: diagnose_workbook reports correct path

Bug #30: diagnose_workbook always assigned active_workbook and
workbook_path from Excel's currently-active book, regardless of
which workbook the caller asked to diagnose. So calling
diagnose_workbook("foo.xlsx") while bar.xlsx was active in Excel
reported active_workbook="bar.xlsx" and workbook_path=<bar's path>
alongside foo's input/output counts.

Worse, the .vmrs lookup used workbook_path (bar's) so it would
silently find bar's sibling vmrs instead of foo's.

Fix: when an explicit workbook_name is supplied, look up that
book's path from list_workbooks and report it. active_workbook
still reflects Excel's active book (informational), workbook_path
now consistently describes the diagnosed workbook.

404 tests pass. Verified live via the round-2 test pass on the
real workbooks.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: vosesoft

CHANGELOG.md

@@ -4,6 +4,16 @@ All notable changes to ModelRisk MCP. Follows [Keep a Changelog](https://keepach
 
 ## [Unreleased]
 
+## [0.3.0-alpha.28] — 2026-05-22
+
+### Fixed
+
+- **Bug #30 — `diagnose_workbook` mixed data sources when called with an explicit `workbook_name`.** Prior versions always assigned `active_workbook = <Excel-active book's name>` and `workbook_path = <active book's path>`, regardless of which workbook the caller asked to diagnose. Result: calling `diagnose_workbook("foo.xlsx")` while `bar.xlsx` was active in Excel reported `active_workbook="bar.xlsx"` and `workbook_path=<bar's path>` alongside foo's input/output counts — misleading. Worse, the downstream `.vmrs` lookup used `workbook_path` (bar's) and would silently find bar's sibling vmrs instead of foo's. Fix: when an explicit `workbook_name` is supplied, look up that book's path from `list_workbooks` and report it in `workbook_path`. The `active_workbook` field still reflects Excel's active book (useful informationally), but `workbook_path` now consistently describes the workbook being diagnosed.
+
+### Tests
+
+404 unit tests pass. Live verification via the round-2 test pass.
+
 ## [0.3.0-alpha.27] — 2026-05-22
 
 ### Fixed

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "modelrisk-mcp"
-version = "0.3.0a27"
+version = "0.3.0a28"
 description = "Open MCP server bridging Anthropic Claude (and any MCP-compatible client) with the ModelRisk Excel add-in."
 readme = "README.md"
 requires-python = ">=3.11"

server.json

@@ -3,12 +3,12 @@
   "name": "io.github.vosesoftware/modelrisk-mcp",
   "title": "ModelRisk",
   "description": "Read, build, fit, and run Monte Carlo risk models in Excel through Vose Software's ModelRisk.",
-  "version": "0.3.0a27",
+  "version": "0.3.0a28",
   "packages": [
     {
       "registryType": "pypi",
       "identifier": "modelrisk-mcp",
-      "version": "0.3.0a27",
+      "version": "0.3.0a28",
       "transport": {
         "type": "stdio"
       }

src/modelrisk_mcp/__init__.py

@@ -1 +1 @@
-__version__ = "0.3.0a27"
+__version__ = "0.3.0a28"

src/modelrisk_mcp/tools/workflows.py

@@ -205,13 +205,50 @@ def diagnose_workbook(
     }
     issues: list[str] = []
 
-    # 1. Excel reachability + active workbook
+    # 1. Excel reachability + workbook resolution.
+    #
+    # Bug #30 (alpha.28): prior versions assigned
+    # `active_workbook = active.name` and `workbook_path = active.path`
+    # regardless of whether the caller passed an explicit
+    # `workbook_name`. Result: calling `diagnose_workbook("foo.xlsx")`
+    # while "bar.xlsx" was active in Excel would report
+    # active_workbook="bar.xlsx" and workbook_path=<bar's path>
+    # alongside foo's input/output counts — misleading by
+    # construction. The downstream `.vmrs` lookup also used the wrong
+    # path and would silently find bar's vmrs instead of foo's.
+    #
+    # Fix: when `workbook_name` is supplied, look up THAT book's path
+    # from `list_workbooks` and report it. The `active_workbook` field
+    # always reflects Excel's actually-active book (informational),
+    # while `workbook_path` describes the diagnosed workbook.
     try:
         active = bridge.excel.get_active_workbook()
         out["excel_connected"] = True
         wb_name = workbook_name or active.name
         out["active_workbook"] = active.name
+        # Default to active.path; override below if a specific
+        # workbook was named.
         out["workbook_path"] = active.path
+        if workbook_name and workbook_name != active.name:
+            try:
+                target = next(
+                    (b for b in bridge.excel.list_workbooks()
+                     if b.name == workbook_name),
+                    None,
+                )
+                if target is None:
+                    issues.append(
+                        f"Workbook {workbook_name!r} is not currently "
+                        f"open. The diagnose result will reflect the "
+                        f"empty fallback values."
+                    )
+                else:
+                    out["workbook_path"] = target.path
+            except Exception:
+                # If list_workbooks fails, keep the active.path
+                # default; the input counts below will still
+                # be sourced from the requested workbook name.
+                pass
     except Exception as exc:
         issues.append(f"Excel not reachable: {exc!s}")
         out["issues"] = issues

tests/unit/test_server_boot.py

@@ -5,7 +5,7 @@
 
 
 def test_version_is_set() -> None:
-    assert __version__ == "0.3.0a27"
+    assert __version__ == "0.3.0a28"
 
 
 def test_server_name() -> None: