chore: add biomejs, replace eslint with biome for linting and formatting

Co-Authored-By: Niels Swimberghe <3382717+Swimburger@users.noreply.github.com>

Author: devin-ai-integration[bot]

.github/workflows/ci.yml

@@ -20,6 +20,9 @@ jobs:
       - name: Install dependencies
         run: npm ci
 
+      - name: Lint
+        run: npm run lint
+
       - name: Run tests
         run: npx vitest run
 

biome.json

@@ -0,0 +1,38 @@
+{
+	"$schema": "https://biomejs.dev/schemas/2.4.5/schema.json",
+	"vcs": {
+		"enabled": true,
+		"clientKind": "git",
+		"useIgnoreFile": true
+	},
+	"files": {
+		"includes": ["**", "!!**/dist"]
+	},
+	"formatter": {
+		"enabled": true,
+		"indentStyle": "space",
+		"indentWidth": 4
+	},
+	"linter": {
+		"enabled": true,
+		"rules": {
+			"recommended": true,
+			"suspicious": {
+				"noExplicitAny": "off"
+			}
+		}
+	},
+	"javascript": {
+		"formatter": {
+			"quoteStyle": "double"
+		}
+	},
+	"assist": {
+		"enabled": true,
+		"actions": {
+			"source": {
+				"organizeImports": "on"
+			}
+		}
+	}
+}

dist/index.js

@@ -39311,14 +39311,14 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.run = run;
+const fs = __importStar(__nccwpck_require__(7561));
+const path = __importStar(__nccwpck_require__(9411));
 const core = __importStar(__nccwpck_require__(2186));
-const github = __importStar(__nccwpck_require__(5438));
 const exec = __importStar(__nccwpck_require__(1514));
+const github = __importStar(__nccwpck_require__(5438));
 const io = __importStar(__nccwpck_require__(7436));
-const fs = __importStar(__nccwpck_require__(7147));
-const path = __importStar(__nccwpck_require__(1017));
-const yaml = __importStar(__nccwpck_require__(1917));
 const glob = __importStar(__nccwpck_require__(8211));
+const yaml = __importStar(__nccwpck_require__(1917));
 const minimatch_1 = __nccwpck_require__(4501);
 async function run() {
     try {
@@ -39400,7 +39400,7 @@ async function updateTargetSpec(token, branch, autoMerge) {
         try {
             fileMapping = JSON.parse(fileMappingInput);
         }
-        catch (jsonError) {
+        catch (_jsonError) {
             throw new Error(`Failed to parse 'sources' input as either YAML or JSON. Please check the format. Error: ${yamlError.message}`);
         }
     }
@@ -39429,7 +39429,7 @@ async function runFernApiUpdate() {
             await exec.exec("fern", ["--version"], { silent: true });
             core.info("Fern CLI is already installed");
         }
-        catch (error) {
+        catch (_error) {
             core.info("Fern CLI not found. Installing Fern CLI...");
             await exec.exec("npm", ["install", "-g", "fern-api"]);
         }
@@ -39468,7 +39468,7 @@ async function cloneRepository(options) {
     try {
         await exec.exec("git", ["clone", repoUrl, repoDir]);
     }
-    catch (error) {
+    catch (_error) {
         throw new Error(`Failed to clone repository. Please ensure your token has 'repo' scope and you have write access to ${options.repository}.`);
     }
     process.chdir(repoDir);
@@ -39528,7 +39528,7 @@ async function branchExists(owner, repo, branchName, octokit) {
         });
         return true;
     }
-    catch (error) {
+    catch (_error) {
         return false;
     }
 }
@@ -39610,7 +39610,7 @@ async function hasDifferenceWithRemote(branchName) {
         const diff = await exec.getExecOutput("git", ["diff", `HEAD`, `origin/${branchName}`], { silent: true });
         return !!diff.stdout.trim();
     }
-    catch (error) {
+    catch (_error) {
         core.info(`Could not fetch remote branch, assuming this is the first push to new branch.`);
         return true;
     }
@@ -39640,7 +39640,9 @@ async function pushChanges(branchName, options) {
 async function pushWithFallback(branchName, owner, repo, octokit) {
     // Try regular push first
     try {
-        await exec.exec("git", ["push", "--verbose", "origin", branchName], { silent: false });
+        await exec.exec("git", ["push", "--verbose", "origin", branchName], {
+            silent: false,
+        });
         return true;
     }
     catch {
@@ -39660,34 +39662,31 @@ async function pushWithFallback(branchName, owner, repo, octokit) {
         }
         // Push after rebase failed — no rebase in progress, don't abort
         errorLabel = "Push error";
-        errorMsg = [
-            pushResult.stderr.trim(),
-            pushResult.stdout.trim(),
-        ]
-            .filter(Boolean)
-            .join("\n") || `git push failed with exit code ${pushResult.exitCode}`;
+        errorMsg =
+            [pushResult.stderr.trim(), pushResult.stdout.trim()]
+                .filter(Boolean)
+                .join("\n") ||
+                `git push failed with exit code ${pushResult.exitCode}`;
         core.info(`Push after rebase failed: ${errorMsg}`);
     }
     else {
         // Rebase itself failed (merge conflicts)
         rebaseFailed = true;
         errorLabel = "Rebase error";
-        errorMsg = [
-            rebaseResult.stderr.trim(),
-            rebaseResult.stdout.trim(),
-        ]
-            .filter(Boolean)
-            .join("\n") || `git pull --rebase failed with exit code ${rebaseResult.exitCode}`;
+        errorMsg =
+            [rebaseResult.stderr.trim(), rebaseResult.stdout.trim()]
+                .filter(Boolean)
+                .join("\n") ||
+                `git pull --rebase failed with exit code ${rebaseResult.exitCode}`;
         core.info(`Rebase failed (likely due to merge conflicts). Aborting rebase.`);
         // Abort the rebase so the working tree is clean
         const abortResult = await exec.getExecOutput("git", ["rebase", "--abort"], { ignoreReturnCode: true });
         if (abortResult.exitCode !== 0) {
-            abortErrorMsg = [
-                abortResult.stderr.trim(),
-                abortResult.stdout.trim(),
-            ]
-                .filter(Boolean)
-                .join("\n") || `rebase --abort failed with exit code ${abortResult.exitCode}`;
+            abortErrorMsg =
+                [abortResult.stderr.trim(), abortResult.stdout.trim()]
+                    .filter(Boolean)
+                    .join("\n") ||
+                    `rebase --abort failed with exit code ${abortResult.exitCode}`;
             core.info(`rebase --abort failed: ${abortErrorMsg}. The working tree may be in an unexpected state.`);
         }
     }
@@ -39745,10 +39744,10 @@ async function updatePR(octokit, owner, repo, prNumber) {
 async function createPR(octokit, owner, repo, branchName, targetBranch, isFromFern) {
     core.info(`Creating new PR from ${branchName} to ${targetBranch}`);
     const date = new Date().toISOString().replace(/[:.]/g, "-");
-    let prTitle = isFromFern
+    const prTitle = isFromFern
         ? `chore: Update API specifications with fern api update (${date})`
         : `chore: Update OpenAPI specifications (${date})`;
-    let prBody = isFromFern
+    const prBody = isFromFern
         ? "Update API specifications by running fern api update."
         : "Update OpenAPI specifications based on changes in the source repository.";
     const prResponse = await octokit.rest.pulls.create({

e2e/run-e2e.ts

@@ -74,7 +74,9 @@ async function githubApi(
 
     if (!resp.ok) {
         const text = await resp.text();
-        throw new Error(`GitHub API ${resp.status} ${resp.statusText}: ${text}`);
+        throw new Error(
+            `GitHub API ${resp.status} ${resp.statusText}: ${text}`,
+        );
     }
 
     // 204 No Content
@@ -128,16 +130,18 @@ async function updateSourceSpec(): Promise<void> {
                     operationId: "getStatus",
                     summary: `Status v${specVersion}`,
                     responses: {
-                        "200": { description: `Returns status v${specVersion}` },
+                        "200": {
+                            description: `Returns status v${specVersion}`,
+                        },
                     },
                 },
             },
         },
     };
 
-    const content = Buffer.from(
-        JSON.stringify(spec, null, 2) + "\n",
-    ).toString("base64");
+    const content = Buffer.from(`${JSON.stringify(spec, null, 2)}\n`).toString(
+        "base64",
+    );
     const sha = await getFileSha(SOURCE_SPEC_PATH);
 
     await githubApi(`/repos/${OWNER}/${REPO}/contents/${SOURCE_SPEC_PATH}`, {
@@ -241,9 +245,7 @@ async function waitForWorkflowRun(
         const runs: WorkflowRun[] = data.workflow_runs;
 
         // Find a run that started after our trigger
-        const run = runs.find(
-            (r: any) => new Date(r.created_at) > afterDate,
-        );
+        const run = runs.find((r: any) => new Date(r.created_at) > afterDate);
 
         if (run) {
             if (run.status === "completed") {
@@ -279,9 +281,7 @@ async function getPRCommitCount(prNumber: number): Promise<number> {
 }
 
 async function getPRComments(prNumber: number): Promise<GitHubComment[]> {
-    return githubApi(
-        `/repos/${OWNER}/${REPO}/issues/${prNumber}/comments`,
-    );
+    return githubApi(`/repos/${OWNER}/${REPO}/issues/${prNumber}/comments`);
 }
 
 async function pushConflictingCommit(branchSha: string): Promise<void> {
@@ -327,26 +327,20 @@ async function pushConflictingCommit(branchSha: string): Promise<void> {
     });
 
     // Create a commit
-    const newCommit = await githubApi(
-        `/repos/${OWNER}/${REPO}/git/commits`,
-        {
-            method: "POST",
-            body: {
-                message: "Conflicting change to force merge conflict",
-                tree: tree.sha,
-                parents: [branchSha],
-            },
+    const newCommit = await githubApi(`/repos/${OWNER}/${REPO}/git/commits`, {
+        method: "POST",
+        body: {
+            message: "Conflicting change to force merge conflict",
+            tree: tree.sha,
+            parents: [branchSha],
         },
-    );
+    });
 
     // Update the branch ref
-    await githubApi(
-        `/repos/${OWNER}/${REPO}/git/refs/heads/${BRANCH}`,
-        {
-            method: "PATCH",
-            body: { sha: newCommit.sha, force: true },
-        },
-    );
+    await githubApi(`/repos/${OWNER}/${REPO}/git/refs/heads/${BRANCH}`, {
+        method: "PATCH",
+        body: { sha: newCommit.sha, force: true },
+    });
 
     console.log(`  Pushed conflicting commit ${newCommit.sha} to ${BRANCH}`);
 }
@@ -356,7 +350,9 @@ async function pushConflictingCommit(branchSha: string): Promise<void> {
 async function testHappyPath(): Promise<void> {
     console.log("\n=== TEST: Happy Path ===");
 
-    console.log("Step 1: Update source spec and trigger workflow (first run - should create PR)");
+    console.log(
+        "Step 1: Update source spec and trigger workflow (first run - should create PR)",
+    );
     await updateSourceSpec();
 
     const before1 = new Date();
@@ -379,7 +375,9 @@ async function testHappyPath(): Promise<void> {
     const commits1 = await getPRCommitCount(prNumber);
     console.log(`  PR #${prNumber} created with ${commits1} commit(s)`);
 
-    console.log("\nStep 2: Reset fern spec on PR branch and trigger workflow (should reuse existing PR)");
+    console.log(
+        "\nStep 2: Reset fern spec on PR branch and trigger workflow (should reuse existing PR)",
+    );
     // Instead of changing the source spec (which is subject to raw.githubusercontent.com CDN cache),
     // we reset fern/openapi/openapi.json on the update-api branch to {} so that fern api update
     // will write the (cached) origin content and produce a git diff.
@@ -415,7 +413,9 @@ async function testHappyPath(): Promise<void> {
         );
     }
 
-    console.log(`  Still 1 PR (#${prNumber}), commits: ${commits1} → ${commits2}`);
+    console.log(
+        `  Still 1 PR (#${prNumber}), commits: ${commits1} → ${commits2}`,
+    );
     console.log("  PASS: Happy path\n");
 }
 
@@ -434,12 +434,16 @@ async function testConflictPath(): Promise<void> {
     const branchData = await githubApi(
         `/repos/${OWNER}/${REPO}/git/refs/heads/${BRANCH}`,
     );
-    const currentSha = branchData.object.sha;
+    const _currentSha = branchData.object.sha;
 
-    console.log("Step 1: Reset fern spec on PR branch so fern api update will produce a diff");
+    console.log(
+        "Step 1: Reset fern spec on PR branch so fern api update will produce a diff",
+    );
     await resetFernSpec(BRANCH);
 
-    console.log("Step 2: Trigger workflow and push conflicting commit during fern install window");
+    console.log(
+        "Step 2: Trigger workflow and push conflicting commit during fern install window",
+    );
     // The workflow takes ~5-10s to install fern-api. During this window,
     // we push a commit to origin/update-api that the action won't have locally,
     // causing its subsequent push to fail (remote has commits the local doesn't).
@@ -448,7 +452,9 @@ async function testConflictPath(): Promise<void> {
 
     // Wait for the workflow to start and pull the branch, then push a conflicting commit.
     // The fern-api install takes ~5-10s, so we have a window.
-    console.log("  Waiting 12s for workflow to pull branch before pushing conflict...");
+    console.log(
+        "  Waiting 12s for workflow to pull branch before pushing conflict...",
+    );
     await sleep(12000);
 
     // Get the updated SHA (after resetFernSpec added a commit)
@@ -473,14 +479,13 @@ async function testConflictPath(): Promise<void> {
     console.log("Step 3: Check for conflict comment on PR");
     const comments = await getPRComments(pr.number);
     const conflictComment = comments.find(
-        (c) => c.body.includes("Sync failed") || c.body.includes("Rebase error"),
+        (c) =>
+            c.body.includes("Sync failed") || c.body.includes("Rebase error"),
     );
 
     if (conflictComment) {
         console.log(`  Found conflict comment on PR #${pr.number}:`);
-        console.log(
-            `    ${conflictComment.body.substring(0, 200)}...`,
-        );
+        console.log(`    ${conflictComment.body.substring(0, 200)}...`);
         console.log("  PASS: Conflict path - error comment posted\n");
     } else if (run.conclusion === "failure") {
         console.log(