Prototype basic publish command (just pushes all changes to git)

packages/foam-cli/package.json

@@ -62,10 +62,9 @@
   "scripts": {
     "cli": "./bin/run",
     "postpack": "rm -f oclif.manifest.json",
-    "posttest": "eslint . --ext .ts --config .eslintrc",
     "prepack": "rm -rf lib && tsc -b && oclif-dev manifest && oclif-dev readme",
     "test": "jest",
     "version": "oclif-dev readme && git add README.md"
   },
   "types": "lib/index.d.ts"
-}
+}

packages/foam-cli/src/commands/janitor.ts

@@ -59,7 +59,7 @@ Successfully generated link references and heading!
           return writeFileToDisk(note.path, file);
         }
 
-        return null;
+        return Promise.resolve(null);
       })
 
       await Promise.all(fileWritePromises);

packages/foam-cli/src/commands/migrate.ts

@@ -49,7 +49,7 @@ Successfully generated link references and heading!
         if (kebabCasedFileName) {
           return renameFile(note.path, kebabCasedFileName);
         }
-        return null;
+        return Promise.resolve(null);
       })
 
       await Promise.all(fileRename);
@@ -80,7 +80,7 @@ Successfully generated link references and heading!
           return writeFileToDisk(note.path, file);
         }
 
-        return null;
+        return Promise.resolve(null);
       }))
 
       await Promise.all(fileWritePromises);

packages/foam-cli/src/commands/publish.ts

@@ -0,0 +1,63 @@
+import {Command, flags} from '@oclif/command'
+import { execSync } from 'child_process';
+import * as ora from 'ora';
+
+
+export default class Publish extends Command {
+  static description = 'Push all changes to git repository';
+
+  static examples = [
+    `$ foam publish -m "Optional log message"`,
+  ]
+
+  static flags = {
+    message: flags.string({
+      char: 'm',
+      description: "optional message"
+    }),
+    remote: flags.string({
+      char: 'r',
+      description: "remote"
+    }),
+    branch: flags.string({
+      char: 'b',
+      description: "branch"
+    })
+  }
+
+  async execWithSpinner(command: string, message: string) {
+    const spinner = ora(message).start();
+
+    // @todo handle errors
+    const response = execSync(command).toString();
+
+    spinner.succeed(`${message} Done!`);
+
+    return response;
+  }
+
+  async printPublishInfo(remote: string) {
+    // @todo actually get this data from GH API
+
+    const [, remotePath] = execSync(`git remote get-url ${remote}`).toString().trim().split(':');
+    const [repo, org] = remotePath.split('/').reverse();
+    console.log('');
+    console.log(`🎉 Your changes will be available shortly at https://${org}.github.io/${repo.replace('.git', '')}`);
+    console.log('');
+
+  }
+
+  async run() {
+    const {flags} = this.parse(Publish);
+
+    // @todo improve
+    const message = flags.message || 'foam publish';
+    const remote = flags.remote || 'origin';
+    const branch = flags.branch || 'master';
+
+    await this.execWithSpinner(`git add -A`, 'Staging changes...');
+    await this.execWithSpinner(`git commit -m "${message}"`, 'Creating a commit...');
+    await this.execWithSpinner(`git push ${remote} ${branch}`, "Publishing...");
+    await this.printPublishInfo(remote);
+  }
+}

packages/foam-cli/src/utils/rename-file.ts

@@ -1,9 +1,15 @@
-import * as fs from 'fs'
+import * as fs from 'fs';
+import * as path from 'path';
 
+/**
+ * 
+ * @param fileUri absolute path for the file that needs to renamed
+ * @param newFileName  "new file name" without the extension
+ */
 export const renameFile = async (fileUri: string, newFileName: string) => {
-  const fileName = fileUri.split('/').pop();
-  const extension = fileName?.split('.').pop();
-  const newFileUri = fileUri.replace(`${fileName}`, `${newFileName}.${extension}`);
+  const dirName = path.dirname(fileUri);
+  const extension = path.extname(fileUri);
+  const newFileUri = path.join(dirName, `${newFileName}${extension}`);
 
   return fs.promises.rename(fileUri, newFileUri);
 }

packages/foam-cli/src/utils/write-file-to-disk.ts

@@ -1,8 +1,5 @@
 import * as fs from 'fs';
 
-export const writeFileToDisk = async (fileUri: string, data: string): Promise<Boolean> => {
-  return fs.promises.writeFile(fileUri, data).then(() => true).catch(err => {
-    console.log('error while writing to file: ', err)
-    return false;
-  })
+export const writeFileToDisk = async (fileUri: string, data: string) => {
+  return fs.promises.writeFile(fileUri, data);
 }

packages/foam-cli/test/rename-file.test.ts

@@ -3,6 +3,8 @@ import { renameFile } from '../src/utils/rename-file'
 import * as fs from 'fs';
 import mockFS from 'mock-fs';
 
+const doesFileExist = (path) => fs.promises.access(path).then(() => true).catch(() => false);
+
 describe('renameFile', () => {
 
   const fileUri = './test/oldFileName.md';
@@ -16,10 +18,11 @@ describe('renameFile', () => {
   });
 
   it('should rename existing file', async () => {
-    expect(fs.existsSync(fileUri)).toBe(true);
+    expect(await doesFileExist(fileUri)).toBe(true);
+
     renameFile(fileUri, 'new-file-name');
 
-    expect(fs.existsSync(fileUri)).toBe(false);
-    expect(fs.existsSync('./test/new-file-name.md')).toBe(true);
+    expect(await doesFileExist(fileUri)).toBe(false);
+    expect(await doesFileExist('./test/new-file-name.md')).toBe(true);
   });
 });

packages/foam-cli/test/write-file-to-disk.test.ts

@@ -18,7 +18,7 @@ describe('writeFileToDisk', () => {
   it('should overrwrite existing file in the disk with the new data', async () => {
     const expected = `content in the new file`;
     await writeFileToDisk(fileUri, expected);
-    const actual = fs.readFileSync(fileUri, { encoding: 'utf8' });
+    const actual = await fs.promises.readFile(fileUri, { encoding: 'utf8' });
     expect(actual).toBe(expected);
   });
 })