mirror of
https://github.com/kuhyx/testsAndMisc.git
synced 2026-11-10 21:07:03 +01:00
clean_audio.sh 419 -> 230 + 204
enforce_vbox_hosts.sh 443 -> 220 + 236
migrate_hosts_guard.sh 444 -> 167 + 162 + 141
migrate_hosts_guard needed two libraries: with all ten work functions moved
the single lib was 295 lines, so migrate and rollback were separated.
The extractor's brace matcher required index > start and so silently skipped
one-line `warn() { ...; }` definitions. Fixed, and regression-checked against
the committed fresh-install/main.sh split byte-for-byte.
The harness earned its keep on a fourth file: adding `export` to libre_translate
to satisfy SC2034 changed two function bodies, and verify_shell_split.sh caught
what lint had accepted. Those exports were reverted, and libre_translate.sh is
parked - what remains after its clean seams both assigns and reads the same
globals, so getting it under the cap needs parse_args restructured rather than
relocated, which is a refactor rather than a verbatim move.
That cost is now written down: a lib must contain the readers of every global
it assigns. The precedent at install_pacman_wrapper.sh:29 is about
definitions-only libs, not about assignments as such, and the pre-commit hook
runs shellcheck with no -x, so a source= directive will not make it follow the
link. This is the shell analogue of the monkeypatch-target trap.
Verified: all 8, 13 and 19 functions verbatim; all four libraries source with
no side effects; shellcheck passes standalone on every file with no
suppression; bash -n passes on all seven. None of the three was executed -
they drive ffmpeg over real media, mutate VirtualBox guest disks, and rewrite
systemd units.
Over-cap files 29 -> 26.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011w2Pj4Ek3ZNMQVYcFHgXDY
108 lines
4.0 KiB
Python
Executable File
108 lines
4.0 KiB
Python
Executable File
#!/usr/bin/env python3
|
|
"""Move a shell script's top-level functions into a sourced library, verbatim.
|
|
|
|
Line-range slicing is the obvious approach and it is wrong: these scripts
|
|
interleave top-level commands between their function definitions, so a range
|
|
slice sweeps those commands into the library, where they run at source time
|
|
and in the wrong order. This walks the file brace-by-brace instead, lifts only
|
|
the function blocks (with the comment directly above each), and leaves every
|
|
other line exactly where it was.
|
|
|
|
Verify the result with meta/scripts/verify_shell_split.sh.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import argparse
|
|
from pathlib import Path
|
|
import re
|
|
import sys
|
|
|
|
_FUNC_RE = re.compile(r"^[a-zA-Z_][a-zA-Z0-9_]*\(\)\s*\{")
|
|
|
|
|
|
def find_function_blocks(lines: list[str]) -> list[tuple[int, int, str]]:
|
|
"""Return (start, end, name) for each top-level function, brace-matched."""
|
|
blocks: list[tuple[int, int, str]] = []
|
|
index = 0
|
|
while index < len(lines):
|
|
if _FUNC_RE.match(lines[index]):
|
|
start, depth = index, 0
|
|
while index < len(lines):
|
|
depth += lines[index].count("{") - lines[index].count("}")
|
|
# A one-line `name() { ...; }` closes on its opening line, so
|
|
# the depth check must accept index == start.
|
|
if depth <= 0:
|
|
break
|
|
index += 1
|
|
blocks.append((start, index, lines[start].split("(")[0]))
|
|
index += 1
|
|
return blocks
|
|
|
|
|
|
def split_script(
|
|
path: Path, lib_path: Path, names: list[str] | None, header: str
|
|
) -> tuple[int, int]:
|
|
"""Move the named functions (or all of them) into *lib_path*."""
|
|
lines = path.read_text().split("\n")
|
|
blocks = find_function_blocks(lines)
|
|
if names:
|
|
blocks = [b for b in blocks if b[2] in names]
|
|
missing = set(names) - {b[2] for b in blocks}
|
|
if missing:
|
|
sys.exit(f"no such top-level function(s): {', '.join(sorted(missing))}")
|
|
|
|
parts: list[str] = []
|
|
drop: set[int] = set()
|
|
for start, end, _name in blocks:
|
|
head = start
|
|
while head - 1 >= 0 and lines[head - 1].lstrip().startswith("#"):
|
|
head -= 1
|
|
parts.append("\n".join(lines[head : end + 1]))
|
|
drop.update(range(head, end + 1))
|
|
|
|
lib_path.parent.mkdir(parents=True, exist_ok=True)
|
|
lib_path.write_text(header.rstrip() + "\n\n" + "\n\n".join(parts) + "\n")
|
|
|
|
kept = [ln for n, ln in enumerate(lines) if n not in drop]
|
|
rel = lib_path.relative_to(path.parent)
|
|
source_line = '. "$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)/' + str(rel) + '"'
|
|
out: list[str] = []
|
|
inserted = False
|
|
for line in kept:
|
|
out.append(line)
|
|
# `set -e # comment` is common here, so match the directive, not the line.
|
|
if not inserted and re.match(r"^set -[euo pipefail]+\b", line.strip()):
|
|
out += ["", f"# shellcheck source={rel}", source_line]
|
|
inserted = True
|
|
if not inserted:
|
|
sys.exit("no `set -e` line found to anchor the source directive")
|
|
path.write_text("\n".join(out))
|
|
return len(out), len(lib_path.read_text().split("\n"))
|
|
|
|
|
|
def main() -> None:
|
|
"""Extract functions from a script into a library beside it."""
|
|
parser = argparse.ArgumentParser(description=__doc__)
|
|
parser.add_argument("script", type=Path)
|
|
parser.add_argument("library", type=Path)
|
|
parser.add_argument(
|
|
"--functions",
|
|
help="comma-separated function names to move (default: all)",
|
|
)
|
|
parser.add_argument(
|
|
"--header",
|
|
default="#!/usr/bin/env bash\n# Helpers sourced by the entry script.",
|
|
help="header comment for the generated library",
|
|
)
|
|
args = parser.parse_args()
|
|
names = args.functions.split(",") if args.functions else None
|
|
entry_len, lib_len = split_script(args.script, args.library, names, args.header)
|
|
sys.stdout.write(
|
|
f"{args.script}: {entry_len} lines\n{args.library}: {lib_len} lines\n"
|
|
)
|
|
|
|
|
|
if __name__ == "__main__":
|
|
main()
|