@humanbased/crosscheck 0.14.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/AGENT.md +207 -0
- package/ISSUE.md +234 -0
- package/LICENSE +21 -0
- package/README.md +234 -0
- package/README.zh.md +169 -0
- package/assets/logo.png +0 -0
- package/assets/screenshot-watch-timing.png +0 -0
- package/assets/screenshot-watch-timing.svg +1 -0
- package/assets/screenshot-watch.png +0 -0
- package/crosscheck.config.example.yml +214 -0
- package/dist/__tests__/annotation.test.d.ts +2 -0
- package/dist/__tests__/annotation.test.d.ts.map +1 -0
- package/dist/__tests__/annotation.test.js +134 -0
- package/dist/__tests__/annotation.test.js.map +1 -0
- package/dist/__tests__/backtrace.test.d.ts +2 -0
- package/dist/__tests__/backtrace.test.d.ts.map +1 -0
- package/dist/__tests__/backtrace.test.js +280 -0
- package/dist/__tests__/backtrace.test.js.map +1 -0
- package/dist/__tests__/board.test.d.ts +2 -0
- package/dist/__tests__/board.test.d.ts.map +1 -0
- package/dist/__tests__/board.test.js +149 -0
- package/dist/__tests__/board.test.js.map +1 -0
- package/dist/__tests__/codex.test.d.ts +2 -0
- package/dist/__tests__/codex.test.d.ts.map +1 -0
- package/dist/__tests__/codex.test.js +92 -0
- package/dist/__tests__/codex.test.js.map +1 -0
- package/dist/__tests__/comment-bodies.test.d.ts +2 -0
- package/dist/__tests__/comment-bodies.test.d.ts.map +1 -0
- package/dist/__tests__/comment-bodies.test.js +75 -0
- package/dist/__tests__/comment-bodies.test.js.map +1 -0
- package/dist/__tests__/conflict-resolve.test.d.ts +2 -0
- package/dist/__tests__/conflict-resolve.test.d.ts.map +1 -0
- package/dist/__tests__/conflict-resolve.test.js +123 -0
- package/dist/__tests__/conflict-resolve.test.js.map +1 -0
- package/dist/__tests__/crosscheck-commit.test.d.ts +2 -0
- package/dist/__tests__/crosscheck-commit.test.d.ts.map +1 -0
- package/dist/__tests__/crosscheck-commit.test.js +13 -0
- package/dist/__tests__/crosscheck-commit.test.js.map +1 -0
- package/dist/__tests__/detector.test.d.ts +2 -0
- package/dist/__tests__/detector.test.d.ts.map +1 -0
- package/dist/__tests__/detector.test.js +112 -0
- package/dist/__tests__/detector.test.js.map +1 -0
- package/dist/__tests__/diagnose.test.d.ts +2 -0
- package/dist/__tests__/diagnose.test.d.ts.map +1 -0
- package/dist/__tests__/diagnose.test.js +164 -0
- package/dist/__tests__/diagnose.test.js.map +1 -0
- package/dist/__tests__/diff-hash.test.d.ts +2 -0
- package/dist/__tests__/diff-hash.test.d.ts.map +1 -0
- package/dist/__tests__/diff-hash.test.js +126 -0
- package/dist/__tests__/diff-hash.test.js.map +1 -0
- package/dist/__tests__/durations.test.d.ts +2 -0
- package/dist/__tests__/durations.test.d.ts.map +1 -0
- package/dist/__tests__/durations.test.js +26 -0
- package/dist/__tests__/durations.test.js.map +1 -0
- package/dist/__tests__/event-fields.test.d.ts +2 -0
- package/dist/__tests__/event-fields.test.d.ts.map +1 -0
- package/dist/__tests__/event-fields.test.js +50 -0
- package/dist/__tests__/event-fields.test.js.map +1 -0
- package/dist/__tests__/filter.test.d.ts +2 -0
- package/dist/__tests__/filter.test.d.ts.map +1 -0
- package/dist/__tests__/filter.test.js +21 -0
- package/dist/__tests__/filter.test.js.map +1 -0
- package/dist/__tests__/fix.test.d.ts +2 -0
- package/dist/__tests__/fix.test.d.ts.map +1 -0
- package/dist/__tests__/fix.test.js +124 -0
- package/dist/__tests__/fix.test.js.map +1 -0
- package/dist/__tests__/github-client.test.d.ts +2 -0
- package/dist/__tests__/github-client.test.d.ts.map +1 -0
- package/dist/__tests__/github-client.test.js +22 -0
- package/dist/__tests__/github-client.test.js.map +1 -0
- package/dist/__tests__/github-scan-client.test.d.ts +2 -0
- package/dist/__tests__/github-scan-client.test.d.ts.map +1 -0
- package/dist/__tests__/github-scan-client.test.js +100 -0
- package/dist/__tests__/github-scan-client.test.js.map +1 -0
- package/dist/__tests__/is-fresh-review-comment.test.d.ts +2 -0
- package/dist/__tests__/is-fresh-review-comment.test.d.ts.map +1 -0
- package/dist/__tests__/is-fresh-review-comment.test.js +86 -0
- package/dist/__tests__/is-fresh-review-comment.test.js.map +1 -0
- package/dist/__tests__/issue.test.d.ts +2 -0
- package/dist/__tests__/issue.test.d.ts.map +1 -0
- package/dist/__tests__/issue.test.js +259 -0
- package/dist/__tests__/issue.test.js.map +1 -0
- package/dist/__tests__/kickass.test.d.ts +2 -0
- package/dist/__tests__/kickass.test.d.ts.map +1 -0
- package/dist/__tests__/kickass.test.js +268 -0
- package/dist/__tests__/kickass.test.js.map +1 -0
- package/dist/__tests__/loader.test.d.ts +2 -0
- package/dist/__tests__/loader.test.d.ts.map +1 -0
- package/dist/__tests__/loader.test.js +180 -0
- package/dist/__tests__/loader.test.js.map +1 -0
- package/dist/__tests__/onboard-preservation.test.d.ts +2 -0
- package/dist/__tests__/onboard-preservation.test.d.ts.map +1 -0
- package/dist/__tests__/onboard-preservation.test.js +506 -0
- package/dist/__tests__/onboard-preservation.test.js.map +1 -0
- package/dist/__tests__/optimize.test.d.ts +2 -0
- package/dist/__tests__/optimize.test.d.ts.map +1 -0
- package/dist/__tests__/optimize.test.js +101 -0
- package/dist/__tests__/optimize.test.js.map +1 -0
- package/dist/__tests__/post-review-comment.test.d.ts +2 -0
- package/dist/__tests__/post-review-comment.test.d.ts.map +1 -0
- package/dist/__tests__/post-review-comment.test.js +44 -0
- package/dist/__tests__/post-review-comment.test.js.map +1 -0
- package/dist/__tests__/pr-lock.test.d.ts +2 -0
- package/dist/__tests__/pr-lock.test.d.ts.map +1 -0
- package/dist/__tests__/pr-lock.test.js +115 -0
- package/dist/__tests__/pr-lock.test.js.map +1 -0
- package/dist/__tests__/pr-picker.test.d.ts +2 -0
- package/dist/__tests__/pr-picker.test.d.ts.map +1 -0
- package/dist/__tests__/pr-picker.test.js +57 -0
- package/dist/__tests__/pr-picker.test.js.map +1 -0
- package/dist/__tests__/pr-status-scan.test.d.ts +2 -0
- package/dist/__tests__/pr-status-scan.test.d.ts.map +1 -0
- package/dist/__tests__/pr-status-scan.test.js +92 -0
- package/dist/__tests__/pr-status-scan.test.js.map +1 -0
- package/dist/__tests__/pr-status.test.d.ts +2 -0
- package/dist/__tests__/pr-status.test.d.ts.map +1 -0
- package/dist/__tests__/pr-status.test.js +346 -0
- package/dist/__tests__/pr-status.test.js.map +1 -0
- package/dist/__tests__/repo-picker.test.d.ts +2 -0
- package/dist/__tests__/repo-picker.test.d.ts.map +1 -0
- package/dist/__tests__/repo-picker.test.js +115 -0
- package/dist/__tests__/repo-picker.test.js.map +1 -0
- package/dist/__tests__/review-comment-body.test.d.ts +2 -0
- package/dist/__tests__/review-comment-body.test.d.ts.map +1 -0
- package/dist/__tests__/review-comment-body.test.js +54 -0
- package/dist/__tests__/review-comment-body.test.js.map +1 -0
- package/dist/__tests__/review-models.test.d.ts +2 -0
- package/dist/__tests__/review-models.test.d.ts.map +1 -0
- package/dist/__tests__/review-models.test.js +39 -0
- package/dist/__tests__/review-models.test.js.map +1 -0
- package/dist/__tests__/review-status.test.d.ts +2 -0
- package/dist/__tests__/review-status.test.d.ts.map +1 -0
- package/dist/__tests__/review-status.test.js +95 -0
- package/dist/__tests__/review-status.test.js.map +1 -0
- package/dist/__tests__/runner.test.d.ts +2 -0
- package/dist/__tests__/runner.test.d.ts.map +1 -0
- package/dist/__tests__/runner.test.js +204 -0
- package/dist/__tests__/runner.test.js.map +1 -0
- package/dist/__tests__/scan-cache.test.d.ts +2 -0
- package/dist/__tests__/scan-cache.test.d.ts.map +1 -0
- package/dist/__tests__/scan-cache.test.js +59 -0
- package/dist/__tests__/scan-cache.test.js.map +1 -0
- package/dist/__tests__/scan-client.test.d.ts +2 -0
- package/dist/__tests__/scan-client.test.d.ts.map +1 -0
- package/dist/__tests__/scan-client.test.js +30 -0
- package/dist/__tests__/scan-client.test.js.map +1 -0
- package/dist/__tests__/scan.test.d.ts +2 -0
- package/dist/__tests__/scan.test.d.ts.map +1 -0
- package/dist/__tests__/scan.test.js +115 -0
- package/dist/__tests__/scan.test.js.map +1 -0
- package/dist/__tests__/scopes.test.d.ts +2 -0
- package/dist/__tests__/scopes.test.d.ts.map +1 -0
- package/dist/__tests__/scopes.test.js +101 -0
- package/dist/__tests__/scopes.test.js.map +1 -0
- package/dist/__tests__/sha-cache.test.d.ts +2 -0
- package/dist/__tests__/sha-cache.test.d.ts.map +1 -0
- package/dist/__tests__/sha-cache.test.js +40 -0
- package/dist/__tests__/sha-cache.test.js.map +1 -0
- package/dist/__tests__/smart-switch.test.d.ts +2 -0
- package/dist/__tests__/smart-switch.test.d.ts.map +1 -0
- package/dist/__tests__/smart-switch.test.js +145 -0
- package/dist/__tests__/smart-switch.test.js.map +1 -0
- package/dist/ck.d.ts +3 -0
- package/dist/ck.d.ts.map +1 -0
- package/dist/ck.js +8 -0
- package/dist/ck.js.map +1 -0
- package/dist/cli.d.ts +3 -0
- package/dist/cli.d.ts.map +1 -0
- package/dist/cli.js +132 -0
- package/dist/cli.js.map +1 -0
- package/dist/commands/diagnose.d.ts +54 -0
- package/dist/commands/diagnose.d.ts.map +1 -0
- package/dist/commands/diagnose.js +294 -0
- package/dist/commands/diagnose.js.map +1 -0
- package/dist/commands/impact.d.ts +38 -0
- package/dist/commands/impact.d.ts.map +1 -0
- package/dist/commands/impact.js +210 -0
- package/dist/commands/impact.js.map +1 -0
- package/dist/commands/init.d.ts +12 -0
- package/dist/commands/init.d.ts.map +1 -0
- package/dist/commands/init.js +183 -0
- package/dist/commands/init.js.map +1 -0
- package/dist/commands/issue.d.ts +25 -0
- package/dist/commands/issue.d.ts.map +1 -0
- package/dist/commands/issue.js +445 -0
- package/dist/commands/issue.js.map +1 -0
- package/dist/commands/kickass.d.ts +59 -0
- package/dist/commands/kickass.d.ts.map +1 -0
- package/dist/commands/kickass.js +288 -0
- package/dist/commands/kickass.js.map +1 -0
- package/dist/commands/onboard.d.ts +70 -0
- package/dist/commands/onboard.d.ts.map +1 -0
- package/dist/commands/onboard.js +883 -0
- package/dist/commands/onboard.js.map +1 -0
- package/dist/commands/optimize.d.ts +16 -0
- package/dist/commands/optimize.d.ts.map +1 -0
- package/dist/commands/optimize.js +244 -0
- package/dist/commands/optimize.js.map +1 -0
- package/dist/commands/review.d.ts +2 -0
- package/dist/commands/review.d.ts.map +1 -0
- package/dist/commands/review.js +118 -0
- package/dist/commands/review.js.map +1 -0
- package/dist/commands/run.d.ts +13 -0
- package/dist/commands/run.d.ts.map +1 -0
- package/dist/commands/run.js +243 -0
- package/dist/commands/run.js.map +1 -0
- package/dist/commands/scan.d.ts +94 -0
- package/dist/commands/scan.d.ts.map +1 -0
- package/dist/commands/scan.js +276 -0
- package/dist/commands/scan.js.map +1 -0
- package/dist/commands/serve.d.ts +9 -0
- package/dist/commands/serve.d.ts.map +1 -0
- package/dist/commands/serve.js +402 -0
- package/dist/commands/serve.js.map +1 -0
- package/dist/commands/status.d.ts +2 -0
- package/dist/commands/status.d.ts.map +1 -0
- package/dist/commands/status.js +89 -0
- package/dist/commands/status.js.map +1 -0
- package/dist/commands/watch.d.ts +9 -0
- package/dist/commands/watch.d.ts.map +1 -0
- package/dist/commands/watch.js +902 -0
- package/dist/commands/watch.js.map +1 -0
- package/dist/config/loader.d.ts +47 -0
- package/dist/config/loader.d.ts.map +1 -0
- package/dist/config/loader.js +334 -0
- package/dist/config/loader.js.map +1 -0
- package/dist/config/schema.d.ts +814 -0
- package/dist/config/schema.d.ts.map +1 -0
- package/dist/config/schema.js +152 -0
- package/dist/config/schema.js.map +1 -0
- package/dist/github/client.d.ts +139 -0
- package/dist/github/client.d.ts.map +1 -0
- package/dist/github/client.js +711 -0
- package/dist/github/client.js.map +1 -0
- package/dist/github/detector.d.ts +12 -0
- package/dist/github/detector.d.ts.map +1 -0
- package/dist/github/detector.js +120 -0
- package/dist/github/detector.js.map +1 -0
- package/dist/github/merge.d.ts +9 -0
- package/dist/github/merge.d.ts.map +1 -0
- package/dist/github/merge.js +33 -0
- package/dist/github/merge.js.map +1 -0
- package/dist/github/review-status.d.ts +6 -0
- package/dist/github/review-status.d.ts.map +1 -0
- package/dist/github/review-status.js +51 -0
- package/dist/github/review-status.js.map +1 -0
- package/dist/github/webhook.d.ts +41 -0
- package/dist/github/webhook.d.ts.map +1 -0
- package/dist/github/webhook.js +50 -0
- package/dist/github/webhook.js.map +1 -0
- package/dist/lib/annotation.d.ts +23 -0
- package/dist/lib/annotation.d.ts.map +1 -0
- package/dist/lib/annotation.js +103 -0
- package/dist/lib/annotation.js.map +1 -0
- package/dist/lib/backtrace.d.ts +40 -0
- package/dist/lib/backtrace.d.ts.map +1 -0
- package/dist/lib/backtrace.js +169 -0
- package/dist/lib/backtrace.js.map +1 -0
- package/dist/lib/board.d.ts +74 -0
- package/dist/lib/board.d.ts.map +1 -0
- package/dist/lib/board.js +640 -0
- package/dist/lib/board.js.map +1 -0
- package/dist/lib/clone.d.ts +12 -0
- package/dist/lib/clone.d.ts.map +1 -0
- package/dist/lib/clone.js +30 -0
- package/dist/lib/clone.js.map +1 -0
- package/dist/lib/comment-bodies.d.ts +17 -0
- package/dist/lib/comment-bodies.d.ts.map +1 -0
- package/dist/lib/comment-bodies.js +51 -0
- package/dist/lib/comment-bodies.js.map +1 -0
- package/dist/lib/crosscheck-commit.d.ts +2 -0
- package/dist/lib/crosscheck-commit.d.ts.map +1 -0
- package/dist/lib/crosscheck-commit.js +4 -0
- package/dist/lib/crosscheck-commit.js.map +1 -0
- package/dist/lib/diff-hash.d.ts +16 -0
- package/dist/lib/diff-hash.d.ts.map +1 -0
- package/dist/lib/diff-hash.js +71 -0
- package/dist/lib/diff-hash.js.map +1 -0
- package/dist/lib/durations.d.ts +5 -0
- package/dist/lib/durations.d.ts.map +1 -0
- package/dist/lib/durations.js +39 -0
- package/dist/lib/durations.js.map +1 -0
- package/dist/lib/event-fields.d.ts +6 -0
- package/dist/lib/event-fields.d.ts.map +1 -0
- package/dist/lib/event-fields.js +20 -0
- package/dist/lib/event-fields.js.map +1 -0
- package/dist/lib/filter.d.ts +2 -0
- package/dist/lib/filter.d.ts.map +1 -0
- package/dist/lib/filter.js +4 -0
- package/dist/lib/filter.js.map +1 -0
- package/dist/lib/fortune.d.ts +2 -0
- package/dist/lib/fortune.d.ts.map +1 -0
- package/dist/lib/fortune.js +26 -0
- package/dist/lib/fortune.js.map +1 -0
- package/dist/lib/languages.d.ts +3 -0
- package/dist/lib/languages.d.ts.map +1 -0
- package/dist/lib/languages.js +26 -0
- package/dist/lib/languages.js.map +1 -0
- package/dist/lib/log-analysis.d.ts +17 -0
- package/dist/lib/log-analysis.d.ts.map +1 -0
- package/dist/lib/log-analysis.js +72 -0
- package/dist/lib/log-analysis.js.map +1 -0
- package/dist/lib/logger.d.ts +14 -0
- package/dist/lib/logger.d.ts.map +1 -0
- package/dist/lib/logger.js +84 -0
- package/dist/lib/logger.js.map +1 -0
- package/dist/lib/port.d.ts +2 -0
- package/dist/lib/port.d.ts.map +1 -0
- package/dist/lib/port.js +21 -0
- package/dist/lib/port.js.map +1 -0
- package/dist/lib/pr-lock.d.ts +4 -0
- package/dist/lib/pr-lock.d.ts.map +1 -0
- package/dist/lib/pr-lock.js +91 -0
- package/dist/lib/pr-lock.js.map +1 -0
- package/dist/lib/pr-picker.d.ts +10 -0
- package/dist/lib/pr-picker.d.ts.map +1 -0
- package/dist/lib/pr-picker.js +80 -0
- package/dist/lib/pr-picker.js.map +1 -0
- package/dist/lib/pr-status.d.ts +206 -0
- package/dist/lib/pr-status.d.ts.map +1 -0
- package/dist/lib/pr-status.js +613 -0
- package/dist/lib/pr-status.js.map +1 -0
- package/dist/lib/repo-picker.d.ts +23 -0
- package/dist/lib/repo-picker.d.ts.map +1 -0
- package/dist/lib/repo-picker.js +411 -0
- package/dist/lib/repo-picker.js.map +1 -0
- package/dist/lib/review-models.d.ts +7 -0
- package/dist/lib/review-models.d.ts.map +1 -0
- package/dist/lib/review-models.js +32 -0
- package/dist/lib/review-models.js.map +1 -0
- package/dist/lib/runner.d.ts +65 -0
- package/dist/lib/runner.d.ts.map +1 -0
- package/dist/lib/runner.js +710 -0
- package/dist/lib/runner.js.map +1 -0
- package/dist/lib/scan-cache.d.ts +31 -0
- package/dist/lib/scan-cache.d.ts.map +1 -0
- package/dist/lib/scan-cache.js +112 -0
- package/dist/lib/scan-cache.js.map +1 -0
- package/dist/lib/scopes.d.ts +16 -0
- package/dist/lib/scopes.d.ts.map +1 -0
- package/dist/lib/scopes.js +37 -0
- package/dist/lib/scopes.js.map +1 -0
- package/dist/lib/sha-cache.d.ts +7 -0
- package/dist/lib/sha-cache.d.ts.map +1 -0
- package/dist/lib/sha-cache.js +44 -0
- package/dist/lib/sha-cache.js.map +1 -0
- package/dist/lib/smart-switch.d.ts +44 -0
- package/dist/lib/smart-switch.d.ts.map +1 -0
- package/dist/lib/smart-switch.js +145 -0
- package/dist/lib/smart-switch.js.map +1 -0
- package/dist/lib/verdict.d.ts +9 -0
- package/dist/lib/verdict.d.ts.map +1 -0
- package/dist/lib/verdict.js +52 -0
- package/dist/lib/verdict.js.map +1 -0
- package/dist/lib/workflow.d.ts +85 -0
- package/dist/lib/workflow.d.ts.map +1 -0
- package/dist/lib/workflow.js +116 -0
- package/dist/lib/workflow.js.map +1 -0
- package/dist/reviewers/address.d.ts +5 -0
- package/dist/reviewers/address.d.ts.map +1 -0
- package/dist/reviewers/address.js +87 -0
- package/dist/reviewers/address.js.map +1 -0
- package/dist/reviewers/claude.d.ts +12 -0
- package/dist/reviewers/claude.d.ts.map +1 -0
- package/dist/reviewers/claude.js +78 -0
- package/dist/reviewers/claude.js.map +1 -0
- package/dist/reviewers/codex.d.ts +9 -0
- package/dist/reviewers/codex.d.ts.map +1 -0
- package/dist/reviewers/codex.js +121 -0
- package/dist/reviewers/codex.js.map +1 -0
- package/dist/reviewers/conflict-resolve.d.ts +15 -0
- package/dist/reviewers/conflict-resolve.d.ts.map +1 -0
- package/dist/reviewers/conflict-resolve.js +219 -0
- package/dist/reviewers/conflict-resolve.js.map +1 -0
- package/dist/reviewers/fix.d.ts +7 -0
- package/dist/reviewers/fix.d.ts.map +1 -0
- package/dist/reviewers/fix.js +197 -0
- package/dist/reviewers/fix.js.map +1 -0
- package/get-started.md +1271 -0
- package/get-started.zh.md +1208 -0
- package/package.json +75 -0
|
@@ -0,0 +1,101 @@
|
|
|
1
|
+
import { describe, it, expect } from 'vitest';
|
|
2
|
+
import { selectOptimizeAgent } from '../commands/optimize.js';
|
|
3
|
+
function makeConfig(claudeEnabled, codexEnabled) {
|
|
4
|
+
return {
|
|
5
|
+
mode: 'cross-vendor',
|
|
6
|
+
clone_protocol: 'ssh',
|
|
7
|
+
orgs: [],
|
|
8
|
+
users: [],
|
|
9
|
+
repos: [],
|
|
10
|
+
routing: { codex_reviews_patterns: [], claude_reviews_patterns: [], claude_branch_prefixes: [], codex_branch_prefixes: [], allowed_authors: [], author_routes: {}, fallback_reviewer: 'auto' },
|
|
11
|
+
server: { port: 7892, webhook_path: '/webhook' },
|
|
12
|
+
quality: { tier: 'balanced', focus: [], custom_prompt: undefined },
|
|
13
|
+
budget: { codex_monthly_usd: null, per_review_usd: 1 },
|
|
14
|
+
vendors: {
|
|
15
|
+
claude: { enabled: claudeEnabled, model: null, auth: 'subscription', effort: 'medium' },
|
|
16
|
+
codex: { enabled: codexEnabled, model: null, auth: 'subscription', effort: 'medium', quality: 'medium' },
|
|
17
|
+
},
|
|
18
|
+
logs: { enabled: false, retention_days: 7 },
|
|
19
|
+
tunnel: { backend: 'localhost.run', smee_channel: '' },
|
|
20
|
+
impact: { assumed_human_review_minutes: 60, hourly_rate_usd: 150, defect_cost_usd: 150 },
|
|
21
|
+
backtrace: { enabled: false },
|
|
22
|
+
display: { theme: { bar_fill: 'blue', bar_empty: 'dim', cr_approve: 'green', cr_needs_work: 'yellow', cr_block: 'red', fix_fill: 'cyan' } },
|
|
23
|
+
post_review: {
|
|
24
|
+
auto_fix: {
|
|
25
|
+
delivery: { mode: 'pull_request', pr_title: 'fix: address CR issues in #{original_pr_title}', label: 'cr-autofix' },
|
|
26
|
+
},
|
|
27
|
+
},
|
|
28
|
+
brand: { service_name: 'crosscheck', comment_header: '', comment_footer: '', reviewer_attribution: '' },
|
|
29
|
+
};
|
|
30
|
+
}
|
|
31
|
+
function makeReport(claudeAttempts = 0, claudeSuccesses = 0, codexAttempts = 0, codexSuccesses = 0) {
|
|
32
|
+
const performance = {};
|
|
33
|
+
if (claudeAttempts > 0) {
|
|
34
|
+
performance['claude'] = {
|
|
35
|
+
attempts: claudeAttempts,
|
|
36
|
+
successes: claudeSuccesses,
|
|
37
|
+
failure_rate: (claudeAttempts - claudeSuccesses) / claudeAttempts,
|
|
38
|
+
};
|
|
39
|
+
}
|
|
40
|
+
if (codexAttempts > 0) {
|
|
41
|
+
performance['codex'] = {
|
|
42
|
+
attempts: codexAttempts,
|
|
43
|
+
successes: codexSuccesses,
|
|
44
|
+
failure_rate: (codexAttempts - codexSuccesses) / codexAttempts,
|
|
45
|
+
};
|
|
46
|
+
}
|
|
47
|
+
return {
|
|
48
|
+
period: { from: 'N/A', to: 'N/A', log_files: 0 },
|
|
49
|
+
summary: { total_reviews: 0, successful: 0, failed: 0, failure_rate: 0 },
|
|
50
|
+
errors: [],
|
|
51
|
+
verdict_distribution: { APPROVE: 0, NEEDS_WORK: 0, BLOCK: 0 },
|
|
52
|
+
verdict_parse_failures: 0,
|
|
53
|
+
repos_seen: [],
|
|
54
|
+
languages_detected: [],
|
|
55
|
+
reviewer_performance: performance,
|
|
56
|
+
suggestions: [],
|
|
57
|
+
};
|
|
58
|
+
}
|
|
59
|
+
describe('selectOptimizeAgent', () => {
|
|
60
|
+
it('returns codex when only codex is enabled', () => {
|
|
61
|
+
const { agent } = selectOptimizeAgent(makeConfig(false, true), makeReport());
|
|
62
|
+
expect(agent).toBe('codex');
|
|
63
|
+
});
|
|
64
|
+
it('returns claude when only claude is enabled', () => {
|
|
65
|
+
const { agent } = selectOptimizeAgent(makeConfig(true, false), makeReport());
|
|
66
|
+
expect(agent).toBe('claude');
|
|
67
|
+
});
|
|
68
|
+
it('returns claude as default when both enabled and no log data', () => {
|
|
69
|
+
const { agent, reason } = selectOptimizeAgent(makeConfig(true, true), makeReport());
|
|
70
|
+
expect(agent).toBe('claude');
|
|
71
|
+
expect(reason).toMatch(/default/);
|
|
72
|
+
});
|
|
73
|
+
it('returns codex when both enabled and codex has higher success rate', () => {
|
|
74
|
+
// codex 80%, claude 50%
|
|
75
|
+
const { agent } = selectOptimizeAgent(makeConfig(true, true), makeReport(10, 5, 10, 8));
|
|
76
|
+
expect(agent).toBe('codex');
|
|
77
|
+
});
|
|
78
|
+
it('returns claude when both enabled and claude has higher success rate', () => {
|
|
79
|
+
// claude 90%, codex 60%
|
|
80
|
+
const { agent } = selectOptimizeAgent(makeConfig(true, true), makeReport(10, 9, 10, 6));
|
|
81
|
+
expect(agent).toBe('claude');
|
|
82
|
+
});
|
|
83
|
+
it('returns claude (default) when both enabled and rates are equal', () => {
|
|
84
|
+
// both 70%
|
|
85
|
+
const { agent } = selectOptimizeAgent(makeConfig(true, true), makeReport(10, 7, 10, 7));
|
|
86
|
+
expect(agent).toBe('claude');
|
|
87
|
+
});
|
|
88
|
+
it('throws when no vendors are enabled', () => {
|
|
89
|
+
expect(() => selectOptimizeAgent(makeConfig(false, false), makeReport())).toThrow(/No vendors enabled/);
|
|
90
|
+
});
|
|
91
|
+
it('reason string mentions source when only one vendor enabled', () => {
|
|
92
|
+
const { reason } = selectOptimizeAgent(makeConfig(true, false), makeReport());
|
|
93
|
+
expect(reason).toMatch(/only enabled vendor/);
|
|
94
|
+
});
|
|
95
|
+
it('reason string includes success rates when chosen by data', () => {
|
|
96
|
+
const { reason } = selectOptimizeAgent(makeConfig(true, true), makeReport(10, 5, 10, 8));
|
|
97
|
+
expect(reason).toMatch(/80%/);
|
|
98
|
+
expect(reason).toMatch(/50%/);
|
|
99
|
+
});
|
|
100
|
+
});
|
|
101
|
+
//# sourceMappingURL=optimize.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"optimize.test.js","sourceRoot":"","sources":["../../src/__tests__/optimize.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAA;AAC7C,OAAO,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAA;AAI7D,SAAS,UAAU,CAAC,aAAsB,EAAE,YAAqB;IAC/D,OAAO;QACL,IAAI,EAAE,cAAc;QACpB,cAAc,EAAE,KAAK;QACrB,IAAI,EAAE,EAAE;QACR,KAAK,EAAE,EAAE;QACT,KAAK,EAAE,EAAE;QACT,OAAO,EAAE,EAAE,sBAAsB,EAAE,EAAE,EAAE,uBAAuB,EAAE,EAAE,EAAE,sBAAsB,EAAE,EAAE,EAAE,qBAAqB,EAAE,EAAE,EAAE,eAAe,EAAE,EAAE,EAAE,aAAa,EAAE,EAAE,EAAE,iBAAiB,EAAE,MAAM,EAAE;QAC9L,MAAM,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE,YAAY,EAAE,UAAU,EAAE;QAChD,OAAO,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,KAAK,EAAE,EAAE,EAAE,aAAa,EAAE,SAAS,EAAE;QAClE,MAAM,EAAE,EAAE,iBAAiB,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC,EAAE;QACtD,OAAO,EAAE;YACP,MAAM,EAAE,EAAE,OAAO,EAAE,aAAa,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,cAAc,EAAE,MAAM,EAAE,QAAQ,EAAE;YACvF,KAAK,EAAE,EAAE,OAAO,EAAE,YAAY,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,cAAc,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,QAAQ,EAAE;SACzG;QACD,IAAI,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE,cAAc,EAAE,CAAC,EAAE;QAC3C,MAAM,EAAE,EAAE,OAAO,EAAE,eAAe,EAAE,YAAY,EAAE,EAAE,EAAE;QACtD,MAAM,EAAE,EAAE,4BAA4B,EAAE,EAAE,EAAE,eAAe,EAAE,GAAG,EAAE,eAAe,EAAE,GAAG,EAAE;QACxF,SAAS,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE;QAC7B,OAAO,EAAE,EAAE,KAAK,EAAE,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,UAAU,EAAE,OAAO,EAAE,aAAa,EAAE,QAAQ,EAAE,QAAQ,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE;QAC3I,WAAW,EAAE;YACX,QAAQ,EAAE;gBACR,QAAQ,EAAE,EAAE,IAAI,EAAE,cAAc,EAAE,QAAQ,EAAE,gDAAgD,EAAE,KAAK,EAAE,YAAY,EAAE;aACpH;SACF;QACD,KAAK,EAAE,EAAE,YAAY,EAAE,YAAY,EAAE,cAAc,EAAE,EAAE,EAAE,cAAc,EAAE,EAAE,EAAE,oBAAoB,EAAE,EAAE,EAAE;KACxG,CAAA;AACH,CAAC;AAED,SAAS,UAAU,CACjB,cAAc,GAAG,CAAC,EAAE,eAAe,GAAG,CAAC,EACvC,aAAa,GAAG,CAAC,EAAE,cAAc,GAAG,CAAC;IAErC,MAAM,WAAW,GAAkF,EAAE,CAAA;IACrG,IAAI,cAAc,GAAG,CAAC,EAAE,CAAC;QACvB,WAAW,CAAC,QAAQ,CAAC,GAAG;YACtB,QAAQ,EAAE,cAAc;YACxB,SAAS,EAAE,eAAe;YAC1B,YAAY,EAAE,CAAC,cAAc,GAAG,eAAe,CAAC,GAAG,cAAc;SAClE,CAAA;IACH,CAAC;IACD,IAAI,aAAa,GAAG,CAAC,EAAE,CAAC;QACtB,WAAW,CAAC,OAAO,CAAC,GAAG;YACrB,QAAQ,EAAE,aAAa;YACvB,SAAS,EAAE,cAAc;YACzB,YAAY,EAAE,CAAC,aAAa,GAAG,cAAc,CAAC,GAAG,aAAa;SAC/D,CAAA;IACH,CAAC;IACD,OAAO;QACL,MAAM,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,EAAE,EAAE,KAAK,EAAE,SAAS,EAAE,CAAC,EAAE;QAChD,OAAO,EAAE,EAAE,aAAa,EAAE,CAAC,EAAE,UAAU,EAAE,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,YAAY,EAAE,CAAC,EAAE;QACxE,MAAM,EAAE,EAAE;QACV,oBAAoB,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE,UAAU,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE;QAC7D,sBAAsB,EAAE,CAAC;QACzB,UAAU,EAAE,EAAE;QACd,kBAAkB,EAAE,EAAE;QACtB,oBAAoB,EAAE,WAAW;QACjC,WAAW,EAAE,EAAE;KAChB,CAAA;AACH,CAAC;AAED,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,KAAK,EAAE,IAAI,CAAC,EAAE,UAAU,EAAE,CAAC,CAAA;QAC5E,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;IAC7B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,KAAK,CAAC,EAAE,UAAU,EAAE,CAAC,CAAA;QAC5E,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;IAC9B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACrE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,UAAU,EAAE,CAAC,CAAA;QACnF,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QAC5B,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,SAAS,CAAC,CAAA;IACnC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,mEAAmE,EAAE,GAAG,EAAE;QAC3E,wBAAwB;QACxB,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,UAAU,CAAC,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC,CAAA;QACvF,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;IAC7B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC7E,wBAAwB;QACxB,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,UAAU,CAAC,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC,CAAA;QACvF,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;IAC9B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;QACxE,WAAW;QACX,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,UAAU,CAAC,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC,CAAA;QACvF,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;IAC9B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,MAAM,CAAC,GAAG,EAAE,CAAC,mBAAmB,CAAC,UAAU,CAAC,KAAK,EAAE,KAAK,CAAC,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,oBAAoB,CAAC,CAAA;IACzG,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACpE,MAAM,EAAE,MAAM,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,KAAK,CAAC,EAAE,UAAU,EAAE,CAAC,CAAA;QAC7E,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,qBAAqB,CAAC,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QAClE,MAAM,EAAE,MAAM,EAAE,GAAG,mBAAmB,CAAC,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,UAAU,CAAC,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC,CAAA;QACxF,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,KAAK,CAAC,CAAA;QAC7B,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,KAAK,CAAC,CAAA;IAC/B,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"post-review-comment.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/post-review-comment.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,44 @@
|
|
|
1
|
+
import { describe, it, expect, vi } from 'vitest';
|
|
2
|
+
import { postReviewComment } from '../github/client.js';
|
|
3
|
+
function makeOctokit() {
|
|
4
|
+
const createComment = vi.fn().mockResolvedValue({ data: { id: 123 } });
|
|
5
|
+
return {
|
|
6
|
+
octokit: {
|
|
7
|
+
rest: {
|
|
8
|
+
issues: { createComment },
|
|
9
|
+
},
|
|
10
|
+
},
|
|
11
|
+
createComment,
|
|
12
|
+
};
|
|
13
|
+
}
|
|
14
|
+
describe('postReviewComment', () => {
|
|
15
|
+
it('emits a v2 review annotation even when verdict parsing failed', async () => {
|
|
16
|
+
const { octokit, createComment } = makeOctokit();
|
|
17
|
+
await postReviewComment(octokit, 'owner', 'repo', 42, 'Review body', 'codex', {}, 'claude', null);
|
|
18
|
+
expect(createComment).toHaveBeenCalledWith(expect.objectContaining({
|
|
19
|
+
body: expect.stringContaining('<!-- crosscheck: origin=claude reviewer=codex model=default type=review round=1 verdict=UNKNOWN service=crosscheck -->'),
|
|
20
|
+
}));
|
|
21
|
+
});
|
|
22
|
+
it('emits type=recheck with the supplied workflow round', async () => {
|
|
23
|
+
const { octokit, createComment } = makeOctokit();
|
|
24
|
+
await postReviewComment(octokit, 'owner', 'repo', 42, 'Recheck body', 'claude', {}, 'codex', 'APPROVE', 99, true, undefined, 'recheck', 2);
|
|
25
|
+
expect(createComment).toHaveBeenCalledWith(expect.objectContaining({
|
|
26
|
+
body: expect.stringContaining('<!-- crosscheck: origin=codex reviewer=claude model=default type=recheck round=2 verdict=APPROVE service=crosscheck -->'),
|
|
27
|
+
}));
|
|
28
|
+
});
|
|
29
|
+
it('threads model, stepType, and round into the annotation', async () => {
|
|
30
|
+
const { octokit, createComment } = makeOctokit();
|
|
31
|
+
await postReviewComment(octokit, 'owner', 'repo', 42, 'Review', 'claude', {}, 'codex', 'NEEDS_WORK', undefined, false, 'claude-opus-4-7', 'review', 3);
|
|
32
|
+
expect(createComment).toHaveBeenCalledWith(expect.objectContaining({
|
|
33
|
+
body: expect.stringContaining('model=claude-opus-4-7 type=review round=3'),
|
|
34
|
+
}));
|
|
35
|
+
});
|
|
36
|
+
it('threads the selected head sha into the annotation when provided', async () => {
|
|
37
|
+
const { octokit, createComment } = makeOctokit();
|
|
38
|
+
await postReviewComment(octokit, 'owner', 'repo', 42, 'Review', 'codex', {}, 'claude', 'BLOCK', undefined, false, 'gpt-5', 'review', 1, 'abc1234');
|
|
39
|
+
expect(createComment).toHaveBeenCalledWith(expect.objectContaining({
|
|
40
|
+
body: expect.stringContaining('service=crosscheck sha=abc1234'),
|
|
41
|
+
}));
|
|
42
|
+
});
|
|
43
|
+
});
|
|
44
|
+
//# sourceMappingURL=post-review-comment.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"post-review-comment.test.js","sourceRoot":"","sources":["../../src/__tests__/post-review-comment.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AACjD,OAAO,EAAE,iBAAiB,EAAE,MAAM,qBAAqB,CAAA;AAEvD,SAAS,WAAW;IAClB,MAAM,aAAa,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,IAAI,EAAE,EAAE,EAAE,EAAE,GAAG,EAAE,EAAE,CAAC,CAAA;IACtE,OAAO;QACL,OAAO,EAAE;YACP,IAAI,EAAE;gBACJ,MAAM,EAAE,EAAE,aAAa,EAAE;aAC1B;SACO;QACV,aAAa;KACd,CAAA;AACH,CAAC;AAED,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,EAAE,OAAO,EAAE,aAAa,EAAE,GAAG,WAAW,EAAE,CAAA;QAEhD,MAAM,iBAAiB,CACrB,OAAO,EACP,OAAO,EACP,MAAM,EACN,EAAE,EACF,aAAa,EACb,OAAO,EACP,EAAE,EACF,QAAQ,EACR,IAAI,CACL,CAAA;QAED,MAAM,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,gBAAgB,CAAC;YACjE,IAAI,EAAE,MAAM,CAAC,gBAAgB,CAC3B,wHAAwH,CACzH;SACF,CAAC,CAAC,CAAA;IACL,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,MAAM,EAAE,OAAO,EAAE,aAAa,EAAE,GAAG,WAAW,EAAE,CAAA;QAEhD,MAAM,iBAAiB,CACrB,OAAO,EACP,OAAO,EACP,MAAM,EACN,EAAE,EACF,cAAc,EACd,QAAQ,EACR,EAAE,EACF,OAAO,EACP,SAAS,EACT,EAAE,EACF,IAAI,EACJ,SAAS,EACT,SAAS,EACT,CAAC,CACF,CAAA;QAED,MAAM,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,gBAAgB,CAAC;YACjE,IAAI,EAAE,MAAM,CAAC,gBAAgB,CAC3B,yHAAyH,CAC1H;SACF,CAAC,CAAC,CAAA;IACL,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;QACtE,MAAM,EAAE,OAAO,EAAE,aAAa,EAAE,GAAG,WAAW,EAAE,CAAA;QAEhD,MAAM,iBAAiB,CACrB,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,EAAE,EAAE,QAAQ,EAAE,QAAQ,EAAE,EAAE,EAAE,OAAO,EAAE,YAAY,EAC3E,SAAS,EAAE,KAAK,EAAE,iBAAiB,EAAE,QAAQ,EAAE,CAAC,CACjD,CAAA;QAED,MAAM,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,gBAAgB,CAAC;YACjE,IAAI,EAAE,MAAM,CAAC,gBAAgB,CAAC,2CAA2C,CAAC;SAC3E,CAAC,CAAC,CAAA;IACL,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iEAAiE,EAAE,KAAK,IAAI,EAAE;QAC/E,MAAM,EAAE,OAAO,EAAE,aAAa,EAAE,GAAG,WAAW,EAAE,CAAA;QAEhD,MAAM,iBAAiB,CACrB,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,EAAE,EAAE,QAAQ,EAAE,OAAO,EAAE,EAAE,EAAE,QAAQ,EAAE,OAAO,EACtE,SAAS,EAAE,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,SAAS,CAClD,CAAA;QAED,MAAM,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,gBAAgB,CAAC;YACjE,IAAI,EAAE,MAAM,CAAC,gBAAgB,CAAC,gCAAgC,CAAC;SAChE,CAAC,CAAC,CAAA;IACL,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"pr-lock.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/pr-lock.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,115 @@
|
|
|
1
|
+
import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
|
|
2
|
+
import { writeFileSync, mkdirSync, utimesSync, existsSync } from 'fs';
|
|
3
|
+
import { homedir } from 'os';
|
|
4
|
+
import { join } from 'path';
|
|
5
|
+
import { acquirePRLock, releasePRLock, handleLockSignal } from '../lib/pr-lock.js';
|
|
6
|
+
const OWNER = 'test-owner';
|
|
7
|
+
const REPO = 'test-repo';
|
|
8
|
+
const PR = 999;
|
|
9
|
+
const SHA = 'abc12345def67890';
|
|
10
|
+
afterEach(() => {
|
|
11
|
+
releasePRLock(OWNER, REPO, PR, SHA);
|
|
12
|
+
releasePRLock(OWNER, REPO, PR + 1, SHA);
|
|
13
|
+
releasePRLock(OWNER, REPO, PR, 'ffffffff00000000');
|
|
14
|
+
});
|
|
15
|
+
describe('acquirePRLock', () => {
|
|
16
|
+
it('returns true on first acquisition', () => {
|
|
17
|
+
expect(acquirePRLock(OWNER, REPO, PR, SHA)).toBe(true);
|
|
18
|
+
});
|
|
19
|
+
it('returns false when lock is already held for same SHA', () => {
|
|
20
|
+
acquirePRLock(OWNER, REPO, PR, SHA);
|
|
21
|
+
expect(acquirePRLock(OWNER, REPO, PR, SHA)).toBe(false);
|
|
22
|
+
});
|
|
23
|
+
it('returns false for a different SHA on the same PR', () => {
|
|
24
|
+
acquirePRLock(OWNER, REPO, PR, SHA);
|
|
25
|
+
const newSha = 'ffffffff00000000';
|
|
26
|
+
expect(acquirePRLock(OWNER, REPO, PR, newSha)).toBe(false);
|
|
27
|
+
});
|
|
28
|
+
it('returns true after lock is released', () => {
|
|
29
|
+
acquirePRLock(OWNER, REPO, PR, SHA);
|
|
30
|
+
releasePRLock(OWNER, REPO, PR, SHA);
|
|
31
|
+
expect(acquirePRLock(OWNER, REPO, PR, SHA)).toBe(true);
|
|
32
|
+
});
|
|
33
|
+
it('does not conflict across different PR numbers', () => {
|
|
34
|
+
expect(acquirePRLock(OWNER, REPO, PR, SHA)).toBe(true);
|
|
35
|
+
expect(acquirePRLock(OWNER, REPO, PR + 1, SHA)).toBe(true);
|
|
36
|
+
});
|
|
37
|
+
it('acquires a stale lock (mtime > 20 min)', () => {
|
|
38
|
+
const lockDir = join(homedir(), '.crosscheck', 'locks');
|
|
39
|
+
mkdirSync(lockDir, { recursive: true });
|
|
40
|
+
const stalePath = join(lockDir, `${OWNER}-${REPO}-${PR}.lock`);
|
|
41
|
+
writeFileSync(stalePath, '');
|
|
42
|
+
// Back-date the mtime to 21 minutes ago
|
|
43
|
+
const staleTime = new Date(Date.now() - 21 * 60 * 1000);
|
|
44
|
+
utimesSync(stalePath, staleTime, staleTime);
|
|
45
|
+
expect(acquirePRLock(OWNER, REPO, PR, SHA)).toBe(true);
|
|
46
|
+
});
|
|
47
|
+
});
|
|
48
|
+
describe('releasePRLock', () => {
|
|
49
|
+
it('does not throw when lock file does not exist', () => {
|
|
50
|
+
expect(() => releasePRLock(OWNER, REPO, PR, SHA)).not.toThrow();
|
|
51
|
+
});
|
|
52
|
+
});
|
|
53
|
+
describe('handleLockSignal', () => {
|
|
54
|
+
// process.kill would actually deliver the signal to the test runner and
|
|
55
|
+
// terminate it. We stub it out and assert the call shape instead. Each
|
|
56
|
+
// test snapshots existing SIGINT listeners and restores them afterward so
|
|
57
|
+
// module-level state (acquirePRLock's one-shot register) doesn't bleed
|
|
58
|
+
// across tests.
|
|
59
|
+
let savedListeners = [];
|
|
60
|
+
beforeEach(() => {
|
|
61
|
+
savedListeners = process.listeners('SIGINT');
|
|
62
|
+
for (const l of savedListeners)
|
|
63
|
+
process.removeListener('SIGINT', l);
|
|
64
|
+
});
|
|
65
|
+
afterEach(() => {
|
|
66
|
+
for (const l of process.listeners('SIGINT'))
|
|
67
|
+
process.removeListener('SIGINT', l);
|
|
68
|
+
for (const l of savedListeners)
|
|
69
|
+
process.on('SIGINT', l);
|
|
70
|
+
});
|
|
71
|
+
it('removes active lock files when signal fires', () => {
|
|
72
|
+
acquirePRLock(OWNER, REPO, PR, SHA);
|
|
73
|
+
const lockFile = join(homedir(), '.crosscheck', 'locks', `${OWNER}-${REPO}-${PR}.lock`);
|
|
74
|
+
expect(existsSync(lockFile)).toBe(true);
|
|
75
|
+
const killSpy = vi.spyOn(process, 'kill').mockImplementation(() => true);
|
|
76
|
+
try {
|
|
77
|
+
handleLockSignal('SIGINT');
|
|
78
|
+
expect(existsSync(lockFile)).toBe(false);
|
|
79
|
+
}
|
|
80
|
+
finally {
|
|
81
|
+
killSpy.mockRestore();
|
|
82
|
+
}
|
|
83
|
+
});
|
|
84
|
+
it('re-raises the signal to terminate the process when it is the sole listener', () => {
|
|
85
|
+
process.on('SIGINT', handleLockSignal);
|
|
86
|
+
expect(process.listenerCount('SIGINT')).toBe(1);
|
|
87
|
+
const killSpy = vi.spyOn(process, 'kill').mockImplementation(() => true);
|
|
88
|
+
try {
|
|
89
|
+
handleLockSignal('SIGINT');
|
|
90
|
+
expect(killSpy).toHaveBeenCalledWith(process.pid, 'SIGINT');
|
|
91
|
+
// Should have removed itself before re-raising so the default action takes over
|
|
92
|
+
expect(process.listenerCount('SIGINT')).toBe(0);
|
|
93
|
+
}
|
|
94
|
+
finally {
|
|
95
|
+
killSpy.mockRestore();
|
|
96
|
+
}
|
|
97
|
+
});
|
|
98
|
+
it('does not re-raise when another listener exists (lets graceful-shutdown handlers drive exit)', () => {
|
|
99
|
+
process.on('SIGINT', handleLockSignal);
|
|
100
|
+
const otherListener = () => { };
|
|
101
|
+
process.on('SIGINT', otherListener);
|
|
102
|
+
expect(process.listenerCount('SIGINT')).toBe(2);
|
|
103
|
+
const killSpy = vi.spyOn(process, 'kill').mockImplementation(() => true);
|
|
104
|
+
try {
|
|
105
|
+
handleLockSignal('SIGINT');
|
|
106
|
+
expect(killSpy).not.toHaveBeenCalled();
|
|
107
|
+
// Our handler should still be registered — only the sole-listener path removes it
|
|
108
|
+
expect(process.listenerCount('SIGINT')).toBe(2);
|
|
109
|
+
}
|
|
110
|
+
finally {
|
|
111
|
+
killSpy.mockRestore();
|
|
112
|
+
}
|
|
113
|
+
});
|
|
114
|
+
});
|
|
115
|
+
//# sourceMappingURL=pr-lock.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"pr-lock.test.js","sourceRoot":"","sources":["../../src/__tests__/pr-lock.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AACxE,OAAO,EAAE,aAAa,EAAE,SAAS,EAAE,UAAU,EAAE,UAAU,EAAE,MAAM,IAAI,CAAA;AACrE,OAAO,EAAE,OAAO,EAAE,MAAM,IAAI,CAAA;AAC5B,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAA;AAC3B,OAAO,EAAE,aAAa,EAAE,aAAa,EAAE,gBAAgB,EAAE,MAAM,mBAAmB,CAAA;AAElF,MAAM,KAAK,GAAG,YAAY,CAAA;AAC1B,MAAM,IAAI,GAAG,WAAW,CAAA;AACxB,MAAM,EAAE,GAAG,GAAG,CAAA;AACd,MAAM,GAAG,GAAG,kBAAkB,CAAA;AAE9B,SAAS,CAAC,GAAG,EAAE;IACb,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAA;IACnC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,GAAG,CAAC,EAAE,GAAG,CAAC,CAAA;IACvC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,kBAAkB,CAAC,CAAA;AACpD,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,eAAe,EAAE,GAAG,EAAE;IAC7B,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,CAAC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IACxD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAA;QACnC,MAAM,CAAC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACzD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAA;QACnC,MAAM,MAAM,GAAG,kBAAkB,CAAA;QACjC,MAAM,CAAC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IAC5D,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC7C,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAA;QACnC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAA;QACnC,MAAM,CAAC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IACxD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,MAAM,CAAC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACtD,MAAM,CAAC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,GAAG,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAC5D,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,EAAE,EAAE,aAAa,EAAE,OAAO,CAAC,CAAA;QACvD,SAAS,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;QACvC,MAAM,SAAS,GAAG,IAAI,CAAC,OAAO,EAAE,GAAG,KAAK,IAAI,IAAI,IAAI,EAAE,OAAO,CAAC,CAAA;QAC9D,aAAa,CAAC,SAAS,EAAE,EAAE,CAAC,CAAA;QAC5B,wCAAwC;QACxC,MAAM,SAAS,GAAG,IAAI,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC,CAAA;QACvD,UAAU,CAAC,SAAS,EAAE,SAAS,EAAE,SAAS,CAAC,CAAA;QAC3C,MAAM,CAAC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IACxD,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,eAAe,EAAE,GAAG,EAAE;IAC7B,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,CAAC,GAAG,EAAE,CAAC,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAA;IACjE,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,wEAAwE;IACxE,uEAAuE;IACvE,0EAA0E;IAC1E,uEAAuE;IACvE,gBAAgB;IAChB,IAAI,cAAc,GAA6B,EAAE,CAAA;IAEjD,UAAU,CAAC,GAAG,EAAE;QACd,cAAc,GAAG,OAAO,CAAC,SAAS,CAAC,QAAQ,CAA6B,CAAA;QACxE,KAAK,MAAM,CAAC,IAAI,cAAc;YAAE,OAAO,CAAC,cAAc,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAA;IACrE,CAAC,CAAC,CAAA;IAEF,SAAS,CAAC,GAAG,EAAE;QACb,KAAK,MAAM,CAAC,IAAI,OAAO,CAAC,SAAS,CAAC,QAAQ,CAAC;YAAE,OAAO,CAAC,cAAc,CAAC,QAAQ,EAAE,CAA2B,CAAC,CAAA;QAC1G,KAAK,MAAM,CAAC,IAAI,cAAc;YAAE,OAAO,CAAC,EAAE,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAA;IACzD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,aAAa,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAA;QACnC,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,EAAE,EAAE,aAAa,EAAE,OAAO,EAAE,GAAG,KAAK,IAAI,IAAI,IAAI,EAAE,OAAO,CAAC,CAAA;QACvF,MAAM,CAAC,UAAU,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAEvC,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAA;QACxE,IAAI,CAAC;YACH,gBAAgB,CAAC,QAAQ,CAAC,CAAA;YAC1B,MAAM,CAAC,UAAU,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC1C,CAAC;gBAAS,CAAC;YACT,OAAO,CAAC,WAAW,EAAE,CAAA;QACvB,CAAC;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4EAA4E,EAAE,GAAG,EAAE;QACpF,OAAO,CAAC,EAAE,CAAC,QAAQ,EAAE,gBAAgB,CAAC,CAAA;QACtC,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAE/C,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAA;QACxE,IAAI,CAAC;YACH,gBAAgB,CAAC,QAAQ,CAAC,CAAA;YAC1B,MAAM,CAAC,OAAO,CAAC,CAAC,oBAAoB,CAAC,OAAO,CAAC,GAAG,EAAE,QAAQ,CAAC,CAAA;YAC3D,gFAAgF;YAChF,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACjD,CAAC;gBAAS,CAAC;YACT,OAAO,CAAC,WAAW,EAAE,CAAA;QACvB,CAAC;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6FAA6F,EAAE,GAAG,EAAE;QACrG,OAAO,CAAC,EAAE,CAAC,QAAQ,EAAE,gBAAgB,CAAC,CAAA;QACtC,MAAM,aAAa,GAAG,GAAS,EAAE,GAA6C,CAAC,CAAA;QAC/E,OAAO,CAAC,EAAE,CAAC,QAAQ,EAAE,aAAa,CAAC,CAAA;QACnC,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAE/C,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAA;QACxE,IAAI,CAAC;YACH,gBAAgB,CAAC,QAAQ,CAAC,CAAA;YAC1B,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;YACtC,kFAAkF;YAClF,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACjD,CAAC;gBAAS,CAAC;YACT,OAAO,CAAC,WAAW,EAAE,CAAA;QACvB,CAAC;IACH,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"pr-picker.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/pr-picker.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,57 @@
|
|
|
1
|
+
import { describe, expect, it } from 'vitest';
|
|
2
|
+
import { actionGroupLabel, formatPickerLabel, parseSelection, sortPRsForPicker, UserInputError } from '../lib/pr-picker.js';
|
|
3
|
+
function pr(number) {
|
|
4
|
+
return {
|
|
5
|
+
owner: 'acme',
|
|
6
|
+
repo: 'web',
|
|
7
|
+
number,
|
|
8
|
+
title: `PR ${number}`,
|
|
9
|
+
author: 'alice',
|
|
10
|
+
url: `https://github.com/acme/web/pull/${number}`,
|
|
11
|
+
headSha: 'abc123',
|
|
12
|
+
headRef: 'feature',
|
|
13
|
+
headRepo: 'acme/web',
|
|
14
|
+
baseRef: 'main',
|
|
15
|
+
freshness: 'stale',
|
|
16
|
+
reviewState: 'PR',
|
|
17
|
+
nextAction: 'review',
|
|
18
|
+
lastActiveAt: '2026-05-29T00:00:00.000Z',
|
|
19
|
+
staleAfterMs: 60_000,
|
|
20
|
+
ageMs: 120_000,
|
|
21
|
+
verdict: null,
|
|
22
|
+
latestAnnotation: null,
|
|
23
|
+
};
|
|
24
|
+
}
|
|
25
|
+
describe('parseSelection', () => {
|
|
26
|
+
const prs = [pr(1), pr(2), pr(3)];
|
|
27
|
+
it('selects all PRs', () => {
|
|
28
|
+
expect(parseSelection('all', prs).map(item => item.number)).toEqual([1, 2, 3]);
|
|
29
|
+
});
|
|
30
|
+
it('selects comma-separated PR indexes in operator order', () => {
|
|
31
|
+
expect(parseSelection('3,1,1', prs).map(item => item.number)).toEqual([3, 1]);
|
|
32
|
+
});
|
|
33
|
+
it('returns an empty selection for blank input', () => {
|
|
34
|
+
expect(parseSelection(' ', prs)).toEqual([]);
|
|
35
|
+
});
|
|
36
|
+
it('rejects out-of-range selections', () => {
|
|
37
|
+
expect(() => parseSelection('4', prs)).toThrow('Invalid selection');
|
|
38
|
+
expect(() => parseSelection('4', prs)).toThrow(UserInputError);
|
|
39
|
+
});
|
|
40
|
+
});
|
|
41
|
+
describe('picker grouping', () => {
|
|
42
|
+
it('orders actionable PRs by next action group', () => {
|
|
43
|
+
const ordered = sortPRsForPicker([
|
|
44
|
+
{ ...pr(4), nextAction: 'merge', reviewState: 'APPROVE' },
|
|
45
|
+
{ ...pr(2), nextAction: 'fix', reviewState: 'NEEDS_WORK' },
|
|
46
|
+
{ ...pr(3), nextAction: 'recheck', reviewState: 'RECHECK' },
|
|
47
|
+
{ ...pr(1), nextAction: 'review', reviewState: 'PR' },
|
|
48
|
+
]);
|
|
49
|
+
expect(ordered.map(item => item.number)).toEqual([1, 2, 3, 4]);
|
|
50
|
+
expect(ordered.map(actionGroupLabel)).toEqual(['CR', 'fix', 'recheck', 'merge']);
|
|
51
|
+
});
|
|
52
|
+
it('renders compact labels with action group and scanned head', () => {
|
|
53
|
+
expect(formatPickerLabel(pr(12))).toContain('CR');
|
|
54
|
+
expect(formatPickerLabel(pr(12))).toContain('acme/web#12@abc123');
|
|
55
|
+
});
|
|
56
|
+
});
|
|
57
|
+
//# sourceMappingURL=pr-picker.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"pr-picker.test.js","sourceRoot":"","sources":["../../src/__tests__/pr-picker.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AAC7C,OAAO,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,cAAc,EAAE,gBAAgB,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAA;AAG3H,SAAS,EAAE,CAAC,MAAc;IACxB,OAAO;QACL,KAAK,EAAE,MAAM;QACb,IAAI,EAAE,KAAK;QACX,MAAM;QACN,KAAK,EAAE,MAAM,MAAM,EAAE;QACrB,MAAM,EAAE,OAAO;QACf,GAAG,EAAE,oCAAoC,MAAM,EAAE;QACjD,OAAO,EAAE,QAAQ;QACjB,OAAO,EAAE,SAAS;QAClB,QAAQ,EAAE,UAAU;QACpB,OAAO,EAAE,MAAM;QACf,SAAS,EAAE,OAAO;QAClB,WAAW,EAAE,IAAI;QACjB,UAAU,EAAE,QAAQ;QACpB,YAAY,EAAE,0BAA0B;QACxC,YAAY,EAAE,MAAM;QACpB,KAAK,EAAE,OAAO;QACd,OAAO,EAAE,IAAI;QACb,gBAAgB,EAAE,IAAI;KACvB,CAAA;AACH,CAAC;AAED,QAAQ,CAAC,gBAAgB,EAAE,GAAG,EAAE;IAC9B,MAAM,GAAG,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAA;IAEjC,EAAE,CAAC,iBAAiB,EAAE,GAAG,EAAE;QACzB,MAAM,CAAC,cAAc,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAA;IAChF,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,MAAM,CAAC,cAAc,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAA;IAC/E,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,CAAC,cAAc,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;QACzC,MAAM,CAAC,GAAG,EAAE,CAAC,cAAc,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAA;QACnE,MAAM,CAAC,GAAG,EAAE,CAAC,cAAc,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,cAAc,CAAC,CAAA;IAChE,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAC/B,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,OAAO,GAAG,gBAAgB,CAAC;YAC/B,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC,EAAE,UAAU,EAAE,OAAO,EAAE,WAAW,EAAE,SAAS,EAAE;YACzD,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC,EAAE,UAAU,EAAE,KAAK,EAAE,WAAW,EAAE,YAAY,EAAE;YAC1D,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC,EAAE,UAAU,EAAE,SAAS,EAAE,WAAW,EAAE,SAAS,EAAE;YAC3D,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC,EAAE,UAAU,EAAE,QAAQ,EAAE,WAAW,EAAE,IAAI,EAAE;SACtD,CAAC,CAAA;QAEF,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAA;QAC9D,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,gBAAgB,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,SAAS,EAAE,OAAO,CAAC,CAAC,CAAA;IAClF,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACnE,MAAM,CAAC,iBAAiB,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,CAAA;QACjD,MAAM,CAAC,iBAAiB,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAA;IACnE,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"pr-status-scan.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/pr-status-scan.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,92 @@
|
|
|
1
|
+
import { beforeEach, describe, expect, it, vi } from 'vitest';
|
|
2
|
+
import { ConfigSchema } from '../config/schema.js';
|
|
3
|
+
vi.mock('../github/client.js', () => ({
|
|
4
|
+
createGithubClient: vi.fn(() => ({})),
|
|
5
|
+
listCheckRuns: vi.fn(),
|
|
6
|
+
listCommitStatuses: vi.fn(),
|
|
7
|
+
listIssueComments: vi.fn(),
|
|
8
|
+
listOpenPRs: vi.fn(),
|
|
9
|
+
listOrgRepos: vi.fn(),
|
|
10
|
+
listPRCommitActivity: vi.fn(),
|
|
11
|
+
listPRReviewComments: vi.fn(),
|
|
12
|
+
listTimelineEvents: vi.fn(),
|
|
13
|
+
listUserRepos: vi.fn(),
|
|
14
|
+
}));
|
|
15
|
+
vi.mock('../github/merge.js', () => ({
|
|
16
|
+
getPRMergeSummary: vi.fn(),
|
|
17
|
+
}));
|
|
18
|
+
vi.mock('../lib/logger.js', async () => {
|
|
19
|
+
const actual = await vi.importActual('../lib/logger.js');
|
|
20
|
+
return {
|
|
21
|
+
...actual,
|
|
22
|
+
logError: vi.fn(),
|
|
23
|
+
};
|
|
24
|
+
});
|
|
25
|
+
const { scanOpenPRStatuses } = await import('../lib/pr-status.js');
|
|
26
|
+
const github = await import('../github/client.js');
|
|
27
|
+
const merge = await import('../github/merge.js');
|
|
28
|
+
const logger = await import('../lib/logger.js');
|
|
29
|
+
const mockListOpenPRs = vi.mocked(github.listOpenPRs);
|
|
30
|
+
const mockListIssueComments = vi.mocked(github.listIssueComments);
|
|
31
|
+
const mockListPRReviewComments = vi.mocked(github.listPRReviewComments);
|
|
32
|
+
const mockListPRCommitActivity = vi.mocked(github.listPRCommitActivity);
|
|
33
|
+
const mockListCommitStatuses = vi.mocked(github.listCommitStatuses);
|
|
34
|
+
const mockListCheckRuns = vi.mocked(github.listCheckRuns);
|
|
35
|
+
const mockListTimelineEvents = vi.mocked(github.listTimelineEvents);
|
|
36
|
+
const mockGetPRMergeSummary = vi.mocked(merge.getPRMergeSummary);
|
|
37
|
+
const mockLogError = vi.mocked(logger.logError);
|
|
38
|
+
function openPR(number) {
|
|
39
|
+
return {
|
|
40
|
+
number,
|
|
41
|
+
title: `PR ${number}`,
|
|
42
|
+
author: 'alice',
|
|
43
|
+
headSha: `sha-${number}`,
|
|
44
|
+
headRef: `branch-${number}`,
|
|
45
|
+
headRepo: 'acme/web',
|
|
46
|
+
baseRef: 'main',
|
|
47
|
+
body: null,
|
|
48
|
+
createdAt: '2026-05-27T00:00:00.000Z',
|
|
49
|
+
updatedAt: '2026-05-27T00:00:00.000Z',
|
|
50
|
+
url: `https://github.com/acme/web/pull/${number}`,
|
|
51
|
+
};
|
|
52
|
+
}
|
|
53
|
+
beforeEach(() => {
|
|
54
|
+
vi.clearAllMocks();
|
|
55
|
+
mockListOpenPRs.mockResolvedValue([]);
|
|
56
|
+
mockListIssueComments.mockResolvedValue([]);
|
|
57
|
+
mockListPRReviewComments.mockResolvedValue([]);
|
|
58
|
+
mockListPRCommitActivity.mockResolvedValue([]);
|
|
59
|
+
mockListCommitStatuses.mockResolvedValue([]);
|
|
60
|
+
mockListCheckRuns.mockResolvedValue([]);
|
|
61
|
+
mockListTimelineEvents.mockResolvedValue([]);
|
|
62
|
+
mockGetPRMergeSummary.mockResolvedValue({ mergeable: null, protectedBase: null });
|
|
63
|
+
});
|
|
64
|
+
describe('scanOpenPRStatuses', () => {
|
|
65
|
+
it('skips one PR when activity aggregation fails and continues scanning the rest', async () => {
|
|
66
|
+
mockListOpenPRs.mockResolvedValue([openPR(1), openPR(2)]);
|
|
67
|
+
mockListIssueComments.mockImplementation(async (_owner, _repo, issueNumber) => {
|
|
68
|
+
if (issueNumber === 1)
|
|
69
|
+
throw new Error('rate limited');
|
|
70
|
+
return [];
|
|
71
|
+
});
|
|
72
|
+
const result = await scanOpenPRStatuses(ConfigSchema.parse({ repos: [{ owner: 'acme', name: 'web' }] }), 'token', { now: new Date('2026-05-29T00:00:00.000Z'), staleAfterMs: 24 * 60 * 60 * 1000 });
|
|
73
|
+
expect(result.prs.map(pr => pr.number)).toEqual([2]);
|
|
74
|
+
expect(result.summary.total).toBe(1);
|
|
75
|
+
expect(mockLogError).toHaveBeenCalledWith(expect.objectContaining({ event: 'scan_pr_skipped', owner: 'acme', repo: 'web', pr: 1 }), expect.any(Error));
|
|
76
|
+
});
|
|
77
|
+
it('caps concurrent per-PR GitHub activity requests', async () => {
|
|
78
|
+
mockListOpenPRs.mockResolvedValue(Array.from({ length: 20 }, (_, index) => openPR(index + 1)));
|
|
79
|
+
let active = 0;
|
|
80
|
+
let maxActive = 0;
|
|
81
|
+
mockListIssueComments.mockImplementation(async () => {
|
|
82
|
+
active += 1;
|
|
83
|
+
maxActive = Math.max(maxActive, active);
|
|
84
|
+
await new Promise(resolve => setTimeout(resolve, 1));
|
|
85
|
+
active -= 1;
|
|
86
|
+
return [];
|
|
87
|
+
});
|
|
88
|
+
await scanOpenPRStatuses(ConfigSchema.parse({ repos: [{ owner: 'acme', name: 'web' }] }), 'token', { now: new Date('2026-05-29T00:00:00.000Z'), staleAfterMs: 24 * 60 * 60 * 1000 });
|
|
89
|
+
expect(maxActive).toBeLessThanOrEqual(8);
|
|
90
|
+
});
|
|
91
|
+
});
|
|
92
|
+
//# sourceMappingURL=pr-status-scan.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"pr-status-scan.test.js","sourceRoot":"","sources":["../../src/__tests__/pr-status-scan.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AAC7D,OAAO,EAAE,YAAY,EAAE,MAAM,qBAAqB,CAAA;AAElD,EAAE,CAAC,IAAI,CAAC,qBAAqB,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,kBAAkB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,CAAC,CAAC;IACrC,aAAa,EAAE,EAAE,CAAC,EAAE,EAAE;IACtB,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC3B,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC1B,WAAW,EAAE,EAAE,CAAC,EAAE,EAAE;IACpB,YAAY,EAAE,EAAE,CAAC,EAAE,EAAE;IACrB,oBAAoB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC7B,oBAAoB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC7B,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC3B,aAAa,EAAE,EAAE,CAAC,EAAE,EAAE;CACvB,CAAC,CAAC,CAAA;AAEH,EAAE,CAAC,IAAI,CAAC,oBAAoB,EAAE,GAAG,EAAE,CAAC,CAAC;IACnC,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE;CAC3B,CAAC,CAAC,CAAA;AAEH,EAAE,CAAC,IAAI,CAAC,kBAAkB,EAAE,KAAK,IAAI,EAAE;IACrC,MAAM,MAAM,GAAG,MAAM,EAAE,CAAC,YAAY,CAAoC,kBAAkB,CAAC,CAAA;IAC3F,OAAO;QACL,GAAG,MAAM;QACT,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;KAClB,CAAA;AACH,CAAC,CAAC,CAAA;AAEF,MAAM,EAAE,kBAAkB,EAAE,GAAG,MAAM,MAAM,CAAC,qBAAqB,CAAC,CAAA;AAClE,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,qBAAqB,CAAC,CAAA;AAClD,MAAM,KAAK,GAAG,MAAM,MAAM,CAAC,oBAAoB,CAAC,CAAA;AAChD,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,kBAAkB,CAAC,CAAA;AAE/C,MAAM,eAAe,GAAG,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAA;AACrD,MAAM,qBAAqB,GAAG,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,iBAAiB,CAAC,CAAA;AACjE,MAAM,wBAAwB,GAAG,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,oBAAoB,CAAC,CAAA;AACvE,MAAM,wBAAwB,GAAG,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,oBAAoB,CAAC,CAAA;AACvE,MAAM,sBAAsB,GAAG,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,kBAAkB,CAAC,CAAA;AACnE,MAAM,iBAAiB,GAAG,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,CAAA;AACzD,MAAM,sBAAsB,GAAG,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,kBAAkB,CAAC,CAAA;AACnE,MAAM,qBAAqB,GAAG,EAAE,CAAC,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,CAAA;AAChE,MAAM,YAAY,GAAG,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAA;AAE/C,SAAS,MAAM,CAAC,MAAc;IAC5B,OAAO;QACL,MAAM;QACN,KAAK,EAAE,MAAM,MAAM,EAAE;QACrB,MAAM,EAAE,OAAO;QACf,OAAO,EAAE,OAAO,MAAM,EAAE;QACxB,OAAO,EAAE,UAAU,MAAM,EAAE;QAC3B,QAAQ,EAAE,UAAU;QACpB,OAAO,EAAE,MAAM;QACf,IAAI,EAAE,IAAI;QACV,SAAS,EAAE,0BAA0B;QACrC,SAAS,EAAE,0BAA0B;QACrC,GAAG,EAAE,oCAAoC,MAAM,EAAE;KAClD,CAAA;AACH,CAAC;AAED,UAAU,CAAC,GAAG,EAAE;IACd,EAAE,CAAC,aAAa,EAAE,CAAA;IAClB,eAAe,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAA;IACrC,qBAAqB,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAA;IAC3C,wBAAwB,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAA;IAC9C,wBAAwB,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAA;IAC9C,sBAAsB,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAA;IAC5C,iBAAiB,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAA;IACvC,sBAAsB,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAA;IAC5C,qBAAqB,CAAC,iBAAiB,CAAC,EAAE,SAAS,EAAE,IAAI,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC,CAAA;AACnF,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IAClC,EAAE,CAAC,8EAA8E,EAAE,KAAK,IAAI,EAAE;QAC5F,eAAe,CAAC,iBAAiB,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAA;QACzD,qBAAqB,CAAC,kBAAkB,CAAC,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,WAAW,EAAE,EAAE;YAC5E,IAAI,WAAW,KAAK,CAAC;gBAAE,MAAM,IAAI,KAAK,CAAC,cAAc,CAAC,CAAA;YACtD,OAAO,EAAE,CAAA;QACX,CAAC,CAAC,CAAA;QAEF,MAAM,MAAM,GAAG,MAAM,kBAAkB,CACrC,YAAY,CAAC,KAAK,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,EAC/D,OAAO,EACP,EAAE,GAAG,EAAE,IAAI,IAAI,CAAC,0BAA0B,CAAC,EAAE,YAAY,EAAE,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,EAAE,CACjF,CAAA;QAED,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,CAAA;QACpD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACpC,MAAM,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACvC,MAAM,CAAC,gBAAgB,CAAC,EAAE,KAAK,EAAE,iBAAiB,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,EACxF,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAClB,CAAA;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,eAAe,CAAC,iBAAiB,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,EAAE,CAAC,MAAM,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,CAAA;QAC9F,IAAI,MAAM,GAAG,CAAC,CAAA;QACd,IAAI,SAAS,GAAG,CAAC,CAAA;QACjB,qBAAqB,CAAC,kBAAkB,CAAC,KAAK,IAAI,EAAE;YAClD,MAAM,IAAI,CAAC,CAAA;YACX,SAAS,GAAG,IAAI,CAAC,GAAG,CAAC,SAAS,EAAE,MAAM,CAAC,CAAA;YACvC,MAAM,IAAI,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,CAAC,CAAC,CAAC,CAAA;YACpD,MAAM,IAAI,CAAC,CAAA;YACX,OAAO,EAAE,CAAA;QACX,CAAC,CAAC,CAAA;QAEF,MAAM,kBAAkB,CACtB,YAAY,CAAC,KAAK,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,EAC/D,OAAO,EACP,EAAE,GAAG,EAAE,IAAI,IAAI,CAAC,0BAA0B,CAAC,EAAE,YAAY,EAAE,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,EAAE,CACjF,CAAA;QAED,MAAM,CAAC,SAAS,CAAC,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAA;IAC1C,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"pr-status.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/pr-status.test.ts"],"names":[],"mappings":""}
|