robot fixes

Author: sergical

src/components/onboarding/index.tsx

@@ -276,35 +276,35 @@ export function updateElementsVisibilityForOptions(
       // Check line before open wrapper
       if (openIndex > 0) {
         const prevLine = allLines[openIndex - 1];
-        const prevText = prevLine.textContent?.trim();
-        if (!prevText || prevText === '') {
-          prevLine.style.display = 'none';
+        if (!prevLine.textContent?.trim()) {
+          prevLine.classList.add('hidden');
+          prevLine.dataset.emptyLineHidden = 'true';
         }
       }
 
       // Check line after close wrapper
       if (closeIndex < allLines.length - 1) {
         const nextLine = allLines[closeIndex + 1];
-        const nextText = nextLine.textContent?.trim();
-        if (!nextText || nextText === '') {
-          nextLine.style.display = 'none';
+        if (!nextLine.textContent?.trim()) {
+          nextLine.classList.add('hidden');
+          nextLine.dataset.emptyLineHidden = 'true';
         }
       }
     } else {
       // Show empty lines when integrations are visible
       if (openIndex > 0) {
         const prevLine = allLines[openIndex - 1];
-        const prevText = prevLine.textContent?.trim();
-        if (!prevText || prevText === '') {
-          prevLine.style.display = '';
+        if (prevLine.dataset.emptyLineHidden === 'true') {
+          prevLine.classList.remove('hidden');
+          delete prevLine.dataset.emptyLineHidden;
         }
       }
 
       if (closeIndex < allLines.length - 1) {
         const nextLine = allLines[closeIndex + 1];
-        const nextText = nextLine.textContent?.trim();
-        if (!nextText || nextText === '') {
-          nextLine.style.display = '';
+        if (nextLine.dataset.emptyLineHidden === 'true') {
+          nextLine.classList.remove('hidden');
+          delete nextLine.dataset.emptyLineHidden;
         }
       }
     }

src/rehype-onboarding-lines.js

@@ -68,18 +68,25 @@ function handle_inline_options(node) {
     if (lineStr.match(/integrations:\s*\[/)) {
       // Mark the opening line and hide it by default
       line.properties['data-integrations-wrapper'] = 'open';
-      line.properties.className = [...(line.properties.className || []), 'hidden'];
+      const openClasses = Array.isArray(line.properties.className)
+        ? line.properties.className
+        : line.properties.className
+          ? [line.properties.className]
+          : [];
+      line.properties.className = [...openClasses, 'hidden'];
 
-      // Find the closing "],"
+      // Find the closing "]," - must be exactly ],
       for (let j = i + 1; j < node.children.length; j++) {
         const closeStr = toString(node.children[j]).trim();
-        if (closeStr.includes('],')) {
+        if (closeStr === '],') {
           // Mark the closing line and hide it by default
           node.children[j].properties['data-integrations-wrapper'] = 'close';
-          node.children[j].properties.className = [
-            ...(node.children[j].properties.className || []),
-            'hidden',
-          ];
+          const closeClasses = Array.isArray(node.children[j].properties.className)
+            ? node.children[j].properties.className
+            : node.children[j].properties.className
+              ? [node.children[j].properties.className]
+              : [];
+          node.children[j].properties.className = [...closeClasses, 'hidden'];
           break;
         }
       }