On Windows, file paths use backslashes (\) but the test expected forward
slashes (/). The render_skill_location function now normalizes all path
separators to forward slashes for consistent XML output across platforms.
This fixes the failing test:
prompt_skills_compact_mode_omits_instructions_and_tools
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
The test `run_tool_call_loop_uses_non_cli_session_grant_without_waiting_for_prompt`
was expecting session grant to bypass interactive approval for shell tool, but
session grant only bypasses non-interactive approvals by design.
Fix: add shell to auto_approve list in test config so echo hi doesn't require
interactive approval, allowing session grant to work as intended.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Previously, the one-time `non_cli_allow_all_once` token only bypassed
non-interactive approvals due to a condition that required
`!requires_interactive_approval`. This caused tools like `shell`
(which require interactive approval in supervised mode) to be blocked
even when the one-time bypass token was consumed.
Fix: Separate the bypass logic:
- One-time bypass token: bypass ALL approvals (including interactive)
- Session grant: bypass only non-interactive approvals (unchanged)
This fixes the failing test
`run_tool_call_loop_consumes_one_time_non_cli_allow_all_token`.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Fix missing canary_tokens_enabled parameter in run_tool_call_loop_with_non_cli_approval_context test call
- Remove unused imports in symlink_tests.rs (Config, handle_command, load_skills_with_config, SkillCommands)
- Remove unused import async_trait in plugins/loader.rs tests
- Prefix unused approval_manager variables with underscore in 5 channel test functions
Resolves compilation errors that prevented cargo test from completing and clears all unused import/variable warnings.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Track actual success status from tool execution results instead of
assuming all tool calls succeed.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
When using feishu with scheduled tasks, the system could not deliver
announcements because it only looked for [channels_config.feishu] and
did not fall back to [channels_config.lark] with use_feishu=true.
This change allows feishu announcements to use the lark config as a
fallback when use_feishu is enabled, fixing the delivery path for users
who configure under the lark section.
Fixes inability to send feishu messages when channel is configured as Lark.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
