vmware-debug 1.11.2__tar.gz → 1.11.3__tar.gz

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/.claude-plugin/plugin.json

@@ -2,7 +2,7 @@
   "$schema": "https://anthropic.com/claude-code/plugin.schema.json",
   "name": "vmware-debug",
   "displayName": "VMware Debug",
-  "version": "1.11.2",
+  "version": "1.11.3",
   "description": "Read-only VMware incident correlation: timeline, spikes, root-cause routing. 2 MCP tools.",
   "author": {
     "name": "Wei Zhou",

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/.mcp.json

@@ -5,7 +5,7 @@
       "command": "uvx",
       "args": [
         "--from",
-        "vmware-debug==1.11.2",
+        "vmware-debug==1.11.3",
         "vmware-debug-mcp"
       ]
     }

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: vmware-debug
-Version: 1.11.2
+Version: 1.11.3
 Summary: VMware diagnostic brain — read-only incident triage, log/event correlation, and root-cause routing across the VMware skill family
 Project-URL: Homepage, https://github.com/vmware-skills/VMware-Debug
 Project-URL: Repository, https://github.com/vmware-skills/VMware-Debug
@@ -18,7 +18,7 @@ Requires-Dist: mcp[cli]<2.0,>=1.10
 Requires-Dist: pyyaml<7.0,>=6.0
 Requires-Dist: rich<15.0,>=13.0
 Requires-Dist: typer<1.0,>=0.12
-Requires-Dist: vmware-policy<2.0,>=1.11.0
+Requires-Dist: vmware-policy<2.0,>=1.12.0
 Description-Content-Type: text/markdown
 
 <!-- mcp-name: io.github.vmware-skills/vmware-debug -->

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/RELEASE_NOTES.md

@@ -1,3 +1,39 @@
+## v1.11.3 — 398 tests never ran
+
+
+**`tests/__init__.py` was missing, and the whole suite -- 398 tests -- failed at
+collection.** `tests/eval/` and `tests/eval/regression/` were already packages,
+so `from tests.eval.regression._vsphere_event_vocabulary import ...` resolved
+only where the namespace-package fallback happened to work. It did not on
+Windows. One file.
+
+**The suite now runs on a cp936 machine.** Round 3 of the VCF 9 field testing ran
+on Windows Server 2025 with locale cp936. Across the family four repos' suites --
+1687 tests -- never executed at all, dying at collection reading our own UTF-8
+sources, and 101 more failed the same way. Most of those were the tests that
+verify the destructive-operation guardrails: the guardrails were fine, the tests
+that check them could not open a file. On the UTF-8 CI every one of them was
+green. A security test that cannot run is not a security test.
+
+Every text read and write here names its encoding now, `tests/` included -- the
+previous round fixed only the package, which is why this came back. A gate in
+`family_smoke` scans both trees by AST, and the whole family's suites were re-run
+under an ASCII locale to confirm: 15 of 15 green, from 1 of 15.
+
+**`--help` no longer dies on a console that cannot encode it.** On any console
+whose encoding cannot carry the characters in our own help text, `--help` exited
+with a `UnicodeEncodeError` traceback -- unavailable exactly on the machines
+where it is most needed. Four repos were affected; the handler is now relaxed in
+all fifteen so a glyph degrades instead of killing the command.
+
+**Unknown tool arguments are refused instead of dropped.** The schema declared
+`additionalProperties: false` and the runtime accepted them anyway, so a filter
+argument whose name a model guessed wrong returned the *unfiltered* result with
+nothing to indicate anything had been discarded. Fixed in vmware-policy 1.12.0
+and in force here.
+
+Requires vmware-policy 1.12.0.
+
 ## v1.11.2 — the resolver hijack, the discarded event identifier, and a parameter that did nothing
 
 Three findings from the 2026-08-30 real-hardware re-test. 

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "vmware-debug"
-version = "1.11.2"
+version = "1.11.3"
 description = "VMware diagnostic brain — read-only incident triage, log/event correlation, and root-cause routing across the VMware skill family"
 readme = "README.md"
 license = "MIT"
@@ -21,7 +21,7 @@ dependencies = [
     "typer>=0.12,<1.0",
     "rich>=13.0,<15.0",
     "mcp[cli]>=1.10,<2.0",
-    "vmware-policy>=1.11.0,<2.0",
+    "vmware-policy>=1.12.0,<2.0",
     # Imported directly by ops/cases/grading.py for the rules file. It has
     # always arrived transitively via vmware-policy; a package should
     # declare what it imports rather than rely on a dependency's dependency.

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/server.json

@@ -7,12 +7,12 @@
     "url": "https://github.com/vmware-skills/VMware-Debug",
     "source": "github"
   },
-  "version": "1.11.2",
+  "version": "1.11.3",
   "packages": [
     {
       "registryType": "pypi",
       "identifier": "vmware-debug",
-      "version": "1.11.2",
+      "version": "1.11.3",
       "runtimeHint": "uvx",
       "packageArguments": [
         {

vmware_debug-1.11.3/tests/__init__.py

@@ -0,0 +1,9 @@
+"""Test package.
+
+``tests/eval/`` and ``tests/eval/regression/`` were already packages while this
+file was missing, so ``from tests.eval.regression._vsphere_event_vocabulary
+import ...`` resolved only where the namespace-package fallback happened to
+work. It did not on Windows, and the whole suite -- 398 tests -- failed at
+collection. The other repos that use cross-module test imports all carry this
+file; this one is the half-built package.
+"""

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/eval/capability/_scoring.py

@@ -132,13 +132,13 @@ class ScoreBoard:
             ),
             "scores": merged,
         }
-        path.write_text(json.dumps(payload, indent=2, sort_keys=True) + "\n")
+        path.write_text(json.dumps(payload, indent=2, sort_keys=True) + "\n", encoding="utf-8")
 
 
 def previous_scores(path: Path = SCORES_PATH) -> dict[str, Any]:
     """Load the last recorded run, or ``{}`` on a first run / unreadable file."""
     try:
-        return json.loads(path.read_text()).get("scores", {})
+        return json.loads(path.read_text(encoding="utf-8")).get("scores", {})
     except (OSError, ValueError):
         return {}
 

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/eval/regression/test_capability_grader.py

@@ -184,7 +184,7 @@ def test_an_error_returned_via_a_local_is_still_seen(tmp_path):
         "    except Exception as exc:\n"
         "        payload = {'error': str(exc), 'hint': _HINT}\n"
         "        return payload\n"
-    )
+    , encoding="utf-8")
     sites = list(_error_returns_in_server(server_dir=tmp_path, module=None))
 
     assert len(sites) == 2, f"a returned local was not seen: {sites}"
@@ -219,7 +219,7 @@ def test_a_partial_run_does_not_erase_metrics_it_did_not_measure(tmp_path):
     partial.add(Score(name="alpha", value=2, maximum=2))
     partial.write(path)
 
-    scores = json.loads(path.read_text())["scores"]
+    scores = json.loads(path.read_text(encoding="utf-8"))["scores"]
     assert set(scores) == {"alpha", "beta"}, "a partial run deleted a metric"
     assert scores["alpha"]["value"] == 2, "the fresh measurement must win"
     assert not scores["alpha"].get("stale"), "a freshly measured metric is not stale"
@@ -238,7 +238,7 @@ def test_an_empty_run_leaves_the_baseline_alone(tmp_path):
     seeded.write(path)
 
     ScoreBoard().write(path)
-    assert set(json.loads(path.read_text())["scores"]) == {"alpha"}
+    assert set(json.loads(path.read_text(encoding="utf-8"))["scores"]) == {"alpha"}
 
 
 def test_credits_a_real_subcommand_under_a_group():

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/eval/regression/test_environment_resolver_isolation.py

@@ -51,7 +51,7 @@ def _sibling_resolver(target: str) -> str | None:
 @pytest.fixture
 def engine(tmp_path):
     path = tmp_path / "rules.yaml"
-    path.write_text(_RULES)
+    path.write_text(_RULES, encoding="utf-8")
     return PolicyEngine(path)
 
 

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/eval/regression/test_investigation_metrics.py

@@ -233,7 +233,7 @@ def test_every_unreachable_source_carries_a_remedy():
 def _kb(tmp_path, name, applies_block):
     p = tmp_path / "vmware" / "knowledge" / "kb" / name
     p.parent.mkdir(parents=True, exist_ok=True)
-    p.write_text(f"---\nid: {name}\n{applies_block}---\nbody\n")
+    p.write_text(f"---\nid: {name}\n{applies_block}---\nbody\n", encoding="utf-8")
 
 
 def _scoped_case(versions):

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/test_case_conclusion.py

@@ -54,18 +54,18 @@ class TestRecording:
 
     def test_conclusion_md_gains_an_entry(self, case):
         record_grade(case, grade_case(case), at="2026-08-28T09:30:00Z")
-        body = (case_dir(case) / "conclusion.md").read_text()
+        body = (case_dir(case) / "conclusion.md").read_text(encoding="utf-8")
         assert "candidate" in body.lower()
         assert "2026-08-28T09:30:00Z" in body
 
     def test_the_entry_names_the_rules_that_produced_it(self, case):
         record_grade(case, grade_case(case), at="2026-08-28T09:30:00Z")
-        assert "grading_rules.yaml" in (case_dir(case) / "conclusion.md").read_text()
+        assert "grading_rules.yaml" in (case_dir(case) / "conclusion.md").read_text(encoding="utf-8")
 
     def test_the_placeholder_is_gone_once_a_grade_exists(self, case):
-        assert "Not graded yet" in (case_dir(case) / "conclusion.md").read_text()
+        assert "Not graded yet" in (case_dir(case) / "conclusion.md").read_text(encoding="utf-8")
         record_grade(case, grade_case(case), at="2026-08-28T09:30:00Z")
-        assert "Not graded yet" not in (case_dir(case) / "conclusion.md").read_text()
+        assert "Not graded yet" not in (case_dir(case) / "conclusion.md").read_text(encoding="utf-8")
 
 
 class TestHistoryIsAppendOnly:
@@ -108,7 +108,7 @@ class TestHistoryIsAppendOnly:
     def test_every_entry_survives_in_the_file(self, case):
         for i, at in enumerate(["09:30", "09:40", "09:50"]):
             record_grade(case, grade_case(case), at=f"2026-08-28T{at}:00Z")
-        body = (case_dir(case) / "conclusion.md").read_text()
+        body = (case_dir(case) / "conclusion.md").read_text(encoding="utf-8")
         for at in ["09:30", "09:40", "09:50"]:
             assert f"2026-08-28T{at}:00Z" in body
 

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/test_case_evidence.py

@@ -73,7 +73,7 @@ class TestRecording:
 
     def test_the_payload_is_stored_beside_the_record(self, case):
         e = record_evidence(case, an_evidence(), payload={"rows": [1, 2, 3]})
-        body = json.loads((case_dir(case) / "evidence" / f"{e.evidence_id}.json").read_text())
+        body = json.loads((case_dir(case) / "evidence" / f"{e.evidence_id}.json").read_text(encoding="utf-8"))
         assert body["payload"] == {"rows": [1, 2, 3]}
 
     def test_recording_against_a_case_that_does_not_exist_is_an_error(self):
@@ -95,7 +95,7 @@ class TestTheTimeBasisContract:
 
     def test_an_unknown_time_source_is_explicit_not_missing(self, case):
         e = record_evidence(case, an_evidence(time_source=None, clock_skew_s=None))
-        body = json.loads((case_dir(case) / "evidence" / f"{e.evidence_id}.json").read_text())
+        body = json.loads((case_dir(case) / "evidence" / f"{e.evidence_id}.json").read_text(encoding="utf-8"))
         assert "time_source" in body and body["time_source"] is None
         assert "clock_skew_s" in body and body["clock_skew_s"] is None
 

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/test_case_grading.py

@@ -42,7 +42,7 @@ def mounted_kb(tmp_path):
     """
     p = tmp_path / "vmware" / "knowledge" / "kb" / "KB-fixture.md"
     p.parent.mkdir(parents=True, exist_ok=True)
-    p.write_text("---\nid: KB-fixture\napplies_to:\n  product: vsphere\n---\nbody\n")
+    p.write_text("---\nid: KB-fixture\napplies_to:\n  product: vsphere\n---\nbody\n", encoding="utf-8")
     return p
 
 
@@ -229,7 +229,7 @@ class TestRules:
         site.parent.mkdir(parents=True)
         site.write_text(
             "grades:\n  probable:\n    min_independent_sources: 1\n    blocked_by_gaps: false\n"
-        )
+        , encoding="utf-8")
         record_evidence(case, evidence(skill="vmware-monitor"))
         r = grade_case(case)
         assert r.grade == "probable"
@@ -242,7 +242,7 @@ class TestRules:
     def test_a_broken_site_file_is_an_error_not_a_silent_fallback(self, tmp_path):
         site = tmp_path / "vmware" / "investigation" / "grading_rules.yaml"
         site.parent.mkdir(parents=True)
-        site.write_text("grades: [this is a list not a mapping")
+        site.write_text("grades: [this is a list not a mapping", encoding="utf-8")
         with pytest.raises(ValueError, match="grading_rules.yaml"):
             load_rules()
 
@@ -263,7 +263,7 @@ class TestTheCeilingIsMeasuredNotAsserted:
     def test_a_populated_knowledge_library_raises_the_ceiling(self, case, tmp_path):
         kb = tmp_path / "vmware" / "knowledge" / "kb"
         kb.mkdir(parents=True)
-        (kb / "KB-2026-0417.md").write_text("---\nid: KB-2026-0417\n---\nbody\n")
+        (kb / "KB-2026-0417.md").write_text("---\nid: KB-2026-0417\n---\nbody\n", encoding="utf-8")
         r = grade_case(case)
         assert r.ceiling == "confirmed"
 
@@ -288,7 +288,7 @@ class TestTheRulesFileHasNoDecorativeKnobs:
     def _site(self, tmp_path, body):
         p = tmp_path / "vmware" / "investigation" / "grading_rules.yaml"
         p.parent.mkdir(parents=True, exist_ok=True)
-        p.write_text(body)
+        p.write_text(body, encoding="utf-8")
 
     def test_relaxing_the_prerequisite_actually_relaxes_it(self, case, tmp_path, mounted_kb):
         """One decisive source, no corroboration: Confirmed under the relaxed

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/test_case_hypotheses.py

@@ -67,7 +67,7 @@ class TestRegistering:
 
     def test_it_lands_in_hypotheses_md_as_readable_text(self, case):
         add_hypothesis(case, "failing NVMe device")
-        body = (case_dir(case) / "hypotheses.md").read_text()
+        body = (case_dir(case) / "hypotheses.md").read_text(encoding="utf-8")
         assert "H1" in body and "failing NVMe device" in body
         assert "_Empty." not in body
 

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/test_case_readiness.py

@@ -122,7 +122,7 @@ class TestReadiness:
         assert readiness()["classes"]["knowledge"]["available"] is False
         kb = tmp_path / "vmware" / "knowledge" / "kb"
         kb.mkdir(parents=True)
-        (kb / "KB-1.md").write_text("---\napplies_to: {}\n---\nbody\n")
+        (kb / "KB-1.md").write_text("---\napplies_to: {}\n---\nbody\n", encoding="utf-8")
         assert readiness()["classes"]["knowledge"]["available"] is True
 
     def test_hardware_stays_unavailable_no_matter_what_is_installed(self):

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/test_case_store.py

@@ -82,18 +82,18 @@ class TestCreate:
         differently to whoever opens the folder."""
         d = case_dir(create_case(a_scope(), at=AT).case_id)
         for name in ("timeline.md", "hypotheses.md", "conclusion.md"):
-            body = (d / name).read_text()
+            body = (d / name).read_text(encoding="utf-8")
             assert body.strip(), f"{name} is blank"
             assert body.lstrip().startswith("#"), f"{name} has no heading"
 
     def test_jsonl_and_json_start_valid_not_blank(self):
         d = case_dir(create_case(a_scope(), at=AT).case_id)
-        assert (d / "plan.jsonl").read_text() == ""
-        assert json.loads((d / "gaps.json").read_text()) == {"gaps": []}
+        assert (d / "plan.jsonl").read_text(encoding="utf-8") == ""
+        assert json.loads((d / "gaps.json").read_text(encoding="utf-8")) == {"gaps": []}
 
     def test_refuses_to_overwrite_an_existing_case(self):
         case = create_case(a_scope(), at=AT)
-        (case_dir(case.case_id) / "evidence" / "E001.json").write_text("{}")
+        (case_dir(case.case_id) / "evidence" / "E001.json").write_text("{}", encoding="utf-8")
         with pytest.raises(CaseExists) as e:
             create_case(a_scope(), at=AT)
         assert case.case_id in str(e.value)
@@ -133,7 +133,7 @@ class TestLoad:
 
     def test_a_case_whose_scope_is_unreadable_is_an_error_not_an_empty_scope(self):
         case = create_case(a_scope(), at=AT)
-        (case_dir(case.case_id) / "scope.json").write_text("{ this is not json")
+        (case_dir(case.case_id) / "scope.json").write_text("{ this is not json", encoding="utf-8")
         with pytest.raises(ValueError, match="scope.json"):
             load_case(case.case_id)
 

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/test_case_timeline_close.py

@@ -97,7 +97,7 @@ class TestTimeline:
     def test_it_writes_timeline_md(self, case):
         submit(case, payload=EVENTS)
         build_case_timeline(case)
-        body = (case_dir(case) / "timeline.md").read_text()
+        body = (case_dir(case) / "timeline.md").read_text(encoding="utf-8")
         assert "esxi-03" in body
         assert "_Empty." not in body
 
@@ -132,7 +132,7 @@ class TestClose:
     def test_the_grade_is_recorded_at_close_not_assumed(self, case):
         out = close_case(case, at=AT)
         assert out["grade"] == "candidate"
-        assert "candidate" in (case_dir(case) / "conclusion.md").read_text().lower()
+        assert "candidate" in (case_dir(case) / "conclusion.md").read_text(encoding="utf-8").lower()
 
     def test_closing_twice_is_refused_rather_than_rewriting_the_record(self, case):
         close_case(case, at=AT)

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/tests/test_knowledge.py

@@ -38,7 +38,7 @@ def home(tmp_path, monkeypatch):
 def write(home, rel, body):
     p = home / rel
     p.parent.mkdir(parents=True, exist_ok=True)
-    p.write_text(body)
+    p.write_text(body, encoding="utf-8")
     return p
 
 

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/uv.lock

@@ -1293,7 +1293,7 @@ wheels = [
 
 [[package]]
 name = "vmware-debug"
-version = "1.11.2"
+version = "1.11.3"
 source = { editable = "." }
 dependencies = [
     { name = "mcp", extra = ["cli"] },
@@ -1316,7 +1316,7 @@ requires-dist = [
     { name = "pyyaml", specifier = ">=6.0,<7.0" },
     { name = "rich", specifier = ">=13.0,<15.0" },
     { name = "typer", specifier = ">=0.12,<1.0" },
-    { name = "vmware-policy", specifier = ">=1.11.0,<2.0" },
+    { name = "vmware-policy", specifier = ">=1.12.0,<2.0" },
 ]
 
 [package.metadata.requires-dev]
@@ -1328,14 +1328,14 @@ dev = [
 
 [[package]]
 name = "vmware-policy"
-version = "1.11.0"
+version = "1.12.0"
 source = { registry = "https://pypi.org/simple" }
 dependencies = [
     { name = "pyyaml" },
     { name = "rich" },
     { name = "typer" },
 ]
-sdist = { url = "https://files.pythonhosted.org/packages/3e/8a/9be3e8f6605e740f067e69d0dceb34a888b89559f35510090459d004c66f/vmware_policy-1.11.0.tar.gz", hash = "sha256:f497be95b4558bf714980ccf05219a9a40b48173de845de4d1562e3679b2ce91", size = 149532, upload-time = "2026-08-30T14:40:05.844Z" }
+sdist = { url = "https://files.pythonhosted.org/packages/24/ce/6d3eda9c10648e34facba0eaeff5a58991246a54e6b0f203a499440e3939/vmware_policy-1.12.0.tar.gz", hash = "sha256:4f9e7b80020eec628d65955ba0d61d8080bde730a257e940912917fd3d5ba36d", size = 156796, upload-time = "2026-08-31T00:20:49.206Z" }
 wheels = [
-    { url = "https://files.pythonhosted.org/packages/a8/89/8ea2b904057b6a3b344237763a1821079e896fcc6adccd699455586a01e8/vmware_policy-1.11.0-py3-none-any.whl", hash = "sha256:de7d434e1c8113acfd91aacac0a76c05d20b3fc0c147ab02b29782413ec8905b", size = 56005, upload-time = "2026-08-30T14:40:04.205Z" },
+    { url = "https://files.pythonhosted.org/packages/f7/58/c13af514586e3d6ba18144305b421e0e3830cc33e58e87b447c4821509e7/vmware_policy-1.12.0-py3-none-any.whl", hash = "sha256:cbd4dbc454678604fedf4c82e3e6bcecb696e3a1ce6c0ba795bb4dac134c1d17", size = 60569, upload-time = "2026-08-31T00:20:47.595Z" },
 ]

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/vmware_debug/__init__.py

@@ -5,4 +5,4 @@ into a unified timeline, detect spikes, rank root-cause hypotheses, and route
 remediation to vmware-aiops / vmware-pilot. Never writes; never executes fixes.
 """
 
-__version__ = "1.11.2"
+__version__ = "1.11.3"

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/vmware_debug/cli.py

@@ -20,6 +20,30 @@ from rich.table import Table
 from vmware_debug import __version__
 from vmware_debug.mcp.tools import incident_timeline, list_symptom_categories
 
+
+def _harden_console_encoding() -> None:
+    """Never let one unrepresentable glyph kill a command.
+
+    On a console whose encoding cannot carry the characters we print -- cp936 on
+    the Chinese Windows boxes this family is tested on, or any ASCII locale --
+    ``print`` raises ``UnicodeEncodeError`` and the whole command dies with a
+    traceback. ``--help`` died that way in four repos. A mangled dash is a
+    cosmetic loss; a dead ``--help`` is an outage, so the error handler is
+    relaxed rather than the vocabulary narrowed.
+
+    Best effort: ``reconfigure`` is absent when stdout has been replaced by a
+    plain object (pytest capture, some MCP hosts), and losing the hardening
+    there is not worth an exception at import.
+    """
+    for stream in (sys.stdout, sys.stderr):
+        try:
+            stream.reconfigure(errors="backslashreplace")
+        except (AttributeError, ValueError, OSError):
+            pass
+
+
+_harden_console_encoding()
+
 app = typer.Typer(
     add_completion=False,
     help="VMware diagnostic brain — read-only incident triage and root-cause routing.",
@@ -54,7 +78,7 @@ def triage(
     top_n: int = typer.Option(5, help="Max hypotheses to return."),
 ) -> None:
     """Correlate a set of pre-collected events into a ranked incident timeline."""
-    raw = events_file.read_text() if events_file else sys.stdin.read()
+    raw = events_file.read_text(encoding="utf-8") if events_file else sys.stdin.read()
     try:
         events = json.loads(raw)
     except json.JSONDecodeError as exc:

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/vmware_debug/ops/cases/conclusion.py

@@ -76,7 +76,7 @@ def record_grade(case_id: str, result: GradeResult, at: str) -> GradeEntry:
     )
 
     try:
-        body = path.read_text()
+        body = path.read_text(encoding="utf-8")
     except OSError as exc:
         raise CaseError(
             f"Cannot read conclusion.md for case {case_id}: {exc}. The case "
@@ -84,13 +84,13 @@ def record_grade(case_id: str, result: GradeResult, at: str) -> GradeEntry:
         ) from exc
 
     body = body.replace(_PLACEHOLDER + "\n\n", "").replace(_PLACEHOLDER + "\n", "")
-    path.write_text(body.rstrip("\n") + "\n\n" + _render(entry))
+    path.write_text(body.rstrip("\n") + "\n\n" + _render(entry), encoding="utf-8")
 
     index_path = d / "case.json"
-    index = json.loads(index_path.read_text())
+    index = json.loads(index_path.read_text(encoding="utf-8"))
     index["grade"] = result.grade
     index["graded_at"] = at
-    index_path.write_text(json.dumps(index, indent=2, ensure_ascii=False) + "\n")
+    index_path.write_text(json.dumps(index, indent=2, ensure_ascii=False) + "\n", encoding="utf-8")
 
     return entry
 
@@ -137,7 +137,7 @@ def grade_history(case_id: str) -> tuple[GradeEntry, ...]:
             f"incomplete; restore it from your copy rather than recreating it."
         )
     out = []
-    for m in _ENTRY_RE.finditer(path.read_text()):
+    for m in _ENTRY_RE.finditer(path.read_text(encoding="utf-8")):
         try:
             d = json.loads(m.group("json"))
         except json.JSONDecodeError as exc:

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/vmware_debug/ops/cases/evidence.py

@@ -215,7 +215,7 @@ def record_evidence(case_id: str, evidence: Evidence, payload: Any = None) -> Ev
     body = stamped.to_json()
     if payload is not None:
         body["payload"] = payload
-    path.write_text(json.dumps(body, indent=2, ensure_ascii=False) + "\n")
+    path.write_text(json.dumps(body, indent=2, ensure_ascii=False) + "\n", encoding="utf-8")
     return stamped
 
 
@@ -243,7 +243,7 @@ def load_evidence(case_id: str) -> tuple[Evidence, ...]:
     out = []
     for path in sorted(d.glob("E*.json")):
         try:
-            out.append(Evidence.from_json(json.loads(path.read_text())))
+            out.append(Evidence.from_json(json.loads(path.read_text(encoding="utf-8"))))
         except (OSError, json.JSONDecodeError) as exc:
             raise ValueError(
                 f"Evidence file {path.name} in case {case_id} is unreadable: "
@@ -271,7 +271,7 @@ def record_gap(case_id: str, gap: Gap) -> Gap:
         gap_id=_next_id([g.gap_id for g in existing], "G"),
     )
     payload = {"gaps": [g.to_json() for g in existing] + [stamped.to_json()]}
-    (d / _GAPS).write_text(json.dumps(payload, indent=2, ensure_ascii=False) + "\n")
+    (d / _GAPS).write_text(json.dumps(payload, indent=2, ensure_ascii=False) + "\n", encoding="utf-8")
     return stamped
 
 
@@ -284,7 +284,7 @@ def load_gaps(case_id: str) -> tuple[Gap, ...]:
     """
     path = _case_dir_or_raise(case_id) / _GAPS
     try:
-        body = json.loads(path.read_text())
+        body = json.loads(path.read_text(encoding="utf-8"))
     except OSError as exc:
         raise ValueError(f"Cannot read {_GAPS} for case {case_id}: {exc}.") from exc
     except json.JSONDecodeError as exc:

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/vmware_debug/ops/cases/hypotheses.py

@@ -55,7 +55,7 @@ def load_hypotheses(case_id: str) -> tuple[Hypothesis, ...]:
     """Every registered hypothesis, in registration order."""
     path = _path(case_id)
     try:
-        body = path.read_text()
+        body = path.read_text(encoding="utf-8")
     except OSError as exc:
         raise CaseError(f"Cannot read {_FILE} for case {case_id}: {exc}") from exc
     out = []
@@ -97,12 +97,12 @@ def add_hypothesis(case_id: str, statement: str, at: str = "") -> Hypothesis:
         statement=statement.strip(),
         at=at,
     )
-    body = path.read_text().replace(_PLACEHOLDER_MARK, "").rstrip("\n")
+    body = path.read_text(encoding="utf-8").replace(_PLACEHOLDER_MARK, "").rstrip("\n")
     machine = json.dumps(
         {"id": h.hypothesis_id, "statement": h.statement, "at": h.at},
         ensure_ascii=False,
     )
-    path.write_text(f"{body}\n\n## {h.hypothesis_id} — {h.statement}\n\n{_ENTRY.format(machine)}\n")
+    path.write_text(f"{body}\n\n## {h.hypothesis_id} — {h.statement}\n\n{_ENTRY.format(machine)}\n", encoding="utf-8")
     return h
 
 

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/vmware_debug/ops/cases/knowledge.py

@@ -298,7 +298,7 @@ def _parse(path: Path, section: str) -> tuple[list[KnowledgeEntry], str | None]:
             return [one(row) for row in csv.DictReader(raw.splitlines(), delimiter=delim)], None
         if ext in (".txt", ".log"):
             sidecar = path.with_suffix(".yaml")
-            d = yaml.safe_load(sidecar.read_text()) if sidecar.is_file() else {}
+            d = yaml.safe_load(sidecar.read_text(encoding="utf-8")) if sidecar.is_file() else {}
             return [one(d or {}, raw.strip())], None
     except Exception as exc:
         return [], f"{path.name}: {type(exc).__name__}: {exc}"

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/vmware_debug/ops/cases/store.py

@@ -65,7 +65,7 @@ def case_dir(case_id: str) -> Path:
 
 
 def _write_json(path: Path, payload: object) -> None:
-    path.write_text(json.dumps(payload, indent=2, ensure_ascii=False) + "\n")
+    path.write_text(json.dumps(payload, indent=2, ensure_ascii=False) + "\n", encoding="utf-8")
 
 
 def _read_json(path: Path, what: str) -> dict:
@@ -76,7 +76,7 @@ def _read_json(path: Path, what: str) -> dict:
     it would be at its most costly here.
     """
     try:
-        raw = path.read_text()
+        raw = path.read_text(encoding="utf-8")
     except OSError as exc:
         raise CaseError(
             f"Cannot read {what} at {path}: {exc}. The case directory may have "
@@ -125,10 +125,10 @@ def create_case(scope: Scope, at: str) -> Case:
     (d / "evidence").mkdir()
 
     _write_json(d / "scope.json", scope.to_json())
-    (d / "plan.jsonl").write_text("")
+    (d / "plan.jsonl").write_text("", encoding="utf-8")
     _write_json(d / "gaps.json", {"gaps": []})
     for name, body in _SKELETON_MD.items():
-        (d / name).write_text(body)
+        (d / name).write_text(body, encoding="utf-8")
     _write_json(
         d / "case.json",
         {"case_id": case_id, "state": "open", "opened_at": at, "grade": None},

{vmware_debug-1.11.2 → vmware_debug-1.11.3}/vmware_debug/ops/cases/timeline.py

@@ -52,7 +52,7 @@ def build_case_timeline(
     for item in evidence:
         path = d / f"{item.evidence_id}.json"
         try:
-            body = json.loads(path.read_text())
+            body = json.loads(path.read_text(encoding="utf-8"))
         except (OSError, json.JSONDecodeError):
             rejected.append(f"{item.evidence_id}: payload unreadable")
             continue
@@ -125,7 +125,7 @@ def _write_timeline_md(case_id: str, result: dict, rows: list) -> None:
     if result.get("rejected"):
         lines += ["", "## Could not be read", ""]
         lines += [f"- {r}" for r in result["rejected"]]
-    (case_dir(case_id) / "timeline.md").write_text("\n".join(lines) + "\n")
+    (case_dir(case_id) / "timeline.md").write_text("\n".join(lines) + "\n", encoding="utf-8")
 
 
 def close_case(case_id: str, at: str) -> dict[str, Any]:
@@ -146,11 +146,11 @@ def close_case(case_id: str, at: str) -> dict[str, Any]:
     open_gaps = [g.gap_id for g in load_gaps(case_id) if g.blocks]
     index_path = case_dir(case_id) / "case.json"
     try:
-        index = json.loads(index_path.read_text())
+        index = json.loads(index_path.read_text(encoding="utf-8"))
     except (OSError, json.JSONDecodeError) as exc:
         raise CaseError(f"Cannot read case.json for {case_id}: {exc}") from exc
     index.update({"state": "closed", "closed_at": at, "grade": result.grade})
-    index_path.write_text(json.dumps(index, indent=2, ensure_ascii=False) + "\n")
+    index_path.write_text(json.dumps(index, indent=2, ensure_ascii=False) + "\n", encoding="utf-8")
 
     note = f"Closed at {result.grade}."
     if open_gaps: