fix(build): Reduce spaces in build output.

lowdefy · Oct 16, 2023 · 2e14c4f · 2e14c4f
1 parent cdd9bdc
commit 2e14c4f
Show file tree

Hide file tree

Showing 19 changed files with 28 additions and 179 deletions.
diff --git a/packages/build/src/build/writeApp.js b/packages/build/src/build/writeApp.js
@@ -16,10 +16,7 @@
 import { serializer } from '@lowdefy/helpers';
 
 async function writeApp({ components, context }) {
-  await context.writeBuildArtifact(
-    'app.json',
-    serializer.serializeToString(components.app ?? {}, { space: 2 })
-  );
+  await context.writeBuildArtifact('app.json', serializer.serializeToString(components.app ?? {}));
 }
 
 export default writeApp;
diff --git a/packages/build/src/build/writeApp.test.js b/packages/build/src/build/writeApp.test.js
@@ -34,14 +34,7 @@ test('writeApp', async () => {
     },
   };
   await writeApp({ components, context });
-  expect(mockWriteBuildArtifact.mock.calls).toEqual([
-    [
-      'app.json',
-      `{
-  "key": "value"
-}`,
-    ],
-  ]);
+  expect(mockWriteBuildArtifact.mock.calls).toEqual([['app.json', `{"key":"value"}`]]);
 });
 
 test('writeApp empty config', async () => {

diff --git a/packages/build/src/build/writeAuth.js b/packages/build/src/build/writeAuth.js
@@ -18,7 +18,7 @@ import { serializer } from '@lowdefy/helpers';
 async function writeAuth({ components, context }) {
   await context.writeBuildArtifact(
     'auth.json',
-    serializer.serializeToString(components.auth ?? {}, { space: 2 })
+    serializer.serializeToString(components.auth ?? {})
   );
 }
 

diff --git a/packages/build/src/build/writeAuth.test.js b/packages/build/src/build/writeAuth.test.js
@@ -34,14 +34,7 @@ test('writeAuth', async () => {
     },
   };
   await writeAuth({ components, context });
-  expect(mockWriteBuildArtifact.mock.calls).toEqual([
-    [
-      'auth.json',
-      `{
-  "key": "value"
-}`,
-    ],
-  ]);
+  expect(mockWriteBuildArtifact.mock.calls).toEqual([['auth.json', `{"key":"value"}`]]);
 });
 
 test('writeAuth empty auth', async () => {

diff --git a/packages/build/src/build/writeConfig.js b/packages/build/src/build/writeConfig.js
@@ -18,7 +18,7 @@ import { serializer } from '@lowdefy/helpers';
 async function writeConfig({ components, context }) {
   await context.writeBuildArtifact(
     'config.json',
-    serializer.serializeToString(components.config ?? {}, { space: 2 })
+    serializer.serializeToString(components.config ?? {})
   );
 }
 

diff --git a/packages/build/src/build/writeConfig.test.js b/packages/build/src/build/writeConfig.test.js
@@ -34,14 +34,7 @@ test('writeConfig', async () => {
     },
   };
   await writeConfig({ components, context });
-  expect(mockWriteBuildArtifact.mock.calls).toEqual([
-    [
-      'config.json',
-      `{
-  "key": "value"
-}`,
-    ],
-  ]);
+  expect(mockWriteBuildArtifact.mock.calls).toEqual([['config.json', `{"key":"value"}`]]);
 });
 
 test('writeConfig empty config', async () => {

diff --git a/packages/build/src/build/writeConnections.js b/packages/build/src/build/writeConnections.js
@@ -24,7 +24,7 @@ async function writeConnections({ components, context }) {
   const writePromises = components.connections.map(async (connection) => {
     await context.writeBuildArtifact(
       `connections/${connection.connectionId}.json`,
-      serializer.serializeToString(connection, { space: 2 })
+      serializer.serializeToString(connection)
     );
   });
   return Promise.all(writePromises);

diff --git a/packages/build/src/build/writeConnections.test.js b/packages/build/src/build/writeConnections.test.js
@@ -43,13 +43,7 @@ test('writeConnections write connection', async () => {
   expect(mockWriteBuildArtifact.mock.calls).toEqual([
     [
       'connections/connection1.json',
-      `{
-  "id": "connection:connection1",
-  "connectionId": "connection1",
-  "properties": {
-    "prop": "val"
-  }
-}`,
+      `{"id":"connection:connection1","connectionId":"connection1","properties":{"prop":"val"}}`,
     ],
   ]);
 });
@@ -71,17 +65,11 @@ test('writeConnections multiple connection', async () => {
   expect(mockWriteBuildArtifact.mock.calls).toEqual([
     [
       'connections/connection1.json',
-      `{
-  "id": "connection:connection1",
-  "connectionId": "connection1"
-}`,
+      `{"id":"connection:connection1","connectionId":"connection1"}`,
     ],
     [
       'connections/connection2.json',
-      `{
-  "id": "connection:connection2",
-  "connectionId": "connection2"
-}`,
+      `{"id":"connection:connection2","connectionId":"connection2"}`,
     ],
   ]);
 });

diff --git a/packages/build/src/build/writeGlobal.js b/packages/build/src/build/writeGlobal.js
@@ -23,10 +23,7 @@ async function writeGlobal({ components, context }) {
   if (!type.isObject(components.global)) {
     throw new Error('Global is not an object.');
   }
-  await context.writeBuildArtifact(
-    'global.json',
-    serializer.serializeToString(components.global, { space: 2 })
-  );
+  await context.writeBuildArtifact('global.json', serializer.serializeToString(components.global));
 }
 
 export default writeGlobal;
diff --git a/packages/build/src/build/writeGlobal.test.js b/packages/build/src/build/writeGlobal.test.js
@@ -34,14 +34,7 @@ test('writeGlobal', async () => {
     },
   };
   await writeGlobal({ components, context });
-  expect(mockWriteBuildArtifact.mock.calls).toEqual([
-    [
-      'global.json',
-      `{
-  "key": "value"
-}`,
-    ],
-  ]);
+  expect(mockWriteBuildArtifact.mock.calls).toEqual([['global.json', `{"key":"value"}`]]);
 });
 
 test('writeGlobal empty global', async () => {

diff --git a/packages/build/src/build/writeMaps.js b/packages/build/src/build/writeMaps.js
@@ -22,14 +22,8 @@ async function writeMaps({ context }) {
   if (!type.isObject(context.refMap)) {
     throw new Error('refMap is not an object.');
   }
-  await context.writeBuildArtifact(
-    'keyMap.json',
-    serializer.serializeToString(context.keyMap, { space: 0 })
-  );
-  await context.writeBuildArtifact(
-    'refMap.json',
-    serializer.serializeToString(context.refMap, { space: 0 })
-  );
+  await context.writeBuildArtifact('keyMap.json', serializer.serializeToString(context.keyMap));
+  await context.writeBuildArtifact('refMap.json', serializer.serializeToString(context.refMap));
 }
 
 export default writeMaps;
diff --git a/packages/build/src/build/writeMenus.js b/packages/build/src/build/writeMenus.js
@@ -20,10 +20,7 @@ async function writeMenus({ components, context }) {
   if (!type.isArray(components.menus)) {
     throw new Error('Menus is not an array.');
   }
-  await context.writeBuildArtifact(
-    'menus.json',
-    serializer.serializeToString(components.menus, { space: 2 })
-  );
+  await context.writeBuildArtifact('menus.json', serializer.serializeToString(components.menus));
 }
 
 export default writeMenus;
diff --git a/packages/build/src/build/writeMenus.test.js b/packages/build/src/build/writeMenus.test.js
@@ -39,16 +39,7 @@ test('writeMenus', async () => {
   };
   await writeMenus({ components, context });
   expect(mockWriteBuildArtifact.mock.calls).toEqual([
-    [
-      'menus.json',
-      `[
-  {
-    "id": "menu:default",
-    "menuId": "default",
-    "links": []
-  }
-]`,
-    ],
+    ['menus.json', `[{"id":"menu:default","menuId":"default","links":[]}]`],
   ]);
 });
 

diff --git a/packages/build/src/build/writePages.js b/packages/build/src/build/writePages.js
@@ -18,7 +18,7 @@ import { serializer } from '@lowdefy/helpers';
 async function writePage({ page, context }) {
   await context.writeBuildArtifact(
     `pages/${page.pageId}/${page.pageId}.json`,
-    serializer.serializeToString(page ?? {}, { space: 2 })
+    serializer.serializeToString(page ?? {})
   );
 }
 

diff --git a/packages/build/src/build/writePages.test.js b/packages/build/src/build/writePages.test.js
@@ -42,12 +42,7 @@ test('writePages write page', async () => {
   expect(mockWriteBuildArtifact.mock.calls).toEqual([
     [
       'pages/page1/page1.json',
-      `{
-  "id": "page:page1",
-  "pageId": "page1",
-  "blockId": "page1",
-  "requests": []
-}`,
+      `{"id":"page:page1","pageId":"page1","blockId":"page1","requests":[]}`,
     ],
   ]);
 });
@@ -73,21 +68,11 @@ test('writePages multiple pages', async () => {
   expect(mockWriteBuildArtifact.mock.calls).toEqual([
     [
       'pages/page1/page1.json',
-      `{
-  "id": "page:page1",
-  "pageId": "page1",
-  "blockId": "page1",
-  "requests": []
-}`,
+      `{"id":"page:page1","pageId":"page1","blockId":"page1","requests":[]}`,
     ],
     [
       'pages/page2/page2.json',
-      `{
-  "id": "page:page2",
-  "pageId": "page2",
-  "blockId": "page2",
-  "requests": []
-}`,
+      `{"id":"page:page2","pageId":"page2","blockId":"page2","requests":[]}`,
     ],
   ]);
 });

diff --git a/packages/build/src/build/writeRequests.js b/packages/build/src/build/writeRequests.js
@@ -20,7 +20,7 @@ async function writeRequestsOnPage({ page, context }) {
     page.requests.map(async (request) => {
       await context.writeBuildArtifact(
         `pages/${page.pageId}/requests/${request.requestId}.json`,
-        serializer.serializeToString(request ?? {}, { space: 2 })
+        serializer.serializeToString(request ?? {})
       );
       delete request.properties;
       delete request.type;

diff --git a/packages/build/src/build/writeRequests.test.js b/packages/build/src/build/writeRequests.test.js
@@ -52,20 +52,7 @@ test('writeRequests write request', async () => {
   expect(mockWriteBuildArtifact.mock.calls).toEqual([
     [
       'pages/page1/requests/request1.json',
-      `{
-  "id": "request:page1:request1",
-  "requestId": "request1",
-  "pageId": "page1",
-  "connectionId": "connection1",
-  "auth": {
-    "public": true
-  },
-  "type": "Request",
-  "payload": {},
-  "properties": {
-    "key": "value"
-  }
-}`,
+      `{"id":"request:page1:request1","requestId":"request1","pageId":"page1","connectionId":"connection1","auth":{"public":true},"type":"Request","payload":{},"properties":{"key":"value"}}`,
     ],
   ]);
 });
@@ -105,37 +92,11 @@ test('writeRequests write multiple requests on a page', async () => {
   expect(mockWriteBuildArtifact.mock.calls).toEqual([
     [
       'pages/page1/requests/request1.json',
-      `{
-  "id": "request:page1:request1",
-  "requestId": "request1",
-  "pageId": "page1",
-  "connectionId": "connection1",
-  "auth": {
-    "public": true
-  },
-  "type": "Request",
-  "payload": {},
-  "properties": {
-    "key": "value"
-  }
-}`,
+      `{"id":"request:page1:request1","requestId":"request1","pageId":"page1","connectionId":"connection1","auth":{"public":true},"type":"Request","payload":{},"properties":{"key":"value"}}`,
     ],
     [
       'pages/page1/requests/request2.json',
-      `{
-  "id": "request:page1:request2",
-  "requestId": "request2",
-  "pageId": "page1",
-  "connectionId": "connection1",
-  "auth": {
-    "public": true
-  },
-  "type": "Request",
-  "payload": {},
-  "properties": {
-    "key": "value"
-  }
-}`,
+      `{"id":"request:page1:request2","requestId":"request2","pageId":"page1","connectionId":"connection1","auth":{"public":true},"type":"Request","payload":{},"properties":{"key":"value"}}`,
     ],
   ]);
 });
@@ -181,37 +142,11 @@ test('writeRequests write requests on a for multiple pages', async () => {
   expect(mockWriteBuildArtifact.mock.calls).toEqual([
     [
       'pages/page1/requests/request1.json',
-      `{
-  "id": "request:page1:request1",
-  "requestId": "request1",
-  "pageId": "page1",
-  "connectionId": "connection1",
-  "auth": {
-    "public": true
-  },
-  "type": "Request",
-  "payload": {},
-  "properties": {
-    "key": "value"
-  }
-}`,
+      `{"id":"request:page1:request1","requestId":"request1","pageId":"page1","connectionId":"connection1","auth":{"public":true},"type":"Request","payload":{},"properties":{"key":"value"}}`,
     ],
     [
       'pages/page2/requests/request1.json',
-      `{
-  "id": "request:page2:request1",
-  "requestId": "request1",
-  "pageId": "page2",
-  "connectionId": "connection1",
-  "auth": {
-    "public": true
-  },
-  "type": "Request",
-  "payload": {},
-  "properties": {
-    "key": "value"
-  }
-}`,
+      `{"id":"request:page2:request1","requestId":"request1","pageId":"page2","connectionId":"connection1","auth":{"public":true},"type":"Request","payload":{},"properties":{"key":"value"}}`,
     ],
   ]);
 });

diff --git a/packages/build/src/build/writeTypes.js b/packages/build/src/build/writeTypes.js
@@ -18,7 +18,7 @@ import { serializer } from '@lowdefy/helpers';
 async function writeTypes({ components, context }) {
   await context.writeBuildArtifact(
     'types.json',
-    serializer.serializeToString(components.types ?? {}, { space: 2 })
+    serializer.serializeToString(components.types ?? {})
   );
 }
 

diff --git a/packages/build/src/build/writeTypes.test.js b/packages/build/src/build/writeTypes.test.js
@@ -33,12 +33,5 @@ test('writeTypes', async () => {
     type: 'value',
   };
   await writeTypes({ components, context });
-  expect(mockWriteBuildArtifact.mock.calls).toEqual([
-    [
-      'types.json',
-      `{
-  "type": "value"
-}`,
-    ],
-  ]);
+  expect(mockWriteBuildArtifact.mock.calls).toEqual([['types.json', `{"type":"value"}`]]);
 });