feat(cli): support multiple files on pull/push

Support pulling/pushing multiple files at once with the
`@mermaidchart/cli` CLI, e.g. like `mermaid-chart pull file1 file2`.

Author: aloisklink

packages/cli/src/commander.test.ts

@@ -218,9 +218,13 @@ describe('link', () => {
 
 describe('pull', () => {
   const diagram = 'test/output/connected-diagram.mmd';
+  const diagram2 = 'test/output/connected-diagram-2.mmd';
 
   beforeEach(async () => {
-    await copyFile('test/fixtures/connected-diagram.mmd', diagram);
+    await Promise.all([
+      copyFile('test/fixtures/connected-diagram.mmd', diagram),
+      copyFile('test/fixtures/connected-diagram.mmd', diagram2),
+    ]);
   });
 
   it('should fail if MermaidChart document has not yet been linked', async () => {
@@ -243,7 +247,7 @@ describe('pull', () => {
     ).rejects.toThrowError(`Diagram at ${diagram} has no code`);
   });
 
-  it('should pull document and add a `id:` field to frontmatter', async () => {
+  it('should pull documents and add a `id:` field to frontmatter', async () => {
     const { program } = mockedProgram();
 
     const mockedDiagram = {
@@ -255,22 +259,30 @@ title: My cool flowchart
       A[I've been updated!]`,
     };
 
-    vi.mocked(MermaidChart.prototype.getDocument).mockResolvedValueOnce(mockedDiagram);
+    vi.mocked(MermaidChart.prototype.getDocument).mockResolvedValue(mockedDiagram);
 
-    await program.parseAsync(['--config', CONFIG_AUTHED, 'pull', diagram], { from: 'user' });
+    await program.parseAsync(['--config', CONFIG_AUTHED, 'pull', diagram, diagram2], {
+      from: 'user',
+    });
 
-    const diagramContents = await readFile(diagram, { encoding: 'utf8' });
+    for (const file of [diagram, diagram2]) {
+      const diagramContents = await readFile(file, { encoding: 'utf8' });
 
-    expect(diagramContents).toContain(`id: ${mockedDiagram.documentID}`);
-    expect(diagramContents).toContain("flowchart TD\n      A[I've been updated!]");
+      expect(diagramContents).toContain(`id: ${mockedDiagram.documentID}`);
+      expect(diagramContents).toContain("flowchart TD\n      A[I've been updated!]");
+    }
   });
 });
 
 describe('push', () => {
   const diagram = 'test/output/connected-diagram.mmd';
+  const diagram2 = 'test/output/connected-diagram-2.mmd';
 
   beforeEach(async () => {
-    await copyFile('test/fixtures/connected-diagram.mmd', diagram);
+    await Promise.all([
+      copyFile('test/fixtures/connected-diagram.mmd', diagram),
+      copyFile('test/fixtures/connected-diagram.mmd', diagram2),
+    ]);
   });
 
   it('should fail if MermaidChart document has not yet been linked', async () => {
@@ -283,16 +295,18 @@ describe('push', () => {
     ).rejects.toThrowError('Diagram at test/fixtures/unsynced.mmd has no id');
   });
 
-  it('should push document and remove the `id:` field front frontmatter', async () => {
+  it('should push documents and remove the `id:` field front frontmatter', async () => {
     const { program } = mockedProgram();
 
-    vi.mocked(MermaidChart.prototype.getDocument).mockResolvedValueOnce(mockedEmptyDiagram);
+    vi.mocked(MermaidChart.prototype.getDocument).mockResolvedValue(mockedEmptyDiagram);
 
     await expect(readFile(diagram, { encoding: 'utf8' })).resolves.not.toContain(/^id:/);
 
-    await program.parseAsync(['--config', CONFIG_AUTHED, 'push', diagram], { from: 'user' });
+    await program.parseAsync(['--config', CONFIG_AUTHED, 'push', diagram, diagram2], {
+      from: 'user',
+    });
 
-    expect(vi.mocked(MermaidChart.prototype.setDocument)).toHaveBeenCalledOnce();
+    expect(vi.mocked(MermaidChart.prototype.setDocument)).toHaveBeenCalledTimes(2);
     expect(vi.mocked(MermaidChart.prototype.setDocument)).toHaveBeenCalledWith(
       expect.objectContaining({
         code: expect.not.stringContaining('id:'),

packages/cli/src/commander.ts

@@ -196,40 +196,48 @@ function linkCmd() {
 
 function pullCmd() {
   return createCommand('pull')
-    .description('Pulls a document from from Mermaid Chart')
-    .addArgument(new Argument('<path>', 'The path of the file to pull.'))
-    .option('--check', 'Check whether the local file would be overwrited')
-    .action(async (path, options, command) => {
+    .description('Pulls documents from Mermaid Chart')
+    .addArgument(new Argument('<path...>', 'The paths of the files to pull.'))
+    .option('--check', 'Check whether the local files would be overwrited')
+    .action(async (paths, options, command) => {
       const optsWithGlobals = command.optsWithGlobals<CommonOptions>();
       const client = await createClient(optsWithGlobals);
-      const text = await readFile(path, { encoding: 'utf8' });
-
-      const newFile = await pull(text, client, { title: path });
-
-      if (text === newFile) {
-        console.log(`✅ - ${path} is up to date`);
-      } else {
-        if (options['check']) {
-          console.log(`❌ - ${path} would be updated`);
-          process.exitCode = 1;
-        } else {
-          await writeFile(path, newFile, { encoding: 'utf8' });
-          console.log(`✅ - ${path} was updated`);
-        }
-      }
+      await Promise.all(
+        paths.map(async (path) => {
+          const text = await readFile(path, { encoding: 'utf8' });
+
+          const newFile = await pull(text, client, { title: path });
+
+          if (text === newFile) {
+            console.log(`✅ - ${path} is up to date`);
+          } else {
+            if (options['check']) {
+              console.log(`❌ - ${path} would be updated`);
+              process.exitCode = 1;
+            } else {
+              await writeFile(path, newFile, { encoding: 'utf8' });
+              console.log(`✅ - ${path} was updated`);
+            }
+          }
+        }),
+      );
     });
 }
 
 function pushCmd() {
   return createCommand('push')
-    .description('Push a local diagram to Mermaid Chart')
-    .addArgument(new Argument('<path>', 'The path of the file to push.'))
-    .action(async (path, _options, command) => {
+    .description('Push local diagrams to Mermaid Chart')
+    .addArgument(new Argument('<path...>', 'The paths of the files to push.'))
+    .action(async (paths, _options, command) => {
       const optsWithGlobals = command.optsWithGlobals<CommonOptions>();
       const client = await createClient(optsWithGlobals);
-      const text = await readFile(path, { encoding: 'utf8' });
+      await Promise.all(
+        paths.map(async (path) => {
+          const text = await readFile(path, { encoding: 'utf8' });
 
-      await push(text, client, { title: path });
+          await push(text, client, { title: path });
+        }),
+      );
     });
 }