Font targetColor target
Highlight target
',
version_count: 1,
});
- await new Promise(resolve => setTimeout(resolve, 450));
}, suffix);
+ await page.waitForSelector('#doc-email-richbody p');
async function selectParagraph(index) {
await page.evaluate(index => {
const rich = document.querySelector('#doc-email-richbody');
const paragraph = rich.querySelectorAll('p')[index];
+ if (!paragraph) throw new Error(`Missing paragraph ${index}: ${rich.innerHTML}`);
rich.focus();
const range = document.createRange();
range.selectNodeContents(paragraph);
diff --git a/tests/test_document_rich_keyboard_shortcuts.py b/tests/test_document_rich_keyboard_shortcuts.py
index ae2130c00..532a58b5c 100644
--- a/tests/test_document_rich_keyboard_shortcuts.py
+++ b/tests/test_document_rich_keyboard_shortcuts.py
@@ -64,7 +64,8 @@ def test_rich_document_shortcuts_work_at_desktop_and_mobile_widths():
await selectText('Align target');
await page.keyboard.press('Control+Shift+e');
const center = await page.locator('#doc-email-richbody').evaluate(root => root.children[1].style.textAlign);
- await page.keyboard.press('Control+Shift+r');
+ await page.locator('[data-dd="align"]').click();
+ await page.locator('#doc-md-dd-menu .doc-overflow-item').filter({ hasText: 'Align right' }).click();
const right = await page.locator('#doc-email-richbody').evaluate(root => root.children[1].style.textAlign);
await page.keyboard.press('Control+Shift+j');
const justify = await page.locator('#doc-email-richbody').evaluate(root => root.children[1].style.textAlign);
diff --git a/tests/test_document_rich_slash_menu.py b/tests/test_document_rich_slash_menu.py
index 3006b54fb..c7a2fafea 100644
--- a/tests/test_document_rich_slash_menu.py
+++ b/tests/test_document_rich_slash_menu.py
@@ -52,7 +52,9 @@ def test_slash_menu_filters_converts_blocks_inserts_tables_and_fits_mobile():
current_content: 'Opening paragraph
',
version_count: 1,
});
- await new Promise(resolve => setTimeout(resolve, 450));
+ });
+ await page.waitForFunction(() => document.querySelectorAll('#doc-email-richbody p').length >= 2);
+ await page.evaluate(() => {
const block = document.querySelector('#doc-email-richbody p:last-child');
const range = document.createRange();
range.selectNodeContents(block);
@@ -111,6 +113,7 @@ def test_slash_menu_filters_converts_blocks_inserts_tables_and_fits_mobile():
});
await page.keyboard.type('/');
await page.waitForSelector('#doc-rich-slash-menu');
+ await page.mouse.move(1, 1);
await page.keyboard.press('End');
const mobile = await page.locator('#doc-rich-slash-menu').evaluate(el => {
const rect = el.getBoundingClientRect();
@@ -131,6 +134,7 @@ def test_slash_menu_filters_converts_blocks_inserts_tables_and_fits_mobile():
};
});
await page.keyboard.press('Home');
+ const homeOptions = await page.locator('.doc-rich-slash-label').allTextContents();
const homeLabel = await page.locator('.doc-rich-slash-item.is-active .doc-rich-slash-label').textContent();
await page.keyboard.press('Escape');
const escaped = await page.locator('#doc-rich-slash-menu').count() === 0;
@@ -142,7 +146,7 @@ def test_slash_menu_filters_converts_blocks_inserts_tables_and_fits_mobile():
popup: el.hasAttribute('aria-haspopup'),
}));
- console.log(JSON.stringify({ filtered, heading, noHeadingQuery, tableFiltered, table, noTableQuery, mobile, homeLabel, escaped, slashRemains, cleanedAria }));
+ console.log(JSON.stringify({ filtered, heading, noHeadingQuery, tableFiltered, table, noTableQuery, mobile, homeOptions, homeLabel, escaped, slashRemains, cleanedAria }));
await browser.close();
"""
result = subprocess.run(
@@ -169,7 +173,8 @@ def test_slash_menu_filters_converts_blocks_inserts_tables_and_fits_mobile():
assert mobile["activeLabel"] == "Image"
assert mobile["activeDescendant"] == mobile["activeId"]
assert mobile["scrollTop"] > 0
- assert data["homeLabel"] == "Text"
+ assert data["homeOptions"][0] == "Text"
+ assert data["homeLabel"] == data["homeOptions"][0], data
assert data["escaped"] is True
assert data["slashRemains"] is True
assert data["cleanedAria"] == {
diff --git a/tests/test_document_rich_toolbar_menus.py b/tests/test_document_rich_toolbar_menus.py
index 950a2c4f1..658d1b83f 100644
--- a/tests/test_document_rich_toolbar_menus.py
+++ b/tests/test_document_rich_toolbar_menus.py
@@ -66,8 +66,8 @@ def test_mobile_toolbar_uses_native_momentum_and_distinct_activation_tokens():
id: 'toggle-doc', title: 'Toggle menu', language: 'richtext',
current_content: 'Toggle target
', version_count: 1,
});
- await new Promise(resolve => setTimeout(resolve, 450));
});
+ await page.waitForSelector('#doc-email-richbody p');
const toggle = page.locator('button[data-dd="font"]');
await toggle.click();
@@ -138,7 +138,9 @@ def test_mobile_toolbar_menu_preserves_selection_and_restores_focus():
current_content: 'Paragraph
',
version_count: 1,
});
- await new Promise(resolve => setTimeout(resolve, 450));
+ });
+ await page.waitForSelector('#doc-email-richbody p');
+ await page.evaluate(() => {
const paragraph = document.querySelector('#doc-email-richbody p');
const range = document.createRange();
range.selectNodeContents(paragraph);
@@ -251,7 +253,9 @@ def test_rich_toolbar_menus_track_live_formatting_values():
current_content: 'Stateful text
',
version_count: 1,
});
- await new Promise(resolve => setTimeout(resolve, 450));
+ });
+ await page.waitForSelector('#doc-email-richbody p');
+ await page.evaluate(() => {
const paragraph = document.querySelector('#doc-email-richbody p');
const range = document.createRange();
range.selectNodeContents(paragraph);
diff --git a/tests/test_document_toolbar_order.py b/tests/test_document_toolbar_order.py
index 4abdafb8a..ddf6355ab 100644
--- a/tests/test_document_toolbar_order.py
+++ b/tests/test_document_toolbar_order.py
@@ -18,8 +18,8 @@ def test_toolbar_groups_define_writing_actions_before_view_controls():
assert ordering.index("name: 'inline-basic'") < ordering.index("name: 'inline-color'")
assert ordering.index("name: 'inline-color'") < ordering.index("name: 'alignment'")
assert ordering.index("name: 'alignment'") < ordering.index("name: 'spacing'")
- assert ordering.index("name: 'spacing'") < ordering.index("name: 'link'")
- assert ordering.index("name: 'link'") < ordering.index("name: 'paragraph'")
+ assert ordering.index("'[data-md=\"link\"]'") < ordering.index("name: 'alignment'")
+ assert ordering.index("name: 'spacing'") < ordering.index("name: 'paragraph'")
assert ordering.index("name: 'paragraph'") < ordering.index("name: 'insert'")
assert ordering.index("name: 'insert'") < ordering.index("name: 'document'")
assert ordering.index("name: 'document'") < ordering.index("name: 'view'")
@@ -49,8 +49,8 @@ def test_rich_toolbar_rendered_order_is_stable_on_desktop_and_mobile():
current_content: 'Writing tools
',
version_count: 1,
});
- await new Promise(resolve => setTimeout(resolve, 450));
}, suffix);
+ await page.waitForSelector('#doc-email-richbody p');
const state = await page.evaluate(() => {
const toolbar = document.querySelector('#md-toolbar-items');
@@ -58,7 +58,8 @@ def test_rich_toolbar_rendered_order_is_stable_on_desktop_and_mobile():
const key = item => item.dataset.dd || item.dataset.md || item.id;
const visible = controls.filter(item => {
const style = getComputedStyle(item);
- return style.display !== 'none' && style.visibility !== 'hidden';
+ return !item.classList.contains('md-toolbar-sep')
+ && style.display !== 'none' && style.visibility !== 'hidden';
});
return {
all: controls.map(item => [item.dataset.toolbarGroup, key(item)]),
@@ -90,7 +91,7 @@ def test_rich_toolbar_rendered_order_is_stable_on_desktop_and_mobile():
expected_groups = [
"display-size", "type", "inline-basic", "inline-color", "alignment", "spacing",
- "link", "paragraph", "insert", "inline-rich", "document", "view",
+ "paragraph", "insert", "inline-rich", "document", "view",
]
expected_separators = [
"display-size-type",
@@ -98,8 +99,7 @@ def test_rich_toolbar_rendered_order_is_stable_on_desktop_and_mobile():
"inline-basic-inline-color",
"inline-color-alignment",
"alignment-spacing",
- "spacing-link",
- "link-paragraph",
+ "spacing-paragraph",
"paragraph-insert",
"insert-inline-rich",
"inline-rich-document",
@@ -110,7 +110,7 @@ def test_rich_toolbar_rendered_order_is_stable_on_desktop_and_mobile():
assert list(dict.fromkeys(groups)) == expected_groups
assert state["separators"] == expected_separators
assert state["visible"][:6] == [
- "doc-fontsize-btn",
+ "doc-ai-writing-btn",
"heading",
"font",
"textsize",
@@ -118,11 +118,11 @@ def test_rich_toolbar_rendered_order_is_stable_on_desktop_and_mobile():
"italic",
]
assert state["visible"].index("link") < state["visible"].index("list")
- assert state["visible"].index("list") < state["visible"].index("md-toolbar-attach-btn")
+ assert state["visible"].index("md-toolbar-attach-btn") < state["visible"].index("list")
assert state["visible"].index("md-toolbar-attach-btn") < state["visible"].index("doc-find-toolbar-btn")
assert state["visible"].index("subscript") > state["visible"].index("md-toolbar-attach-btn")
if "doc-outline-toolbar-btn" in state["visible"]:
- assert state["visible"].index("doc-fontsize-btn") < state["visible"].index("doc-outline-toolbar-btn")
+ assert state["visible"].index("doc-ai-writing-btn") < state["visible"].index("doc-outline-toolbar-btn")
assert state["pageOverflow"] == 0
assert data["mobile"]["toolbarOverflow"] is True
diff --git a/tests/test_escape_inner_layers.py b/tests/test_escape_inner_layers.py
index a9915531b..e63d3fc6c 100644
--- a/tests/test_escape_inner_layers.py
+++ b/tests/test_escape_inner_layers.py
@@ -22,7 +22,9 @@ def test_rich_escape_closes_toolbar_then_selection_badge() -> None:
id: 'escape-doc', title: 'Escape', language: 'richtext',
current_content: 'Selected sentence for testing
', version_count: 1,
});
- await new Promise(resolve => setTimeout(resolve, 700));
+ });
+ await page.waitForSelector('#doc-email-richbody p');
+ await page.evaluate(() => {
const rich = document.querySelector('#doc-email-richbody');
const range = document.createRange();
range.selectNodeContents(rich.querySelector('p'));
@@ -32,7 +34,8 @@ def test_rich_escape_closes_toolbar_then_selection_badge() -> None:
rich.focus();
rich.dispatchEvent(new MouseEvent('mouseup', { bubbles: true, clientX: 20, clientY: 20 }));
});
- await page.waitForTimeout(100);
+ await page.waitForSelector('#doc-rich-selection-toolbar');
+ await page.waitForSelector('#doc-selection-badge', { state: 'attached' });
const before = await page.evaluate(() => ({
toolbar: !!document.querySelector('#doc-rich-selection-toolbar'),
badge: document.querySelector('#doc-selection-badge')?.style.display ?? 'missing',
@@ -63,7 +66,7 @@ def test_rich_escape_closes_toolbar_then_selection_badge() -> None:
data = json.loads(result.stdout)
assert data['before']['toolbar'] is True
assert data['before']['badge'] != 'missing'
- assert data['afterOne'] == {'toolbar': False, 'badge': ''}
+ assert data['afterOne'] == {'toolbar': False, 'badge': 'none'}
assert data['afterTwo'] == {'toolbar': False, 'badge': 'none'}
@@ -95,7 +98,6 @@ def test_email_escape_closes_inner_states_without_closing_library() -> None:
const state = (await import('/static/js/emailLibrary/state.js')).state;
const modal = document.querySelector('#email-lib-modal');
state._selectMode = true;
- modal.classList.add('email-reading');
});
await page.keyboard.press('Escape');
const select = await page.evaluate(async () => ({
@@ -103,6 +105,7 @@ def test_email_escape_closes_inner_states_without_closing_library() -> None:
select: (await import('/static/js/emailLibrary/state.js')).state._selectMode,
reading: document.querySelector('#email-lib-modal')?.classList.contains('email-reading'),
}));
+ await page.evaluate(() => document.querySelector('#email-lib-modal').classList.add('email-reading'));
await page.keyboard.press('Escape');
const reading = await page.evaluate(() => ({
modal: !!document.querySelector('#email-lib-modal'),
@@ -121,5 +124,5 @@ def test_email_escape_closes_inner_states_without_closing_library() -> None:
assert result.returncode == 0, result.stderr
data = json.loads(result.stdout)
assert data['settings'] == {'modal': True, 'settings': False}
- assert data['select'] == {'modal': True, 'select': False, 'reading': True}
+ assert data['select'] == {'modal': True, 'select': False, 'reading': False}
assert data['reading'] == {'modal': True, 'reading': False}
diff --git a/tests/test_research_synapse_ui.py b/tests/test_research_synapse_ui.py
index 2a337a359..fb27c1abd 100644
--- a/tests/test_research_synapse_ui.py
+++ b/tests/test_research_synapse_ui.py
@@ -8,11 +8,11 @@ def test_active_research_visualizer_has_live_hierarchy_and_balanced_layout():
source = (ROOT / "static/js/researchSynapse.js").read_text(encoding="utf-8")
assert "Live research map" in source
- assert "rs-phase-chip" in source
+ assert 'starting…' in source
assert "rs-root-halo" in source
assert "const angles = [-90, 90, 180, 0" in source
assert "rs-node-tone-${tone}" in source
- assert "wrap.classList.add(`rs-phase-${phase}`)" in source
+ assert "if (phase) wrap.classList.add(`rs-phase-${phase}`)" in source
assert "const delta = total - previousTotal" in source
assert "rs-source-node" in source
assert "_rememberSource(extra.title, extra.url)" in source
From 9bc1cdcab0595d810bcc03730e18229737d9b145 Mon Sep 17 00:00:00 2001
From: Alexandre Teixeira <111787685+alteixeira20@users.noreply.github.com>
Date: Tue, 22 Sep 2026 23:27:08 +0100
Subject: [PATCH 09/14] test(harness): isolate imports and stabilize browser
and DNS fixtures
---
tests/conftest.py | 12 +++---
tests/streaming/markdownHarness.mjs | 2 +-
.../test_consolidate_memory_explicit_drops.py | 2 +-
tests/test_copy_message_strips_thinking_js.py | 2 +-
tests/test_fenced_inline_args.py | 36 +-----------------
tests/test_fenced_invoke_no_raw_xml.py | 36 +-----------------
tests/test_integrations_url_join.py | 1 +
tests/test_markdown_lazy_lib_loading_js.py | 2 +-
tests/test_markdown_rendering_js.py | 2 +-
tests/test_review_document_conversion.py | 4 +-
tests/test_startup_session_bootstrap_js.py | 18 ++++-----
tests/test_tile_manager_snap_zones_js.py | 3 +-
tests/test_upload_multifile.py | 2 +-
tests/test_web_search_raw_json_tool_call.py | 35 +----------------
tests/test_web_search_time_filter.py | 38 +------------------
15 files changed, 31 insertions(+), 164 deletions(-)
diff --git a/tests/conftest.py b/tests/conftest.py
index 0e8faebde..3dab8dbd1 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -112,8 +112,7 @@ def _serve_test_static():
sock.close()
if is_bound:
- yield
- return
+ raise RuntimeError("port 7011 is already in use; browser tests require this worktree's static server")
root_dir = Path(__file__).resolve().parent.parent
@@ -134,12 +133,11 @@ def _serve_test_static():
class _Server(socketserver.TCPServer):
allow_reuse_address = True
+ server = _Server(("127.0.0.1", 7011), _Handler)
+ thread = threading.Thread(target=server.serve_forever, daemon=True)
+ thread.start()
try:
- server = _Server(("127.0.0.1", 7011), _Handler)
- thread = threading.Thread(target=server.serve_forever, daemon=True)
- thread.start()
yield
+ finally:
server.shutdown()
server.server_close()
- except Exception:
- yield
diff --git a/tests/streaming/markdownHarness.mjs b/tests/streaming/markdownHarness.mjs
index af34b3372..430599719 100644
--- a/tests/streaming/markdownHarness.mjs
+++ b/tests/streaming/markdownHarness.mjs
@@ -26,7 +26,7 @@ export async function loadMarkdown() {
globalThis.MutationObserver = class { observe() {} };
let src = fs.readFileSync(path.join(REPO, 'static/js/markdown.js'), 'utf8');
- src = src.replace(/import uiModule from ['"]\.\/ui\.js(?:\?[^'"]*)?['"];?/, '');
+ src = src.replace(/import uiModule from ['"]\.\/ui\.js(?:[?#][^'"]*)?['"];?/, '');
src = src.replace(
/import \{ splitTableRow \} from ['"]\.\/markdown\/tableRow\.js['"];/,
() => `function splitTableRow(row){return (row||'').replace(/^\\s*\\|/,'').replace(/\\|\\s*$/,'').split('|').map((c)=>c.trim());}`,
diff --git a/tests/test_consolidate_memory_explicit_drops.py b/tests/test_consolidate_memory_explicit_drops.py
index b03e651bc..d52fd7580 100644
--- a/tests/test_consolidate_memory_explicit_drops.py
+++ b/tests/test_consolidate_memory_explicit_drops.py
@@ -36,7 +36,7 @@ def test_omitted_memory_survives_only_explicit_drop(monkeypatch):
monkeypatch.setattr(src.memory, "MemoryManager", _FakeMM)
monkeypatch.setattr(
src.task_endpoint, "resolve_task_candidates",
- lambda owner=None: [("http://x/v1", "model", {})],
+ lambda owner=None, **kwargs: [("http://x/v1", "model", {})],
)
async def fake_llm(_candidates, **kwargs):
diff --git a/tests/test_copy_message_strips_thinking_js.py b/tests/test_copy_message_strips_thinking_js.py
index 74a03f80c..86dea41e0 100644
--- a/tests/test_copy_message_strips_thinking_js.py
+++ b/tests/test_copy_message_strips_thinking_js.py
@@ -58,7 +58,7 @@ def _extract_thinking_blocks(text: str) -> dict:
let source = fs.readFileSync('./static/js/markdown.js', 'utf8');
source = source.replace(
- /import uiModule from ['"]\.\/ui\.js(?:\?[^'"]*)?['"];?/,
+ /import uiModule from ['"]\.\/ui\.js(?:[?#][^'"]*)?['"];?/,
''
);
source = source.replace(
diff --git a/tests/test_fenced_inline_args.py b/tests/test_fenced_inline_args.py
index 5de7d9a93..76d691010 100644
--- a/tests/test_fenced_inline_args.py
+++ b/tests/test_fenced_inline_args.py
@@ -5,41 +5,7 @@ with the args on the same line as the tag; the parser must execute those. The
relaxed tag pattern must NOT prefix-match longer fence tags: ```python3 is a
language hint, not a "python" tool call with content "3\n...".
"""
-import sys
-from unittest.mock import MagicMock
-
-# This module needs the real agent-tool stack; importing it pulls in heavy
-# DB/auth deps, so we stub those just long enough to import, then restore them.
-# We deliberately do NOT pop src.tool_execution: popping and re-importing it
-# rebinds the `src` package's `tool_execution` attribute, so a later
-# `import src.tool_execution as te` resolves to a different module object than
-# the one its functions live in - which silently breaks tests that monkeypatch
-# it (e.g. test_edit_file's admin gate) and breaks request-scoped ContextVars.
-_ABSENT = object()
-_AGENT_MODULES = ["src.agent_tools", "src.tool_parsing", "src.tool_schemas"]
-_STUBBED = [
- "sqlalchemy", "sqlalchemy.orm", "sqlalchemy.ext", "sqlalchemy.ext.declarative",
- "sqlalchemy.ext.hybrid", "sqlalchemy.sql", "sqlalchemy.sql.expression",
- "src.database", "core.models", "core.database", "core.auth",
-]
-_saved_stubs = {name: sys.modules.get(name, _ABSENT) for name in _STUBBED}
-
-for _mod in _AGENT_MODULES:
- sys.modules.pop(_mod, None)
-for _mod in _STUBBED:
- if _mod not in sys.modules:
- sys.modules[_mod] = MagicMock()
-
-import src.agent_tools # noqa: E402, F401
-from src.tool_parsing import parse_tool_blocks, strip_tool_blocks # noqa: E402
-
-# Drop the stubs we installed so they do not leak into later tests.
-for _name, _original in _saved_stubs.items():
- if _original is _ABSENT:
- sys.modules.pop(_name, None)
- else:
- sys.modules[_name] = _original
-
+from src.tool_parsing import parse_tool_blocks, strip_tool_blocks
def test_inline_args_on_tag_line_parse():
# The original bug: ```list_email_accounts {} (args on the tag line)
diff --git a/tests/test_fenced_invoke_no_raw_xml.py b/tests/test_fenced_invoke_no_raw_xml.py
index e8a66a76a..e9b6db441 100644
--- a/tests/test_fenced_invoke_no_raw_xml.py
+++ b/tests/test_fenced_invoke_no_raw_xml.py
@@ -3,41 +3,7 @@ can't be converted (e.g. a hyphenated/namespaced tool name that _XML_INVOKE_RE's
\\w+ won't match, or an unknown tool) must NOT fall through and ship the raw XML
to the code executor as if it were python/bash.
"""
-import sys
-from unittest.mock import MagicMock
-
-# This module needs the real agent-tool stack; importing it pulls in heavy
-# DB/auth deps, so we stub those just long enough to import, then restore them.
-# We deliberately do NOT pop src.tool_execution: popping and re-importing it
-# rebinds the `src` package's `tool_execution` attribute, so a later
-# `import src.tool_execution as te` resolves to a different module object than
-# the one its functions live in - which silently breaks tests that monkeypatch
-# it (e.g. test_edit_file's admin gate) and breaks request-scoped ContextVars.
-_ABSENT = object()
-_AGENT_MODULES = ["src.agent_tools", "src.tool_parsing", "src.tool_schemas"]
-_STUBBED = [
- "sqlalchemy", "sqlalchemy.orm", "sqlalchemy.ext", "sqlalchemy.ext.declarative",
- "sqlalchemy.ext.hybrid", "sqlalchemy.sql", "sqlalchemy.sql.expression",
- "src.database", "core.models", "core.database", "core.auth",
-]
-_saved_stubs = {name: sys.modules.get(name, _ABSENT) for name in _STUBBED}
-
-for _mod in _AGENT_MODULES:
- sys.modules.pop(_mod, None)
-for _mod in _STUBBED:
- if _mod not in sys.modules:
- sys.modules[_mod] = MagicMock()
-
-import src.agent_tools # noqa: E402, F401
-from src.tool_parsing import parse_tool_blocks # noqa: E402
-
-# Drop the stubs we installed so they do not leak into later tests.
-for _name, _original in _saved_stubs.items():
- if _original is _ABSENT:
- sys.modules.pop(_name, None)
- else:
- sys.modules[_name] = _original
-
+from src.tool_parsing import parse_tool_blocks
def test_unconvertible_invoke_in_fence_is_not_executed_as_code():
text = '```python\n\n1\n\n```'
diff --git a/tests/test_integrations_url_join.py b/tests/test_integrations_url_join.py
index a5e19722e..461b6e380 100644
--- a/tests/test_integrations_url_join.py
+++ b/tests/test_integrations_url_join.py
@@ -105,6 +105,7 @@ async def test_api_call_root_path_has_no_trailing_slash():
with (
patch.object(integrations, "_find_integration", return_value=DISCORD_INTEGRATION),
+ patch("src.url_safety._default_resolver", return_value=["93.184.216.34"]),
patch("httpx.AsyncClient", return_value=mock_client),
):
result = await integrations.execute_api_call(
diff --git a/tests/test_markdown_lazy_lib_loading_js.py b/tests/test_markdown_lazy_lib_loading_js.py
index 9a3b6e43b..ee334a828 100644
--- a/tests/test_markdown_lazy_lib_loading_js.py
+++ b/tests/test_markdown_lazy_lib_loading_js.py
@@ -102,7 +102,7 @@ globalThis.document = {
globalThis.MutationObserver = class { observe() {} };
let source = fs.readFileSync('./static/js/markdown.js', 'utf8');
-source = source.replace(/import uiModule from ['"]\.\/ui\.js(?:\?[^'"]*)?['"];?/, '');
+source = source.replace(/import uiModule from ['"]\.\/ui\.js(?:[?#][^'"]*)?['"];?/, '');
source = source.replace(
/import \{ splitTableRow \} from ['"]\.\/markdown\/tableRow\.js['"];/,
`function splitTableRow(row) {
diff --git a/tests/test_markdown_rendering_js.py b/tests/test_markdown_rendering_js.py
index 2391f98ba..10f70c8e5 100644
--- a/tests/test_markdown_rendering_js.py
+++ b/tests/test_markdown_rendering_js.py
@@ -53,7 +53,7 @@ def _run_markdown_case(markdown: str, render_expr: str = "mod.mdToHtml(input)",
let source = fs.readFileSync('./static/js/markdown.js', 'utf8');
source = source.replace(
- /import uiModule from ['"]\.\/ui\.js(?:\?[^'"]*)?['"];?/,
+ /import uiModule from ['"]\.\/ui\.js(?:[?#][^'"]*)?['"];?/,
''
);
source = source.replace(
diff --git a/tests/test_review_document_conversion.py b/tests/test_review_document_conversion.py
index 10499bbdf..4309b2c0a 100644
--- a/tests/test_review_document_conversion.py
+++ b/tests/test_review_document_conversion.py
@@ -114,6 +114,7 @@ async def test_docx_preview_runs_off_loop_and_checks_owner(monkeypatch, conversi
def test_imported_office_document_is_owned_at_first_commit(monkeypatch, tmp_path):
+ import importlib
from sqlalchemy import create_engine, event
from sqlalchemy.orm import sessionmaker
import src.database as database
@@ -122,7 +123,8 @@ def test_imported_office_document_is_owned_at_first_commit(monkeypatch, tmp_path
database.Base.metadata.create_all(engine)
factory = sessionmaker(bind=engine)
monkeypatch.setattr(database, "SessionLocal", factory)
- monkeypatch.setattr("src.agent_tools.document_tools.set_active_document", lambda doc_id: None)
+ document_tools = importlib.import_module("src.agent_tools.document_tools")
+ monkeypatch.setattr(document_tools, "set_active_document", lambda doc_id: None)
owners_at_commit = []
def inspect_new_rows(session):
owners_at_commit.extend(row.owner for row in session.new if isinstance(row, database.Document))
diff --git a/tests/test_startup_session_bootstrap_js.py b/tests/test_startup_session_bootstrap_js.py
index 3fe37748b..353646751 100644
--- a/tests/test_startup_session_bootstrap_js.py
+++ b/tests/test_startup_session_bootstrap_js.py
@@ -302,39 +302,39 @@ def results(tmp_path_factory):
source = _SESSIONS.read_text(encoding="utf-8")
versioned_rewrites = (
(
- r"import Storage from './storage\.js(?:\?[^']+)?';",
+ r"import Storage from './storage\.js(?:[?#][^']*)?';",
"import Storage from './storage.mjs';",
),
(
- r"import uiModule, \{ autoResize, styledPrompt \} from './ui\.js(?:\?[^']+)?';",
+ r"import uiModule, \{ autoResize, styledPrompt \} from './ui\.js(?:[?#][^']*)?';",
"import uiModule, { autoResize, styledPrompt } from './ui.mjs';",
),
(
- r"import chatRenderer from './chatRenderer\.js(?:\?[^']+)?';",
+ r"import chatRenderer from './chatRenderer\.js(?:[?#][^']*)?';",
"import chatRenderer from './chatRenderer.mjs';",
),
(
- r"import \{ providerLogo \} from './providers\.js(?:\?[^']+)?';",
+ r"import \{ providerLogo \} from './providers\.js(?:[?#][^']*)?';",
"import { providerLogo } from './providers.mjs';",
),
(
- r"import \{ initModelPicker, updateModelPicker \} from './modelPicker\.js(?:\?[^']+)?';",
+ r"import \{ initModelPicker, updateModelPicker \} from './modelPicker\.js(?:[?#][^']*)?';",
"import { initModelPicker, updateModelPicker } from './modelPicker.mjs';",
),
(
- r"import themeModule from './theme\.js(?:\?[^']+)?';",
+ r"import themeModule from './theme\.js(?:[?#][^']*)?';",
"import themeModule from './theme.mjs';",
),
(
- r"import spinnerModule from './spinner\.js(?:\?[^']+)?';",
+ r"import spinnerModule from './spinner\.js(?:[?#][^']*)?';",
"import spinnerModule from './spinner.mjs';",
),
(
- r"import \{ actionMenuRank, orderActionMenuItems, SELECT_MENU_ICON \} from './actionMenuOrder\.js(?:\?[^']+)?';",
+ r"import \{ actionMenuRank, orderActionMenuItems, SELECT_MENU_ICON \} from './actionMenuOrder\.js(?:[?#][^']*)?';",
"import { actionMenuRank, orderActionMenuItems, SELECT_MENU_ICON } from './actionMenuOrder.mjs';",
),
(
- r"import \{ registerEscapeLayer, bindMenuDismiss \} from './escMenuStack\.js(?:\?[^']+)?';",
+ r"import \{ registerEscapeLayer, bindMenuDismiss \} from './escMenuStack\.js(?:[?#][^']*)?';",
"import { registerEscapeLayer, bindMenuDismiss } from './escMenuStack.mjs';",
),
)
diff --git a/tests/test_tile_manager_snap_zones_js.py b/tests/test_tile_manager_snap_zones_js.py
index 85f2b8da1..f97c8aa1e 100644
--- a/tests/test_tile_manager_snap_zones_js.py
+++ b/tests/test_tile_manager_snap_zones_js.py
@@ -20,11 +20,12 @@ def _run_tile_case():
innerWidth: 1200,
innerHeight: 800,
addEventListener() {{}},
+ getComputedStyle() {{ return {{ display: 'block' }}; }},
}};
let sidebarVisible = false;
const sidebar = {{
classList: {{ contains(name) {{ return name === 'hidden' ? !sidebarVisible : false; }} }},
- getBoundingClientRect() {{ return {{ right: 240 }}; }},
+ getBoundingClientRect() {{ return {{ left: 0, right: 240, width: 240, top: 0, bottom: 800 }}; }},
}};
globalThis.document = {{
readyState: 'loading',
diff --git a/tests/test_upload_multifile.py b/tests/test_upload_multifile.py
index 2e40948e6..cf692e610 100644
--- a/tests/test_upload_multifile.py
+++ b/tests/test_upload_multifile.py
@@ -191,7 +191,7 @@ async def test_chat_image_upload_is_added_to_gallery(tmp_path, monkeypatch):
up.setup_upload_routes(h)
endpoint = _endpoint(up.router)
- result = await endpoint(_request(user="alice"), [_image_upload()])
+ result = await endpoint(_request(user="alice"), [_image_upload()], gallery_id=None)
uploaded = result["files"][0]
assert uploaded["gallery_id"]
diff --git a/tests/test_web_search_raw_json_tool_call.py b/tests/test_web_search_raw_json_tool_call.py
index 57abc47d7..09a5b57ba 100644
--- a/tests/test_web_search_raw_json_tool_call.py
+++ b/tests/test_web_search_raw_json_tool_call.py
@@ -9,41 +9,8 @@ That is an intended tool call in non-native/textual tool mode, but older parsing
only recognized fenced blocks, [TOOL_CALL], XML invoke, and tool_code markup.
"""
import json
-import sys
-from unittest.mock import MagicMock
-
-# This module needs the real agent-tool stack; importing it pulls in heavy
-# DB/auth deps, so we stub those just long enough to import, then restore them.
-# We deliberately do NOT pop src.tool_execution: popping and re-importing it
-# rebinds the `src` package's `tool_execution` attribute, so a later
-# `import src.tool_execution as te` resolves to a different module object than
-# the one its functions live in - which silently breaks tests that monkeypatch
-# it (e.g. test_edit_file's admin gate) and breaks request-scoped ContextVars.
-_ABSENT = object()
-_AGENT_MODULES = ["src.agent_tools", "src.tool_parsing", "src.tool_schemas"]
-_STUBBED = [
- "sqlalchemy", "sqlalchemy.orm", "sqlalchemy.ext", "sqlalchemy.ext.declarative",
- "sqlalchemy.ext.hybrid", "sqlalchemy.sql", "sqlalchemy.sql.expression",
- "src.database", "core.models", "core.database", "core.auth",
-]
-_saved_stubs = {name: sys.modules.get(name, _ABSENT) for name in _STUBBED}
-
-for _mod in _AGENT_MODULES:
- sys.modules.pop(_mod, None)
-for _mod in _STUBBED:
- if _mod not in sys.modules:
- sys.modules[_mod] = MagicMock()
-
-import src.agent_tools # noqa: E402, F401
-from src.tool_parsing import parse_tool_blocks, strip_tool_blocks # noqa: E402
-
-# Drop the stubs we installed so they do not leak into later tests.
-for _name, _original in _saved_stubs.items():
- if _original is _ABSENT:
- sys.modules.pop(_name, None)
- else:
- sys.modules[_name] = _original
+from src.tool_parsing import parse_tool_blocks, strip_tool_blocks
def test_raw_json_after_web_search_phrase_runs_as_web_search():
text = (
diff --git a/tests/test_web_search_time_filter.py b/tests/test_web_search_time_filter.py
index d54369413..88a42336a 100644
--- a/tests/test_web_search_time_filter.py
+++ b/tests/test_web_search_time_filter.py
@@ -6,43 +6,9 @@ web_search branch emitted a bare query string and dropped time_filter. These pin
that a valid filter is passed through as JSON, while plain/invalid cases stay a
bare string (back-compat).
"""
-import sys
-from unittest.mock import MagicMock
-
-# This module needs the real agent-tool stack; importing it pulls in heavy
-# DB/auth deps, so we stub those just long enough to import, then restore them.
-# We deliberately do NOT pop src.tool_execution: popping and re-importing it
-# rebinds the `src` package's `tool_execution` attribute, so a later
-# `import src.tool_execution as te` resolves to a different module object than
-# the one its functions live in - which silently breaks tests that monkeypatch
-# it (e.g. test_edit_file's admin gate) and breaks request-scoped ContextVars.
-_ABSENT = object()
-_AGENT_MODULES = ["src.agent_tools", "src.tool_parsing", "src.tool_schemas"]
-_STUBBED = [
- "sqlalchemy", "sqlalchemy.orm", "sqlalchemy.ext", "sqlalchemy.ext.declarative",
- "sqlalchemy.ext.hybrid", "sqlalchemy.sql", "sqlalchemy.sql.expression",
- "src.database", "core.models", "core.database", "core.auth",
-]
-_saved_stubs = {name: sys.modules.get(name, _ABSENT) for name in _STUBBED}
-
-for _mod in _AGENT_MODULES:
- sys.modules.pop(_mod, None)
-for _mod in _STUBBED:
- if _mod not in sys.modules:
- sys.modules[_mod] = MagicMock()
-
-import json # noqa: E402
-
-import src.agent_tools # noqa: E402, F401
-from src.tool_schemas import function_call_to_tool_block # noqa: E402
-
-# Drop the stubs we installed so they do not leak into later tests.
-for _name, _original in _saved_stubs.items():
- if _original is _ABSENT:
- sys.modules.pop(_name, None)
- else:
- sys.modules[_name] = _original
+import json
+from src.tool_schemas import function_call_to_tool_block
def test_time_filter_is_preserved_as_json():
block = function_call_to_tool_block(
From c0a1cecc00606d5a6d6c6f055e6b87006cc3df17 Mon Sep 17 00:00:00 2001
From: Alexandre Teixeira <111787685+alteixeira20@users.noreply.github.com>
Date: Tue, 22 Sep 2026 23:27:17 +0100
Subject: [PATCH 10/14] test(baseline): align UI and model profile expectations
---
tests/test_action_menu_order.py | 4 +--
tests/test_builtin_themes_static.py | 2 +-
tests/test_chat_ttft_timer_static.py | 5 ++--
tests/test_clawmm_r47_malformed_write_body.py | 2 +-
tests/test_compare_mobile_tabs_static.py | 2 +-
tests/test_docs_no_orphan_images.py | 3 ++-
.../test_document_library_expanded_export.py | 2 +-
tests/test_document_library_mobile_footer.py | 10 ++++---
tests/test_editor_canvas_navigation_js.py | 4 ++-
tests/test_editor_layer_thumbnail_render.py | 4 +--
tests/test_editor_text_layers_js.py | 4 ++-
tests/test_editor_topbar_static.py | 18 ++++++-------
tests/test_email_attachment_search.py | 8 +++---
tests/test_email_folder_chip_static.py | 2 +-
tests/test_email_open_dedup_js.py | 7 ++---
tests/test_email_picker_exclusivity_static.py | 9 ++++---
tests/test_harness_version.py | 3 +--
tests/test_live_thinking_chat_integration.py | 6 ++---
tests/test_llm_core_ollama_thinking.py | 2 +-
...st_notes_mobile_checklist_toggle_static.py | 12 ++++++---
tests/test_notes_sidebar_new_note.py | 4 +--
tests/test_pr6020_rebase_regressions.py | 26 ++++++++++---------
tests/test_research_history_expansion.py | 23 +++++++++++-----
tests/test_research_panel_ui.py | 12 ++++-----
tests/test_tasks_completed_default_static.py | 3 ++-
tests/test_tool_approval_frontend_routing.py | 8 +++---
26 files changed, 108 insertions(+), 77 deletions(-)
diff --git a/tests/test_action_menu_order.py b/tests/test_action_menu_order.py
index c4e957054..84472662e 100644
--- a/tests/test_action_menu_order.py
+++ b/tests/test_action_menu_order.py
@@ -161,13 +161,13 @@ def test_email_settings_show_away_account_and_compact_display_controls() -> None
assert "enabled ? 'Show' : 'Hide'" in source
assert "email-settings-inline-link" in source
assert "email-auto-reply-exclude" not in source
- assert "_emailWritingStyleHtml(writingStyle) + _emailDisplaySettingsHtml()" in source
+ assert "_emailWritingStyleHtml(writingStyle) + _emailDisplaySettingsHtml(cfg)" in source
assert ".email-settings-status.is-success" in style
assert "var(--color-success, #4caf50)" in style
assert ".email-style-settings-extract svg" in style
assert "export async function mountEmailSettings(host)" in source
assert "_openGlobalEmailSettings('show-tags')" in source
- assert source.count('class="admin-card email-settings-section') == 4
+ assert source.count('class="admin-card email-settings-section') == 5
assert 'id="settings-email-default-card"' in (ROOT / "static/index.html").read_text(encoding="utf-8")
assert "multipleAccounts" in source
diff --git a/tests/test_builtin_themes_static.py b/tests/test_builtin_themes_static.py
index dde1514fd..f1fbcee60 100644
--- a/tests/test_builtin_themes_static.py
+++ b/tests/test_builtin_themes_static.py
@@ -8,7 +8,7 @@ THEME_JS = (ROOT / "static/js/theme.js").read_text(encoding="utf-8")
def test_five_distinct_builtin_themes_are_available() -> None:
expected = {
- "eclipse": "constellations",
+ "eclipse": "starfield-depth",
"porcelain": "dots",
"arcade": "synapse",
"blueprint": "dots",
diff --git a/tests/test_chat_ttft_timer_static.py b/tests/test_chat_ttft_timer_static.py
index d5d28b430..303f66699 100644
--- a/tests/test_chat_ttft_timer_static.py
+++ b/tests/test_chat_ttft_timer_static.py
@@ -25,8 +25,9 @@ def test_measured_ttft_is_shown_in_message_stats():
def test_compact_footer_and_details_show_real_performance_counters():
assert "`${Number(tps).toFixed(2)} tok/s`" in RENDERER
- assert "`${Number(ttft).toFixed(3)}s TTFT`" in RENDERER
- assert "`${Number(injectedTokens).toLocaleString()} in`" in RENDERER
+ assert "const visibleTtft = metrics.client_ttft ?? metrics.time_to_first_token" in RENDERER
+ assert "${Number(visibleTtft).toFixed(3)}s" in RENDERER
+ assert "${Number(injectedTokens).toLocaleString()}" in RENDERER
assert 'Input' in RENDERER
assert 'Injected' in RENDERER
assert 'all rounds' not in RENDERER
diff --git a/tests/test_clawmm_r47_malformed_write_body.py b/tests/test_clawmm_r47_malformed_write_body.py
index ea2b36ee8..00a35b800 100644
--- a/tests/test_clawmm_r47_malformed_write_body.py
+++ b/tests/test_clawmm_r47_malformed_write_body.py
@@ -87,7 +87,7 @@ async def test_malformed_text_artifact_write_uses_one_bounded_raw_body_handoff(m
events = [json.loads(chunk[6:]) for chunk in raw if '[DONE]' not in chunk]
assert len(requests) == 2
assert 'tools' not in requests[1]
- assert requests[1]['max_tokens'] == 4096
+ assert requests[1]['max_tokens'] == 8192
assert len(executed) == 1
assert executed[0].tool_type == 'write_file'
assert executed[0].content == (
diff --git a/tests/test_compare_mobile_tabs_static.py b/tests/test_compare_mobile_tabs_static.py
index 15d8366ce..14270050b 100644
--- a/tests/test_compare_mobile_tabs_static.py
+++ b/tests/test_compare_mobile_tabs_static.py
@@ -12,7 +12,7 @@ def test_mobile_compare_mounts_accessible_tabs_without_removing_panes():
index = _read("static/js/compare/index.js")
panes = _read("static/js/compare/panes.js")
- assert "mountMobilePaneTabs(container, grid)" in index
+ assert "mountMobilePaneTabs(container, grid, (anchor) => _addPane(anchor))" in index
assert "role', 'tablist'" in panes
assert "role', 'tab'" in panes
assert "role', 'tabpanel'" in panes
diff --git a/tests/test_docs_no_orphan_images.py b/tests/test_docs_no_orphan_images.py
index f6ed43560..02e0c4494 100644
--- a/tests/test_docs_no_orphan_images.py
+++ b/tests/test_docs_no_orphan_images.py
@@ -83,7 +83,8 @@ def test_pages_site_owns_its_entrypoint_and_media():
assert REPO / "website/index.html" in website_files
assert REPO / "docs/index.html" not in docs_files
- assert not [p for p in docs_files if p.suffix.lower() in VIDEO_EXTS | {".md"}]
+ assert not [p for p in docs_files if p.suffix.lower() in VIDEO_EXTS]
+ assert not [p for p in docs_files if p.name in PUBLIC_GUIDES]
website_paths = {p.relative_to(REPO / "website").as_posix() for p in website_files}
assert PUBLIC_GUIDES <= website_paths
diff --git a/tests/test_document_library_expanded_export.py b/tests/test_document_library_expanded_export.py
index 812048752..9d9fc5ecc 100644
--- a/tests/test_document_library_expanded_export.py
+++ b/tests/test_document_library_expanded_export.py
@@ -18,7 +18,7 @@ def test_expanded_document_card_has_export_beside_clone():
def test_expanded_export_reuses_download_function_without_proxy_click():
- assert "const exportDocumentFile = async () =>" in DOC_LIBRARY_JS
+ assert "const exportDocumentFile = async (format = 'original') =>" in DOC_LIBRARY_JS
assert "await exportDocumentFile();" in DOC_LIBRARY_JS
assert "exportItem.click();" not in DOC_LIBRARY_JS
assert "exportItem.type = 'button';" in DOC_LIBRARY_JS
diff --git a/tests/test_document_library_mobile_footer.py b/tests/test_document_library_mobile_footer.py
index 186d50be1..e690ecc30 100644
--- a/tests/test_document_library_mobile_footer.py
+++ b/tests/test_document_library_mobile_footer.py
@@ -10,24 +10,26 @@ SOURCE = (ROOT / "static/js/documentLibrary.js").read_text(encoding="utf-8")
STYLE = (ROOT / "static/style.css").read_text(encoding="utf-8")
-def test_mobile_footer_exposes_open_and_more_only():
+def test_mobile_footer_exposes_delete_open_and_more():
assert "doclib-expanded-open-btn" in SOURCE
assert "doclib-expanded-mobile-more" in SOURCE
assert "label: 'Open in new chat'" in SOURCE
assert "'Open in original' : 'Open document'" in SOURCE
- assert "label: 'Export file'" in SOURCE
+ assert "label: 'Export file ›'" in SOURCE
+ assert "label: 'Original format'" in SOURCE
+ assert "label: 'Markdown (.md)'" in SOURCE
assert "'Restore document' : 'Archive document'" in SOURCE
assert "label: 'Delete document'" in SOURCE
- mobile_css = STYLE.split("The Documents preview footer only exposes Open and More", 1)[1]
+ mobile_css = STYLE.split("On phones, keep Delete explicit", 1)[1]
mobile_css = mobile_css.split("/* Chat top bar", 1)[0]
for hidden_action in (
- ".doclib-expanded-delete-btn",
".doclib-expanded-archive-btn",
".doclib-expanded-clone-btn",
".doclib-expanded-export-btn",
):
assert hidden_action in mobile_css
+ assert ".doclib-expanded-delete-btn {\n display: inline-flex !important" in mobile_css
assert ".doclib-expanded-mobile-more" in mobile_css
assert "display: inline-flex" in mobile_css
assert "box-sizing: border-box" in mobile_css
diff --git a/tests/test_editor_canvas_navigation_js.py b/tests/test_editor_canvas_navigation_js.py
index e61eaee05..f20568cbf 100644
--- a/tests/test_editor_canvas_navigation_js.py
+++ b/tests/test_editor_canvas_navigation_js.py
@@ -91,13 +91,15 @@ def test_apply_pan_updates_state_canvas_overlay_and_cursor_classes():
def test_hand_navigation_is_wired_to_toolbar_keyboard_and_offline_graph():
toolbar = (ROOT / "static/js/editor/build/toolbar.js").read_text(encoding="utf-8")
+ shortcuts = (ROOT / "static/js/editor/tool-shortcuts.js").read_text(encoding="utf-8")
keyboard = (ROOT / "static/js/editor/keyboard-shortcuts.js").read_text(encoding="utf-8")
events = (ROOT / "static/js/editor/canvas-events.js").read_text(encoding="utf-8")
editor = (ROOT / "static/js/galleryEditor.js").read_text(encoding="utf-8")
service_worker = (ROOT / "static/sw.js").read_text(encoding="utf-8")
assert "{ id: 'hand', label: 'Hand'" in toolbar
- assert "key: 'H'" in toolbar
+ assert "t.key = TOOL_SHORTCUTS[t.id]" in toolbar
+ assert "hand: 'H'" in shortcuts
assert "e.code === 'Space'" in keyboard
assert "setTemporaryPan?.(true)" in keyboard
assert "isDirectPanIntent(state.tool, state.spacePanActive" in events
diff --git a/tests/test_editor_layer_thumbnail_render.py b/tests/test_editor_layer_thumbnail_render.py
index 3af077197..aa9e283e1 100644
--- a/tests/test_editor_layer_thumbnail_render.py
+++ b/tests/test_editor_layer_thumbnail_render.py
@@ -20,8 +20,8 @@ def test_hover_thumbnail_uses_rendered_layer_output():
def test_inline_layer_rows_share_the_common_thumbnail_renderer():
panel = (ROOT / "static/js/editor/layer-panel.js").read_text()
- assert "const previewCanvas = renderLayer?.(layer) || layer.canvas;" in panel
- assert "const thumb = createInlineThumbnail(previewCanvas, `${layer.name} preview`);" in panel
+ assert "const thumb = createInlineThumbnail(() => renderLayer?.(layer) || layer.canvas, `${layer.name} preview`);" in panel
+ assert "thumb._refreshPreview = draw;" in panel
assert "thumbCtx.fillStyle = ((x / tile + y / tile) & 1)" not in panel
diff --git a/tests/test_editor_text_layers_js.py b/tests/test_editor_text_layers_js.py
index 9cd5f4815..bf61dd880 100644
--- a/tests/test_editor_text_layers_js.py
+++ b/tests/test_editor_text_layers_js.py
@@ -119,6 +119,7 @@ def test_rasterize_text_layer_clears_retained_metadata_only_for_text():
def test_text_tool_is_retained_across_editor_subsystems():
editor = (ROOT / "static/js/galleryEditor.js").read_text(encoding="utf-8")
toolbar = (ROOT / "static/js/editor/build/toolbar.js").read_text(encoding="utf-8")
+ shortcuts = (ROOT / "static/js/editor/tool-shortcuts.js").read_text(encoding="utf-8")
controls = (ROOT / "static/js/editor/build/controls.js").read_text(encoding="utf-8")
codec = (ROOT / "static/js/editor/document-codec.js").read_text(encoding="utf-8")
geometry = (ROOT / "static/js/editor/document-geometry.js").read_text(encoding="utf-8")
@@ -127,7 +128,8 @@ def test_text_tool_is_retained_across_editor_subsystems():
merge = (ROOT / "static/js/editor/wire-merge-buttons.js").read_text(encoding="utf-8")
assert "{ id: 'text', label: 'Text'" in toolbar
- assert "key: 'T'" in toolbar
+ assert "t.key = TOOL_SHORTCUTS[t.id]" in toolbar
+ assert "text: 'T'" in shortcuts
assert 'id="ge-text-section"' in controls
assert "if (state.tool === 'text') return _placeText(e);" in editor
assert "kind: l.kind || 'raster'" in editor
diff --git a/tests/test_editor_topbar_static.py b/tests/test_editor_topbar_static.py
index 0f26b42d5..8f2ed3d50 100644
--- a/tests/test_editor_topbar_static.py
+++ b/tests/test_editor_topbar_static.py
@@ -32,15 +32,15 @@ def test_stacked_labels_share_one_size_and_position_rule():
assert "top: 2px;" in STYLE
-def test_narrow_topbar_reflows_essential_actions_instead_of_clipping_them():
- assert "ge-topbar-overflow" in OVERFLOW
- assert "topbar.classList.add('ge-topbar-overflow')" in OVERFLOW
- assert "(max-width: 700px)" in OVERFLOW
- assert ".ge-topbar.ge-topbar-overflow" in STYLE
- assert "flex: 1 0 100%;" in STYLE
- right_start = STYLE.index(".ge-topbar-overflow .ge-topbar-right")
- overflow_right = STYLE[right_start:STYLE.index("}", right_start)]
- assert "justify-content: flex-start;" in overflow_right
+def test_narrow_topbar_scrolls_essential_actions_and_hides_ai_group_first():
+ assert "if (topbar.scrollWidth > topbar.clientWidth)" in OVERFLOW
+ assert "aiGroup.forEach(el => { el.style.display = 'none'; });" in OVERFLOW
+ assert "new ResizeObserver(() => syncOverflow())" in OVERFLOW
+ mobile = STYLE[STYLE.index("/* Editor topbar — scrolls horizontally") :]
+ topbar = mobile[mobile.index(".ge-topbar {") : mobile.index("}", mobile.index(".ge-topbar {"))]
+ assert "overflow-x: auto;" in topbar
+ assert "flex-wrap: nowrap;" in topbar
+ assert "justify-content: flex-start;" in topbar
def test_desktop_tool_rail_keeps_long_tool_names_readable():
diff --git a/tests/test_email_attachment_search.py b/tests/test_email_attachment_search.py
index 98dc39f78..02470334b 100644
--- a/tests/test_email_attachment_search.py
+++ b/tests/test_email_attachment_search.py
@@ -160,8 +160,10 @@ def test_attachment_cache_backfill_preserves_message_id(tmp_path, monkeypatch):
def test_single_email_tag_has_no_more_control():
library = open("static/js/emailLibrary.js", encoding="utf-8").read()
-
- assert 'class="email-tags-more email-tags-more-single"' not in library
+ group = library[library.index("function _emailTagGroupHtml("):library.index("function _fitEmailCardTags(")]
+ assert "if (visible.length === 1) return visible[0];" in group
+ assert "if (visible.length === 2) return visible.join('');" in group
+ assert group.index("if (visible.length === 2)") < group.index("email-tags-more-single")
def test_email_folder_and_filter_pickers_treat_their_buttons_as_inside_clicks():
@@ -181,7 +183,7 @@ def test_empty_reply_has_two_editable_rows_and_reply_survives_compact_toolbar():
fit_start = library.index("function _fitReaderActions")
fit_end = library.index("const _readerActionFitObserver", fit_start)
fit = library[fit_start:fit_end]
- assert '[data-act="ai-reply"], [data-act="reply-all"]' in fit
+ assert '[data-act="reply-all"], [data-act="forward"]' in fit
assert '[data-act="reply"]' not in fit
diff --git a/tests/test_email_folder_chip_static.py b/tests/test_email_folder_chip_static.py
index 41f6d0a3b..193cfa6aa 100644
--- a/tests/test_email_folder_chip_static.py
+++ b/tests/test_email_folder_chip_static.py
@@ -16,7 +16,7 @@ def test_folder_chip_stays_with_date_and_moves_down():
folder_rule_start = css.index(".email-folder-chip {", first_folder_rule + 1)
folder_rule = css[folder_rule_start:][:220]
assert "position: relative;" in folder_rule
- assert "top: 2px;" in folder_rule
+ assert "top: 4px;" in folder_rule
group_rule = css[css.index(".email-meta-date-group {"):][:180]
assert "display: inline-flex;" in group_rule
assert "white-space: nowrap;" in group_rule
diff --git a/tests/test_email_open_dedup_js.py b/tests/test_email_open_dedup_js.py
index b5fea966a..9e5bd9f1c 100644
--- a/tests/test_email_open_dedup_js.py
+++ b/tests/test_email_open_dedup_js.py
@@ -127,15 +127,16 @@ def test_attachment_warning_only_checks_authored_reply_text():
assert "text.split(/^>|^On .* wrote:/m)" not in helper
-def test_email_send_saves_recovery_draft_before_send_and_closes_only_on_success():
+def test_email_send_saves_recovery_draft_before_send_and_retains_it_on_failure():
source = (_REPO / "static/js/document.js").read_text(encoding="utf-8")
send = source[source.index("async function _sendEmail"):source.index("\n\n async function _saveDraft", source.index("async function _sendEmail"))]
assert "async function _saveEmailDraftForRecovery" in source
assert send.index("_saveEmailDraftForRecovery({") < send.index("fetch(`${API_BASE}/api/email/send`")
+ assert send.index("if (isLibraryOpen()) closeLibrary();") > send.index("const sendRequest = fetch(")
+ assert send.index("if (isLibraryOpen()) closeLibrary();") < send.index("const res = await sendRequest")
success_branch = send[send.index("if (data.success) {"):]
- assert "closeLibrary()" in success_branch
- assert send.index("closeLibrary()") > send.index("const res = await sendRequest")
+ assert "fetch(`${API_BASE}/api/document/${sendDocId}`, { method: 'DELETE' })" in success_branch
assert "Draft kept in Drafts." in send
assert "recoveryDraft.draft_uid" in send
diff --git a/tests/test_email_picker_exclusivity_static.py b/tests/test_email_picker_exclusivity_static.py
index 18f217cfa..365514f43 100644
--- a/tests/test_email_picker_exclusivity_static.py
+++ b/tests/test_email_picker_exclusivity_static.py
@@ -10,8 +10,11 @@ def test_email_folder_and_filter_pickers_are_exclusive_and_escape_safe():
assert source.count("const filterMenu = document.getElementById('email-filter-menu');") == 1
assert source.count("const folderMenu = document.getElementById('email-folder-menu');") == 1
- assert source.count("e.stopImmediatePropagation();") >= 2
- assert "_wireEmailPickerEscapeGuard();" in source
- assert "window.addEventListener('keydown'" in source
+ assert source.count("e.stopImmediatePropagation?.();") >= 2
+ assert "filterMenu?._dismiss?.();" in source
+ assert "folderMenu?._dismiss?.();" in source
+ escape_handler = source[source.index("state._libInnerEscHandler = (e) => {"):source.index("window.addEventListener('keydown', state._libInnerEscHandler, true)")]
+ assert "if (dismissTopMenu()) {" in escape_handler
+ assert "window.addEventListener('keydown', state._libInnerEscHandler, true)" in source
inbox = (ROOT / "static/js/emailInbox.js").read_text(encoding="utf-8")
assert re.search(r"from './emailLibrary\.js\?v=[A-Za-z0-9_-]+'", inbox)
diff --git a/tests/test_harness_version.py b/tests/test_harness_version.py
index d83c5fff0..999491e8f 100644
--- a/tests/test_harness_version.py
+++ b/tests/test_harness_version.py
@@ -5,10 +5,9 @@ import re
ROOT = Path(__file__).resolve().parents[1]
-def test_original_harness_version_is_canonical_semver() -> None:
+def test_harness_version_is_canonical_semver() -> None:
version = (ROOT / "HARNESS_VERSION").read_text(encoding="utf-8").strip()
- assert version == "0.20.18"
assert re.fullmatch(r"\d+\.\d+\.\d+", version)
diff --git a/tests/test_live_thinking_chat_integration.py b/tests/test_live_thinking_chat_integration.py
index ca6e1c864..e8636d310 100644
--- a/tests/test_live_thinking_chat_integration.py
+++ b/tests/test_live_thinking_chat_integration.py
@@ -145,15 +145,15 @@ def test_final_response_skips_duplicate_end_of_stream_render():
"// --- Final render (skip if stream was ever backgrounded or currently in background) ---",
"} // end if (!_isBgFinal)",
)
- assert "if (!terminalFinalResponseRendered) _renderStream();" in post_loop
+ assert "if (!terminalFinalResponseRendered && !_turnRendering.isVisible(roundHolder.querySelector('.body'), _terminalAnswerHtml(roundText, roundHolder.querySelector('.body')))) _renderStream();" in post_loop
assert "const finalDisplay = terminalFinalResponseRendered ? '' : _streamDisplayText" in post_loop
def test_only_rich_completed_turns_reload_saved_canonical_assistant_row():
assert "if (_canonicalTerminalSaved && _savedAssistantMessageId && !_pendingContinue)" not in _CHAT
assert "if (!_pendingContinue)" in _CHAT
- assert "fall back to its latest" in _CHAT
- assert ": [...history].reverse().find(msg => msg && msg.role === 'assistant');" in _CHAT
+ assert "if (!streamSessionId || !_savedAssistantMessageId) return false;" in _CHAT
+ assert "String(msg.metadata?._db_id || '') === _savedAssistantMessageId" in _CHAT
assert "const _needsCanonicalTurnRebuild" in _CHAT
assert "lastToolThread" in _CHAT
assert "_streamTurnMarker.remove();" in _CHAT
diff --git a/tests/test_llm_core_ollama_thinking.py b/tests/test_llm_core_ollama_thinking.py
index d7d1485ec..305cd6ac8 100644
--- a/tests/test_llm_core_ollama_thinking.py
+++ b/tests/test_llm_core_ollama_thinking.py
@@ -172,4 +172,4 @@ class TestThinkSuppression:
"qwen35-9b-tool-router-v4-firstaction-noschema-adapter",
)
assert "think" not in payload
- assert payload["max_tokens"] == 1024
+ assert payload["max_tokens"] == llm_core.LLMConfig.DEFAULT_MAX_TOKENS
diff --git a/tests/test_notes_mobile_checklist_toggle_static.py b/tests/test_notes_mobile_checklist_toggle_static.py
index 1ff792f38..eb8dfdaa9 100644
--- a/tests/test_notes_mobile_checklist_toggle_static.py
+++ b/tests/test_notes_mobile_checklist_toggle_static.py
@@ -87,6 +87,12 @@ def test_mobile_bulk_select_long_press_is_shared_across_card_types():
assert ".skill-card[data-skill-name]" in helper
assert ".task-card[data-id]" in helper
assert "const HOLD_MS = 450" in helper
- assert "body:has(#memory-select-btn.active)" in css
- assert "body:has(#skills-select-btn.active)" in css
- assert "body:has(#tasks-select-btn.active)" in css
+ shared_buttons = (
+ "#memory-select-btn,", "#skills-select-btn,",
+ "#notes-select-btn,", "#tasks-select-btn,",
+ )
+ shared_css = css.split("/* Shared bulk-selection trigger.", 1)[1]
+ selectors = re.findall(r":is\(([^)]*)\)(?:\.active)?::before", shared_css)
+ assert len(selectors) == 2
+ for selector in selectors:
+ assert all(button in selector for button in shared_buttons)
diff --git a/tests/test_notes_sidebar_new_note.py b/tests/test_notes_sidebar_new_note.py
index c60bbefb2..6caf9f9a3 100644
--- a/tests/test_notes_sidebar_new_note.py
+++ b/tests/test_notes_sidebar_new_note.py
@@ -9,7 +9,7 @@ def test_notes_sidebar_new_action_matches_library_structure_and_animation():
css = (ROOT / "static/style.css").read_text(encoding="utf-8")
assert 'class="list-item-plus-btn sidebar-new-item-btn" id="notes-new-note-btn"' in html
- assert 'note' in html
+ assert 'new' in html[html.index('id="notes-new-note-btn"'):]
assert "#tool-notes-btn:hover #notes-new-note-btn" in css
assert "#notes-new-note-btn:hover svg" in css
@@ -72,7 +72,7 @@ def test_notes_body_and_tag_arrow_are_nudged_down():
assert css.count("padding: 10px 8px 8px") >= 2
assert ".notes-pane-body .doclib-chip-scroll-arrow.right" in css
- assert "top: calc(50% - 2px) !important" in css
+ assert "top: calc(50% - 6px) !important" in css[css.index(".notes-pane-body .doclib-chip-scroll-arrow.right"):]
def test_notes_tag_strip_cannot_grow_into_blank_space():
diff --git a/tests/test_pr6020_rebase_regressions.py b/tests/test_pr6020_rebase_regressions.py
index cacd0ebc0..c46669a27 100644
--- a/tests/test_pr6020_rebase_regressions.py
+++ b/tests/test_pr6020_rebase_regressions.py
@@ -15,6 +15,11 @@ NOTES_TOOLS = {
"ask_user",
"update_plan",
}
+GENERAL_COMPACT_TOOLS = {"ask_user", "web_search", "web_fetch"}
+CALENDAR_COMPACT_TOOLS = {
+ "ask_user", "bash", "get_workspace", "manage_calendar", "manage_notes",
+ "python", "read_file", "web_fetch", "web_search",
+}
def _collect(generator):
@@ -102,9 +107,7 @@ def test_odysseus_notes_mode_clamps_without_overriding_caller_denials(monkeypatc
assert {"manage_notes", "manage_calendar", "manage_tasks"} <= route["disabled_tools"]
-def test_odysseus_general_mode_disables_every_tool(monkeypatch):
- from src.tool_policy import known_tool_names
-
+def test_odysseus_general_mode_uses_compact_core(monkeypatch):
prompt_calls, _ = _install_route_probe(monkeypatch)
_run_probe(
@@ -113,11 +116,10 @@ def test_odysseus_general_mode_disables_every_tool(monkeypatch):
)
route = prompt_calls[0]
- assert route["relevant_tools"] == set()
- assert known_tool_names() <= route["disabled_tools"]
+ assert route["relevant_tools"] == GENERAL_COMPACT_TOOLS
-def test_odysseus_calendar_intent_uses_notes_mode(monkeypatch):
+def test_odysseus_calendar_intent_uses_compact_calendar_route(monkeypatch):
prompt_calls, _ = _install_route_probe(monkeypatch)
_run_probe(
@@ -125,10 +127,10 @@ def test_odysseus_calendar_intent_uses_notes_mode(monkeypatch):
relevant_tools={"manage_notes", "manage_calendar", "manage_tasks", "bash"},
)
- assert prompt_calls[0]["relevant_tools"] == NOTES_TOOLS
+ assert prompt_calls[0]["relevant_tools"] == CALENDAR_COMPACT_TOOLS
-def test_odysseus_calendar_followup_keeps_notes_mode(monkeypatch):
+def test_odysseus_calendar_followup_keeps_compact_calendar_route(monkeypatch):
prompt_calls, _ = _install_route_probe(monkeypatch)
messages = [
{"role": "user", "content": "Add lunch tomorrow to my calendar."},
@@ -153,7 +155,7 @@ def test_odysseus_calendar_followup_keeps_notes_mode(monkeypatch):
relevant_tools={"manage_notes", "manage_calendar", "manage_tasks", "bash"},
)
- assert prompt_calls[0]["relevant_tools"] == NOTES_TOOLS
+ assert prompt_calls[0]["relevant_tools"] == CALENDAR_COMPACT_TOOLS
def test_agent_route_passes_workspace_to_system_prompt(monkeypatch):
@@ -169,7 +171,7 @@ def test_agent_route_passes_workspace_to_system_prompt(monkeypatch):
assert prompt_calls[0]["workspace"] == "/tmp/example-repo"
-def test_odysseus_qwen_temperature_is_capped_for_agent_requests(monkeypatch):
+def test_odysseus_compact_primary_uses_deterministic_temperature(monkeypatch):
_, stream_calls = _install_route_probe(monkeypatch)
_run_probe(
@@ -178,7 +180,7 @@ def test_odysseus_qwen_temperature_is_capped_for_agent_requests(monkeypatch):
temperature=1.2,
)
- assert stream_calls[0]["temperature"] == 0.2
+ assert stream_calls[0]["temperature"] == 0.0
def test_qwen_fallback_candidate_gets_capped_temperature(monkeypatch):
@@ -214,7 +216,7 @@ def test_non_qwen_fallback_keeps_requested_temperature(monkeypatch):
fallbacks=[("https://backup.example/v1", "gpt-4o", {})],
)
- assert stream_calls[0]["temperature"] == 0.2
+ assert stream_calls[0]["temperature"] == 0.0
factory = stream_calls[0]["candidate_request_factory"]
request = asyncio.run(factory(1, "https://backup.example/v1", "gpt-4o", {}))
assert request["kwargs"]["temperature"] == 1.2
diff --git a/tests/test_research_history_expansion.py b/tests/test_research_history_expansion.py
index 948dbdaca..d1c692f17 100644
--- a/tests/test_research_history_expansion.py
+++ b/tests/test_research_history_expansion.py
@@ -1,6 +1,7 @@
"""Regression coverage for full-height Deep Research history expansion."""
from pathlib import Path
+import re
ROOT = Path(__file__).resolve().parent.parent
@@ -11,7 +12,11 @@ def test_history_list_participates_in_shared_library_expansion_layout():
css = (ROOT / "static/style.css").read_text(encoding="utf-8")
assert 'id="research-past-list" class="doclib-grid memory-list research-jobs-list"' in panel
- assert "width:min(560px, 90vw);max-height:78vh" in panel
+ card = css[css.index("#research-pane .research-history-card {"):]
+ card = card[:card.index("}")]
+ assert "flex: 1 1 0;" in card
+ assert "min-height: 0;" in card
+ assert "overflow: hidden;" in card
assert 'class="memory-tab-panel research-tab-panel"' in panel
assert 'class="admin-card research-new-job"' in panel
assert 'id="research-history-filters" class="skills-summary-strip"' in panel
@@ -21,7 +26,11 @@ def test_history_list_participates_in_shared_library_expansion_layout():
assert "pane.classList.toggle('research-results-view', tab !== 'research')" in panel
assert "display: flex; flex-direction: column; gap: 10px;" in css
assert ".research-tabs {\n flex: 0 0 auto;\n margin: -4px -4px 0;" in css
- assert "font-size:11px; letter-spacing:0;" in css
+ title = css[css.index("#research-pane .research-history-title-row h2 {"):]
+ title = title[:title.index("}")]
+ assert "letter-spacing: 0;" in title
+ desc = css[css.index("#research-pane .research-history-desc {"):]
+ assert "font-size: 11px;" in desc[:desc.index("}")]
assert "font: inherit; font-size: 12px; cursor: pointer; text-align: left;" in css
assert "#research-pane .research-history-search-wrap .memory-search-input" in css
assert "height: 30px;" in css
@@ -65,8 +74,8 @@ def test_research_panel_uses_one_versioned_module_instance():
renderer = (ROOT / "static/js/chatRenderer.js").read_text(encoding="utf-8")
index = (ROOT / "static/index.html").read_text(encoding="utf-8")
- version = "20260902researchhistoryalign34"
- assert f"research/panel.js?v={version}" in app
- assert renderer.count(f"research/panel.js?v={version}") == 2
- assert f"style.css?v={version}" in index
- assert f"app.js?v={version}" in index
+ panel_version = re.search(r"research/panel\.js\?v=([^'\"]+)", app)
+ asset_version = re.search(r"/static/app\.js\?v=([^'\"]+)", index)
+ style_version = re.search(r"/static/style\.css\?v=([^'\"]+)", index)
+ assert panel_version and renderer.count(f"research/panel.js?v={panel_version.group(1)}") == 2
+ assert asset_version and style_version and asset_version.group(1) == style_version.group(1)
diff --git a/tests/test_research_panel_ui.py b/tests/test_research_panel_ui.py
index 0cfa5a4e3..304b273a7 100644
--- a/tests/test_research_panel_ui.py
+++ b/tests/test_research_panel_ui.py
@@ -25,18 +25,18 @@ def test_research_settings_use_custom_pickers_and_shared_provider_icons():
assert "provider === 'google' ? 'google_pse'" in icons
first_picker_offset = style.split(".research-setting:has(#research-rounds),", 1)[1].split("}", 1)[0]
assert ".research-setting:has(#research-category)" in first_picker_offset
- assert "top: -4px;" in first_picker_offset
+ assert "top: 0;" in first_picker_offset
def test_completed_research_keeps_primary_and_utility_actions_visible_without_format_icon():
panel = (ROOT / "static/js/research/panel.js").read_text(encoding="utf-8")
assert "research-job-format-icon" not in panel
- assert "Visual Report" in panel
- assert "Discuss" in panel
- assert 'data-action="copy" title="Copy report to clipboard"' in panel
- assert 'data-action="dismiss" title="Clear from list"' in panel
- assert 'data-action="delete" title="Delete from disk"' in panel
+ assert 'data-action="report" title="Open visual report"' in panel
+ assert 'data-action="chat" title="Open follow-up chat with this research as context"' in panel
+ assert "{ action: 'copy', icon: _copyIcon, label: 'Copy report' }" in panel
+ assert "{ action: 'dismiss', icon: _cancelIcon, label: 'Hide from list' }" in panel
+ assert "{ action: 'delete', icon: _trashIcon, label: 'Delete from disk', danger: true }" in panel
def test_research_format_survives_live_and_reconnected_jobs():
diff --git a/tests/test_tasks_completed_default_static.py b/tests/test_tasks_completed_default_static.py
index ae37bdedc..44ae48fac 100644
--- a/tests/test_tasks_completed_default_static.py
+++ b/tests/test_tasks_completed_default_static.py
@@ -40,7 +40,8 @@ def test_tasks_filter_chips_include_active_paused_switch():
assert "mkChip(`active (${activeCount})`, 'active', _taskStatusFilter === 'active', 'status');" in src
assert "mkChip(`paused (${pausedCount})`, 'paused', _taskStatusFilter === 'paused', 'status');" in src
assert "if (_taskStatusFilter && String(t.status || '').toLowerCase() !== _taskStatusFilter) return false;" in src
- assert "if (value === null) _taskStatusFilter = null;" in src
+ assert "_taskStatusFilter = _taskStatusFilter === value ? null : value;" in src
+ assert "_taskStatusFilter = null;" in src[src.index("if (kind === 'status')"):src.index("_renderList();", src.index("if (kind === 'status')"))]
def test_tasks_completed_view_exposes_active_paused_shortcuts():
diff --git a/tests/test_tool_approval_frontend_routing.py b/tests/test_tool_approval_frontend_routing.py
index ecf7f3b33..8b2f66d9a 100644
--- a/tests/test_tool_approval_frontend_routing.py
+++ b/tests/test_tool_approval_frontend_routing.py
@@ -155,12 +155,12 @@ def test_every_changed_approval_module_is_cache_busted_together():
assert found, f"missing cache-busted reference for {module_name}"
assert len(set(found)) == 1, f"split module graph for {module_name}: {found}"
- # These shared modules are imported throughout the graph. Keep their URL
- # canonical and unversioned; mixing a query URL with plain relative imports
- # creates a second singleton with separate state and listeners.
- for module_name in ("sessions.js", "ui.js", "memory.js", "markdown.js", "models.js"):
+ # Shared modules must have one URL apiece. ui.js is consistently versioned
+ # throughout the graph; the other shared modules remain unversioned.
+ for module_name in ("sessions.js", "memory.js", "markdown.js", "models.js"):
assert any(module_name in source for source in sources)
assert not versions(module_name), f"split module graph for {module_name}"
+ assert len(set(versions("ui.js"))) == 1
compare_stream = (root / "static/js/compare/stream.js").read_text(encoding="utf-8")
compare_vote = (root / "static/js/compare/vote.js").read_text(encoding="utf-8")
From 0012baa17c612293c8abffb0ad2c2ce46a6f2968 Mon Sep 17 00:00:00 2001
From: Alexandre Teixeira <111787685+alteixeira20@users.noreply.github.com>
Date: Tue, 22 Sep 2026 23:27:29 +0100
Subject: [PATCH 11/14] ci(trivy): free build cache before image scan
---
.github/workflows/container-trivy.yml | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/.github/workflows/container-trivy.yml b/.github/workflows/container-trivy.yml
index ad5674f18..fece5727c 100644
--- a/.github/workflows/container-trivy.yml
+++ b/.github/workflows/container-trivy.yml
@@ -62,6 +62,8 @@ jobs:
- name: Set up Buildx
uses: docker/setup-buildx-action@37fe631027851001ddb9b187196cc803df7f5f0e # v4.3.0
+ with:
+ driver: docker
# Build without pushing so a broken Dockerfile is caught here, and the
# exact image we ship is what gets scanned.
@@ -73,6 +75,9 @@ jobs:
load: true
tags: odysseus:ci
+ - name: Free build cache before vulnerability database download
+ run: docker builder prune --all --force
+
- name: Scan image with Trivy
uses: aquasecurity/trivy-action@ed142fd0673e97e23eac54620cfb913e5ce36c25 # v0.36.0
with:
@@ -103,6 +108,8 @@ jobs:
- name: Set up Buildx
uses: docker/setup-buildx-action@37fe631027851001ddb9b187196cc803df7f5f0e # v4.3.0
+ with:
+ driver: docker
- name: Build image
uses: docker/build-push-action@53b7df96c91f9c12dcc8a07bcb9ccacbed38856a # v7.3.0
@@ -112,6 +119,9 @@ jobs:
load: true
tags: odysseus:ci
+ - name: Free build cache before vulnerability database download
+ run: docker builder prune --all --force
+
- name: Scan image with Trivy
uses: aquasecurity/trivy-action@ed142fd0673e97e23eac54620cfb913e5ce36c25 # v0.36.0
with:
From c0b71a5cef32a9f9764999c9c50348e72e99692c Mon Sep 17 00:00:00 2001
From: Alexandre Teixeira <111787685+alteixeira20@users.noreply.github.com>
Date: Wed, 23 Sep 2026 00:10:50 +0100
Subject: [PATCH 12/14] fix(tools): constrain Python environment namespace
mount
---
src/agent_tools/subprocess_tools.py | 28 +++++++++-
tests/test_workspace_artifact_tool_floor.py | 61 ++++++++++++++++++++-
2 files changed, 87 insertions(+), 2 deletions(-)
diff --git a/src/agent_tools/subprocess_tools.py b/src/agent_tools/subprocess_tools.py
index 27ae30886..c8ffe9ddb 100644
--- a/src/agent_tools/subprocess_tools.py
+++ b/src/agent_tools/subprocess_tools.py
@@ -262,10 +262,36 @@ def _wrap_workspace_namespace(
# tools keep their installed packages without exposing the host /tmp.
if interpreter_prefix:
prefix = os.path.abspath(interpreter_prefix)
+ resolved_prefix = os.path.realpath(prefix)
mounted_roots = ("/usr", "/home", "/mnt")
- if os.path.isdir(prefix) and not any(
+ reserved_roots = {
+ "/", "/tmp", "/var", "/opt", "/etc", "/workspace",
+ "/root", "/run", "/proc", "/dev", "/sys", *mounted_roots,
+ }
+ already_visible = any(
prefix == root or prefix.startswith(root + os.sep)
for root in mounted_roots
+ )
+ # A prefix is trusted only when it names a specific interpreter tree.
+ # In particular, never overlay the private root, tmpfs, or workspace
+ # with a broad host directory. Reject symlinked prefixes too: bwrap
+ # would otherwise bind the resolved source at a different destination.
+ has_environment_layout = (
+ os.path.isfile(os.path.join(prefix, "pyvenv.cfg"))
+ or (
+ os.path.isfile(os.path.join(prefix, "bin", "python"))
+ and os.path.isdir(os.path.join(
+ prefix, "lib", f"python{sys.version_info.major}.{sys.version_info.minor}",
+ ))
+ )
+ )
+ if (
+ not already_visible
+ and prefix == resolved_prefix
+ and prefix not in reserved_roots
+ and len(prefix.split(os.sep)) >= 3
+ and os.path.isdir(prefix)
+ and has_environment_layout
):
parents = []
parent = os.path.dirname(prefix)
diff --git a/tests/test_workspace_artifact_tool_floor.py b/tests/test_workspace_artifact_tool_floor.py
index 1d65ea9e3..3de10155e 100644
--- a/tests/test_workspace_artifact_tool_floor.py
+++ b/tests/test_workspace_artifact_tool_floor.py
@@ -2194,6 +2194,9 @@ def test_python_loaded_code_sees_virtual_workspace_alias(monkeypatch, tmp_path):
"""Absolute /workspace paths must work inside generated Python scripts."""
import asyncio
import shutil
+ import sys
+ import venv
+ from types import SimpleNamespace
if not shutil.which("bwrap"):
return
@@ -2203,10 +2206,21 @@ def test_python_loaded_code_sees_virtual_workspace_alias(monkeypatch, tmp_path):
from src.agent_tools import subprocess_tools
from src import tool_execution
workspace = tmp_path
+ environment = tmp_path / "confined-venv"
+ venv.EnvBuilder(with_pip=False).create(environment)
+ monkeypatch.setattr(subprocess_tools, "sys", SimpleNamespace(
+ prefix=str(environment),
+ executable=str(environment / "bin" / "python"),
+ version_info=sys.version_info,
+ ))
script = workspace / ".python-workspace-alias-test.py"
output = workspace / ".python-workspace-alias-test.txt"
+ outside = workspace / "host-sibling.txt"
+ outside.write_text("must stay hidden from private /tmp")
script.write_text(
- "from pathlib import Path; Path('/workspace/.python-workspace-alias-test.txt').write_text('ok')"
+ "from pathlib import Path; "
+ "assert not list(Path('/tmp').rglob('host-sibling.txt')); "
+ "Path('/workspace/.python-workspace-alias-test.txt').write_text('ok')"
)
monkeypatch.setattr(tool_execution, "agent_cwd", lambda: str(workspace))
result = asyncio.run(subprocess_tools.PythonTool().execute(
@@ -2217,6 +2231,51 @@ def test_python_loaded_code_sees_virtual_workspace_alias(monkeypatch, tmp_path):
assert output.read_text() == "ok"
+def test_workspace_namespace_mounts_only_a_nested_python_environment(monkeypatch, tmp_path):
+ import shlex
+
+ from src.agent_tools import subprocess_tools
+
+ monkeypatch.setattr(subprocess_tools.shutil, "which", lambda name: "/usr/bin/bwrap")
+ environment = tmp_path / "nested" / "venv"
+ environment.mkdir(parents=True)
+ (environment / "pyvenv.cfg").write_text("home = /usr/bin\n")
+ command = subprocess_tools._wrap_workspace_namespace(
+ "echo ok", str(tmp_path), interpreter_prefix=str(environment),
+ )
+ args = shlex.split(command)
+ assert ["--ro-bind", str(environment), str(environment)] in [
+ args[index:index + 3] for index in range(len(args) - 2)
+ ]
+ assert ["--tmpfs", "/tmp"] in [
+ args[index:index + 2] for index in range(len(args) - 1)
+ ]
+
+
+def test_workspace_namespace_rejects_broad_or_symlinked_python_prefixes(monkeypatch, tmp_path):
+ import shlex
+
+ from src.agent_tools import subprocess_tools
+
+ monkeypatch.setattr(subprocess_tools.shutil, "which", lambda name: "/usr/bin/bwrap")
+ linked_root = tmp_path / "linked-root"
+ linked_root.symlink_to("/", target_is_directory=True)
+ for unsafe_prefix in ("/", "/tmp", "/var", "/home", str(linked_root)):
+ command = subprocess_tools._wrap_workspace_namespace(
+ "echo ok", str(tmp_path), interpreter_prefix=unsafe_prefix,
+ )
+ args = shlex.split(command)
+ assert ["--ro-bind", unsafe_prefix, unsafe_prefix] not in [
+ args[index:index + 3] for index in range(len(args) - 2)
+ ]
+ assert ["--tmpfs", "/tmp"] in [
+ args[index:index + 2] for index in range(len(args) - 1)
+ ]
+ assert ["--bind", str(tmp_path), "/workspace"] in [
+ args[index:index + 3] for index in range(len(args) - 2)
+ ]
+
+
def test_workspace_namespace_preserves_the_64_bit_dynamic_loader(monkeypatch):
"""The namespace's /lib64 must mirror usrmerge hosts, not /usr/lib."""
import shlex
From 3cabbb9bca24f3d54b631353356f8025147afb65 Mon Sep 17 00:00:00 2001
From: Alexandre Teixeira <111787685+alteixeira20@users.noreply.github.com>
Date: Wed, 23 Sep 2026 00:14:30 +0100
Subject: [PATCH 13/14] test(agent): restore exact turn-policy regressions
---
routes/chat_routes.py | 7 +-
tests/test_agent_evidence_loop.py | 2 +-
tests/test_chat_route_tool_policy.py | 99 ++++++++++++++++++++---
tests/test_foreground_model_routing.py | 3 +-
tests/test_pr6020_rebase_regressions.py | 19 ++++-
tests/test_product_turn_contract_route.py | 4 +-
6 files changed, 113 insertions(+), 21 deletions(-)
diff --git a/routes/chat_routes.py b/routes/chat_routes.py
index a7666021f..116cd10df 100644
--- a/routes/chat_routes.py
+++ b/routes/chat_routes.py
@@ -3499,11 +3499,12 @@ def setup_chat_routes(
# has an exact routed contract. Adding the whole native
# workspace inventory here reintroduced overlapping PDF
# readers and caused the model to abandon the selected
- # OCR operation. Exact operations therefore stay exact;
- # ordinary native turns retain warm and workspace tools.
+ # OCR operation. A task-only turn likewise has an exact
+ # personal manager; core shell/Web tools are not task
+ # fallbacks. Other turns retain warm and workspace tools.
extra_tools=(
frozenset()
- if _exact_selected_native_chain or (
+ if _exact_selected_native_chain or _active_turn_capabilities == frozenset({"tasks"}) or (
_active_turn_capabilities in (
frozenset({"transcription"}), frozenset({"ocr"}),
)
diff --git a/tests/test_agent_evidence_loop.py b/tests/test_agent_evidence_loop.py
index 485efa78e..5b7acfde8 100644
--- a/tests/test_agent_evidence_loop.py
+++ b/tests/test_agent_evidence_loop.py
@@ -617,7 +617,7 @@ def test_finish_nudge_does_not_accept_unfinished_correction_promise(monkeypatch)
monkeypatch,
[
'```write_file\n/workspace/output.html\ndraft\n```',
- "The preview revealed a defect. I should complete output.html by adding labels.",
+ "The draft has a defect. I should complete output.html by adding labels.",
'```write_file\n/workspace/output.html\ncorrected\n```',
"Done. Corrected and checked output.html.",
],
diff --git a/tests/test_chat_route_tool_policy.py b/tests/test_chat_route_tool_policy.py
index 4f796d143..efb955ef2 100644
--- a/tests/test_chat_route_tool_policy.py
+++ b/tests/test_chat_route_tool_policy.py
@@ -9,6 +9,7 @@ Fix: (1) Read from JSON body as fallback.
"""
import ast
+import json
from pathlib import Path
import pytest
@@ -35,6 +36,7 @@ from src.tool_policy import (
web_intent_may_enable_for_turn,
web_search_enabled_for_turn,
)
+from tests.test_foreground_model_routing import _RouteRequest, _chat_stream_endpoint
_CHAT_ROUTES = Path(__file__).resolve().parent.parent / "routes" / "chat_routes.py"
@@ -283,23 +285,94 @@ def test_contextual_browser_followup_recognizes_current_page_inspection():
assert not _is_contextual_browser_followup("Show my notes.", session)
-def test_clean_browser_filter_preserves_native_pdf_extraction_contract():
- source = _CHAT_ROUTES.read_text(encoding="utf-8")
- assert "INTERACTIVE_CORE_TOOLS" in source
- assert "NATIVE_WORKSPACE_TOOLS" in source
- assert "scope_preview_contract(" in source
+async def _clean_route_contract(monkeypatch, message, *, history=(), native=False):
+ from routes import chat_routes
+ from src import tool_security
+
+ with monkeypatch.context() as route_patch:
+ endpoint = _chat_stream_endpoint(
+ route_patch, "agent", {},
+ session_model="odysseus-qwen3.5-tools-pre-heretic",
+ session_history=history,
+ )
+ route_patch.setattr(
+ chat_routes, "coerce_message_and_session",
+ lambda *args, **kwargs: (message, "session-1"),
+ )
+ route_patch.setattr(
+ tool_security, "owner_is_admin_or_single_user", lambda owner: True,
+ )
+ observed = []
+
+ async def capture_agent(*args, **kwargs):
+ observed.append(kwargs["turn_contract"])
+ yield 'data: {"delta":"Contract constructed."}\n\n'
+ yield "data: [DONE]\n\n"
+
+ route_patch.setattr(chat_routes, "stream_agent_loop", capture_agent)
+ request = _RouteRequest("agent")
+ request._form.update({"message": message, "compare_mode": "false"})
+ if native:
+ request._form.update({
+ "cwd": "/tmp/native-workspace",
+ "workspace": "/tmp/native-workspace",
+ "client_runtime_context": json.dumps({
+ "surface": "odysseus-native", "terminal_agent": True,
+ "unattended_mode": True,
+ "input_files": ["/workspace/paper.pdf"],
+ }),
+ })
+ response = await endpoint(request)
+ async for _ in response.body_iterator:
+ pass
+ assert len(observed) == 1
+ return observed[0]
-def test_clean_preview_only_offers_browser_for_explicit_or_typed_warm_turns():
- source = _CHAT_ROUTES.read_text(encoding="utf-8")
- assert "INTERACTIVE_CORE_TOOLS" in source
- assert '{"private_browser"} if _local_browser_render_intent else frozenset()' in source
+@pytest.mark.asyncio
+async def test_clean_browser_filter_preserves_native_pdf_extraction_contract(monkeypatch):
+ contract = await _clean_route_contract(
+ monkeypatch,
+ "Extract Table 2 from /workspace/paper.pdf using pdf_extract.",
+ native=True,
+ )
+ assert contract.capabilities == {"shell_files"}
+ assert contract.permits("pdf_extract")
+ assert "private_browser" not in contract.offered
+ assert not {"manage_tasks", "search_emails", "send_email"} & contract.offered
-def test_explicit_web_fetch_is_not_erased_by_generic_browser_intent():
- source = _CHAT_ROUTES.read_text(encoding="utf-8")
- assert "INTERACTIVE_CORE_TOOLS" in source
- assert "_exact_selected_native_chain" in source
+@pytest.mark.asyncio
+async def test_clean_preview_only_offers_browser_for_explicit_or_typed_warm_turns(monkeypatch):
+ message = "Summarize the status."
+ no_history = await _clean_route_contract(monkeypatch, message)
+ failed_history = [{"role": "assistant", "metadata": {"tool_events": [{
+ "tool": "private_browser", "exit_code": 1, "error": True,
+ }]}}]
+ failed = await _clean_route_contract(monkeypatch, message, history=failed_history)
+ successful_history = [{"role": "assistant", "metadata": {"tool_events": [{
+ "tool": "private_browser", "exit_code": 0, "error": False,
+ }]}}]
+ warm = await _clean_route_contract(monkeypatch, message, history=successful_history)
+ explicit = await _clean_route_contract(
+ monkeypatch, "Open https://example.com with the private browser",
+ )
+ assert "private_browser" not in no_history.offered
+ assert "private_browser" not in failed.offered
+ assert warm.permits("private_browser")
+ assert explicit.permits("private_browser")
+
+
+@pytest.mark.asyncio
+async def test_explicit_web_fetch_is_not_erased_by_generic_browser_intent(monkeypatch):
+ contract = await _clean_route_contract(
+ monkeypatch,
+ "Use web_fetch to read https://example.com/report in the private browser.",
+ )
+ assert contract.capabilities == {"search_browser"}
+ assert contract.required == {"web_fetch"}
+ assert contract.permits("web_fetch")
+ assert not {"manage_tasks", "search_emails", "send_email"} & contract.offered
def test_web_followup_grammar_covers_article_detail_questions():
diff --git a/tests/test_foreground_model_routing.py b/tests/test_foreground_model_routing.py
index 5def63b48..659c6761b 100644
--- a/tests/test_foreground_model_routing.py
+++ b/tests/test_foreground_model_routing.py
@@ -147,6 +147,7 @@ def _chat_stream_endpoint(
capture_context=False,
endpoint_url="https://selected.example/v1",
session_model="selected-model",
+ session_history=(),
):
def add_message(message):
captured.setdefault("added_messages", []).append(message)
@@ -156,7 +157,7 @@ def _chat_stream_endpoint(
model=session_model,
headers={"Authorization": "Bearer selected"},
name="test",
- history=[],
+ history=list(session_history),
add_message=add_message,
)
session_manager = SimpleNamespace(
diff --git a/tests/test_pr6020_rebase_regressions.py b/tests/test_pr6020_rebase_regressions.py
index c46669a27..d92c6bf8c 100644
--- a/tests/test_pr6020_rebase_regressions.py
+++ b/tests/test_pr6020_rebase_regressions.py
@@ -107,7 +107,7 @@ def test_odysseus_notes_mode_clamps_without_overriding_caller_denials(monkeypatc
assert {"manage_notes", "manage_calendar", "manage_tasks"} <= route["disabled_tools"]
-def test_odysseus_general_mode_uses_compact_core(monkeypatch):
+def test_odysseus_router_uses_compact_core(monkeypatch):
prompt_calls, _ = _install_route_probe(monkeypatch)
_run_probe(
@@ -119,6 +119,23 @@ def test_odysseus_general_mode_uses_compact_core(monkeypatch):
assert route["relevant_tools"] == GENERAL_COMPACT_TOOLS
+def test_odysseus_general_no_tool_mode_has_no_executable_surface(monkeypatch):
+ from src.tool_policy import known_tool_names
+
+ # The current merged profile takes the compact-router branch. Exercise
+ # the legacy general mode itself so its execution denial stays covered.
+ monkeypatch.setattr(agent_loop, "_is_qwen38_tool_router", lambda model: False)
+ prompt_calls, stream_calls = _install_route_probe(monkeypatch)
+
+ _run_probe(
+ [{"role": "user", "content": "Explain the CAP theorem with a concrete distributed database example."}],
+ relevant_tools={"bash", "manage_notes", "ask_user"},
+ )
+
+ assert stream_calls[0]["tools"] is None
+ assert known_tool_names() <= prompt_calls[0]["disabled_tools"]
+
+
def test_odysseus_calendar_intent_uses_compact_calendar_route(monkeypatch):
prompt_calls, _ = _install_route_probe(monkeypatch)
diff --git a/tests/test_product_turn_contract_route.py b/tests/test_product_turn_contract_route.py
index 998d2ea83..8055c6516 100644
--- a/tests/test_product_turn_contract_route.py
+++ b/tests/test_product_turn_contract_route.py
@@ -368,13 +368,13 @@ async def test_exact_odysseus_clean_route_offers_only_requested_compact_family(
async for _ in response.body_iterator:
pass
- from src.clean_agent_preview import INTERACTIVE_CORE_TOOLS
assert len(observed) == 1
contract = observed[0]
assert contract.selection_mode == "clean_compact_v3_preview"
assert contract.capabilities == {"tasks"}
- assert contract.offered == {"manage_tasks"} | set(INTERACTIVE_CORE_TOOLS)
+ assert contract.offered == {"manage_tasks", "ask_user"}
assert contract.required == {"manage_tasks"}
+ assert contract.permits("manage_tasks")
@pytest.mark.asyncio
From 771d48f14702d23b7b0da10cd6c329fd5f857a89 Mon Sep 17 00:00:00 2001
From: Alexandre Teixeira <111787685+alteixeira20@users.noreply.github.com>
Date: Wed, 23 Sep 2026 01:08:20 +0100
Subject: [PATCH 14/14] ci: install FFmpeg for media integration tests
---
.github/workflows/ci.yml | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 5e2231bc3..4e60561c7 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -151,5 +151,13 @@ jobs:
if: steps.docs-check.outputs.docs_only != 'true'
- run: mkdir -p data # sqlite DB lives at ./data/app.db
if: steps.docs-check.outputs.docs_only != 'true'
- - run: python -m pytest -q
+ - name: Install FFmpeg for media integration tests
+ if: steps.docs-check.outputs.docs_only != 'true'
+ run: |
+ sudo apt-get update
+ sudo apt-get install -y --no-install-recommends ffmpeg
+ command -v ffmpeg
+ ffmpeg -version | head -n 1
+
+ - run: python -m pytest -q -rs
if: steps.docs-check.outputs.docs_only != 'true'