feat(remote/github): add dry run option

src/config/args.ts

@@ -14,6 +14,7 @@ interface Parser<TData> {
 
 export interface ParsedArgs {
   config: string;
+  dryrun: boolean;
   remote: string;
 }
 
@@ -38,6 +39,12 @@ export function createParser(modeset: Modeback): Parser<ParsedArgs> {
         demand: true,
         type: 'string',
       },
+      dryrun: {
+        alias: ['d'],
+        default: true,
+        demand: false,
+        type: 'boolean',
+      },
       remote: {
         alias: ['r'],
         demand: true,

src/main.ts

@@ -57,7 +57,10 @@ export async function main(argv: Array<string>): Promise<number> {
   });
 
   for (const project of config.projects) {
-    const remote = new GithubRemote(project.remote);
+    const remote = new GithubRemote({
+      ...project.remote,
+      dryrun: args.dryrun,
+    });
     await remote.connect();
 
     // mode switch

src/remote/github.ts

@@ -13,6 +13,9 @@ export class GithubRemote implements Remote {
 
   constructor(options: RemoteOptions) {
     this.options = options;
+
+    /* eslint-disable-next-line */
+    console.log('options:', options);
   }
 
   public async connect() {
@@ -46,13 +49,15 @@ export class GithubRemote implements Remote {
 
     // TODO: check if the label already exists
 
-    await mustExist(this.request).issues.createLabel({
-      color: options.color,
-      description: options.desc,
-      name: options.name,
-      owner: path.owner,
-      repo: path.repo,
-    });
+    if (this.forReal) {
+      await mustExist(this.request).issues.createLabel({
+        color: options.color,
+        description: options.desc,
+        name: options.name,
+        owner: path.owner,
+        repo: path.repo,
+      });
+    }
 
     return options;
   }
@@ -62,11 +67,13 @@ export class GithubRemote implements Remote {
 
     // TODO: check if the label is in use
 
-    await mustExist(this.request).issues.deleteLabel({
-      name: options.name,
-      owner: path.owner,
-      repo: path.repo,
-    });
+    if (this.forReal) {
+      await mustExist(this.request).issues.deleteLabel({
+        name: options.name,
+        owner: path.owner,
+        repo: path.repo,
+      });
+    }
 
     return options;
   }
@@ -81,33 +88,39 @@ export class GithubRemote implements Remote {
 
   public async listIssues(options: ProjectQuery): Promise<Array<IssueUpdate>> {
     const path = await this.splitProject(options.project);
-    const repo = await mustExist(this.request).issues.listForRepo(path);
 
     const issues: Array<IssueUpdate> = [];
-    for (const issue of repo.data) {
-      issues.push({
-        issue: issue.id.toString(10),
-        labels: issue.labels.map((l) => l.name),
-        name: issue.title,
-        project: options.project,
-      });
+
+    if (this.forReal) {
+      const repo = await mustExist(this.request).issues.listForRepo(path);
+      for (const issue of repo.data) {
+        issues.push({
+          issue: issue.id.toString(10),
+          labels: issue.labels.map((l) => l.name),
+          name: issue.title,
+          project: options.project,
+        });
+      }
     }
 
     return issues;
   }
 
   public async listLabels(options: ProjectQuery): Promise<Array<LabelUpdate>> {
     const path = await this.splitProject(options.project);
-    const repo = await mustExist(this.request).issues.listLabelsForRepo(path);
 
     const labels: Array<LabelUpdate> = [];
-    for (const label of repo.data) {
-      labels.push({
-        color: label.color,
-        desc: label.description,
-        name: label.name,
-        project: options.project,
-      });
+
+    if (this.forReal) {
+      const repo = await mustExist(this.request).issues.listLabelsForRepo(path);
+      for (const label of repo.data) {
+        labels.push({
+          color: label.color,
+          desc: label.description,
+          name: label.name,
+          project: options.project,
+        });
+      }
     }
 
     return labels;
@@ -119,19 +132,28 @@ export class GithubRemote implements Remote {
 
   public async updateLabel(options: LabelUpdate): Promise<LabelUpdate> {
     const path = await this.splitProject(options.project);
-    const data = await mustExist(this.request).issues.updateLabel({
-      color: options.color,
-      description: options.desc,
-      name: options.name,
-      owner: path.owner,
-      repo: path.repo,
-    });
 
-    return {
-      color: data.data.color,
-      desc: data.data.description,
-      name: data.data.name,
-      project: options.project,
-    };
+    if (this.forReal) {
+      const data = await mustExist(this.request).issues.updateLabel({
+        color: options.color,
+        description: options.desc,
+        name: options.name,
+        owner: path.owner,
+        repo: path.repo,
+      });
+
+      return {
+        color: data.data.color,
+        desc: data.data.description,
+        name: data.data.name,
+        project: options.project,
+      };
+    } else {
+      return options;
+    }
+  }
+
+  protected get forReal(): boolean {
+    return !this.options.dryrun;
   }
 }

src/remote/index.ts

@@ -25,7 +25,19 @@ export interface LabelUpdate extends LabelQuery {
 }
 
 export interface RemoteOptions {
+  /**
+   * Arbitrary key-value data for this remote, usually credentials and base URLs.
+   */
   data: Record<string, string>;
+
+  /**
+   * If set, do not make any real changes.
+   */
+  dryrun: boolean;
+
+  /**
+   * Remote class/type name.
+   */
   type: string;
 }