@aiready/cli 0.12.12 → 0.12.16
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.
- package/.turbo/turbo-build.log +8 -8
- package/.turbo/turbo-test.log +76 -21
- package/dist/chunk-2QOU5KKW.mjs +301 -0
- package/dist/chunk-55ZUD52M.mjs +290 -0
- package/dist/chunk-BD7XZYET.mjs +302 -0
- package/dist/chunk-VOKP7FGM.mjs +303 -0
- package/dist/cli.js +40 -15
- package/dist/cli.mjs +4 -1
- package/dist/index.js +37 -15
- package/dist/index.mjs +1 -1
- package/package.json +12 -12
- package/src/__tests__/config-shape.test.ts +106 -0
- package/src/commands/scan.ts +3 -0
- package/src/index.ts +44 -17
|
@@ -0,0 +1,303 @@
|
|
|
1
|
+
var __require = /* @__PURE__ */ ((x) => typeof require !== "undefined" ? require : typeof Proxy !== "undefined" ? new Proxy(x, {
|
|
2
|
+
get: (a, b) => (typeof require !== "undefined" ? require : a)[b]
|
|
3
|
+
}) : x)(function(x) {
|
|
4
|
+
if (typeof require !== "undefined") return require.apply(this, arguments);
|
|
5
|
+
throw Error('Dynamic require of "' + x + '" is not supported');
|
|
6
|
+
});
|
|
7
|
+
|
|
8
|
+
// src/index.ts
|
|
9
|
+
import {
|
|
10
|
+
ToolRegistry,
|
|
11
|
+
ToolName,
|
|
12
|
+
calculateOverallScore,
|
|
13
|
+
calculateTokenBudget,
|
|
14
|
+
GLOBAL_INFRA_OPTIONS,
|
|
15
|
+
COMMON_FINE_TUNING_OPTIONS,
|
|
16
|
+
initializeParsers
|
|
17
|
+
} from "@aiready/core";
|
|
18
|
+
import "@aiready/pattern-detect";
|
|
19
|
+
import "@aiready/context-analyzer";
|
|
20
|
+
import "@aiready/consistency";
|
|
21
|
+
import "@aiready/ai-signal-clarity";
|
|
22
|
+
import "@aiready/agent-grounding";
|
|
23
|
+
import "@aiready/testability";
|
|
24
|
+
import "@aiready/doc-drift";
|
|
25
|
+
import "@aiready/deps";
|
|
26
|
+
import "@aiready/change-amplification";
|
|
27
|
+
var TOOL_PACKAGE_MAP = {
|
|
28
|
+
[ToolName.PatternDetect]: "@aiready/pattern-detect",
|
|
29
|
+
[ToolName.ContextAnalyzer]: "@aiready/context-analyzer",
|
|
30
|
+
[ToolName.NamingConsistency]: "@aiready/consistency",
|
|
31
|
+
[ToolName.AiSignalClarity]: "@aiready/ai-signal-clarity",
|
|
32
|
+
[ToolName.AgentGrounding]: "@aiready/agent-grounding",
|
|
33
|
+
[ToolName.TestabilityIndex]: "@aiready/testability",
|
|
34
|
+
[ToolName.DocDrift]: "@aiready/doc-drift",
|
|
35
|
+
[ToolName.DependencyHealth]: "@aiready/deps",
|
|
36
|
+
[ToolName.ChangeAmplification]: "@aiready/change-amplification",
|
|
37
|
+
// Aliases handled by registry
|
|
38
|
+
patterns: "@aiready/pattern-detect",
|
|
39
|
+
duplicates: "@aiready/pattern-detect",
|
|
40
|
+
context: "@aiready/context-analyzer",
|
|
41
|
+
fragmentation: "@aiready/context-analyzer",
|
|
42
|
+
consistency: "@aiready/consistency",
|
|
43
|
+
"ai-signal": "@aiready/ai-signal-clarity",
|
|
44
|
+
grounding: "@aiready/agent-grounding",
|
|
45
|
+
testability: "@aiready/testability",
|
|
46
|
+
"deps-health": "@aiready/deps",
|
|
47
|
+
"change-amp": "@aiready/change-amplification"
|
|
48
|
+
};
|
|
49
|
+
function sanitizeConfigRecursive(obj) {
|
|
50
|
+
if (!obj || typeof obj !== "object" || Array.isArray(obj)) return obj;
|
|
51
|
+
const sanitized = {};
|
|
52
|
+
const infraToStrip = [
|
|
53
|
+
"rootDir",
|
|
54
|
+
"onProgress",
|
|
55
|
+
"progressCallback",
|
|
56
|
+
"streamResults",
|
|
57
|
+
"batchSize",
|
|
58
|
+
"useSmartDefaults"
|
|
59
|
+
];
|
|
60
|
+
for (const [key, value] of Object.entries(obj)) {
|
|
61
|
+
if (infraToStrip.includes(key)) continue;
|
|
62
|
+
if (typeof value === "object" && value !== null && !Array.isArray(value)) {
|
|
63
|
+
sanitized[key] = sanitizeConfigRecursive(value);
|
|
64
|
+
} else {
|
|
65
|
+
sanitized[key] = value;
|
|
66
|
+
}
|
|
67
|
+
}
|
|
68
|
+
return sanitized;
|
|
69
|
+
}
|
|
70
|
+
function sanitizeToolConfig(config) {
|
|
71
|
+
return sanitizeConfigRecursive(config);
|
|
72
|
+
}
|
|
73
|
+
async function analyzeUnified(options) {
|
|
74
|
+
await initializeParsers();
|
|
75
|
+
const startTime = Date.now();
|
|
76
|
+
const requestedTools = options.tools || [
|
|
77
|
+
"patterns",
|
|
78
|
+
"context",
|
|
79
|
+
"consistency"
|
|
80
|
+
];
|
|
81
|
+
const result = {
|
|
82
|
+
summary: {
|
|
83
|
+
totalIssues: 0,
|
|
84
|
+
criticalIssues: 0,
|
|
85
|
+
// Added as per instruction
|
|
86
|
+
majorIssues: 0,
|
|
87
|
+
// Added as per instruction
|
|
88
|
+
totalFiles: 0,
|
|
89
|
+
toolsRun: [],
|
|
90
|
+
executionTime: 0,
|
|
91
|
+
config: options,
|
|
92
|
+
toolConfigs: {}
|
|
93
|
+
}
|
|
94
|
+
};
|
|
95
|
+
for (const toolName of requestedTools) {
|
|
96
|
+
let provider = ToolRegistry.find(toolName);
|
|
97
|
+
if (!provider) {
|
|
98
|
+
const packageName = TOOL_PACKAGE_MAP[toolName] || (toolName.startsWith("@aiready/") ? toolName : `@aiready/${toolName}`);
|
|
99
|
+
try {
|
|
100
|
+
await import(packageName);
|
|
101
|
+
provider = ToolRegistry.find(toolName);
|
|
102
|
+
if (provider) {
|
|
103
|
+
console.log(
|
|
104
|
+
`\u2705 Successfully loaded tool provider: ${toolName} from ${packageName}`
|
|
105
|
+
);
|
|
106
|
+
} else {
|
|
107
|
+
console.log(
|
|
108
|
+
`\u26A0\uFE0F Loaded ${packageName} but provider ${toolName} still not found in registry.`
|
|
109
|
+
);
|
|
110
|
+
}
|
|
111
|
+
} catch (err) {
|
|
112
|
+
console.log(
|
|
113
|
+
`\u274C Failed to dynamically load tool ${toolName} (${packageName}):`,
|
|
114
|
+
err.message
|
|
115
|
+
);
|
|
116
|
+
}
|
|
117
|
+
}
|
|
118
|
+
if (!provider) {
|
|
119
|
+
console.warn(
|
|
120
|
+
`\u26A0\uFE0F Warning: Tool provider for '${toolName}' not found. Skipping.`
|
|
121
|
+
);
|
|
122
|
+
continue;
|
|
123
|
+
}
|
|
124
|
+
try {
|
|
125
|
+
const sanitizedOptions = { ...options };
|
|
126
|
+
delete sanitizedOptions.onProgress;
|
|
127
|
+
delete sanitizedOptions.progressCallback;
|
|
128
|
+
const toolOptions = {
|
|
129
|
+
rootDir: options.rootDir
|
|
130
|
+
// Always include rootDir
|
|
131
|
+
};
|
|
132
|
+
[...GLOBAL_INFRA_OPTIONS, ...COMMON_FINE_TUNING_OPTIONS].forEach(
|
|
133
|
+
(key) => {
|
|
134
|
+
if (key in options && key !== "toolConfigs" && key !== "tools") {
|
|
135
|
+
toolOptions[key] = options[key];
|
|
136
|
+
}
|
|
137
|
+
}
|
|
138
|
+
);
|
|
139
|
+
if (options.toolConfigs?.[provider.id]) {
|
|
140
|
+
Object.assign(toolOptions, options.toolConfigs[provider.id]);
|
|
141
|
+
} else if (options.tools && !Array.isArray(options.tools) && typeof options.tools === "object" && options.tools[provider.id]) {
|
|
142
|
+
Object.assign(toolOptions, options.tools[provider.id]);
|
|
143
|
+
} else if (options[provider.id]) {
|
|
144
|
+
Object.assign(toolOptions, options[provider.id]);
|
|
145
|
+
}
|
|
146
|
+
toolOptions.onProgress = (processed, total, message) => {
|
|
147
|
+
if (options.progressCallback) {
|
|
148
|
+
options.progressCallback({
|
|
149
|
+
tool: provider.id,
|
|
150
|
+
processed,
|
|
151
|
+
total,
|
|
152
|
+
message
|
|
153
|
+
});
|
|
154
|
+
}
|
|
155
|
+
};
|
|
156
|
+
const output = await provider.analyze(toolOptions);
|
|
157
|
+
if (output.metadata) {
|
|
158
|
+
output.metadata.config = sanitizeToolConfig(toolOptions);
|
|
159
|
+
}
|
|
160
|
+
if (options.progressCallback) {
|
|
161
|
+
options.progressCallback({ tool: provider.id, data: output });
|
|
162
|
+
}
|
|
163
|
+
result[provider.id] = output;
|
|
164
|
+
result.summary.toolsRun.push(provider.id);
|
|
165
|
+
if (output.summary?.config) {
|
|
166
|
+
result.summary.toolConfigs[provider.id] = sanitizeToolConfig(
|
|
167
|
+
output.summary.config
|
|
168
|
+
);
|
|
169
|
+
} else if (output.metadata?.config) {
|
|
170
|
+
result.summary.toolConfigs[provider.id] = sanitizeToolConfig(
|
|
171
|
+
output.metadata.config
|
|
172
|
+
);
|
|
173
|
+
} else {
|
|
174
|
+
result.summary.toolConfigs[provider.id] = sanitizeToolConfig(toolOptions);
|
|
175
|
+
}
|
|
176
|
+
const toolFiles = output.summary?.totalFiles || output.summary?.filesAnalyzed || 0;
|
|
177
|
+
if (toolFiles > result.summary.totalFiles) {
|
|
178
|
+
result.summary.totalFiles = toolFiles;
|
|
179
|
+
}
|
|
180
|
+
const issueCount = output.results.reduce(
|
|
181
|
+
(sum, file) => sum + (file.issues?.length || 0),
|
|
182
|
+
0
|
|
183
|
+
);
|
|
184
|
+
result.summary.totalIssues += issueCount;
|
|
185
|
+
if (provider.alias && Array.isArray(provider.alias)) {
|
|
186
|
+
for (const alias of provider.alias) {
|
|
187
|
+
if (!result[alias]) {
|
|
188
|
+
result[alias] = output;
|
|
189
|
+
}
|
|
190
|
+
}
|
|
191
|
+
}
|
|
192
|
+
const camelCaseId = provider.id.replace(
|
|
193
|
+
/-([a-z])/g,
|
|
194
|
+
(g) => g[1].toUpperCase()
|
|
195
|
+
);
|
|
196
|
+
if (camelCaseId !== provider.id && !result[camelCaseId]) {
|
|
197
|
+
result[camelCaseId] = output;
|
|
198
|
+
}
|
|
199
|
+
} catch (err) {
|
|
200
|
+
console.error(`\u274C Error running tool '${provider.id}':`, err);
|
|
201
|
+
}
|
|
202
|
+
}
|
|
203
|
+
result.summary.config = sanitizeConfigRecursive({
|
|
204
|
+
scan: {
|
|
205
|
+
tools: requestedTools,
|
|
206
|
+
include: options.include,
|
|
207
|
+
exclude: options.exclude
|
|
208
|
+
},
|
|
209
|
+
// Use 'tools' for tool-specific configurations to match AIReadyConfig
|
|
210
|
+
tools: result.summary.toolConfigs
|
|
211
|
+
});
|
|
212
|
+
result.summary.executionTime = Date.now() - startTime;
|
|
213
|
+
return result;
|
|
214
|
+
}
|
|
215
|
+
async function scoreUnified(results, options) {
|
|
216
|
+
const toolScores = /* @__PURE__ */ new Map();
|
|
217
|
+
for (const toolId of results.summary.toolsRun) {
|
|
218
|
+
const provider = ToolRegistry.get(toolId);
|
|
219
|
+
if (!provider) continue;
|
|
220
|
+
const output = results[toolId];
|
|
221
|
+
if (!output) continue;
|
|
222
|
+
try {
|
|
223
|
+
const toolScore = provider.score(output, options);
|
|
224
|
+
if (!toolScore.tokenBudget) {
|
|
225
|
+
if (toolId === ToolName.PatternDetect && output.duplicates) {
|
|
226
|
+
const wastedTokens = output.duplicates.reduce(
|
|
227
|
+
(sum, d) => sum + (d.tokenCost || 0),
|
|
228
|
+
0
|
|
229
|
+
);
|
|
230
|
+
toolScore.tokenBudget = calculateTokenBudget({
|
|
231
|
+
totalContextTokens: wastedTokens * 2,
|
|
232
|
+
wastedTokens: {
|
|
233
|
+
duplication: wastedTokens,
|
|
234
|
+
fragmentation: 0,
|
|
235
|
+
chattiness: 0
|
|
236
|
+
}
|
|
237
|
+
});
|
|
238
|
+
} else if (toolId === ToolName.ContextAnalyzer && output.summary) {
|
|
239
|
+
toolScore.tokenBudget = calculateTokenBudget({
|
|
240
|
+
totalContextTokens: output.summary.totalTokens,
|
|
241
|
+
wastedTokens: {
|
|
242
|
+
duplication: 0,
|
|
243
|
+
fragmentation: output.summary.totalPotentialSavings || 0,
|
|
244
|
+
chattiness: 0
|
|
245
|
+
}
|
|
246
|
+
});
|
|
247
|
+
}
|
|
248
|
+
}
|
|
249
|
+
toolScores.set(toolId, toolScore);
|
|
250
|
+
} catch (err) {
|
|
251
|
+
console.error(`\u274C Error scoring tool '${toolId}':`, err);
|
|
252
|
+
}
|
|
253
|
+
}
|
|
254
|
+
if (toolScores.size === 0) {
|
|
255
|
+
return {
|
|
256
|
+
overall: 0,
|
|
257
|
+
rating: "Critical",
|
|
258
|
+
timestamp: (/* @__PURE__ */ new Date()).toISOString(),
|
|
259
|
+
toolsUsed: [],
|
|
260
|
+
breakdown: [],
|
|
261
|
+
calculation: {
|
|
262
|
+
formula: "0 / 0 = 0",
|
|
263
|
+
weights: {},
|
|
264
|
+
normalized: "0 / 0 = 0"
|
|
265
|
+
}
|
|
266
|
+
};
|
|
267
|
+
}
|
|
268
|
+
return calculateOverallScore(toolScores, options, void 0);
|
|
269
|
+
}
|
|
270
|
+
function generateUnifiedSummary(result) {
|
|
271
|
+
const { summary } = result;
|
|
272
|
+
let output = `\u{1F680} AIReady Analysis Complete
|
|
273
|
+
|
|
274
|
+
`;
|
|
275
|
+
output += `\u{1F4CA} Summary:
|
|
276
|
+
`;
|
|
277
|
+
output += ` Tools run: ${summary.toolsRun.join(", ")}
|
|
278
|
+
`;
|
|
279
|
+
output += ` Total issues found: ${summary.totalIssues}
|
|
280
|
+
`;
|
|
281
|
+
output += ` Execution time: ${(summary.executionTime / 1e3).toFixed(2)}s
|
|
282
|
+
|
|
283
|
+
`;
|
|
284
|
+
for (const provider of ToolRegistry.getAll()) {
|
|
285
|
+
const toolResult = result[provider.id];
|
|
286
|
+
if (toolResult) {
|
|
287
|
+
const issueCount = toolResult.results.reduce(
|
|
288
|
+
(sum, r) => sum + (r.issues?.length || 0),
|
|
289
|
+
0
|
|
290
|
+
);
|
|
291
|
+
output += `\u2022 ${provider.id}: ${issueCount} issues
|
|
292
|
+
`;
|
|
293
|
+
}
|
|
294
|
+
}
|
|
295
|
+
return output;
|
|
296
|
+
}
|
|
297
|
+
|
|
298
|
+
export {
|
|
299
|
+
__require,
|
|
300
|
+
analyzeUnified,
|
|
301
|
+
scoreUnified,
|
|
302
|
+
generateUnifiedSummary
|
|
303
|
+
};
|
package/dist/cli.js
CHANGED
|
@@ -68,17 +68,32 @@ var TOOL_PACKAGE_MAP = {
|
|
|
68
68
|
"deps-health": "@aiready/deps",
|
|
69
69
|
"change-amp": "@aiready/change-amplification"
|
|
70
70
|
};
|
|
71
|
-
function
|
|
72
|
-
if (!
|
|
73
|
-
const sanitized = {
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
71
|
+
function sanitizeConfigRecursive(obj) {
|
|
72
|
+
if (!obj || typeof obj !== "object" || Array.isArray(obj)) return obj;
|
|
73
|
+
const sanitized = {};
|
|
74
|
+
const infraToStrip = [
|
|
75
|
+
"rootDir",
|
|
76
|
+
"onProgress",
|
|
77
|
+
"progressCallback",
|
|
78
|
+
"streamResults",
|
|
79
|
+
"batchSize",
|
|
80
|
+
"useSmartDefaults"
|
|
81
|
+
];
|
|
82
|
+
for (const [key, value] of Object.entries(obj)) {
|
|
83
|
+
if (infraToStrip.includes(key)) continue;
|
|
84
|
+
if (typeof value === "object" && value !== null && !Array.isArray(value)) {
|
|
85
|
+
sanitized[key] = sanitizeConfigRecursive(value);
|
|
86
|
+
} else {
|
|
87
|
+
sanitized[key] = value;
|
|
77
88
|
}
|
|
78
|
-
}
|
|
89
|
+
}
|
|
79
90
|
return sanitized;
|
|
80
91
|
}
|
|
92
|
+
function sanitizeToolConfig(config) {
|
|
93
|
+
return sanitizeConfigRecursive(config);
|
|
94
|
+
}
|
|
81
95
|
async function analyzeUnified(options) {
|
|
96
|
+
await (0, import_core.initializeParsers)();
|
|
82
97
|
const startTime = Date.now();
|
|
83
98
|
const requestedTools = options.tools || [
|
|
84
99
|
"patterns",
|
|
@@ -129,22 +144,24 @@ async function analyzeUnified(options) {
|
|
|
129
144
|
continue;
|
|
130
145
|
}
|
|
131
146
|
try {
|
|
132
|
-
const
|
|
133
|
-
delete
|
|
134
|
-
delete
|
|
147
|
+
const sanitizedOptions = { ...options };
|
|
148
|
+
delete sanitizedOptions.onProgress;
|
|
149
|
+
delete sanitizedOptions.progressCallback;
|
|
135
150
|
const toolOptions = {
|
|
136
151
|
rootDir: options.rootDir
|
|
137
152
|
// Always include rootDir
|
|
138
153
|
};
|
|
139
154
|
[...import_core.GLOBAL_INFRA_OPTIONS, ...import_core.COMMON_FINE_TUNING_OPTIONS].forEach(
|
|
140
155
|
(key) => {
|
|
141
|
-
if (key in options && key !== "toolConfigs") {
|
|
156
|
+
if (key in options && key !== "toolConfigs" && key !== "tools") {
|
|
142
157
|
toolOptions[key] = options[key];
|
|
143
158
|
}
|
|
144
159
|
}
|
|
145
160
|
);
|
|
146
161
|
if (options.toolConfigs?.[provider.id]) {
|
|
147
162
|
Object.assign(toolOptions, options.toolConfigs[provider.id]);
|
|
163
|
+
} else if (options.tools && !Array.isArray(options.tools) && typeof options.tools === "object" && options.tools[provider.id]) {
|
|
164
|
+
Object.assign(toolOptions, options.tools[provider.id]);
|
|
148
165
|
} else if (options[provider.id]) {
|
|
149
166
|
Object.assign(toolOptions, options[provider.id]);
|
|
150
167
|
}
|
|
@@ -205,10 +222,15 @@ async function analyzeUnified(options) {
|
|
|
205
222
|
console.error(`\u274C Error running tool '${provider.id}':`, err);
|
|
206
223
|
}
|
|
207
224
|
}
|
|
208
|
-
result.summary.config = {
|
|
209
|
-
|
|
210
|
-
|
|
211
|
-
|
|
225
|
+
result.summary.config = sanitizeConfigRecursive({
|
|
226
|
+
scan: {
|
|
227
|
+
tools: requestedTools,
|
|
228
|
+
include: options.include,
|
|
229
|
+
exclude: options.exclude
|
|
230
|
+
},
|
|
231
|
+
// Use 'tools' for tool-specific configurations to match AIReadyConfig
|
|
232
|
+
tools: result.summary.toolConfigs
|
|
233
|
+
});
|
|
212
234
|
result.summary.executionTime = Date.now() - startTime;
|
|
213
235
|
return result;
|
|
214
236
|
}
|
|
@@ -608,6 +630,9 @@ async function scanAction(directory, options) {
|
|
|
608
630
|
console.log(
|
|
609
631
|
` Total issues (all tools): ${import_chalk3.default.bold(String(results.summary.totalIssues || 0))}`
|
|
610
632
|
);
|
|
633
|
+
console.log(
|
|
634
|
+
` Execution time: ${import_chalk3.default.bold(((Date.now() - startTime) / 1e3).toFixed(2) + "s")}`
|
|
635
|
+
);
|
|
611
636
|
let scoringResult;
|
|
612
637
|
if (options.score || finalOptions.scoring?.showBreakdown) {
|
|
613
638
|
scoringResult = await scoreUnified(results, finalOptions);
|
package/dist/cli.mjs
CHANGED
|
@@ -3,7 +3,7 @@ import {
|
|
|
3
3
|
__require,
|
|
4
4
|
analyzeUnified,
|
|
5
5
|
scoreUnified
|
|
6
|
-
} from "./chunk-
|
|
6
|
+
} from "./chunk-VOKP7FGM.mjs";
|
|
7
7
|
|
|
8
8
|
// src/cli.ts
|
|
9
9
|
import { Command } from "commander";
|
|
@@ -370,6 +370,9 @@ async function scanAction(directory, options) {
|
|
|
370
370
|
console.log(
|
|
371
371
|
` Total issues (all tools): ${chalk3.bold(String(results.summary.totalIssues || 0))}`
|
|
372
372
|
);
|
|
373
|
+
console.log(
|
|
374
|
+
` Execution time: ${chalk3.bold(((Date.now() - startTime) / 1e3).toFixed(2) + "s")}`
|
|
375
|
+
);
|
|
373
376
|
let scoringResult;
|
|
374
377
|
if (options.score || finalOptions.scoring?.showBreakdown) {
|
|
375
378
|
scoringResult = await scoreUnified(results, finalOptions);
|
package/dist/index.js
CHANGED
|
@@ -57,17 +57,32 @@ var TOOL_PACKAGE_MAP = {
|
|
|
57
57
|
"deps-health": "@aiready/deps",
|
|
58
58
|
"change-amp": "@aiready/change-amplification"
|
|
59
59
|
};
|
|
60
|
-
function
|
|
61
|
-
if (!
|
|
62
|
-
const sanitized = {
|
|
63
|
-
|
|
64
|
-
|
|
65
|
-
|
|
60
|
+
function sanitizeConfigRecursive(obj) {
|
|
61
|
+
if (!obj || typeof obj !== "object" || Array.isArray(obj)) return obj;
|
|
62
|
+
const sanitized = {};
|
|
63
|
+
const infraToStrip = [
|
|
64
|
+
"rootDir",
|
|
65
|
+
"onProgress",
|
|
66
|
+
"progressCallback",
|
|
67
|
+
"streamResults",
|
|
68
|
+
"batchSize",
|
|
69
|
+
"useSmartDefaults"
|
|
70
|
+
];
|
|
71
|
+
for (const [key, value] of Object.entries(obj)) {
|
|
72
|
+
if (infraToStrip.includes(key)) continue;
|
|
73
|
+
if (typeof value === "object" && value !== null && !Array.isArray(value)) {
|
|
74
|
+
sanitized[key] = sanitizeConfigRecursive(value);
|
|
75
|
+
} else {
|
|
76
|
+
sanitized[key] = value;
|
|
66
77
|
}
|
|
67
|
-
}
|
|
78
|
+
}
|
|
68
79
|
return sanitized;
|
|
69
80
|
}
|
|
81
|
+
function sanitizeToolConfig(config) {
|
|
82
|
+
return sanitizeConfigRecursive(config);
|
|
83
|
+
}
|
|
70
84
|
async function analyzeUnified(options) {
|
|
85
|
+
await (0, import_core.initializeParsers)();
|
|
71
86
|
const startTime = Date.now();
|
|
72
87
|
const requestedTools = options.tools || [
|
|
73
88
|
"patterns",
|
|
@@ -118,22 +133,24 @@ async function analyzeUnified(options) {
|
|
|
118
133
|
continue;
|
|
119
134
|
}
|
|
120
135
|
try {
|
|
121
|
-
const
|
|
122
|
-
delete
|
|
123
|
-
delete
|
|
136
|
+
const sanitizedOptions = { ...options };
|
|
137
|
+
delete sanitizedOptions.onProgress;
|
|
138
|
+
delete sanitizedOptions.progressCallback;
|
|
124
139
|
const toolOptions = {
|
|
125
140
|
rootDir: options.rootDir
|
|
126
141
|
// Always include rootDir
|
|
127
142
|
};
|
|
128
143
|
[...import_core.GLOBAL_INFRA_OPTIONS, ...import_core.COMMON_FINE_TUNING_OPTIONS].forEach(
|
|
129
144
|
(key) => {
|
|
130
|
-
if (key in options && key !== "toolConfigs") {
|
|
145
|
+
if (key in options && key !== "toolConfigs" && key !== "tools") {
|
|
131
146
|
toolOptions[key] = options[key];
|
|
132
147
|
}
|
|
133
148
|
}
|
|
134
149
|
);
|
|
135
150
|
if (options.toolConfigs?.[provider.id]) {
|
|
136
151
|
Object.assign(toolOptions, options.toolConfigs[provider.id]);
|
|
152
|
+
} else if (options.tools && !Array.isArray(options.tools) && typeof options.tools === "object" && options.tools[provider.id]) {
|
|
153
|
+
Object.assign(toolOptions, options.tools[provider.id]);
|
|
137
154
|
} else if (options[provider.id]) {
|
|
138
155
|
Object.assign(toolOptions, options[provider.id]);
|
|
139
156
|
}
|
|
@@ -194,10 +211,15 @@ async function analyzeUnified(options) {
|
|
|
194
211
|
console.error(`\u274C Error running tool '${provider.id}':`, err);
|
|
195
212
|
}
|
|
196
213
|
}
|
|
197
|
-
result.summary.config = {
|
|
198
|
-
|
|
199
|
-
|
|
200
|
-
|
|
214
|
+
result.summary.config = sanitizeConfigRecursive({
|
|
215
|
+
scan: {
|
|
216
|
+
tools: requestedTools,
|
|
217
|
+
include: options.include,
|
|
218
|
+
exclude: options.exclude
|
|
219
|
+
},
|
|
220
|
+
// Use 'tools' for tool-specific configurations to match AIReadyConfig
|
|
221
|
+
tools: result.summary.toolConfigs
|
|
222
|
+
});
|
|
201
223
|
result.summary.executionTime = Date.now() - startTime;
|
|
202
224
|
return result;
|
|
203
225
|
}
|
package/dist/index.mjs
CHANGED
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@aiready/cli",
|
|
3
|
-
"version": "0.12.
|
|
3
|
+
"version": "0.12.16",
|
|
4
4
|
"description": "Unified CLI for AIReady analysis tools",
|
|
5
5
|
"main": "./dist/index.js",
|
|
6
6
|
"module": "./dist/index.mjs",
|
|
@@ -11,17 +11,17 @@
|
|
|
11
11
|
"dependencies": {
|
|
12
12
|
"chalk": "^5.3.0",
|
|
13
13
|
"commander": "^14.0.0",
|
|
14
|
-
"@aiready/agent-grounding": "0.11.
|
|
15
|
-
"@aiready/consistency": "0.18.
|
|
16
|
-
"@aiready/
|
|
17
|
-
"@aiready/deps": "0.11.
|
|
18
|
-
"@aiready/doc-drift": "0.11.
|
|
19
|
-
"@aiready/
|
|
20
|
-
"@aiready/
|
|
21
|
-
"@aiready/
|
|
22
|
-
"@aiready/
|
|
23
|
-
"@aiready/testability": "0.4.
|
|
24
|
-
"@aiready/
|
|
14
|
+
"@aiready/agent-grounding": "0.11.14",
|
|
15
|
+
"@aiready/consistency": "0.18.14",
|
|
16
|
+
"@aiready/context-analyzer": "0.19.14",
|
|
17
|
+
"@aiready/deps": "0.11.14",
|
|
18
|
+
"@aiready/doc-drift": "0.11.14",
|
|
19
|
+
"@aiready/change-amplification": "0.11.14",
|
|
20
|
+
"@aiready/pattern-detect": "0.14.14",
|
|
21
|
+
"@aiready/visualizer": "0.4.15",
|
|
22
|
+
"@aiready/ai-signal-clarity": "0.11.14",
|
|
23
|
+
"@aiready/testability": "0.4.14",
|
|
24
|
+
"@aiready/core": "0.21.14"
|
|
25
25
|
},
|
|
26
26
|
"devDependencies": {
|
|
27
27
|
"@types/node": "^24.0.0",
|
|
@@ -0,0 +1,106 @@
|
|
|
1
|
+
import { describe, it, expect, beforeEach, afterEach } from 'vitest';
|
|
2
|
+
import { analyzeUnified } from '../index';
|
|
3
|
+
import { ToolRegistry, ToolName, SpokeOutputSchema } from '@aiready/core';
|
|
4
|
+
|
|
5
|
+
describe('CLI Configuration Shape', () => {
|
|
6
|
+
beforeEach(() => {
|
|
7
|
+
ToolRegistry.clear();
|
|
8
|
+
|
|
9
|
+
// Register a mock provider that returns its input config in metadata
|
|
10
|
+
ToolRegistry.register({
|
|
11
|
+
id: ToolName.PatternDetect,
|
|
12
|
+
alias: ['patterns'],
|
|
13
|
+
analyze: async (options) =>
|
|
14
|
+
SpokeOutputSchema.parse({
|
|
15
|
+
results: [],
|
|
16
|
+
summary: { config: options },
|
|
17
|
+
metadata: {
|
|
18
|
+
toolName: ToolName.PatternDetect,
|
|
19
|
+
version: '1.0.0',
|
|
20
|
+
config: options,
|
|
21
|
+
},
|
|
22
|
+
}),
|
|
23
|
+
score: () => ({
|
|
24
|
+
toolName: ToolName.PatternDetect,
|
|
25
|
+
score: 80,
|
|
26
|
+
factors: [],
|
|
27
|
+
recommendations: [],
|
|
28
|
+
rawMetrics: {},
|
|
29
|
+
}),
|
|
30
|
+
defaultWeight: 10,
|
|
31
|
+
});
|
|
32
|
+
});
|
|
33
|
+
|
|
34
|
+
afterEach(() => {
|
|
35
|
+
ToolRegistry.clear();
|
|
36
|
+
});
|
|
37
|
+
|
|
38
|
+
it('should generate a strictly portable AIReadyConfig in summary', async () => {
|
|
39
|
+
const results = await analyzeUnified({
|
|
40
|
+
rootDir: '/tmp/fake-repo',
|
|
41
|
+
tools: [ToolName.PatternDetect],
|
|
42
|
+
exclude: ['**/node_modules/**'],
|
|
43
|
+
// Pass a tool-specific override
|
|
44
|
+
toolConfigs: {
|
|
45
|
+
[ToolName.PatternDetect]: {
|
|
46
|
+
minSimilarity: 0.9,
|
|
47
|
+
// This should be stripped
|
|
48
|
+
rootDir: '/tmp/fake-repo',
|
|
49
|
+
},
|
|
50
|
+
},
|
|
51
|
+
});
|
|
52
|
+
|
|
53
|
+
const config = results.summary.config;
|
|
54
|
+
|
|
55
|
+
// 1. Check top-level structure
|
|
56
|
+
expect(config).toHaveProperty('scan');
|
|
57
|
+
expect(config).toHaveProperty('tools');
|
|
58
|
+
|
|
59
|
+
// 2. Ensure rootDir is STRIPPED from top level
|
|
60
|
+
expect(config).not.toHaveProperty('rootDir');
|
|
61
|
+
|
|
62
|
+
// 3. Ensure internal keys are stripped from scan section
|
|
63
|
+
expect(config.scan).toHaveProperty('tools');
|
|
64
|
+
expect(config.scan).toHaveProperty('exclude');
|
|
65
|
+
expect(config.scan).not.toHaveProperty('rootDir');
|
|
66
|
+
|
|
67
|
+
// 4. Ensure recursive stripping in tools section
|
|
68
|
+
const patternConfig = config.tools[ToolName.PatternDetect];
|
|
69
|
+
expect(patternConfig).toHaveProperty('minSimilarity', 0.9);
|
|
70
|
+
expect(patternConfig).not.toHaveProperty('rootDir');
|
|
71
|
+
expect(patternConfig).not.toHaveProperty('onProgress');
|
|
72
|
+
});
|
|
73
|
+
|
|
74
|
+
it('should strip internal keys like useSmartDefaults and batchSize', async () => {
|
|
75
|
+
const results = await analyzeUnified({
|
|
76
|
+
rootDir: '/test',
|
|
77
|
+
tools: [ToolName.PatternDetect],
|
|
78
|
+
useSmartDefaults: true,
|
|
79
|
+
// @ts-ignore - testing internal key stripping
|
|
80
|
+
batchSize: 50,
|
|
81
|
+
});
|
|
82
|
+
|
|
83
|
+
const config = results.summary.config;
|
|
84
|
+
|
|
85
|
+
expect(config).not.toHaveProperty('useSmartDefaults');
|
|
86
|
+
expect(config.scan).not.toHaveProperty('useSmartDefaults');
|
|
87
|
+
|
|
88
|
+
// Check tool level too
|
|
89
|
+
const patternConfig = config.tools[ToolName.PatternDetect];
|
|
90
|
+
expect(patternConfig).not.toHaveProperty('useSmartDefaults');
|
|
91
|
+
expect(patternConfig).not.toHaveProperty('batchSize');
|
|
92
|
+
});
|
|
93
|
+
|
|
94
|
+
it('should produce a config that is compatible with tool specific collection', async () => {
|
|
95
|
+
// This test ensures that the toolConfigs collected from individual tools
|
|
96
|
+
// are also sanitized before being merged into the final report.
|
|
97
|
+
const results = await analyzeUnified({
|
|
98
|
+
rootDir: '/test',
|
|
99
|
+
tools: [ToolName.PatternDetect],
|
|
100
|
+
});
|
|
101
|
+
|
|
102
|
+
const toolConfigs = results.summary.toolConfigs;
|
|
103
|
+
expect(toolConfigs).toBeDefined();
|
|
104
|
+
expect(toolConfigs![ToolName.PatternDetect]).not.toHaveProperty('rootDir');
|
|
105
|
+
});
|
|
106
|
+
});
|
package/src/commands/scan.ts
CHANGED
|
@@ -192,6 +192,9 @@ export async function scanAction(directory: string, options: ScanOptions) {
|
|
|
192
192
|
console.log(
|
|
193
193
|
` Total issues (all tools): ${chalk.bold(String(results.summary.totalIssues || 0))}`
|
|
194
194
|
);
|
|
195
|
+
console.log(
|
|
196
|
+
` Execution time: ${chalk.bold(((Date.now() - startTime) / 1000).toFixed(2) + 's')}`
|
|
197
|
+
);
|
|
195
198
|
|
|
196
199
|
let scoringResult: ScoringResult | undefined;
|
|
197
200
|
if (options.score || finalOptions.scoring?.showBreakdown) {
|