incorporate reviews round 2

.github/workflows/ci.yml

@@ -39,10 +39,10 @@ jobs:
                   cache: npm
             - name: Install npm packages
               run: npm ci
+            - name: Compile JavaScript assets
+              run: npm run build --workspace=memory-viz
             - name: Run jest tests
-              run: |
-                  npm run build --workspace=memory-viz
-                  npm run test-cov
+              run: npm run test-cov
             - name: Coveralls (jest)
               uses: coverallsapp/github-action@v2
               with:

docs/docs/06-cli.md

@@ -14,7 +14,7 @@ To run the MemoryViz CLI, run:
 $ npx memory-viz <path-to-file>
 ```
 
-replacing `<path-to-file>` with the path to a file containing MemoryViz-compatible JSON. If the file extension is not `.json` or if the file content is not compatible with MemoryViz, an error will be thrown.
+replacing `<path-to-file>` with the path to a file containing MemoryViz-compatible JSON. If the file content is not compatible with MemoryViz, an error will be thrown.
 
 ## Output
 

memory-viz/bin/cli.js

@@ -27,7 +27,7 @@ let data;
 try {
     data = JSON.parse(fileContent);
 } catch (err) {
-    console.error(`Error: Invalid JSON.`);
+    console.error(`Error: Invalid JSON\n${err.message}.`);
     process.exit(1);
 }
 

memory-viz/src/tests/cli.spec.tsx

@@ -58,7 +58,7 @@ describe.each([
     },
     {
         errorType: "invalid json",
-        expectedErrorMessage: "Error: Invalid JSON.",
+        expectedErrorMessage: "Error: Invalid JSON",
     },
     {
         errorType: "invalid memory-viz json",

package.json

@@ -10,7 +10,6 @@
         "docs"
     ],
     "devDependencies": {
-        "child_process": "^1.0.2",
         "husky": "^8.0.3",
         "lint-staged": "^14.0.1",
         "prettier": "2.8.1",