chore: enable lint and CI

Author: ymc9

.eslintignore

@@ -0,0 +1 @@
+**/dist/**

.eslintrc.json

@@ -0,0 +1,22 @@
+{
+    "root": true,
+    "parser": "@typescript-eslint/parser",
+    "parserOptions": {
+        "ecmaVersion": 6,
+        "sourceType": "module"
+    },
+    "plugins": ["@typescript-eslint"],
+    "extends": [
+        "eslint:recommended",
+        "plugin:@typescript-eslint/eslint-recommended",
+        "plugin:@typescript-eslint/recommended"
+    ],
+    "rules": {
+        "@typescript-eslint/no-unused-vars": [
+            "error",
+            { "varsIgnorePattern": "^_", "argsIgnorePattern": "^_" }
+        ],
+        "@typescript-eslint/no-explicit-any": "off",
+        "@typescript-eslint/ban-ts-comment": "off"
+    }
+}

.github/workflows/build-test.yml

@@ -0,0 +1,77 @@
+name: CI
+
+on:
+    pull_request:
+        branches:
+            - main
+            - dev
+
+permissions:
+    contents: read
+
+jobs:
+    build-test:
+        runs-on: buildjet-8vcpu-ubuntu-2204
+
+        services:
+            postgres:
+                image: postgres
+                env:
+                    POSTGRES_PASSWORD: abc123
+                # Set health checks to wait until postgres has started
+                options: >-
+                    --health-cmd pg_isready
+                    --health-interval 10s
+                    --health-timeout 5s
+                    --health-retries 5
+                ports:
+                    - 5432:5432
+
+        strategy:
+            matrix:
+                node-version: [20.x]
+
+        steps:
+            - name: Checkout
+              uses: actions/checkout@v3
+
+            - name: Install pnpm
+              uses: pnpm/action-setup@v2
+              with:
+                  version: 10.12.1
+
+            - name: Use Node.js ${{ matrix.node-version }}
+              uses: buildjet/setup-node@v3
+              with:
+                  node-version: ${{ matrix.node-version }}
+                  cache: 'pnpm'
+
+            - name: Get pnpm store directory
+              id: pnpm-cache
+              shell: bash
+              run: |
+                  echo "STORE_PATH=$(pnpm store path)" >> $GITHUB_OUTPUT
+
+            - name: Setup pnpm cache
+              uses: buildjet/cache@v3
+              with:
+                  path: ${{ steps.pnpm-cache.outputs.STORE_PATH }}
+                  key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
+                  restore-keys: |
+                      ${{ runner.os }}-pnpm-store-
+
+            - name: Install dependencies
+              run: pnpm install --frozen-lockfile
+
+            - name: Build
+              run: pnpm run build
+
+            - name: Lint
+              run: pnpm run lint
+
+            # install again for internal dependencies
+            - name: Install internal dependencies
+              run: pnpm install --frozen-lockfile
+
+            - name: Test
+              run: pnpm run test

package.json

@@ -2,17 +2,19 @@
     "name": "zenstack-v3",
     "version": "3.0.0-alpha.1",
     "description": "ZenStack",
-    "packageManager": "pnpm@10.0.0",
+    "packageManager": "pnpm@10.12.1",
     "scripts": {
         "build": "pnpm -r build",
         "watch": "pnpm -r --parallel watch",
+        "lint": "pnpm -r lint",
         "test": "pnpm vitest"
     },
     "keywords": [],
     "author": "",
     "license": "ISC",
     "devDependencies": {
         "@swc/core": "^1.10.15",
+        "eslint": "~8.57.1",
         "npm-run-all": "^4.1.5",
         "tsup": "^8.3.5",
         "tsx": "^4.19.2",

packages/cli/package.json

@@ -23,6 +23,7 @@
     "scripts": {
         "build": "tsup-node",
         "watch": "tsup-node --watch",
+        "lint": "eslint src --ext ts",
         "test": "vitest",
         "pack": "pnpm pack"
     },

packages/language/.eslintrc.json

@@ -1,12 +1,10 @@
 import { AstUtils, type AstNode, type ValidationAcceptor } from 'langium';
 import {
     BinaryExpr,
-    DataModelAttribute,
     Expression,
     isArrayExpr,
     isDataModel,
     isDataModelAttribute,
-    isDataModelField,
     isEnum,
     isLiteralExpr,
     isMemberAccessExpr,
@@ -18,7 +16,6 @@ import {
 
 import {
     findUpAst,
-    getAttributeArgLiteral,
     isAuthInvocation,
     isAuthOrAuthMemberAccess,
     isDataModelFieldReference,

packages/language/src/validators/expression-validator.ts

@@ -1,10 +1,8 @@
 import type { LangiumDocuments, ValidationAcceptor } from 'langium';
 import { PLUGIN_MODULE_NAME, STD_LIB_MODULE_NAME } from '../constants';
-import { isDataModel, isDataSource, type Model } from '../generated/ast';
+import { isDataSource, type Model } from '../generated/ast';
 import {
     getAllDeclarationsIncludingImports,
-    getDataModelAndTypeDefs,
-    hasAttribute,
     resolveImport,
     resolveTransitiveImports,
 } from '../utils';

packages/language/src/validators/schema-validator.ts

@@ -7,6 +7,7 @@
         "build": "tsup-node && pnpm test-typecheck",
         "test-typecheck": "tsc --project tsconfig.test.json",
         "watch": "tsup-node --watch",
+        "lint": "eslint src --ext ts",
         "test": "vitest",
         "pack": "pnpm pack"
     },
@@ -85,6 +86,7 @@
     "dependencies": {
         "@paralleldrive/cuid2": "^2.2.2",
         "decimal.js": "^10.4.3",
+        "is-plain-object": "^5.0.0",
         "kysely": "^0.27.5",
         "nanoid": "^5.0.9",
         "pg-connection-string": "^2.9.0",
@@ -112,8 +114,8 @@
         "@types/pg": "^8.0.0",
         "@types/tmp": "^0.2.6",
         "@zenstackhq/language": "workspace:*",
-        "@zenstackhq/testtools": "workspace:*",
         "@zenstackhq/sdk": "workspace:*",
+        "@zenstackhq/testtools": "workspace:*",
         "tmp": "^0.2.3"
     }
 }

packages/runtime/package.json

@@ -199,6 +199,7 @@ export class ClientImpl<Schema extends SchemaDef> {
             );
         }
 
+        // eslint-disable-next-line @typescript-eslint/ban-types
         return (procOptions[name] as Function).apply(this, [this, ...args]);
     }