test: add 10 tests for v1.2 features

[kienbui1995]

Add test coverage for all 6 new v1.2 features.

Module	New Tests	Coverage
session.rs	to_markdown_basic, to_markdown_with_tool_calls	/export markdown
parallel_tools.rs	simple_diff_*, diff_preview_*, is_write_tool_check	/diff-preview
plugin.rs	discover_all_script_types, reads_description, execute_sh_plugin	Plugin system

Total: 152 → 162 tests, all passing.

Summary by CodeRabbit

• Tests
  • Added comprehensive unit test coverage for diff operations, session markdown conversion, and plugin discovery/execution.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Three test modules were added to validate existing functionality: parallel_tools.rs tests for diff operations and tool classification, session.rs tests for markdown formatting output, and plugin.rs tests for plugin discovery and execution across supported script types. No production logic was modified.

Changes

Cohort / File(s)	Summary
Diff and Tool Classification Tests mc/crates/mc-core/src/parallel_tools.rs	Added diff_tests module with unit tests validating simple_diff behavior for identical and changed inputs, diff_preview_summary formatting for write_file and edit_file operations, and is_write_tool classification logic.
Session Markdown Output Tests mc/crates/mc-core/src/session.rs	Added unit tests verifying Session::to_markdown() output structure, including markdown headers, role labels, message text, tool-call formatting with backticked command names, and tool-result icon formatting.
Plugin Discovery and Execution Tests mc/crates/mc-tools/src/plugin.rs	Added unit tests validating discovery of supported script types (.sh, .py, .js), extraction of tool descriptions from script first-line comments, and execution of shell plugins with PLUGIN_INPUT environment variable passing.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• feat: plugin system — auto-discover and execute custom tools #16: Plugin discovery and execution tests directly validate the plugin discovery and execution logic introduced in this PR.
• feat: /diff-preview — approve/reject file changes with diff #15: Diff-preview and tool classification tests directly target and validate the diff-preview implementation logic from this PR.

Poem

🐰 A bunny hops through test-filed halls,
With diffs and sessions' happy calls,
Each test a seed in fertile ground,
Plugin plugins safely sound!
No bugs shall sneak past this kind hare,
Coverage blooming everywhere! 🌱

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'test: add 10 tests for v1.2 features' accurately describes the main change—adding comprehensive test coverage for v1.2 features across multiple modules with clear, specific focus.
Docstring Coverage	✅ Passed	Docstring coverage is 92.86% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch test/v1.2-features

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}