fix: fix duplicate deck names

server/lib/anki/getDeckname.test.ts

@@ -0,0 +1,13 @@
+import getDeckName from './getDeckname';
+
+describe('getDeckname', () => {
+  it('has no parent', () => {
+    expect(getDeckName(undefined, 'test')).toBe('test');
+  });
+  it('has parent', () => {
+    expect(getDeckName('parent', 'test')).toBe('parent::test');
+  })
+  it("ignores parent is same as child", () => {
+    expect(getDeckName('test', 'test')).toBe('test');
+  })
+});

server/lib/anki/getDeckname.ts

@@ -0,0 +1,16 @@
+/**
+ * This will collapse children with exact same title to the parent.
+ * This is safe to do because the parent will be the one that is used to create the deck. 
+ * The user is potentially losing the ability to create a deck with the same name as a parent.
+ * In the real world this does not really matter but adding this note in case that assumption 
+ * changes.
+ */
+export default function getDeckName(
+  parent: string | undefined,
+  name: string
+): string {
+    if (parent && parent !== name) {
+        return `${parent}::${name}`;
+    }
+    return name;
+}

server/lib/notion/BlockHandler.ts

@@ -39,6 +39,7 @@ import isTesting from './helpers/isTesting';
 import BlockEquation from './blocks/BlockEquation';
 import renderFront from './helpers/renderFront';
 import perserveNewlinesIfApplicable from './helpers/perserveNewlinesIfApplicable';
+import getDeckName from '../anki/getDeckname';
 
 class BlockHandler {
   api: NotionAPIWrapper;
@@ -399,7 +400,7 @@ class BlockHandler {
       'utf8'
     );
     const deck = new Deck(
-      parentName ? `${parentName}::${title}` : title,
+      getDeckName(parentName, title),
       cards,
       undefined,
       NOTION_STYLE,