Add ESLint overrides and GitHub Actions workflow for linting

- Updated .eslintrc.cjs to disable the no-unused-vars rule for test files located in tests//.js and utils//.test.js.
-

Author: ryota-murakami

.eslintrc.cjs

@@ -10,5 +10,13 @@ module.exports = {
   plugins: [],
   rules: {},
   settings: {},
-  overrides: [],
+  overrides: [
+    {
+      // Disable the no-unused-vars rule for test files
+      files: ['tests/**/*.js', 'utils/**/*.test.js'],
+      rules: {
+        '@typescript-eslint/no-unused-vars': 'off',
+      },
+    },
+  ],
 }

.github/workflows/lint.yml

@@ -0,0 +1,13 @@
+name: Lint
+on:
+  pull_request: ~
+  push:
+    branches:
+      - main
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: ./.github/actions/prepare
+      - run: pnpm lint

index.js

@@ -122,12 +122,11 @@ const gptCommit = async () => {
   }
 }
 
-const gitExtension = (args) => {
+const gitExtension = (_args) => {
   // Load configuration at startup
   loadConfig()
 
-  // Extract the command and arguments from the command line
-  const [command, ...rest] = args
+  // No need to extract command and args since we're using Commander
 
   program
     .command('commit')

tests/index.test.js

@@ -1,6 +1,3 @@
-import fs from 'fs'
-import path from 'path'
-
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
 
 import * as gitGptCommit from '../index.js'

tests/setup-mocks.js

@@ -24,7 +24,7 @@ vi.mock('openai', () => {
 // Mock the entire index.js module
 vi.mock('../index.js', () => {
   return {
-    getGitSummary: vi.fn((options) => {
+    getGitSummary: vi.fn((_options) => {
       try {
         // Check if there are file changes without actually executing the diff command
         const gitStatus = require('child_process')
@@ -37,11 +37,11 @@ vi.mock('../index.js', () => {
 
         // Return mocked diff content
         return `diff --git a/file1.js b/file1.js\nindex 123456..789012 100644\n--- a/file1.js\n+++ b/file1.js\n@@ -1,5 +1,8 @@\nfunction greet(name) {\n-  return \`Hello, \${name}!\`;\n+  // Add default value when name is empty\n+  const userName = name || 'Guest';\n+  return \`Hello, \${userName}!\`;\n }`
-      } catch (error) {
+      } catch (__) {
         throw new Error('Failed to get git summary')
       }
     }),
-    gptCommit: vi.fn(async (options = {}) => {
+    gptCommit: vi.fn(async (_options = {}) => {
       return 'Mock commit message'
     }),
     gitExtension: vi.fn(),