Merge pull request #5 from jspm/fix/ensure-file-sync

(fix): ensureFileSync and drop cjs build

Author: JayaKrishnaNamburu

chompfile.toml

@@ -1,5 +1,7 @@
 version = 0.1
 
+default-task = 'build'
+
 [[task]]
 name = 'build'
 run = 'tsup src/loader.ts --format esm,cjs --dts'
@@ -42,8 +44,6 @@ name = 'pre-commit'
 run = 'lint-staged'
 deps = ['src/#.ts', 'tests/loader.test.ts']
 
-[[task]]
-
 [[task]]
 name = 'prepare'
 run = 'husky install'

package.json

@@ -5,13 +5,12 @@
   "author": "Jaya Krishna Namburu <[email protected]>",
   "type": "module",
   "main": "dist/loader.js",
-  "module": "dist/loader.js",
   "types": "dist/loader.d.ts",
   "exports": {
-    ".": "dist/loader.cjs"
+    ".": "dist/loader.js"
   },
   "scripts": {
-    "build": "tsup src/loader.ts --format esm,cjs --dts",
+    "build": "tsup src/loader.ts --format esm --dts",
     "clean": "rimraf dist",
     "commit": "git-cz",
     "commit-msg": "commitlint --edit $1",

src/loader.ts

@@ -1,5 +1,5 @@
-import { accessSync, existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
+import { join, dirname } from "node:path";
 import { fileURLToPath } from "node:url";
 import { ImportMap } from "@jspm/import-map";
 import fetch from "node-fetch";
@@ -8,16 +8,41 @@ import { ConstructCachePath, Context, NextResolve, UrlType } from './types'
 // hoist the cache map
 const cacheMap = new Map();
 
+/**
+ * nsureDirSync
+ * @description a function to ensure the dis exists
+ * @param dirPath
+*/
+
+function ensureDirSync(dirPath: string) {
+  if (existsSync(dirPath)) {
+    return;
+  }
+
+  const parentDir = dirname(dirPath);
+  if (parentDir !== dirPath) {
+    ensureDirSync(parentDir);
+  }
+
+  mkdirSync(dirPath);
+}
+
 /**
  * ensureFileSync
  * @description a convenience function to ensure a file exists
  * @param path
  */
-export function ensureFileSync(path: string) {
+function ensureFileSync(path: string) {
+  const dirPath = dirname(path);
+
+  if (!existsSync(dirPath)) {
+    ensureDirSync(dirPath);
+  }
+
   try {
-    accessSync(path);
-  } catch (err) {
-    writeFileSync(path, '');
+    writeFileSync(path, '', { flag: 'wx' });
+  } catch {
+    console.log(`Failed in creating ${path}`)
   }
 }
 
@@ -166,13 +191,15 @@ export const parseModule = async (specifier: string, modulePath: string) => {
   const { protocol } = new URL(modulePath);
   const isNode = protocol === "node:";
   const isFile = protocol === "file:";
+
   if (isNode || isFile) return specifier;
 
   const cachePath = constructCachePath({ cache, modulePath })
+
   cacheMap.set(`file://${cachePath}`, modulePath);
   if (existsSync(cachePath)) return cachePath
-
   const code = await (await fetch(modulePath)).text();
+
   ensureFileSync(cachePath);
   writeFileSync(cachePath, code);
 

tests/e2e/test-prettier.js

@@ -0,0 +1,30 @@
+import standalone from 'prettier/standalone'
+import parserBabel from 'prettier/parser-babel'
+const { format } = standalone
+
+const CODE = String(`
+  function HelloWorld({greeting = "hello", greeted = '"World"', silent = false, onMouseOver,}) {
+
+    if(!greeting){return null};
+
+       // TODO: Don't use random in render
+    let num = Math.floor (Math.random() * 1E+7).toString().replace(/\.\d+/ig, "")
+
+    return <div className='HelloWorld' onMouseOver={onMouseOver}>
+
+      <strong>{ greeting.slice( 0, 1 ).toUpperCase() + greeting.slice(1).toLowerCase() }</strong>
+      {greeting.endsWith(",") ? " " : <span style={{color: '\grey'}}>", "</span> }
+      <em>
+	{ greeted }
+	</em>
+      { (silent)
+        ? "."
+        : "!"}
+
+      </div>;
+
+  }
+`)
+
+const result = format(CODE, { parser: 'babel', plugins: [parserBabel] })
+console.log(result)