Merge branch 'Add-Figma-Changelog-entry-for-4.15.0' of https://github.com/hashicorp/design-system into Add-Figma-Changelog-entry-for-4.15.0

Author: andgen404

.changeset/cold-humans-mix.md

@@ -0,0 +1,5 @@
+---
+"@hashicorp/design-system-components": minor
+---
+
+`Table` - Updated the visual design of `Table` cells by adding borders, making them more distinguishable when spanning rows or columns.

.changeset/gorgeous-days-call.md

@@ -0,0 +1,5 @@
+---
+"@hashicorp/design-system-components": patch
+---
+
+`Table` - fixed the aria-labels for select row and select all checkboxes so they do not change based on the state of the checkbox.

.changeset/lovely-goats-work.md

@@ -4,4 +4,4 @@ enableGlobalCache: false
 
 nodeLinker: node-modules
 
-yarnPath: .yarn/releases/yarn-4.4.0.cjs
+yarnPath: .yarn/releases/yarn-4.5.3.cjs

.changeset/smart-seahorses-cough.md

@@ -90,7 +90,7 @@ Note: If you want to ignore a changeset bump in terminal (e.g. major bump for se
 
 ## Releasing
 
-See the [release docs](packages/components/RELEASE.md) for the process we follow to publish a new package version.
+See the [release docs](https://hashicorp.atlassian.net/wiki/x/HIBT0Q) for the process we follow to publish a new package version.
 
 ## License
 

.changeset/sweet-penguins-shave.md

@@ -16,7 +16,7 @@
     "release-packages": "yarn changeset publish",
     "version-packages": "yarn changeset version && yarn install --mode update-lockfile && yarn workspace website generate-changelog-markdown-files"
   },
-  "packageManager": "yarn@4.4.0",
+  "packageManager": "yarn@4.5.3",
   "resolutions": {
     "broccoli-asset-rewrite@^2.0.0": "patch:broccoli-asset-rewrite@npm%3A2.0.0#./.yarn/patches/broccoli-asset-rewrite-npm-2.0.0-c4ce42084a.patch",
     "@glint/environment-ember-loose": "patch:@glint/environment-ember-loose@npm%3A1.4.0#~/.yarn/patches/@glint-environment-ember-loose-npm-1.4.0-31c2f31bcb.patch"

.changeset/thin-games-doubt.md

@@ -1,5 +1,15 @@
 # @hashicorp/design-system-codemods
 
+## 0.5.0
+
+### Minor Changes
+
+Update `v4/dropdown-list-item-interactive` codemod to successfully parse nested content blocks such as conditionals
+
+<small class="doc-whats-new-changelog-metadata">[#2575](https://github.com/hashicorp/design-system/pull/2575)</small>
+
+<div class="doc-whats-new-changelog-separator"></div>
+
 ## 0.4.0
 
 ### Minor Changes

.yarn/releases/yarn-4.4.0.cjs

@@ -30,6 +30,7 @@ node ./packages/codemods/bin/cli.js v3/dropdown path/to/some/glob/**/*.hbs
 ### v4
 <!--TRANSFORMS_START-->
 * [v4/contextual-components](transforms/v4/contextual-components/README.md)
+* [v4/dropdown-list-item-interactive](transforms/v4/dropdown-list-item-interactive/README.md)
 * [v4/table](transforms/v4/table/README.md)
 * [v4/icon](transforms/v4/icon/README.md)
 <!--TRANSFORMS_END-->

.yarn/releases/yarn-4.5.3.cjs

@@ -1,7 +1,7 @@
 {
   "name": "@hashicorp/design-system-codemods",
   "description": "Helios Design System codemods",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "scripts": {
     "lint": "eslint --cache .",
     "lint:fix": "eslint . --fix",

.yarnrc.yml

@@ -0,0 +1,10 @@
+{{!
+  Copyright (c) HashiCorp, Inc.
+  SPDX-License-Identifier: MPL-2.0
+}}
+<Hds::Dropdown as |dd|>
+  <dd.Title @text='Title text' />
+  {{#if this.conditional}}
+    <dd.Interactive @href='#' @text='Test' />
+  {{/if}}
+</Hds::Dropdown>

README.md

@@ -0,0 +1,10 @@
+{{!
+  Copyright (c) HashiCorp, Inc.
+  SPDX-License-Identifier: MPL-2.0
+}}
+<Hds::Dropdown as |dd|>
+  <dd.Title @text='Title text' />
+  {{#if this.conditional}}
+    <dd.Interactive @href='#'>Test</dd.Interactive>
+  {{/if}}
+</Hds::Dropdown>

package.json

@@ -5,6 +5,85 @@
 
 const CODEMOD_ANALYSIS = process.env.CODEMOD_ANALYSIS;
 
+function processChildren(children, asPrefix, b) {
+  let hasUpdatedChildren = false;
+  let processedChildren = [];
+
+  children.forEach((child) => {
+    let updatedChild;
+    let isProcessed = false;
+
+    // Check if the child is an ElementNode with the specified tag
+    if (child.type === 'ElementNode' && child.tag === `${asPrefix}.Interactive`) {
+      const textAttr = child.attributes.find((a) => a.name === '@text');
+
+      if (textAttr) {
+        const childOutputAttributes = child.attributes.filter((a) => a.name !== '@text');
+
+        const isHandlebarsAttr = textAttr.value.type === 'MustacheStatement';
+
+        let children = [];
+
+        // Handle different types of MustacheStatement values
+        if (isHandlebarsAttr) {
+          if (textAttr.value.path.type === 'NumberLiteral') {
+            children = [b.mustache(b.number(textAttr.value.path.value))];
+          } else if (textAttr.value.path.type === 'StringLiteral') {
+            children = [b.mustache(b.string(textAttr.value.path.value))];
+          } else {
+            children = [
+              b.mustache(
+                textAttr.value.path.original,
+                [...textAttr.value.params],
+                textAttr.value.hash
+              ),
+            ];
+          }
+        } else {
+          children = [b.text(textAttr.value.chars)];
+        }
+
+        // Create a new element with the updated children and attributes
+        updatedChild = b.element(
+          { name: child.tag, selfClosing: false },
+          {
+            children,
+            attrs: childOutputAttributes,
+            modifiers: child.modifiers,
+            blockParams: child.blockParams,
+          }
+        );
+
+        isProcessed = true;
+      } else {
+        updatedChild = child;
+      }
+    } else if (child.type === 'BlockStatement') {
+      // Recursively process children of BlockStatement nodes
+      const { hasUpdatedChildren: nestedHasUpdated, processedChildren: nestedProcessed } =
+        processChildren(child.program.body, asPrefix, b);
+
+      if (nestedHasUpdated) {
+        updatedChild = b.block(
+          child.path,
+          child.params,
+          child.hash,
+          b.program(nestedProcessed, child.program.blockParams),
+          child.inverse
+        );
+        isProcessed = true;
+      } else {
+        updatedChild = child;
+      }
+    }
+
+    processedChildren.push(updatedChild || child);
+    hasUpdatedChildren = hasUpdatedChildren || isProcessed;
+  });
+
+  return { hasUpdatedChildren, processedChildren };
+}
+
 module.exports = function ({ source /*, path*/ }, { parse, visit }) {
   const ast = parse(source);
 
@@ -13,85 +92,33 @@ module.exports = function ({ source /*, path*/ }, { parse, visit }) {
 
     return {
       ElementNode(node) {
+        // Check if the node is an Hds::Dropdown element
         if (node.type === 'ElementNode' && node.tag === 'Hds::Dropdown') {
-          let hasUpdatedChildren = false;
-          let processedChildren;
-
           if (node.blockParams && node.blockParams.length > 0) {
             const asPrefix = node.blockParams[0];
 
-            processedChildren = [];
-
-            if (node.children) {
-              node.children.forEach((child) => {
-                let updatedChild;
-                let isProcessed = false;
-
-                if (child.type === 'ElementNode' && child.tag === `${asPrefix}.Interactive`) {
-                  const textAttr = child.attributes.find((a) => a.name === '@text');
-
-                  if (textAttr) {
-                    const childOutputAttributes = child.attributes.filter(
-                      (a) => a.name !== '@text'
-                    );
-
-                    const isHandlebarsAttr = textAttr.value.type === 'MustacheStatement';
-
-                    let children = [];
-
-                    if (isHandlebarsAttr) {
-                      if (textAttr.value.path.type === 'NumberLiteral') {
-                        children = [b.mustache(b.number(textAttr.value.path.value))];
-                      } else if (textAttr.value.path.type === 'StringLiteral') {
-                        children = [b.mustache(b.string(textAttr.value.path.value))];
-                      } else {
-                        children = [
-                          b.mustache(
-                            textAttr.value.path.original,
-                            [...textAttr.value.params],
-                            textAttr.value.hash
-                          ),
-                        ];
-                      }
-                    } else {
-                      children = [b.text(textAttr.value.chars)];
-                    }
-
-                    updatedChild = b.element(
-                      { name: child.tag, selfClosing: false },
-                      {
-                        children,
-                        attrs: childOutputAttributes,
-                        modifiers: child.modifiers,
-                        blockParams: child.blockParams,
-                      }
-                    );
-
-                    isProcessed = true;
-                  } else {
-                    updatedChild = child;
+            // Process the children of the Hds::Dropdown element
+            const { hasUpdatedChildren, processedChildren } = processChildren(
+              node.children,
+              asPrefix,
+              b
+            );
+
+            // Return the updated element if any children were updated
+            if (hasUpdatedChildren && !CODEMOD_ANALYSIS) {
+              return [
+                b.element(
+                  { name: node.tag, selfClosing: false },
+                  {
+                    attrs: node.attributes,
+                    children: processedChildren,
+                    modifiers: node.modifiers,
+                    blockParams: node.blockParams,
                   }
-                }
-
-                processedChildren.push(updatedChild || child);
-                hasUpdatedChildren = hasUpdatedChildren || isProcessed;
-              });
+                ),
+              ];
             }
           }
-
-          if (hasUpdatedChildren && !CODEMOD_ANALYSIS) {
-            return [
-              b.element(
-                { name: node.tag, selfClosing: false },
-                {
-                  attrs: node.attributes,
-                  children: processedChildren || node.children,
-                  modifiers: node.modifiers,
-                  blockParams: node.blockParams,
-                }
-              ),
-            ];
-          }
         }
       },
     };