v1.0.0: Bug fixes and version bump

src/PlanViewer.App/Controls/PlanViewerControl.axaml.cs

@@ -1783,8 +1783,9 @@ private object BuildNodeTooltipContent(PlanNode node, List<PlanWarning>? allWarn
             || node.ActualRebinds > 0 || node.ActualRewinds > 0)
         {
             AddTooltipSection(stack, "Rebinds / Rewinds");
-            if (node.EstimateRebinds > 0) AddTooltipRow(stack, "Est. Rebinds", $"{node.EstimateRebinds:N1}");
-            if (node.EstimateRewinds > 0) AddTooltipRow(stack, "Est. Rewinds", $"{node.EstimateRewinds:N1}");
+            // Always show both estimated values when section is visible
+            AddTooltipRow(stack, "Est. Rebinds", $"{node.EstimateRebinds:N1}");
+            AddTooltipRow(stack, "Est. Rewinds", $"{node.EstimateRewinds:N1}");
             if (node.ActualRebinds > 0) AddTooltipRow(stack, "Actual Rebinds", $"{node.ActualRebinds:N0}");
             if (node.ActualRewinds > 0) AddTooltipRow(stack, "Actual Rewinds", $"{node.ActualRewinds:N0}");
         }

src/PlanViewer.App/Info.plist

@@ -11,9 +11,9 @@
     <key>CFBundleExecutable</key>
     <string>PlanViewer.App</string>
     <key>CFBundleVersion</key>
-    <string>0.9.0</string>
+    <string>1.0.0</string>
     <key>CFBundleShortVersionString</key>
-    <string>0.9.0</string>
+    <string>1.0.0</string>
     <key>CFBundlePackageType</key>
     <string>APPL</string>
     <key>NSHumanReadableCopyright</key>

src/PlanViewer.App/PlanViewer.App.csproj

@@ -5,7 +5,7 @@
     <Nullable>enable</Nullable>
     <ApplicationManifest>app.manifest</ApplicationManifest>
     <AvaloniaUseCompiledBindingsByDefault>true</AvaloniaUseCompiledBindingsByDefault>
-    <Version>0.9.0</Version>
+    <Version>1.0.0</Version>
     <Authors>Erik Darling</Authors>
     <Company>Darling Data LLC</Company>
     <Product>Performance Studio</Product>

src/PlanViewer.App/Services/AdviceContentBuilder.cs

@@ -215,7 +215,9 @@ public static StackPanel Build(string content)
             }
 
             // Expensive operator timing lines: "4,616ms CPU (61%), 586ms elapsed (62%)"
-            if (trimmed.Contains("ms CPU") || trimmed.Contains("ms elapsed"))
+            // Must start with a digit to avoid catching "Runtime: 1,234ms elapsed, 1,200ms CPU"
+            if ((trimmed.Contains("ms CPU") || trimmed.Contains("ms elapsed"))
+                && trimmed.Length > 0 && char.IsDigit(trimmed[0]))
             {
                 panel.Children.Add(CreateOperatorTimingLine(trimmed));
                 continue;

src/PlanViewer.Cli/PlanViewer.Cli.csproj

@@ -11,7 +11,7 @@
     <Nullable>enable</Nullable>
     <RootNamespace>PlanViewer.Cli</RootNamespace>
     <AssemblyName>planview</AssemblyName>
-    <Version>0.9.0</Version>
+    <Version>1.0.0</Version>
     <Authors>Erik Darling</Authors>
     <Company>Darling Data LLC</Company>
     <Product>Performance Studio</Product>

tests/PlanViewer.Core.Tests/PlanAnalyzerTests.cs

@@ -743,4 +743,20 @@ public void ParallelSkew_DetectedInSkewedPlan()
         }
         return null;
     }
+
+    // ---------------------------------------------------------------
+    // NoJoinPredicate: verify it flows through to TextFormatter output
+    // ---------------------------------------------------------------
+
+    [Fact]
+    public void NoJoinPredicate_AppearsInTextFormatterOutput()
+    {
+        var plan = PlanTestHelper.LoadAndAnalyze("missing-join-predicate.sqlplan");
+        var analysis = PlanViewer.Core.Output.ResultMapper.Map(plan, "file");
+        var text = PlanViewer.Core.Output.TextFormatter.Format(analysis);
+
+        Assert.Contains("[Warning]", text);
+        Assert.Contains("No Join Predicate", text);
+        Assert.Contains("often misleading", text);
+    }
 }