@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,282 +0,0 @@
|
|
|
1
|
-
import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
|
|
2
|
-
import { parseJsonFromAgentOutput, parseDescribeOutput } from './describe-parser.js';
|
|
3
|
-
import { mkdir, writeFile, rm } from 'fs/promises';
|
|
4
|
-
import { join } from 'path';
|
|
5
|
-
import { tmpdir } from 'os';
|
|
6
|
-
describe('describe-parser', () => {
|
|
7
|
-
describe('parseJsonFromAgentOutput', () => {
|
|
8
|
-
it('should parse JSON from code fence', () => {
|
|
9
|
-
const raw = '```json\n{"type": "feature", "title": "Test"}\n```';
|
|
10
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
11
|
-
expect(result).toEqual({ type: 'feature', title: 'Test' });
|
|
12
|
-
});
|
|
13
|
-
it('should parse JSON from code fence with extra whitespace', () => {
|
|
14
|
-
const raw = '```json\n\n {"type": "feature", "title": "Test"} \n\n```';
|
|
15
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
16
|
-
expect(result).toEqual({ type: 'feature', title: 'Test' });
|
|
17
|
-
});
|
|
18
|
-
it('should parse JSON from code fence with case-insensitive marker', () => {
|
|
19
|
-
const raw = '```JSON\n{"type": "feature", "title": "Test"}\n```';
|
|
20
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
21
|
-
expect(result).toEqual({ type: 'feature', title: 'Test' });
|
|
22
|
-
});
|
|
23
|
-
it('should parse embedded JSON from mixed text', () => {
|
|
24
|
-
const raw = 'Here is the output: {"outputType":"describe_output","outputPath":".drs/describe-output.json"} Done!';
|
|
25
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
26
|
-
expect(result).toEqual({
|
|
27
|
-
outputType: 'describe_output',
|
|
28
|
-
outputPath: '.drs/describe-output.json',
|
|
29
|
-
});
|
|
30
|
-
});
|
|
31
|
-
it('should parse pure JSON string', () => {
|
|
32
|
-
const raw = '{"type": "feature", "title": "Test"}';
|
|
33
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
34
|
-
expect(result).toEqual({ type: 'feature', title: 'Test' });
|
|
35
|
-
});
|
|
36
|
-
it('should handle nested JSON objects', () => {
|
|
37
|
-
const raw = `{"type": "feature", "title": "Test"}`;
|
|
38
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
39
|
-
expect(result).toEqual({
|
|
40
|
-
type: 'feature',
|
|
41
|
-
title: 'Test',
|
|
42
|
-
});
|
|
43
|
-
});
|
|
44
|
-
it('should handle JSON with arrays', () => {
|
|
45
|
-
const raw = '{"items": [1, 2, 3]}';
|
|
46
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
47
|
-
expect(result).toEqual({
|
|
48
|
-
items: [1, 2, 3],
|
|
49
|
-
});
|
|
50
|
-
});
|
|
51
|
-
it('should handle JSON with escaped quotes', () => {
|
|
52
|
-
const raw = '{"message": "This is a \\"quoted\\" string"}';
|
|
53
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
54
|
-
expect(result).toEqual({
|
|
55
|
-
message: 'This is a "quoted" string',
|
|
56
|
-
});
|
|
57
|
-
});
|
|
58
|
-
it('should prefer last JSON candidate when multiple exist', () => {
|
|
59
|
-
const raw = 'First: {"id": 1} Second: {"id": 2}';
|
|
60
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
61
|
-
// Should return the last valid JSON found
|
|
62
|
-
expect(result).toEqual({ id: 2 });
|
|
63
|
-
});
|
|
64
|
-
it('should prefer code fence over embedded JSON', () => {
|
|
65
|
-
const raw = `
|
|
66
|
-
{"id": 1}
|
|
67
|
-
\`\`\`json
|
|
68
|
-
{"id": 2}
|
|
69
|
-
\`\`\`
|
|
70
|
-
`;
|
|
71
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
72
|
-
expect(result).toEqual({ id: 2 });
|
|
73
|
-
});
|
|
74
|
-
it('should throw error for invalid JSON', () => {
|
|
75
|
-
const raw = 'This is not JSON at all';
|
|
76
|
-
expect(() => parseJsonFromAgentOutput(raw)).toThrow();
|
|
77
|
-
});
|
|
78
|
-
it('should throw error for incomplete JSON', () => {
|
|
79
|
-
const raw = '{"type": "feature", "title":';
|
|
80
|
-
expect(() => parseJsonFromAgentOutput(raw)).toThrow();
|
|
81
|
-
});
|
|
82
|
-
it('should handle empty string', () => {
|
|
83
|
-
const raw = '';
|
|
84
|
-
expect(() => parseJsonFromAgentOutput(raw)).toThrow();
|
|
85
|
-
});
|
|
86
|
-
it('should handle JSON with null values', () => {
|
|
87
|
-
const raw = '{"value": null, "name": "test"}';
|
|
88
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
89
|
-
expect(result).toEqual({ value: null, name: 'test' });
|
|
90
|
-
});
|
|
91
|
-
it('should handle JSON with boolean values', () => {
|
|
92
|
-
const raw = '{"enabled": true, "disabled": false}';
|
|
93
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
94
|
-
expect(result).toEqual({ enabled: true, disabled: false });
|
|
95
|
-
});
|
|
96
|
-
it('should handle JSON with numbers', () => {
|
|
97
|
-
const raw = '{"integer": 42, "float": 3.14, "negative": -10}';
|
|
98
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
99
|
-
expect(result).toEqual({ integer: 42, float: 3.14, negative: -10 });
|
|
100
|
-
});
|
|
101
|
-
it('should handle markdown text with embedded JSON', () => {
|
|
102
|
-
const raw = `
|
|
103
|
-
# Title
|
|
104
|
-
|
|
105
|
-
Here is some explanation text.
|
|
106
|
-
|
|
107
|
-
{"outputType":"describe_output","outputPath":".drs/output.json"}
|
|
108
|
-
|
|
109
|
-
More text after.
|
|
110
|
-
`;
|
|
111
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
112
|
-
expect(result).toEqual({
|
|
113
|
-
outputType: 'describe_output',
|
|
114
|
-
outputPath: '.drs/output.json',
|
|
115
|
-
});
|
|
116
|
-
});
|
|
117
|
-
it('should handle JSON with escaped characters', () => {
|
|
118
|
-
const raw = '{"pattern": "function() with \\"quotes\\""}';
|
|
119
|
-
const result = parseJsonFromAgentOutput(raw);
|
|
120
|
-
expect(result).toEqual({
|
|
121
|
-
pattern: 'function() with "quotes"',
|
|
122
|
-
});
|
|
123
|
-
});
|
|
124
|
-
});
|
|
125
|
-
describe('parseDescribeOutput', () => {
|
|
126
|
-
let testDir;
|
|
127
|
-
beforeEach(async () => {
|
|
128
|
-
// Create a unique temporary directory for each test
|
|
129
|
-
testDir = join(tmpdir(), `drs-test-${Date.now()}-${Math.random().toString(36).slice(2)}`);
|
|
130
|
-
await mkdir(testDir, { recursive: true });
|
|
131
|
-
await mkdir(join(testDir, '.drs'), { recursive: true });
|
|
132
|
-
});
|
|
133
|
-
afterEach(async () => {
|
|
134
|
-
// Clean up test directory
|
|
135
|
-
try {
|
|
136
|
-
await rm(testDir, { recursive: true, force: true });
|
|
137
|
-
}
|
|
138
|
-
catch {
|
|
139
|
-
// Ignore cleanup errors
|
|
140
|
-
}
|
|
141
|
-
});
|
|
142
|
-
it('should read describe output from default path', async () => {
|
|
143
|
-
const describeData = {
|
|
144
|
-
type: 'feature',
|
|
145
|
-
title: 'Add new feature',
|
|
146
|
-
summary: ['Added authentication', 'Updated database schema'],
|
|
147
|
-
};
|
|
148
|
-
await writeFile(join(testDir, '.drs/describe-output.json'), JSON.stringify(describeData));
|
|
149
|
-
const result = await parseDescribeOutput(testDir, false);
|
|
150
|
-
expect(result).toEqual(describeData);
|
|
151
|
-
});
|
|
152
|
-
it('should read describe output from pointer path', async () => {
|
|
153
|
-
const describeData = {
|
|
154
|
-
type: 'bugfix',
|
|
155
|
-
title: 'Fix authentication bug',
|
|
156
|
-
summary: ['Fixed login issue'],
|
|
157
|
-
};
|
|
158
|
-
await writeFile(join(testDir, '.drs/custom-output.json'), JSON.stringify(describeData));
|
|
159
|
-
const rawOutput = JSON.stringify({
|
|
160
|
-
outputType: 'describe_output',
|
|
161
|
-
outputPath: '.drs/custom-output.json',
|
|
162
|
-
});
|
|
163
|
-
const result = await parseDescribeOutput(testDir, false, rawOutput);
|
|
164
|
-
expect(result).toEqual(describeData);
|
|
165
|
-
});
|
|
166
|
-
it('should fall back to default path if pointer path not found', async () => {
|
|
167
|
-
const describeData = {
|
|
168
|
-
type: 'refactor',
|
|
169
|
-
title: 'Refactor code',
|
|
170
|
-
summary: ['Improved code quality'],
|
|
171
|
-
};
|
|
172
|
-
await writeFile(join(testDir, '.drs/describe-output.json'), JSON.stringify(describeData));
|
|
173
|
-
const rawOutput = JSON.stringify({
|
|
174
|
-
outputType: 'describe_output',
|
|
175
|
-
outputPath: '.drs/nonexistent.json',
|
|
176
|
-
});
|
|
177
|
-
const result = await parseDescribeOutput(testDir, false, rawOutput);
|
|
178
|
-
expect(result).toEqual(describeData);
|
|
179
|
-
});
|
|
180
|
-
it('should throw error if no describe output found', async () => {
|
|
181
|
-
await expect(parseDescribeOutput(testDir, false)).rejects.toThrow('Describe output file not found');
|
|
182
|
-
});
|
|
183
|
-
it('should throw error for invalid output type in pointer', async () => {
|
|
184
|
-
const rawOutput = JSON.stringify({
|
|
185
|
-
outputType: 'invalid_type',
|
|
186
|
-
outputPath: '.drs/output.json',
|
|
187
|
-
});
|
|
188
|
-
await expect(parseDescribeOutput(testDir, false, rawOutput)).rejects.toThrow('Unexpected output type for describe output: invalid_type');
|
|
189
|
-
});
|
|
190
|
-
it('should handle pointer with only outputType field', async () => {
|
|
191
|
-
const describeData = {
|
|
192
|
-
type: 'docs',
|
|
193
|
-
title: 'Update documentation',
|
|
194
|
-
};
|
|
195
|
-
await writeFile(join(testDir, '.drs/describe-output.json'), JSON.stringify(describeData));
|
|
196
|
-
const rawOutput = JSON.stringify({
|
|
197
|
-
outputType: 'describe_output',
|
|
198
|
-
});
|
|
199
|
-
const result = await parseDescribeOutput(testDir, false, rawOutput);
|
|
200
|
-
expect(result).toEqual(describeData);
|
|
201
|
-
});
|
|
202
|
-
it('should log debug messages when debug=true', async () => {
|
|
203
|
-
const logSpy = vi.spyOn(console, 'log').mockImplementation(() => { });
|
|
204
|
-
const describeData = { type: 'test' };
|
|
205
|
-
await writeFile(join(testDir, '.drs/describe-output.json'), JSON.stringify(describeData));
|
|
206
|
-
await parseDescribeOutput(testDir, true);
|
|
207
|
-
expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('Describe output loaded from'));
|
|
208
|
-
logSpy.mockRestore();
|
|
209
|
-
});
|
|
210
|
-
it('should not log debug messages when debug=false', async () => {
|
|
211
|
-
const logSpy = vi.spyOn(console, 'log').mockImplementation(() => { });
|
|
212
|
-
const describeData = { type: 'test' };
|
|
213
|
-
await writeFile(join(testDir, '.drs/describe-output.json'), JSON.stringify(describeData));
|
|
214
|
-
await parseDescribeOutput(testDir, false);
|
|
215
|
-
expect(logSpy).not.toHaveBeenCalled();
|
|
216
|
-
logSpy.mockRestore();
|
|
217
|
-
});
|
|
218
|
-
it('should refuse to read outside working directory', async () => {
|
|
219
|
-
const rawOutput = JSON.stringify({
|
|
220
|
-
outputType: 'describe_output',
|
|
221
|
-
outputPath: '../../../etc/passwd',
|
|
222
|
-
});
|
|
223
|
-
await expect(parseDescribeOutput(testDir, false, rawOutput)).rejects.toThrow('Refusing to read outside working directory');
|
|
224
|
-
});
|
|
225
|
-
it('should handle absolute paths within working directory', async () => {
|
|
226
|
-
const describeData = { type: 'feature' };
|
|
227
|
-
const outputPath = join(testDir, '.drs/describe-output.json');
|
|
228
|
-
await writeFile(outputPath, JSON.stringify(describeData));
|
|
229
|
-
const rawOutput = JSON.stringify({
|
|
230
|
-
outputType: 'describe_output',
|
|
231
|
-
outputPath: outputPath,
|
|
232
|
-
});
|
|
233
|
-
const result = await parseDescribeOutput(testDir, false, rawOutput);
|
|
234
|
-
expect(result).toEqual(describeData);
|
|
235
|
-
});
|
|
236
|
-
it('should handle invalid JSON in describe output file', async () => {
|
|
237
|
-
await writeFile(join(testDir, '.drs/describe-output.json'), 'not valid json');
|
|
238
|
-
await expect(parseDescribeOutput(testDir, false)).rejects.toThrow();
|
|
239
|
-
});
|
|
240
|
-
it('should use process.cwd() as default working directory', async () => {
|
|
241
|
-
// This test verifies the default parameter works
|
|
242
|
-
// We can't easily test the actual cwd behavior in isolation
|
|
243
|
-
const rawOutput = 'invalid pointer';
|
|
244
|
-
// Should attempt to use cwd and likely fail (unless file exists there)
|
|
245
|
-
await expect(parseDescribeOutput(undefined, false, rawOutput)).rejects.toThrow();
|
|
246
|
-
});
|
|
247
|
-
it('should handle pointer with invalid JSON format', async () => {
|
|
248
|
-
const logSpy = vi.spyOn(console, 'log').mockImplementation(() => { });
|
|
249
|
-
const describeData = { type: 'test' };
|
|
250
|
-
await writeFile(join(testDir, '.drs/describe-output.json'), JSON.stringify(describeData));
|
|
251
|
-
const rawOutput = 'not valid json at all';
|
|
252
|
-
// Should fall back to default path since pointer parsing fails
|
|
253
|
-
const result = await parseDescribeOutput(testDir, true, rawOutput);
|
|
254
|
-
expect(result).toEqual(describeData);
|
|
255
|
-
expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('Describe output pointer parse failed'));
|
|
256
|
-
logSpy.mockRestore();
|
|
257
|
-
});
|
|
258
|
-
it('should handle complex describe output structure', async () => {
|
|
259
|
-
const describeData = {
|
|
260
|
-
type: 'feature',
|
|
261
|
-
title: 'Complex feature',
|
|
262
|
-
summary: ['Point 1', 'Point 2', 'Point 3'],
|
|
263
|
-
walkthrough: [
|
|
264
|
-
{
|
|
265
|
-
file: 'src/app.ts',
|
|
266
|
-
changeType: 'modified',
|
|
267
|
-
semanticLabel: 'feature',
|
|
268
|
-
title: 'Added new function',
|
|
269
|
-
changes: ['Added authentication', 'Updated routes'],
|
|
270
|
-
significance: 'major',
|
|
271
|
-
},
|
|
272
|
-
],
|
|
273
|
-
labels: ['enhancement', 'security'],
|
|
274
|
-
recommendations: ['Add tests', 'Update docs'],
|
|
275
|
-
};
|
|
276
|
-
await writeFile(join(testDir, '.drs/describe-output.json'), JSON.stringify(describeData));
|
|
277
|
-
const result = await parseDescribeOutput(testDir, false);
|
|
278
|
-
expect(result).toEqual(describeData);
|
|
279
|
-
});
|
|
280
|
-
});
|
|
281
|
-
});
|
|
282
|
-
//# sourceMappingURL=describe-parser.test.js.map
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"describe-parser.test.js","sourceRoot":"","sources":["../../src/lib/describe-parser.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AACzE,OAAO,EAAE,wBAAwB,EAAE,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AACrF,OAAO,EAAE,KAAK,EAAE,SAAS,EAAE,EAAE,EAAE,MAAM,aAAa,CAAC;AACnD,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAC;AAC5B,OAAO,EAAE,MAAM,EAAE,MAAM,IAAI,CAAC;AAE5B,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAC/B,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;QACxC,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;YAC3C,MAAM,GAAG,GAAG,oDAAoD,CAAC;YACjE,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAC;QAC7D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yDAAyD,EAAE,GAAG,EAAE;YACjE,MAAM,GAAG,GAAG,4DAA4D,CAAC;YACzE,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAC;QAC7D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;YACxE,MAAM,GAAG,GAAG,oDAAoD,CAAC;YACjE,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAC;QAC7D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;YACpD,MAAM,GAAG,GACP,qGAAqG,CAAC;YACxG,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;gBACrB,UAAU,EAAE,iBAAiB;gBAC7B,UAAU,EAAE,2BAA2B;aACxC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+BAA+B,EAAE,GAAG,EAAE;YACvC,MAAM,GAAG,GAAG,sCAAsC,CAAC;YACnD,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAC;QAC7D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;YAC3C,MAAM,GAAG,GAAG,sCAAsC,CAAC;YACnD,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;gBACrB,IAAI,EAAE,SAAS;gBACf,KAAK,EAAE,MAAM;aACd,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;YACxC,MAAM,GAAG,GAAG,sBAAsB,CAAC;YACnC,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;gBACrB,KAAK,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;aACjB,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,GAAG,GAAG,8CAA8C,CAAC;YAC3D,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;gBACrB,OAAO,EAAE,2BAA2B;aACrC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;YAC/D,MAAM,GAAG,GAAG,oCAAoC,CAAC;YACjD,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,0CAA0C;YAC1C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC;QACpC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;YACrD,MAAM,GAAG,GAAG;;;;;OAKX,CAAC;YACF,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC;QACpC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;YAC7C,MAAM,GAAG,GAAG,yBAAyB,CAAC;YACtC,MAAM,CAAC,GAAG,EAAE,CAAC,wBAAwB,CAAC,GAAG,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;QACxD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,GAAG,GAAG,8BAA8B,CAAC;YAC3C,MAAM,CAAC,GAAG,EAAE,CAAC,wBAAwB,CAAC,GAAG,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;QACxD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4BAA4B,EAAE,GAAG,EAAE;YACpC,MAAM,GAAG,GAAG,EAAE,CAAC;YACf,MAAM,CAAC,GAAG,EAAE,CAAC,wBAAwB,CAAC,GAAG,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;QACxD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;YAC7C,MAAM,GAAG,GAAG,iCAAiC,CAAC;YAC9C,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC,CAAC;QACxD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,GAAG,GAAG,sCAAsC,CAAC;YACnD,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,KAAK,EAAE,CAAC,CAAC;QAC7D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;YACzC,MAAM,GAAG,GAAG,iDAAiD,CAAC;YAC9D,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QACtE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;YACxD,MAAM,GAAG,GAAG;;;;;;;;OAQX,CAAC;YACF,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;gBACrB,UAAU,EAAE,iBAAiB;gBAC7B,UAAU,EAAE,kBAAkB;aAC/B,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;YACpD,MAAM,GAAG,GAAG,6CAA6C,CAAC;YAC1D,MAAM,MAAM,GAAG,wBAAwB,CAAC,GAAG,CAAC,CAAC;YAE7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;gBACrB,OAAO,EAAE,0BAA0B;aACpC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;QACnC,IAAI,OAAe,CAAC;QAEpB,UAAU,CAAC,KAAK,IAAI,EAAE;YACpB,oDAAoD;YACpD,OAAO,GAAG,IAAI,CAAC,MAAM,EAAE,EAAE,YAAY,IAAI,CAAC,GAAG,EAAE,IAAI,IAAI,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;YAC1F,MAAM,KAAK,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;YAC1C,MAAM,KAAK,CAAC,IAAI,CAAC,OAAO,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC1D,CAAC,CAAC,CAAC;QAEH,SAAS,CAAC,KAAK,IAAI,EAAE;YACnB,0BAA0B;YAC1B,IAAI,CAAC;gBACH,MAAM,EAAE,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;YACtD,CAAC;YAAC,MAAM,CAAC;gBACP,wBAAwB;YAC1B,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;YAC7D,MAAM,YAAY,GAAG;gBACnB,IAAI,EAAE,SAAS;gBACf,KAAK,EAAE,iBAAiB;gBACxB,OAAO,EAAE,CAAC,sBAAsB,EAAE,yBAAyB,CAAC;aAC7D,CAAC;YAEF,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC;YAE1F,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC;YACzD,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;YAC7D,MAAM,YAAY,GAAG;gBACnB,IAAI,EAAE,QAAQ;gBACd,KAAK,EAAE,wBAAwB;gBAC/B,OAAO,EAAE,CAAC,mBAAmB,CAAC;aAC/B,CAAC;YAEF,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,yBAAyB,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC;YAExF,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,CAAC;gBAC/B,UAAU,EAAE,iBAAiB;gBAC7B,UAAU,EAAE,yBAAyB;aACtC,CAAC,CAAC;YAEH,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,KAAK,EAAE,SAAS,CAAC,CAAC;YACpE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;YAC1E,MAAM,YAAY,GAAG;gBACnB,IAAI,EAAE,UAAU;gBAChB,KAAK,EAAE,eAAe;gBACtB,OAAO,EAAE,CAAC,uBAAuB,CAAC;aACnC,CAAC;YAEF,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC;YAE1F,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,CAAC;gBAC/B,UAAU,EAAE,iBAAiB;gBAC7B,UAAU,EAAE,uBAAuB;aACpC,CAAC,CAAC;YAEH,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,KAAK,EAAE,SAAS,CAAC,CAAC;YACpE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;YAC9D,MAAM,MAAM,CAAC,mBAAmB,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC/D,gCAAgC,CACjC,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;YACrE,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,CAAC;gBAC/B,UAAU,EAAE,cAAc;gBAC1B,UAAU,EAAE,kBAAkB;aAC/B,CAAC,CAAC;YAEH,MAAM,MAAM,CAAC,mBAAmB,CAAC,OAAO,EAAE,KAAK,EAAE,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC1E,0DAA0D,CAC3D,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;YAChE,MAAM,YAAY,GAAG;gBACnB,IAAI,EAAE,MAAM;gBACZ,KAAK,EAAE,sBAAsB;aAC9B,CAAC;YAEF,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC;YAE1F,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,CAAC;gBAC/B,UAAU,EAAE,iBAAiB;aAC9B,CAAC,CAAC;YAEH,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,KAAK,EAAE,SAAS,CAAC,CAAC;YACpE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;YACzD,MAAM,MAAM,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;YAErE,MAAM,YAAY,GAAG,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC;YACtC,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC;YAE1F,MAAM,mBAAmB,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC;YAEzC,MAAM,CAAC,MAAM,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,gBAAgB,CAAC,6BAA6B,CAAC,CAAC,CAAC;YAE5F,MAAM,CAAC,WAAW,EAAE,CAAC;QACvB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;YAC9D,MAAM,MAAM,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;YAErE,MAAM,YAAY,GAAG,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC;YACtC,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC;YAE1F,MAAM,mBAAmB,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC;YAE1C,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YAEtC,MAAM,CAAC,WAAW,EAAE,CAAC;QACvB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;YAC/D,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,CAAC;gBAC/B,UAAU,EAAE,iBAAiB;gBAC7B,UAAU,EAAE,qBAAqB;aAClC,CAAC,CAAC;YAEH,MAAM,MAAM,CAAC,mBAAmB,CAAC,OAAO,EAAE,KAAK,EAAE,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC1E,4CAA4C,CAC7C,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;YACrE,MAAM,YAAY,GAAG,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC;YACzC,MAAM,UAAU,GAAG,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,CAAC;YAE9D,MAAM,SAAS,CAAC,UAAU,EAAE,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC;YAE1D,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,CAAC;gBAC/B,UAAU,EAAE,iBAAiB;gBAC7B,UAAU,EAAE,UAAU;aACvB,CAAC,CAAC;YAEH,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,KAAK,EAAE,SAAS,CAAC,CAAC;YACpE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;YAClE,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,EAAE,gBAAgB,CAAC,CAAC;YAE9E,MAAM,MAAM,CAAC,mBAAmB,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;QACtE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;YACrE,iDAAiD;YACjD,4DAA4D;YAC5D,MAAM,SAAS,GAAG,iBAAiB,CAAC;YAEpC,uEAAuE;YACvE,MAAM,MAAM,CAAC,mBAAmB,CAAC,SAAS,EAAE,KAAK,EAAE,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;QACnF,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;YAC9D,MAAM,MAAM,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;YAErE,MAAM,YAAY,GAAG,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC;YACtC,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC;YAE1F,MAAM,SAAS,GAAG,uBAAuB,CAAC;YAE1C,+DAA+D;YAC/D,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,IAAI,EAAE,SAAS,CAAC,CAAC;YACnE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;YAErC,MAAM,CAAC,MAAM,CAAC,CAAC,oBAAoB,CACjC,MAAM,CAAC,gBAAgB,CAAC,sCAAsC,CAAC,CAChE,CAAC;YAEF,MAAM,CAAC,WAAW,EAAE,CAAC;QACvB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;YAC/D,MAAM,YAAY,GAAG;gBACnB,IAAI,EAAE,SAAS;gBACf,KAAK,EAAE,iBAAiB;gBACxB,OAAO,EAAE,CAAC,SAAS,EAAE,SAAS,EAAE,SAAS,CAAC;gBAC1C,WAAW,EAAE;oBACX;wBACE,IAAI,EAAE,YAAY;wBAClB,UAAU,EAAE,UAAU;wBACtB,aAAa,EAAE,SAAS;wBACxB,KAAK,EAAE,oBAAoB;wBAC3B,OAAO,EAAE,CAAC,sBAAsB,EAAE,gBAAgB,CAAC;wBACnD,YAAY,EAAE,OAAO;qBACtB;iBACF;gBACD,MAAM,EAAE,CAAC,aAAa,EAAE,UAAU,CAAC;gBACnC,eAAe,EAAE,CAAC,WAAW,EAAE,aAAa,CAAC;aAC9C,CAAC;YAEF,MAAM,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC;YAE1F,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC;YACzD,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"description-executor.test.d.ts","sourceRoot":"","sources":["../../src/lib/description-executor.test.ts"],"names":[],"mappings":""}
|
|
@@ -1,128 +0,0 @@
|
|
|
1
|
-
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
|
|
2
|
-
import { runDescribeIfEnabled } from './description-executor.js';
|
|
3
|
-
vi.mock('./describe-core.js', () => ({
|
|
4
|
-
buildDescribeInstructions: vi.fn(() => 'describe instructions'),
|
|
5
|
-
}));
|
|
6
|
-
vi.mock('./context-compression.js', () => ({
|
|
7
|
-
prepareDiffsForAgent: vi.fn((files) => ({ files, generated: [] })),
|
|
8
|
-
formatCompressionSummary: vi.fn(() => undefined),
|
|
9
|
-
resolveCompressionBudget: vi.fn((_contextWindow, options) => options ?? {}),
|
|
10
|
-
}));
|
|
11
|
-
vi.mock('./review-orchestrator.js', () => ({
|
|
12
|
-
filterIgnoredFiles: vi.fn((files) => files),
|
|
13
|
-
}));
|
|
14
|
-
vi.mock('./context-loader.js', () => ({
|
|
15
|
-
loadGlobalContext: vi.fn(() => null),
|
|
16
|
-
}));
|
|
17
|
-
vi.mock('./describe-parser.js', () => ({
|
|
18
|
-
parseDescribeOutput: vi.fn().mockResolvedValue({
|
|
19
|
-
type: 'refactor',
|
|
20
|
-
title: 'Pi migration',
|
|
21
|
-
summary: ['Switches runtime to Pi in-process mode'],
|
|
22
|
-
}),
|
|
23
|
-
}));
|
|
24
|
-
vi.mock('./description-formatter.js', () => ({
|
|
25
|
-
displayDescription: vi.fn(),
|
|
26
|
-
normalizeDescription: vi.fn((description) => description),
|
|
27
|
-
postDescription: vi.fn().mockResolvedValue(undefined),
|
|
28
|
-
}));
|
|
29
|
-
describe('description-executor', () => {
|
|
30
|
-
let runtimeClient;
|
|
31
|
-
let platformClient;
|
|
32
|
-
let config;
|
|
33
|
-
let pr;
|
|
34
|
-
let files;
|
|
35
|
-
let consoleLogSpy;
|
|
36
|
-
beforeEach(() => {
|
|
37
|
-
consoleLogSpy = vi.spyOn(console, 'log').mockImplementation(() => { });
|
|
38
|
-
runtimeClient = {
|
|
39
|
-
createSession: vi.fn().mockResolvedValue({ id: 'session-1' }),
|
|
40
|
-
streamMessages: vi.fn(async function* () {
|
|
41
|
-
yield {
|
|
42
|
-
id: 'message-1',
|
|
43
|
-
role: 'assistant',
|
|
44
|
-
content: '{"outputType":"describe_output","outputPath":".drs/describe-output.json"}',
|
|
45
|
-
timestamp: new Date(),
|
|
46
|
-
};
|
|
47
|
-
}),
|
|
48
|
-
getMinContextWindow: vi.fn(() => undefined),
|
|
49
|
-
getModelContextWindow: vi.fn(() => undefined),
|
|
50
|
-
};
|
|
51
|
-
platformClient = {};
|
|
52
|
-
config = {
|
|
53
|
-
review: {
|
|
54
|
-
agents: ['security'],
|
|
55
|
-
ignorePatterns: [],
|
|
56
|
-
mode: 'multi-agent',
|
|
57
|
-
},
|
|
58
|
-
contextCompression: {
|
|
59
|
-
enabled: true,
|
|
60
|
-
},
|
|
61
|
-
};
|
|
62
|
-
pr = {
|
|
63
|
-
number: 84,
|
|
64
|
-
title: 'Pi migration',
|
|
65
|
-
author: 'manojlds',
|
|
66
|
-
sourceBranch: 'ralph/pi-migration',
|
|
67
|
-
targetBranch: 'pi-migration-base',
|
|
68
|
-
headSha: 'abc123',
|
|
69
|
-
platformData: {},
|
|
70
|
-
};
|
|
71
|
-
files = [
|
|
72
|
-
{
|
|
73
|
-
filename: 'src/lib/review-core.ts',
|
|
74
|
-
patch: '+ new line',
|
|
75
|
-
},
|
|
76
|
-
];
|
|
77
|
-
});
|
|
78
|
-
afterEach(() => {
|
|
79
|
-
consoleLogSpy.mockRestore();
|
|
80
|
-
vi.clearAllMocks();
|
|
81
|
-
});
|
|
82
|
-
it('suppresses terminal description output when posting is enabled', async () => {
|
|
83
|
-
const { displayDescription, postDescription } = await import('./description-formatter.js');
|
|
84
|
-
await runDescribeIfEnabled(runtimeClient, config, platformClient, 'manojlds/drs', pr, files, true, process.cwd(), false);
|
|
85
|
-
expect(displayDescription).not.toHaveBeenCalled();
|
|
86
|
-
expect(postDescription).toHaveBeenCalledWith(platformClient, 'manojlds/drs', 84, expect.objectContaining({ title: 'Pi migration' }), 'PR', expect.objectContaining({
|
|
87
|
-
total: expect.objectContaining({
|
|
88
|
-
input: expect.any(Number),
|
|
89
|
-
output: expect.any(Number),
|
|
90
|
-
totalTokens: expect.any(Number),
|
|
91
|
-
}),
|
|
92
|
-
}));
|
|
93
|
-
});
|
|
94
|
-
it('prints description to terminal when posting is disabled', async () => {
|
|
95
|
-
const { displayDescription, postDescription } = await import('./description-formatter.js');
|
|
96
|
-
await runDescribeIfEnabled(runtimeClient, config, platformClient, 'manojlds/drs', pr, files, false, process.cwd(), false);
|
|
97
|
-
expect(displayDescription).toHaveBeenCalledWith(expect.objectContaining({ title: 'Pi migration' }), 'PR', expect.objectContaining({
|
|
98
|
-
total: expect.objectContaining({
|
|
99
|
-
totalTokens: expect.any(Number),
|
|
100
|
-
}),
|
|
101
|
-
}));
|
|
102
|
-
expect(postDescription).not.toHaveBeenCalled();
|
|
103
|
-
});
|
|
104
|
-
it('uses only describe model IDs when sizing compression budget', async () => {
|
|
105
|
-
const configWithExplicitDescribeModel = {
|
|
106
|
-
...config,
|
|
107
|
-
review: {
|
|
108
|
-
...config.review,
|
|
109
|
-
default: {
|
|
110
|
-
model: 'provider/default-8k',
|
|
111
|
-
},
|
|
112
|
-
},
|
|
113
|
-
describe: {
|
|
114
|
-
model: 'provider/describe-200k',
|
|
115
|
-
},
|
|
116
|
-
};
|
|
117
|
-
await runDescribeIfEnabled(runtimeClient, configWithExplicitDescribeModel, platformClient, 'manojlds/drs', pr, files, false, process.cwd(), false);
|
|
118
|
-
expect(runtimeClient.getMinContextWindow).toHaveBeenCalledWith(['provider/describe-200k']);
|
|
119
|
-
});
|
|
120
|
-
it('falls back to describe context window when model-specific window is unavailable', async () => {
|
|
121
|
-
runtimeClient.getMinContextWindow.mockReturnValue(1000);
|
|
122
|
-
runtimeClient.getModelContextWindow.mockReturnValue(undefined);
|
|
123
|
-
await runDescribeIfEnabled(runtimeClient, config, platformClient, 'manojlds/drs', pr, files, false, process.cwd(), false);
|
|
124
|
-
const output = consoleLogSpy.mock.calls.map((call) => String(call[0])).join('\n');
|
|
125
|
-
expect(output).toContain('% of 1,000 context');
|
|
126
|
-
});
|
|
127
|
-
});
|
|
128
|
-
//# sourceMappingURL=description-executor.test.js.map
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"description-executor.test.js","sourceRoot":"","sources":["../../src/lib/description-executor.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AACzE,OAAO,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAC;AAMjE,EAAE,CAAC,IAAI,CAAC,oBAAoB,EAAE,GAAG,EAAE,CAAC,CAAC;IACnC,yBAAyB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,uBAAuB,CAAC;CAChE,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,0BAA0B,EAAE,GAAG,EAAE,CAAC,CAAC;IACzC,oBAAoB,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,KAAqB,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,SAAS,EAAE,EAAE,EAAE,CAAC,CAAC;IAClF,wBAAwB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC;IAChD,wBAAwB,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,cAAuB,EAAE,OAAgB,EAAE,EAAE,CAAC,OAAO,IAAI,EAAE,CAAC;CAC9F,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,0BAA0B,EAAE,GAAG,EAAE,CAAC,CAAC;IACzC,kBAAkB,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,KAAe,EAAE,EAAE,CAAC,KAAK,CAAC;CACtD,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,qBAAqB,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,iBAAiB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC;CACrC,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,sBAAsB,EAAE,GAAG,EAAE,CAAC,CAAC;IACrC,mBAAmB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC;QAC7C,IAAI,EAAE,UAAU;QAChB,KAAK,EAAE,cAAc;QACrB,OAAO,EAAE,CAAC,wCAAwC,CAAC;KACpD,CAAC;CACH,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,4BAA4B,EAAE,GAAG,EAAE,CAAC,CAAC;IAC3C,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC3B,oBAAoB,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,WAAoB,EAAE,EAAE,CAAC,WAAW,CAAC;IAClE,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,SAAS,CAAC;CACtD,CAAC,CAAC,CAAC;AAEJ,QAAQ,CAAC,sBAAsB,EAAE,GAAG,EAAE;IACpC,IAAI,aAA4B,CAAC;IACjC,IAAI,cAA8B,CAAC;IACnC,IAAI,MAAiB,CAAC;IACtB,IAAI,EAAe,CAAC;IACpB,IAAI,KAAqB,CAAC;IAC1B,IAAI,aAAkB,CAAC;IAEvB,UAAU,CAAC,GAAG,EAAE;QACd,aAAa,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QAEtE,aAAa,GAAG;YACd,aAAa,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,EAAE,EAAE,WAAW,EAAE,CAAC;YAC7D,cAAc,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,SAAS,CAAC;gBACnC,MAAM;oBACJ,EAAE,EAAE,WAAW;oBACf,IAAI,EAAE,WAAW;oBACjB,OAAO,EAAE,2EAA2E;oBACpF,SAAS,EAAE,IAAI,IAAI,EAAE;iBACtB,CAAC;YACJ,CAAC,CAAC;YACF,mBAAmB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC;YAC3C,qBAAqB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC;SAClB,CAAC;QAE9B,cAAc,GAAG,EAA+B,CAAC;QAEjD,MAAM,GAAG;YACP,MAAM,EAAE;gBACN,MAAM,EAAE,CAAC,UAAU,CAAC;gBACpB,cAAc,EAAE,EAAE;gBAClB,IAAI,EAAE,aAAa;aACpB;YACD,kBAAkB,EAAE;gBAClB,OAAO,EAAE,IAAI;aACd;SACsB,CAAC;QAE1B,EAAE,GAAG;YACH,MAAM,EAAE,EAAE;YACV,KAAK,EAAE,cAAc;YACrB,MAAM,EAAE,UAAU;YAClB,YAAY,EAAE,oBAAoB;YAClC,YAAY,EAAE,mBAAmB;YACjC,OAAO,EAAE,QAAQ;YACjB,YAAY,EAAE,EAAE;SACjB,CAAC;QAEF,KAAK,GAAG;YACN;gBACE,QAAQ,EAAE,wBAAwB;gBAClC,KAAK,EAAE,YAAY;aACpB;SACF,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,aAAa,CAAC,WAAW,EAAE,CAAC;QAC5B,EAAE,CAAC,aAAa,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC9E,MAAM,EAAE,kBAAkB,EAAE,eAAe,EAAE,GAAG,MAAM,MAAM,CAAC,4BAA4B,CAAC,CAAC;QAE3F,MAAM,oBAAoB,CACxB,aAAa,EACb,MAAM,EACN,cAAc,EACd,cAAc,EACd,EAAE,EACF,KAAK,EACL,IAAI,EACJ,OAAO,CAAC,GAAG,EAAE,EACb,KAAK,CACN,CAAC;QAEF,MAAM,CAAC,kBAAkB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAClD,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAC1C,cAAc,EACd,cAAc,EACd,EAAE,EACF,MAAM,CAAC,gBAAgB,CAAC,EAAE,KAAK,EAAE,cAAc,EAAE,CAAC,EAClD,IAAI,EACJ,MAAM,CAAC,gBAAgB,CAAC;YACtB,KAAK,EAAE,MAAM,CAAC,gBAAgB,CAAC;gBAC7B,KAAK,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC;gBACzB,MAAM,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC;gBAC1B,WAAW,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC;aAChC,CAAC;SACH,CAAC,CACH,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,MAAM,EAAE,kBAAkB,EAAE,eAAe,EAAE,GAAG,MAAM,MAAM,CAAC,4BAA4B,CAAC,CAAC;QAE3F,MAAM,oBAAoB,CACxB,aAAa,EACb,MAAM,EACN,cAAc,EACd,cAAc,EACd,EAAE,EACF,KAAK,EACL,KAAK,EACL,OAAO,CAAC,GAAG,EAAE,EACb,KAAK,CACN,CAAC;QAEF,MAAM,CAAC,kBAAkB,CAAC,CAAC,oBAAoB,CAC7C,MAAM,CAAC,gBAAgB,CAAC,EAAE,KAAK,EAAE,cAAc,EAAE,CAAC,EAClD,IAAI,EACJ,MAAM,CAAC,gBAAgB,CAAC;YACtB,KAAK,EAAE,MAAM,CAAC,gBAAgB,CAAC;gBAC7B,WAAW,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC;aAChC,CAAC;SACH,CAAC,CACH,CAAC;QACF,MAAM,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;QAC3E,MAAM,+BAA+B,GAAG;YACtC,GAAG,MAAM;YACT,MAAM,EAAE;gBACN,GAAG,MAAM,CAAC,MAAM;gBAChB,OAAO,EAAE;oBACP,KAAK,EAAE,qBAAqB;iBAC7B;aACF;YACD,QAAQ,EAAE;gBACR,KAAK,EAAE,wBAAwB;aAChC;SACW,CAAC;QAEf,MAAM,oBAAoB,CACxB,aAAa,EACb,+BAA+B,EAC/B,cAAc,EACd,cAAc,EACd,EAAE,EACF,KAAK,EACL,KAAK,EACL,OAAO,CAAC,GAAG,EAAE,EACb,KAAK,CACN,CAAC;QAEF,MAAM,CAAC,aAAa,CAAC,mBAAmB,CAAC,CAAC,oBAAoB,CAAC,CAAC,wBAAwB,CAAC,CAAC,CAAC;IAC7F,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iFAAiF,EAAE,KAAK,IAAI,EAAE;QAC9F,aAAqB,CAAC,mBAAmB,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;QAChE,aAAqB,CAAC,qBAAqB,CAAC,eAAe,CAAC,SAAS,CAAC,CAAC;QAExE,MAAM,oBAAoB,CACxB,aAAa,EACb,MAAM,EACN,cAAc,EACd,cAAc,EACd,EAAE,EACF,KAAK,EACL,KAAK,EACL,OAAO,CAAC,GAAG,EAAE,EACb,KAAK,CACN,CAAC;QAEF,MAAM,MAAM,GAAG,aAAa,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,IAAe,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7F,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"description-formatter.test.d.ts","sourceRoot":"","sources":["../../src/lib/description-formatter.test.ts"],"names":[],"mappings":""}
|
|
@@ -1,57 +0,0 @@
|
|
|
1
|
-
import { describe, expect, it } from 'vitest';
|
|
2
|
-
import { formatDescriptionAsMarkdown } from './description-formatter.js';
|
|
3
|
-
describe('description-formatter', () => {
|
|
4
|
-
it('includes usage section in markdown when usage is provided', () => {
|
|
5
|
-
const description = {
|
|
6
|
-
type: 'feature',
|
|
7
|
-
title: 'Add usage visibility',
|
|
8
|
-
summary: ['Shows model usage and cost for describe output'],
|
|
9
|
-
walkthrough: [
|
|
10
|
-
{
|
|
11
|
-
file: 'src/lib/description-formatter.ts',
|
|
12
|
-
changeType: 'modified',
|
|
13
|
-
semanticLabel: 'observability',
|
|
14
|
-
title: 'Adds usage section rendering',
|
|
15
|
-
changes: ['Add expandable model usage details block'],
|
|
16
|
-
},
|
|
17
|
-
],
|
|
18
|
-
};
|
|
19
|
-
const usage = {
|
|
20
|
-
total: {
|
|
21
|
-
input: 1500,
|
|
22
|
-
output: 250,
|
|
23
|
-
cacheRead: 0,
|
|
24
|
-
cacheWrite: 0,
|
|
25
|
-
totalTokens: 1750,
|
|
26
|
-
cost: 0.0312,
|
|
27
|
-
},
|
|
28
|
-
agents: [
|
|
29
|
-
{
|
|
30
|
-
agentType: 'describe/pr-describer',
|
|
31
|
-
model: 'opencode/glm-5-free',
|
|
32
|
-
turns: 1,
|
|
33
|
-
success: true,
|
|
34
|
-
usage: {
|
|
35
|
-
input: 1500,
|
|
36
|
-
output: 250,
|
|
37
|
-
cacheRead: 0,
|
|
38
|
-
cacheWrite: 0,
|
|
39
|
-
totalTokens: 1750,
|
|
40
|
-
cost: 0.0312,
|
|
41
|
-
},
|
|
42
|
-
},
|
|
43
|
-
],
|
|
44
|
-
};
|
|
45
|
-
const markdown = formatDescriptionAsMarkdown(description, 'PR', usage);
|
|
46
|
-
expect(markdown).toContain('# 📋 PR Description Analysis');
|
|
47
|
-
expect(markdown).toContain('## 🧭 Change Summary');
|
|
48
|
-
expect(markdown).toContain('## 📌 Summary');
|
|
49
|
-
expect(markdown).toContain('## 📂 Changes Walkthrough');
|
|
50
|
-
expect(markdown).toContain('## 💰 Model Usage');
|
|
51
|
-
expect(markdown).toContain('View token and cost breakdown');
|
|
52
|
-
expect(markdown).toContain('describe/pr-describer');
|
|
53
|
-
expect(markdown).toContain('opencode/glm-5-free');
|
|
54
|
-
expect(markdown).toContain('$0.0312');
|
|
55
|
-
});
|
|
56
|
-
});
|
|
57
|
-
//# sourceMappingURL=description-formatter.test.js.map
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"description-formatter.test.js","sourceRoot":"","sources":["../../src/lib/description-formatter.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,2BAA2B,EAAoB,MAAM,4BAA4B,CAAC;AAG3F,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;IACrC,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACnE,MAAM,WAAW,GAAgB;YAC/B,IAAI,EAAE,SAAS;YACf,KAAK,EAAE,sBAAsB;YAC7B,OAAO,EAAE,CAAC,gDAAgD,CAAC;YAC3D,WAAW,EAAE;gBACX;oBACE,IAAI,EAAE,kCAAkC;oBACxC,UAAU,EAAE,UAAU;oBACtB,aAAa,EAAE,eAAe;oBAC9B,KAAK,EAAE,8BAA8B;oBACrC,OAAO,EAAE,CAAC,0CAA0C,CAAC;iBACtD;aACF;SACF,CAAC;QAEF,MAAM,KAAK,GAAuB;YAChC,KAAK,EAAE;gBACL,KAAK,EAAE,IAAI;gBACX,MAAM,EAAE,GAAG;gBACX,SAAS,EAAE,CAAC;gBACZ,UAAU,EAAE,CAAC;gBACb,WAAW,EAAE,IAAI;gBACjB,IAAI,EAAE,MAAM;aACb;YACD,MAAM,EAAE;gBACN;oBACE,SAAS,EAAE,uBAAuB;oBAClC,KAAK,EAAE,qBAAqB;oBAC5B,KAAK,EAAE,CAAC;oBACR,OAAO,EAAE,IAAI;oBACb,KAAK,EAAE;wBACL,KAAK,EAAE,IAAI;wBACX,MAAM,EAAE,GAAG;wBACX,SAAS,EAAE,CAAC;wBACZ,UAAU,EAAE,CAAC;wBACb,WAAW,EAAE,IAAI;wBACjB,IAAI,EAAE,MAAM;qBACb;iBACF;aACF;SACF,CAAC;QAEF,MAAM,QAAQ,GAAG,2BAA2B,CAAC,WAAW,EAAE,IAAI,EAAE,KAAK,CAAC,CAAC;QAEvE,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,8BAA8B,CAAC,CAAC;QAC3D,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;QACnD,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QAC5C,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,2BAA2B,CAAC,CAAC;QACxD,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;QAChD,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,+BAA+B,CAAC,CAAC;QAC5D,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,uBAAuB,CAAC,CAAC;QACpD,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,qBAAqB,CAAC,CAAC;QAClD,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"diff-parser.test.d.ts","sourceRoot":"","sources":["../../src/lib/diff-parser.test.ts"],"names":[],"mappings":""}
|