2 Commits

Include renames

Author	SHA1	Message	Date
kevinveenbirkenbachandClaude Opus 4.7	7dc6d6e491	chore(ruff): enable TC + N rules ... TC (flake8-type-checking): typing-only imports moved into ``if TYPE_CHECKING:`` blocks across 8 source files via auto-fix (117 hits). Reduces runtime import surface and rules out circular imports for type-only deps. Project-wide ``from __future__ import annotations`` (FA, already enabled) keeps annotations as strings, so the moved imports never need to load at runtime. N (pep8-naming): 32 hits handled. Renames (semantic + lint): - ``isSmallerThenMaximumBackupSize`` -> ``is_larger_than_maximum_backup_size`` (snake_case + fixes both the typo "Then -> Than" and the semantic mismatch: the function returns ``current > maximum``) - ``isDirectoryDeletable`` / ``deleteVersion`` / ``getAmountOfIteration`` / ``deleteIteration`` -> snake_case - 8 test functions with ALL-CAPS env-var fragments (``test_..._DOMAIN_PRIMARY_...`` etc.) -> snake_case - Mock alias ``VH`` -> ``vh`` in test_host_vars.py Exception class renames (``N818`` Error suffix): - ``TimeLimitExceededException`` -> ``TimeLimitExceededError`` - ``AttemptException`` -> ``AttemptError`` - ``PathNotFound`` -> ``PathNotFoundError`` (propagated across 4 callers under tests/integration/lookups/config/) - ``_ExitJson`` / ``_FailJson`` -> ``*_Error`` Refactor: - ``METADATA = {...}`` local constant in two functions of ``plugins/filter/invokable_paths.py`` lifted to module-level ``_METADATA_KEYS``. - ``from typing import TYPE_CHECKING`` in ``tasks/utils/migrate_meta_layout/__main__.py`` moved above the ``sys.path.insert`` hack to satisfy E402 (collateral from the TC auto-fix that landed it after the path mutation). Per-file ignores in pyproject.toml for the legitimate cases the rule cannot accept: - ``pull-specific-host.py`` -> ``N999`` (Ansible-invoked role-level script, hyphenated filename pinned by the surrounding stack). - 4 matrix-deploy files -> ``N806`` for the ``include_R`` / ``raw_include_R`` round-index notation; the uppercase ``R`` here mirrors the formula in the docstring rather than naming a constant. - ``plugins/filter/value_generator.py`` -> ``N806`` for the lazy-loaded class binding ``ValueGenerator = _value_generator_cls()``, which retains the imported class identifier per PEP 8. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-08 18:57:07 +02:00
kevinveenbirkenbachandClaude Opus 4.7	a080dc4f98	refactor(cli): inventory tools under administration/, drop role scaffolder + dead SSO indirection ... Move: - cli/build/inventory/full -> cli/administration/inventory/devices - cli/create/inventory -> cli/administration/inventory/provision - cli/create/credentials -> cli/administration/inventory/credentials - cli/encrypt/inventory -> cli/administration/inventory/encrypt Delete: - cli/create/role + templates/roles/ (stale scaffolder) - roles/web-app-baserow/templates/javascript.js.j2 + dead sso_warning snippet - js_application_name across akaunting/baserow/opencloud/opentalk/taiga, javascript service disabled for baserow + taiga Wire-through: - argv lists in cli/deploy/{container,development}/* + 2 shell scripts now invoke 'infinito administration inventory provision' - subprocess module names + docs (contributing/* + administration/deploy.md) - discovery resolver test + folder-item-limit whitelist updated - ruff fixes in devices/command.py (RET504 inline + Path.open over open) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-08 18:41:04 +02:00