added mcp ci config, upgrade all packages

Author: ndbroadbent

.github/workflows/mcp.yml

@@ -0,0 +1,86 @@
+name: MCP Server CI
+
+on:
+  push:
+    branches: [main]
+    paths:
+      - 'renamify-mcp/**'
+      - '.github/workflows/mcp.yml'
+  pull_request:
+    branches: [main]
+    paths:
+      - 'renamify-mcp/**'
+      - '.github/workflows/mcp.yml'
+
+defaults:
+  run:
+    working-directory: renamify-mcp
+
+jobs:
+  test:
+    name: Test MCP Server
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
+        node-version: [18.x, 20.x]
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: pnpm/action-setup@v2
+        with:
+          version: 8
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+          cache: 'pnpm'
+          cache-dependency-path: renamify-mcp/pnpm-lock.yaml
+
+      - name: Install dependencies
+        run: pnpm install --frozen-lockfile
+
+      - name: Run TypeScript checks
+        run: pnpm exec tsc --noEmit
+
+      - name: Run Biome checks
+        run: pnpm run check
+
+      - name: Run tests
+        run: pnpm test
+
+      - name: Build
+        run: pnpm run build
+
+  coverage:
+    name: Code Coverage
+    runs-on: ubuntu-latest
+    
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: pnpm/action-setup@v2
+        with:
+          version: 8
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20.x
+          cache: 'pnpm'
+          cache-dependency-path: renamify-mcp/pnpm-lock.yaml
+
+      - name: Install dependencies
+        run: pnpm install --frozen-lockfile
+
+      - name: Run tests with coverage
+        run: pnpm exec vitest run --coverage
+
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v3
+        with:
+          files: ./renamify-mcp/coverage/coverage-final.json
+          flags: mcp
+          name: mcp-coverage

lefthook.yml

@@ -33,6 +33,11 @@ pre-commit:
       glob: "*.ts"
       run: pnpm run test
 
+    mcp-tsc:
+      root: renamify-mcp/
+      glob: "*.ts"
+      run: tsc --noEmit
+
 # pre-push:
 #   commands:
 #     test-all:

renamify-mcp/RELEASES.md

@@ -46,10 +46,10 @@ cd renamify-mcp
 npm login
 
 # Test the publish (dry run)
-npm publish --access public --dry-run
+npm publish --dry-run
 
 # Actually publish
-npm publish --access public
+npm publish
 ```
 
 ## Release Process

renamify-mcp/biome.jsonc

@@ -1,4 +1,16 @@
 {
-  "$schema": "https://biomejs.dev/schemas/2.1.4/schema.json",
-  "extends": ["ultracite"]
+  "$schema": "https://biomejs.dev/schemas/2.2.0/schema.json",
+  "extends": ["ultracite"],
+  "overrides": [
+    {
+      "includes": ["**/*.test.ts", "**/*.spec.ts"],
+      "linter": {
+        "rules": {
+          "style": {
+            "noMagicNumbers": "off"
+          }
+        }
+      }
+    }
+  ]
 }

renamify-mcp/package.json

@@ -40,16 +40,17 @@
   "license": "MIT",
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.17.3",
-    "execa": "^8.0.1",
-    "zod": "^3.22.0"
+    "execa": "^9.6.0",
+    "zod": "^3.25.76"
   },
   "devDependencies": {
-    "@biomejs/biome": "2.1.2",
-    "@types/node": "^20.0.0",
-    "tsx": "^4.0.0",
-    "typescript": "^5.0.0",
-    "ultracite": "5.1.2",
-    "vitest": "^1.0.0"
+    "@biomejs/biome": "2.2.0",
+    "@types/node": "^24.3.0",
+    "@vitest/coverage-v8": "^3.2.4",
+    "tsx": "^4.20.4",
+    "typescript": "^5.9.2",
+    "ultracite": "5.1.8",
+    "vitest": "^3.2.4"
   },
   "engines": {
     "node": ">=18.0.0"

renamify-mcp/src/renamify-service.ts

@@ -2,7 +2,7 @@ import { access } from 'node:fs/promises';
 import { join } from 'node:path';
 import { type ExecaError, execa } from 'execa';
 
-export interface PlanOptions {
+export type PlanOptions = {
   old: string;
   new: string;
   includes?: string[];
@@ -12,23 +12,23 @@ export interface PlanOptions {
   dryRun?: boolean;
   renameFiles?: boolean;
   renameDirs?: boolean;
-}
+};
 
-export interface ApplyOptions {
+export type ApplyOptions = {
   planId?: string;
   planPath?: string;
   atomic?: boolean;
   commit?: boolean;
-}
+};
 
-export interface PreviewOptions {
+export type PreviewOptions = {
   planId?: string;
   planPath?: string;
   format?: 'table' | 'diff' | 'json' | 'summary';
-}
+};
 
 export class RenamifyService {
-  private renamifyPath: string;
+  private readonly renamifyPath: string;
 
   constructor(renamifyPath?: string) {
     // Default to 'renamify' which should be in PATH
@@ -70,23 +70,23 @@ export class RenamifyService {
   }
 
   private addIncludeArgs(args: string[], includes?: string[]): void {
-    if (includes && includes.length > 0) {
+    if (includes?.length) {
       for (const pattern of includes) {
         args.push('--include', pattern);
       }
     }
   }
 
   private addExcludeArgs(args: string[], excludes?: string[]): void {
-    if (excludes && excludes.length > 0) {
+    if (excludes?.length) {
       for (const pattern of excludes) {
         args.push('--exclude', pattern);
       }
     }
   }
 
   private addStylesArg(args: string[], styles?: string[]): void {
-    if (styles && styles.length > 0) {
+    if (styles?.length) {
       args.push('--styles', styles.join(','));
     }
   }
@@ -165,15 +165,23 @@ export class RenamifyService {
   /**
    * Undo a renaming
    */
-  async undo(id: string): Promise<string> {
-    return await this.executeCommand(['undo', id], 'undo');
+  async undo(id?: string): Promise<string> {
+    const args = ['undo'];
+    if (id) {
+      args.push(id);
+    }
+    return await this.executeCommand(args, 'undo');
   }
 
   /**
    * Redo a renaming
    */
-  async redo(id: string): Promise<string> {
-    return await this.executeCommand(['redo', id], 'redo');
+  async redo(id?: string): Promise<string> {
+    const args = ['redo'];
+    if (id) {
+      args.push(id);
+    }
+    return await this.executeCommand(args, 'redo');
   }
 
   /**

renamify-mcp/src/tools.ts

@@ -6,7 +6,7 @@ import type {
 } from './renamify-service.js';
 
 export class RenamifyTools {
-  private renamifyService: RenamifyService;
+  private readonly renamifyService: RenamifyService;
 
   constructor(renamifyService: RenamifyService) {
     this.renamifyService = renamifyService;
@@ -82,7 +82,7 @@ TROUBLESHOOTING:
   /**
    * Undo a renaming
    */
-  async undo(params: { id: string }): Promise<string> {
+  async undo(params: { id?: string }): Promise<string> {
     try {
       const result = await this.renamifyService.undo(params.id);
       return `${result}
@@ -96,7 +96,7 @@ Renaming has been undone successfully.`;
   /**
    * Redo a renaming
    */
-  async redo(params: { id: string }): Promise<string> {
+  async redo(params: { id?: string }): Promise<string> {
     try {
       const result = await this.renamifyService.redo(params.id);
       return `${result}

renamify-mcp/vitest.config.ts

@@ -6,14 +6,21 @@ export default defineConfig({
     environment: 'node',
     coverage: {
       provider: 'v8',
-      reporter: ['text', 'json', 'html'],
+      reporter: ['text', 'json', 'html', 'lcov'],
       exclude: [
         'node_modules/**',
         'dist/**',
         '**/*.test.ts',
         '**/*.spec.ts',
         'vitest.config.ts',
+        'biome.config.ts',
       ],
+      thresholds: {
+        branches: 80,
+        functions: 80,
+        lines: 80,
+        statements: 80,
+      },
     },
   },
 });