ci: satisfy formatting checks

Author: yelixir-dev

.gitignore

@@ -2,6 +2,7 @@ node_modules/
 dist/
 coverage/
 .vitest-tmp/
+.internal/
 .env
 .env.*
 !.env.example

.prettierignore

@@ -0,0 +1,7 @@
+node_modules/
+dist/
+coverage/
+.vitest-tmp/
+release/
+.internal/
+*.tgz

eslint.config.js

@@ -5,7 +5,7 @@ export default tseslint.config(
   js.configs.recommended,
   ...tseslint.configs.recommended,
   {
-    ignores: ["dist/**", "node_modules/**", "release/**"],
+    ignores: ["dist/**", "node_modules/**", "release/**", ".internal/**"],
   },
   {
     files: ["**/*.ts"],

src/config.ts

@@ -178,7 +178,8 @@ export function loadBridgeConfig(options: LoadBridgeConfigOptions = {}): BridgeC
     defaultModel,
     allowedModels,
     allowUnknownModels: parseBoolean(env.COMMANDCODE_ALLOW_UNKNOWN_MODELS, false),
-    bridgeApiKey: stringValue(dashboardConfig.bridgeApiKey) || env.BRIDGE_API_KEY?.trim() || undefined,
+    bridgeApiKey:
+      stringValue(dashboardConfig.bridgeApiKey) || env.BRIDGE_API_KEY?.trim() || undefined,
     commandCodeApiKey: commandCodeCredentials[0]?.apiKey,
     commandCodeCredentials,
     commandCodeRoutingPolicy: routingPolicy,

src/server.ts

@@ -196,7 +196,9 @@ function hasSameHostnameReferer(request: FastifyRequest): boolean {
 function isDashboardWriteSourceAllowed(request: FastifyRequest): boolean {
   if (sameHostnameOrigin(request)) return true;
   if (hasSameHostnameReferer(request)) return true;
-  return !request.headers.origin && !request.headers.referer && isLoopbackHost(request.headers.host);
+  return (
+    !request.headers.origin && !request.headers.referer && isLoopbackHost(request.headers.host)
+  );
 }
 
 function asOpenAIRequest(
@@ -240,7 +242,8 @@ function duplicateCommandCodeApiKeyIds(update: DashboardConfigUpdate): string[]
   for (const credential of update.credentials ?? []) {
     const apiKey = typeof credential.apiKey === "string" ? credential.apiKey.trim() : "";
     if (!apiKey) continue;
-    const id = typeof credential.id === "string" && credential.id.trim() ? credential.id.trim() : "unknown";
+    const id =
+      typeof credential.id === "string" && credential.id.trim() ? credential.id.trim() : "unknown";
     const existingId = seen.get(apiKey);
     if (existingId) {
       duplicateIds.add(existingId);

tests/admin-config.test.ts

@@ -283,7 +283,6 @@ describe("JSON dashboard configuration", () => {
     await app.close();
   });
 
-
   it("saves dashboard JSON without requiring the bridge client API key", async () => {
     const file = tempConfigFile({
       bridgeApiKey: "bridge-secret",

tests/dashboard-ui.test.ts

@@ -120,7 +120,9 @@ describe("dashboard UI", () => {
       bridgeApiKey: "test-admin-token",
     });
 
-    expect(html).toContain("authKey(cfg?.bridgeApiKey)||authKey(localStorage.getItem('bridgeApiKey'))");
+    expect(html).toContain(
+      "authKey(cfg?.bridgeApiKey)||authKey(localStorage.getItem('bridgeApiKey'))",
+    );
     expect(html).toContain("'authorization':'Bearer '+key");
   });
 
@@ -199,12 +201,16 @@ describe("dashboard UI", () => {
     });
 
     expect(html).toContain("async function load(){");
-    expect(html).toContain("return {ok:true,dirty:!!cfg.dirty,restartRequired:!!cfg.restart_required}");
+    expect(html).toContain(
+      "return {ok:true,dirty:!!cfg.dirty,restartRequired:!!cfg.restart_required}",
+    );
     expect(html).toContain("async function waitForRestart()");
     expect(html).toContain("elapsed=5;elapsed<=30;elapsed+=5");
     expect(html).toContain("setTimeout(r,5000)");
     expect(html).toContain("showPopup('Restart requested. Checking bridge state in '");
-    expect(html).toContain("if(state?.ok&&!state.dirty&&!state.restartRequired){hidePopup(); return true;}");
+    expect(html).toContain(
+      "if(state?.ok&&!state.dirty&&!state.restartRequired){hidePopup(); return true;}",
+    );
     expect(html).toContain("Restart did not finish cleanly within 30s");
     expect(html).not.toContain("if($('online').textContent==='online')return");
   });