@diff-review-system/drs 3.3.1 → 4.0.1
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/.pi/agents/describe/pr-describer.md +14 -0
- package/.pi/agents/review/unified-reviewer.md +31 -1
- package/.pi/agents/task/agents-md-updater.md +26 -0
- package/.pi/agents/task/changelog-updater.md +29 -0
- package/.pi/agents/task/review-issue-fixer.md +42 -0
- package/.pi/agents/visual/pr-explainer.md +205 -0
- package/.pi/workflows/github-pr-describe.yaml +26 -0
- package/.pi/workflows/github-pr-fix-review-issues-stacked.yaml +148 -0
- package/.pi/workflows/github-pr-post-comment.yaml +19 -0
- package/.pi/workflows/github-pr-review-post.yaml +43 -0
- package/.pi/workflows/github-pr-review.yaml +364 -0
- package/.pi/workflows/github-pr-show-changes.yaml +25 -0
- package/.pi/workflows/github-pr-update-agents-md-stacked.yaml +103 -0
- package/.pi/workflows/github-pr-visual-explain.yaml +35 -0
- package/.pi/workflows/gitlab-mr-describe.yaml +24 -0
- package/.pi/workflows/gitlab-mr-fix-review-issues-stacked.yaml +144 -0
- package/.pi/workflows/gitlab-mr-post-comment.yaml +17 -0
- package/.pi/workflows/gitlab-mr-review.yaml +364 -0
- package/.pi/workflows/gitlab-mr-show-changes.yaml +23 -0
- package/.pi/workflows/gitlab-mr-update-agents-md-stacked.yaml +100 -0
- package/.pi/workflows/gitlab-mr-visual-explain.yaml +33 -0
- package/.pi/workflows/local-changelog-update.yaml +23 -0
- package/.pi/workflows/local-fix-review-issues.yaml +111 -0
- package/.pi/workflows/local-review.yaml +24 -0
- package/.pi/workflows/local-update-agents-md.yaml +24 -0
- package/.pi/workflows/local-visual-explain.yaml +31 -0
- package/.pi/workflows/release-changelog-finalize.yaml +47 -0
- package/.pi/workflows/tag-changelog-update.yaml +26 -0
- package/README.md +281 -104
- package/dist/ci/runner.d.ts.map +1 -1
- package/dist/ci/runner.js +9 -8
- package/dist/ci/runner.js.map +1 -1
- package/dist/cli/index.js +95 -325
- package/dist/cli/index.js.map +1 -1
- package/dist/cli/init.d.ts.map +1 -1
- package/dist/cli/init.js +25 -23
- package/dist/cli/init.js.map +1 -1
- package/dist/cli/run-agent.d.ts +26 -0
- package/dist/cli/run-agent.d.ts.map +1 -0
- package/dist/cli/run-agent.js +143 -0
- package/dist/cli/run-agent.js.map +1 -0
- package/dist/cli/workflow.d.ts +105 -0
- package/dist/cli/workflow.d.ts.map +1 -0
- package/dist/cli/workflow.js +3309 -0
- package/dist/cli/workflow.js.map +1 -0
- package/dist/github/client.d.ts +12 -0
- package/dist/github/client.d.ts.map +1 -1
- package/dist/github/client.js +27 -0
- package/dist/github/client.js.map +1 -1
- package/dist/github/platform-adapter.d.ts +6 -1
- package/dist/github/platform-adapter.d.ts.map +1 -1
- package/dist/github/platform-adapter.js +84 -8
- package/dist/github/platform-adapter.js.map +1 -1
- package/dist/gitlab/client.d.ts +11 -0
- package/dist/gitlab/client.d.ts.map +1 -1
- package/dist/gitlab/client.js +11 -0
- package/dist/gitlab/client.js.map +1 -1
- package/dist/gitlab/platform-adapter.d.ts +3 -1
- package/dist/gitlab/platform-adapter.d.ts.map +1 -1
- package/dist/gitlab/platform-adapter.js +32 -1
- package/dist/gitlab/platform-adapter.js.map +1 -1
- package/dist/lib/agent-id.d.ts +9 -0
- package/dist/lib/agent-id.d.ts.map +1 -0
- package/dist/lib/agent-id.js +32 -0
- package/dist/lib/agent-id.js.map +1 -0
- package/dist/lib/comment-formatter.d.ts +15 -1
- package/dist/lib/comment-formatter.d.ts.map +1 -1
- package/dist/lib/comment-formatter.js +53 -4
- package/dist/lib/comment-formatter.js.map +1 -1
- package/dist/lib/comment-manager.d.ts +4 -0
- package/dist/lib/comment-manager.d.ts.map +1 -1
- package/dist/lib/comment-manager.js +7 -1
- package/dist/lib/comment-manager.js.map +1 -1
- package/dist/lib/comment-poster.d.ts +2 -2
- package/dist/lib/comment-poster.d.ts.map +1 -1
- package/dist/lib/comment-poster.js +31 -4
- package/dist/lib/comment-poster.js.map +1 -1
- package/dist/lib/config.d.ts +160 -44
- package/dist/lib/config.d.ts.map +1 -1
- package/dist/lib/config.js +475 -101
- package/dist/lib/config.js.map +1 -1
- package/dist/lib/context-compression.d.ts +10 -0
- package/dist/lib/context-compression.d.ts.map +1 -1
- package/dist/lib/context-compression.js +101 -13
- package/dist/lib/context-compression.js.map +1 -1
- package/dist/lib/context-loader.d.ts +5 -4
- package/dist/lib/context-loader.d.ts.map +1 -1
- package/dist/lib/context-loader.js +79 -7
- package/dist/lib/context-loader.js.map +1 -1
- package/dist/lib/describe-core.d.ts.map +1 -1
- package/dist/lib/describe-core.js +3 -2
- package/dist/lib/describe-core.js.map +1 -1
- package/dist/lib/description-executor.js +1 -1
- package/dist/lib/description-executor.js.map +1 -1
- package/dist/lib/diff-lines.d.ts +18 -0
- package/dist/lib/diff-lines.d.ts.map +1 -0
- package/dist/lib/diff-lines.js +40 -0
- package/dist/lib/diff-lines.js.map +1 -0
- package/dist/lib/exit.js +4 -4
- package/dist/lib/exit.js.map +1 -1
- package/dist/lib/html-artifact.d.ts +14 -0
- package/dist/lib/html-artifact.d.ts.map +1 -0
- package/dist/lib/html-artifact.js +59 -0
- package/dist/lib/html-artifact.js.map +1 -0
- package/dist/lib/issue-parser.js +3 -3
- package/dist/lib/issue-parser.js.map +1 -1
- package/dist/lib/json-output-schema.d.ts +70 -0
- package/dist/lib/json-output-schema.d.ts.map +1 -1
- package/dist/lib/json-output-schema.js +40 -0
- package/dist/lib/json-output-schema.js.map +1 -1
- package/dist/lib/logger.d.ts +1 -1
- package/dist/lib/logger.d.ts.map +1 -1
- package/dist/lib/platform-client.d.ts +26 -0
- package/dist/lib/platform-client.d.ts.map +1 -1
- package/dist/lib/review-artifact.d.ts +69 -0
- package/dist/lib/review-artifact.d.ts.map +1 -0
- package/dist/lib/review-artifact.js +171 -0
- package/dist/lib/review-artifact.js.map +1 -0
- package/dist/lib/review-core.d.ts +6 -4
- package/dist/lib/review-core.d.ts.map +1 -1
- package/dist/lib/review-core.js +88 -173
- package/dist/lib/review-core.js.map +1 -1
- package/dist/lib/review-orchestrator.d.ts +23 -0
- package/dist/lib/review-orchestrator.d.ts.map +1 -1
- package/dist/lib/review-orchestrator.js +31 -21
- package/dist/lib/review-orchestrator.js.map +1 -1
- package/dist/lib/review-usage.d.ts +4 -0
- package/dist/lib/review-usage.d.ts.map +1 -1
- package/dist/lib/review-usage.js +25 -0
- package/dist/lib/review-usage.js.map +1 -1
- package/dist/lib/trace-collector.d.ts +105 -0
- package/dist/lib/trace-collector.d.ts.map +1 -0
- package/dist/lib/trace-collector.js +255 -0
- package/dist/lib/trace-collector.js.map +1 -0
- package/dist/lib/trace-html.d.ts +3 -0
- package/dist/lib/trace-html.d.ts.map +1 -0
- package/dist/lib/trace-html.js +349 -0
- package/dist/lib/trace-html.js.map +1 -0
- package/dist/lib/workflow-artifacts.d.ts +54 -0
- package/dist/lib/workflow-artifacts.d.ts.map +1 -0
- package/dist/lib/workflow-artifacts.js +150 -0
- package/dist/lib/workflow-artifacts.js.map +1 -0
- package/dist/pi/sdk.d.ts.map +1 -1
- package/dist/pi/sdk.js +605 -16
- package/dist/pi/sdk.js.map +1 -1
- package/dist/runtime/agent-loader.d.ts +10 -6
- package/dist/runtime/agent-loader.d.ts.map +1 -1
- package/dist/runtime/agent-loader.js +55 -29
- package/dist/runtime/agent-loader.js.map +1 -1
- package/dist/runtime/built-in-paths.d.ts +1 -0
- package/dist/runtime/built-in-paths.d.ts.map +1 -1
- package/dist/runtime/built-in-paths.js +7 -0
- package/dist/runtime/built-in-paths.js.map +1 -1
- package/dist/runtime/client.d.ts +14 -0
- package/dist/runtime/client.d.ts.map +1 -1
- package/dist/runtime/client.js +87 -56
- package/dist/runtime/client.js.map +1 -1
- package/dist/runtime/path-config.d.ts +2 -2
- package/dist/runtime/path-config.d.ts.map +1 -1
- package/dist/runtime/path-config.js +8 -8
- package/dist/runtime/path-config.js.map +1 -1
- package/package.json +22 -16
- package/.pi/agents/review/documentation.md +0 -56
- package/.pi/agents/review/performance.md +0 -53
- package/.pi/agents/review/quality.md +0 -59
- package/.pi/agents/review/security.md +0 -53
- package/.pi/agents/review/style.md +0 -132
- package/dist/cli/describe-mr.d.ts +0 -11
- package/dist/cli/describe-mr.d.ts.map +0 -1
- package/dist/cli/describe-mr.js +0 -134
- package/dist/cli/describe-mr.js.map +0 -1
- package/dist/cli/describe-pr.d.ts +0 -12
- package/dist/cli/describe-pr.d.ts.map +0 -1
- package/dist/cli/describe-pr.js +0 -135
- package/dist/cli/describe-pr.js.map +0 -1
- package/dist/cli/post-comments.d.ts +0 -20
- package/dist/cli/post-comments.d.ts.map +0 -1
- package/dist/cli/post-comments.js +0 -225
- package/dist/cli/post-comments.js.map +0 -1
- package/dist/cli/review-local.d.ts +0 -13
- package/dist/cli/review-local.d.ts.map +0 -1
- package/dist/cli/review-local.integration.test.d.ts +0 -2
- package/dist/cli/review-local.integration.test.d.ts.map +0 -1
- package/dist/cli/review-local.integration.test.js +0 -343
- package/dist/cli/review-local.integration.test.js.map +0 -1
- package/dist/cli/review-local.js +0 -90
- package/dist/cli/review-local.js.map +0 -1
- package/dist/cli/review-local.live.e2e.test.d.ts +0 -2
- package/dist/cli/review-local.live.e2e.test.d.ts.map +0 -1
- package/dist/cli/review-local.live.e2e.test.js +0 -153
- package/dist/cli/review-local.live.e2e.test.js.map +0 -1
- package/dist/cli/review-local.test.d.ts +0 -2
- package/dist/cli/review-local.test.d.ts.map +0 -1
- package/dist/cli/review-local.test.js +0 -164
- package/dist/cli/review-local.test.js.map +0 -1
- package/dist/cli/review-mr.d.ts +0 -22
- package/dist/cli/review-mr.d.ts.map +0 -1
- package/dist/cli/review-mr.js +0 -181
- package/dist/cli/review-mr.js.map +0 -1
- package/dist/cli/review-mr.test.d.ts +0 -2
- package/dist/cli/review-mr.test.d.ts.map +0 -1
- package/dist/cli/review-mr.test.js +0 -142
- package/dist/cli/review-mr.test.js.map +0 -1
- package/dist/cli/review-pr.d.ts +0 -22
- package/dist/cli/review-pr.d.ts.map +0 -1
- package/dist/cli/review-pr.js +0 -181
- package/dist/cli/review-pr.js.map +0 -1
- package/dist/cli/review-pr.test.d.ts +0 -2
- package/dist/cli/review-pr.test.d.ts.map +0 -1
- package/dist/cli/review-pr.test.js +0 -137
- package/dist/cli/review-pr.test.js.map +0 -1
- package/dist/cli/review-url.d.ts +0 -35
- package/dist/cli/review-url.d.ts.map +0 -1
- package/dist/cli/review-url.js +0 -110
- package/dist/cli/review-url.js.map +0 -1
- package/dist/cli/review-url.test.d.ts +0 -2
- package/dist/cli/review-url.test.d.ts.map +0 -1
- package/dist/cli/review-url.test.js +0 -132
- package/dist/cli/review-url.test.js.map +0 -1
- package/dist/cli/show-changes.d.ts +0 -15
- package/dist/cli/show-changes.d.ts.map +0 -1
- package/dist/cli/show-changes.js +0 -184
- package/dist/cli/show-changes.js.map +0 -1
- package/dist/github/client.test.d.ts +0 -2
- package/dist/github/client.test.d.ts.map +0 -1
- package/dist/github/client.test.js +0 -206
- package/dist/github/client.test.js.map +0 -1
- package/dist/github/platform-adapter.test.d.ts +0 -2
- package/dist/github/platform-adapter.test.d.ts.map +0 -1
- package/dist/github/platform-adapter.test.js +0 -40
- package/dist/github/platform-adapter.test.js.map +0 -1
- package/dist/gitlab/diff-parser.test.d.ts +0 -2
- package/dist/gitlab/diff-parser.test.d.ts.map +0 -1
- package/dist/gitlab/diff-parser.test.js +0 -315
- package/dist/gitlab/diff-parser.test.js.map +0 -1
- package/dist/gitlab/platform-adapter.test.d.ts +0 -2
- package/dist/gitlab/platform-adapter.test.d.ts.map +0 -1
- package/dist/gitlab/platform-adapter.test.js +0 -21
- package/dist/gitlab/platform-adapter.test.js.map +0 -1
- package/dist/index.test.d.ts +0 -2
- package/dist/index.test.d.ts.map +0 -1
- package/dist/index.test.js +0 -7
- package/dist/index.test.js.map +0 -1
- package/dist/lib/code-quality-report.test.d.ts +0 -2
- package/dist/lib/code-quality-report.test.d.ts.map +0 -1
- package/dist/lib/code-quality-report.test.js +0 -327
- package/dist/lib/code-quality-report.test.js.map +0 -1
- package/dist/lib/comment-formatter.test.d.ts +0 -2
- package/dist/lib/comment-formatter.test.d.ts.map +0 -1
- package/dist/lib/comment-formatter.test.js +0 -694
- package/dist/lib/comment-formatter.test.js.map +0 -1
- package/dist/lib/comment-manager.test.d.ts +0 -2
- package/dist/lib/comment-manager.test.d.ts.map +0 -1
- package/dist/lib/comment-manager.test.js +0 -680
- package/dist/lib/comment-manager.test.js.map +0 -1
- package/dist/lib/comment-poster.test.d.ts +0 -5
- package/dist/lib/comment-poster.test.d.ts.map +0 -1
- package/dist/lib/comment-poster.test.js +0 -245
- package/dist/lib/comment-poster.test.js.map +0 -1
- package/dist/lib/config-model-overrides.test.d.ts +0 -12
- package/dist/lib/config-model-overrides.test.d.ts.map +0 -1
- package/dist/lib/config-model-overrides.test.js +0 -254
- package/dist/lib/config-model-overrides.test.js.map +0 -1
- package/dist/lib/config.test.d.ts +0 -2
- package/dist/lib/config.test.d.ts.map +0 -1
- package/dist/lib/config.test.js +0 -73
- package/dist/lib/config.test.js.map +0 -1
- package/dist/lib/context-compression.test.d.ts +0 -2
- package/dist/lib/context-compression.test.d.ts.map +0 -1
- package/dist/lib/context-compression.test.js +0 -337
- package/dist/lib/context-compression.test.js.map +0 -1
- package/dist/lib/context-loader.test.d.ts +0 -2
- package/dist/lib/context-loader.test.d.ts.map +0 -1
- package/dist/lib/context-loader.test.js +0 -207
- package/dist/lib/context-loader.test.js.map +0 -1
- package/dist/lib/cursor-fix-link.test.d.ts +0 -2
- package/dist/lib/cursor-fix-link.test.d.ts.map +0 -1
- package/dist/lib/cursor-fix-link.test.js +0 -70
- package/dist/lib/cursor-fix-link.test.js.map +0 -1
- package/dist/lib/describe-core.test.d.ts +0 -2
- package/dist/lib/describe-core.test.d.ts.map +0 -1
- package/dist/lib/describe-core.test.js +0 -208
- package/dist/lib/describe-core.test.js.map +0 -1
- package/dist/lib/describe-output-path.test.d.ts +0 -2
- package/dist/lib/describe-output-path.test.d.ts.map +0 -1
- package/dist/lib/describe-output-path.test.js +0 -51
- package/dist/lib/describe-output-path.test.js.map +0 -1
- package/dist/lib/describe-parser.test.d.ts +0 -2
- package/dist/lib/describe-parser.test.d.ts.map +0 -1
- package/dist/lib/describe-parser.test.js +0 -282
- package/dist/lib/describe-parser.test.js.map +0 -1
- package/dist/lib/description-executor.test.d.ts +0 -2
- package/dist/lib/description-executor.test.d.ts.map +0 -1
- package/dist/lib/description-executor.test.js +0 -128
- package/dist/lib/description-executor.test.js.map +0 -1
- package/dist/lib/description-formatter.test.d.ts +0 -2
- package/dist/lib/description-formatter.test.d.ts.map +0 -1
- package/dist/lib/description-formatter.test.js +0 -57
- package/dist/lib/description-formatter.test.js.map +0 -1
- package/dist/lib/diff-parser.test.d.ts +0 -2
- package/dist/lib/diff-parser.test.d.ts.map +0 -1
- package/dist/lib/diff-parser.test.js +0 -335
- package/dist/lib/diff-parser.test.js.map +0 -1
- package/dist/lib/error-comment-poster.test.d.ts +0 -2
- package/dist/lib/error-comment-poster.test.d.ts.map +0 -1
- package/dist/lib/error-comment-poster.test.js +0 -128
- package/dist/lib/error-comment-poster.test.js.map +0 -1
- package/dist/lib/exit.test.d.ts +0 -2
- package/dist/lib/exit.test.d.ts.map +0 -1
- package/dist/lib/exit.test.js +0 -120
- package/dist/lib/exit.test.js.map +0 -1
- package/dist/lib/issue-parser.test.d.ts +0 -2
- package/dist/lib/issue-parser.test.d.ts.map +0 -1
- package/dist/lib/issue-parser.test.js +0 -281
- package/dist/lib/issue-parser.test.js.map +0 -1
- package/dist/lib/json-output-schema.test.d.ts +0 -2
- package/dist/lib/json-output-schema.test.d.ts.map +0 -1
- package/dist/lib/json-output-schema.test.js +0 -92
- package/dist/lib/json-output-schema.test.js.map +0 -1
- package/dist/lib/json-output.test.d.ts +0 -2
- package/dist/lib/json-output.test.d.ts.map +0 -1
- package/dist/lib/json-output.test.js +0 -141
- package/dist/lib/json-output.test.js.map +0 -1
- package/dist/lib/logger.test.d.ts +0 -2
- package/dist/lib/logger.test.d.ts.map +0 -1
- package/dist/lib/logger.test.js +0 -324
- package/dist/lib/logger.test.js.map +0 -1
- package/dist/lib/position-validator.test.d.ts +0 -2
- package/dist/lib/position-validator.test.d.ts.map +0 -1
- package/dist/lib/position-validator.test.js +0 -128
- package/dist/lib/position-validator.test.js.map +0 -1
- package/dist/lib/prompt-budget.test.d.ts +0 -2
- package/dist/lib/prompt-budget.test.d.ts.map +0 -1
- package/dist/lib/prompt-budget.test.js +0 -55
- package/dist/lib/prompt-budget.test.js.map +0 -1
- package/dist/lib/repository-validator.test.d.ts +0 -5
- package/dist/lib/repository-validator.test.d.ts.map +0 -1
- package/dist/lib/repository-validator.test.js +0 -341
- package/dist/lib/repository-validator.test.js.map +0 -1
- package/dist/lib/review-core.test.d.ts +0 -2
- package/dist/lib/review-core.test.d.ts.map +0 -1
- package/dist/lib/review-core.test.js +0 -600
- package/dist/lib/review-core.test.js.map +0 -1
- package/dist/lib/review-orchestrator.test.d.ts +0 -2
- package/dist/lib/review-orchestrator.test.d.ts.map +0 -1
- package/dist/lib/review-orchestrator.test.js +0 -531
- package/dist/lib/review-orchestrator.test.js.map +0 -1
- package/dist/lib/review-output-path.test.d.ts +0 -2
- package/dist/lib/review-output-path.test.d.ts.map +0 -1
- package/dist/lib/review-output-path.test.js +0 -83
- package/dist/lib/review-output-path.test.js.map +0 -1
- package/dist/lib/review-parser.test.d.ts +0 -2
- package/dist/lib/review-parser.test.d.ts.map +0 -1
- package/dist/lib/review-parser.test.js +0 -130
- package/dist/lib/review-parser.test.js.map +0 -1
- package/dist/lib/review-usage.test.d.ts +0 -2
- package/dist/lib/review-usage.test.d.ts.map +0 -1
- package/dist/lib/review-usage.test.js +0 -83
- package/dist/lib/review-usage.test.js.map +0 -1
- package/dist/lib/unified-review-executor.d.ts +0 -60
- package/dist/lib/unified-review-executor.d.ts.map +0 -1
- package/dist/lib/unified-review-executor.js +0 -207
- package/dist/lib/unified-review-executor.js.map +0 -1
- package/dist/lib/unified-review-executor.test.d.ts +0 -5
- package/dist/lib/unified-review-executor.test.d.ts.map +0 -1
- package/dist/lib/unified-review-executor.test.js +0 -472
- package/dist/lib/unified-review-executor.test.js.map +0 -1
- package/dist/lib/write-json-output.test.d.ts +0 -2
- package/dist/lib/write-json-output.test.d.ts.map +0 -1
- package/dist/lib/write-json-output.test.js +0 -259
- package/dist/lib/write-json-output.test.js.map +0 -1
- package/dist/pi/sdk.test.d.ts +0 -2
- package/dist/pi/sdk.test.d.ts.map +0 -1
- package/dist/pi/sdk.test.js +0 -449
- package/dist/pi/sdk.test.js.map +0 -1
- package/dist/runtime/agent-loader.test.d.ts +0 -2
- package/dist/runtime/agent-loader.test.d.ts.map +0 -1
- package/dist/runtime/agent-loader.test.js +0 -280
- package/dist/runtime/agent-loader.test.js.map +0 -1
- package/dist/runtime/client.test.d.ts +0 -2
- package/dist/runtime/client.test.d.ts.map +0 -1
- package/dist/runtime/client.test.js +0 -523
- package/dist/runtime/client.test.js.map +0 -1
- package/dist/runtime/path-config.test.d.ts +0 -2
- package/dist/runtime/path-config.test.d.ts.map +0 -1
- package/dist/runtime/path-config.test.js +0 -112
- package/dist/runtime/path-config.test.js.map +0 -1
|
@@ -1,137 +0,0 @@
|
|
|
1
|
-
import { beforeEach, describe, expect, it, vi } from 'vitest';
|
|
2
|
-
import { reviewPR } from './review-pr.js';
|
|
3
|
-
const { mockGitHubClient, createGitHubClient, executeUnifiedReview } = vi.hoisted(() => ({
|
|
4
|
-
mockGitHubClient: {
|
|
5
|
-
getPullRequest: vi.fn(),
|
|
6
|
-
getPRFiles: vi.fn(),
|
|
7
|
-
},
|
|
8
|
-
createGitHubClient: vi.fn(),
|
|
9
|
-
executeUnifiedReview: vi.fn(),
|
|
10
|
-
}));
|
|
11
|
-
vi.mock('../github/client.js', () => ({
|
|
12
|
-
createGitHubClient,
|
|
13
|
-
}));
|
|
14
|
-
vi.mock('../lib/unified-review-executor.js', () => ({
|
|
15
|
-
executeUnifiedReview,
|
|
16
|
-
}));
|
|
17
|
-
const baseConfig = {
|
|
18
|
-
pi: {},
|
|
19
|
-
gitlab: { url: 'https://gitlab.com', token: 'token' },
|
|
20
|
-
github: { token: 'token' },
|
|
21
|
-
review: {
|
|
22
|
-
agents: ['security', 'quality'],
|
|
23
|
-
ignorePatterns: [],
|
|
24
|
-
},
|
|
25
|
-
};
|
|
26
|
-
const baseOptions = {
|
|
27
|
-
owner: 'octocat',
|
|
28
|
-
repo: 'hello-world',
|
|
29
|
-
prNumber: 17,
|
|
30
|
-
postComments: true,
|
|
31
|
-
postErrorComment: true,
|
|
32
|
-
describe: false,
|
|
33
|
-
postDescription: false,
|
|
34
|
-
};
|
|
35
|
-
describe('review-pr', () => {
|
|
36
|
-
beforeEach(() => {
|
|
37
|
-
vi.clearAllMocks();
|
|
38
|
-
createGitHubClient.mockReturnValue(mockGitHubClient);
|
|
39
|
-
executeUnifiedReview.mockResolvedValue(undefined);
|
|
40
|
-
mockGitHubClient.getPullRequest.mockResolvedValue({
|
|
41
|
-
number: 17,
|
|
42
|
-
title: 'Test PR',
|
|
43
|
-
body: 'Test description',
|
|
44
|
-
user: { login: 'octocat' },
|
|
45
|
-
head: {
|
|
46
|
-
ref: 'feature/pi-migration',
|
|
47
|
-
sha: 'head-sha',
|
|
48
|
-
},
|
|
49
|
-
base: {
|
|
50
|
-
ref: 'main',
|
|
51
|
-
},
|
|
52
|
-
});
|
|
53
|
-
mockGitHubClient.getPRFiles.mockResolvedValue([
|
|
54
|
-
{
|
|
55
|
-
filename: 'src/app.ts',
|
|
56
|
-
status: 'modified',
|
|
57
|
-
additions: 2,
|
|
58
|
-
deletions: 1,
|
|
59
|
-
changes: 3,
|
|
60
|
-
patch: '@@ -8,2 +8,3 @@\n context line\n+new line\n-removed line',
|
|
61
|
-
},
|
|
62
|
-
{
|
|
63
|
-
filename: 'src/old.ts',
|
|
64
|
-
status: 'removed',
|
|
65
|
-
additions: 0,
|
|
66
|
-
deletions: 5,
|
|
67
|
-
changes: 5,
|
|
68
|
-
patch: '@@ -1 +0,0 @@\n-old line',
|
|
69
|
-
},
|
|
70
|
-
]);
|
|
71
|
-
});
|
|
72
|
-
it('loads PR context once and forwards diff-aware validators to unified review', async () => {
|
|
73
|
-
await reviewPR(baseConfig, baseOptions);
|
|
74
|
-
expect(createGitHubClient).toHaveBeenCalledTimes(1);
|
|
75
|
-
expect(mockGitHubClient.getPullRequest).toHaveBeenCalledWith('octocat', 'hello-world', 17);
|
|
76
|
-
expect(mockGitHubClient.getPRFiles).toHaveBeenCalledWith('octocat', 'hello-world', 17);
|
|
77
|
-
const unifiedOptions = vi.mocked(executeUnifiedReview).mock.calls[0][1];
|
|
78
|
-
expect(unifiedOptions.pullRequest).toEqual(expect.objectContaining({
|
|
79
|
-
number: 17,
|
|
80
|
-
title: 'Test PR',
|
|
81
|
-
headSha: 'head-sha',
|
|
82
|
-
}));
|
|
83
|
-
expect(unifiedOptions.changedFiles).toEqual(expect.arrayContaining([
|
|
84
|
-
expect.objectContaining({ filename: 'src/app.ts', status: 'modified' }),
|
|
85
|
-
expect.objectContaining({ filename: 'src/old.ts', status: 'removed' }),
|
|
86
|
-
]));
|
|
87
|
-
expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 8)).toBe(true);
|
|
88
|
-
expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 9)).toBe(true);
|
|
89
|
-
expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 10)).toBe(false);
|
|
90
|
-
expect(unifiedOptions.lineValidator?.isValidLine('src/old.ts', 1)).toBe(false);
|
|
91
|
-
const inlinePosition = unifiedOptions.createInlinePosition?.({
|
|
92
|
-
category: 'QUALITY',
|
|
93
|
-
severity: 'HIGH',
|
|
94
|
-
title: 'Example issue',
|
|
95
|
-
file: 'src/app.ts',
|
|
96
|
-
line: 9,
|
|
97
|
-
problem: 'Problem',
|
|
98
|
-
solution: 'Solution',
|
|
99
|
-
agent: 'quality',
|
|
100
|
-
}, {
|
|
101
|
-
head: {
|
|
102
|
-
sha: 'ignored-sha',
|
|
103
|
-
},
|
|
104
|
-
});
|
|
105
|
-
expect(inlinePosition).toEqual({
|
|
106
|
-
path: 'src/app.ts',
|
|
107
|
-
line: 9,
|
|
108
|
-
commitSha: 'head-sha',
|
|
109
|
-
});
|
|
110
|
-
});
|
|
111
|
-
it('rejects with actionable guidance when GITHUB_TOKEN is missing', async () => {
|
|
112
|
-
createGitHubClient.mockImplementationOnce(() => {
|
|
113
|
-
throw new Error('GITHUB_TOKEN environment variable is required');
|
|
114
|
-
});
|
|
115
|
-
await expect(reviewPR(baseConfig, baseOptions)).rejects.toThrow('Set GITHUB_TOKEN');
|
|
116
|
-
expect(executeUnifiedReview).not.toHaveBeenCalled();
|
|
117
|
-
});
|
|
118
|
-
it('maps GitHub authentication failures to actionable errors', async () => {
|
|
119
|
-
const authError = Object.assign(new Error('Bad credentials'), { status: 401 });
|
|
120
|
-
mockGitHubClient.getPullRequest.mockRejectedValueOnce(authError);
|
|
121
|
-
await expect(reviewPR(baseConfig, baseOptions)).rejects.toThrow('GitHub authentication failed for octocat/hello-world#17');
|
|
122
|
-
expect(executeUnifiedReview).not.toHaveBeenCalled();
|
|
123
|
-
});
|
|
124
|
-
it('maps GitHub not found failures with repository/pr remediation', async () => {
|
|
125
|
-
const notFoundError = Object.assign(new Error('Not Found'), { status: 404 });
|
|
126
|
-
mockGitHubClient.getPullRequest.mockRejectedValueOnce(notFoundError);
|
|
127
|
-
await expect(reviewPR(baseConfig, baseOptions)).rejects.toThrow('GitHub pull request not found: octocat/hello-world#17');
|
|
128
|
-
expect(executeUnifiedReview).not.toHaveBeenCalled();
|
|
129
|
-
});
|
|
130
|
-
it('maps GitHub rate limit failures with retry guidance', async () => {
|
|
131
|
-
const rateLimitError = Object.assign(new Error('API rate limit exceeded'), { status: 403 });
|
|
132
|
-
mockGitHubClient.getPRFiles.mockRejectedValueOnce(rateLimitError);
|
|
133
|
-
await expect(reviewPR(baseConfig, baseOptions)).rejects.toThrow('GitHub API rate limit reached while loading octocat/hello-world#17');
|
|
134
|
-
expect(executeUnifiedReview).not.toHaveBeenCalled();
|
|
135
|
-
});
|
|
136
|
-
});
|
|
137
|
-
//# sourceMappingURL=review-pr.test.js.map
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"review-pr.test.js","sourceRoot":"","sources":["../../src/cli/review-pr.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAI9D,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAE1C,MAAM,EAAE,gBAAgB,EAAE,kBAAkB,EAAE,oBAAoB,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IACvF,gBAAgB,EAAE;QAChB,cAAc,EAAE,EAAE,CAAC,EAAE,EAAE;QACvB,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;KACpB;IACD,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC3B,oBAAoB,EAAE,EAAE,CAAC,EAAE,EAAE;CAC9B,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,qBAAqB,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,kBAAkB;CACnB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,mCAAmC,EAAE,GAAG,EAAE,CAAC,CAAC;IAClD,oBAAoB;CACrB,CAAC,CAAC,CAAC;AAEJ,MAAM,UAAU,GAAG;IACjB,EAAE,EAAE,EAAE;IACN,MAAM,EAAE,EAAE,GAAG,EAAE,oBAAoB,EAAE,KAAK,EAAE,OAAO,EAAE;IACrD,MAAM,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE;IAC1B,MAAM,EAAE;QACN,MAAM,EAAE,CAAC,UAAU,EAAE,SAAS,CAAC;QAC/B,cAAc,EAAE,EAAE;KACnB;CACsB,CAAC;AAE1B,MAAM,WAAW,GAAG;IAClB,KAAK,EAAE,SAAS;IAChB,IAAI,EAAE,aAAa;IACnB,QAAQ,EAAE,EAAE;IACZ,YAAY,EAAE,IAAI;IAClB,gBAAgB,EAAE,IAAI;IACtB,QAAQ,EAAE,KAAK;IACf,eAAe,EAAE,KAAK;CACvB,CAAC;AAEF,QAAQ,CAAC,WAAW,EAAE,GAAG,EAAE;IACzB,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QAEnB,kBAAkB,CAAC,eAAe,CAAC,gBAAgB,CAAC,CAAC;QACrD,oBAAoB,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QAElD,gBAAgB,CAAC,cAAc,CAAC,iBAAiB,CAAC;YAChD,MAAM,EAAE,EAAE;YACV,KAAK,EAAE,SAAS;YAChB,IAAI,EAAE,kBAAkB;YACxB,IAAI,EAAE,EAAE,KAAK,EAAE,SAAS,EAAE;YAC1B,IAAI,EAAE;gBACJ,GAAG,EAAE,sBAAsB;gBAC3B,GAAG,EAAE,UAAU;aAChB;YACD,IAAI,EAAE;gBACJ,GAAG,EAAE,MAAM;aACZ;SACF,CAAC,CAAC;QAEH,gBAAgB,CAAC,UAAU,CAAC,iBAAiB,CAAC;YAC5C;gBACE,QAAQ,EAAE,YAAY;gBACtB,MAAM,EAAE,UAAU;gBAClB,SAAS,EAAE,CAAC;gBACZ,SAAS,EAAE,CAAC;gBACZ,OAAO,EAAE,CAAC;gBACV,KAAK,EAAE,0DAA0D;aAClE;YACD;gBACE,QAAQ,EAAE,YAAY;gBACtB,MAAM,EAAE,SAAS;gBACjB,SAAS,EAAE,CAAC;gBACZ,SAAS,EAAE,CAAC;gBACZ,OAAO,EAAE,CAAC;gBACV,KAAK,EAAE,0BAA0B;aAClC;SACF,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC;QAExC,MAAM,CAAC,kBAAkB,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,gBAAgB,CAAC,cAAc,CAAC,CAAC,oBAAoB,CAAC,SAAS,EAAE,aAAa,EAAE,EAAE,CAAC,CAAC;QAC3F,MAAM,CAAC,gBAAgB,CAAC,UAAU,CAAC,CAAC,oBAAoB,CAAC,SAAS,EAAE,aAAa,EAAE,EAAE,CAAC,CAAC;QAEvF,MAAM,cAAc,GAAG,EAAE,CAAC,MAAM,CAAC,oBAAoB,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAyB,CAAC;QAEhG,MAAM,CAAC,cAAc,CAAC,WAAW,CAAC,CAAC,OAAO,CACxC,MAAM,CAAC,gBAAgB,CAAC;YACtB,MAAM,EAAE,EAAE;YACV,KAAK,EAAE,SAAS;YAChB,OAAO,EAAE,UAAU;SACpB,CAAC,CACH,CAAC;QACF,MAAM,CAAC,cAAc,CAAC,YAAY,CAAC,CAAC,OAAO,CACzC,MAAM,CAAC,eAAe,CAAC;YACrB,MAAM,CAAC,gBAAgB,CAAC,EAAE,QAAQ,EAAE,YAAY,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC;YACvE,MAAM,CAAC,gBAAgB,CAAC,EAAE,QAAQ,EAAE,YAAY,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC;SACvE,CAAC,CACH,CAAC;QAEF,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC9E,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC9E,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAChF,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAE/E,MAAM,cAAc,GAAG,cAAc,CAAC,oBAAoB,EAAE,CAC1D;YACE,QAAQ,EAAE,SAAS;YACnB,QAAQ,EAAE,MAAM;YAChB,KAAK,EAAE,eAAe;YACtB,IAAI,EAAE,YAAY;YAClB,IAAI,EAAE,CAAC;YACP,OAAO,EAAE,SAAS;YAClB,QAAQ,EAAE,UAAU;YACpB,KAAK,EAAE,SAAS;SACF,EAChB;YACE,IAAI,EAAE;gBACJ,GAAG,EAAE,aAAa;aACnB;SACF,CACF,CAAC;QAEF,MAAM,CAAC,cAAc,CAAC,CAAC,OAAO,CAAC;YAC7B,IAAI,EAAE,YAAY;YAClB,IAAI,EAAE,CAAC;YACP,SAAS,EAAE,UAAU;SACtB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,kBAAkB,CAAC,sBAAsB,CAAC,GAAG,EAAE;YAC7C,MAAM,IAAI,KAAK,CAAC,+CAA+C,CAAC,CAAC;QACnE,CAAC,CAAC,CAAC;QAEH,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;QACpF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;QACxE,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,iBAAiB,CAAC,EAAE,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC,CAAC;QAC/E,gBAAgB,CAAC,cAAc,CAAC,qBAAqB,CAAC,SAAS,CAAC,CAAC;QAEjE,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC7D,yDAAyD,CAC1D,CAAC;QACF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,aAAa,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,WAAW,CAAC,EAAE,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC,CAAC;QAC7E,gBAAgB,CAAC,cAAc,CAAC,qBAAqB,CAAC,aAAa,CAAC,CAAC;QAErE,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC7D,uDAAuD,CACxD,CAAC;QACF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,MAAM,cAAc,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,yBAAyB,CAAC,EAAE,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC,CAAC;QAC5F,gBAAgB,CAAC,UAAU,CAAC,qBAAqB,CAAC,cAAc,CAAC,CAAC;QAElE,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC7D,oEAAoE,CACrE,CAAC;QACF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
package/dist/cli/review-url.d.ts
DELETED
|
@@ -1,35 +0,0 @@
|
|
|
1
|
-
import type { DRSConfig } from '../lib/config.js';
|
|
2
|
-
export interface ReviewUrlOptions {
|
|
3
|
-
url: string;
|
|
4
|
-
postComments: boolean;
|
|
5
|
-
postErrorComment: boolean;
|
|
6
|
-
fixInCursor?: boolean;
|
|
7
|
-
skipFixInCursor?: boolean;
|
|
8
|
-
describe: boolean;
|
|
9
|
-
postDescription: boolean;
|
|
10
|
-
codeQualityReport?: string;
|
|
11
|
-
outputPath?: string;
|
|
12
|
-
jsonOutput?: boolean;
|
|
13
|
-
baseBranch?: string;
|
|
14
|
-
debug?: boolean;
|
|
15
|
-
thinkingLevel?: string;
|
|
16
|
-
}
|
|
17
|
-
export type ParsedReviewUrl = {
|
|
18
|
-
platform: 'github';
|
|
19
|
-
owner: string;
|
|
20
|
-
repo: string;
|
|
21
|
-
prNumber: number;
|
|
22
|
-
} | {
|
|
23
|
-
platform: 'gitlab';
|
|
24
|
-
projectId: string;
|
|
25
|
-
mrIid: number;
|
|
26
|
-
};
|
|
27
|
-
/**
|
|
28
|
-
* Parse a GitHub or GitLab PR/MR URL into platform-specific identifiers.
|
|
29
|
-
*/
|
|
30
|
-
export declare function parseReviewUrl(url: string): ParsedReviewUrl;
|
|
31
|
-
/**
|
|
32
|
-
* Route review execution based on PR/MR URL.
|
|
33
|
-
*/
|
|
34
|
-
export declare function reviewByUrl(config: DRSConfig, options: ReviewUrlOptions): Promise<void>;
|
|
35
|
-
//# sourceMappingURL=review-url.d.ts.map
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"review-url.d.ts","sourceRoot":"","sources":["../../src/cli/review-url.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAIlD,MAAM,WAAW,gBAAgB;IAC/B,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,EAAE,OAAO,CAAC;IACtB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,QAAQ,EAAE,OAAO,CAAC;IAClB,eAAe,EAAE,OAAO,CAAC;IACzB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,MAAM,eAAe,GACvB;IACE,QAAQ,EAAE,QAAQ,CAAC;IACnB,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;CAClB,GACD;IACE,QAAQ,EAAE,QAAQ,CAAC;IACnB,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,CAAC;CACf,CAAC;AAaN;;GAEG;AACH,wBAAgB,cAAc,CAAC,GAAG,EAAE,MAAM,GAAG,eAAe,CAkE3D;AAED;;GAEG;AACH,wBAAsB,WAAW,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,gBAAgB,GAAG,OAAO,CAAC,IAAI,CAAC,CA+C7F"}
|
package/dist/cli/review-url.js
DELETED
|
@@ -1,110 +0,0 @@
|
|
|
1
|
-
import chalk from 'chalk';
|
|
2
|
-
import { reviewMR } from './review-mr.js';
|
|
3
|
-
import { reviewPR } from './review-pr.js';
|
|
4
|
-
function sanitizeUrlForError(url) {
|
|
5
|
-
try {
|
|
6
|
-
const parsed = new URL(url);
|
|
7
|
-
parsed.username = '';
|
|
8
|
-
parsed.password = '';
|
|
9
|
-
return `${parsed.origin}${parsed.pathname}`;
|
|
10
|
-
}
|
|
11
|
-
catch {
|
|
12
|
-
return url.replace(/:\/\/[^@/\s]+@/g, '://').replace(/[?#].*$/, '');
|
|
13
|
-
}
|
|
14
|
-
}
|
|
15
|
-
/**
|
|
16
|
-
* Parse a GitHub or GitLab PR/MR URL into platform-specific identifiers.
|
|
17
|
-
*/
|
|
18
|
-
export function parseReviewUrl(url) {
|
|
19
|
-
const safeUrl = sanitizeUrlForError(url);
|
|
20
|
-
let parsedUrl;
|
|
21
|
-
try {
|
|
22
|
-
parsedUrl = new URL(url);
|
|
23
|
-
}
|
|
24
|
-
catch {
|
|
25
|
-
throw new Error(`Invalid URL: ${safeUrl}`);
|
|
26
|
-
}
|
|
27
|
-
const pathParts = parsedUrl.pathname.split('/').filter(Boolean);
|
|
28
|
-
// GitHub: /owner/repo/pull/<number>
|
|
29
|
-
if (pathParts.length >= 4 && pathParts[2] === 'pull') {
|
|
30
|
-
const owner = pathParts[0];
|
|
31
|
-
const repo = pathParts[1];
|
|
32
|
-
const prNumber = parseInt(pathParts[3] ?? '', 10);
|
|
33
|
-
if (!owner || !repo || !Number.isSafeInteger(prNumber) || prNumber <= 0) {
|
|
34
|
-
throw new Error(`Invalid GitHub PR URL format: ${safeUrl}. Expected https://<host>/<owner>/<repo>/pull/<number>.`);
|
|
35
|
-
}
|
|
36
|
-
return {
|
|
37
|
-
platform: 'github',
|
|
38
|
-
owner,
|
|
39
|
-
repo,
|
|
40
|
-
prNumber,
|
|
41
|
-
};
|
|
42
|
-
}
|
|
43
|
-
// GitLab: /group/subgroup/repo/-/merge_requests/<number>
|
|
44
|
-
const mergeRequestIndex = pathParts.indexOf('merge_requests');
|
|
45
|
-
if (mergeRequestIndex >= 0) {
|
|
46
|
-
if (mergeRequestIndex < 2 || pathParts[mergeRequestIndex - 1] !== '-') {
|
|
47
|
-
throw new Error(`Invalid GitLab MR URL format: ${safeUrl}. Expected .../<group>/<repo>/-/merge_requests/<number>.`);
|
|
48
|
-
}
|
|
49
|
-
const mrIid = parseInt(pathParts[mergeRequestIndex + 1] ?? '', 10);
|
|
50
|
-
if (!Number.isSafeInteger(mrIid) || mrIid <= 0) {
|
|
51
|
-
throw new Error(`Invalid GitLab MR URL format: ${safeUrl}. Merge request IID must be a positive integer.`);
|
|
52
|
-
}
|
|
53
|
-
const repo = pathParts[mergeRequestIndex - 2];
|
|
54
|
-
const ownerParts = pathParts.slice(0, mergeRequestIndex - 2);
|
|
55
|
-
if (!repo || ownerParts.length === 0) {
|
|
56
|
-
throw new Error(`Invalid GitLab MR URL format: ${safeUrl}. Expected .../<group>/<repo>/-/merge_requests/<number>.`);
|
|
57
|
-
}
|
|
58
|
-
return {
|
|
59
|
-
platform: 'gitlab',
|
|
60
|
-
projectId: `${ownerParts.join('/')}/${repo}`,
|
|
61
|
-
mrIid,
|
|
62
|
-
};
|
|
63
|
-
}
|
|
64
|
-
throw new Error(`Unsupported review URL: ${safeUrl}. Expected a GitHub pull request URL (.../pull/<number>) or GitLab merge request URL (.../-/merge_requests/<number>).`);
|
|
65
|
-
}
|
|
66
|
-
/**
|
|
67
|
-
* Route review execution based on PR/MR URL.
|
|
68
|
-
*/
|
|
69
|
-
export async function reviewByUrl(config, options) {
|
|
70
|
-
const parsed = parseReviewUrl(options.url);
|
|
71
|
-
if (parsed.platform === 'github') {
|
|
72
|
-
if (options.codeQualityReport) {
|
|
73
|
-
console.warn(chalk.yellow('⚠ --code-quality-report is only supported for GitLab MRs. Ignoring this option for GitHub PR reviews.'));
|
|
74
|
-
}
|
|
75
|
-
await reviewPR(config, {
|
|
76
|
-
owner: parsed.owner,
|
|
77
|
-
repo: parsed.repo,
|
|
78
|
-
prNumber: parsed.prNumber,
|
|
79
|
-
postComments: options.postComments,
|
|
80
|
-
postErrorComment: options.postErrorComment,
|
|
81
|
-
fixInCursor: options.fixInCursor,
|
|
82
|
-
skipFixInCursor: options.skipFixInCursor,
|
|
83
|
-
describe: options.describe,
|
|
84
|
-
postDescription: options.postDescription,
|
|
85
|
-
outputPath: options.outputPath,
|
|
86
|
-
jsonOutput: options.jsonOutput,
|
|
87
|
-
baseBranch: options.baseBranch,
|
|
88
|
-
debug: options.debug,
|
|
89
|
-
thinkingLevel: options.thinkingLevel,
|
|
90
|
-
});
|
|
91
|
-
return;
|
|
92
|
-
}
|
|
93
|
-
await reviewMR(config, {
|
|
94
|
-
projectId: parsed.projectId,
|
|
95
|
-
mrIid: parsed.mrIid,
|
|
96
|
-
postComments: options.postComments,
|
|
97
|
-
postErrorComment: options.postErrorComment,
|
|
98
|
-
fixInCursor: options.fixInCursor,
|
|
99
|
-
skipFixInCursor: options.skipFixInCursor,
|
|
100
|
-
describe: options.describe,
|
|
101
|
-
postDescription: options.postDescription,
|
|
102
|
-
codeQualityReport: options.codeQualityReport,
|
|
103
|
-
outputPath: options.outputPath,
|
|
104
|
-
jsonOutput: options.jsonOutput,
|
|
105
|
-
baseBranch: options.baseBranch,
|
|
106
|
-
debug: options.debug,
|
|
107
|
-
thinkingLevel: options.thinkingLevel,
|
|
108
|
-
});
|
|
109
|
-
}
|
|
110
|
-
//# sourceMappingURL=review-url.js.map
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"review-url.js","sourceRoot":"","sources":["../../src/cli/review-url.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,MAAM,OAAO,CAAC;AAE1B,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAC1C,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AA+B1C,SAAS,mBAAmB,CAAC,GAAW;IACtC,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,IAAI,GAAG,CAAC,GAAG,CAAC,CAAC;QAC5B,MAAM,CAAC,QAAQ,GAAG,EAAE,CAAC;QACrB,MAAM,CAAC,QAAQ,GAAG,EAAE,CAAC;QACrB,OAAO,GAAG,MAAM,CAAC,MAAM,GAAG,MAAM,CAAC,QAAQ,EAAE,CAAC;IAC9C,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,GAAG,CAAC,OAAO,CAAC,iBAAiB,EAAE,KAAK,CAAC,CAAC,OAAO,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC;IACtE,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,cAAc,CAAC,GAAW;IACxC,MAAM,OAAO,GAAG,mBAAmB,CAAC,GAAG,CAAC,CAAC;IAEzC,IAAI,SAAc,CAAC;IACnB,IAAI,CAAC;QACH,SAAS,GAAG,IAAI,GAAG,CAAC,GAAG,CAAC,CAAC;IAC3B,CAAC;IAAC,MAAM,CAAC;QACP,MAAM,IAAI,KAAK,CAAC,gBAAgB,OAAO,EAAE,CAAC,CAAC;IAC7C,CAAC;IAED,MAAM,SAAS,GAAG,SAAS,CAAC,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;IAEhE,oCAAoC;IACpC,IAAI,SAAS,CAAC,MAAM,IAAI,CAAC,IAAI,SAAS,CAAC,CAAC,CAAC,KAAK,MAAM,EAAE,CAAC;QACrD,MAAM,KAAK,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC;QAC3B,MAAM,IAAI,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC;QAC1B,MAAM,QAAQ,GAAG,QAAQ,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,EAAE,EAAE,EAAE,CAAC,CAAC;QAElD,IAAI,CAAC,KAAK,IAAI,CAAC,IAAI,IAAI,CAAC,MAAM,CAAC,aAAa,CAAC,QAAQ,CAAC,IAAI,QAAQ,IAAI,CAAC,EAAE,CAAC;YACxE,MAAM,IAAI,KAAK,CACb,iCAAiC,OAAO,yDAAyD,CAClG,CAAC;QACJ,CAAC;QAED,OAAO;YACL,QAAQ,EAAE,QAAQ;YAClB,KAAK;YACL,IAAI;YACJ,QAAQ;SACT,CAAC;IACJ,CAAC;IAED,yDAAyD;IACzD,MAAM,iBAAiB,GAAG,SAAS,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAC;IAC9D,IAAI,iBAAiB,IAAI,CAAC,EAAE,CAAC;QAC3B,IAAI,iBAAiB,GAAG,CAAC,IAAI,SAAS,CAAC,iBAAiB,GAAG,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;YACtE,MAAM,IAAI,KAAK,CACb,iCAAiC,OAAO,0DAA0D,CACnG,CAAC;QACJ,CAAC;QAED,MAAM,KAAK,GAAG,QAAQ,CAAC,SAAS,CAAC,iBAAiB,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,EAAE,CAAC,CAAC;QACnE,IAAI,CAAC,MAAM,CAAC,aAAa,CAAC,KAAK,CAAC,IAAI,KAAK,IAAI,CAAC,EAAE,CAAC;YAC/C,MAAM,IAAI,KAAK,CACb,iCAAiC,OAAO,iDAAiD,CAC1F,CAAC;QACJ,CAAC;QAED,MAAM,IAAI,GAAG,SAAS,CAAC,iBAAiB,GAAG,CAAC,CAAC,CAAC;QAC9C,MAAM,UAAU,GAAG,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,iBAAiB,GAAG,CAAC,CAAC,CAAC;QAC7D,IAAI,CAAC,IAAI,IAAI,UAAU,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACrC,MAAM,IAAI,KAAK,CACb,iCAAiC,OAAO,0DAA0D,CACnG,CAAC;QACJ,CAAC;QAED,OAAO;YACL,QAAQ,EAAE,QAAQ;YAClB,SAAS,EAAE,GAAG,UAAU,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,IAAI,EAAE;YAC5C,KAAK;SACN,CAAC;IACJ,CAAC;IAED,MAAM,IAAI,KAAK,CACb,2BAA2B,OAAO,uHAAuH,CAC1J,CAAC;AACJ,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,MAAiB,EAAE,OAAyB;IAC5E,MAAM,MAAM,GAAG,cAAc,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;IAE3C,IAAI,MAAM,CAAC,QAAQ,KAAK,QAAQ,EAAE,CAAC;QACjC,IAAI,OAAO,CAAC,iBAAiB,EAAE,CAAC;YAC9B,OAAO,CAAC,IAAI,CACV,KAAK,CAAC,MAAM,CACV,uGAAuG,CACxG,CACF,CAAC;QACJ,CAAC;QAED,MAAM,QAAQ,CAAC,MAAM,EAAE;YACrB,KAAK,EAAE,MAAM,CAAC,KAAK;YACnB,IAAI,EAAE,MAAM,CAAC,IAAI;YACjB,QAAQ,EAAE,MAAM,CAAC,QAAQ;YACzB,YAAY,EAAE,OAAO,CAAC,YAAY;YAClC,gBAAgB,EAAE,OAAO,CAAC,gBAAgB;YAC1C,WAAW,EAAE,OAAO,CAAC,WAAW;YAChC,eAAe,EAAE,OAAO,CAAC,eAAe;YACxC,QAAQ,EAAE,OAAO,CAAC,QAAQ;YAC1B,eAAe,EAAE,OAAO,CAAC,eAAe;YACxC,UAAU,EAAE,OAAO,CAAC,UAAU;YAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;YAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;YAC9B,KAAK,EAAE,OAAO,CAAC,KAAK;YACpB,aAAa,EAAE,OAAO,CAAC,aAAa;SACrC,CAAC,CAAC;QACH,OAAO;IACT,CAAC;IAED,MAAM,QAAQ,CAAC,MAAM,EAAE;QACrB,SAAS,EAAE,MAAM,CAAC,SAAS;QAC3B,KAAK,EAAE,MAAM,CAAC,KAAK;QACnB,YAAY,EAAE,OAAO,CAAC,YAAY;QAClC,gBAAgB,EAAE,OAAO,CAAC,gBAAgB;QAC1C,WAAW,EAAE,OAAO,CAAC,WAAW;QAChC,eAAe,EAAE,OAAO,CAAC,eAAe;QACxC,QAAQ,EAAE,OAAO,CAAC,QAAQ;QAC1B,eAAe,EAAE,OAAO,CAAC,eAAe;QACxC,iBAAiB,EAAE,OAAO,CAAC,iBAAiB;QAC5C,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,KAAK,EAAE,OAAO,CAAC,KAAK;QACpB,aAAa,EAAE,OAAO,CAAC,aAAa;KACrC,CAAC,CAAC;AACL,CAAC"}
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"review-url.test.d.ts","sourceRoot":"","sources":["../../src/cli/review-url.test.ts"],"names":[],"mappings":""}
|
|
@@ -1,132 +0,0 @@
|
|
|
1
|
-
import { beforeEach, describe, expect, it, vi } from 'vitest';
|
|
2
|
-
import { parseReviewUrl, reviewByUrl } from './review-url.js';
|
|
3
|
-
const { reviewPR, reviewMR } = vi.hoisted(() => ({
|
|
4
|
-
reviewPR: vi.fn(),
|
|
5
|
-
reviewMR: vi.fn(),
|
|
6
|
-
}));
|
|
7
|
-
vi.mock('./review-pr.js', () => ({
|
|
8
|
-
reviewPR,
|
|
9
|
-
}));
|
|
10
|
-
vi.mock('./review-mr.js', () => ({
|
|
11
|
-
reviewMR,
|
|
12
|
-
}));
|
|
13
|
-
const baseConfig = {
|
|
14
|
-
pi: {},
|
|
15
|
-
gitlab: { url: 'https://gitlab.com', token: 'token' },
|
|
16
|
-
github: { token: 'token' },
|
|
17
|
-
review: {
|
|
18
|
-
agents: ['security', 'quality'],
|
|
19
|
-
ignorePatterns: [],
|
|
20
|
-
},
|
|
21
|
-
};
|
|
22
|
-
describe('parseReviewUrl', () => {
|
|
23
|
-
it('parses GitHub pull request URLs', () => {
|
|
24
|
-
const parsed = parseReviewUrl('https://github.com/octocat/hello-world/pull/123');
|
|
25
|
-
expect(parsed).toEqual({
|
|
26
|
-
platform: 'github',
|
|
27
|
-
owner: 'octocat',
|
|
28
|
-
repo: 'hello-world',
|
|
29
|
-
prNumber: 123,
|
|
30
|
-
});
|
|
31
|
-
});
|
|
32
|
-
it('parses GitLab merge request URLs with subgroups', () => {
|
|
33
|
-
const parsed = parseReviewUrl('https://gitlab.example.com/org/security/tools/drs/-/merge_requests/42');
|
|
34
|
-
expect(parsed).toEqual({
|
|
35
|
-
platform: 'gitlab',
|
|
36
|
-
projectId: 'org/security/tools/drs',
|
|
37
|
-
mrIid: 42,
|
|
38
|
-
});
|
|
39
|
-
});
|
|
40
|
-
it('throws for unsupported URLs', () => {
|
|
41
|
-
expect(() => parseReviewUrl('https://example.com/org/repo/issues/10')).toThrow('Unsupported review URL');
|
|
42
|
-
});
|
|
43
|
-
it('redacts URL credentials in error messages', () => {
|
|
44
|
-
let message = '';
|
|
45
|
-
try {
|
|
46
|
-
parseReviewUrl('https://alice:super-secret-token@example.com/not-a-pr');
|
|
47
|
-
}
|
|
48
|
-
catch (error) {
|
|
49
|
-
message = error instanceof Error ? error.message : String(error);
|
|
50
|
-
}
|
|
51
|
-
expect(message).toContain('Unsupported review URL: https://example.com/not-a-pr');
|
|
52
|
-
expect(message).not.toContain('alice');
|
|
53
|
-
expect(message).not.toContain('super-secret-token');
|
|
54
|
-
});
|
|
55
|
-
});
|
|
56
|
-
describe('reviewByUrl', () => {
|
|
57
|
-
beforeEach(() => {
|
|
58
|
-
vi.clearAllMocks();
|
|
59
|
-
reviewPR.mockResolvedValue(undefined);
|
|
60
|
-
reviewMR.mockResolvedValue(undefined);
|
|
61
|
-
});
|
|
62
|
-
it('routes GitHub URLs to reviewPR', async () => {
|
|
63
|
-
await reviewByUrl(baseConfig, {
|
|
64
|
-
url: 'https://github.com/octocat/hello-world/pull/10',
|
|
65
|
-
postComments: true,
|
|
66
|
-
postErrorComment: true,
|
|
67
|
-
describe: true,
|
|
68
|
-
postDescription: false,
|
|
69
|
-
outputPath: 'review.json',
|
|
70
|
-
jsonOutput: true,
|
|
71
|
-
baseBranch: 'release/2026-01',
|
|
72
|
-
debug: true,
|
|
73
|
-
});
|
|
74
|
-
expect(reviewPR).toHaveBeenCalledWith(baseConfig, {
|
|
75
|
-
owner: 'octocat',
|
|
76
|
-
repo: 'hello-world',
|
|
77
|
-
prNumber: 10,
|
|
78
|
-
postComments: true,
|
|
79
|
-
postErrorComment: true,
|
|
80
|
-
describe: true,
|
|
81
|
-
postDescription: false,
|
|
82
|
-
outputPath: 'review.json',
|
|
83
|
-
jsonOutput: true,
|
|
84
|
-
baseBranch: 'release/2026-01',
|
|
85
|
-
debug: true,
|
|
86
|
-
});
|
|
87
|
-
expect(reviewMR).not.toHaveBeenCalled();
|
|
88
|
-
});
|
|
89
|
-
it('routes GitLab URLs to reviewMR', async () => {
|
|
90
|
-
await reviewByUrl(baseConfig, {
|
|
91
|
-
url: 'https://gitlab.com/group/subgroup/repo/-/merge_requests/88',
|
|
92
|
-
postComments: false,
|
|
93
|
-
postErrorComment: false,
|
|
94
|
-
describe: false,
|
|
95
|
-
postDescription: false,
|
|
96
|
-
codeQualityReport: 'gl-code-quality-report.json',
|
|
97
|
-
outputPath: 'review.json',
|
|
98
|
-
jsonOutput: false,
|
|
99
|
-
baseBranch: 'main',
|
|
100
|
-
debug: false,
|
|
101
|
-
});
|
|
102
|
-
expect(reviewMR).toHaveBeenCalledWith(baseConfig, {
|
|
103
|
-
projectId: 'group/subgroup/repo',
|
|
104
|
-
mrIid: 88,
|
|
105
|
-
postComments: false,
|
|
106
|
-
postErrorComment: false,
|
|
107
|
-
describe: false,
|
|
108
|
-
postDescription: false,
|
|
109
|
-
codeQualityReport: 'gl-code-quality-report.json',
|
|
110
|
-
outputPath: 'review.json',
|
|
111
|
-
jsonOutput: false,
|
|
112
|
-
baseBranch: 'main',
|
|
113
|
-
debug: false,
|
|
114
|
-
});
|
|
115
|
-
expect(reviewPR).not.toHaveBeenCalled();
|
|
116
|
-
});
|
|
117
|
-
it('warns and ignores --code-quality-report for GitHub URLs', async () => {
|
|
118
|
-
const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined);
|
|
119
|
-
await reviewByUrl(baseConfig, {
|
|
120
|
-
url: 'https://github.com/octocat/hello-world/pull/9',
|
|
121
|
-
postComments: false,
|
|
122
|
-
postErrorComment: false,
|
|
123
|
-
describe: false,
|
|
124
|
-
postDescription: false,
|
|
125
|
-
codeQualityReport: 'gl-code-quality-report.json',
|
|
126
|
-
jsonOutput: false,
|
|
127
|
-
});
|
|
128
|
-
expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining('--code-quality-report is only supported for GitLab MRs'));
|
|
129
|
-
warnSpy.mockRestore();
|
|
130
|
-
});
|
|
131
|
-
});
|
|
132
|
-
//# sourceMappingURL=review-url.test.js.map
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"review-url.test.js","sourceRoot":"","sources":["../../src/cli/review-url.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE9D,OAAO,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,iBAAiB,CAAC;AAE9D,MAAM,EAAE,QAAQ,EAAE,QAAQ,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IAC/C,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;IACjB,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;CAClB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,gBAAgB,EAAE,GAAG,EAAE,CAAC,CAAC;IAC/B,QAAQ;CACT,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,gBAAgB,EAAE,GAAG,EAAE,CAAC,CAAC;IAC/B,QAAQ;CACT,CAAC,CAAC,CAAC;AAEJ,MAAM,UAAU,GAAG;IACjB,EAAE,EAAE,EAAE;IACN,MAAM,EAAE,EAAE,GAAG,EAAE,oBAAoB,EAAE,KAAK,EAAE,OAAO,EAAE;IACrD,MAAM,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE;IAC1B,MAAM,EAAE;QACN,MAAM,EAAE,CAAC,UAAU,EAAE,SAAS,CAAC;QAC/B,cAAc,EAAE,EAAE;KACnB;CACsB,CAAC;AAE1B,QAAQ,CAAC,gBAAgB,EAAE,GAAG,EAAE;IAC9B,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;QACzC,MAAM,MAAM,GAAG,cAAc,CAAC,iDAAiD,CAAC,CAAC;QAEjF,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;YACrB,QAAQ,EAAE,QAAQ;YAClB,KAAK,EAAE,SAAS;YAChB,IAAI,EAAE,aAAa;YACnB,QAAQ,EAAE,GAAG;SACd,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QACzD,MAAM,MAAM,GAAG,cAAc,CAC3B,uEAAuE,CACxE,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;YACrB,QAAQ,EAAE,QAAQ;YAClB,SAAS,EAAE,wBAAwB;YACnC,KAAK,EAAE,EAAE;SACV,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,CAAC,GAAG,EAAE,CAAC,cAAc,CAAC,wCAAwC,CAAC,CAAC,CAAC,OAAO,CAC5E,wBAAwB,CACzB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,IAAI,OAAO,GAAG,EAAE,CAAC;QAEjB,IAAI,CAAC;YACH,cAAc,CAAC,uDAAuD,CAAC,CAAC;QAC1E,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACnE,CAAC;QAED,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,sDAAsD,CAAC,CAAC;QAClF,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;QACvC,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;IAC3B,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,QAAQ,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QACtC,QAAQ,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,KAAK,IAAI,EAAE;QAC9C,MAAM,WAAW,CAAC,UAAU,EAAE;YAC5B,GAAG,EAAE,gDAAgD;YACrD,YAAY,EAAE,IAAI;YAClB,gBAAgB,EAAE,IAAI;YACtB,QAAQ,EAAE,IAAI;YACd,eAAe,EAAE,KAAK;YACtB,UAAU,EAAE,aAAa;YACzB,UAAU,EAAE,IAAI;YAChB,UAAU,EAAE,iBAAiB;YAC7B,KAAK,EAAE,IAAI;SACZ,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,CAAC,oBAAoB,CAAC,UAAU,EAAE;YAChD,KAAK,EAAE,SAAS;YAChB,IAAI,EAAE,aAAa;YACnB,QAAQ,EAAE,EAAE;YACZ,YAAY,EAAE,IAAI;YAClB,gBAAgB,EAAE,IAAI;YACtB,QAAQ,EAAE,IAAI;YACd,eAAe,EAAE,KAAK;YACtB,UAAU,EAAE,aAAa;YACzB,UAAU,EAAE,IAAI;YAChB,UAAU,EAAE,iBAAiB;YAC7B,KAAK,EAAE,IAAI;SACZ,CAAC,CAAC;QACH,MAAM,CAAC,QAAQ,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,KAAK,IAAI,EAAE;QAC9C,MAAM,WAAW,CAAC,UAAU,EAAE;YAC5B,GAAG,EAAE,4DAA4D;YACjE,YAAY,EAAE,KAAK;YACnB,gBAAgB,EAAE,KAAK;YACvB,QAAQ,EAAE,KAAK;YACf,eAAe,EAAE,KAAK;YACtB,iBAAiB,EAAE,6BAA6B;YAChD,UAAU,EAAE,aAAa;YACzB,UAAU,EAAE,KAAK;YACjB,UAAU,EAAE,MAAM;YAClB,KAAK,EAAE,KAAK;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,CAAC,oBAAoB,CAAC,UAAU,EAAE;YAChD,SAAS,EAAE,qBAAqB;YAChC,KAAK,EAAE,EAAE;YACT,YAAY,EAAE,KAAK;YACnB,gBAAgB,EAAE,KAAK;YACvB,QAAQ,EAAE,KAAK;YACf,eAAe,EAAE,KAAK;YACtB,iBAAiB,EAAE,6BAA6B;YAChD,UAAU,EAAE,aAAa;YACzB,UAAU,EAAE,KAAK;YACjB,UAAU,EAAE,MAAM;YAClB,KAAK,EAAE,KAAK;SACb,CAAC,CAAC;QACH,MAAM,CAAC,QAAQ,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC,CAAC;QAE9E,MAAM,WAAW,CAAC,UAAU,EAAE;YAC5B,GAAG,EAAE,+CAA+C;YACpD,YAAY,EAAE,KAAK;YACnB,gBAAgB,EAAE,KAAK;YACvB,QAAQ,EAAE,KAAK;YACf,eAAe,EAAE,KAAK;YACtB,iBAAiB,EAAE,6BAA6B;YAChD,UAAU,EAAE,KAAK;SAClB,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,CAAC,oBAAoB,CAClC,MAAM,CAAC,gBAAgB,CAAC,wDAAwD,CAAC,CAClF,CAAC;QAEF,OAAO,CAAC,WAAW,EAAE,CAAC;IACxB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -1,15 +0,0 @@
|
|
|
1
|
-
import type { DRSConfig } from '../lib/config.js';
|
|
2
|
-
export interface ShowChangesOptions {
|
|
3
|
-
owner?: string;
|
|
4
|
-
repo?: string;
|
|
5
|
-
prNumber?: number;
|
|
6
|
-
projectId?: string;
|
|
7
|
-
mrIid?: number;
|
|
8
|
-
baseBranch?: string;
|
|
9
|
-
file?: string;
|
|
10
|
-
outputPath?: string;
|
|
11
|
-
jsonOutput?: boolean;
|
|
12
|
-
workingDir?: string;
|
|
13
|
-
}
|
|
14
|
-
export declare function showChanges(config: DRSConfig, options: ShowChangesOptions): Promise<void>;
|
|
15
|
-
//# sourceMappingURL=show-changes.d.ts.map
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"show-changes.d.ts","sourceRoot":"","sources":["../../src/cli/show-changes.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAQlD,MAAM,WAAW,kBAAkB;IACjC,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAyGD,wBAAsB,WAAW,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,kBAAkB,GAAG,OAAO,CAAC,IAAI,CAAC,CAqH/F"}
|
package/dist/cli/show-changes.js
DELETED
|
@@ -1,184 +0,0 @@
|
|
|
1
|
-
import chalk from 'chalk';
|
|
2
|
-
import { writeFile } from 'fs/promises';
|
|
3
|
-
import { resolve } from 'path';
|
|
4
|
-
import { buildBaseInstructions } from '../lib/review-core.js';
|
|
5
|
-
import { filterIgnoredFiles } from '../lib/review-orchestrator.js';
|
|
6
|
-
import { createGitHubClient } from '../github/client.js';
|
|
7
|
-
import { GitHubPlatformAdapter } from '../github/platform-adapter.js';
|
|
8
|
-
import { createGitLabClient } from '../gitlab/client.js';
|
|
9
|
-
import { GitLabPlatformAdapter } from '../gitlab/platform-adapter.js';
|
|
10
|
-
function parseNumber(value, label) {
|
|
11
|
-
if (typeof value === 'number' && Number.isFinite(value))
|
|
12
|
-
return value;
|
|
13
|
-
if (typeof value === 'string') {
|
|
14
|
-
const parsed = parseInt(value, 10);
|
|
15
|
-
if (Number.isFinite(parsed))
|
|
16
|
-
return parsed;
|
|
17
|
-
}
|
|
18
|
-
throw new Error(`${label} is required`);
|
|
19
|
-
}
|
|
20
|
-
function ensureFileMatch(files, filename) {
|
|
21
|
-
if (!filename)
|
|
22
|
-
return files;
|
|
23
|
-
const matches = files.filter((file) => file.filename === filename);
|
|
24
|
-
if (matches.length === 0) {
|
|
25
|
-
throw new Error(`No matching file "${filename}" found in PR/MR changes.`);
|
|
26
|
-
}
|
|
27
|
-
return matches;
|
|
28
|
-
}
|
|
29
|
-
function normalizeBaseBranch(baseBranch) {
|
|
30
|
-
if (!baseBranch)
|
|
31
|
-
return undefined;
|
|
32
|
-
return baseBranch.startsWith('origin/') ? baseBranch : `origin/${baseBranch}`;
|
|
33
|
-
}
|
|
34
|
-
function resolveBaseBranch(cliBaseBranch, targetBranch) {
|
|
35
|
-
if (cliBaseBranch) {
|
|
36
|
-
const resolved = normalizeBaseBranch(cliBaseBranch);
|
|
37
|
-
return {
|
|
38
|
-
baseBranch: cliBaseBranch,
|
|
39
|
-
resolvedBaseBranch: resolved,
|
|
40
|
-
diffCommand: `git diff ${resolved}...HEAD -- <file>`,
|
|
41
|
-
source: 'cli',
|
|
42
|
-
};
|
|
43
|
-
}
|
|
44
|
-
if (process.env.DRS_BASE_BRANCH) {
|
|
45
|
-
const resolved = normalizeBaseBranch(process.env.DRS_BASE_BRANCH);
|
|
46
|
-
return {
|
|
47
|
-
baseBranch: process.env.DRS_BASE_BRANCH,
|
|
48
|
-
resolvedBaseBranch: resolved,
|
|
49
|
-
diffCommand: `git diff ${resolved}...HEAD -- <file>`,
|
|
50
|
-
source: 'env:DRS_BASE_BRANCH',
|
|
51
|
-
};
|
|
52
|
-
}
|
|
53
|
-
if (process.env.GITHUB_BASE_REF) {
|
|
54
|
-
const resolved = normalizeBaseBranch(process.env.GITHUB_BASE_REF);
|
|
55
|
-
return {
|
|
56
|
-
baseBranch: process.env.GITHUB_BASE_REF,
|
|
57
|
-
resolvedBaseBranch: resolved,
|
|
58
|
-
diffCommand: `git diff ${resolved}...HEAD -- <file>`,
|
|
59
|
-
source: 'env:GITHUB_BASE_REF',
|
|
60
|
-
};
|
|
61
|
-
}
|
|
62
|
-
if (process.env.CI_MERGE_REQUEST_TARGET_BRANCH_NAME) {
|
|
63
|
-
const resolved = normalizeBaseBranch(process.env.CI_MERGE_REQUEST_TARGET_BRANCH_NAME);
|
|
64
|
-
return {
|
|
65
|
-
baseBranch: process.env.CI_MERGE_REQUEST_TARGET_BRANCH_NAME,
|
|
66
|
-
resolvedBaseBranch: resolved,
|
|
67
|
-
diffCommand: `git diff ${resolved}...HEAD -- <file>`,
|
|
68
|
-
source: 'env:CI_MERGE_REQUEST_TARGET_BRANCH_NAME',
|
|
69
|
-
};
|
|
70
|
-
}
|
|
71
|
-
if (targetBranch) {
|
|
72
|
-
const resolved = normalizeBaseBranch(targetBranch);
|
|
73
|
-
return {
|
|
74
|
-
baseBranch: targetBranch,
|
|
75
|
-
resolvedBaseBranch: resolved,
|
|
76
|
-
diffCommand: `git diff ${resolved}...HEAD -- <file>`,
|
|
77
|
-
source: 'pr:targetBranch',
|
|
78
|
-
};
|
|
79
|
-
}
|
|
80
|
-
return {
|
|
81
|
-
diffCommand: 'git diff HEAD~1 -- <file>',
|
|
82
|
-
};
|
|
83
|
-
}
|
|
84
|
-
export async function showChanges(config, options) {
|
|
85
|
-
const workingDir = options.workingDir ?? process.cwd();
|
|
86
|
-
const isGitHub = Boolean(options.owner ?? options.repo ?? options.prNumber);
|
|
87
|
-
const isGitLab = Boolean(options.projectId ?? options.mrIid);
|
|
88
|
-
if (isGitHub && isGitLab) {
|
|
89
|
-
throw new Error('Specify either GitHub options (--owner/--repo/--pr) or GitLab options (--project/--mr), not both.');
|
|
90
|
-
}
|
|
91
|
-
if (!isGitHub && !isGitLab) {
|
|
92
|
-
throw new Error('Specify GitHub options (--owner/--repo/--pr) or GitLab options (--project/--mr).');
|
|
93
|
-
}
|
|
94
|
-
if (isGitHub) {
|
|
95
|
-
const owner = options.owner;
|
|
96
|
-
const repo = options.repo;
|
|
97
|
-
if (!owner || !repo) {
|
|
98
|
-
throw new Error('GitHub requires --owner and --repo');
|
|
99
|
-
}
|
|
100
|
-
const prNumber = parseNumber(options.prNumber, 'PR number (--pr)');
|
|
101
|
-
const projectId = `${owner}/${repo}`;
|
|
102
|
-
const githubClient = createGitHubClient();
|
|
103
|
-
const platformClient = new GitHubPlatformAdapter(githubClient);
|
|
104
|
-
const pr = await platformClient.getPullRequest(projectId, prNumber);
|
|
105
|
-
const allFiles = await platformClient.getChangedFiles(projectId, prNumber);
|
|
106
|
-
const filesWithDiffs = allFiles
|
|
107
|
-
.filter((file) => file.status !== 'removed')
|
|
108
|
-
.map((file) => ({ filename: file.filename, patch: file.patch }));
|
|
109
|
-
const filteredNames = filterIgnoredFiles(filesWithDiffs.map((file) => file.filename), config);
|
|
110
|
-
const filteredFiles = filesWithDiffs.filter((file) => filteredNames.includes(file.filename));
|
|
111
|
-
const scopedFiles = ensureFileMatch(filteredFiles, options.file);
|
|
112
|
-
const label = `PR/MR #${pr.number}`;
|
|
113
|
-
const baseBranchResolution = resolveBaseBranch(options.baseBranch, pr.targetBranch);
|
|
114
|
-
const instructions = buildBaseInstructions(label, scopedFiles.map((file) => ({ filename: file.filename })), baseBranchResolution.diffCommand);
|
|
115
|
-
const payload = {
|
|
116
|
-
label,
|
|
117
|
-
files: scopedFiles,
|
|
118
|
-
instructions,
|
|
119
|
-
resolvedBaseBranch: baseBranchResolution.resolvedBaseBranch,
|
|
120
|
-
diffCommand: baseBranchResolution.diffCommand,
|
|
121
|
-
diffCommandSource: baseBranchResolution.source,
|
|
122
|
-
metadata: {
|
|
123
|
-
projectId,
|
|
124
|
-
prNumber: pr.number,
|
|
125
|
-
title: pr.title,
|
|
126
|
-
sourceBranch: pr.sourceBranch,
|
|
127
|
-
targetBranch: pr.targetBranch,
|
|
128
|
-
headSha: pr.headSha,
|
|
129
|
-
},
|
|
130
|
-
};
|
|
131
|
-
await writeOutput(payload, options.outputPath, options.jsonOutput, workingDir);
|
|
132
|
-
return;
|
|
133
|
-
}
|
|
134
|
-
const projectId = options.projectId;
|
|
135
|
-
if (!projectId) {
|
|
136
|
-
throw new Error('GitLab requires --project');
|
|
137
|
-
}
|
|
138
|
-
const mrIid = parseNumber(options.mrIid, 'MR IID (--mr)');
|
|
139
|
-
const gitlabClient = createGitLabClient();
|
|
140
|
-
const platformClient = new GitLabPlatformAdapter(gitlabClient);
|
|
141
|
-
const pr = await platformClient.getPullRequest(projectId, mrIid);
|
|
142
|
-
const allFiles = await platformClient.getChangedFiles(projectId, mrIid);
|
|
143
|
-
const filesWithDiffs = allFiles
|
|
144
|
-
.filter((file) => file.status !== 'removed')
|
|
145
|
-
.map((file) => ({ filename: file.filename, patch: file.patch }));
|
|
146
|
-
const filteredNames = filterIgnoredFiles(filesWithDiffs.map((file) => file.filename), config);
|
|
147
|
-
const filteredFiles = filesWithDiffs.filter((file) => filteredNames.includes(file.filename));
|
|
148
|
-
const scopedFiles = ensureFileMatch(filteredFiles, options.file);
|
|
149
|
-
const label = `PR/MR #${pr.number}`;
|
|
150
|
-
const baseBranchResolution = resolveBaseBranch(options.baseBranch, pr.targetBranch);
|
|
151
|
-
const instructions = buildBaseInstructions(label, scopedFiles.map((file) => ({ filename: file.filename })), baseBranchResolution.diffCommand);
|
|
152
|
-
const payload = {
|
|
153
|
-
label,
|
|
154
|
-
files: scopedFiles,
|
|
155
|
-
instructions,
|
|
156
|
-
resolvedBaseBranch: baseBranchResolution.resolvedBaseBranch,
|
|
157
|
-
diffCommand: baseBranchResolution.diffCommand,
|
|
158
|
-
diffCommandSource: baseBranchResolution.source,
|
|
159
|
-
metadata: {
|
|
160
|
-
projectId,
|
|
161
|
-
prNumber: pr.number,
|
|
162
|
-
title: pr.title,
|
|
163
|
-
sourceBranch: pr.sourceBranch,
|
|
164
|
-
targetBranch: pr.targetBranch,
|
|
165
|
-
headSha: pr.headSha,
|
|
166
|
-
},
|
|
167
|
-
};
|
|
168
|
-
await writeOutput(payload, options.outputPath, options.jsonOutput, workingDir);
|
|
169
|
-
}
|
|
170
|
-
async function writeOutput(payload, outputPath, jsonOutput, workingDir = process.cwd()) {
|
|
171
|
-
let output = jsonOutput ? JSON.stringify(payload, null, 2) : payload.instructions;
|
|
172
|
-
if (!jsonOutput && payload.resolvedBaseBranch) {
|
|
173
|
-
output = `${output}\n\nBase branch resolved to: ${payload.resolvedBaseBranch} (${payload.diffCommandSource})`;
|
|
174
|
-
}
|
|
175
|
-
if (outputPath) {
|
|
176
|
-
const fullPath = resolve(workingDir, outputPath);
|
|
177
|
-
await writeFile(fullPath, output, 'utf-8');
|
|
178
|
-
console.log(chalk.green(`✓ Output written to ${outputPath}\n`));
|
|
179
|
-
}
|
|
180
|
-
if (jsonOutput ?? !outputPath) {
|
|
181
|
-
console.log(output);
|
|
182
|
-
}
|
|
183
|
-
}
|
|
184
|
-
//# sourceMappingURL=show-changes.js.map
|