refactor(1/12): remove deprecated logging tools

[cameroncooke]

Summary

This is PR 1 of 12 in a stacked PR series that decouples the rendering pipeline from MCP transport. This PR can be reviewed and merged independently.

Removes all deprecated logging tools that were superseded by the unified log capture approach. These tools (start_sim_log_cap, stop_sim_log_cap, start_device_log_cap, stop_device_log_cap, launch_app_logs_sim) used a start/stop session model that was fragile and leaked resources. The replacement (already shipped in earlier releases) integrates log capture directly into build-and-run workflows, making these standalone logging tools dead code.

What changes

• Delete all 4 logging tool implementations and their tests (src/mcp/tools/logging/)
• Delete launch_app_logs_sim tool (simulator variant that combined launch + log capture)
• Remove corresponding YAML manifests and the logging workflow manifest
• Remove logging tool references from device and simulator workflow manifests
• Delete the e2e smoke test for logging tools

Why

These tools have been deprecated since the log capture was integrated into build-run workflows. Removing them before the rendering pipeline refactor avoids migrating dead code to the new handler contract, keeping subsequent PRs in this stack cleaner.

Stack navigation

• PR 1/12 (this PR): Remove deprecated logging tools
• PR 2/12: Pipeline event types, parsers, and event builders
• PR 3/12: Rendering engine and output formatting
• PR 4/12: Build/test utility extraction, platform steps, xcodebuild pipeline
• PR 5/12: Runtime handler contract and tool invoker
• PR 6/12: Tool migrations - Simulator tools
• PR 7/12: Tool migrations - Device + macOS tools
• PR 8/12: Tool migrations - UI automation tools
• PR 9/12: Tool migrations - Remaining tools
• PR 10/12: CLI, daemon, and MCP server boundaries
• PR 11/12: Manifests, config, docs, and examples
• PR 12/12: Snapshot test fixtures and benchmarks

Test plan

• npx vitest run passes (no tests reference removed tools)
• No remaining imports of deleted modules in the codebase
• Manifest validation passes without the removed YAML files

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/xcodebuildmcp@319

commit: d820dcf

[cameroncooke]

• refactor(12/12): add snapshot test fixtures and benchmarks #330
• refactor(11/12): update manifests, config, docs, and example projects #329
• refactor(10/12): rewire CLI, daemon, and MCP server boundaries #328
• refactor(9/12): migrate debugging, swift-package, and remaining tools #327
• refactor(8/12): migrate UI automation tools to event-based handlers #326
• refactor(7/12): migrate device and macOS tools to event-based handlers #325
• refactor(6/12): migrate simulator tools to event-based handlers #324
• refactor(5/12): change handler contract to event-based emission #323
• refactor(4/12): extract build/test utilities, platform steps, and xcodebuild pipeline #322
• refactor(3/12): add rendering engine and output formatting #321
• refactor(2/12): add pipeline event types, parsers, and event builders #320
• refactor(1/12): remove deprecated logging tools #319 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.