Skip to content

Bump version to 1.20.0 - #444

Merged
erikdarlingdata merged 1 commit into
devfrom
release/v1.20.0
Aug 21, 2026
Merged

Bump version to 1.20.0#444
erikdarlingdata merged 1 commit into
devfrom
release/v1.20.0

Conversation

@erikdarlingdata

Copy link
Copy Markdown
Owner

Version bump for v1.20.0. Merging this to dev publishes nothing — the release fires when dev is merged to main.

Why minor, not patch

1.19.2 would understate it. #439 adds a source field to every warning in the JSON and MCP output and a new badge in the app and CLI, and #437 changes what an existing analysis rule concludes about a plan. Both are noticeable to a consumer, and one of them is output-shape.

What ships

User-facing

Not user-facing, but relevant to anyone building from this tag

Issues this closes on release

#430, #435, #436 — all three have been waiting on a release, and two have reporters who supplied a crash log and a repro script.

Verification

  • dotnet build clean (the 9 warnings are the pre-existing MCP9005 obsolete-API uses in McpSmokeTests.cs).
  • Full dotnet test on macOS ARM64: 307 total, 305 passed, 0 failed, 2 skipped.
  • CI green on every constituent PR.
  • Same four files the v1.19.1 bump touched, no more.

🤖 Generated with Claude Code

Minor rather than patch. 1.19.x would understate it: #439 adds a "source" field
to every warning in the JSON and MCP output and a new badge in the app and CLI, and
#437 changes what an existing analysis rule concludes about a plan. Both are things
a consumer can notice, and one of them is output-shape.

What ships:

- #437 Rule 12 no longer calls a conversion non-SARGable when it converts the
  parameter rather than the column. Plans carrying a parameter-side conversion on a
  scan lose that warning and report the scan's residual predicate instead. Verified
  against all 38 committed plans: no other plan's verdict moves.
- #439 SQL Server's own warnings are now told apart from ours, tagged [SQL Server]
  in the app and CLI and carried as "source" in JSON/MCP. Additive, but it changes
  the bytes of analyze --compact.
- #431 Robot Advice no longer takes the app down on a deep plan.
- #438 querystore gets the same depth ceiling analyze got; it had been failing
  quietly on deep plans, one ERROR row per plan.
- #443 the macOS handle resolver no longer corrupts memory on Apple silicon.
- #425 Entra MFA works again (WAM parent window handle).

Not user-facing but worth knowing for anyone building from this tag: the suite runs
on Microsoft.Testing.Platform now (#442), and `dotnet test` finishes on macOS for
the first time (#443) - 307 tests, 305 passing, 13 seconds.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@claude

claude Bot commented Aug 21, 2026

Copy link
Copy Markdown

Reviewed. Straightforward version bump — verified all four version-bearing spots are consistent: src/Directory.Build.props (<Version>), PlanViewer.Ssms/Properties/AssemblyInfo.cs (both AssemblyVersion and AssemblyFileVersion), and PlanViewer.Ssms/source.extension.vsixmanifest all now read 1.20.0, and CITATION.cff's version/date match. No code changes, no T-SQL, no new warnings. LGTM.

@erikdarlingdata
erikdarlingdata merged commit 3e8ff03 into dev Aug 21, 2026
3 checks passed
@erikdarlingdata
erikdarlingdata deleted the release/v1.20.0 branch August 21, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant