chore: use mypy for lint check

Author: avallete

.github/workflows/validate-python-types.yml

@@ -33,7 +33,7 @@ jobs:
 
       - name: Install Python dependencies
         run: |
-          pip install pydantic typing-extensions
+          pip install pydantic mypy
 
       - name: Start test database
         working-directory: test/db
@@ -57,46 +57,13 @@ jobs:
           echo "Generated Python types (first 30 lines):"
           head -30 generated_types.py
 
-      - name: Validate Python types syntax
+      - name: Validate Python types runtime
         run: |
-          python -m py_compile generated_types.py
-          echo "✓ Python syntax is valid"
+          python -c "import generated_types; print('✓ Generated Python types are valid and can be imported')"
 
-      - name: Validate Python types imports
+      - name: Validate Python types with mypy
         run: |
-          python << 'EOF'
-          import sys
-          import importlib.util
-
-          try:
-              # Load the module from file
-              spec = importlib.util.spec_from_file_location("generated_types", "generated_types.py")
-              if spec is None or spec.loader is None:
-                  print("✗ Failed to create module spec")
-                  sys.exit(1)
-              
-              module = importlib.util.module_from_spec(spec)
-              spec.loader.exec_module(module)
-              
-              print("✓ All imports are valid")
-              print("✓ Generated types can be imported successfully")
-              print(f"✓ Module loaded: {module.__name__}")
-          except ImportError as e:
-              print(f"✗ Import error: {e}")
-              import traceback
-              traceback.print_exc()
-              sys.exit(1)
-          except SyntaxError as e:
-              print(f"✗ Syntax error: {e}")
-              import traceback
-              traceback.print_exc()
-              sys.exit(1)
-          except Exception as e:
-              print(f"✗ Unexpected error: {e}")
-              import traceback
-              traceback.print_exc()
-              sys.exit(1)
-          EOF
+          mypy generated_types.py --strict
 
       - name: Cleanup
         if: always()

test/lib/tables.ts

@@ -37,7 +37,8 @@ test('list', async () => {
       id: expect.any(Number),
       live_rows_estimate: expect.any(Number),
       size: expect.any(String),
-    }, `
+    },
+    `
     {
       "bytes": Any<Number>,
       "columns": [
@@ -173,7 +174,8 @@ test('list', async () => {
       "schema": "public",
       "size": Any<String>,
     }
-  `)
+  `
+  )
 })
 
 test('list without columns', async () => {

test/lib/types.ts

@@ -64,7 +64,8 @@ test('list types with include Table Types', async () => {
   })
 
   expect(res.data?.find(({ name }) => name === 'todos')).toMatchInlineSnapshot(
-    { id: expect.any(Number) }, `
+    { id: expect.any(Number) },
+    `
     {
       "attributes": [],
       "comment": null,
@@ -75,7 +76,8 @@ test('list types with include Table Types', async () => {
       "schema": "public",
       "type_relation_id": 16403,
     }
-  `)
+  `
+  )
 })
 
 test('list types without Table Types', async () => {

test/lib/views.ts

@@ -4,7 +4,8 @@ import { pgMeta } from './utils'
 test('list', async () => {
   const res = await pgMeta.views.list()
   expect(res.data?.find(({ name }) => name === 'todos_view')).toMatchInlineSnapshot(
-    { id: expect.any(Number) }, `
+    { id: expect.any(Number) },
+    `
     {
       "columns": [
         {
@@ -74,7 +75,8 @@ test('list', async () => {
       "name": "todos_view",
       "schema": "public",
     }
-  `)
+  `
+  )
 })
 
 test('list without columns', async () => {
@@ -98,7 +100,8 @@ test('list without columns', async () => {
 test('retrieve', async () => {
   const res = await pgMeta.views.retrieve({ schema: 'public', name: 'todos_view' })
   expect(res).toMatchInlineSnapshot(
-    { data: { id: expect.any(Number) } }, `
+    { data: { id: expect.any(Number) } },
+    `
     {
       "data": {
         "columns": [
@@ -171,5 +174,6 @@ test('retrieve', async () => {
       },
       "error": null,
     }
-  `)
+  `
+  )
 })