stata-code 0.6.4__tar.gz → 0.6.5__tar.gz

{stata_code-0.6.4 → stata_code-0.6.5}/CHANGELOG.md

@@ -6,6 +6,16 @@ to semver-major.minor for the result schema (see `SCHEMA.md` §6).
 
 ## Unreleased
 
+## 0.6.5 — 2026-05-22
+
+### Fixed
+
+- **OpenAI tool-schema compatibility.** `notebook_locate` and
+  `notebook_insert_cell` no longer advertise top-level `oneOf` constraints in
+  their MCP input schemas. OpenAI rejects those schemas during tool
+  registration, while the server-side runtime guards still enforce the
+  "exactly one query/anchor" rules.
+
 ## 0.6.4 — 2026-05-21
 
 ### Added

{stata_code-0.6.4 → stata_code-0.6.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: stata-code
-Version: 0.6.4
+Version: 0.6.5
 Summary: Agent-native Stata bridge — one core, multiple frontends (MCP, Jupyter, VSCode)
 Project-URL: Homepage, https://github.com/brycewang-stanford/stata-code
 Project-URL: Repository, https://github.com/brycewang-stanford/stata-code
@@ -263,6 +263,19 @@ When `stata-code-mcp` lives inside a project virtualenv (recommended for reprodu
 
 For `uvx`-only setups, set `"command": "uvx"` and `"args": ["--from", "stata-code", "stata-code-mcp"]`.
 
+#### MCP troubleshooting
+
+If `stata_run` reports `adapter_crash` with `worker emitted non-JSON: '\n'`,
+upgrade to `stata-code>=0.6.4`, then restart the MCP client so it launches a
+fresh server process. Also check that the client is resolving the expected
+`stata-code-mcp` binary; project virtualenv installs should use the absolute
+`.venv/bin/stata-code-mcp` path instead of relying on a global `PATH` entry.
+
+If an OpenAI-backed client reports `API Error: 400 Invalid schema for function
+'mcp__stata-code__notebook_insert_cell'` and mentions a top-level `oneOf`,
+upgrade to `stata-code>=0.6.5`, then restart the MCP client. Older server
+processes keep advertising the stale schema until they are restarted.
+
 The MCP server registers 15 tools:
 
 | Tool | Purpose |

{stata_code-0.6.4 → stata_code-0.6.5}/README.md

@@ -224,6 +224,19 @@ When `stata-code-mcp` lives inside a project virtualenv (recommended for reprodu
 
 For `uvx`-only setups, set `"command": "uvx"` and `"args": ["--from", "stata-code", "stata-code-mcp"]`.
 
+#### MCP troubleshooting
+
+If `stata_run` reports `adapter_crash` with `worker emitted non-JSON: '\n'`,
+upgrade to `stata-code>=0.6.4`, then restart the MCP client so it launches a
+fresh server process. Also check that the client is resolving the expected
+`stata-code-mcp` binary; project virtualenv installs should use the absolute
+`.venv/bin/stata-code-mcp` path instead of relying on a global `PATH` entry.
+
+If an OpenAI-backed client reports `API Error: 400 Invalid schema for function
+'mcp__stata-code__notebook_insert_cell'` and mentions a top-level `oneOf`,
+upgrade to `stata-code>=0.6.5`, then restart the MCP client. Older server
+processes keep advertising the stale schema until they are restarted.
+
 The MCP server registers 15 tools:
 
 | Tool | Purpose |

{stata_code-0.6.4 → stata_code-0.6.5}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "stata-code"
-version = "0.6.4"
+version = "0.6.5"
 description = "Agent-native Stata bridge — one core, multiple frontends (MCP, Jupyter, VSCode)"
 readme = "README.md"
 license = "MIT"

{stata_code-0.6.4 → stata_code-0.6.5}/stata_code/__init__.py

@@ -174,7 +174,7 @@ def is_available() -> bool:
     return True
 
 
-__version__ = "0.6.4"
+__version__ = "0.6.5"
 
 __all__ = [
     # Primary entry points

{stata_code-0.6.4 → stata_code-0.6.5}/stata_code/mcp/server.py

@@ -95,7 +95,7 @@ from stata_code.core.runner import (
 )
 from stata_code.core.schema import RunResult
 
-__version__ = "0.6.4"
+__version__ = "0.6.5"
 
 SERVER_INSTRUCTIONS = (
     "Use stata-code for running and inspecting Stata code. Prefer structuredContent "
@@ -894,15 +894,6 @@ def _tool_definitions() -> list[Tool]:
                     },
                 },
                 "required": ["path"],
-                # Schema-level expression of the "exactly one of snippet /
-                # regex / error_text" rule. The runtime guard in
-                # notebook.locate_cells still enforces this for clients that
-                # don't validate inputs against the schema.
-                "oneOf": [
-                    {"required": ["snippet"]},
-                    {"required": ["regex"]},
-                    {"required": ["error_text"]},
-                ],
             },
             outputSchema=_NOTEBOOK_LOCATE_OUTPUT_SCHEMA,
             annotations=ToolAnnotations(
@@ -989,17 +980,6 @@ def _tool_definitions() -> list[Tool]:
                     "at_end": {"type": "boolean"},
                 },
                 "required": ["path", "source"],
-                # Exactly one anchor must be present. Booleans for
-                # at_start/at_end must additionally be `true` to count as
-                # "specified" — the JSON Schema below treats them as
-                # required-presence; the runtime guard in
-                # notebook.insert_cell additionally rejects the false-y form.
-                "oneOf": [
-                    {"required": ["after_cell_id"]},
-                    {"required": ["before_cell_id"]},
-                    {"required": ["at_start"]},
-                    {"required": ["at_end"]},
-                ],
             },
             outputSchema=_NOTEBOOK_INSERT_OUTPUT_SCHEMA,
             annotations=ToolAnnotations(

{stata_code-0.6.4 → stata_code-0.6.5}/tests/test_mcp.py

@@ -87,30 +87,33 @@ class TestToolRegistry:
         assert gg.annotations is not None
         assert gg.annotations.readOnlyHint is True
 
-    def test_notebook_locate_schema_oneof(self):
+    def test_tool_input_schemas_avoid_openai_forbidden_top_level_keywords(self):
+        from stata_code.mcp.server import _tool_definitions
+
+        forbidden = {"oneOf", "anyOf", "allOf", "enum", "not"}
+        for tool in _tool_definitions():
+            schema = tool.inputSchema
+            assert schema is not None
+            assert schema.get("type") == "object"
+            assert forbidden.isdisjoint(schema), tool.name
+
+    def test_notebook_locate_schema_keeps_query_fields_optional(self):
         from stata_code.mcp.server import _tool_definitions
 
         loc = next(t for t in _tool_definitions() if t.name == "notebook_locate")
         schema = loc.inputSchema
-        assert "oneOf" in schema
-        required_sets = [tuple(sorted(o.get("required", []))) for o in schema["oneOf"]]
-        assert ("snippet",) in required_sets
-        assert ("regex",) in required_sets
-        assert ("error_text",) in required_sets
+        assert schema["required"] == ["path"]
+        assert {"snippet", "regex", "error_text"} <= set(schema["properties"])
 
-    def test_notebook_insert_cell_schema_oneof(self):
+    def test_notebook_insert_cell_schema_keeps_anchor_fields_optional(self):
         from stata_code.mcp.server import _tool_definitions
 
         ins = next(t for t in _tool_definitions() if t.name == "notebook_insert_cell")
         schema = ins.inputSchema
-        assert "oneOf" in schema
-        anchors = {tuple(sorted(o.get("required", []))) for o in schema["oneOf"]}
-        assert anchors == {
-            ("after_cell_id",),
-            ("before_cell_id",),
-            ("at_start",),
-            ("at_end",),
-        }
+        assert schema["required"] == ["path", "source"]
+        assert {"after_cell_id", "before_cell_id", "at_start", "at_end"} <= set(
+            schema["properties"]
+        )
 
     def test_resource_templates_include_ref_shapes(self):
         from stata_code.mcp.server import _resource_templates