unity-devkit 0.1.2__tar.gz → 0.1.4__tar.gz

unity_devkit-0.1.4/.gitattributes

@@ -0,0 +1 @@
+ruff.base.toml -text

{unity_devkit-0.1.2 → unity_devkit-0.1.4}/.github/workflows/ci.yml

@@ -28,3 +28,6 @@ jobs:
 
       - name: Type check
         run: uv run basedpyright
+
+      - name: Ruff config drift
+        run: uvx --from python-devkit==0.1.2 sync-ruff --check

{unity_devkit-0.1.2 → unity_devkit-0.1.4}/.github/workflows/release.yml

@@ -22,4 +22,4 @@ jobs:
       - uses: astral-sh/setup-uv@v7
 
       - name: Publish
-        run: uvx --from release-devkit==0.1.0 publish-packages --config publish-config.json
+        run: uvx --from release-devkit==0.1.1 publish-packages --config publish-config.json

{unity_devkit-0.1.2 → unity_devkit-0.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: unity-devkit
-Version: 0.1.2
+Version: 0.1.4
 Summary: Unity project discovery, build, license, and CI tooling
 License-File: LICENSE
 License-File: NOTICE

{unity_devkit-0.1.2 → unity_devkit-0.1.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "unity-devkit"
-version = "0.1.2"
+version = "0.1.4"
 description = "Unity project discovery, build, license, and CI tooling"
 requires-python = ">=3.13"
 dependencies = [

unity_devkit-0.1.4/ruff.base.toml

@@ -0,0 +1,198 @@
+# Org-canonical ruff configuration, hosted in python-devkit.
+# Consuming repos carry this file verbatim as ruff.base.toml, written by the
+# sync-ruff verb (uvx --from python-devkit sync-ruff) and drift-gated in CI.
+# Never edit a synced copy: change this canonical, release python-devkit, and
+# re-run sync-ruff in each consuming repo.
+target-version = "py313"
+line-length = 120
+preview = true
+required-version = ">=0.14.11"
+
+[lint]
+select = ["ALL"]
+ignore = [
+    # ── Temporarily ignored (inline noqas exist for these rules but are redundant
+    # while the rules are globally ignored — RUF100 suppresses the "unused noqa" warning
+    # until we un-ignore each ticket's rules)
+    "RUF100",
+
+    # ── Permanently ignored ──────────────────────────────────────────────
+    # Type annotations — basedpyright strict mode handles this
+    "ANN",
+    # EM101 (string literal in exception) — the msg = ... indirection adds verbosity without benefit
+    "EM101",
+    # TRY003 (raise vanilla args) — requires custom exception class for every raise site;
+    # the real problem (interpolation) is caught by EM102/EM103
+    "TRY003",
+    # A003 (class attribute shadowing) — class attributes via self.x never conflict with builtins
+    "A003",
+    # Function complexity counting — C901 already covers structural complexity;
+    # these counting rules are largely redundant
+    "PLR0911",
+    "PLR0912",
+    "PLR0913",
+    "PLR0914",
+    "PLR0915",
+    "PLR0917",
+
+    # ── Temporarily ignored (will enable when conventions are established) ─
+    # Docstrings — project policy is "no docstrings" for now
+    "D",
+    # Copyright notices — not yet established
+    "CPY",
+    # Docstring formatting — blocked on D
+    "DOC",
+    # TODO/FIXME comment formatting
+    "TD",
+    "FIX",
+
+    # The "Ticket N" blocks below are the transitional cleanup slices tracked
+    # by PLE-248 (Enforce the deferred ruff rules):
+    #   https://linear.app/plerionplatforms/issue/PLE-248
+    # Ticket 3 (T201, print → logging) is tracked by PLE-336:
+    #   https://linear.app/plerionplatforms/issue/PLE-336
+    # Lifting a rule = remove it here in python-devkit's canonical config,
+    # release, and run sync-ruff in every consuming repo.
+
+    # ── Ticket 1: Auto-fix formatting & type modernization ───────────────
+    # Zero risk. Purely mechanical, all auto-fixable via ruff --fix.
+    "COM812",   # missing trailing commas
+    "I001",     # import sorting
+    "ISC003",   # explicit string concatenation
+    "RUF031",   # parenthesized tuple in subscript
+    "TC001",    # move import into TYPE_CHECKING block
+    "TC002",    # move third-party import into TYPE_CHECKING block
+    "TC003",    # move stdlib import into TYPE_CHECKING block
+    "TC006",    # move typing import into TYPE_CHECKING block (PEP 695)
+    "TC008",    # move import into TYPE_CHECKING block (quoted annotation)
+    "UP006",    # use X instead of typing.X
+    "UP007",    # use X | Y instead of Union
+    "UP011",    # unnecessary parentheses to functools.lru_cache
+    "UP015",    # unnecessary open mode parameters
+    "UP017",    # use datetime.UTC alias
+    "UP034",    # extraneous parentheses
+    "UP035",    # deprecated import (use collections.abc, etc.)
+    "UP040",    # use PEP 695 type alias
+    "UP042",    # use PEP 695 type parameters
+    "UP045",    # use X | None instead of Optional
+
+    # ── Ticket 2: Auto-fix code simplifications ──────────────────────────
+    # Very low risk. Auto-fixable with trivial semantic changes.
+    "C401",     # unnecessary generator — use set/dict comprehension
+    "C416",     # unnecessary comprehension — use list/dict
+    "ERA001",   # commented-out code
+    "FURB101",  # read-text can replace open+read
+    "FURB103",  # write-text can replace open+write
+    "FURB110",  # use ternary instead of try/except for default
+    "FURB113",  # use list.extend instead of repeated append
+    "FURB118",  # use operator module instead of lambda
+    "LOG004",   # use __name__ as logger name
+    "PERF401",  # use list comprehension instead of manual append
+    "PIE810",   # unnecessary spread of single-element starred
+    "PLC0207",  # use a single __all__ definition
+    "PLC1901",  # compare to empty string using falsy check
+    "PLR1711",  # useless return at end of function
+    "PLR6104",  # use augmented assignment
+    "PLR6201",  # use a set literal for membership test
+    "PLW1510",  # subprocess.run without explicit check= argument
+    "PLW1514",  # open without explicit encoding
+    "PT001",    # use @pytest.fixture over @pytest.fixture()
+    "PT018",    # composite assertion — split into multiple asserts
+    "PTH101",   # os.chmod → Path.chmod
+    "PTH111",   # os.path.expanduser → Path.expanduser
+    "PTH112",   # os.path.isdir → Path.is_dir
+    "PTH118",   # os.path.join → Path /
+    "PTH123",   # open() → Path.open()
+    "PTH201",   # pathlib.Path(".")
+    "RET501",   # unnecessary explicit return None
+    "RET504",   # unnecessary assignment before return
+    "RET505",   # unnecessary else after return
+    "RET506",   # unnecessary else after raise
+    "RET507",   # unnecessary else after continue
+    "RSE102",   # unnecessary parentheses on raised exception
+    "RUF005",   # consider spread operator instead of concatenation
+    "RUF010",   # use explicit conversion flag in f-string
+    "RUF015",   # prefer next() over single-element slice
+    "RUF029",   # function is declared async but has no await
+    "RUF052",   # local variable is assigned but never used (underscore prefix)
+    "RUF056",   # unnecessary default value for keyword argument
+    "SIM102",   # collapsible if statements
+    "SIM105",   # use contextlib.suppress instead of try/except/pass
+    "SIM115",   # use context manager for opening files
+    "SIM117",   # use single with statement for multiple context managers
+    "SIM118",   # use key in dict instead of key in dict.keys()
+    "TID252",   # prefer absolute imports over relative
+
+    # ── Ticket 3: print → logging ────────────────────────────────────────
+    # All code should use logging, including CLI scripts.
+    "T201",     # print found
+
+    # ── Ticket 4: Exception message interpolation ────────────────────────
+    # Remove f-strings from exception messages — use string literals.
+    "EM102",    # f-string in exception message
+
+    # ── Ticket 5: Missing __init__.py + shebangs ─────────────────────────
+    # Add __init__.py to installable packages; remove vestigial shebangs.
+    "INP001",   # implicit namespace package — add __init__.py
+    "EXE001",   # shebang present but file not executable
+
+    # ── Ticket 6: Type ignore specificity ────────────────────────────────
+    # Replace blanket type: ignore with specific error codes.
+    "PGH003",   # blanket type: ignore
+
+    # ── Ticket 7: Boolean trap refactoring ───────────────────────────────
+    # Change function signatures — callers must update too.
+    "FBT001",   # boolean positional argument
+    "FBT002",   # boolean default value in function definition
+    "FBT003",   # boolean positional value in call
+
+    # ── Ticket 8: Exception handling patterns ────────────────────────────
+    # Behavioral changes to error handling.
+    "B020",     # loop variable overwritten by except
+    "B903",     # generator should use yield from
+    "B904",     # raise without from inside except
+    "B905",     # zip() without strict= parameter
+    "BLE001",   # blind except Exception
+    "S101",     # assert outside tests
+    "TRY002",   # create custom exception class
+    "TRY300",   # try/except/return — move return into else
+
+    # ── Ticket 9: Complexity, naming & refactoring ───────────────────────
+    # Structural changes requiring judgment.
+    "A001",     # builtin shadowing (dir, bytes)
+    "ARG001",   # unused function argument
+    "ARG002",   # unused method argument
+    "C901",     # function too complex
+    "E501",     # line too long
+    "N802",     # function name should be lowercase
+    "N806",     # variable in function should be lowercase
+    "N815",     # mixed-case variable in class scope
+    "N818",     # exception name should end in Error
+    "PLC0415",  # import not at top level
+    "PLE0704",  # bare raise not inside except handler
+    "PLR1702",  # too many nested blocks
+    "PLR2004",  # magic value used in comparison
+    "PLR6301",  # method could be function or static method
+    "PLW0603",  # global statement
+    "PLW2901",  # loop variable overwritten
+    "RUF001",   # ambiguous unicode — replace with ASCII
+    "RUF003",   # ambiguous unicode in comment
+    "S106",     # possible hardcoded password
+    "S108",     # probable insecure temp file/dir usage
+    "S110",     # try/except/pass on broad exception
+    "S202",     # use of tarfile.extractall
+    # S404/S603: prerequisite — bash.py needs env, stderr=STDOUT merge, and no-raise-with-capture
+    # before context_sha.py, deptry_check.py, test_context_sha.py, list_debug_targets.py can migrate
+    "S404",     # suspicious subprocess import
+    "S603",     # subprocess call with non-literal
+    "S606",     # starting process without shell
+    "S607",     # starting process with partial path
+]
+
+[lint.per-file-ignores]
+# Tests use assert as the standard pytest mechanism; fixtures take arguments
+# that pytest consumes implicitly; pytest-style test classes need no self use
+"**/tests/**" = ["S101", "ARG001", "ARG002", "PLR6301"]
+
+[lint.flake8-builtins]
+ignorelist = ["id", "map"]

unity_devkit-0.1.4/ruff.toml

@@ -0,0 +1,5 @@
+extend = "ruff.base.toml"
+
+[lint.extend-per-file-ignores]
+# chmod'ing the installed Linux executable to 0o755 before launch is the intended permission
+"src/unity_devkit/install.py" = ["bad-file-permissions"]

{unity_devkit-0.1.2 → unity_devkit-0.1.4}/src/unity_devkit/build_unity.py

@@ -4,6 +4,7 @@ import json
 import os
 import shutil
 from pathlib import Path
+from typing import Annotated
 
 import typer
 from pydantic_settings import BaseSettings
@@ -28,16 +29,16 @@ app = typer.Typer(add_completion=False, pretty_exceptions_show_locals=False)
 
 @app.command()
 def main(
-    project: str = typer.Option(help="Project name"),
-    project_path: Path = typer.Option(help="Path to Unity project"),
-    platform: str = typer.Option(help="Target platform"),
-    cache_key: str = typer.Option(help="Cache key prefix"),
-    run_number: int = typer.Option(0, help="CI run number"),
-    branch: str = typer.Option("dev", help="Git branch name"),
-    registry: str = typer.Option(help="OCI registry path"),
-    build_env: str = typer.Option(
-        "", help="Newline-separated KEY=VALUE pairs injected into the Unity build process environment"
-    ),
+    project: Annotated[str, typer.Option(help="Project name")],
+    project_path: Annotated[Path, typer.Option(help="Path to Unity project")],
+    platform: Annotated[str, typer.Option(help="Target platform")],
+    cache_key: Annotated[str, typer.Option(help="Cache key prefix")],
+    registry: Annotated[str, typer.Option(help="OCI registry path")],
+    run_number: Annotated[int, typer.Option(help="CI run number")] = 0,
+    branch: Annotated[str, typer.Option(help="Git branch name")] = "dev",
+    build_env: Annotated[
+        str, typer.Option(help="Newline-separated KEY=VALUE pairs injected into the Unity build process environment")
+    ] = "",
 ) -> None:
     for line in build_env.splitlines():
         entry = line.strip()

unity_devkit-0.1.2/ruff.toml

@@ -1,95 +0,0 @@
-target-version = "py313"
-line-length = 120
-preview = true
-
-[lint]
-select = ["ALL"]
-ignore = [
-    # ── Permanent ─────────────────────────────────────────────────────────
-    # Type annotations — basedpyright strict mode handles this
-    "ANN",
-    # No docstrings, no copyright headers
-    "D",
-    "DOC",
-    "CPY",
-    # TODO/FIXME comment formatting
-    "TD",
-    "FIX",
-    # Exception message ergonomics — literal-indirection and custom-class churn without benefit
-    "EM101",
-    "EM102",
-    "TRY003",
-    # Class attribute names never conflict with builtins via self.x
-    "A003",
-    # Function-size counters — C901 already covers structural complexity
-    "PLR0911",
-    "PLR0912",
-    "PLR0913",
-    "PLR0914",
-    "PLR0915",
-    "PLR0917",
-    # Typer's Option/Argument in argument defaults is typer's standard API pattern
-    "B008",
-    # Boolean parameters read fine as keyword-only flags here
-    "FBT001",
-    "FBT002",
-    "FBT003",
-    # Relative imports are the intra-package convention
-    "TID252",
-    # Formatter-owned; enforced by `ruff format`
-    "COM812",
-    "I001",
-    # Magic values in comparisons are readable in this CLI
-    "PLR2004",
-    # The formatter owns wrapping; long string literals and URLs stay on one line
-    "E501",
-    # src-layout tests need no __init__.py
-    "INP001",
-    # Runtime-used typing imports stay at module level
-    "TC001",
-    "TC002",
-    "TC003",
-    # Small literal tuples read fine in membership tests
-    "PLR6201",
-
-    # ── Deferred to PLE-248 — Enforce the deferred ruff rules
-    # https://linear.app/plerionplatforms/issue/PLE-248
-    # Mirrors placeframe's root ruff.toml transitional Ticket-N blocks; lifting
-    # a rule here should happen alongside lifting it from placeframe's ruff.toml.
-    "TC006",    # move import into TYPE_CHECKING block (quoted annotation) — Ticket 1
-    "UP015",    # unnecessary open mode parameters — Ticket 1
-    "FURB101",  # read-text can replace open+read — Ticket 2
-    "FURB103",  # write-text can replace open+write — Ticket 2
-    "FURB113",  # use list.extend instead of repeated append — Ticket 2
-    "PLW1514",  # open/read_text without explicit encoding — Ticket 2
-    "PTH101",   # os.chmod → Path.chmod — Ticket 2
-    "PTH112",   # os.path.isdir → Path.is_dir — Ticket 2
-    "PTH118",   # os.path.join → Path / — Ticket 2
-    "PTH123",   # open() → Path.open() — Ticket 2
-    "RET504",   # unnecessary assignment before return — Ticket 2
-    "RET505",   # unnecessary else after return — Ticket 2
-    "RUF005",   # consider spread operator instead of concatenation — Ticket 2
-    "SIM102",   # collapsible if statements — Ticket 2
-    "SIM115",   # use context manager for opening files — Ticket 2
-    "SIM117",   # use single with statement for multiple context managers — Ticket 2
-    "B904",     # raise ... from inside except — Ticket 8
-    "BLE001",   # blind except Exception — Ticket 8
-    "TRY300",   # try/except/return — move return into else — Ticket 8
-    "C901",     # function too complex — Ticket 9
-    "PLC0415",  # import not at top level — Ticket 9
-    "PLW0603",  # global statement — Ticket 9
-    "PLW2901",  # loop variable overwritten — Ticket 9
-    "PTH201",   # do not pass "." explicitly to Path — Ticket 2
-    "S103",     # bad-file-permissions (0o755 on executable) — Ticket 9
-    "S108",     # hardcoded temp file / directory — Ticket 9
-    "S404",     # suspicious subprocess import — Ticket 9
-    "S603",     # subprocess call with non-literal — Ticket 9
-    "S607",     # start-process-with-partial-path — Ticket 9
-
-    # ── Deferred to PLE-336 — Replace print() with structured logging
-    # https://linear.app/plerionplatforms/issue/PLE-336
-    "T201",     # print found
-]
-
-[lint.flake8-builtins]
-ignorelist = ["id", "map"]