@intellectronica/ruler 0.3.13 → 0.3.14
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
|
@@ -49,6 +49,37 @@ class CrushAgent {
|
|
|
49
49
|
mcp: path.join(projectRoot, '.crush.json'),
|
|
50
50
|
};
|
|
51
51
|
}
|
|
52
|
+
/**
|
|
53
|
+
* Transform MCP server types for Crush compatibility.
|
|
54
|
+
* Crush expects "http" for HTTP servers and "sse" for SSE servers, not "remote".
|
|
55
|
+
*/
|
|
56
|
+
transformMcpServersForCrush(mcpServers) {
|
|
57
|
+
const transformedServers = {};
|
|
58
|
+
for (const [name, serverDef] of Object.entries(mcpServers)) {
|
|
59
|
+
if (serverDef && typeof serverDef === 'object') {
|
|
60
|
+
const server = serverDef;
|
|
61
|
+
const transformedServer = { ...server };
|
|
62
|
+
// Transform type: "remote" to appropriate Crush types
|
|
63
|
+
if (server.type === 'remote' &&
|
|
64
|
+
server.url &&
|
|
65
|
+
typeof server.url === 'string') {
|
|
66
|
+
const url = server.url;
|
|
67
|
+
// Check if URL suggests SSE (contains /sse path segment)
|
|
68
|
+
if (/\/sse(\/|$)/i.test(url)) {
|
|
69
|
+
transformedServer.type = 'sse';
|
|
70
|
+
}
|
|
71
|
+
else {
|
|
72
|
+
transformedServer.type = 'http';
|
|
73
|
+
}
|
|
74
|
+
}
|
|
75
|
+
transformedServers[name] = transformedServer;
|
|
76
|
+
}
|
|
77
|
+
else {
|
|
78
|
+
transformedServers[name] = serverDef;
|
|
79
|
+
}
|
|
80
|
+
}
|
|
81
|
+
return transformedServers;
|
|
82
|
+
}
|
|
52
83
|
async applyRulerConfig(concatenatedRules, projectRoot, rulerMcpJson, agentConfig) {
|
|
53
84
|
const outputPaths = this.getDefaultOutputPath(projectRoot);
|
|
54
85
|
const instructionsPath = agentConfig?.outputPathInstructions ?? outputPaths['instructions'];
|
|
@@ -59,24 +90,27 @@ class CrushAgent {
|
|
|
59
90
|
try {
|
|
60
91
|
const existingMcpConfig = JSON.parse(await fs.readFile(mcpPath, 'utf-8'));
|
|
61
92
|
if (existingMcpConfig && typeof existingMcpConfig === 'object') {
|
|
93
|
+
const transformedServers = this.transformMcpServersForCrush((rulerMcpJson?.mcpServers ?? {}));
|
|
62
94
|
finalMcpConfig = {
|
|
63
95
|
...existingMcpConfig,
|
|
64
96
|
mcp: {
|
|
65
97
|
...(existingMcpConfig.mcp || {}),
|
|
66
|
-
...
|
|
98
|
+
...transformedServers,
|
|
67
99
|
},
|
|
68
100
|
};
|
|
69
101
|
}
|
|
70
102
|
else if (rulerMcpJson) {
|
|
103
|
+
const transformedServers = this.transformMcpServersForCrush((rulerMcpJson?.mcpServers ?? {}));
|
|
71
104
|
finalMcpConfig = {
|
|
72
|
-
mcp:
|
|
105
|
+
mcp: transformedServers,
|
|
73
106
|
};
|
|
74
107
|
}
|
|
75
108
|
}
|
|
76
109
|
catch {
|
|
77
110
|
if (rulerMcpJson) {
|
|
111
|
+
const transformedServers = this.transformMcpServersForCrush((rulerMcpJson?.mcpServers ?? {}));
|
|
78
112
|
finalMcpConfig = {
|
|
79
|
-
mcp:
|
|
113
|
+
mcp: transformedServers,
|
|
80
114
|
};
|
|
81
115
|
}
|
|
82
116
|
}
|
|
@@ -442,7 +442,8 @@ async function applyMcpConfiguration(agent, filteredMcpJson, dest, agentConfig,
|
|
|
442
442
|
// Agents that handle MCP configuration internally should not have external MCP handling
|
|
443
443
|
if (agent.getIdentifier() === 'zed' ||
|
|
444
444
|
agent.getIdentifier() === 'gemini-cli' ||
|
|
445
|
-
agent.getIdentifier() === 'amazon-q-cli'
|
|
445
|
+
agent.getIdentifier() === 'amazon-q-cli' ||
|
|
446
|
+
agent.getIdentifier() === 'crush') {
|
|
446
447
|
(0, constants_1.logVerbose)(`Skipping external MCP config for ${agent.getName()} - handled internally by agent`, verbose);
|
|
447
448
|
return;
|
|
448
449
|
}
|