few manual fixes

Author: mark-torres10

.github/workflows/ci.yml

@@ -157,7 +157,7 @@ jobs:
       - name: Check Alembic metadata does not drift (db/schema.py)
         run: uv run python scripts/check_db_schema_drift.py
 
-  import_lint:
+  python_lint_import_dependency_boundaries:
     runs-on: ubuntu-latest
     timeout-minutes: 15
     steps:
@@ -175,9 +175,9 @@ jobs:
           path: |
             ~/.cache/uv
             uv.lock
-          key: ${{ runner.os }}-uv-import-lint-3.12-${{ hashFiles('pyproject.toml', 'uv.lock') }}
+          key: ${{ runner.os }}-uv-python-lint-import-dependency-boundaries-3.12-${{ hashFiles('pyproject.toml', 'uv.lock') }}
           restore-keys: |
-            ${{ runner.os }}-uv-import-lint-3.12-
+            ${{ runner.os }}-uv-python-lint-import-dependency-boundaries-3.12-
 
       - name: Install uv
         uses: astral-sh/setup-uv@v4
@@ -193,7 +193,7 @@ jobs:
       - name: Add project directory to PYTHONPATH
         run: echo "PYTHONPATH=${{ github.workspace }}" >> $GITHUB_ENV
 
-      - name: Run import-linter (PY-1)
+      - name: python_lint_import_dependency_boundaries
         run: uv run lint-imports --config pyproject.toml
 
   test:

.pre-commit-config.yaml

@@ -40,8 +40,8 @@ repos:
         entry: bash -lc "root=$(git rev-parse --show-toplevel) && cd \"$root\" && PYTHONPATH=\"$root\" uv run --extra test pyright ."
         language: system
         pass_filenames: false
-      - id: import-linter
-        name: import-linter (PY-1)
+      - id: python_lint_import_dependency_boundaries
+        name: python_lint_import_dependency_boundaries
         entry: bash -lc "root=$(git rev-parse --show-toplevel) && cd \"$root\" && PYTHONPATH=\"$root\" uv run --extra test lint-imports --config pyproject.toml"
         language: system
         pass_filenames: false

pyproject.toml

@@ -110,12 +110,12 @@ typeCheckingMode = "basic"
 reportMissingTypeStubs = false
 
 [tool.importlinter]
-root_packages = ["simulation", "db", "lib", "feeds", "ai", "jobs", "ml_tooling", "ai_tools"]
+root_packages = ["simulation", "db", "lib", "feeds", "ai", "jobs", "ml_tooling"]
 include_external_packages = false
 exclude_type_checking_imports = true
 
 [[tool.importlinter.contracts]]
-name = "PY-1 domain purity: simulation.core.models"
+name = "Domain purity: simulation.core.models"
 type = "forbidden"
 source_modules = ["simulation.core.models"]
 forbidden_modules = [
@@ -141,19 +141,19 @@ forbidden_modules = [
 ]
 
 [[tool.importlinter.contracts]]
-name = "PY-1 lib is leaf"
+name = "lib is leaf"
 type = "forbidden"
 source_modules = ["lib"]
 forbidden_modules = ["simulation", "db", "feeds", "ai", "jobs", "ml_tooling"]
 
 [[tool.importlinter.contracts]]
-name = "PY-1 ml_tooling constraints"
+name = "ml_tooling constraints"
 type = "forbidden"
 source_modules = ["ml_tooling"]
 forbidden_modules = ["simulation", "db", "feeds", "ai", "jobs"]
 
 [[tool.importlinter.contracts]]
-name = "PY-1 API routes boundary: simulation.api.routes"
+name = "API routes boundary: simulation.api.routes"
 type = "forbidden"
 source_modules = ["simulation.api.routes"]
 forbidden_modules = ["simulation.core", "db", "feeds", "ai", "jobs", "ml_tooling"]

simulation/core/agent_actions.py

@@ -1,8 +1,4 @@
-"""Action generation helpers for simulation agents.
-
-This module intentionally lives outside `simulation.core.models` to keep domain models
-pure per PY-1 import contracts.
-"""
+"""Action generation helpers for simulation agents."""
 
 from __future__ import annotations
 

simulation/core/models/runs.py

@@ -25,7 +25,7 @@ class RunConfig(BaseModel):
     def validate_metric_keys_config(cls, v: list[str] | None) -> list[str] | None:
         """Validate that metric_keys, when provided, is non-empty and contains non-empty strings.
 
-        Note: registry-level validation belongs outside the domain models layer (PY-1).
+        Note: registry-level validation belongs outside the domain models layer.
         """
         if v is None:
             return None