Add lldb + improve session defaults (#161)

Author: cameroncooke

AGENTS.md

@@ -1,210 +1,53 @@
-This file provides guidance to AI assisants (Claude Code, Cursor etc) when working with code in this repository.
-
-## Project Overview
-
-XcodeBuildMCP is a Model Context Protocol (MCP) server providing standardized tools for AI assistants to interact with Xcode projects, iOS simulators, devices, and Apple development workflows. It's a TypeScript/Node.js project that runs as a stdio-based MCP server.
-
-## Common Commands
-
-### Build & Development
-```bash
-npm run build         # Compile TypeScript with tsup, generates version info
-npm run dev           # Watch mode development
-npm run bundle:axe    # Bundle axe CLI tool for simulator automation (needed when using local MCP server)
-npm run test          # Run complete Vitest test suite
-npm run test:watch    # Watch mode testing
-npm run lint          # ESLint code checking
-npm run lint:fix       # ESLint code checking and fixing
-npm run format:check  # Prettier code checking
-npm run format        # Prettier code formatting
-npm run typecheck     # TypeScript type checking
-npm run inspect       # Run interactive MCP protocol inspector
-npm run doctor        # Doctor CLI
-```
-
-### Development with Reloaderoo
-
-**Reloaderoo** (v1.1.2+) provides CLI-based testing and hot-reload capabilities for XcodeBuildMCP without requiring MCP client configuration.
-
-#### Quick Start
-
-**CLI Mode (Testing & Development):**
-```bash
-# List all tools
-npx reloaderoo inspect list-tools -- node build/index.js
-
-# Call any tool
-npx reloaderoo inspect call-tool list_devices --params '{}' -- node build/index.js
-
-# Get server information
-npx reloaderoo inspect server-info -- node build/index.js
-
-# List and read resources
-npx reloaderoo inspect list-resources -- node build/index.js
-npx reloaderoo inspect read-resource "xcodebuildmcp://devices" -- node build/index.js
-```
-
-**Proxy Mode (MCP Client Integration):**
-```bash
-# Start persistent server for MCP clients
-npx reloaderoo proxy -- node build/index.js
-
-# With debug logging
-npx reloaderoo proxy --log-level debug -- node build/index.js
-
-# Then ask AI: "Please restart the MCP server to load my changes"
-```
-
-#### All CLI Inspect Commands
-
-Reloaderoo provides 8 inspect subcommands for comprehensive MCP server testing:
-
-```bash
-# Server capabilities and information
-npx reloaderoo inspect server-info -- node build/index.js
-
-# Tool management
-npx reloaderoo inspect list-tools -- node build/index.js
-npx reloaderoo inspect call-tool <tool_name> --params '<json>' -- node build/index.js
-
-# Resource access
-npx reloaderoo inspect list-resources -- node build/index.js
-npx reloaderoo inspect read-resource "<uri>" -- node build/index.js
-
-# Prompt management
-npx reloaderoo inspect list-prompts -- node build/index.js
-npx reloaderoo inspect get-prompt <name> --args '<json>' -- node build/index.js
-
-# Connectivity testing
-npx reloaderoo inspect ping -- node build/index.js
-```
-
-#### Advanced Options
-
-```bash
-# Custom working directory
-npx reloaderoo inspect list-tools --working-dir /custom/path -- node build/index.js
-
-# Timeout configuration
-npx reloaderoo inspect call-tool slow_tool --timeout 60000 --params '{}' -- node build/index.js
-
-# Use timeout configuration if needed
-npx reloaderoo inspect server-info --timeout 60000 -- node build/index.js
-
-# Debug logging (use proxy mode for detailed logging)
-npx reloaderoo proxy --log-level debug -- node build/index.js
-```
-
-#### Key Benefits
-
-- ✅ **No MCP Client Setup**: Direct CLI access to all tools
-- ✅ **Raw JSON Output**: Perfect for AI agents and programmatic use
-- ✅ **Hot-Reload Support**: `restart_server` tool for MCP client development
-- ✅ **Claude Code Compatible**: Automatic content block consolidation
-- ✅ **8 Inspect Commands**: Complete MCP protocol testing capabilities
-- ✅ **Universal Compatibility**: Works on any system via npx
-
-For complete documentation, examples, and troubleshooting, see @docs/dev/RELOADEROO.md
-
-## Architecture Overview
-
-### Plugin-Based MCP architecture
-
-XcodeBuildMCP uses the concept of configuration by convention for MCP exposing and running MCP capabilities like tools and resources. This means to add a new tool or resource, you simply create a new file in the appropriate directory and it will be automatically loaded and exposed to MCP clients.
-
-#### Tools
-
-Tools are the core of the MCP server and are the primary way to interact with the server. They are organized into directories by their functionality and are automatically loaded and exposed to MCP clients.
-
-For more information see @docs/dev/PLUGIN_DEVELOPMENT.md
-
-#### Resources
-
-Resources are the secondary way to interact with the server. They are used to provide data to tools and are organized into directories by their functionality and are automatically loaded and exposed to MCP clients.
-
-For more information see @docs/dev/PLUGIN_DEVELOPMENT.md
-
-### Tool Registration
-
-XcodeBuildMCP loads tools at startup. To limit the toolset, set `XCODEBUILDMCP_ENABLED_WORKFLOWS` to a comma-separated list of workflow directory names (for example: `simulator,project-discovery`). The `session-management` workflow is always auto-included since other tools depend on it.
-
-#### Claude Code Compatibility Workaround
-- **Detection**: Automatic detection when running under Claude Code.
-- **Purpose**: Workaround for Claude Code's MCP specification violation where it only displays the first content block in tool responses.
-- **Behavior**: When Claude Code is detected, multiple content blocks are automatically consolidated into a single text response, separated by `---` dividers. This ensures all information (including test results and stderr warnings) is visible to Claude Code users.
-
-### Core Architecture Layers
-1. **MCP Transport**: stdio protocol communication
-2. **Plugin Discovery**: Automatic tool AND resource registration system
-3. **MCP Resources**: URI-based data access (e.g., `xcodebuildmcp://simulators`)
-4. **Tool Implementation**: Self-contained workflow modules
-5. **Shared Utilities**: Command execution, build management, validation
-6. **Types**: Shared interfaces and Zod schemas
-
-For more information see @docs/dev/ARCHITECTURE.md
-
-## Testing
-
-The project enforces a strict **Dependency Injection (DI)** testing philosophy.
-
-- **NO Vitest Mocking**: The use of `vi.mock()`, `vi.fn()`, `vi.spyOn()`, etc., is **completely banned**.
-- **Executors**: All external interactions (like running commands or accessing the file system) are handled through injectable "executors".
-    - `CommandExecutor`: For running shell commands.
-    - `FileSystemExecutor`: For file system operations.
-- **Testing Logic**: Tests import the core `...Logic` function from a tool file and pass in a mock executor (`createMockExecutor` or `createMockFileSystemExecutor`) to simulate different outcomes.
-
-This approach ensures that tests are robust, easy to maintain, and verify the actual integration between components without being tightly coupled to implementation details.
-
-For complete guidelines, refer to @docs/dev/TESTING.md.
-
-## TypeScript Import Standards
-
-This project uses **TypeScript file extensions** (`.ts`) for all relative imports to ensure compatibility with native TypeScript runtimes.
-
-### Import Rules
-
-- ✅ **Use `.ts` extensions**: `import { tool } from './tool.ts'`
-- ✅ **Use `.ts` for re-exports**: `export { default } from '../shared/tool.ts'`
-- ✅ **External packages use `.js`**: `import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js'`
-- ❌ **Never use `.js` for internal files**: `import { tool } from './tool.js'` ← ESLint error
-
-### Benefits
-
-1. **Future-proof**: Compatible with native TypeScript runtimes (Bun, Deno, Node.js --loader)
-2. **IDE Experience**: Direct navigation to source TypeScript files
-3. **Consistency**: Import path matches the actual file you're editing
-4. **Modern Standard**: Aligns with TypeScript 4.7+ `allowImportingTsExtensions`
-
-### ESLint Enforcement
-
-The project automatically enforces this standard:
-
-```bash
-npm run lint  # Will catch .js imports for internal files
-```
-
-This ensures all new code follows the `.ts` import pattern and maintains compatibility with both current and future TypeScript execution environments.
-
-## Release Process
-
-Follow standardized development workflow with feature branches, structured pull requests, and linear commit history. **Never push to main directly or force push without permission.**
-
-For complete guidelines, refer to @docs/dev/RELEASE_PROCESS.md
-
-## Useful external resources
-
-### Model Context Protocol
-
-https://modelcontextprotocol.io/llms-full.txt
-
-### MCP Specification
-
-https://modelcontextprotocol.io/specification
-
-### MCP Inspector
-
-https://github.com/modelcontextprotocol/inspector
-
-### MCP Client SDKs
-
-https://github.com/modelcontextprotocol/typescript-sdk
+# Development Rules
+
+## Code Quality
+- No `any` types unless absolutely necessary
+- Check node_modules for external API type definitions instead of guessing
+- **NEVER use inline imports** - no `await import("./foo.js")`, no `import("pkg").Type` in type positions, no dynamic imports for types. Always use standard top-level imports.
+- NEVER remove or downgrade code to fix type errors from outdated dependencies; upgrade the dependency instead
+- Always ask before removing functionality or code that appears to be intentional
+- Follow TypeScript best practices
+
+## Commands
+- NEVER commit unless user asks
+
+## GitHub
+When reading issues:
+- Always read all comments on the issue
+-
+## Tools
+- GitHub CLI for issues/PRs
+-
+## Style
+- Keep answers short and concise
+- No emojis in commits, issues, PR comments, or code
+- No fluff or cheerful filler text
+- Technical prose only, be kind but direct (e.g., "Thanks @user" not "Thanks so much @user!")
+
+## Docs
+- If modifying or adding/removing tools run `npm run docs:update` to update the TOOLS.md file, never edit this file directly.
+-
+### Changelog
+Location: `CHANGELOG.md`
+
+#### Format
+Use these sections under `## [Unreleased]`:
+- `### Added` - New features
+- `### Changed` - Changes to existing functionality
+- `### Fixed` - Bug fixes
+- `### Removed` - Removed features
+-
+#### Rules
+- Before adding entries, read the full `[Unreleased]` section to see which subsections already exist
+- New entries ALWAYS go under `## [Unreleased]` section
+- Append to existing subsections (e.g., `### Fixed`), do not create duplicates
+- NEVER modify already-released version sections (e.g., `## [0.12.2]`)
+- Each version section is immutable once released
+-
+#### Attribution
+- **Internal changes (from issues)**: `Fixed foo bar ([#123](https://github.com/cameroncook/XcodeBuildMCP/issues/123))`
+- **External contributions**: `Added feature X ([#456](https://github.com/cameroncook/XcodeBuildMCP/pull/456) by [@username](https://github.com/username))`
+
+## **CRITICAL** Tool Usage Rules **CRITICAL**
+- NEVER use sed/cat to read a file or a range of a file. Always use the native read tool.
+- You MUST read every file you modify in full before editing.

CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## [Unreleased]
+### Added
+- Add Smithery support for packaging/distribution.
+- Add DAP-based debugger backend and simulator debugging toolset (attach, breakpoints, stack, variables, LLDB command).
+- Add session-status MCP resource with session identifiers.
+- Add UI automation guard that blocks UI tools when the debugger is paused.
+
+### Changed
+- Migrate to Zod v4.
+- Improve session default handling (reconcile mutual exclusivity and ignore explicit undefined clears).
+
+### Fixed
+- Update UI automation guard guidance to point at `debug_continue` when paused.
+- Fix tool loading bugs in static tool registration.
+
 ## [1.16.0] - 2025-12-30
 - Remove dynamic tool discovery (`discover_tools`) and `XCODEBUILDMCP_DYNAMIC_TOOLS`. Use `XCODEBUILDMCP_ENABLED_WORKFLOWS` to limit startup tool registration.
 - Add MCP tool annotations to all tools.

build-plugins/plugin-discovery.js

@@ -96,7 +96,7 @@ function generateWorkflowLoader(workflowName, toolFiles) {
   const toolImports = toolFiles
     .map((file, index) => {
       const toolName = file.replace(/\.(ts|js)$/, '');
-      return `const tool_${index} = await import('../mcp/tools/${workflowName}/${toolName}.js').then(m => m.default)`;
+      return `const tool_${index} = await import('../mcp/tools/${workflowName}/${toolName}.ts').then(m => m.default)`;
     })
     .join(';\n    ');
 
@@ -108,7 +108,7 @@ function generateWorkflowLoader(workflowName, toolFiles) {
     .join(',\n      ');
 
   return `async () => {
-    const { workflow } = await import('../mcp/tools/${workflowName}/index.js');
+    const { workflow } = await import('../mcp/tools/${workflowName}/index.ts');
     ${toolImports ? toolImports + ';\n    ' : ''}
     return {
       workflow,
@@ -215,7 +215,7 @@ async function generateResourceLoaders() {
 
     // Generate dynamic loader for this resource
     resourceLoaders[resourceName] = `async () => {
-    const module = await import('../mcp/resources/${resourceName}.js');
+    const module = await import('../mcp/resources/${resourceName}.ts');
     return module.default;
   }`;
 

build-plugins/plugin-discovery.ts

@@ -101,7 +101,7 @@ function generateWorkflowLoader(workflowName: string, toolFiles: string[]): stri
   const toolImports = toolFiles
     .map((file, index) => {
       const toolName = file.replace(/\.(ts|js)$/, '');
-      return `const tool_${index} = await import('../mcp/tools/${workflowName}/${toolName}.js').then(m => m.default)`;
+      return `const tool_${index} = await import('../mcp/tools/${workflowName}/${toolName}.ts').then(m => m.default)`;
     })
     .join(';\n    ');
 
@@ -113,7 +113,7 @@ function generateWorkflowLoader(workflowName: string, toolFiles: string[]): stri
     .join(',\n      ');
 
   return `async () => {
-    const { workflow } = await import('../mcp/tools/${workflowName}/index.js');
+    const { workflow } = await import('../mcp/tools/${workflowName}/index.ts');
     ${toolImports ? toolImports + ';\n    ' : ''}
     return {
       workflow,
@@ -217,7 +217,7 @@ export async function generateResourceLoaders(): Promise<void> {
   for (const fileName of resourceFiles) {
     const resourceName = fileName.replace(/\.(ts|js)$/, '');
     resourceLoaders[resourceName] = `async () => {
-    const module = await import('../mcp/resources/${resourceName}.js');
+    const module = await import('../mcp/resources/${resourceName}.ts');
     return module.default;
   }`;
 

docs/CONFIGURATION.md

@@ -20,17 +20,19 @@ XcodeBuildMCP is configured through environment variables provided by your MCP c
 By default, XcodeBuildMCP loads all tools at startup. If you want a smaller tool surface for a specific workflow, set `XCODEBUILDMCP_ENABLED_WORKFLOWS` to a comma-separated list of workflow directory names. The `session-management` workflow is always auto-included since other tools depend on it.
 
 **Available workflows:**
-- `device` (7 tools) - iOS Device Development
-- `simulator` (12 tools) - iOS Simulator Development
-- `simulator-management` (5 tools) - Simulator Management
-- `swift-package` (6 tools) - Swift Package Manager
-- `project-discovery` (5 tools) - Project Discovery
-- `macos` (6 tools) - macOS Development
-- `ui-testing` (11 tools) - UI Testing & Automation
+- `device` (14 tools) - iOS Device Development
+- `simulator` (19 tools) - iOS Simulator Development
 - `logging` (4 tools) - Log Capture & Management
+- `macos` (11 tools) - macOS Development
+- `project-discovery` (5 tools) - Project Discovery
 - `project-scaffolding` (2 tools) - Project Scaffolding
 - `utilities` (1 tool) - Project Utilities
+- `session-management` (3 tools) - session-management
+- `debugging` (8 tools) - Simulator Debugging
+- `simulator-management` (8 tools) - Simulator Management
+- `swift-package` (6 tools) - Swift Package Manager
 - `doctor` (1 tool) - System Doctor
+- `ui-testing` (11 tools) - UI Testing & Automation
 
 ## Incremental build support