@sillsdev/docu-notion 1.0.0-alpha.1 → 1.0.0-alpha.4

package/dist/plugins/HeadingTranformer.spec.js

@@ -74,6 +74,14 @@ test("Adds anchor to headings", () => __awaiter(void 0, void 0, void 0, function
     ]);
     expect(result.trim()).toBe(`# Heading One {/* #${headingBlockId.replaceAll("-", "")} */}`);
 }));
+test("Adds anchor to H4 headings", () => __awaiter(void 0, void 0, void 0, function* () {
+    const headingBlockId = "86f746f4-1c79-4ba1-a2f6-a1d59c2f9d23";
+    const config = { plugins: [HeadingTransformer_1.standardHeadingTransformer] };
+    const result = yield (0, pluginTestRun_1.blocksToMarkdown)(config, [
+        makeHeadingBlock(headingBlockId, "Heading Four", "heading_4"),
+    ]);
+    expect(result.trim()).toBe(`#### Heading Four {/* #${headingBlockId.replaceAll("-", "")} */}`);
+}));
 test("docusaurus-v2 flag keeps legacy heading id syntax", () => __awaiter(void 0, void 0, void 0, function* () {
     const headingBlockId = "86f746f4-1c79-4ba1-a2f6-a1d59c2f9d23";
     const config = { plugins: [HeadingTransformer_1.standardHeadingTransformer] };

package/dist/plugins/HeadingTransformer.js

@@ -11,13 +11,38 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.standardHeadingTransformer = void 0;
 const log_1 = require("../log");
+function renderHeadingText(context, block, type) {
+    // Work around notion-to-md only shipping built-in heading renderers for
+    // heading_1 through heading_3. For heading_4 and above we still reuse its
+    // inline annotation logic, but we assemble the heading markdown ourselves.
+    const headingBlock = block[type];
+    const blockContent = (headingBlock === null || headingBlock === void 0 ? void 0 : headingBlock.text) || (headingBlock === null || headingBlock === void 0 ? void 0 : headingBlock.rich_text) || [];
+    return blockContent
+        .map((content) => {
+        if (content.type === "equation" && content.equation) {
+            return `$${content.equation.expression}$`;
+        }
+        let plainText = context.notionToMarkdown.annotatePlainText(content.plain_text, content.annotations);
+        if (content.href) {
+            plainText = `[${plainText}](${content.href})`;
+        }
+        return plainText;
+    })
+        .join("");
+}
 // Makes links to headings work in docusaurus
 // https://github.com/sillsdev/docu-notion/issues/20
 function headingTransformer(context, block) {
     return __awaiter(this, void 0, void 0, function* () {
         // First, remove the prefix we added to the heading type
-        block.type = block.type.replace("DN_", "");
-        const markdown = yield context.notionToMarkdown.blockToMarkdown(block);
+        const type = block.type.replace("DN_", "");
+        block.type = type;
+        const headingLevel = Number(type.replace("heading_", ""));
+        const markdown = headingLevel <= 3
+            ? yield context.notionToMarkdown.blockToMarkdown(block)
+            // notion-to-md 3.1.1 falls through on heading_4+ and crashes trying to
+            // read block[type].text, so render those levels locally.
+            : `${"#".repeat(headingLevel)} ${renderHeadingText(context, block, type)}`;
         (0, log_1.logDebug)("headingTransformer, markdown of a heading before adding id", markdown);
         // To make heading links work in Docusaurus, we add a stable block-id anchor.
         // Docusaurus v2 uses explicit heading IDs, while the v3 default can use the
@@ -61,5 +86,11 @@ exports.standardHeadingTransformer = {
             type: "DN_heading_3",
             getStringFromBlock: (context, block) => headingTransformer(context, block),
         },
+        {
+            type: "DN_heading_4",
+            // Keep this explicit so H4 blocks take the local workaround path instead
+            // of being handed back to notion-to-md's unsupported default branch.
+            getStringFromBlock: (context, block) => headingTransformer(context, block),
+        },
     ],
 };

package/dist/pull.js

@@ -66,16 +66,19 @@ function getOptionsForLogging(options) {
 const kNotionApiVersion = "2026-03-11";
 let layoutStrategy;
 let notionToMarkdown;
-const pages = new Array();
-const counts = {
-    output_normally: 0,
-    skipped_because_empty: 0,
-    skipped_because_status: 0,
-    skipped_because_level_cannot_have_content: 0,
-    error_because_no_slug: 0,
-};
 function notionPull(options) {
     return __awaiter(this, void 0, void 0, function* () {
+        // These are local to each pull so that repeated calls (e.g. in tests or
+        // programmatic multi-run scenarios) don't accumulate pages or counts from
+        // previous runs.
+        const pages = new Array();
+        const counts = {
+            output_normally: 0,
+            skipped_because_empty: 0,
+            skipped_because_status: 0,
+            skipped_because_level_cannot_have_content: 0,
+            error_because_no_slug: 0,
+        };
         // It's helpful when troubleshooting CI secrets and environment variables to see what options actually made it to docu-notion.
         const optionsForLogging = getOptionsForLogging(options);
         const config = yield (0, configuration_1.loadConfigAsync)();
@@ -97,12 +100,12 @@ function notionPull(options) {
             (0, process_1.exit)(1);
         }
         (0, log_1.group)("Stage 1: walk children of the page named 'Outline', looking for pages...");
-        yield getPagesRecursively(options, "", options.rootPage, 0, true);
+        yield getPagesRecursively(options, "", options.rootPage, 0, true, pages, counts);
         (0, log_1.logDebug)("getPagesRecursively", JSON.stringify(pages, null, 2));
         (0, log_1.info)(`Found ${pages.length} pages`);
         (0, log_1.endGroup)();
         (0, log_1.group)(`Stage 2: convert ${pages.length} Notion pages to markdown and save locally...`);
-        yield outputPages(options, config, pages);
+        yield outputPages(options, config, pages, counts);
         (0, log_1.endGroup)();
         (0, log_1.group)("Stage 3: clean up old files & images...");
         yield layoutStrategy.cleanupOldFiles();
@@ -110,7 +113,7 @@ function notionPull(options) {
         (0, log_1.endGroup)();
     });
 }
-function outputPages(options, config, pages) {
+function outputPages(options, config, pages, counts) {
     return __awaiter(this, void 0, void 0, function* () {
         const context = {
             getBlockChildren: getBlockChildren,
@@ -124,7 +127,7 @@ function outputPages(options, config, pages) {
             notionToMarkdown: notionToMarkdown,
             options: options,
             pages: pages,
-            counts: counts, // review will this get copied or pointed to?
+            counts: counts,
             imports: [],
             convertNotionLinkToLocalDocusaurusLink: (url) => (0, internalLinks_1.convertInternalUrl)(context, url),
         };
@@ -141,7 +144,7 @@ function outputPages(options, config, pages) {
                 context.options.statusTag != "*" &&
                 page.status !== context.options.statusTag) {
                 (0, log_1.verbose)(`Skipping page because status is not '${context.options.statusTag}': ${page.nameOrTitle}`);
-                ++context.counts.skipped_because_status;
+                ++counts.skipped_because_status;
             }
             else {
                 if (options.requireSlugs && !page.hasExplicitSlug) {
@@ -149,7 +152,7 @@ function outputPages(options, config, pages) {
                     ++counts.error_because_no_slug;
                 }
                 const markdown = yield (0, transform_1.getMarkdownForPage)(config, context, page);
-                writePage(page, markdown);
+                writePage(page, markdown, counts);
             }
         }
         if (counts.error_because_no_slug > 0)
@@ -164,7 +167,7 @@ function outputPages(options, config, pages) {
 // outline that contain content (which we call "Simple" pages). Later, we can
 // then step through this list creating the files we need, and, crucially, be
 // able to figure out what the url will be for any links between content pages.
-function getPagesRecursively(options, incomingContext, pageIdOfThisParent, orderOfThisParent, rootLevel) {
+function getPagesRecursively(options, incomingContext, pageIdOfThisParent, orderOfThisParent, rootLevel, pages, counts) {
     return __awaiter(this, void 0, void 0, function* () {
         var _a;
         const pageInTheOutline = yield fromPageId(incomingContext, pageIdOfThisParent, orderOfThisParent, true);
@@ -197,7 +200,7 @@ function getPagesRecursively(options, incomingContext, pageIdOfThisParent, order
                 layoutContext = layoutStrategy.newLevel(options.markdownOutputPath, pageInTheOutline.order, incomingContext, pageInTheOutline.nameOrTitle);
             }
             for (const childPageInfo of pageInfo.childPageIdsAndOrder) {
-                yield getPagesRecursively(options, layoutContext, childPageInfo.id, childPageInfo.order, false);
+                yield getPagesRecursively(options, layoutContext, childPageInfo.id, childPageInfo.order, false, pages, counts);
             }
             for (const linkPageInfo of pageInfo.linksPageIdsAndOrder) {
                 pages.push(yield fromPageId(layoutContext, linkPageInfo.id, linkPageInfo.order, false));
@@ -209,7 +212,7 @@ function getPagesRecursively(options, incomingContext, pageIdOfThisParent, order
         }
     });
 }
-function writePage(page, finalMarkdown) {
+function writePage(page, finalMarkdown, counts) {
     const mdPath = layoutStrategy.getPathForPage(page, ".md");
     (0, log_1.verbose)(`writing ${mdPath}`);
     fs.writeFileSync(mdPath, finalMarkdown, {});

package/package.json

@@ -54,6 +54,7 @@
     "lint-staged": "^10.5.4",
     "prettier": "^2.2.1",
     "rimraf": "^4.1.2",
+    "semantic-release": "25.0.3",
     "typescript": "^5.9.3",
     "vite": "^4.2.1",
     "vitest": "^0.30.1"
@@ -87,8 +88,15 @@
   "publishConfig": {
     "access": "public"
   },
+  "packageManager": "npm@11.16.0",
   "volta": {
-    "node": "22.21.0"
+    "node": "24.16.0",
+    "npm": "11.16.0"
   },
-  "version": "1.0.0-alpha.1"
+  "allowScripts": {
+    "core-js@2.6.12": true,
+    "esbuild@0.17.17": true,
+    "fsevents": false
+  },
+  "version": "1.0.0-alpha.4"
 }