Merge pull request #130 from jongalloway/copilot/sub-pr-129

Fix stroke-linecap emission, complete PackageReadme DSL list, align target test ring order with docs

Author: jongalloway

doc/PackageReadme.md

@@ -59,7 +59,7 @@ dnx DiagramForge.Tool diagram.mmd --theme dracula --transparent -o overlay.svg
 ## Supported today
 
 - Mermaid subset: flowchart, block, sequence, state, mindmap, timeline, venn, architecture, and xychart
-- Conceptual DSL: matrix, pyramid, cycle, pillars, funnel, radial, and target
+- Conceptual DSL: matrix, pyramid, cycle, pillars, funnel, radial, target, chevrons, snake, tablist, tree, and more
 - Theme presets, theme JSON files, palette overrides, and frontmatter styling
 
 DiagramForge intentionally implements a focused Mermaid subset rather than full Mermaid.js parity.

src/DiagramForge/Rendering/SvgStructureWriter.cs

@@ -316,6 +316,7 @@ private static void AppendConnectorPath(
         string extraAttributes = "")
     {
         double strokeWidth = baseStrokeWidth ?? theme.StrokeWidth;
+        string escapedLineCap = SvgRenderSupport.Escape(lineCap);
 
         if (outlinedTargetConnector)
         {
@@ -325,18 +326,14 @@ private static void AppendConnectorPath(
             double outlineWidth = connectorStrokeWidth + (haloPerSide * 2);
             string appliedMarkerStart = includeMarkers ? markerStart : " ";
             string appliedMarkerEnd = includeMarkers ? markerEnd : " ";
-            string escapedLineCap = SvgRenderSupport.Escape(lineCap);
             sb.AppendLine($"""  <path d="{pathData}" fill="none" stroke="{outlineColor}" stroke-width="{SvgRenderSupport.F(outlineWidth)}" stroke-linecap="{escapedLineCap}" stroke-linejoin="round"{extraAttributes}{strokeDash}/>""");
             sb.AppendLine($"""  <path d="{pathData}" fill="none" stroke="{strokeColor}" stroke-width="{SvgRenderSupport.F(connectorStrokeWidth)}" stroke-linecap="{escapedLineCap}" stroke-linejoin="round"{extraAttributes}{strokeDash}{appliedMarkerStart}{appliedMarkerEnd}/>""");
             return;
         }
 
         string startMarker = includeMarkers ? markerStart : " ";
         string endMarker = includeMarkers ? markerEnd : " ";
-        string lineCapAttribute = string.Equals(lineCap, "round", StringComparison.Ordinal)
-            ? string.Empty
-            : $" stroke-linecap=\"{SvgRenderSupport.Escape(lineCap)}\"";
-        sb.AppendLine($"""  <path d="{pathData}" fill="none" stroke="{strokeColor}" stroke-width="{SvgRenderSupport.F(strokeWidth)}"{lineCapAttribute}{extraAttributes}{strokeDash}{startMarker}{endMarker}/>""");
+        sb.AppendLine($"""  <path d="{pathData}" fill="none" stroke="{strokeColor}" stroke-width="{SvgRenderSupport.F(strokeWidth)}" stroke-linecap="{escapedLineCap}"{extraAttributes}{strokeDash}{startMarker}{endMarker}/>""");
     }
 
     private static bool TryGetMetadataDouble(IReadOnlyDictionary<string, object> metadata, string key, out double value)