Get tests passing

Author: ericallam

.github/workflows/unit-tests.yml

@@ -38,7 +38,9 @@ jobs:
           SESSION_SECRET: "secret"
           MAGIC_LINK_SECRET: "secret"
           ENCRYPTION_KEY: "secret"
-
+      
+      - name: 🧪 Run Package Unit Tests
+        run: pnpm run test --filter "@trigger.dev/*"
 
       - name: 🧪 Run Internal Unit Tests
         run: pnpm run test --filter "@internal/*"

packages/core/src/v3/utils/flattenAttributes.ts

@@ -97,17 +97,15 @@ export function unflattenAttributes(
   for (const [key, value] of Object.entries(obj)) {
     const parts = key.split(".").reduce(
       (acc, part) => {
-        if (part.includes("[")) {
-          // Handling nested array indices
+        if (part.startsWith("[") && part.endsWith("]")) {
+          // Handle array indices more precisely
           const match = part.match(/^\[(\d+)\]$/);
-          const number = match?.[1];
-
-          if (number) {
-            acc.push(parseInt(number));
-            return acc;
+          if (match && match[1]) {
+            acc.push(parseInt(match[1]));
+          } else {
+            // Remove brackets for non-numeric array keys
+            acc.push(part.slice(1, -1));
           }
-
-          acc.push(part.replace("[", "").replace("]", ""));
         } else {
           acc.push(part);
         }
@@ -119,22 +117,25 @@ export function unflattenAttributes(
     let current: any = result;
     for (let i = 0; i < parts.length - 1; i++) {
       const part = parts[i];
+      const nextPart = parts[i + 1];
 
-      if (!part) {
+      if (!part && part !== 0) {
         continue;
       }
 
-      const nextPart = parts[i + 1];
-      const isArray = typeof nextPart === "number";
-      if (isArray && !Array.isArray(current[part])) {
-        current[part] = [];
-      } else if (!isArray && current[part] === undefined) {
+      if (typeof nextPart === "number") {
+        // Ensure we create an array for numeric indices
+        current[part] = Array.isArray(current[part]) ? current[part] : [];
+      } else if (current[part] === undefined) {
+        // Create an object for non-numeric paths
         current[part] = {};
       }
+
       current = current[part];
     }
+
     const lastPart = parts[parts.length - 1];
-    if (lastPart) {
+    if (lastPart !== undefined) {
       current[lastPart] = rehydrateNull(value);
     }
   }

packages/core/test/flattenAttributes.test.ts

@@ -1,6 +1,20 @@
 import { flattenAttributes, unflattenAttributes } from "../src/v3/utils/flattenAttributes.js";
 
 describe("flattenAttributes", () => {
+  it("handles number keys correctl", () => {
+    expect(flattenAttributes({ bar: { "25": "foo" } })).toEqual({ "bar.25": "foo" });
+    expect(unflattenAttributes({ "bar.25": "foo" })).toEqual({ bar: { "25": "foo" } });
+    expect(flattenAttributes({ bar: ["foo", "baz"] })).toEqual({
+      "bar.[0]": "foo",
+      "bar.[1]": "baz",
+    });
+    expect(unflattenAttributes({ "bar.[0]": "foo", "bar.[1]": "baz" })).toEqual({
+      bar: ["foo", "baz"],
+    });
+    expect(flattenAttributes({ bar: { 25: "foo" } })).toEqual({ "bar.25": "foo" });
+    expect(unflattenAttributes({ "bar.25": "foo" })).toEqual({ bar: { 25: "foo" } });
+  });
+
   it("handles null correctly", () => {
     expect(flattenAttributes(null)).toEqual({ "": "$@null((" });
     expect(unflattenAttributes({ "": "$@null((" })).toEqual(null);