@moxn/kb-migrate 0.4.38 → 0.4.39

package/dist/sources/notion.d.ts

@@ -150,7 +150,7 @@ export declare function slugify(title: string): string;
  * so a second same-titled sibling was rejected by that assert (audit finding D).
  * Mutates `siblingSlugCounts` (keyed on the BASE slug) the way the builder expects.
  */
-export declare function disambiguateSibling(title: string, siblingSlugCounts: Map<string, number>): {
+export declare function disambiguateSibling(title: string, siblingSlugCounts: Map<string, number>, slugifyFn?: (s: string) => string): {
     name: string;
     slug: string;
 };

package/dist/sources/notion.js

@@ -632,12 +632,12 @@ export function slugify(title) {
  * so a second same-titled sibling was rejected by that assert (audit finding D).
  * Mutates `siblingSlugCounts` (keyed on the BASE slug) the way the builder expects.
  */
-export function disambiguateSibling(title, siblingSlugCounts) {
-    const baseSlug = slugify(title);
+export function disambiguateSibling(title, siblingSlugCounts, slugifyFn = slugify) {
+    const baseSlug = slugifyFn(title);
     const existing = siblingSlugCounts.get(baseSlug) ?? 0;
     siblingSlugCounts.set(baseSlug, existing + 1);
     if (existing === 0)
         return { name: title, slug: baseSlug };
     const name = `${title} ${existing + 1}`;
-    return { name, slug: slugify(name) };
+    return { name, slug: slugifyFn(name) };
 }

package/dist/sources/notion.test.js

@@ -68,3 +68,16 @@ describe('disambiguateSibling (D — dedup name AND path together)', () => {
         expect(disambiguateSibling('Beta', counts).slug).toBe('beta');
     });
 });
+describe('disambiguateSibling — custom slugify (OneNote uses slugifyPathSegment)', () => {
+    const upperSlug = (s) => s.toUpperCase().replace(/ /g, '_');
+    it('derives the slug with the provided function, keeping the bijection', () => {
+        const counts = new Map();
+        expect(disambiguateSibling('My Page', counts, upperSlug)).toEqual({
+            name: 'My Page',
+            slug: 'MY_PAGE',
+        });
+        const second = disambiguateSibling('My Page', counts, upperSlug);
+        expect(second).toEqual({ name: 'My Page 2', slug: 'MY_PAGE_2' });
+        expect(upperSlug(second.name)).toBe(second.slug);
+    });
+});

package/dist/sources/onenote/onenote-tree.js

@@ -11,6 +11,7 @@
  * own ID or any ancestor's ID is in the set. If the set is empty, every
  * page is included.
  */
+import { disambiguateSibling } from '../notion.js';
 const MAX_GROUP_DEPTH = 4;
 /**
  * Normalize any human text into a ltree-safe slug segment.
@@ -96,12 +97,22 @@ async function walkSection(client, section, pathSegs, ctx, ancestorSelected, sel
         createdBefore: opts.createdBefore,
     });
     const sectionSeg = slugifyPathSegment(section.displayName);
+    // Dedup same-titled sibling pages within a section — suffix the title AND the
+    // path segment together so slugify(name) === pathTail (the server's hard
+    // bijection assert) and two pages never collide on one path. Without this,
+    // two "Untitled" pages produce the same kbPath → the second silently clobbers
+    // or skips the first on import. (Notion finding D; OneNote had no dedup at all.)
+    const pageSlugCounts = new Map();
     for (const page of pages) {
         const pageSelected = ancestorSelected || (!!sel && sel.has(page.id));
         if (!pageSelected)
             continue;
-        const pageSeg = slugifyPathSegment(page.title ?? 'untitled');
-        out.push(toDiscovered(page, section, [...pathSegs, sectionSeg, pageSeg], ctx));
+        const { name: dedupTitle, slug: pageSeg } = disambiguateSibling(page.title ?? 'untitled', pageSlugCounts, slugifyPathSegment);
+        out.push(toDiscovered({ ...page, title: dedupTitle }, section, [
+            ...pathSegs,
+            sectionSeg,
+            pageSeg,
+        ], ctx));
     }
 }
 function toDiscovered(page, section, segs, ctx) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moxn/kb-migrate",
-  "version": "0.4.38",
+  "version": "0.4.39",
   "description": "Migration tool for importing documents into Moxn Knowledge Base from local files, Notion, Google Docs, and more",
   "type": "module",
   "main": "dist/index.js",