fix: ruff lint - datetime.UTC, X|None, E501, B904, F821

Author: Revenue Holdings

.github/dependabot.yml

@@ -0,0 +1,12 @@
+version: 2
+updates:
+  - package-ecosystem: pip
+    directory: "/"
+    schedule:
+      interval: weekly
+    open-pull-requests-limit: 5
+  - package-ecosystem: github-actions
+    directory: "/"
+    schedule:
+      interval: weekly
+    open-pull-requests-limit: 3

.github/workflows/publish.yml

@@ -24,6 +24,9 @@ jobs:
           python -m pip install --upgrade pip
           pip install build twine
 
+      - name: Lint with ruff
+        run: pip install ruff && ruff check src/ --target-version py310
+
       - name: Build package
         run: |
           python -m build

.gitignore

@@ -70,3 +70,4 @@ Thumbs.db
 # Project specific
 research/
 fixtures/generated/
+.ruff_cache/

fixtures/sample.alembic.py

@@ -1,11 +1,11 @@
 """Initial schema
 
 Revision ID: sample
-Revises: 
+Revises:
 Create Date: 2026-05-15 03:00:00.000000
 """
-from alembic import op
 import sqlalchemy as sa
+from alembic import op
 
 # revision identifiers, used by Alembic.
 revision = 'sample'
@@ -18,7 +18,7 @@ def upgrade() -> None:
         sa.Column('name', sa.String(100), nullable=False),
         sa.Column('email', sa.String(255), nullable=False, unique=True),
         sa.Column('role', sa.Enum('admin', 'editor', 'viewer'), nullable=False),
-        sa.Column('is_active', sa.Boolean(), server_default=true),
+        sa.Column('is_active', sa.Boolean(), server_default=True),
         sa.Column('created_at', sa.DateTime(), server_default=sa.func.now()),
     )
 

fixtures/sample.sqlalchemy.py

@@ -1,13 +1,24 @@
 """SchemaForge Demo: Blog Schema (SQLAlchemy Declarative Models)
-Convert to any ORM format with: schemaforge convert --from sqlalchemy --to <format> --input fixtures/sample.sqlalchemy.py
+Convert to any ORM format with:
+  schemaforge convert --from sqlalchemy --to <format> --input fixtures/sample.sqlalchemy.py
 """
 
+import enum
+from sqlalchemy import (
+    Boolean,
+    Column,
+    DateTime,
+    ForeignKey,
+    Integer,
+    Numeric,
+    String,
+    Text,
+    func,
+)
 from sqlalchemy import (
-    Column, Integer, String, Boolean, DateTime, Text, Float, Numeric,
-    ForeignKey, Enum as SAEnum, func,
+    Enum as SAEnum,
 )
 from sqlalchemy.orm import declarative_base
-import enum
 
 Base = declarative_base()
 

scripts/check_consistency.py

@@ -1,11 +1,10 @@
 """Schema consistency check script for CI/CD pipelines."""
-import sys
 import os
+import sys
 
 sys.path.insert(0, os.path.join(os.path.dirname(__file__), ".."))
 
 from schemaforge.convert import convert_schema
-from schemaforge.check import check_directory
 
 
 def main():

src/schemaforge/cli.py

@@ -20,7 +20,10 @@ def decorator(func):
 from .type_config import TypeConfig
 
 # All supported format names (used for CLI choices and detection)
-_FORMATS = ["sql", "prisma", "drizzle", "typeorm", "django", "sqlalchemy", "alembic", "json_schema", "graphql", "ef", "scala"]
+_FORMATS = [
+    "sql", "prisma", "drizzle", "typeorm", "django",
+    "sqlalchemy", "alembic", "json_schema", "graphql", "ef", "scala",
+]
 
 
 @click.group()

src/schemaforge/mcp_server.py

@@ -23,7 +23,10 @@
 
 
 # All supported formats
-_FORMATS = ["sql", "prisma", "drizzle", "typeorm", "django", "sqlalchemy", "alembic", "json_schema", "graphql", "ef", "scala"]
+_FORMATS = [
+    "sql", "prisma", "drizzle", "typeorm", "django",
+    "sqlalchemy", "alembic", "json_schema", "graphql", "ef", "scala",
+]
 _FORMAT_DESCRIPTIONS = {
     "sql": "SQL DDL (Data Definition Language)",
     "prisma": "Prisma schema",

src/schemaforge/parsers/sql_parser.py

@@ -31,7 +31,11 @@ def parse(self, text: str) -> Schema:
             if not stmt:
                 continue
             upper = stmt.upper()
-            if upper.startswith("CREATE TABLE") or upper.startswith("CREATE TEMPORARY TABLE") or upper.startswith("CREATE OR REPLACE TABLE"):
+            if (
+                upper.startswith("CREATE TABLE")
+                or upper.startswith("CREATE TEMPORARY TABLE")
+                or upper.startswith("CREATE OR REPLACE TABLE")
+            ):
                 table = self._parse_create_table(stmt)
                 if table:
                     schema.tables.append(table)
@@ -196,8 +200,19 @@ def _parse_column_def(self, defn: str) -> Column | None:
         """Parse a column definition."""
         # Skip table constraints
         upper = defn.upper()
-        if (any(kw in upper for kw in ["PRIMARY KEY", "FOREIGN KEY", "INDEX", "KEY", "CHECK", "UNIQUE", "CONSTRAINT"])
-                and (not defn.split()[0].isidentifier() or defn.split()[0].upper() in ("PRIMARY", "FOREIGN", "INDEX", "KEY", "CHECK", "UNIQUE", "CONSTRAINT"))):
+        if (
+            any(kw in upper for kw in [
+                "PRIMARY KEY", "FOREIGN KEY", "INDEX",
+                "KEY", "CHECK", "UNIQUE", "CONSTRAINT",
+            ])
+            and (
+                not defn.split()[0].isidentifier()
+                or defn.split()[0].upper() in (
+                    "PRIMARY", "FOREIGN", "INDEX",
+                    "KEY", "CHECK", "UNIQUE", "CONSTRAINT",
+                )
+            )
+        ):
             return None
 
         # Parse: column_name TYPE [(args)] [constraints...]

src/schemaforge/parsers/typeorm_parser.py

@@ -91,7 +91,10 @@ def _split_entities(self, text: str) -> list[str]:
             for line in text.split("\n"):
                 stripped = line.strip()
                 # Detect @Entity, @ViewEntity, or bare class extends pattern
-                if (stripped.startswith("@") or stripped.startswith("export class")) and (stripped.startswith("export class") or "class " in stripped):
+                if (
+            (stripped.startswith("@") or stripped.startswith("export class"))
+            and (stripped.startswith("export class") or "class " in stripped)
+        ):
                     in_class = True
                     current = line + "\n"
                     brace_depth = 0