Add Biome linter/formatter (#27)

Author: vladavoX

.github/workflows/ci.yml

@@ -6,16 +6,34 @@ on:
   pull_request:
 
 jobs:
+  quality:
+    name: Biome Check
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v6
+        with:
+          persist-credentials: false
+      - name: Setup Biome
+        uses: biomejs/setup-biome@v2
+        with:
+          version: latest
+      - name: Run Biome
+        run: biome ci .
+
   build-mcp-server:
     name: Build MCP Server
+    needs: quality
     runs-on: ubuntu-latest
 
     steps:
       - name: Checkout
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Setup Node
-        uses: actions/setup-node@v5
+        uses: actions/setup-node@v6
         with:
           node-version: "22"
           cache: "yarn"

biome.json

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

package.json

@@ -39,7 +39,9 @@
     "dev": "vite build --watch",
     "build": "tsc --noEmit && vite build",
     "typecheck": "tsc --noEmit",
-    "start": "node dist/index.js"
+    "start": "node dist/index.js",
+    "biome:check": "biome check",
+    "biome:fix": "biome check --write"
   },
   "packageManager": "yarn@4.9.2",
   "dependencies": {
@@ -48,6 +50,7 @@
     "zod": "^3.25.76"
   },
   "devDependencies": {
+    "@biomejs/biome": "2.3.8",
     "@smithery/sdk": "^1.7.5",
     "@types/node": "^22.10.6",
     "typescript": "^5.7.3",

src/axiosConfig.ts

@@ -1,11 +1,8 @@
-import axios, { AxiosInstance } from "axios";
+import axios, { type AxiosInstance } from "axios";
 import { PACKAGE_NAME, PACKAGE_VERSION } from "./contants";
 import { PlainlyApiAuthenticationError, PlainlyApiError } from "./sdk/errors";
 
-export function createApiClient(config: {
-  baseUrl: string;
-  apiKey: string;
-}): AxiosInstance {
+export function createApiClient(config: { baseUrl: string; apiKey: string }): AxiosInstance {
   const baseUrl = config.baseUrl;
   const apiKey = config.apiKey;
 
@@ -40,7 +37,7 @@ export function createApiClient(config: {
       }
       // Network or other errors
       return Promise.reject(error);
-    }
+    },
   );
 
   return instance;

src/cli.ts

@@ -1,8 +1,8 @@
 #!/usr/bin/env node
 
-import { PlainlyMcpServer } from "./server";
-import env from "./env";
 import { createApiClient } from "./axiosConfig";
+import env from "./env";
+import { PlainlyMcpServer } from "./server";
 
 // Validate required environment variables
 if (!env.PLAINLY_API_KEY) {

src/env.ts

@@ -5,9 +5,7 @@ type Env = {
 };
 
 export default {
-  PLAINLY_APP_URL:
-    process.env.PLAINLY_APP_URL || "https://app.plainlyvideos.com",
-  PLAINLY_API_URL:
-    process.env.PLAINLY_API_URL || "https://api.plainlyvideos.com",
+  PLAINLY_APP_URL: process.env.PLAINLY_APP_URL || "https://app.plainlyvideos.com",
+  PLAINLY_API_URL: process.env.PLAINLY_API_URL || "https://api.plainlyvideos.com",
   PLAINLY_API_KEY: process.env.PLAINLY_API_KEY,
 } as Env;

src/sdk/api/getRenderItem.ts

@@ -1,13 +1,8 @@
-import { AxiosInstance, AxiosResponse } from "axios";
-import { Render } from "../types";
+import type { AxiosInstance, AxiosResponse } from "axios";
+import type { Render } from "../types";
 
-export const getRenderItem = async (
-  client: AxiosInstance,
-  renderingId: string
-): Promise<Render> => {
-  const response = await client.get<Render, AxiosResponse<Render>, void>(
-    `/api/v2/renders/${renderingId}`
-  );
+export const getRenderItem = async (client: AxiosInstance, renderingId: string): Promise<Render> => {
+  const response = await client.get<Render, AxiosResponse<Render>, void>(`/api/v2/renders/${renderingId}`);
 
   return response.data;
 };

src/sdk/api/getRenderableItemDetails.ts

@@ -1,5 +1,5 @@
-import { AxiosInstance } from "axios";
-import {
+import type { AxiosInstance } from "axios";
+import type {
   DesignDetails,
   DesignVariant,
   Layer,
@@ -11,7 +11,7 @@ import {
 export const getRenderableItemsDetails = async (
   client: AxiosInstance,
   renderableItemId: string,
-  isDesign: boolean
+  isDesign: boolean,
 ): Promise<RenderableItemDetails[]> => {
   if (isDesign) {
     return await getDesignVariants(client, renderableItemId);
@@ -20,13 +20,8 @@ export const getRenderableItemsDetails = async (
   }
 };
 
-const getDesignVariants = async (
-  client: AxiosInstance,
-  designId: string
-): Promise<RenderableItemDetails[]> => {
-  const response = await client.get<DesignDetails>(
-    `/api/v2/designs/${designId}`
-  );
+const getDesignVariants = async (client: AxiosInstance, designId: string): Promise<RenderableItemDetails[]> => {
+  const response = await client.get<DesignDetails>(`/api/v2/designs/${designId}`);
   const designDetails = response.data;
 
   // flatten variants into renderable items with parameter details
@@ -46,13 +41,8 @@ const getDesignVariants = async (
   }));
 };
 
-const getProjectTemplates = async (
-  client: AxiosInstance,
-  projectId: string
-): Promise<RenderableItemDetails[]> => {
-  const response = await client.get<ProjectDetails>(
-    `/api/v2/projects/${projectId}`
-  );
+const getProjectTemplates = async (client: AxiosInstance, projectId: string): Promise<RenderableItemDetails[]> => {
+  const response = await client.get<ProjectDetails>(`/api/v2/projects/${projectId}`);
   const projectDetails = response.data;
 
   // flatten templates into renderable items with parameter details
@@ -72,11 +62,11 @@ const getProjectTemplates = async (
 };
 
 const isDynamicLayer = (
-  layer: Layer
+  layer: Layer,
 ): layer is Layer & {
   parametrization: { value: string; mandatory: boolean };
 } => {
-  return layer.parametrization != null && layer.parametrization.expression;
+  return !!layer.parametrization?.expression;
 };
 
 const getExampleVideoUrl = (variant: DesignVariant): string | undefined => {

src/sdk/api/listRenderableItems.ts

@@ -1,32 +1,21 @@
-import { AxiosInstance } from "axios";
+import type { AxiosInstance } from "axios";
 import { getAspectRatio } from "../../utils/aspectRatio";
-import {
-  Design,
-  Project,
-  RenderableItem,
-  RenderableItemsListOptions,
-} from "../types";
+import type { Design, Project, RenderableItem, RenderableItemsListOptions } from "../types";
 
 export const listRenderableItems = async (
   client: AxiosInstance,
   options: RenderableItemsListOptions = {
     excludeDesigns: false,
     excludeProjects: false,
-  }
+  },
 ): Promise<RenderableItem[]> => {
-  const designs = options.excludeDesigns
-    ? []
-    : await listRenderableDesigns(client);
-  const projects = options.excludeProjects
-    ? []
-    : await listRenderableProjects(client);
+  const designs = options.excludeDesigns ? [] : await listRenderableDesigns(client);
+  const projects = options.excludeProjects ? [] : await listRenderableProjects(client);
 
   return [...projects, ...designs];
 };
 
-const listRenderableDesigns = async (
-  client: AxiosInstance
-): Promise<RenderableItem[]> => {
+const listRenderableDesigns = async (client: AxiosInstance): Promise<RenderableItem[]> => {
   const response = await client.get<Design[]>("/api/v2/designs");
 
   return response.data
@@ -46,9 +35,7 @@ const listRenderableDesigns = async (
     }));
 };
 
-const listRenderableProjects = async (
-  client: AxiosInstance
-): Promise<RenderableItem[]> => {
+const listRenderableProjects = async (client: AxiosInstance): Promise<RenderableItem[]> => {
   const response = await client.get<Project[]>("/api/v2/projects");
 
   return response.data
@@ -62,10 +49,7 @@ const listRenderableProjects = async (
       templates: project.templates.map((template) => ({
         id: template.id,
         name: template.name,
-        aspectRatio: getAspectRatio(
-          template.resolution.width,
-          template.resolution.height
-        ),
+        aspectRatio: getAspectRatio(template.resolution.width, template.resolution.height),
         durationSeconds: template.duration,
       })),
     }));

src/sdk/api/renderItem.ts

@@ -1,16 +1,9 @@
-import { AxiosInstance, AxiosResponse } from "axios";
-import { ProjectRenderDto, Render, RenderItemParams } from "../types";
+import type { AxiosInstance, AxiosResponse } from "axios";
 import { PACKAGE_NAME } from "../../contants";
+import type { ProjectRenderDto, Render, RenderItemParams } from "../types";
 
-export const renderItem = async (
-  client: AxiosInstance,
-  params: RenderItemParams
-): Promise<Render> => {
-  const response = await client.post<
-    Render,
-    AxiosResponse<Render>,
-    ProjectRenderDto
-  >("/api/v2/renders", {
+export const renderItem = async (client: AxiosInstance, params: RenderItemParams): Promise<Render> => {
+  const response = await client.post<Render, AxiosResponse<Render>, ProjectRenderDto>("/api/v2/renders", {
     projectId: params.projectDesignId,
     templateId: params.templateVariantId,
     parameters: params.parameters,