From 1f7dc10993e7119e4fe40e942cf649f5b5fb8eef Mon Sep 17 00:00:00 2001 From: Junghwan <70629228+shaun0927@users.noreply.github.com> Date: Mon, 25 May 2026 18:00:58 +0900 Subject: [PATCH] fix(canonical): call Result.is_ok/is_err as properties MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Result.is_ok and Result.is_err are @property, and Result has no unwrap_err method — only the .error property. The canonical live-run assertion was calling them as methods, so OUROBOROS_RUN_CANONICAL=1 raised TypeError before any failure evidence could surface. Co-Authored-By: Claude Opus 4.7 (1M context) --- tests/canonical/test_canonical.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/canonical/test_canonical.py b/tests/canonical/test_canonical.py index 4b4006409..169f1f677 100644 --- a/tests/canonical/test_canonical.py +++ b/tests/canonical/test_canonical.py @@ -245,9 +245,9 @@ async def test_scenario_live_run_or_skip( result = await _invoke_ouroboros_auto(scenario, workdir) - assert result.is_ok(), ( + assert result.is_ok, ( f"{scenario.slug}: ouroboros_auto returned MCP error: " - f"{result.unwrap_err() if not result.is_ok() else 'unknown'}" + f"{result.error if result.is_err else 'unknown'}" ) tool_result = result.unwrap()