mirror of
https://github.com/kevinveenbirkenbach/computer-playbook.git
synced 2026-11-06 10:59:59 +00:00
kevinveenbirkenbachandClaude Opus 4.7
9a578952c3
Replace 8 bash env modules with a single Python generator
(cli.meta.env) that materialises env/static.env + runtime context
into a gitignored .env. Each variable lives in its own handler
under utils/env/handlers/; defaults carry inline one-line `# ...`
comments.
Rename to the INFINITO_* schema:
- workflow inputs: APPS, SERVICES_DISABLED, CMD, DEBUG, WHITELIST,
BUNDLES, FULL_CYCLE, VARIANT, RUN_FLAGS, LIMIT_HOST, PARALLEL
- inventory: INVENTORY_DIR / INVENTORY_FILE / HOST_VARS_FILE,
PW_FILE -> INVENTORY_PASSWORD_FILE
- venv: VENV_BASE / VENV_FALLBACK / VENV_NAME
- new SPOT defaults: COMPILE_SILENCE, CONTAINER_NO_CA,
WAIT_HEALTH_TIMEOUT_S, PACKAGE_CACHE_FRONTEND_INIT_IMAGE,
promoted IMAGE_TAG from GHA-only to general passthrough
compose.yml hardening:
- inline \${VAR:-default} removed; strict \${VAR:?...} everywhere
- new integration lint blocks regressions
Tooling:
- scripts/install/all.sh owns the stamp-file install logic
(Makefile target delegates)
- scripts/meta/env/load.sh is the single shell entry point;
8 bash modules deleted
- new shell lint: no INFINITO_* defaults outside env/static.env
Tests:
- 93 new unit tests for utils/env/{builder,parser,runtime,writer}
- restructure tests/lint into repository/ (file-content rules) vs
filesystem/{shell,python,env}/ (filesystem-shape rules)
- drift + smoke integration tests against the generator
Docs:
- variables.md is now a pointer; per-variable docs live next to
the key in env/static.env / env/ci.env
- compose.yml.md drops the Default column
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
42 lines
1.2 KiB
Python
Executable File
42 lines
1.2 KiB
Python
Executable File
#!/usr/bin/env python3
|
|
|
|
from __future__ import annotations
|
|
|
|
import stat
|
|
import unittest
|
|
|
|
from . import PROJECT_ROOT
|
|
|
|
|
|
class TestShellScriptsExecutable(unittest.TestCase):
|
|
@classmethod
|
|
def setUpClass(cls):
|
|
cls.repo_root = PROJECT_ROOT
|
|
cls.scripts_root = cls.repo_root / "scripts"
|
|
|
|
def test_all_shell_scripts_under_scripts_are_executable(self):
|
|
missing_executable_bit: list[str] = []
|
|
|
|
for path in sorted(self.scripts_root.rglob("*.sh")): # nocheck: project-walk
|
|
if not path.is_file():
|
|
continue
|
|
|
|
mode = path.stat().st_mode
|
|
has_any_execute_bit = bool(
|
|
mode & (stat.S_IXUSR | stat.S_IXGRP | stat.S_IXOTH)
|
|
)
|
|
if not has_any_execute_bit:
|
|
rel = path.relative_to(self.repo_root).as_posix()
|
|
missing_executable_bit.append(rel)
|
|
|
|
if missing_executable_bit:
|
|
self.fail(
|
|
"The following shell scripts are missing the executable bit:\n- "
|
|
+ "\n- ".join(missing_executable_bit)
|
|
+ "\n\nSet it with: chmod +x <file>"
|
|
)
|
|
|
|
|
|
if __name__ == "__main__":
|
|
unittest.main()
|