diff --git a/DiffPalReviewDevV1/task.json b/DiffPalReviewDevV1/task.json index b2494ae..2e06231 100644 --- a/DiffPalReviewDevV1/task.json +++ b/DiffPalReviewDevV1/task.json @@ -10,7 +10,7 @@ "version": { "Major": 1, "Minor": 6, - "Patch": 7 + "Patch": 8 }, "minimumAgentVersion": "3.224.0", "instanceNameFormat": "DiffPal Review Dev", @@ -120,7 +120,7 @@ "balanced": "Balanced", "inline": "Inline" }, - "helpMarkDown": "Review feedback shape. Summary posts status and one PR summary. Balanced adds actionable inline threads. Inline uses a more permissive inline threshold." + "helpMarkDown": "Review feedback shape. Summary posts status and one PR summary. Balanced and Inline publish Azure threads for all findings; blocking findings stay active and non-blocking findings are closed immediately." }, { "name": "language", diff --git a/DiffPalReviewV1/task.json b/DiffPalReviewV1/task.json index 5db2c3d..68365ca 100644 --- a/DiffPalReviewV1/task.json +++ b/DiffPalReviewV1/task.json @@ -10,7 +10,7 @@ "version": { "Major": 1, "Minor": 6, - "Patch": 7 + "Patch": 8 }, "minimumAgentVersion": "3.224.0", "instanceNameFormat": "DiffPal Review", @@ -120,7 +120,7 @@ "balanced": "Balanced", "inline": "Inline" }, - "helpMarkDown": "Review feedback shape. Summary posts status and one PR summary. Balanced adds actionable inline threads. Inline uses a more permissive inline threshold." + "helpMarkDown": "Review feedback shape. Summary posts status and one PR summary. Balanced and Inline publish Azure threads for all findings; blocking findings stay active and non-blocking findings are closed immediately." }, { "name": "language", diff --git a/README.md b/README.md index 02afe9c..77222aa 100644 --- a/README.md +++ b/README.md @@ -20,6 +20,8 @@ VSIX packaging, and Marketplace release flow. The task installs `@diffpal/diffpal` by default and runs `diffpal review ado`. Bring the provider recipe you want to use; the Azure review flow stays the same. +By default it installs `@diffpal/diffpal@latest`, so pin `diffpalVersion` when +you need rollout-safe CLI behavior. ## Behavior @@ -35,6 +37,11 @@ defaults are ignored, and explicit invalid paths fail with task-level messages. Set `explain: true` to print the resolved PR id, branches, commits, merge-base, base/head, and redacted CLI arguments before the review starts. +With `feedback: balanced` or `feedback: inline`, DiffPal publishes Azure threads +for all findings. Blocking findings stay active; non-blocking findings are +published as closed immediately. Findings without canonical file/line mapping +fall back to non-file PR threads. + ## Examples ### Minimal PR validation diff --git a/package-lock.json b/package-lock.json index 24f3819..7bbcc39 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1,12 +1,12 @@ { "name": "@diffpal/azure-devops-extension", - "version": "0.1.29", + "version": "0.1.30", "lockfileVersion": 3, "requires": true, "packages": { "": { "name": "@diffpal/azure-devops-extension", - "version": "0.1.29", + "version": "0.1.30", "license": "MIT", "dependencies": { "azure-pipelines-task-lib": "5.2.10" diff --git a/package.json b/package.json index 89f6167..713ba08 100644 --- a/package.json +++ b/package.json @@ -34,5 +34,5 @@ "publish:prod": "npm run build \u0026\u0026 tfx extension publish --manifest-globs vss-extension.json", "smoke": "npm run build \u0026\u0026 node scripts/smoke-test.js" }, - "version": "0.1.29" + "version": "0.1.30" } diff --git a/vss-extension.dev.json b/vss-extension.dev.json index be0304d..32a83a6 100644 --- a/vss-extension.dev.json +++ b/vss-extension.dev.json @@ -46,5 +46,5 @@ "id": "Microsoft.VisualStudio.Services" } ], - "version": "0.1.29" + "version": "0.1.30" } diff --git a/vss-extension.json b/vss-extension.json index 7a781a7..b6ded06 100644 --- a/vss-extension.json +++ b/vss-extension.json @@ -46,5 +46,5 @@ "id": "Microsoft.VisualStudio.Services" } ], - "version": "0.1.29" + "version": "0.1.30" }