kshitijk4poor fcac0f94d4 fix(openviking): guard empty tool_id in batch skip set; reuse env_var_enabled ... Two follow-up fixes on top of the cherry-picked structured-sync work: - _messages_to_openviking_batch only added a recall tool result's id to skipped_tool_ids when the id was non-empty. An empty tool_call_id (which the canonical transcript can carry; agent_runtime_helpers defaults it to "") poisoned the skip set with "", silently dropping any *other* tool result that also lacked an id. Move the recall-skip add inside the existing `if tool_id:` guard. Adds a regression test (mutation-checked: fails on pre-fix code, passes after). - _sync_trace_enabled() open-coded the canonical truthy-env check; reuse utils.env_var_enabled (byte-identical {1,true,yes,on} semantics).		2026-06-19 13:53:39 +05:30
..
test_openviking.py	fix(openviking): guard empty tool_id in batch skip set; reuse env_var_enabled	2026-06-19 13:53:39 +05:30