Convert to ESM module (#93)

* Update dependencies

* Update to ESM module

* Update GH Actions

* Migrate eslint config file

* npm run lint-fix

Author: mheap

.eslintrc

@@ -2,20 +2,23 @@ name: Node.js CI
 
 on: [push]
 
+permissions:
+  contents: read
+
 jobs:
   build:
     runs-on: ubuntu-latest
 
     strategy:
       matrix:
-        node-version: [10.x, 12.x, 14.x]
+        node-version: [18, 20, 22]
 
     env:
       CI: true
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
       - run: npm ci

.github/workflows/ci.yml

@@ -3,15 +3,19 @@ name: NPM Publish
 on:
   release:
     types: [published]
+
+permissions:
+  contents: write
+
 jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
       with:
         ref: ${{ github.event.release.target_commitish }}
     - name: Use Node.js 12
-      uses: actions/setup-node@v1
+      uses: actions/setup-node@v4
       with:
         node-version: 12
         registry-url: https://registry.npmjs.org/

.github/workflows/release.yml

@@ -0,0 +1,26 @@
+import { defineConfig } from "eslint/config";
+import prettier from "eslint-plugin-prettier";
+
+export default defineConfig([{
+    plugins: {
+        prettier,
+    },
+
+    languageOptions: {
+        ecmaVersion: 2020,
+        sourceType: "module",
+
+        parserOptions: {
+            impliedStrict: true,
+
+            ecmaFeatures: {
+                impliedStrict: true,
+                experimentalObjectRestSpread: true,
+            },
+        },
+    },
+
+    rules: {
+        "prettier/prettier": "error",
+    },
+}]);

eslint.config.mjs

@@ -1,4 +1,4 @@
-module.exports = function (octokit, opts) {
+export default function (octokit, opts) {
   return new Promise(async (resolve, reject) => {
     opts = {
       visibility: "all",
@@ -108,7 +108,7 @@ module.exports = function (octokit, opts) {
 
       if (!is_org && team) {
         throw new Error(
-          "The provided 'owner' is not an organization, and so can not have teams"
+          "The provided 'owner' is not an organization, and so can not have teams",
         );
       }
 
@@ -120,4 +120,4 @@ module.exports = function (octokit, opts) {
       throw e;
     }
   }
-};
+}

fetch-repos.js

@@ -1,9 +1,13 @@
-const plugin = require("./fetch-repos");
-const { Octokit } = require("@octokit/rest");
+import plugin from "./fetch-repos";
+import { Octokit } from "@octokit/rest";
+import nock from "nock";
 
-const nock = require("nock");
 nock.disableNetConnect();
-const octokit = new Octokit();
+const octokit = new Octokit({
+  log: {
+    error: () => {},
+  },
+});
 
 function run(body) {
   return plugin(octokit, body);
@@ -18,7 +22,7 @@ test(`owner does not exist`, async () => {
   mockGetUser(owner, null);
 
   await expect(run({ owner })).rejects.toEqual(
-    `The user/org '${owner}' could not be found`
+    `The user/org '${owner}' could not be found`,
   );
 });
 
@@ -27,7 +31,7 @@ test(`team requested, but user is not an org`, async () => {
   mockGetUser("valid-user", "User");
 
   await expect(run({ owner })).rejects.toEqual(
-    `The provided 'owner' is not an organization, and so can not have teams`
+    `The provided 'owner' is not an organization, and so can not have teams`,
   );
 });
 
@@ -76,7 +80,7 @@ test(`runs with defaults`, async () => {
   await expect(
     run({
       owner: "mheap",
-    })
+    }),
   ).resolves.toEqual([
     publicRepo,
     privateRepo,
@@ -94,7 +98,7 @@ test(`can disable forks`, async () => {
     run({
       owner: "mheap",
       include_forks: false,
-    })
+    }),
   ).resolves.toEqual([]);
 });
 
@@ -105,7 +109,7 @@ test(`allows archived`, async () => {
     run({
       owner: "mheap",
       include_archived: true,
-    })
+    }),
   ).resolves.toEqual([archivedRepo]);
 });
 
@@ -116,7 +120,7 @@ test(`allows templates`, async () => {
     run({
       owner: "mheap",
       include_templates: true,
-    })
+    }),
   ).resolves.toEqual([templateRepo]);
 });
 
@@ -127,7 +131,7 @@ test(`filters to only public`, async () => {
     run({
       owner: "mheap",
       visibility: "public",
-    })
+    }),
   ).resolves.toEqual([publicRepo]);
 });
 
@@ -138,7 +142,7 @@ test(`filters to only private`, async () => {
     run({
       owner: "mheap",
       visibility: "private",
-    })
+    }),
   ).resolves.toEqual([privateRepo]);
 });
 
@@ -149,7 +153,7 @@ test(`permissions: pull (success)`, async () => {
     run({
       owner: "mheap",
       minimum_access: "pull",
-    })
+    }),
   ).resolves.toEqual([repoWithPull]);
 });
 
@@ -160,7 +164,7 @@ test(`permissions: push (success)`, async () => {
     run({
       owner: "mheap",
       minimum_access: "push",
-    })
+    }),
   ).resolves.toEqual([repoWithPush]);
 });
 
@@ -171,7 +175,7 @@ test(`permissions: pull (success)`, async () => {
     run({
       owner: "mheap",
       minimum_access: "admin",
-    })
+    }),
   ).resolves.toEqual([repoWithAdmin]);
 });
 
@@ -182,7 +186,7 @@ test(`permissions: pull (no auth, implicitly allowed)`, async () => {
     run({
       owner: "mheap",
       minimum_access: "pull",
-    })
+    }),
   ).resolves.toEqual([publicRepo]);
 });
 
@@ -193,7 +197,7 @@ test(`permissions: pull (no auth, implicitly disallowed)`, async () => {
     run({
       owner: "mheap",
       minimum_access: "push",
-    })
+    }),
   ).resolves.toEqual([]);
 });
 
@@ -204,7 +208,7 @@ test(`permissions: push (failure)`, async () => {
     run({
       owner: "mheap",
       minimum_access: "push",
-    })
+    }),
   ).resolves.toEqual([]);
 });
 
@@ -215,7 +219,7 @@ test(`permissions: admin (failure)`, async () => {
     run({
       owner: "mheap",
       minimum_access: "admin",
-    })
+    }),
   ).resolves.toEqual([]);
 });
 
@@ -226,7 +230,7 @@ test(`permissions: minimum_access is case insensitive`, async () => {
     run({
       owner: "mheap",
       minimum_access: "ADMIN",
-    })
+    }),
   ).resolves.toEqual([repoWithAdmin]);
 });
 
@@ -254,7 +258,7 @@ function mockGetOrgRepos(owner, repos) {
 
 function mockGetTeamRepos(owner, team_slug, repos) {
   const m = nock("https://api.github.com").get(
-    `/orgs/${owner}/teams/${team_slug}/repos`
+    `/orgs/${owner}/teams/${team_slug}/repos`,
   );
   return m.reply(200, repos);
 }