fix: get admonition label order correct

Author: hatton

package.json

@@ -8,7 +8,6 @@
     "lib/**/*"
   ],
   "scripts": {
-    "write-version": "cross-var echo {\\\"version\\\":\\\"$npm_package_version\\\"} > version.json",
     "build": "npm run write-version && tsc",
     "clean": "rm -rf ./lib/",
     "semantic-release": "semantic-release",

src/DocusaurusTweaks.ts

@@ -19,7 +19,7 @@ export function tweakForDocusaurus(input: string): string {
     if (docusaurusAdmonition) {
       output = output.replace(
         string,
-        `${docusaurusAdmonition}:::\n\n${content}\n\n:::\n\n`
+        `:::${docusaurusAdmonition}\n\n${content}\n\n:::\n\n`
       );
     }
   }

src/index.ts

@@ -3,13 +3,6 @@
 import { program } from "commander";
 
 import { notionPull } from "./pull";
-// note: just getting process.en.npm_package_version will return the version of the client package!
-// So in our npm scripts we write out this version info and then pull it in here (tsc can
-// handle json more readily than just txt)
-//const version = require("./version.json");
-
-// eslint-disable-next-line @typescript-eslint/restrict-template-expressions
-//console.log(`notion-pull version ${version.version}`);
 const pkg = require("../package.json");
 // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
 console.log(`notion-pull version ${pkg.version}`);
@@ -23,7 +16,7 @@ program
   )
   .requiredOption(
     "-m, --markdown-output-path  <string>",
-    "root of the hierarchy for md files. WARNING: node-pull will delete this directory."
+    "root of the hierarchy for md files. WARNING: node-pull will delete files from this directory."
   )
   .requiredOption(
     "-i, --img-output-path  <string>",

src/pull.ts

@@ -46,8 +46,9 @@ export async function notionPull(options: any): Promise<void> {
   layoutStrategy = new HierarchicalNamedLayoutStrategy();
   //layoutStrategy = new FlatGuidLayoutStrategy();
 
-  layoutStrategy.setRootDirectoryForMarkdown(markdownOutputPath);
   await fs.mkdir(markdownOutputPath, { recursive: true });
+  layoutStrategy.setRootDirectoryForMarkdown(markdownOutputPath);
+
   console.log("Connecting to Notion...");
   // About the complication here of getting all the pages first and then output
   // them all. It would be simpler to just do it all in one pass, however the