feat: convert to ESM (breaking change)

.eslintignore

@@ -1,3 +1,3 @@
 coverage/
 dist/
-!.alexrc.js
+exe/

.eslintrc

@@ -1,9 +1,6 @@
 {
-  "extends": ["@readme/eslint-config", "@readme/eslint-config/typescript"],
+  "extends": ["@readme/eslint-config", "@readme/eslint-config/typescript", "@readme/eslint-config/esm"],
   "root": true,
-  "parserOptions": {
-    "ecmaVersion": 2020
-  },
   "overrides": [
     {
       "files": ["bin/*.js"],
@@ -12,6 +9,13 @@
       }
     }
   ],
+  "env": {
+    "es2021": true,
+    "node": true
+  },
+  "parserOptions": {
+    "sourceType": "module"
+  },
   "rules": {
     "@typescript-eslint/ban-types": [
       "error",
@@ -37,7 +41,7 @@
      * console.log() calls, hurting our ability to write resilient tests and easily debug issues.
      *
      * We should be returning Promise-wrapped values in our main command functions
-     * so we can write robust tests and take advantage of `bin/rdme`,
+     * so we can write robust tests and take advantage of `bin/rdme.js`,
      * which we use for printing function outputs and returning correct exit codes.
      *
      * Furthermore, we should also be using our custom loggers (see src/lib/logger.js)

.github/dependabot.yml

@@ -33,19 +33,7 @@ updates:
       prefix: chore(deps)
       prefix-development: chore(deps-dev)
     ignore:
-      # All of these packages are now ESM-only and can't be used here without a rewrite.
-      - dependency-name: chalk
-        versions:
-          - '>= 5'
-      - dependency-name: configstore
-        versions:
-          - '>= 6'
-      - dependency-name: node-fetch
-        versions:
-          - '>= 3'
-      - dependency-name: open
-        versions:
-          - '>= 9'
+      # See https://github.com/sindresorhus/ora/issues/229
       - dependency-name: ora
         versions:
-          - '>= 6'
+          - '>= 7'

.github/workflows/ci.yml

@@ -15,8 +15,6 @@ jobs:
       fail-fast: false
       matrix:
         node-version:
-          - 14
-          - 16
           - 18
           - 20
 

.gitignore

@@ -1,4 +1,5 @@
+.env
 coverage/
 dist/
-.env
+exe/
 node_modules/

.npmignore

@@ -4,7 +4,7 @@ __tests__/
 .github/
 .husky/
 .prettier*
-bin/*.js
+bin/**
 coverage/
 packages/
 vitest.*

.prettierignore

@@ -2,3 +2,4 @@ __tests__/__fixtures__/invalid-json/yikes.json
 CHANGELOG.md
 coverage/
 dist/
+exe/

CONTRIBUTING.md

@@ -2,11 +2,11 @@
 
 ## Running Shell Commands Locally 🐚
 
-To run test commands from within the repository, run the build and then run your commands from the root of the repository and use `./bin/rdme` instead of `rdme` so it properly points to the command executable, like so:
+To run test commands from within the repository, run the build and then run your commands from the root of the repository and use `./bin/rdme.js` instead of `rdme` so it properly points to the command executable, like so:
 
 ```sh
 npm run build
-./bin/rdme openapi:validate __tests__/__fixtures__/ref-oas/petstore.json
+./bin/rdme.js openapi:validate __tests__/__fixtures__/ref-oas/petstore.json
 ```
 
 If you need to debug commands quicker and re-building TS everytime is becoming cumbersome, you can use the debug command, like so:
@@ -70,7 +70,7 @@ act -j simple
 
 ### Usage of `console`
 
-As you'll learn in our commands logic (see [`bin/rdme`](bin/rdme) and the [`src/cmds`](src/cmds) directory), we wrap our command outputs in resolved/rejected [`Promise` objects](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise) and use [`bin/rdme`](bin/rdme) file to log the results to the console and return the correct status code. This is so we can write more resilient tests, ensure that the proper exit codes are being returned, and make debugging easier.
+As you'll learn in our commands logic (see [`bin/rdme.js`](bin/rdme.js) and the [`src/cmds`](src/cmds) directory), we wrap our command outputs in resolved/rejected [`Promise` objects](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise) and use [`bin/rdme.js`](bin/rdme.js) file to log the results to the console and return the correct status code. This is so we can write more resilient tests, ensure that the proper exit codes are being returned, and make debugging easier.
 
 When writing command logic, avoid using `console` statements (and correspondingly, avoid mocking `console` statements in tests) when possible.
 

Dockerfile

@@ -1,8 +1,8 @@
-FROM node:16-alpine as builder
+FROM node:18-alpine as builder
 
 COPY . /rdme
 
-RUN cd /rdme && npm ci && npm run build && npx pkg@5 . --target host --out-path exe
+RUN cd /rdme && npm ci && npm run build:exe
 
 FROM alpine:3.14
 

__tests__/bin.test.ts

@@ -1,4 +1,4 @@
-import { exec } from 'child_process';
+import { exec } from 'node:child_process';
 
 import { describe, it, expect } from 'vitest';
 
@@ -7,7 +7,7 @@ describe('bin', () => {
     expect.assertions(1);
 
     await new Promise(resolve => {
-      exec(`node ${__dirname}/../bin/rdme`, (error, stdout) => {
+      exec(`node ${__dirname}/../bin/rdme.js`, (error, stdout) => {
         expect(stdout).toContain('a utility for interacting with ReadMe');
         resolve(true);
       });

__tests__/cmds/categories/create.test.ts

@@ -2,8 +2,8 @@ import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterEach, it, expect, vi } from 'vitest';
 
-import CategoriesCreateCommand from '../../../src/cmds/categories/create';
-import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock';
+import CategoriesCreateCommand from '../../../src/cmds/categories/create.js';
+import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock.js';
 
 const categoriesCreate = new CategoriesCreateCommand();
 

__tests__/cmds/categories/index.test.ts

@@ -2,8 +2,8 @@ import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterEach, it, expect, vi } from 'vitest';
 
-import CategoriesCommand from '../../../src/cmds/categories';
-import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock';
+import CategoriesCommand from '../../../src/cmds/categories/index.js';
+import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock.js';
 
 const categories = new CategoriesCommand();
 

__tests__/cmds/changelogs/index.test.ts

@@ -1,16 +1,16 @@
-import fs from 'fs';
-import path from 'path';
+import fs from 'node:fs';
+import path from 'node:path';
 
 import chalk from 'chalk';
 import frontMatter from 'gray-matter';
 import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterAll, beforeEach, it, expect, vi } from 'vitest';
 
-import ChangelogsCommand from '../../../src/cmds/changelogs';
-import APIError from '../../../src/lib/apiError';
-import getAPIMock from '../../helpers/get-api-mock';
-import hashFileContents from '../../helpers/hash-file-contents';
+import ChangelogsCommand from '../../../src/cmds/changelogs.js';
+import APIError from '../../../src/lib/apiError.js';
+import getAPIMock from '../../helpers/get-api-mock.js';
+import hashFileContents from '../../helpers/hash-file-contents.js';
 
 const changelogs = new ChangelogsCommand();
 

__tests__/cmds/changelogs/single.test.ts

@@ -1,16 +1,16 @@
-import fs from 'fs';
-import path from 'path';
+import fs from 'node:fs';
+import path from 'node:path';
 
 import chalk from 'chalk';
 import frontMatter from 'gray-matter';
 import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterAll, beforeEach, it, expect, vi } from 'vitest';
 
-import ChangelogsCommand from '../../../src/cmds/changelogs';
-import APIError from '../../../src/lib/apiError';
-import getAPIMock from '../../helpers/get-api-mock';
-import hashFileContents from '../../helpers/hash-file-contents';
+import ChangelogsCommand from '../../../src/cmds/changelogs.js';
+import APIError from '../../../src/lib/apiError.js';
+import getAPIMock from '../../helpers/get-api-mock.js';
+import hashFileContents from '../../helpers/hash-file-contents.js';
 
 const changelogs = new ChangelogsCommand();
 

__tests__/cmds/custompages/index.test.ts

@@ -1,16 +1,16 @@
-import fs from 'fs';
-import path from 'path';
+import fs from 'node:fs';
+import path from 'node:path';
 
 import chalk from 'chalk';
 import frontMatter from 'gray-matter';
 import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterAll, beforeEach, it, expect, vi } from 'vitest';
 
-import CustomPagesCommand from '../../../src/cmds/custompages';
-import APIError from '../../../src/lib/apiError';
-import getAPIMock from '../../helpers/get-api-mock';
-import hashFileContents from '../../helpers/hash-file-contents';
+import CustomPagesCommand from '../../../src/cmds/custompages.js';
+import APIError from '../../../src/lib/apiError.js';
+import getAPIMock from '../../helpers/get-api-mock.js';
+import hashFileContents from '../../helpers/hash-file-contents.js';
 
 const custompages = new CustomPagesCommand();
 

__tests__/cmds/custompages/single.test.ts

@@ -1,16 +1,16 @@
-import fs from 'fs';
-import path from 'path';
+import fs from 'node:fs';
+import path from 'node:path';
 
 import chalk from 'chalk';
 import frontMatter from 'gray-matter';
 import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterAll, beforeEach, it, expect, vi } from 'vitest';
 
-import CustomPagesCommand from '../../../src/cmds/custompages';
-import APIError from '../../../src/lib/apiError';
-import getAPIMock from '../../helpers/get-api-mock';
-import hashFileContents from '../../helpers/hash-file-contents';
+import CustomPagesCommand from '../../../src/cmds/custompages.js';
+import APIError from '../../../src/lib/apiError.js';
+import getAPIMock from '../../helpers/get-api-mock.js';
+import hashFileContents from '../../helpers/hash-file-contents.js';
 
 const custompages = new CustomPagesCommand();
 

__tests__/cmds/docs/edit.test.ts

@@ -1,12 +1,12 @@
-import fs from 'fs';
+import fs from 'node:fs';
 
 import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterAll, beforeEach, afterEach, it, expect, vi } from 'vitest';
 
-import DocsEditCommand from '../../../src/cmds/docs/edit';
-import APIError from '../../../src/lib/apiError';
-import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock';
+import DocsEditCommand from '../../../src/cmds/docs/edit.js';
+import APIError from '../../../src/lib/apiError.js';
+import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock.js';
 
 const docsEdit = new DocsEditCommand();
 

__tests__/cmds/docs/index.test.ts

@@ -1,21 +1,21 @@
 /* eslint-disable no-console */
-import fs from 'fs';
-import path from 'path';
+import fs from 'node:fs';
+import path from 'node:path';
 
 import chalk from 'chalk';
 import frontMatter from 'gray-matter';
 import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterAll, beforeEach, afterEach, it, expect, vi } from 'vitest';
 
-import DocsCommand from '../../../src/cmds/docs';
-import GuidesCommand from '../../../src/cmds/guides';
-import APIError from '../../../src/lib/apiError';
-import configstore from '../../../src/lib/configstore';
-import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock';
-import { after, before } from '../../helpers/get-gha-setup';
-import hashFileContents from '../../helpers/hash-file-contents';
-import { after as afterGHAEnv, before as beforeGHAEnv } from '../../helpers/setup-gha-env';
+import DocsCommand from '../../../src/cmds/docs/index.js';
+import GuidesCommand from '../../../src/cmds/guides/index.js';
+import APIError from '../../../src/lib/apiError.js';
+import configstore from '../../../src/lib/configstore.js';
+import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock.js';
+import { after, before } from '../../helpers/get-gha-setup.js';
+import hashFileContents from '../../helpers/hash-file-contents.js';
+import { after as afterGHAEnv, before as beforeGHAEnv } from '../../helpers/setup-gha-env.js';
 
 const docs = new DocsCommand();
 const guides = new GuidesCommand();

__tests__/cmds/docs/prune.test.ts

@@ -2,9 +2,9 @@ import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterAll, it, expect, vi } from 'vitest';
 
-import DocsPruneCommand from '../../../src/cmds/docs/prune';
-import GuidesPruneCommand from '../../../src/cmds/guides/prune';
-import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock';
+import DocsPruneCommand from '../../../src/cmds/docs/prune.js';
+import GuidesPruneCommand from '../../../src/cmds/guides/prune.js';
+import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock.js';
 
 const docsPrune = new DocsPruneCommand();
 const guidesPrune = new GuidesPruneCommand();

__tests__/cmds/docs/single.test.ts

@@ -1,17 +1,17 @@
-import fs from 'fs';
-import path from 'path';
+import fs from 'node:fs';
+import path from 'node:path';
 
 import chalk from 'chalk';
 import frontMatter from 'gray-matter';
 import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterAll, beforeEach, afterEach, it, expect, vi } from 'vitest';
 
-import DocsCommand from '../../../src/cmds/docs';
-import APIError from '../../../src/lib/apiError';
-import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock';
-import hashFileContents from '../../helpers/hash-file-contents';
-import { after as afterGHAEnv, before as beforeGHAEnv } from '../../helpers/setup-gha-env';
+import DocsCommand from '../../../src/cmds/docs/index.js';
+import APIError from '../../../src/lib/apiError.js';
+import getAPIMock, { getAPIMockWithVersionHeader } from '../../helpers/get-api-mock.js';
+import hashFileContents from '../../helpers/hash-file-contents.js';
+import { after as afterGHAEnv, before as beforeGHAEnv } from '../../helpers/setup-gha-env.js';
 
 const docs = new DocsCommand();
 

__tests__/cmds/login.test.ts

@@ -2,10 +2,10 @@ import nock from 'nock';
 import prompts from 'prompts';
 import { describe, beforeAll, afterAll, afterEach, it, expect } from 'vitest';
 
-import Command from '../../src/cmds/login';
-import APIError from '../../src/lib/apiError';
-import configStore from '../../src/lib/configstore';
-import getAPIMock from '../helpers/get-api-mock';
+import Command from '../../src/cmds/login.js';
+import APIError from '../../src/lib/apiError.js';
+import configStore from '../../src/lib/configstore.js';
+import getAPIMock from '../helpers/get-api-mock.js';
 
 const cmd = new Command();
 

__tests__/cmds/logout.test.ts

@@ -1,8 +1,8 @@
 import { describe, afterEach, it, expect } from 'vitest';
 
-import Command from '../../src/cmds/logout';
-import config from '../../src/lib/config';
-import configStore from '../../src/lib/configstore';
+import Command from '../../src/cmds/logout.js';
+import config from '../../src/lib/config.js';
+import configStore from '../../src/lib/configstore.js';
 
 const cmd = new Command();
 

__tests__/cmds/open.test.ts

@@ -1,12 +1,12 @@
-import type { Version } from '../../src/cmds/versions';
+import type { Version } from '../../src/cmds/versions/index.js';
 
 import chalk from 'chalk';
 import { describe, afterEach, it, expect } from 'vitest';
 
-import Command from '../../src/cmds/open';
-import config from '../../src/lib/config';
-import configStore from '../../src/lib/configstore';
-import getAPIMock from '../helpers/get-api-mock';
+import Command from '../../src/cmds/open.js';
+import config from '../../src/lib/config.js';
+import configStore from '../../src/lib/configstore.js';
+import getAPIMock from '../helpers/get-api-mock.js';
 
 const cmd = new Command();
 

__tests__/cmds/openapi/convert.test.ts

@@ -1,9 +1,9 @@
-import fs from 'fs';
+import fs from 'node:fs';
 
 import prompts from 'prompts';
 import { describe, it, expect, vi } from 'vitest';
 
-import OpenAPIConvertCommand from '../../../src/cmds/openapi/convert';
+import OpenAPIConvertCommand from '../../../src/cmds/openapi/convert.js';
 
 const convert = new OpenAPIConvertCommand();