Merge pull request #704 from opsmill/develop

Merge develop into infrahub-develop

Author: ogenstad

.github/workflows/ci.yml

@@ -91,6 +91,8 @@ jobs:
         run: "uv run ruff check ."
       - name: "Linting: ruff format"
         run: "uv run ruff format --check --diff ."
+      - name: "Linting: ty check"
+        run: "uv run ty check ."
 
 
   markdown-lint:

infrahub_sdk/analyzer.py

@@ -30,10 +30,10 @@ class GraphQLOperation(BaseModel):
 
 
 class GraphQLQueryAnalyzer:
-    def __init__(self, query: str, schema: GraphQLSchema | None = None) -> None:
+    def __init__(self, query: str, schema: GraphQLSchema | None = None, document: DocumentNode | None = None) -> None:
         self.query: str = query
         self.schema: GraphQLSchema | None = schema
-        self.document: DocumentNode = parse(self.query)
+        self.document: DocumentNode = document or parse(self.query)
         self._fields: dict | None = None
 
     @property

pyproject.toml

@@ -82,6 +82,7 @@ lint = [
     "mypy==1.11.2",
     "ruff==0.14.5",
     "astroid>=3.1,<4.0",
+    "ty==0.0.4",
 ]
 types = [
     "types-ujson",
@@ -116,6 +117,68 @@ filterwarnings = [
 ]
 addopts = "-vs --cov-report term-missing --cov-report xml --dist loadscope"
 
+[tool.ty]
+
+[tool.ty.environment]
+python-version = "3.10"
+
+[[tool.ty.overrides]]
+include = ["infrahub_sdk/**"]
+
+[tool.ty.overrides.rules]
+##################################################################################################
+# The ignored rules below should be removed once the code has been updated, they are included    #
+# like this so that we can reactivate them one by one.                                           #
+##################################################################################################
+division-by-zero = "ignore"
+invalid-argument-type = "ignore"
+invalid-assignment = "ignore"
+invalid-await = "ignore"
+invalid-return-type = "ignore"
+invalid-type-form = "ignore"
+missing-argument = "ignore"
+no-matching-overload = "ignore"
+possibly-unresolved-reference = "ignore"
+redundant-cast = "ignore"
+too-many-positional-arguments = "ignore"
+type-assertion-failure = "ignore"
+unknown-argument = "ignore"
+unresolved-attribute = "ignore"
+unresolved-import = "ignore"
+unsupported-operator = "ignore"
+
+
+[[tool.ty.overrides]]
+include = ["tests/**"]
+
+[tool.ty.overrides.rules]
+##################################################################################################
+# The ignored rules below should be removed once the code has been updated, they are included    #
+# like this so that we can reactivate them one by one.                                           #
+##################################################################################################
+invalid-argument-type = "ignore"
+invalid-assignment = "ignore"
+invalid-method-override = "ignore"
+invalid-return-type = "ignore"
+no-matching-overload = "ignore"
+non-subscriptable = "ignore"
+not-iterable = "ignore"
+possibly-missing-attribute = "ignore"
+unresolved-attribute = "ignore"
+unresolved-import = "ignore"
+
+
+[[tool.ty.overrides]]
+include = ["docs/**"]
+
+[tool.ty.overrides.rules]
+##################################################################################################
+# The ignored rules below should be removed once the code has been updated, they are included    #
+# like this so that we can reactivate them one by one.                                           #
+##################################################################################################
+invalid-assignment = "ignore"
+
+
 [tool.mypy]
 pretty = true
 ignore_missing_imports = true

tasks.py

@@ -173,6 +173,15 @@ def lint_mypy(context: Context) -> None:
         context.run(exec_cmd)
 
 
+@task
+def lint_ty(context: Context) -> None:
+    """Run ty type checker against all Python files."""
+    print(" - Check code with ty")
+    exec_cmd = "uv run ty check ."
+    with context.cd(MAIN_DIRECTORY_PATH):
+        context.run(exec_cmd)
+
+
 @task
 def lint_ruff(context: Context) -> None:
     """Run Linter to check all Python files."""
@@ -220,6 +229,7 @@ def lint_all(context: Context) -> None:
     """Run all linters."""
     lint_yaml(context)
     lint_ruff(context)
+    lint_ty(context)
     lint_mypy(context)
     lint_docs(context)