Add auto indentation for modules.json to template generator app for g…

…it tracking
source-academy · Mar 22, 2021 · a58c40a · a58c40a
1 parent e45c88f
commit a58c40a
Show file tree

Hide file tree

Showing 3 changed files with 54 additions and 38 deletions.
diff --git a/modules.json b/modules.json
@@ -1 +1,12 @@
-{"repeat":{"tabs":["Repeat"]}, "pix_n_flix":{"tabs":["Pixnflix"]}}
+{
+  "repeat": {
+    "tabs": [
+      "Repeat"
+    ]
+  },
+  "pix_n_flix": {
+    "tabs": [
+      "Pixnflix"
+    ]
+  }
+}
diff --git a/templates/module.js b/templates/module.js
@@ -4,18 +4,8 @@ const print = require('./print');
 const utilities = require('./utilities');
 const manifest = require('../modules.json');
 
-async function addNew() {
-  const moduleName = await askModuleName();
-  const bundleDestination = `${paths.root}/src/bundles/${moduleName}`;
-  await fs.mkdir(bundleDestination, { recursive: true });
-  await fs.copyFile(paths.bundleTemplate, `${bundleDestination}/index.ts`);
-  await fs.writeFile(
-    paths.manifest,
-    JSON.stringify({ ...manifest, [moduleName]: { tabs: [] } })
-  );
-  print.success(
-    `Bundle for module ${moduleName} created at ${bundleDestination}.`
-  );
+function check(moduleName) {
+  return Object.keys(manifest).includes(moduleName);
 }
 
 async function askModuleName() {
@@ -33,8 +23,18 @@ async function askModuleName() {
   return name;
 }
 
-function check(moduleName) {
-  return Object.keys(manifest).includes(moduleName);
+async function addNew() {
+  const moduleName = await askModuleName();
+  const bundleDestination = `${paths.root}/src/bundles/${moduleName}`;
+  await fs.mkdir(bundleDestination, { recursive: true });
+  await fs.copyFile(paths.bundleTemplate, `${bundleDestination}/index.ts`);
+  await fs.writeFile(
+    paths.manifest,
+    JSON.stringify({ ...manifest, [moduleName]: { tabs: [] } }, null, 2)
+  );
+  print.success(
+    `Bundle for module ${moduleName} created at ${bundleDestination}.`
+  );
 }
 
 module.exports = {

diff --git a/templates/tab.js b/templates/tab.js
@@ -5,28 +5,13 @@ const utilities = require('./utilities');
 const modules = require('./module');
 const manifest = require('../modules.json');
 
-const existingTabs = Object.keys(manifest).reduce((accumulator, current) => {
-  return accumulator.concat(manifest[current].tabs);
-}, []);
+const existingTabs = Object.keys(manifest).reduce(
+  (accumulator, current) => accumulator.concat(manifest[current].tabs),
+  []
+);
 
-async function addNew() {
-  const moduleName = await askModuleName();
-  const tabName = await askTabName();
-
-  // Copy module tab template into correct destination and show success message
-  const tabDestination = `${paths.root}/src/tabs/${tabName}`;
-  await fs.mkdir(tabDestination, { recursive: true });
-  await fs.copyFile(paths.tabTemplate, `${tabDestination}/index.tsx`);
-  await fs.writeFile(
-    paths.manifest,
-    JSON.stringify({
-      ...manifest,
-      [moduleName]: { tabs: [...manifest[moduleName].tabs, tabName] },
-    })
-  );
-  print.success(
-    `Tab ${tabName} for module ${moduleName} created at ${tabDestination}.`
-  );
+function check(tabName) {
+  return existingTabs.includes(tabName);
 }
 
 async function askModuleName() {
@@ -53,8 +38,28 @@ async function askTabName() {
   return name;
 }
 
-function check(tabName) {
-  return existingTabs.includes(tabName);
+async function addNew() {
+  const moduleName = await askModuleName();
+  const tabName = await askTabName();
+
+  // Copy module tab template into correct destination and show success message
+  const tabDestination = `${paths.root}/src/tabs/${tabName}`;
+  await fs.mkdir(tabDestination, { recursive: true });
+  await fs.copyFile(paths.tabTemplate, `${tabDestination}/index.tsx`);
+  await fs.writeFile(
+    paths.manifest,
+    JSON.stringify(
+      {
+        ...manifest,
+        [moduleName]: { tabs: [...manifest[moduleName].tabs, tabName] },
+      },
+      null,
+      2
+    )
+  );
+  print.success(
+    `Tab ${tabName} for module ${moduleName} created at ${tabDestination}.`
+  );
 }
 
 module.exports = {