Merge pull request #68 from phantom5099/workflow

add check and fix test failed

Author: phantom5099

.github/workflows/pr-check.yml

@@ -0,0 +1,102 @@
+name: PR Check
+
+on:
+  pull_request:
+    branches: [main, master]
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number }}
+  cancel-in-progress: true
+
+permissions:
+  contents: read
+
+jobs:
+  changes:
+    runs-on: ubuntu-latest
+    outputs:
+      code: ${{ steps.filter.outputs.code }}
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dorny/paths-filter@v3
+        id: filter
+        with:
+          filters: |
+            code:
+              - 'packages/**'
+              - 'package.json'
+              - 'pnpm-lock.yaml'
+              - 'tsconfig*.json'
+              - 'vitest.config.ts'
+              - 'eslint.config.mjs'
+              - '.prettierrc'
+
+  lint:
+    needs: changes
+    if: ${{ needs.changes.outputs.code == 'true' }}
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: pnpm/action-setup@v3
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '22'
+          cache: 'pnpm'
+      - run: pnpm install
+      - run: pnpm run lint
+
+  format-check:
+    needs: changes
+    if: ${{ needs.changes.outputs.code == 'true' }}
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: pnpm/action-setup@v3
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '22'
+          cache: 'pnpm'
+      - run: pnpm install
+      - run: pnpm run format:check
+
+  typecheck:
+    needs: changes
+    if: ${{ needs.changes.outputs.code == 'true' }}
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: pnpm/action-setup@v3
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '22'
+          cache: 'pnpm'
+      - run: pnpm install
+      - run: pnpm run typecheck
+
+  test:
+    needs: changes
+    if: ${{ needs.changes.outputs.code == 'true' }}
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: pnpm/action-setup@v3
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '22'
+          cache: 'pnpm'
+      - run: pnpm install
+      - run: pnpm vitest run
+
+  build-desktop:
+    needs: [changes, lint, format-check, typecheck, test]
+    if: ${{ needs.changes.outputs.code == 'true' }}
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: pnpm/action-setup@v3
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '22'
+          cache: 'pnpm'
+      - run: pnpm install
+      - run: pnpm --filter @codingcode/desktop run build

.pnpmfile.cjs

@@ -0,0 +1,15 @@
+module.exports = {
+  hooks: {
+    readPackage(pkg) {
+      if (pkg.name === 'vitest' || pkg.name === '@vitest/mocker' || pkg.name === 'vite-node') {
+        if (pkg.dependencies?.vite) {
+          pkg.dependencies.vite = '^6.4.2';
+        }
+        if (pkg.peerDependencies?.vite) {
+          pkg.peerDependencies.vite = '^6.4.2';
+        }
+      }
+      return pkg;
+    }
+  }
+};

.prettierignore

@@ -0,0 +1,6 @@
+dist/
+out/
+node_modules/
+package-lock.json
+pnpm-lock.yaml
+*.md

.prettierrc

@@ -0,0 +1,7 @@
+{
+  "semi": true,
+  "singleQuote": true,
+  "tabWidth": 2,
+  "trailingComma": "es5",
+  "printWidth": 100
+}

eslint.config.mjs

@@ -0,0 +1,26 @@
+import eslint from '@eslint/js';
+import tseslint from 'typescript-eslint';
+
+export default tseslint.config(
+  eslint.configs.recommended,
+  tseslint.configs.recommended,
+  {
+    ignores: ['dist/**', 'out/**', 'node_modules/**', '**/electron/**'],
+  },
+  {
+    rules: {
+      '@typescript-eslint/no-explicit-any': 'off',
+      '@typescript-eslint/no-unused-vars': [
+        'warn',
+        { argsIgnorePattern: '^_', varsIgnorePattern: '^_' },
+      ],
+      'no-empty': ['warn', { allowEmptyCatch: true }],
+      'no-useless-escape': 'warn',
+      'require-yield': 'warn',
+      'no-useless-assignment': 'warn',
+      'prefer-const': 'warn',
+      '@typescript-eslint/ban-ts-comment': 'warn',
+      '@typescript-eslint/no-this-alias': 'warn',
+    },
+  }
+);

package.json

@@ -1,6 +1,7 @@
 {
   "name": "coding-agent",
   "version": "0.1.0",
+  "packageManager": "pnpm@11.3.0",
   "type": "module",
   "workspaces": [
     "packages/infra",
@@ -12,7 +13,11 @@
     "start": "tsx packages/codingcode/src/cli.ts",
     "dev": "tsx --watch packages/codingcode/src/cli.ts",
     "test": "vitest",
-    "typecheck": "tsc --noEmit",
+    "typecheck": "tsc -b",
+    "lint": "eslint \"packages/*/src/**/*.ts\" \"packages/*/src/**/*.tsx\" \"packages/*/test/**/*.ts\"",
+    "lint:fix": "eslint \"packages/*/src/**/*.ts\" \"packages/*/src/**/*.tsx\" \"packages/*/test/**/*.ts\" --fix",
+    "format": "prettier --write \"packages/**/*.{ts,tsx,json}\" \"*.json\" \"*.ts\"",
+    "format:check": "prettier --check \"packages/**/*.{ts,tsx,json}\" \"*.json\" \"*.ts\"",
     "debug": "tsx src/debug.ts"
   },
   "dependencies": {
@@ -35,10 +40,20 @@
     "zod": "^4.4.3"
   },
   "devDependencies": {
+    "@eslint/js": "^10.0.1",
     "@types/node": "^22.0.0",
+    "eslint": "^10.4.1",
     "playwright-core": "^1.60.0",
+    "prettier": "^3.8.3",
     "tsx": "^4.19.0",
     "typescript": "^5.8.0",
+    "typescript-eslint": "^8.60.1",
+    "vite": "^6.4.2",
     "vitest": "^3.0.0"
+  },
+  "pnpm": {
+    "overrides": {
+      "vite": "^6.4.2"
+    }
   }
 }

packages/codingcode/package.json

@@ -11,10 +11,12 @@
   "dependencies": {
     "@ai-sdk/deepseek": "^2.0.35",
     "@ai-sdk/openai": "^3.0.63",
+    "@ai-sdk/provider": "^3.0.0",
+    "@ai-sdk/provider-utils": "^4.0.0",
     "ai": "^6.0.180",
     "effect": "^3.21.2",
     "hono": "^4.12.19",
     "zod": "^4.4.3",
-    "@codingcode/infra": "*"
+    "@codingcode/infra": "workspace:*"
   }
 }