@skillsmith/mcp-server 0.1.0
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/dist/.tsbuildinfo +1 -0
- package/dist/src/__tests__/get-skill.test.d.ts +6 -0
- package/dist/src/__tests__/get-skill.test.d.ts.map +1 -0
- package/dist/src/__tests__/get-skill.test.js +88 -0
- package/dist/src/__tests__/get-skill.test.js.map +1 -0
- package/dist/src/__tests__/middleware/errorFormatter.test.d.ts +7 -0
- package/dist/src/__tests__/middleware/errorFormatter.test.d.ts.map +1 -0
- package/dist/src/__tests__/middleware/errorFormatter.test.js +304 -0
- package/dist/src/__tests__/middleware/errorFormatter.test.js.map +1 -0
- package/dist/src/__tests__/middleware/license.test.d.ts +7 -0
- package/dist/src/__tests__/middleware/license.test.d.ts.map +1 -0
- package/dist/src/__tests__/middleware/license.test.js +500 -0
- package/dist/src/__tests__/middleware/license.test.js.map +1 -0
- package/dist/src/__tests__/search.test.d.ts +6 -0
- package/dist/src/__tests__/search.test.d.ts.map +1 -0
- package/dist/src/__tests__/search.test.js +86 -0
- package/dist/src/__tests__/search.test.js.map +1 -0
- package/dist/src/__tests__/test-utils.d.ts +19 -0
- package/dist/src/__tests__/test-utils.d.ts.map +1 -0
- package/dist/src/__tests__/test-utils.js +87 -0
- package/dist/src/__tests__/test-utils.js.map +1 -0
- package/dist/src/context/index.d.ts +19 -0
- package/dist/src/context/index.d.ts.map +1 -0
- package/dist/src/context/index.js +25 -0
- package/dist/src/context/index.js.map +1 -0
- package/dist/src/context/project-detector.d.ts +145 -0
- package/dist/src/context/project-detector.d.ts.map +1 -0
- package/dist/src/context/project-detector.js +321 -0
- package/dist/src/context/project-detector.js.map +1 -0
- package/dist/src/context.d.ts +100 -0
- package/dist/src/context.d.ts.map +1 -0
- package/dist/src/context.js +157 -0
- package/dist/src/context.js.map +1 -0
- package/dist/src/core-shim.d.ts +7 -0
- package/dist/src/core-shim.d.ts.map +1 -0
- package/dist/src/core-shim.js +9 -0
- package/dist/src/core-shim.js.map +1 -0
- package/dist/src/health/healthCheck.d.ts +88 -0
- package/dist/src/health/healthCheck.d.ts.map +1 -0
- package/dist/src/health/healthCheck.js +117 -0
- package/dist/src/health/healthCheck.js.map +1 -0
- package/dist/src/health/index.d.ts +21 -0
- package/dist/src/health/index.d.ts.map +1 -0
- package/dist/src/health/index.js +21 -0
- package/dist/src/health/index.js.map +1 -0
- package/dist/src/health/readinessCheck.d.ts +139 -0
- package/dist/src/health/readinessCheck.d.ts.map +1 -0
- package/dist/src/health/readinessCheck.js +266 -0
- package/dist/src/health/readinessCheck.js.map +1 -0
- package/dist/src/index.d.ts +8 -0
- package/dist/src/index.d.ts.map +1 -0
- package/dist/src/index.js +178 -0
- package/dist/src/index.js.map +1 -0
- package/dist/src/index.test.d.ts +2 -0
- package/dist/src/index.test.d.ts.map +1 -0
- package/dist/src/index.test.js +43 -0
- package/dist/src/index.test.js.map +1 -0
- package/dist/src/logger.d.ts +26 -0
- package/dist/src/logger.d.ts.map +1 -0
- package/dist/src/logger.js +179 -0
- package/dist/src/logger.js.map +1 -0
- package/dist/src/middleware/__tests__/csp.test.d.ts +2 -0
- package/dist/src/middleware/__tests__/csp.test.d.ts.map +1 -0
- package/dist/src/middleware/__tests__/csp.test.js +389 -0
- package/dist/src/middleware/__tests__/csp.test.js.map +1 -0
- package/dist/src/middleware/csp.d.ts +87 -0
- package/dist/src/middleware/csp.d.ts.map +1 -0
- package/dist/src/middleware/csp.js +273 -0
- package/dist/src/middleware/csp.js.map +1 -0
- package/dist/src/middleware/degradation.d.ts +99 -0
- package/dist/src/middleware/degradation.d.ts.map +1 -0
- package/dist/src/middleware/degradation.js +315 -0
- package/dist/src/middleware/degradation.js.map +1 -0
- package/dist/src/middleware/errorFormatter.d.ts +119 -0
- package/dist/src/middleware/errorFormatter.d.ts.map +1 -0
- package/dist/src/middleware/errorFormatter.js +294 -0
- package/dist/src/middleware/errorFormatter.js.map +1 -0
- package/dist/src/middleware/index.d.ts +10 -0
- package/dist/src/middleware/index.d.ts.map +1 -0
- package/dist/src/middleware/index.js +14 -0
- package/dist/src/middleware/index.js.map +1 -0
- package/dist/src/middleware/license.d.ts +161 -0
- package/dist/src/middleware/license.d.ts.map +1 -0
- package/dist/src/middleware/license.js +281 -0
- package/dist/src/middleware/license.js.map +1 -0
- package/dist/src/middleware/toolFeatureMapping.d.ts +36 -0
- package/dist/src/middleware/toolFeatureMapping.d.ts.map +1 -0
- package/dist/src/middleware/toolFeatureMapping.js +90 -0
- package/dist/src/middleware/toolFeatureMapping.js.map +1 -0
- package/dist/src/onboarding/first-run.d.ts +64 -0
- package/dist/src/onboarding/first-run.d.ts.map +1 -0
- package/dist/src/onboarding/first-run.js +77 -0
- package/dist/src/onboarding/first-run.js.map +1 -0
- package/dist/src/onboarding/index.d.ts +7 -0
- package/dist/src/onboarding/index.d.ts.map +1 -0
- package/dist/src/onboarding/index.js +7 -0
- package/dist/src/onboarding/index.js.map +1 -0
- package/dist/src/suggestions/index.d.ts +21 -0
- package/dist/src/suggestions/index.d.ts.map +1 -0
- package/dist/src/suggestions/index.js +20 -0
- package/dist/src/suggestions/index.js.map +1 -0
- package/dist/src/suggestions/suggestion-engine.d.ts +185 -0
- package/dist/src/suggestions/suggestion-engine.d.ts.map +1 -0
- package/dist/src/suggestions/suggestion-engine.js +352 -0
- package/dist/src/suggestions/suggestion-engine.js.map +1 -0
- package/dist/src/suggestions/types.d.ts +88 -0
- package/dist/src/suggestions/types.d.ts.map +1 -0
- package/dist/src/suggestions/types.js +21 -0
- package/dist/src/suggestions/types.js.map +1 -0
- package/dist/src/tools/analyze.d.ts +151 -0
- package/dist/src/tools/analyze.d.ts.map +1 -0
- package/dist/src/tools/analyze.js +205 -0
- package/dist/src/tools/analyze.js.map +1 -0
- package/dist/src/tools/compare.d.ts +149 -0
- package/dist/src/tools/compare.d.ts.map +1 -0
- package/dist/src/tools/compare.js +464 -0
- package/dist/src/tools/compare.js.map +1 -0
- package/dist/src/tools/get-skill.d.ts +116 -0
- package/dist/src/tools/get-skill.d.ts.map +1 -0
- package/dist/src/tools/get-skill.js +224 -0
- package/dist/src/tools/get-skill.js.map +1 -0
- package/dist/src/tools/index.d.ts +20 -0
- package/dist/src/tools/index.d.ts.map +1 -0
- package/dist/src/tools/index.js +20 -0
- package/dist/src/tools/index.js.map +1 -0
- package/dist/src/tools/install.d.ts +122 -0
- package/dist/src/tools/install.d.ts.map +1 -0
- package/dist/src/tools/install.js +314 -0
- package/dist/src/tools/install.js.map +1 -0
- package/dist/src/tools/recommend.d.ts +171 -0
- package/dist/src/tools/recommend.d.ts.map +1 -0
- package/dist/src/tools/recommend.js +325 -0
- package/dist/src/tools/recommend.js.map +1 -0
- package/dist/src/tools/search.d.ts +121 -0
- package/dist/src/tools/search.d.ts.map +1 -0
- package/dist/src/tools/search.js +249 -0
- package/dist/src/tools/search.js.map +1 -0
- package/dist/src/tools/suggest.d.ts +181 -0
- package/dist/src/tools/suggest.d.ts.map +1 -0
- package/dist/src/tools/suggest.js +342 -0
- package/dist/src/tools/suggest.js.map +1 -0
- package/dist/src/tools/uninstall.d.ts +123 -0
- package/dist/src/tools/uninstall.d.ts.map +1 -0
- package/dist/src/tools/uninstall.js +250 -0
- package/dist/src/tools/uninstall.js.map +1 -0
- package/dist/src/tools/validate.d.ts +122 -0
- package/dist/src/tools/validate.d.ts.map +1 -0
- package/dist/src/tools/validate.js +497 -0
- package/dist/src/tools/validate.js.map +1 -0
- package/dist/src/utils/installed-skills.d.ts +101 -0
- package/dist/src/utils/installed-skills.d.ts.map +1 -0
- package/dist/src/utils/installed-skills.js +220 -0
- package/dist/src/utils/installed-skills.js.map +1 -0
- package/dist/src/utils/validation.d.ts +76 -0
- package/dist/src/utils/validation.d.ts.map +1 -0
- package/dist/src/utils/validation.js +153 -0
- package/dist/src/utils/validation.js.map +1 -0
- package/dist/src/webhooks/index.d.ts +8 -0
- package/dist/src/webhooks/index.d.ts.map +1 -0
- package/dist/src/webhooks/index.js +9 -0
- package/dist/src/webhooks/index.js.map +1 -0
- package/dist/src/webhooks/webhook-endpoint.d.ts +149 -0
- package/dist/src/webhooks/webhook-endpoint.d.ts.map +1 -0
- package/dist/src/webhooks/webhook-endpoint.js +339 -0
- package/dist/src/webhooks/webhook-endpoint.js.map +1 -0
- package/dist/tests/compare.test.d.ts +6 -0
- package/dist/tests/compare.test.d.ts.map +1 -0
- package/dist/tests/compare.test.js +225 -0
- package/dist/tests/compare.test.js.map +1 -0
- package/dist/tests/context/project-detector.test.d.ts +6 -0
- package/dist/tests/context/project-detector.test.d.ts.map +1 -0
- package/dist/tests/context/project-detector.test.js +719 -0
- package/dist/tests/context/project-detector.test.js.map +1 -0
- package/dist/tests/e2e/compare.e2e.test.d.ts +10 -0
- package/dist/tests/e2e/compare.e2e.test.d.ts.map +1 -0
- package/dist/tests/e2e/compare.e2e.test.js +286 -0
- package/dist/tests/e2e/compare.e2e.test.js.map +1 -0
- package/dist/tests/e2e/install-flow.e2e.test.d.ts +10 -0
- package/dist/tests/e2e/install-flow.e2e.test.d.ts.map +1 -0
- package/dist/tests/e2e/install-flow.e2e.test.js +209 -0
- package/dist/tests/e2e/install-flow.e2e.test.js.map +1 -0
- package/dist/tests/e2e/recommend.e2e.test.d.ts +12 -0
- package/dist/tests/e2e/recommend.e2e.test.d.ts.map +1 -0
- package/dist/tests/e2e/recommend.e2e.test.js +347 -0
- package/dist/tests/e2e/recommend.e2e.test.js.map +1 -0
- package/dist/tests/e2e/skill-flow.e2e.test.d.ts +10 -0
- package/dist/tests/e2e/skill-flow.e2e.test.d.ts.map +1 -0
- package/dist/tests/e2e/skill-flow.e2e.test.js +280 -0
- package/dist/tests/e2e/skill-flow.e2e.test.js.map +1 -0
- package/dist/tests/e2e/suggest.e2e.test.d.ts +13 -0
- package/dist/tests/e2e/suggest.e2e.test.d.ts.map +1 -0
- package/dist/tests/e2e/suggest.e2e.test.js +347 -0
- package/dist/tests/e2e/suggest.e2e.test.js.map +1 -0
- package/dist/tests/e2e/utils/baseline-collector.d.ts +107 -0
- package/dist/tests/e2e/utils/baseline-collector.d.ts.map +1 -0
- package/dist/tests/e2e/utils/baseline-collector.js +211 -0
- package/dist/tests/e2e/utils/baseline-collector.js.map +1 -0
- package/dist/tests/e2e/utils/hardcoded-detector.d.ts +46 -0
- package/dist/tests/e2e/utils/hardcoded-detector.d.ts.map +1 -0
- package/dist/tests/e2e/utils/hardcoded-detector.js +255 -0
- package/dist/tests/e2e/utils/hardcoded-detector.js.map +1 -0
- package/dist/tests/e2e/utils/index.d.ts +7 -0
- package/dist/tests/e2e/utils/index.d.ts.map +1 -0
- package/dist/tests/e2e/utils/index.js +7 -0
- package/dist/tests/e2e/utils/index.js.map +1 -0
- package/dist/tests/e2e/utils/linear-reporter.d.ts +60 -0
- package/dist/tests/e2e/utils/linear-reporter.d.ts.map +1 -0
- package/dist/tests/e2e/utils/linear-reporter.js +232 -0
- package/dist/tests/e2e/utils/linear-reporter.js.map +1 -0
- package/dist/tests/health.test.d.ts +9 -0
- package/dist/tests/health.test.d.ts.map +1 -0
- package/dist/tests/health.test.js +308 -0
- package/dist/tests/health.test.js.map +1 -0
- package/dist/tests/integration/analyze.integration.test.d.ts +2 -0
- package/dist/tests/integration/analyze.integration.test.d.ts.map +1 -0
- package/dist/tests/integration/analyze.integration.test.js +244 -0
- package/dist/tests/integration/analyze.integration.test.js.map +1 -0
- package/dist/tests/integration/compare.integration.test.d.ts +2 -0
- package/dist/tests/integration/compare.integration.test.d.ts.map +1 -0
- package/dist/tests/integration/compare.integration.test.js +120 -0
- package/dist/tests/integration/compare.integration.test.js.map +1 -0
- package/dist/tests/integration/fixtures/test-skills.d.ts +62 -0
- package/dist/tests/integration/fixtures/test-skills.d.ts.map +1 -0
- package/dist/tests/integration/fixtures/test-skills.js +644 -0
- package/dist/tests/integration/fixtures/test-skills.js.map +1 -0
- package/dist/tests/integration/get-skill.integration.test.d.ts +6 -0
- package/dist/tests/integration/get-skill.integration.test.d.ts.map +1 -0
- package/dist/tests/integration/get-skill.integration.test.js +203 -0
- package/dist/tests/integration/get-skill.integration.test.js.map +1 -0
- package/dist/tests/integration/github-api.integration.test.d.ts +14 -0
- package/dist/tests/integration/github-api.integration.test.d.ts.map +1 -0
- package/dist/tests/integration/github-api.integration.test.js +190 -0
- package/dist/tests/integration/github-api.integration.test.js.map +1 -0
- package/dist/tests/integration/install.integration.test.d.ts +6 -0
- package/dist/tests/integration/install.integration.test.d.ts.map +1 -0
- package/dist/tests/integration/install.integration.test.js +282 -0
- package/dist/tests/integration/install.integration.test.js.map +1 -0
- package/dist/tests/integration/recommend.integration.test.d.ts +2 -0
- package/dist/tests/integration/recommend.integration.test.d.ts.map +1 -0
- package/dist/tests/integration/recommend.integration.test.js +215 -0
- package/dist/tests/integration/recommend.integration.test.js.map +1 -0
- package/dist/tests/integration/search.integration.test.d.ts +6 -0
- package/dist/tests/integration/search.integration.test.d.ts.map +1 -0
- package/dist/tests/integration/search.integration.test.js +229 -0
- package/dist/tests/integration/search.integration.test.js.map +1 -0
- package/dist/tests/integration/setup.d.ts +71 -0
- package/dist/tests/integration/setup.d.ts.map +1 -0
- package/dist/tests/integration/setup.js +124 -0
- package/dist/tests/integration/setup.js.map +1 -0
- package/dist/tests/integration/uninstall.integration.test.d.ts +6 -0
- package/dist/tests/integration/uninstall.integration.test.d.ts.map +1 -0
- package/dist/tests/integration/uninstall.integration.test.js +296 -0
- package/dist/tests/integration/uninstall.integration.test.js.map +1 -0
- package/dist/tests/integration/validate.integration.test.d.ts +2 -0
- package/dist/tests/integration/validate.integration.test.d.ts.map +1 -0
- package/dist/tests/integration/validate.integration.test.js +181 -0
- package/dist/tests/integration/validate.integration.test.js.map +1 -0
- package/dist/tests/onboarding/first-run.test.d.ts +7 -0
- package/dist/tests/onboarding/first-run.test.d.ts.map +1 -0
- package/dist/tests/onboarding/first-run.test.js +258 -0
- package/dist/tests/onboarding/first-run.test.js.map +1 -0
- package/dist/tests/performance/search-performance.test.d.ts +10 -0
- package/dist/tests/performance/search-performance.test.d.ts.map +1 -0
- package/dist/tests/performance/search-performance.test.js +218 -0
- package/dist/tests/performance/search-performance.test.js.map +1 -0
- package/dist/tests/recommend.test.d.ts +6 -0
- package/dist/tests/recommend.test.d.ts.map +1 -0
- package/dist/tests/recommend.test.js +208 -0
- package/dist/tests/recommend.test.js.map +1 -0
- package/dist/tests/suggestions/suggestion-engine.test.d.ts +6 -0
- package/dist/tests/suggestions/suggestion-engine.test.d.ts.map +1 -0
- package/dist/tests/suggestions/suggestion-engine.test.js +448 -0
- package/dist/tests/suggestions/suggestion-engine.test.js.map +1 -0
- package/dist/tests/test-utils.d.ts +74 -0
- package/dist/tests/test-utils.d.ts.map +1 -0
- package/dist/tests/test-utils.js +98 -0
- package/dist/tests/test-utils.js.map +1 -0
- package/dist/tests/tools.test.d.ts +5 -0
- package/dist/tests/tools.test.d.ts.map +1 -0
- package/dist/tests/tools.test.js +138 -0
- package/dist/tests/tools.test.js.map +1 -0
- package/dist/tests/unit/installed-skills.test.d.ts +6 -0
- package/dist/tests/unit/installed-skills.test.d.ts.map +1 -0
- package/dist/tests/unit/installed-skills.test.js +285 -0
- package/dist/tests/unit/installed-skills.test.js.map +1 -0
- package/dist/tests/unit/logger.test.d.ts +6 -0
- package/dist/tests/unit/logger.test.d.ts.map +1 -0
- package/dist/tests/unit/logger.test.js +281 -0
- package/dist/tests/unit/logger.test.js.map +1 -0
- package/dist/tests/validate.test.d.ts +5 -0
- package/dist/tests/validate.test.d.ts.map +1 -0
- package/dist/tests/validate.test.js +303 -0
- package/dist/tests/validate.test.js.map +1 -0
- package/dist/tests/webhooks/proxy-trust.security.test.d.ts +8 -0
- package/dist/tests/webhooks/proxy-trust.security.test.d.ts.map +1 -0
- package/dist/tests/webhooks/proxy-trust.security.test.js +145 -0
- package/dist/tests/webhooks/proxy-trust.security.test.js.map +1 -0
- package/dist/tests/webhooks/rate-limiter.security.test.d.ts +8 -0
- package/dist/tests/webhooks/rate-limiter.security.test.d.ts.map +1 -0
- package/dist/tests/webhooks/rate-limiter.security.test.js +122 -0
- package/dist/tests/webhooks/rate-limiter.security.test.js.map +1 -0
- package/dist/vitest.config.d.ts +6 -0
- package/dist/vitest.config.d.ts.map +1 -0
- package/dist/vitest.config.js +13 -0
- package/dist/vitest.config.js.map +1 -0
- package/package.json +63 -0
|
@@ -0,0 +1,339 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* SMI-645: Webhook Endpoint - HTTP server for GitHub webhooks
|
|
3
|
+
*
|
|
4
|
+
* Provides:
|
|
5
|
+
* - Express/Node.js HTTP server for receiving webhooks
|
|
6
|
+
* - Signature validation middleware
|
|
7
|
+
* - Rate limiting for security
|
|
8
|
+
* - Event routing to WebhookHandler
|
|
9
|
+
*
|
|
10
|
+
* Usage:
|
|
11
|
+
* import { createWebhookServer, startWebhookServer } from './webhooks/webhook-endpoint.js';
|
|
12
|
+
*
|
|
13
|
+
* const server = createWebhookServer({
|
|
14
|
+
* secret: process.env.GITHUB_WEBHOOK_SECRET,
|
|
15
|
+
* onIndexUpdate: (repoUrl, filePath) => { ... },
|
|
16
|
+
* });
|
|
17
|
+
*
|
|
18
|
+
* startWebhookServer(server, { port: 3000 });
|
|
19
|
+
*/
|
|
20
|
+
import { createServer } from 'http';
|
|
21
|
+
import { WebhookHandler, WebhookQueue, } from '@skillsmith/core';
|
|
22
|
+
/**
|
|
23
|
+
* Create rate limiter with automatic cleanup (SMI-681)
|
|
24
|
+
* @param limit - Maximum requests per window
|
|
25
|
+
* @param windowMs - Window duration in milliseconds
|
|
26
|
+
*/
|
|
27
|
+
export function createRateLimiter(limit, windowMs) {
|
|
28
|
+
const state = {
|
|
29
|
+
requests: new Map(),
|
|
30
|
+
limit,
|
|
31
|
+
window: windowMs,
|
|
32
|
+
};
|
|
33
|
+
// SMI-681: Periodic cleanup to prevent memory leak
|
|
34
|
+
state.cleanupTimer = setInterval(() => {
|
|
35
|
+
const now = Date.now();
|
|
36
|
+
const windowStart = now - windowMs;
|
|
37
|
+
for (const [ip, timestamps] of state.requests.entries()) {
|
|
38
|
+
const valid = timestamps.filter((t) => t > windowStart);
|
|
39
|
+
if (valid.length === 0) {
|
|
40
|
+
state.requests.delete(ip);
|
|
41
|
+
}
|
|
42
|
+
else {
|
|
43
|
+
state.requests.set(ip, valid);
|
|
44
|
+
}
|
|
45
|
+
}
|
|
46
|
+
}, windowMs);
|
|
47
|
+
// Don't block process exit
|
|
48
|
+
if (state.cleanupTimer.unref) {
|
|
49
|
+
state.cleanupTimer.unref();
|
|
50
|
+
}
|
|
51
|
+
return state;
|
|
52
|
+
}
|
|
53
|
+
/**
|
|
54
|
+
* Destroy rate limiter and clean up resources (SMI-681)
|
|
55
|
+
*/
|
|
56
|
+
export function destroyRateLimiter(state) {
|
|
57
|
+
if (state.cleanupTimer) {
|
|
58
|
+
clearInterval(state.cleanupTimer);
|
|
59
|
+
state.cleanupTimer = undefined;
|
|
60
|
+
}
|
|
61
|
+
state.requests.clear();
|
|
62
|
+
}
|
|
63
|
+
/**
|
|
64
|
+
* Check if request is rate limited
|
|
65
|
+
*/
|
|
66
|
+
export function isRateLimited(limiter, ip) {
|
|
67
|
+
const now = Date.now();
|
|
68
|
+
const windowStart = now - limiter.window;
|
|
69
|
+
// Get existing requests for this IP
|
|
70
|
+
let requests = limiter.requests.get(ip) || [];
|
|
71
|
+
// Filter to only requests within the window
|
|
72
|
+
requests = requests.filter((time) => time > windowStart);
|
|
73
|
+
// Check if over limit
|
|
74
|
+
if (requests.length >= limiter.limit) {
|
|
75
|
+
return true;
|
|
76
|
+
}
|
|
77
|
+
// Add this request
|
|
78
|
+
requests.push(now);
|
|
79
|
+
limiter.requests.set(ip, requests);
|
|
80
|
+
return false;
|
|
81
|
+
}
|
|
82
|
+
/**
|
|
83
|
+
* Get client IP from request (SMI-682: Added trusted proxy validation)
|
|
84
|
+
* @param req - Incoming HTTP request
|
|
85
|
+
* @param config - Server configuration with trust proxy settings
|
|
86
|
+
*/
|
|
87
|
+
export function getClientIp(req, config) {
|
|
88
|
+
// SMI-682: Only trust X-Forwarded-For if explicitly configured
|
|
89
|
+
if (config.trustProxy) {
|
|
90
|
+
const forwarded = req.headers['x-forwarded-for'];
|
|
91
|
+
if (typeof forwarded === 'string') {
|
|
92
|
+
const clientIp = forwarded.split(',')[0].trim();
|
|
93
|
+
// If trustedProxies specified, verify the request came from one
|
|
94
|
+
if (config.trustedProxies?.length) {
|
|
95
|
+
const remoteIp = req.socket.remoteAddress;
|
|
96
|
+
if (!config.trustedProxies.includes(remoteIp || '')) {
|
|
97
|
+
// Don't trust forwarded header from untrusted source
|
|
98
|
+
return remoteIp || 'unknown';
|
|
99
|
+
}
|
|
100
|
+
}
|
|
101
|
+
return clientIp;
|
|
102
|
+
}
|
|
103
|
+
if (Array.isArray(forwarded) && forwarded.length > 0) {
|
|
104
|
+
const clientIp = forwarded[0].split(',')[0].trim();
|
|
105
|
+
// If trustedProxies specified, verify the request came from one
|
|
106
|
+
if (config.trustedProxies?.length) {
|
|
107
|
+
const remoteIp = req.socket.remoteAddress;
|
|
108
|
+
if (!config.trustedProxies.includes(remoteIp || '')) {
|
|
109
|
+
return remoteIp || 'unknown';
|
|
110
|
+
}
|
|
111
|
+
}
|
|
112
|
+
return clientIp;
|
|
113
|
+
}
|
|
114
|
+
}
|
|
115
|
+
// Fall back to socket address
|
|
116
|
+
return req.socket.remoteAddress || 'unknown';
|
|
117
|
+
}
|
|
118
|
+
/**
|
|
119
|
+
* Read request body with size limit
|
|
120
|
+
*/
|
|
121
|
+
async function readBody(req, maxSize) {
|
|
122
|
+
return new Promise((resolve, reject) => {
|
|
123
|
+
const chunks = [];
|
|
124
|
+
let size = 0;
|
|
125
|
+
req.on('data', (chunk) => {
|
|
126
|
+
size += chunk.length;
|
|
127
|
+
if (size > maxSize) {
|
|
128
|
+
req.destroy();
|
|
129
|
+
reject(new Error('Request body too large'));
|
|
130
|
+
return;
|
|
131
|
+
}
|
|
132
|
+
chunks.push(chunk);
|
|
133
|
+
});
|
|
134
|
+
req.on('end', () => {
|
|
135
|
+
resolve(Buffer.concat(chunks).toString('utf-8'));
|
|
136
|
+
});
|
|
137
|
+
req.on('error', reject);
|
|
138
|
+
});
|
|
139
|
+
}
|
|
140
|
+
/**
|
|
141
|
+
* Send JSON response
|
|
142
|
+
*/
|
|
143
|
+
function sendJson(res, statusCode, data) {
|
|
144
|
+
res.writeHead(statusCode, { 'Content-Type': 'application/json' });
|
|
145
|
+
res.end(JSON.stringify(data));
|
|
146
|
+
}
|
|
147
|
+
/**
|
|
148
|
+
* Create a webhook server
|
|
149
|
+
*/
|
|
150
|
+
export function createWebhookServer(options) {
|
|
151
|
+
const { secret, maxBodySize = 1024 * 1024, // 1MB
|
|
152
|
+
rateLimit = 100, rateLimitWindow = 60000, // 1 minute
|
|
153
|
+
trustProxy = false, // SMI-682: Default to not trusting proxy
|
|
154
|
+
trustedProxies, onIndexUpdate, onLog = () => { }, queueOptions = {}, } = options;
|
|
155
|
+
// SMI-682: Config for getClientIp
|
|
156
|
+
const serverConfig = {
|
|
157
|
+
secret,
|
|
158
|
+
trustProxy,
|
|
159
|
+
trustedProxies,
|
|
160
|
+
};
|
|
161
|
+
// Create rate limiter
|
|
162
|
+
const rateLimiter = createRateLimiter(rateLimit, rateLimitWindow);
|
|
163
|
+
// Create queue with processor
|
|
164
|
+
const queue = new WebhookQueue({
|
|
165
|
+
debounceMs: queueOptions.debounceMs ?? 5000,
|
|
166
|
+
maxRetries: queueOptions.maxRetries ?? 3,
|
|
167
|
+
retryDelayMs: queueOptions.retryDelayMs ?? 1000,
|
|
168
|
+
processor: onIndexUpdate,
|
|
169
|
+
onLog,
|
|
170
|
+
});
|
|
171
|
+
// Create handler
|
|
172
|
+
const handler = new WebhookHandler({
|
|
173
|
+
secret,
|
|
174
|
+
queue,
|
|
175
|
+
onLog,
|
|
176
|
+
});
|
|
177
|
+
// Create HTTP server
|
|
178
|
+
const server = createServer(async (req, res) => {
|
|
179
|
+
const url = req.url || '';
|
|
180
|
+
const method = req.method || 'GET';
|
|
181
|
+
// Health check endpoint
|
|
182
|
+
if (url === '/health' && method === 'GET') {
|
|
183
|
+
sendJson(res, 200, {
|
|
184
|
+
status: 'healthy',
|
|
185
|
+
queue: queue.getStats(),
|
|
186
|
+
});
|
|
187
|
+
return;
|
|
188
|
+
}
|
|
189
|
+
// Webhook endpoint
|
|
190
|
+
if (url === '/webhooks/github' && method === 'POST') {
|
|
191
|
+
// SMI-682: Use config for trusted proxy validation
|
|
192
|
+
const clientIp = getClientIp(req, serverConfig);
|
|
193
|
+
// Rate limiting
|
|
194
|
+
if (isRateLimited(rateLimiter, clientIp)) {
|
|
195
|
+
onLog('warn', 'Rate limit exceeded', { ip: clientIp });
|
|
196
|
+
sendJson(res, 429, {
|
|
197
|
+
error: 'Too many requests',
|
|
198
|
+
retryAfter: Math.ceil(rateLimitWindow / 1000),
|
|
199
|
+
});
|
|
200
|
+
return;
|
|
201
|
+
}
|
|
202
|
+
// Get event type from header
|
|
203
|
+
const eventType = req.headers['x-github-event'];
|
|
204
|
+
if (!eventType) {
|
|
205
|
+
sendJson(res, 400, { error: 'Missing X-GitHub-Event header' });
|
|
206
|
+
return;
|
|
207
|
+
}
|
|
208
|
+
// Get signature from header
|
|
209
|
+
const signature = req.headers['x-hub-signature-256'];
|
|
210
|
+
if (!signature) {
|
|
211
|
+
sendJson(res, 401, { error: 'Missing X-Hub-Signature-256 header' });
|
|
212
|
+
return;
|
|
213
|
+
}
|
|
214
|
+
// Get delivery ID for idempotency
|
|
215
|
+
const deliveryId = req.headers['x-github-delivery'];
|
|
216
|
+
// Read body
|
|
217
|
+
let body;
|
|
218
|
+
try {
|
|
219
|
+
body = await readBody(req, maxBodySize);
|
|
220
|
+
}
|
|
221
|
+
catch (error) {
|
|
222
|
+
sendJson(res, 413, { error: 'Request body too large' });
|
|
223
|
+
return;
|
|
224
|
+
}
|
|
225
|
+
// Process webhook (pass delivery ID for idempotency)
|
|
226
|
+
let result;
|
|
227
|
+
try {
|
|
228
|
+
result = await handler.handleWebhook(eventType, body, signature, deliveryId);
|
|
229
|
+
}
|
|
230
|
+
catch (error) {
|
|
231
|
+
onLog('error', 'Webhook processing error', {
|
|
232
|
+
error: error instanceof Error ? error.message : String(error),
|
|
233
|
+
});
|
|
234
|
+
sendJson(res, 500, { error: 'Internal server error' });
|
|
235
|
+
return;
|
|
236
|
+
}
|
|
237
|
+
// Send response
|
|
238
|
+
if (result.success) {
|
|
239
|
+
sendJson(res, 200, {
|
|
240
|
+
success: true,
|
|
241
|
+
message: result.message,
|
|
242
|
+
changesDetected: result.changesDetected,
|
|
243
|
+
itemsQueued: result.itemsQueued,
|
|
244
|
+
});
|
|
245
|
+
}
|
|
246
|
+
else {
|
|
247
|
+
const statusCode = result.error?.includes('Signature') ? 401 : 400;
|
|
248
|
+
sendJson(res, statusCode, {
|
|
249
|
+
success: false,
|
|
250
|
+
error: result.error,
|
|
251
|
+
});
|
|
252
|
+
}
|
|
253
|
+
return;
|
|
254
|
+
}
|
|
255
|
+
// Queue status endpoint (for monitoring)
|
|
256
|
+
if (url === '/webhooks/status' && method === 'GET') {
|
|
257
|
+
sendJson(res, 200, {
|
|
258
|
+
queue: queue.getStats(),
|
|
259
|
+
hasPending: queue.hasPendingItems(),
|
|
260
|
+
});
|
|
261
|
+
return;
|
|
262
|
+
}
|
|
263
|
+
// 404 for unknown routes
|
|
264
|
+
sendJson(res, 404, { error: 'Not found' });
|
|
265
|
+
});
|
|
266
|
+
return { server, handler, queue };
|
|
267
|
+
}
|
|
268
|
+
/**
|
|
269
|
+
* Start the webhook server
|
|
270
|
+
*/
|
|
271
|
+
export function startWebhookServer(webhookServer, options = {}) {
|
|
272
|
+
const { port = 3000, host = '0.0.0.0' } = options;
|
|
273
|
+
return new Promise((resolve) => {
|
|
274
|
+
webhookServer.server.listen(port, host, () => {
|
|
275
|
+
console.log(`Webhook server listening on http://${host}:${port}`);
|
|
276
|
+
console.log(`GitHub webhook URL: http://${host}:${port}/webhooks/github`);
|
|
277
|
+
resolve();
|
|
278
|
+
});
|
|
279
|
+
});
|
|
280
|
+
}
|
|
281
|
+
/**
|
|
282
|
+
* Stop the webhook server
|
|
283
|
+
*/
|
|
284
|
+
export function stopWebhookServer(webhookServer) {
|
|
285
|
+
return new Promise((resolve, reject) => {
|
|
286
|
+
webhookServer.queue.clear();
|
|
287
|
+
webhookServer.server.close((err) => {
|
|
288
|
+
if (err) {
|
|
289
|
+
reject(err);
|
|
290
|
+
}
|
|
291
|
+
else {
|
|
292
|
+
resolve();
|
|
293
|
+
}
|
|
294
|
+
});
|
|
295
|
+
});
|
|
296
|
+
}
|
|
297
|
+
/**
|
|
298
|
+
* Main entry point for standalone webhook server
|
|
299
|
+
*/
|
|
300
|
+
export async function main() {
|
|
301
|
+
const secret = process.env.GITHUB_WEBHOOK_SECRET;
|
|
302
|
+
if (!secret) {
|
|
303
|
+
console.error('Error: GITHUB_WEBHOOK_SECRET environment variable is required');
|
|
304
|
+
process.exit(1);
|
|
305
|
+
}
|
|
306
|
+
const port = parseInt(process.env.WEBHOOK_PORT || '3000', 10);
|
|
307
|
+
const host = process.env.WEBHOOK_HOST || '0.0.0.0';
|
|
308
|
+
const webhookServer = createWebhookServer({
|
|
309
|
+
secret,
|
|
310
|
+
onLog: (level, message, data) => {
|
|
311
|
+
const timestamp = new Date().toISOString();
|
|
312
|
+
console.log(`[${timestamp}] [${level.toUpperCase()}] ${message}`, data ? JSON.stringify(data) : '');
|
|
313
|
+
},
|
|
314
|
+
onIndexUpdate: async (item) => {
|
|
315
|
+
// In standalone mode, just log the update
|
|
316
|
+
// In production, this would trigger re-indexing
|
|
317
|
+
console.log(`[INDEX UPDATE] ${item.type}: ${item.repoFullName} - ${item.filePath}`);
|
|
318
|
+
},
|
|
319
|
+
});
|
|
320
|
+
await startWebhookServer(webhookServer, { port, host });
|
|
321
|
+
// Handle graceful shutdown
|
|
322
|
+
const shutdown = async () => {
|
|
323
|
+
console.log('\nShutting down webhook server...');
|
|
324
|
+
await stopWebhookServer(webhookServer);
|
|
325
|
+
console.log('Webhook server stopped');
|
|
326
|
+
process.exit(0);
|
|
327
|
+
};
|
|
328
|
+
process.on('SIGINT', shutdown);
|
|
329
|
+
process.on('SIGTERM', shutdown);
|
|
330
|
+
}
|
|
331
|
+
// Run if this is the main module
|
|
332
|
+
// Note: ESM doesn't have require.main === module, so we check for CLI flag
|
|
333
|
+
if (process.argv.includes('--standalone')) {
|
|
334
|
+
main().catch((error) => {
|
|
335
|
+
console.error('Failed to start webhook server:', error);
|
|
336
|
+
process.exit(1);
|
|
337
|
+
});
|
|
338
|
+
}
|
|
339
|
+
//# sourceMappingURL=webhook-endpoint.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"webhook-endpoint.js","sourceRoot":"","sources":["../../../src/webhooks/webhook-endpoint.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAEH,OAAO,EAAE,YAAY,EAA2C,MAAM,MAAM,CAAA;AAC5E,OAAO,EACL,cAAc,EACd,YAAY,GAGb,MAAM,kBAAkB,CAAA;AAwFzB;;;;GAIG;AACH,MAAM,UAAU,iBAAiB,CAAC,KAAa,EAAE,QAAgB;IAC/D,MAAM,KAAK,GAAqB;QAC9B,QAAQ,EAAE,IAAI,GAAG,EAAE;QACnB,KAAK;QACL,MAAM,EAAE,QAAQ;KACjB,CAAA;IAED,mDAAmD;IACnD,KAAK,CAAC,YAAY,GAAG,WAAW,CAAC,GAAG,EAAE;QACpC,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAA;QACtB,MAAM,WAAW,GAAG,GAAG,GAAG,QAAQ,CAAA;QAElC,KAAK,MAAM,CAAC,EAAE,EAAE,UAAU,CAAC,IAAI,KAAK,CAAC,QAAQ,CAAC,OAAO,EAAE,EAAE,CAAC;YACxD,MAAM,KAAK,GAAG,UAAU,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,GAAG,WAAW,CAAC,CAAA;YACvD,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;gBACvB,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC,CAAA;YAC3B,CAAC;iBAAM,CAAC;gBACN,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,EAAE,KAAK,CAAC,CAAA;YAC/B,CAAC;QACH,CAAC;IACH,CAAC,EAAE,QAAQ,CAAC,CAAA;IAEZ,2BAA2B;IAC3B,IAAI,KAAK,CAAC,YAAY,CAAC,KAAK,EAAE,CAAC;QAC7B,KAAK,CAAC,YAAY,CAAC,KAAK,EAAE,CAAA;IAC5B,CAAC;IAED,OAAO,KAAK,CAAA;AACd,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,kBAAkB,CAAC,KAAuB;IACxD,IAAI,KAAK,CAAC,YAAY,EAAE,CAAC;QACvB,aAAa,CAAC,KAAK,CAAC,YAAY,CAAC,CAAA;QACjC,KAAK,CAAC,YAAY,GAAG,SAAS,CAAA;IAChC,CAAC;IACD,KAAK,CAAC,QAAQ,CAAC,KAAK,EAAE,CAAA;AACxB,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,aAAa,CAAC,OAAyB,EAAE,EAAU;IACjE,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAA;IACtB,MAAM,WAAW,GAAG,GAAG,GAAG,OAAO,CAAC,MAAM,CAAA;IAExC,oCAAoC;IACpC,IAAI,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC,IAAI,EAAE,CAAA;IAE7C,4CAA4C;IAC5C,QAAQ,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,GAAG,WAAW,CAAC,CAAA;IAExD,sBAAsB;IACtB,IAAI,QAAQ,CAAC,MAAM,IAAI,OAAO,CAAC,KAAK,EAAE,CAAC;QACrC,OAAO,IAAI,CAAA;IACb,CAAC;IAED,mBAAmB;IACnB,QAAQ,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA;IAClB,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,EAAE,QAAQ,CAAC,CAAA;IAElC,OAAO,KAAK,CAAA;AACd,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,WAAW,CAAC,GAAoB,EAAE,MAA2B;IAC3E,+DAA+D;IAC/D,IAAI,MAAM,CAAC,UAAU,EAAE,CAAC;QACtB,MAAM,SAAS,GAAG,GAAG,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAA;QAChD,IAAI,OAAO,SAAS,KAAK,QAAQ,EAAE,CAAC;YAClC,MAAM,QAAQ,GAAG,SAAS,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAA;YAE/C,gEAAgE;YAChE,IAAI,MAAM,CAAC,cAAc,EAAE,MAAM,EAAE,CAAC;gBAClC,MAAM,QAAQ,GAAG,GAAG,CAAC,MAAM,CAAC,aAAa,CAAA;gBACzC,IAAI,CAAC,MAAM,CAAC,cAAc,CAAC,QAAQ,CAAC,QAAQ,IAAI,EAAE,CAAC,EAAE,CAAC;oBACpD,qDAAqD;oBACrD,OAAO,QAAQ,IAAI,SAAS,CAAA;gBAC9B,CAAC;YACH,CAAC;YAED,OAAO,QAAQ,CAAA;QACjB,CAAC;QACD,IAAI,KAAK,CAAC,OAAO,CAAC,SAAS,CAAC,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACrD,MAAM,QAAQ,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAA;YAElD,gEAAgE;YAChE,IAAI,MAAM,CAAC,cAAc,EAAE,MAAM,EAAE,CAAC;gBAClC,MAAM,QAAQ,GAAG,GAAG,CAAC,MAAM,CAAC,aAAa,CAAA;gBACzC,IAAI,CAAC,MAAM,CAAC,cAAc,CAAC,QAAQ,CAAC,QAAQ,IAAI,EAAE,CAAC,EAAE,CAAC;oBACpD,OAAO,QAAQ,IAAI,SAAS,CAAA;gBAC9B,CAAC;YACH,CAAC;YAED,OAAO,QAAQ,CAAA;QACjB,CAAC;IACH,CAAC;IAED,8BAA8B;IAC9B,OAAO,GAAG,CAAC,MAAM,CAAC,aAAa,IAAI,SAAS,CAAA;AAC9C,CAAC;AAED;;GAEG;AACH,KAAK,UAAU,QAAQ,CAAC,GAAoB,EAAE,OAAe;IAC3D,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE;QACrC,MAAM,MAAM,GAAa,EAAE,CAAA;QAC3B,IAAI,IAAI,GAAG,CAAC,CAAA;QAEZ,GAAG,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,KAAa,EAAE,EAAE;YAC/B,IAAI,IAAI,KAAK,CAAC,MAAM,CAAA;YACpB,IAAI,IAAI,GAAG,OAAO,EAAE,CAAC;gBACnB,GAAG,CAAC,OAAO,EAAE,CAAA;gBACb,MAAM,CAAC,IAAI,KAAK,CAAC,wBAAwB,CAAC,CAAC,CAAA;gBAC3C,OAAM;YACR,CAAC;YACD,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACpB,CAAC,CAAC,CAAA;QAEF,GAAG,CAAC,EAAE,CAAC,KAAK,EAAE,GAAG,EAAE;YACjB,OAAO,CAAC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAA;QAClD,CAAC,CAAC,CAAA;QAEF,GAAG,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC,CAAA;IACzB,CAAC,CAAC,CAAA;AACJ,CAAC;AAED;;GAEG;AACH,SAAS,QAAQ,CAAC,GAAmB,EAAE,UAAkB,EAAE,IAA6B;IACtF,GAAG,CAAC,SAAS,CAAC,UAAU,EAAE,EAAE,cAAc,EAAE,kBAAkB,EAAE,CAAC,CAAA;IACjE,GAAG,CAAC,GAAG,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAA;AAC/B,CAAC;AAsBD;;GAEG;AACH,MAAM,UAAU,mBAAmB,CAAC,OAA6B;IAC/D,MAAM,EACJ,MAAM,EACN,WAAW,GAAG,IAAI,GAAG,IAAI,EAAE,MAAM;IACjC,SAAS,GAAG,GAAG,EACf,eAAe,GAAG,KAAK,EAAE,WAAW;IACpC,UAAU,GAAG,KAAK,EAAE,yCAAyC;IAC7D,cAAc,EACd,aAAa,EACb,KAAK,GAAG,GAAG,EAAE,GAAE,CAAC,EAChB,YAAY,GAAG,EAAE,GAClB,GAAG,OAAO,CAAA;IAEX,kCAAkC;IAClC,MAAM,YAAY,GAAwB;QACxC,MAAM;QACN,UAAU;QACV,cAAc;KACf,CAAA;IAED,sBAAsB;IACtB,MAAM,WAAW,GAAG,iBAAiB,CAAC,SAAS,EAAE,eAAe,CAAC,CAAA;IAEjE,8BAA8B;IAC9B,MAAM,KAAK,GAAG,IAAI,YAAY,CAAC;QAC7B,UAAU,EAAE,YAAY,CAAC,UAAU,IAAI,IAAI;QAC3C,UAAU,EAAE,YAAY,CAAC,UAAU,IAAI,CAAC;QACxC,YAAY,EAAE,YAAY,CAAC,YAAY,IAAI,IAAI;QAC/C,SAAS,EAAE,aAAa;QACxB,KAAK;KACN,CAAC,CAAA;IAEF,iBAAiB;IACjB,MAAM,OAAO,GAAG,IAAI,cAAc,CAAC;QACjC,MAAM;QACN,KAAK;QACL,KAAK;KACN,CAAC,CAAA;IAEF,qBAAqB;IACrB,MAAM,MAAM,GAAG,YAAY,CAAC,KAAK,EAAE,GAAG,EAAE,GAAG,EAAE,EAAE;QAC7C,MAAM,GAAG,GAAG,GAAG,CAAC,GAAG,IAAI,EAAE,CAAA;QACzB,MAAM,MAAM,GAAG,GAAG,CAAC,MAAM,IAAI,KAAK,CAAA;QAElC,wBAAwB;QACxB,IAAI,GAAG,KAAK,SAAS,IAAI,MAAM,KAAK,KAAK,EAAE,CAAC;YAC1C,QAAQ,CAAC,GAAG,EAAE,GAAG,EAAE;gBACjB,MAAM,EAAE,SAAS;gBACjB,KAAK,EAAE,KAAK,CAAC,QAAQ,EAAE;aACxB,CAAC,CAAA;YACF,OAAM;QACR,CAAC;QAED,mBAAmB;QACnB,IAAI,GAAG,KAAK,kBAAkB,IAAI,MAAM,KAAK,MAAM,EAAE,CAAC;YACpD,mDAAmD;YACnD,MAAM,QAAQ,GAAG,WAAW,CAAC,GAAG,EAAE,YAAY,CAAC,CAAA;YAE/C,gBAAgB;YAChB,IAAI,aAAa,CAAC,WAAW,EAAE,QAAQ,CAAC,EAAE,CAAC;gBACzC,KAAK,CAAC,MAAM,EAAE,qBAAqB,EAAE,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC,CAAA;gBACtD,QAAQ,CAAC,GAAG,EAAE,GAAG,EAAE;oBACjB,KAAK,EAAE,mBAAmB;oBAC1B,UAAU,EAAE,IAAI,CAAC,IAAI,CAAC,eAAe,GAAG,IAAI,CAAC;iBAC9C,CAAC,CAAA;gBACF,OAAM;YACR,CAAC;YAED,6BAA6B;YAC7B,MAAM,SAAS,GAAG,GAAG,CAAC,OAAO,CAAC,gBAAgB,CAAuB,CAAA;YACrE,IAAI,CAAC,SAAS,EAAE,CAAC;gBACf,QAAQ,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE,KAAK,EAAE,+BAA+B,EAAE,CAAC,CAAA;gBAC9D,OAAM;YACR,CAAC;YAED,4BAA4B;YAC5B,MAAM,SAAS,GAAG,GAAG,CAAC,OAAO,CAAC,qBAAqB,CAAuB,CAAA;YAC1E,IAAI,CAAC,SAAS,EAAE,CAAC;gBACf,QAAQ,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE,KAAK,EAAE,oCAAoC,EAAE,CAAC,CAAA;gBACnE,OAAM;YACR,CAAC;YAED,kCAAkC;YAClC,MAAM,UAAU,GAAG,GAAG,CAAC,OAAO,CAAC,mBAAmB,CAAuB,CAAA;YAEzE,YAAY;YACZ,IAAI,IAAY,CAAA;YAChB,IAAI,CAAC;gBACH,IAAI,GAAG,MAAM,QAAQ,CAAC,GAAG,EAAE,WAAW,CAAC,CAAA;YACzC,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,QAAQ,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE,KAAK,EAAE,wBAAwB,EAAE,CAAC,CAAA;gBACvD,OAAM;YACR,CAAC;YAED,qDAAqD;YACrD,IAAI,MAA2B,CAAA;YAC/B,IAAI,CAAC;gBACH,MAAM,GAAG,MAAM,OAAO,CAAC,aAAa,CAAC,SAAS,EAAE,IAAI,EAAE,SAAS,EAAE,UAAU,CAAC,CAAA;YAC9E,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,KAAK,CAAC,OAAO,EAAE,0BAA0B,EAAE;oBACzC,KAAK,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;iBAC9D,CAAC,CAAA;gBACF,QAAQ,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE,KAAK,EAAE,uBAAuB,EAAE,CAAC,CAAA;gBACtD,OAAM;YACR,CAAC;YAED,gBAAgB;YAChB,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;gBACnB,QAAQ,CAAC,GAAG,EAAE,GAAG,EAAE;oBACjB,OAAO,EAAE,IAAI;oBACb,OAAO,EAAE,MAAM,CAAC,OAAO;oBACvB,eAAe,EAAE,MAAM,CAAC,eAAe;oBACvC,WAAW,EAAE,MAAM,CAAC,WAAW;iBAChC,CAAC,CAAA;YACJ,CAAC;iBAAM,CAAC;gBACN,MAAM,UAAU,GAAG,MAAM,CAAC,KAAK,EAAE,QAAQ,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,CAAA;gBAClE,QAAQ,CAAC,GAAG,EAAE,UAAU,EAAE;oBACxB,OAAO,EAAE,KAAK;oBACd,KAAK,EAAE,MAAM,CAAC,KAAK;iBACpB,CAAC,CAAA;YACJ,CAAC;YACD,OAAM;QACR,CAAC;QAED,yCAAyC;QACzC,IAAI,GAAG,KAAK,kBAAkB,IAAI,MAAM,KAAK,KAAK,EAAE,CAAC;YACnD,QAAQ,CAAC,GAAG,EAAE,GAAG,EAAE;gBACjB,KAAK,EAAE,KAAK,CAAC,QAAQ,EAAE;gBACvB,UAAU,EAAE,KAAK,CAAC,eAAe,EAAE;aACpC,CAAC,CAAA;YACF,OAAM;QACR,CAAC;QAED,yBAAyB;QACzB,QAAQ,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE,KAAK,EAAE,WAAW,EAAE,CAAC,CAAA;IAC5C,CAAC,CAAC,CAAA;IAEF,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,KAAK,EAAE,CAAA;AACnC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,kBAAkB,CAChC,aAA4B,EAC5B,UAA8B,EAAE;IAEhC,MAAM,EAAE,IAAI,GAAG,IAAI,EAAE,IAAI,GAAG,SAAS,EAAE,GAAG,OAAO,CAAA;IAEjD,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE;QAC7B,aAAa,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE,IAAI,EAAE,GAAG,EAAE;YAC3C,OAAO,CAAC,GAAG,CAAC,sCAAsC,IAAI,IAAI,IAAI,EAAE,CAAC,CAAA;YACjE,OAAO,CAAC,GAAG,CAAC,8BAA8B,IAAI,IAAI,IAAI,kBAAkB,CAAC,CAAA;YACzE,OAAO,EAAE,CAAA;QACX,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;AACJ,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,iBAAiB,CAAC,aAA4B;IAC5D,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE;QACrC,aAAa,CAAC,KAAK,CAAC,KAAK,EAAE,CAAA;QAC3B,aAAa,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,GAAG,EAAE,EAAE;YACjC,IAAI,GAAG,EAAE,CAAC;gBACR,MAAM,CAAC,GAAG,CAAC,CAAA;YACb,CAAC;iBAAM,CAAC;gBACN,OAAO,EAAE,CAAA;YACX,CAAC;QACH,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;AACJ,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,IAAI;IACxB,MAAM,MAAM,GAAG,OAAO,CAAC,GAAG,CAAC,qBAAqB,CAAA;IAEhD,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,OAAO,CAAC,KAAK,CAAC,+DAA+D,CAAC,CAAA;QAC9E,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACjB,CAAC;IAED,MAAM,IAAI,GAAG,QAAQ,CAAC,OAAO,CAAC,GAAG,CAAC,YAAY,IAAI,MAAM,EAAE,EAAE,CAAC,CAAA;IAC7D,MAAM,IAAI,GAAG,OAAO,CAAC,GAAG,CAAC,YAAY,IAAI,SAAS,CAAA;IAElD,MAAM,aAAa,GAAG,mBAAmB,CAAC;QACxC,MAAM;QACN,KAAK,EAAE,CAAC,KAAK,EAAE,OAAO,EAAE,IAAI,EAAE,EAAE;YAC9B,MAAM,SAAS,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAA;YAC1C,OAAO,CAAC,GAAG,CACT,IAAI,SAAS,MAAM,KAAK,CAAC,WAAW,EAAE,KAAK,OAAO,EAAE,EACpD,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,EAAE,CACjC,CAAA;QACH,CAAC;QACD,aAAa,EAAE,KAAK,EAAE,IAAI,EAAE,EAAE;YAC5B,0CAA0C;YAC1C,gDAAgD;YAChD,OAAO,CAAC,GAAG,CAAC,kBAAkB,IAAI,CAAC,IAAI,KAAK,IAAI,CAAC,YAAY,MAAM,IAAI,CAAC,QAAQ,EAAE,CAAC,CAAA;QACrF,CAAC;KACF,CAAC,CAAA;IAEF,MAAM,kBAAkB,CAAC,aAAa,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC,CAAA;IAEvD,2BAA2B;IAC3B,MAAM,QAAQ,GAAG,KAAK,IAAI,EAAE;QAC1B,OAAO,CAAC,GAAG,CAAC,mCAAmC,CAAC,CAAA;QAChD,MAAM,iBAAiB,CAAC,aAAa,CAAC,CAAA;QACtC,OAAO,CAAC,GAAG,CAAC,wBAAwB,CAAC,CAAA;QACrC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACjB,CAAC,CAAA;IAED,OAAO,CAAC,EAAE,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAA;IAC9B,OAAO,CAAC,EAAE,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAA;AACjC,CAAC;AAED,iCAAiC;AACjC,2EAA2E;AAC3E,IAAI,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,cAAc,CAAC,EAAE,CAAC;IAC1C,IAAI,EAAE,CAAC,KAAK,CAAC,CAAC,KAAK,EAAE,EAAE;QACrB,OAAO,CAAC,KAAK,CAAC,iCAAiC,EAAE,KAAK,CAAC,CAAA;QACvD,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACjB,CAAC,CAAC,CAAA;AACJ,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"compare.test.d.ts","sourceRoot":"","sources":["../../tests/compare.test.ts"],"names":[],"mappings":"AAAA;;;GAGG"}
|
|
@@ -0,0 +1,225 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Tests for SMI-743: MCP Skill Compare Tool
|
|
3
|
+
* Updated for SMI-791: Wire to SkillRepository
|
|
4
|
+
*/
|
|
5
|
+
import { describe, it, expect, beforeAll, afterAll } from 'vitest';
|
|
6
|
+
import { executeCompare, formatComparisonResults, compareInputSchema, } from '../src/tools/compare.js';
|
|
7
|
+
import { SkillsmithError, ErrorCodes } from '@skillsmith/core';
|
|
8
|
+
import { createSeededTestContext } from '../src/__tests__/test-utils.js';
|
|
9
|
+
let context;
|
|
10
|
+
beforeAll(() => {
|
|
11
|
+
context = createSeededTestContext();
|
|
12
|
+
});
|
|
13
|
+
afterAll(() => {
|
|
14
|
+
context.db.close();
|
|
15
|
+
});
|
|
16
|
+
describe('Skill Compare Tool', () => {
|
|
17
|
+
describe('compareInputSchema', () => {
|
|
18
|
+
it('should require both skill_a and skill_b', () => {
|
|
19
|
+
expect(() => compareInputSchema.parse({})).toThrow();
|
|
20
|
+
expect(() => compareInputSchema.parse({ skill_a: 'a/b' })).toThrow();
|
|
21
|
+
expect(() => compareInputSchema.parse({ skill_b: 'a/b' })).toThrow();
|
|
22
|
+
});
|
|
23
|
+
it('should accept valid skill IDs', () => {
|
|
24
|
+
const result = compareInputSchema.parse({
|
|
25
|
+
skill_a: 'community/jest-helper',
|
|
26
|
+
skill_b: 'community/vitest-helper',
|
|
27
|
+
});
|
|
28
|
+
expect(result.skill_a).toBe('community/jest-helper');
|
|
29
|
+
expect(result.skill_b).toBe('community/vitest-helper');
|
|
30
|
+
});
|
|
31
|
+
it('should reject empty skill IDs', () => {
|
|
32
|
+
expect(() => compareInputSchema.parse({
|
|
33
|
+
skill_a: '',
|
|
34
|
+
skill_b: 'community/vitest-helper',
|
|
35
|
+
})).toThrow();
|
|
36
|
+
});
|
|
37
|
+
});
|
|
38
|
+
describe('executeCompare', () => {
|
|
39
|
+
it('should compare two valid skills', async () => {
|
|
40
|
+
const result = await executeCompare({
|
|
41
|
+
skill_a: 'community/jest-helper',
|
|
42
|
+
skill_b: 'community/vitest-helper',
|
|
43
|
+
}, context);
|
|
44
|
+
expect(result.comparison).toBeDefined();
|
|
45
|
+
expect(result.comparison.a).toBeDefined();
|
|
46
|
+
expect(result.comparison.b).toBeDefined();
|
|
47
|
+
expect(result.comparison.a.id).toBe('community/jest-helper');
|
|
48
|
+
expect(result.comparison.b.id).toBe('community/vitest-helper');
|
|
49
|
+
expect(result.differences).toBeDefined();
|
|
50
|
+
expect(result.differences.length).toBeGreaterThan(0);
|
|
51
|
+
expect(result.recommendation).toBeDefined();
|
|
52
|
+
expect(result.recommendation.length).toBeGreaterThan(0);
|
|
53
|
+
expect(['a', 'b', 'tie']).toContain(result.winner);
|
|
54
|
+
expect(result.timing.totalMs).toBeGreaterThanOrEqual(0);
|
|
55
|
+
});
|
|
56
|
+
it('should return skill summaries with all fields', async () => {
|
|
57
|
+
const result = await executeCompare({
|
|
58
|
+
skill_a: 'anthropic/commit',
|
|
59
|
+
skill_b: 'anthropic/review-pr',
|
|
60
|
+
}, context);
|
|
61
|
+
// Check skill A summary
|
|
62
|
+
expect(result.comparison.a.id).toBe('anthropic/commit');
|
|
63
|
+
expect(result.comparison.a.name).toBe('commit');
|
|
64
|
+
expect(result.comparison.a.description).toBeDefined();
|
|
65
|
+
expect(result.comparison.a.author).toBe('anthropic');
|
|
66
|
+
expect(result.comparison.a.quality_score).toBeGreaterThanOrEqual(0);
|
|
67
|
+
expect(result.comparison.a.trust_tier).toBeDefined();
|
|
68
|
+
expect(result.comparison.a.tags).toBeInstanceOf(Array);
|
|
69
|
+
// Check skill B summary
|
|
70
|
+
expect(result.comparison.b.id).toBe('anthropic/review-pr');
|
|
71
|
+
expect(result.comparison.b.name).toBe('review-pr');
|
|
72
|
+
});
|
|
73
|
+
it('should include quality score comparison in differences', async () => {
|
|
74
|
+
const result = await executeCompare({
|
|
75
|
+
skill_a: 'community/jest-helper',
|
|
76
|
+
skill_b: 'community/vitest-helper',
|
|
77
|
+
}, context);
|
|
78
|
+
const qualityDiff = result.differences.find((d) => d.field === 'quality_score');
|
|
79
|
+
expect(qualityDiff).toBeDefined();
|
|
80
|
+
expect(qualityDiff?.a_value).toBeDefined();
|
|
81
|
+
expect(qualityDiff?.b_value).toBeDefined();
|
|
82
|
+
expect(['a', 'b', 'tie']).toContain(qualityDiff?.winner);
|
|
83
|
+
});
|
|
84
|
+
it('should include trust tier comparison', async () => {
|
|
85
|
+
const result = await executeCompare({
|
|
86
|
+
skill_a: 'anthropic/commit',
|
|
87
|
+
skill_b: 'community/jest-helper',
|
|
88
|
+
}, context);
|
|
89
|
+
const trustDiff = result.differences.find((d) => d.field === 'trust_tier');
|
|
90
|
+
expect(trustDiff).toBeDefined();
|
|
91
|
+
expect(trustDiff?.a_value).toBe('verified');
|
|
92
|
+
expect(trustDiff?.b_value).toBe('community');
|
|
93
|
+
expect(trustDiff?.winner).toBe('a'); // verified > community
|
|
94
|
+
});
|
|
95
|
+
it('should include dependencies count comparison', async () => {
|
|
96
|
+
const result = await executeCompare({
|
|
97
|
+
skill_a: 'community/jest-helper',
|
|
98
|
+
skill_b: 'community/vitest-helper',
|
|
99
|
+
}, context);
|
|
100
|
+
const depsDiff = result.differences.find((d) => d.field === 'dependencies_count');
|
|
101
|
+
expect(depsDiff).toBeDefined();
|
|
102
|
+
expect(typeof depsDiff?.a_value).toBe('number');
|
|
103
|
+
expect(typeof depsDiff?.b_value).toBe('number');
|
|
104
|
+
});
|
|
105
|
+
it('should throw SKILL_INVALID_ID for malformed skill_a', async () => {
|
|
106
|
+
try {
|
|
107
|
+
await executeCompare({
|
|
108
|
+
skill_a: 'invalid-format',
|
|
109
|
+
skill_b: 'community/jest-helper',
|
|
110
|
+
}, context);
|
|
111
|
+
expect.fail('Should have thrown an error');
|
|
112
|
+
}
|
|
113
|
+
catch (error) {
|
|
114
|
+
expect(error).toBeInstanceOf(SkillsmithError);
|
|
115
|
+
expect(error.code).toBe(ErrorCodes.SKILL_INVALID_ID);
|
|
116
|
+
}
|
|
117
|
+
});
|
|
118
|
+
it('should throw SKILL_INVALID_ID for malformed skill_b', async () => {
|
|
119
|
+
try {
|
|
120
|
+
await executeCompare({
|
|
121
|
+
skill_a: 'community/jest-helper',
|
|
122
|
+
skill_b: 'invalid-format',
|
|
123
|
+
}, context);
|
|
124
|
+
expect.fail('Should have thrown an error');
|
|
125
|
+
}
|
|
126
|
+
catch (error) {
|
|
127
|
+
expect(error).toBeInstanceOf(SkillsmithError);
|
|
128
|
+
expect(error.code).toBe(ErrorCodes.SKILL_INVALID_ID);
|
|
129
|
+
}
|
|
130
|
+
});
|
|
131
|
+
it('should throw SKILL_NOT_FOUND for non-existent skill_a', async () => {
|
|
132
|
+
try {
|
|
133
|
+
await executeCompare({
|
|
134
|
+
skill_a: 'nonexistent/skill',
|
|
135
|
+
skill_b: 'community/jest-helper',
|
|
136
|
+
}, context);
|
|
137
|
+
expect.fail('Should have thrown an error');
|
|
138
|
+
}
|
|
139
|
+
catch (error) {
|
|
140
|
+
expect(error).toBeInstanceOf(SkillsmithError);
|
|
141
|
+
expect(error.code).toBe(ErrorCodes.SKILL_NOT_FOUND);
|
|
142
|
+
}
|
|
143
|
+
});
|
|
144
|
+
it('should throw SKILL_NOT_FOUND for non-existent skill_b', async () => {
|
|
145
|
+
try {
|
|
146
|
+
await executeCompare({
|
|
147
|
+
skill_a: 'community/jest-helper',
|
|
148
|
+
skill_b: 'nonexistent/skill',
|
|
149
|
+
}, context);
|
|
150
|
+
expect.fail('Should have thrown an error');
|
|
151
|
+
}
|
|
152
|
+
catch (error) {
|
|
153
|
+
expect(error).toBeInstanceOf(SkillsmithError);
|
|
154
|
+
expect(error.code).toBe(ErrorCodes.SKILL_NOT_FOUND);
|
|
155
|
+
}
|
|
156
|
+
});
|
|
157
|
+
it('should throw error when comparing skill with itself', async () => {
|
|
158
|
+
try {
|
|
159
|
+
await executeCompare({
|
|
160
|
+
skill_a: 'community/jest-helper',
|
|
161
|
+
skill_b: 'community/jest-helper',
|
|
162
|
+
}, context);
|
|
163
|
+
expect.fail('Should have thrown an error');
|
|
164
|
+
}
|
|
165
|
+
catch (error) {
|
|
166
|
+
expect(error).toBeInstanceOf(SkillsmithError);
|
|
167
|
+
expect(error.code).toBe(ErrorCodes.VALIDATION_INVALID_TYPE);
|
|
168
|
+
}
|
|
169
|
+
});
|
|
170
|
+
it('should generate meaningful recommendation', async () => {
|
|
171
|
+
const result = await executeCompare({
|
|
172
|
+
skill_a: 'anthropic/commit',
|
|
173
|
+
skill_b: 'community/jest-helper',
|
|
174
|
+
}, context);
|
|
175
|
+
expect(result.recommendation.length).toBeGreaterThan(20);
|
|
176
|
+
});
|
|
177
|
+
it('should determine winner based on comparison metrics', async () => {
|
|
178
|
+
const result = await executeCompare({
|
|
179
|
+
skill_a: 'anthropic/commit',
|
|
180
|
+
skill_b: 'community/docker-compose',
|
|
181
|
+
}, context);
|
|
182
|
+
expect(['a', 'b', 'tie']).toContain(result.winner);
|
|
183
|
+
});
|
|
184
|
+
});
|
|
185
|
+
describe('formatComparisonResults', () => {
|
|
186
|
+
it('should format comparison as side-by-side table', async () => {
|
|
187
|
+
const result = await executeCompare({
|
|
188
|
+
skill_a: 'community/jest-helper',
|
|
189
|
+
skill_b: 'community/vitest-helper',
|
|
190
|
+
}, context);
|
|
191
|
+
const formatted = formatComparisonResults(result);
|
|
192
|
+
expect(formatted).toContain('Skill Comparison');
|
|
193
|
+
expect(formatted).toContain('jest-helper');
|
|
194
|
+
expect(formatted).toContain('vitest-helper');
|
|
195
|
+
expect(formatted).toContain('Quality Score');
|
|
196
|
+
expect(formatted).toContain('Trust Tier');
|
|
197
|
+
});
|
|
198
|
+
it('should show winner in formatted output', async () => {
|
|
199
|
+
const result = await executeCompare({
|
|
200
|
+
skill_a: 'anthropic/commit',
|
|
201
|
+
skill_b: 'community/jest-helper',
|
|
202
|
+
}, context);
|
|
203
|
+
const formatted = formatComparisonResults(result);
|
|
204
|
+
expect(formatted).toContain('Winner:');
|
|
205
|
+
});
|
|
206
|
+
it('should include recommendation text', async () => {
|
|
207
|
+
const result = await executeCompare({
|
|
208
|
+
skill_a: 'community/jest-helper',
|
|
209
|
+
skill_b: 'community/vitest-helper',
|
|
210
|
+
}, context);
|
|
211
|
+
const formatted = formatComparisonResults(result);
|
|
212
|
+
expect(formatted).toContain('Recommendation:');
|
|
213
|
+
});
|
|
214
|
+
it('should include timing information', async () => {
|
|
215
|
+
const result = await executeCompare({
|
|
216
|
+
skill_a: 'community/jest-helper',
|
|
217
|
+
skill_b: 'community/vitest-helper',
|
|
218
|
+
}, context);
|
|
219
|
+
const formatted = formatComparisonResults(result);
|
|
220
|
+
expect(formatted).toContain('Completed in');
|
|
221
|
+
expect(formatted).toContain('ms');
|
|
222
|
+
});
|
|
223
|
+
});
|
|
224
|
+
});
|
|
225
|
+
//# sourceMappingURL=compare.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"compare.test.js","sourceRoot":"","sources":["../../tests/compare.test.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,QAAQ,CAAA;AAClE,OAAO,EACL,cAAc,EACd,uBAAuB,EACvB,kBAAkB,GACnB,MAAM,yBAAyB,CAAA;AAChC,OAAO,EAAE,eAAe,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAA;AAC9D,OAAO,EAAE,uBAAuB,EAAE,MAAM,gCAAgC,CAAA;AAGxE,IAAI,OAAoB,CAAA;AAExB,SAAS,CAAC,GAAG,EAAE;IACb,OAAO,GAAG,uBAAuB,EAAE,CAAA;AACrC,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,GAAG,EAAE;IACZ,OAAO,CAAC,EAAE,CAAC,KAAK,EAAE,CAAA;AACpB,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IAClC,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;QAClC,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;YACjD,MAAM,CAAC,GAAG,EAAE,CAAC,kBAAkB,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,EAAE,CAAA;YACpD,MAAM,CAAC,GAAG,EAAE,CAAC,kBAAkB,CAAC,KAAK,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,OAAO,EAAE,CAAA;YACpE,MAAM,CAAC,GAAG,EAAE,CAAC,kBAAkB,CAAC,KAAK,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,OAAO,EAAE,CAAA;QACtE,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,+BAA+B,EAAE,GAAG,EAAE;YACvC,MAAM,MAAM,GAAG,kBAAkB,CAAC,KAAK,CAAC;gBACtC,OAAO,EAAE,uBAAuB;gBAChC,OAAO,EAAE,yBAAyB;aACnC,CAAC,CAAA;YACF,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAA;YACpD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,yBAAyB,CAAC,CAAA;QACxD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,+BAA+B,EAAE,GAAG,EAAE;YACvC,MAAM,CAAC,GAAG,EAAE,CACV,kBAAkB,CAAC,KAAK,CAAC;gBACvB,OAAO,EAAE,EAAE;gBACX,OAAO,EAAE,yBAAyB;aACnC,CAAC,CACH,CAAC,OAAO,EAAE,CAAA;QACb,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,gBAAgB,EAAE,GAAG,EAAE;QAC9B,EAAE,CAAC,iCAAiC,EAAE,KAAK,IAAI,EAAE;YAC/C,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,uBAAuB;gBAChC,OAAO,EAAE,yBAAyB;aACnC,EACD,OAAO,CACR,CAAA;YAED,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,WAAW,EAAE,CAAA;YACvC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,CAAA;YACzC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,CAAA;YACzC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAA;YAC5D,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,yBAAyB,CAAC,CAAA;YAC9D,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,WAAW,EAAE,CAAA;YACxC,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;YACpD,MAAM,CAAC,MAAM,CAAC,cAAc,CAAC,CAAC,WAAW,EAAE,CAAA;YAC3C,MAAM,CAAC,MAAM,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;YACvD,MAAM,CAAC,CAAC,GAAG,EAAE,GAAG,EAAE,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,MAAM,CAAC,CAAA;YAClD,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAA;QACzD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;YAC7D,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,kBAAkB;gBAC3B,OAAO,EAAE,qBAAqB;aAC/B,EACD,OAAO,CACR,CAAA;YAED,wBAAwB;YACxB,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAA;YACvD,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;YAC/C,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,WAAW,EAAE,CAAA;YACrD,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAA;YACpD,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAA;YACnE,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,WAAW,EAAE,CAAA;YACpD,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,cAAc,CAAC,KAAK,CAAC,CAAA;YAEtD,wBAAwB;YACxB,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAA;YAC1D,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAA;QACpD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;YACtE,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,uBAAuB;gBAChC,OAAO,EAAE,yBAAyB;aACnC,EACD,OAAO,CACR,CAAA;YAED,MAAM,WAAW,GAAG,MAAM,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,eAAe,CAAC,CAAA;YAC/E,MAAM,CAAC,WAAW,CAAC,CAAC,WAAW,EAAE,CAAA;YACjC,MAAM,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC,WAAW,EAAE,CAAA;YAC1C,MAAM,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC,WAAW,EAAE,CAAA;YAC1C,MAAM,CAAC,CAAC,GAAG,EAAE,GAAG,EAAE,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC,WAAW,EAAE,MAAM,CAAC,CAAA;QAC1D,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,sCAAsC,EAAE,KAAK,IAAI,EAAE;YACpD,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,kBAAkB;gBAC3B,OAAO,EAAE,uBAAuB;aACjC,EACD,OAAO,CACR,CAAA;YAED,MAAM,SAAS,GAAG,MAAM,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,YAAY,CAAC,CAAA;YAC1E,MAAM,CAAC,SAAS,CAAC,CAAC,WAAW,EAAE,CAAA;YAC/B,MAAM,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;YAC3C,MAAM,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAA;YAC5C,MAAM,CAAC,SAAS,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA,CAAC,uBAAuB;QAC7D,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;YAC5D,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,uBAAuB;gBAChC,OAAO,EAAE,yBAAyB;aACnC,EACD,OAAO,CACR,CAAA;YAED,MAAM,QAAQ,GAAG,MAAM,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,oBAAoB,CAAC,CAAA;YACjF,MAAM,CAAC,QAAQ,CAAC,CAAC,WAAW,EAAE,CAAA;YAC9B,MAAM,CAAC,OAAO,QAAQ,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;YAC/C,MAAM,CAAC,OAAO,QAAQ,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACjD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;YACnE,IAAI,CAAC;gBACH,MAAM,cAAc,CAClB;oBACE,OAAO,EAAE,gBAAgB;oBACzB,OAAO,EAAE,uBAAuB;iBACjC,EACD,OAAO,CACR,CAAA;gBACD,MAAM,CAAC,IAAI,CAAC,6BAA6B,CAAC,CAAA;YAC5C,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,CAAC,KAAK,CAAC,CAAC,cAAc,CAAC,eAAe,CAAC,CAAA;gBAC7C,MAAM,CAAE,KAAyB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,gBAAgB,CAAC,CAAA;YAC3E,CAAC;QACH,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;YACnE,IAAI,CAAC;gBACH,MAAM,cAAc,CAClB;oBACE,OAAO,EAAE,uBAAuB;oBAChC,OAAO,EAAE,gBAAgB;iBAC1B,EACD,OAAO,CACR,CAAA;gBACD,MAAM,CAAC,IAAI,CAAC,6BAA6B,CAAC,CAAA;YAC5C,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,CAAC,KAAK,CAAC,CAAC,cAAc,CAAC,eAAe,CAAC,CAAA;gBAC7C,MAAM,CAAE,KAAyB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,gBAAgB,CAAC,CAAA;YAC3E,CAAC;QACH,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;YACrE,IAAI,CAAC;gBACH,MAAM,cAAc,CAClB;oBACE,OAAO,EAAE,mBAAmB;oBAC5B,OAAO,EAAE,uBAAuB;iBACjC,EACD,OAAO,CACR,CAAA;gBACD,MAAM,CAAC,IAAI,CAAC,6BAA6B,CAAC,CAAA;YAC5C,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,CAAC,KAAK,CAAC,CAAC,cAAc,CAAC,eAAe,CAAC,CAAA;gBAC7C,MAAM,CAAE,KAAyB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,eAAe,CAAC,CAAA;YAC1E,CAAC;QACH,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;YACrE,IAAI,CAAC;gBACH,MAAM,cAAc,CAClB;oBACE,OAAO,EAAE,uBAAuB;oBAChC,OAAO,EAAE,mBAAmB;iBAC7B,EACD,OAAO,CACR,CAAA;gBACD,MAAM,CAAC,IAAI,CAAC,6BAA6B,CAAC,CAAA;YAC5C,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,CAAC,KAAK,CAAC,CAAC,cAAc,CAAC,eAAe,CAAC,CAAA;gBAC7C,MAAM,CAAE,KAAyB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,eAAe,CAAC,CAAA;YAC1E,CAAC;QACH,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;YACnE,IAAI,CAAC;gBACH,MAAM,cAAc,CAClB;oBACE,OAAO,EAAE,uBAAuB;oBAChC,OAAO,EAAE,uBAAuB;iBACjC,EACD,OAAO,CACR,CAAA;gBACD,MAAM,CAAC,IAAI,CAAC,6BAA6B,CAAC,CAAA;YAC5C,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,CAAC,KAAK,CAAC,CAAC,cAAc,CAAC,eAAe,CAAC,CAAA;gBAC7C,MAAM,CAAE,KAAyB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,uBAAuB,CAAC,CAAA;YAClF,CAAC;QACH,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;YACzD,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,kBAAkB;gBAC3B,OAAO,EAAE,uBAAuB;aACjC,EACD,OAAO,CACR,CAAA;YAED,MAAM,CAAC,MAAM,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,EAAE,CAAC,CAAA;QAC1D,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;YACnE,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,kBAAkB;gBAC3B,OAAO,EAAE,0BAA0B;aACpC,EACD,OAAO,CACR,CAAA;YAED,MAAM,CAAC,CAAC,GAAG,EAAE,GAAG,EAAE,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,MAAM,CAAC,CAAA;QACpD,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;QACvC,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;YAC9D,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,uBAAuB;gBAChC,OAAO,EAAE,yBAAyB;aACnC,EACD,OAAO,CACR,CAAA;YACD,MAAM,SAAS,GAAG,uBAAuB,CAAC,MAAM,CAAC,CAAA;YAEjD,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAA;YAC/C,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAA;YAC1C,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAA;YAC5C,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAA;YAC5C,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAA;QAC3C,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,wCAAwC,EAAE,KAAK,IAAI,EAAE;YACtD,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,kBAAkB;gBAC3B,OAAO,EAAE,uBAAuB;aACjC,EACD,OAAO,CACR,CAAA;YACD,MAAM,SAAS,GAAG,uBAAuB,CAAC,MAAM,CAAC,CAAA;YAEjD,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAA;QACxC,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,oCAAoC,EAAE,KAAK,IAAI,EAAE;YAClD,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,uBAAuB;gBAChC,OAAO,EAAE,yBAAyB;aACnC,EACD,OAAO,CACR,CAAA;YACD,MAAM,SAAS,GAAG,uBAAuB,CAAC,MAAM,CAAC,CAAA;YAEjD,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAA;QAChD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,mCAAmC,EAAE,KAAK,IAAI,EAAE;YACjD,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;gBACE,OAAO,EAAE,uBAAuB;gBAChC,OAAO,EAAE,yBAAyB;aACnC,EACD,OAAO,CACR,CAAA;YACD,MAAM,SAAS,GAAG,uBAAuB,CAAC,MAAM,CAAC,CAAA;YAEjD,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,cAAc,CAAC,CAAA;YAC3C,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,CAAA;QACnC,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"project-detector.test.d.ts","sourceRoot":"","sources":["../../../tests/context/project-detector.test.ts"],"names":[],"mappings":"AAAA;;;GAGG"}
|