feat/atlas-scan

dist_enumerate.main() calls sys.exit() — silently aborts the full pipeline

scripts/atlas/distribucion/pilot_run.py:49

If no substations are found for the division, dist_enumerate.main() calls sys.exit() and the process dies. Stages 2/3 and 3/3 never run and 'PILOTO COMPLETO' is never printed. Consider wrapping in try/except SystemExit to emit a clear diagnostic and exit cleanly.

Proxy presence skips WFS availability probe unconditionally

scripts/atlas/distribucion/pilot_run.py:41

When C._PROXIES is non-empty the poll loop is skipped entirely and the pipeline runs immediately — even if the WFS is still returning the shell HTML. dist_enumerate.main() would then operate on empty/malformed data. Consider running at least one availability probe even with proxies configured, or document the assumption explicitly.

No per-stage error handling — a failure in stage N silently skips remaining stages

scripts/atlas/distribucion/pilot_run.py:49

dist_enumerate.main(), dist_capacity.main(), and dist_compare.main() are called sequentially with no try/except. If any stage raises, subsequent stages are skipped with no message. Wrap each call to at minimum print the stage name before re-raising.

Actual poll interval is longer than POLL_MIN due to probe execution time

scripts/atlas/distribucion/pilot_run.py:45

time.sleep(POLL_MIN * 60) is called after the probe completes. On slow connections the probe itself adds seconds. Negligible at 15-minute intervals, but the interval is drift-by-design rather than fixed.

C._PROXIES accesses a private/internal attribute

scripts/atlas/distribucion/pilot_run.py:41

If dist_common renames or restructures _PROXIES, the check silently evaluates to falsy and the poll loop always runs — even with proxies configured. Prefer a documented public constant or predicate in dist_common.

Duplicate 'import dist_common as C' statement

scripts/atlas/distribucion/pilot_run.py:14

dist_common is imported twice (lines 13-14). Harmless in Python but is a copy-paste artifact. Remove the redundant import.

Broad except swallows unexpected errors in disponible()

scripts/atlas/distribucion/pilot_run.py:35

except Exception as e catches all non-ShellBlocked errors (including bugs like AttributeError) and treats them as 'blocked'. Intentional per the comment, but could hide bugs during development. Acceptable for a research script.

No tests — consistent with directory-wide pattern

scripts/atlas/distribucion/pilot_run.py

No tests exist for any sibling scripts in this directory. This is a research/operational script; the pattern is consistent. No action required.

15-minute sleep is not interruptible — Ctrl-C unresponsive mid-wait

scripts/atlas/distribucion/pilot_run.py:41

time.sleep(POLL_MIN * 60) holds for up to 15 minutes. Splitting into 60 × 1-second sleeps or using threading.Event.wait() would let KeyboardInterrupt respond immediately without changing the effective poll frequency.

POLL_MIN unit conversion repeated in log message and sleep call

scripts/atlas/distribucion/pilot_run.py:45

POLL_MIN * 60 appears implicitly in both the sleep and the printed message. Naming poll_seconds = POLL_MIN * 60 at the top and using it in both places eliminates the silent unit conversion.

Pipeline stages called with positional args — fragile if signatures change

scripts/atlas/distribucion/pilot_run.py:49

dist_enumerate.main(DIV, STEP, BBOX) relies on positional order where the third param is bbox_override. Keyword args (bbox_override=BBOX) would survive reordering and clarify intent.