@learnpack/learnpack 5.0.360 → 5.0.361

package/lib/creatorDist/assets/{index-GYxGB_X0.js → index-CqVEKiJR.js}

@@ -18492,7 +18492,7 @@ function Rae(e, t) {
   return e[1] - t[1];
 }
 function HU(e, t) {
-  const a = /^([\d.]+)\s*-\s*(.*?)\s*\[(\w+):\s*(.+)\]$/,
+  const a = /^([\d.]+)\s*-\s*(.*?)\s*\[(\w+):\s*(?!\s*\]?\s*$)(.+?)\s*\]?\s*$/,
     n = e.match(a);
   if (!n) return null;
   const [, r, s, c, h] = n,

package/lib/creatorDist/index.html

@@ -10,7 +10,7 @@
     />
 
     <title>Learnpack Creator: Craft tutorials in seconds!</title>
-    <script type="module" crossorigin src="/creator/assets/index-GYxGB_X0.js"></script>
+    <script type="module" crossorigin src="/creator/assets/index-CqVEKiJR.js"></script>
     <link rel="stylesheet" crossorigin href="/creator/assets/index-B7s5Nmq8.css">
   </head>
   <body>

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@learnpack/learnpack",
   "description": "Seamlessly build, sell and/or take interactive & auto-graded tutorials, start learning now or build a new tutorial to your audience.",
-  "version": "5.0.360",
+  "version": "5.0.361",
   "author": "Alejandro Sanchez @alesanchezr",
   "contributors": [
     {

package/src/creator/src/utils/lib.test.ts

@@ -0,0 +1,101 @@
+import { describe, it, expect } from "vitest";
+import { parseLesson } from "./lib";
+import { Lesson } from "../components/LessonItem";
+
+describe("parseLesson", () => {
+  it("parses a complete step", () => {
+    const lesson = parseLesson("01.0 - Income [read: List your income.]", []);
+
+    expect(lesson).toMatchObject({
+      id: "01.0",
+      title: "Income",
+      type: "READ",
+      description: "List your income.",
+    });
+  });
+
+  it("accepts a step without the closing bracket", () => {
+    const lesson = parseLesson("01.0 - Income [READ: List your income.", []);
+
+    expect(lesson).toMatchObject({
+      id: "01.0",
+      title: "Income",
+      type: "READ",
+      description: "List your income.",
+    });
+  });
+
+  it("ignores spaces after the closing bracket", () => {
+    const lesson = parseLesson("01.0 - Income [READ: List your income.] ", []);
+
+    expect(lesson?.description).toBe("List your income.");
+  });
+
+  it("keeps brackets inside the description", () => {
+    expect(
+      parseLesson("01.0 - Arrays [CODE: Use arr[0] in a loop.]", [])
+        ?.description
+    ).toBe("Use arr[0] in a loop.");
+    expect(
+      parseLesson("01.0 - Arrays [CODE: Use arr[0] in a loop.", [])?.description
+    ).toBe("Use arr[0] in a loop.");
+  });
+
+  it("keeps brackets inside the title", () => {
+    const lesson = parseLesson(
+      "01.0 - Arrays [basics] [READ: Learn arrays.",
+      []
+    );
+
+    expect(lesson?.title).toBe("Arrays [basics]");
+    expect(lesson?.type).toBe("READ");
+  });
+
+  it.each([
+    ["01.0 - Income [READ: ]"],
+    ["01.0 - Income [READ:   ]"],
+    ["01.0 - Income [READ:]"],
+    ["01.0 - Income [READ: "],
+    ["01.0 - Income [READ:"],
+    ["01.0 - Income"],
+    ["01.0 - Income [READ"],
+    ["Income [READ: List your income.]"],
+  ])("rejects %j", (input) => {
+    expect(parseLesson(input, [])).toBeNull();
+  });
+
+  it("rejects a step with a line break", () => {
+    expect(
+      parseLesson("01.0 - Income [READ: List\nyour income.]", [])
+    ).toBeNull();
+  });
+
+  it("keeps the identity of a previous lesson when the bracket is missing", () => {
+    const previous: Lesson[] = [
+      {
+        id: "01.0",
+        uid: "lesson-1",
+        title: "Income",
+        type: "READ",
+        description: "Old description.",
+        duration: 7,
+        locked: true,
+      },
+    ];
+
+    const lesson = parseLesson(
+      "01.0 - Income [READ: New description.",
+      previous
+    );
+
+    expect(lesson).toEqual({
+      id: "01.0",
+      uid: "lesson-1",
+      title: "Income",
+      type: "READ",
+      description: "New description.",
+      duration: 7,
+      locked: true,
+    });
+  });
+});

package/src/creator/src/utils/lib.ts

@@ -6,7 +6,10 @@ import { randomUUID } from "./creatorUtils";
 import { Syllabus } from "./store";
 
 export function parseLesson(input: string, previous: Lesson[]): Lesson | null {
-  const pattern = /^([\d.]+)\s*-\s*(.*?)\s*\[(\w+):\s*(.+)\]$/;
+  // The closing bracket is optional because the model sometimes drops it, and a
+  // step without it would otherwise be lost. Blank descriptions are still rejected.
+  const pattern =
+    /^([\d.]+)\s*-\s*(.*?)\s*\[(\w+):\s*(?!\s*\]?\s*$)(.+?)\s*\]?\s*$/;
   const match = input.match(pattern);
 
   if (!match) return null;

package/src/creatorDist/assets/{index-GYxGB_X0.js → index-CqVEKiJR.js}

@@ -18492,7 +18492,7 @@ function Rae(e, t) {
   return e[1] - t[1];
 }
 function HU(e, t) {
-  const a = /^([\d.]+)\s*-\s*(.*?)\s*\[(\w+):\s*(.+)\]$/,
+  const a = /^([\d.]+)\s*-\s*(.*?)\s*\[(\w+):\s*(?!\s*\]?\s*$)(.+?)\s*\]?\s*$/,
     n = e.match(a);
   if (!n) return null;
   const [, r, s, c, h] = n,

package/src/creatorDist/index.html

@@ -10,7 +10,7 @@
     />
 
     <title>Learnpack Creator: Craft tutorials in seconds!</title>
-    <script type="module" crossorigin src="/creator/assets/index-GYxGB_X0.js"></script>
+    <script type="module" crossorigin src="/creator/assets/index-CqVEKiJR.js"></script>
     <link rel="stylesheet" crossorigin href="/creator/assets/index-B7s5Nmq8.css">
   </head>
   <body>