Skip to content

Commit cfc7ae0

Browse files
committed
Fix incorrect test file other missed references to old diagnostic tool name
1 parent f919c9d commit cfc7ae0

4 files changed

Lines changed: 27 additions & 27 deletions

File tree

.github/ISSUE_TEMPLATE/bug_report.yml

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ body:
77
attributes:
88
value: |
99
Thanks for taking the time to report an issue with XcodeBuildMCP!
10-
10+
1111
- type: textarea
1212
id: description
1313
attributes:
@@ -32,12 +32,12 @@ body:
3232
## System Information
3333
- platform: darwin
3434
- release: 25.0.0
35-
- arch: arm64
35+
- arch: arm64
3636
...
37-
```
37+
```
3838
validations:
39-
required: true
40-
39+
required: true
40+
4141
- type: input
4242
id: editor-client
4343
attributes:
@@ -46,7 +46,7 @@ body:
4646
placeholder: Cursor 0.49.1
4747
validations:
4848
required: true
49-
49+
5050
- type: input
5151
id: mcp-server-version
5252
attributes:
@@ -55,7 +55,7 @@ body:
5555
placeholder: 1.2.2
5656
validations:
5757
required: true
58-
58+
5959
- type: input
6060
id: llm
6161
attributes:
@@ -64,7 +64,7 @@ body:
6464
placeholder: Claude 3.5 Sonnet
6565
validations:
6666
required: true
67-
67+
6868
- type: textarea
6969
id: mcp-config
7070
attributes:
@@ -79,7 +79,7 @@ body:
7979
}
8080
```
8181
render: json
82-
82+
8383
- type: textarea
8484
id: steps
8585
attributes:
@@ -91,7 +91,7 @@ body:
9191
3. What failed or didn't work as expected
9292
validations:
9393
required: true
94-
94+
9595
- type: textarea
9696
id: expected
9797
attributes:
@@ -100,7 +100,7 @@ body:
100100
placeholder: The AI should have been able to...
101101
validations:
102102
required: true
103-
103+
104104
- type: textarea
105105
id: actual
106106
attributes:
@@ -109,7 +109,7 @@ body:
109109
placeholder: Instead, the AI...
110110
validations:
111111
required: true
112-
112+
113113
- type: textarea
114114
id: error
115115
attributes:

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ A Model Context Protocol (MCP) server that provides Xcode-related tools for inte
3333
- [Client Compatibility](#client-compatibility)
3434
- [Code Signing for Device Deployment](#code-signing-for-device-deployment)
3535
- [Troubleshooting](#troubleshooting)
36-
- [Doctor Tool](#doctor-tool)
36+
- [Doctor Tool](#doctor-tool)
3737
- [Privacy](#privacy)
3838
- [What is sent to Sentry?](#what-is-sent-to-sentry)
3939
- [Opting Out of Sentry](#opting-out-of-sentry)
@@ -105,7 +105,7 @@ For clients that support MCP resources XcodeBuildMCP provides efficient URI-base
105105

106106
- **Simulators Resource** (`xcodebuildmcp://simulators`): Direct access to available iOS simulators with UUIDs and states
107107
- **Devices Resource** (`xcodebuildmcp://devices`): Direct access to connected physical Apple devices with UDIDs and states
108-
- **Environment Resource** (`xcodebuildmcp://environment`): Direct access to environment information such as Xcode version, macOS version, and Node.js version
108+
- **Doctor Resource** (`xcodebuildmcp://doctor`): Direct access to environment information such as Xcode version, macOS version, and Node.js version
109109

110110
## Getting started
111111

docs/TOOLS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -211,4 +211,4 @@ For clients that support MCP resources, XcodeBuildMCP provides efficient URI-bas
211211
|--------------|-------------|---------------|
212212
| `xcodebuildmcp://simulators` | Available iOS simulators with UUIDs and states | `list_sims` |
213213
| `xcodebuildmcp://devices` | Available physical Apple devices with UUIDs, names, and connection status | `list_devices` |
214-
| `xcodebuildmcp://environment` | System health checks and environment validation | `doctor` |
214+
| `xcodebuildmcp://doctor` | System health checks and environment validation | `doctor` |

src/mcp/resources/__tests__/environment.test.ts renamed to src/mcp/resources/__tests__/doctor.test.ts

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,26 @@
11
import { describe, it, expect } from 'vitest';
22

3-
import environmentResource, { environmentResourceLogic } from '../environment.js';
3+
import doctorResource, { doctorResourceLogic } from '../doctor.js';
44
import { createMockExecutor } from '../../../utils/command.js';
55

6-
describe('environment resource', () => {
6+
describe('doctor resource', () => {
77
describe('Export Field Validation', () => {
88
it('should export correct uri', () => {
9-
expect(environmentResource.uri).toBe('xcodebuildmcp://environment');
9+
expect(doctorResource.uri).toBe('xcodebuildmcp://doctor');
1010
});
1111

1212
it('should export correct description', () => {
13-
expect(environmentResource.description).toBe(
14-
'Comprehensive development environment doctor information and configuration status',
13+
expect(doctorResource.description).toBe(
14+
'Comprehensive development environment diagnostic information and configuration status',
1515
);
1616
});
1717

1818
it('should export correct mimeType', () => {
19-
expect(environmentResource.mimeType).toBe('text/plain');
19+
expect(doctorResource.mimeType).toBe('text/plain');
2020
});
2121

2222
it('should export handler function', () => {
23-
expect(typeof environmentResource.handler).toBe('function');
23+
expect(typeof doctorResource.handler).toBe('function');
2424
});
2525
});
2626

@@ -31,7 +31,7 @@ describe('environment resource', () => {
3131
output: 'Mock command output',
3232
});
3333

34-
const result = await environmentResourceLogic(mockExecutor);
34+
const result = await doctorResourceLogic(mockExecutor);
3535

3636
expect(result.contents).toHaveLength(1);
3737
expect(result.contents[0].text).toContain('XcodeBuildMCP Doctor');
@@ -45,7 +45,7 @@ describe('environment resource', () => {
4545
it('should handle spawn errors by showing doctor info', async () => {
4646
const mockExecutor = createMockExecutor(new Error('spawn xcrun ENOENT'));
4747

48-
const result = await environmentResourceLogic(mockExecutor);
48+
const result = await doctorResourceLogic(mockExecutor);
4949

5050
expect(result.contents).toHaveLength(1);
5151
expect(result.contents[0].text).toContain('XcodeBuildMCP Doctor');
@@ -63,7 +63,7 @@ describe('environment resource', () => {
6363
output: 'Mock output',
6464
});
6565

66-
const result = await environmentResourceLogic(mockExecutor);
66+
const result = await doctorResourceLogic(mockExecutor);
6767

6868
expect(result.contents[0].text).toContain('## Troubleshooting Tips');
6969
expect(result.contents[0].text).toContain('brew tap cameroncooke/axe');
@@ -85,7 +85,7 @@ describe('environment resource', () => {
8585
output: 'Mock output',
8686
});
8787

88-
const result = await environmentResourceLogic(mockExecutor);
88+
const result = await doctorResourceLogic(mockExecutor);
8989

9090
expect(result.contents[0].text).toContain('### UI Automation (axe)');
9191
expect(result.contents[0].text).toContain('### Incremental Builds');
@@ -100,7 +100,7 @@ describe('environment resource', () => {
100100
error: 'Command failed',
101101
});
102102

103-
const result = await environmentResourceLogic(mockExecutor);
103+
const result = await doctorResourceLogic(mockExecutor);
104104

105105
expect(result.contents).toHaveLength(1);
106106
expect(result.contents[0].text).toContain('XcodeBuildMCP Doctor');

0 commit comments

Comments
 (0)