feat!: drop CommonJS support (#59)

## Summary

- publish an ESM-only package from one Bazel build target
- ignore `src/generated` in Oxlint and Oxfmt
- enforce Conventional Commits with Commitlint in Lefthook and CI
- replace Release Doctor with the upstream Release Please action used by
Codescythe

## Breaking change

CommonJS output and export conditions are removed. Consumers must use
ESM imports.

## Test

- `bazel test //...`
- `pnpm build`
- Commitlint hook
- ESM package import smoke test

Author: longlho

.github/workflows/ci.yml

@@ -22,6 +22,8 @@ jobs:
     if: (github.event_name == 'push' || github.event.pull_request.head.repo.fork) && (github.event_name != 'push' || github.event.head_commit.message != 'codegen metadata')
     steps:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        with:
+          fetch-depth: 0
 
       - name: Set up Bazel
         uses: bazel-contrib/setup-bazel@c5acdfb288317d0b5c0bbd7a396a3dc868bb0f86
@@ -41,6 +43,18 @@ jobs:
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
 
+      - name: Lint commits
+        env:
+          BASE_SHA: ${{ github.event.pull_request.base.sha || github.event.before }}
+          HEAD_SHA: ${{ github.event.pull_request.head.sha || github.sha }}
+        run: |
+          set -euo pipefail
+          if git cat-file -e "$BASE_SHA^{commit}" 2>/dev/null && [ "$BASE_SHA" != "0000000000000000000000000000000000000000" ]; then
+            pnpm exec commitlint --from "$BASE_SHA" --to "$HEAD_SHA" --verbose
+          else
+            pnpm exec commitlint --last --verbose
+          fi
+
       - name: Lint
         run: bazel test //:lint
 

.github/workflows/release-doctor.yml

@@ -0,0 +1,28 @@
+name: Release Please
+
+on:
+  push:
+    branches:
+      - main
+  workflow_dispatch:
+
+concurrency:
+  group: ${{ github.workflow }}
+  cancel-in-progress: false
+
+jobs:
+  release-please:
+    name: Create or update release PR
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+      issues: write
+      pull-requests: write
+
+    steps:
+      - name: Run Release Please
+        uses: googleapis/release-please-action@45996ed1f6d02564a971a2fa1b5860e934307cf7
+        with:
+          token: ${{ secrets.RELEASE_TOKEN }}
+          config-file: release-please-config.json
+          manifest-file: .release-please-manifest.json

.github/workflows/release-please.yml

@@ -5,5 +5,5 @@
   "singleQuote": true,
   "trailingComma": "all",
   "sortPackageJson": false,
-  "ignorePatterns": ["dist/**", "bazel-*/**"]
+  "ignorePatterns": ["dist/**", "bazel-*/**", "src/generated/**"]
 }

.oxfmtrc.json

@@ -1,6 +1,6 @@
 {
   "$schema": "./node_modules/oxlint/configuration_schema.json",
-  "ignorePatterns": ["dist/**", "bazel-*/**"],
+  "ignorePatterns": ["dist/**", "bazel-*/**", "src/generated/**"],
   "categories": {
     "correctness": "off"
   },

.oxlintrc.json

@@ -32,38 +32,11 @@ TS_SOURCES = glob(["src/**/*.ts"])
 TEST_SOURCES = glob(["tests/**/*.ts"])
 
 ts_project(
-    name = "dist_cjs",
+    name = "dist",
     srcs = TS_SOURCES,
     declaration = True,
     declaration_map = True,
-    out_dir = "dist/cjs",
-    root_dir = "src",
-    source_map = True,
-    transpiler = "tsc",
-    tsc = ":tsc",
-    tsconfig = {
-        "compilerOptions": dict(
-            BASE_COMPILER_OPTIONS,
-            declaration = True,
-            declarationMap = True,
-            module = "Node16",
-            moduleResolution = "Node16",
-            noEmit = False,
-            outDir = "dist/cjs",
-            rootDir = "src",
-            sourceMap = True,
-            types = ["node"],
-        ),
-    },
-    deps = [":node_modules/@types/node"],
-)
-
-ts_project(
-    name = "dist_esm",
-    srcs = TS_SOURCES,
-    declaration = True,
-    declaration_map = True,
-    out_dir = "dist/esm",
+    out_dir = "dist",
     root_dir = "src",
     source_map = True,
     transpiler = "tsc",
@@ -76,7 +49,7 @@ ts_project(
             module = "ESNext",
             moduleResolution = "Bundler",
             noEmit = False,
-            outDir = "dist/esm",
+            outDir = "dist",
             rootDir = "src",
             sourceMap = True,
             types = ["node"],
@@ -91,21 +64,13 @@ npm_package(
         "CHANGELOG.md",
         "LICENSE",
         "README.md",
-        ":dist_cjs",
-        ":dist_esm",
+        ":dist",
         "//npm:package",
-        "//npm:index.d.cts",
-        "//npm:index.js",
-    ] + TS_SOURCES + [
-        "esm/package.json",
-    ],
+    ] + TS_SOURCES,
     out = "package",
     package = "@perplexity-ai/perplexity_ai",
     publishable = True,
     replace_prefixes = {
-        "esm": "dist/esm",
-        "npm/index.d.cts": "dist/index.d.cts",
-        "npm/index.js": "dist/index.js",
         "npm/package.json": "package.json",
     },
 )
@@ -126,8 +91,8 @@ ts_project(
     tsconfig = {
         "compilerOptions": dict(
             BASE_COMPILER_OPTIONS,
-            module = "Node16",
-            moduleResolution = "Node16",
+            module = "ESNext",
+            moduleResolution = "Bundler",
             noEmit = False,
             outDir = "test-dist",
             rootDir = "tests",
@@ -166,9 +131,19 @@ FORMAT_SOURCES = glob(
         "tests/**/*.ts",
         "*.json",
         "*.md",
+        "*.mjs",
         "*.yml",
     ],
     allow_empty = True,
+    exclude = ["src/generated/**"],
+)
+
+LINT_SOURCES = glob(
+    [
+        "src/**/*.ts",
+        "tests/**/*.ts",
+    ],
+    exclude = ["src/generated/**"],
 )
 
 oxfmt_bin.oxfmt_test(
@@ -188,10 +163,10 @@ oxlint_bin.oxlint_test(
     name = "oxlint",
     args = [
         "--config=$(rootpath :.oxlintrc.json)",
-    ] + TS_SOURCES + TEST_SOURCES,
+    ] + LINT_SOURCES,
     data = [
         ".oxlintrc.json",
-    ] + TS_SOURCES + TEST_SOURCES,
+    ] + LINT_SOURCES,
 )
 
 publint_bin.publint_test(
@@ -211,7 +186,6 @@ attw_bin.attw_test(
         "--pack",
         "--ignore-rules",
         "cjs-resolves-to-esm",
-        "missing-export-equals",
         "internal-resolution-error",
     ],
     data = [":pkg"],

BUILD.bazel

@@ -0,0 +1,3 @@
+export default {
+  extends: ['@commitlint/config-conventional'],
+};

commitlint.config.mjs

@@ -4,6 +4,11 @@ output:
 
 no_auto_install: true
 
+commit-msg:
+  commands:
+    commitlint:
+      run: ./node_modules/.bin/commitlint --edit {1}
+
 pre-commit:
   commands:
     oxlint:

esm/package.json

@@ -1,10 +1,5 @@
 load("@jq.bzl//jq:jq.bzl", "jq")
 
-exports_files([
-    "index.d.cts",
-    "index.js",
-])
-
 jq(
     name = "package",
     srcs = ["//:package.json"],