Update Ruff and Unit Tests

Author: Kaweees

.envrc

@@ -0,0 +1 @@
+use nix

.gitignore

@@ -1,10 +1,11 @@
 *
 
 !.github/
-!.github/**/
+!.github/**
 !assets/
-!assets/**/
+!assets/**
 !.editorconfig
+!.envrc
 !.gitattributes
 !.gitignore
 !.gitkeep
@@ -16,13 +17,13 @@
 !README.md
 
 !scripts/
-!scripts/**/
+!scripts/**
 !shared/
-!shared/**/
+!shared/**
 !src/
-!src/**/
+!src/**
 !tests/
-!tests/**/
+!tests/**
 
 !*.md
 !*.mdc

.pre-commit-config.yaml

@@ -4,24 +4,39 @@ repos:
     hooks:
       # Basic safety checks
       - id: check-added-large-files
-      - id: check-merge-conflict
+      - id: check-ast
+      - id: check-builtin-literals
       - id: check-case-conflict
-      - id: check-symlinks
-      - id: destroyed-symlinks
+      - id: check-docstring-first
       - id: check-executables-have-shebangs
-      - id: check-shebang-scripts-are-executable
+      - id: check-json
+      - id: check-merge-conflict
+      - id: check-symlinks
       - id: check-toml
+      - id: check-vcs-permalinks
+      - id: check-xml
       - id: check-yaml
+        args: ["--allow-multiple-documents"]
+      - id: destroyed-symlinks
+      - id: check-shebang-scripts-are-executable
 
       # Simple auto-fixers
       - id: end-of-file-fixer
+      - id: fix-byte-order-marker
+      - id: mixed-line-ending
+      - id: pretty-format-json
+        args: ["--autofix"]
+      - id: requirements-txt-fixer
+      - id: sort-simple-yaml
       - id: trailing-whitespace
 
       # Quick correctness checks
       - id: debug-statements
+      - id: name-tests-test
 
       # Security checks (can also be placed last)
       - id: detect-private-key
+      - id: forbid-new-submodules
 
   - repo: https://github.com/PyCQA/autoflake
     rev: "v2.3.1"
@@ -48,4 +63,4 @@ repos:
       - id: ruff-format
       # Then linting + autofixes
       - id: ruff
-        args: [--fix]
+        args: ["--output-format=full", "--fix", "--config", "ruff.toml"]

pyproject.toml

@@ -27,8 +27,6 @@ dev = [
   "pre-commit>=4.1.0",
   "ipykernel>=6.29.5",
   "pytest>=8.3.2",
-  "deptry>=0.23.0",
-  "mypy>=1.15.0",
   "pytest-cov>=6.0.0",
 ]
 
@@ -38,55 +36,6 @@ allow-direct-references = true
 [tool.hatch.build.targets.wheel]
 packages = ["shared", "src"]
 
-[tool.mypy]
-exclude = "^.tmp/"
-ignore_missing_imports = true
-
-[tool.bandit]
-skips = ['B101']
-
-[tool.ruff]
-
-extend-exclude = [
-  "__pycache__",
-  "docs",
-  "site",
-  ".eggs",
-  ".git",
-  ".venv",
-  "build",
-  "dist",
-  "notebooks",
-  ".cache",
-]
-line-length = 120
-
-[tool.ruff.lint]
-ignore = [
-  "E402",   # Module level import not at top of file
-  "E501",   # Line too long
-  "E203",   # Whitespace before ':' -> conflicts with black
-  "D401",   # First line should be in imperative mood
-  "RET504", # Unnecessary variable assignment before return statement
-  "RET505", # Unnecessary elif after return statement
-  "SIM102", # Use a single if-statement instead of nested if-statements
-  "SIM117", # Merge with statements for context managers that have same scope
-  "SIM118", # Checks for key-existence checks against dict.keys() calls
-]
-select = ["E", "F", "I"]
-
-[tool.ruff.lint.per-file-ignores]
-"*/__init__.py" = ["F401"] # Unused import
-
-[tool.ruff.lint.isort]
-known-first-party = ["modern_python", "core"]
-
-[tool.ruff.lint.pydocstyle]
-convention = "google"
-
-[tool.ruff.lint.mccabe]
-max-complexity = 30
-
 [tool.pytest.ini_options]
 cache_dir = ".pytest_cache"
 pythonpath = [".", "scripts", "src"]

ruff.toml

@@ -0,0 +1,44 @@
+extend-exclude = [
+  "__pycache__",
+  "docs",
+  "site",
+  ".eggs",
+  ".git",
+  ".venv",
+  "build",
+  "dist",
+  "notebooks",
+  ".cache",
+]
+line-length = 120
+
+[lint]
+extend-select = [
+  # Ruff-specific rules
+  "RUF",
+]
+ignore = [
+  "E402",   # Module level import not at top of file
+  "E501",   # Line too long
+  "E203",   # Whitespace before ':' -> conflicts with black
+  "D401",   # First line should be in imperative mood
+  "RET504", # Unnecessary variable assignment before return statement
+  "RET505", # Unnecessary elif after return statement
+  "SIM102", # Use a single if-statement instead of nested if-statements
+  "SIM117", # Merge with statements for context managers that have same scope
+  "SIM118", # Checks for key-existence checks against dict.keys() calls
+]
+select = ["E", "F", "I"]
+
+[lint.per-file-ignores]
+"*/__init__.py" = ["F401"] # Unused import
+
+[lint.isort]
+known-first-party = ["modern_python", "core"]
+
+
+[lint.pydocstyle]
+convention = "google"
+
+[lint.mccabe]
+max-complexity = 30

tests/conftest.py

@@ -1,5 +1,5 @@
 from this import return_one  # type: ignore
 
 
-def test_this():
+def return_one_test():
     assert return_one() == 1

tests/scripts/test_this.py tests/scripts/this_test.pytests/scripts/test_this.py renamed to tests/scripts/this_test.py

@@ -1,7 +1,7 @@
 from utils import get_token
 
 
-def test_get_token(monkeypatch):
+def get_token_test(monkeypatch):
     # Set up mock environment variables for the test
     monkeypatch.setenv("TOKEN", "test_token")