Skip to content

Commit ac0249d

Browse files
wheels-bot[bot]github-actions[bot]
wheels-bot[bot]
and
github-actions[bot]
authored
fix(mcp): replace phantom mcp-configuration-guide.md path with live integration guide URL (#3019)
* fix(mcp): replace phantom mcp-configuration-guide.md path with live integration guide URL PR ##2888 standardized the deprecated /wheels/mcp HTTP transport's doc pointer on https://guides.wheels.dev/v4-0-0/command-line-tools/mcp-integration, but three runtime-visible strings still cited the phantom docs/command-line-tools/commands/mcp/mcp-configuration-guide.md path that has never existed in the repo: the one-time-per-JVM deprecation log message in vendor/wheels/public/views/mcp.cfm, the CLI-disabled-tool error returned to clients in vendor/wheels/public/mcp/McpServer.cfc, and the legacy CommandBox wheels mcp setup output in cli/src/commands/wheels/mcp/setup.cfc. Replace all three (plus the file-header comment in mcp.cfm) with the URL #2888 already standardized on, matching the serverInfo.deprecationNotice field that PR did fix. Refs #3016 Signed-off-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com> * docs(web/guides): replace deprecated "dependencies" key with "requires" in package.json example Signed-off-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com> --------- Signed-off-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com> Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
1 parent 383146b commit ac0249d

6 files changed

Lines changed: 65 additions & 5 deletions

File tree

‎changelog.d/3016-mcp-deprecation-stale-path.fixed.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
- Deprecated `/wheels/mcp` HTTP transport's runtime warnings (log message, CLI-disabled-tool error, and legacy `wheels mcp setup` output) now point to the live integration guide (`https://guides.wheels.dev/v4-0-0/command-line-tools/mcp-integration`) instead of the phantom `docs/command-line-tools/commands/mcp/mcp-configuration-guide.md` path that #2888 missed (#3016)

‎cli/src/commands/wheels/mcp/setup.cfc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ component extends="../base" {
9191
print.indentedLine("• Test runner (test)");
9292
print.indentedLine("• Browser automation (via Browser MCP)");
9393
print.line();
94-
print.indentedLine("Full guide: docs/command-line-tools/commands/mcp/mcp-configuration-guide.md");
94+
print.indentedLine("Full guide: https://guides.wheels.dev/v4-0-0/command-line-tools/mcp-integration");
9595
print.line();
9696
}
9797

‎vendor/wheels/public/mcp/McpServer.cfc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1286,7 +1286,7 @@ Provide migration code following Wheels conventions."
12861286
// intact for existing clients.
12871287
return "Error: CLI-backed tools are disabled on the deprecated /wheels/mcp HTTP endpoint. "
12881288
& "Use the stdio MCP server instead ('wheels mcp wheels'; see "
1289-
& "docs/command-line-tools/commands/mcp/mcp-configuration-guide.md). "
1289+
& "https://guides.wheels.dev/v4-0-0/command-line-tools/mcp-integration). "
12901290
& "Requested command: " & arguments.command;
12911291
}
12921292

‎vendor/wheels/public/views/mcp.cfm‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
// and doesn't require a running dev server for file-based operations
1515
// (generate, migrate, stats, etc.). This HTTP endpoint will be removed in
1616
// a future release. See:
17-
// docs/command-line-tools/commands/mcp/mcp-configuration-guide.md
17+
// https://guides.wheels.dev/v4-0-0/command-line-tools/mcp-integration
1818
1919
// Log one-time deprecation warning per JVM
2020
if (!structKeyExists(application, "mcpHttpDeprecationLogged")) {
@@ -24,7 +24,7 @@ if (!structKeyExists(application, "mcpHttpDeprecationLogged")) {
2424
type="warning",
2525
text="The in-dev-server MCP endpoint at /wheels/mcp is deprecated. "
2626
& "Use 'wheels mcp wheels' (LuCLI stdio MCP) instead. "
27-
& "See docs/command-line-tools/commands/mcp/mcp-configuration-guide.md"
27+
& "See https://guides.wheels.dev/v4-0-0/command-line-tools/mcp-integration"
2828
);
2929
} catch (any ignored) { /* logging is best-effort */ }
3030
application.mcpHttpDeprecationLogged = true;

‎vendor/wheels/tests/specs/cli/McpDeprecationNoticeStaleDocPathSpec.cfc‎

Lines changed: 59 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,59 @@
1+
/**
2+
* Regression: #2888 standardized the deprecated `/wheels/mcp` HTTP transport's
3+
* doc pointer on the live integration guide
4+
* (`https://guides.wheels.dev/v4-0-0/command-line-tools/mcp-integration`),
5+
* but three runtime-visible strings still cite a phantom path that has
6+
* never existed in the repo:
7+
*
8+
* - vendor/wheels/public/views/mcp.cfm (deprecation log message)
9+
* - vendor/wheels/public/mcp/McpServer.cfc (CLI-disabled-tool error)
10+
* - cli/src/commands/wheels/mcp/setup.cfc (legacy CommandBox CLI output)
11+
*
12+
* The phantom path is `docs/command-line-tools/commands/mcp/mcp-configuration-guide.md`.
13+
* Issue ##3016.
14+
*/
15+
component extends="wheels.WheelsTest" {
16+
17+
function run() {
18+
19+
describe("MCP deprecation pointers cite the live integration guide", () => {
20+
21+
// expandPath("/wheels") resolves to vendor/wheels via the configured
22+
// Lucee mapping; the repo root is two levels above.
23+
var repoRoot = expandPath("/wheels/../..");
24+
var phantomPath = "docs/command-line-tools/commands/mcp/mcp-configuration-guide.md";
25+
var canonical = "https://guides.wheels.dev/v4-0-0/command-line-tools/mcp-integration";
26+
var targets = [
27+
"vendor/wheels/public/views/mcp.cfm",
28+
"vendor/wheels/public/mcp/McpServer.cfc",
29+
"cli/src/commands/wheels/mcp/setup.cfc"
30+
];
31+
32+
for (var rel in targets) {
33+
// Capture the loop variable so the closure body binds the
34+
// current value, not the final iteration's value.
35+
(function(relPath) {
36+
it("references " & canonical & " in " & relPath, () => {
37+
var absolute = repoRoot & "/" & relPath;
38+
expect(fileExists(absolute)).toBeTrue("Missing file: " & absolute);
39+
40+
var content = fileRead(absolute);
41+
42+
expect(content contains canonical).toBeTrue(
43+
relPath & " should reference " & canonical
44+
& " — the URL ##2888 standardized on for the deprecated /wheels/mcp transport."
45+
);
46+
47+
expect(content contains phantomPath).toBeFalse(
48+
relPath & " still references the phantom path " & phantomPath
49+
& " that has never existed in the repo."
50+
);
51+
});
52+
})(rel);
53+
}
54+
55+
});
56+
57+
}
58+
59+
}

‎web/sites/guides/src/content/docs/v4-0-0/digging-deeper/packages.mdx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ Every package declares a `package.json` at its root. The loader parses it before
8888
"services": [],
8989
"middleware": []
9090
},
91-
"dependencies": {}
91+
"requires": {}
9292
}
9393
```
9494

0 commit comments

Comments
 (0)