@tekyzinc/gsd-t 2.73.21 → 2.73.23

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 All notable changes to GSD-T are documented here. Updated with each release.
 
+## [2.73.23] - 2026-04-09
+
+### Fixed
+- **Container props auto-redirect to parent** — setting `gap`, `borderRadius`, or `overflow` on a bar segment (child) now auto-targets the parent flex/grid container. Previously only worked when the container itself was selected.
+
+## [2.73.22] - 2026-04-09
+
+### Added
+- **Stack-level border-radius** — setting `borderRadius` on a flex/grid bar column container auto-sets `overflow: hidden` (so rounded corners clip child segments) and propagates to all sibling columns. One change rounds all stacked bars.
+
 ## [2.73.21] - 2026-04-09
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tekyzinc/gsd-t",
-  "version": "2.73.21",
+  "version": "2.73.23",
   "description": "GSD-T: Contract-Driven Development for Claude Code — 56 slash commands with headless CI/CD mode, graph-powered code analysis, real-time agent dashboard, execution intelligence, task telemetry, doc-ripple enforcement, backlog management, impact analysis, test sync, milestone archival, and PRD generation",
   "author": "Tekyz, Inc.",
   "license": "MIT",

package/scripts/gsd-t-design-review-inject.js

@@ -712,8 +712,22 @@
           const propagate = msg.propagate !== false; // default true
           let propagatedCount = 0;
 
-          // Apply to primary element
-          lockedEl.style.setProperty(cssName, msg.value, "important");
+          // Container-only props: if applied to a non-container child, redirect to parent
+          const containerOnlyProps = new Set(["gap", "row-gap", "column-gap", "border-radius"]);
+          let targetEl = lockedEl;
+          if (containerOnlyProps.has(cssName)) {
+            const elStyle = getComputedStyle(lockedEl);
+            const isContainer = elStyle.display === "flex" || elStyle.display === "inline-flex" || elStyle.display === "grid";
+            if (!isContainer && lockedEl.parentElement) {
+              const parentStyle = getComputedStyle(lockedEl.parentElement);
+              if (parentStyle.display === "flex" || parentStyle.display === "inline-flex" || parentStyle.display === "grid") {
+                targetEl = lockedEl.parentElement;
+              }
+            }
+          }
+
+          // Apply to target element (may be parent for container-only props)
+          targetEl.style.setProperty(cssName, msg.value, "important");
 
           // Typography props cascade to descendants
           const inheritProps = new Set(["font-size", "font-weight", "font-family", "line-height",
@@ -792,18 +806,28 @@
             } else if (tag === "div" && parent) {
               const elStyle = getComputedStyle(lockedEl);
               const parentStyle = getComputedStyle(parent);
-              const layoutProps = new Set(["gap", "row-gap", "column-gap"]);
-
-              if (layoutProps.has(cssName)) {
-                // Layout props on a flex/grid container: find all sibling containers with same structure
-                // (e.g., all bar columns in a chart — same parent, same display type)
-                const elDisplay = elStyle.display;
-                if (elDisplay === "flex" || elDisplay === "inline-flex" || elDisplay === "grid") {
-                  for (const sib of parent.children) {
-                    if (sib === lockedEl || sib.tagName !== "DIV") continue;
+
+              // Props that propagate across sibling containers (all bar columns)
+              const containerProps = new Set(["gap", "row-gap", "column-gap", "border-radius", "overflow"]);
+
+              if (containerProps.has(cssName)) {
+                // Use targetEl (may be parent if redirected) for container propagation
+                const containerEl = targetEl;
+                const containerParent = containerEl.parentElement;
+                const containerDisplay = getComputedStyle(containerEl).display;
+                if (containerParent && (containerDisplay === "flex" || containerDisplay === "inline-flex" || containerDisplay === "grid")) {
+                  // Auto-set overflow:hidden when border-radius is applied to a container
+                  if (cssName === "border-radius" && msg.value && msg.value !== "0px" && msg.value !== "0") {
+                    containerEl.style.setProperty("overflow", "hidden", "important");
+                  }
+                  for (const sib of containerParent.children) {
+                    if (sib === containerEl || sib.tagName !== "DIV") continue;
                     const sibStyle = getComputedStyle(sib);
-                    if (sibStyle.display === elDisplay) {
+                    if (sibStyle.display === containerDisplay) {
                       sib.style.setProperty(cssName, msg.value, "important");
+                      if (cssName === "border-radius" && msg.value && msg.value !== "0px" && msg.value !== "0") {
+                        sib.style.setProperty("overflow", "hidden", "important");
+                      }
                       propagatedCount++;
                     }
                   }
@@ -826,7 +850,7 @@
             const tag = lockedEl.tagName.toLowerCase();
             const columnProps = new Set(["text-align", "width", "min-width", "max-width"]);
             const rowProps = new Set(["height", "min-height", "max-height"]);
-            const layoutProps = new Set(["gap", "row-gap", "column-gap"]);
+            const containerPropsLabel = new Set(["gap", "row-gap", "column-gap", "border-radius", "overflow"]);
             if ((tag === "td" || tag === "th") && columnProps.has(cssName)) {
               const row = lockedEl.closest("tr");
               const colIdx = row ? Array.from(row.children).indexOf(lockedEl) + 1 : 0;
@@ -835,7 +859,7 @@
               propagateScope = "all rows";
             } else if (tag === "tr") {
               propagateScope = "all rows";
-            } else if (layoutProps.has(cssName)) {
+            } else if (containerPropsLabel.has(cssName)) {
               propagateScope = "all columns";
             } else {
               propagateScope = "similar";