Skip to content

fix: api/cli/registry bridges + high-value codebase-audit fixes - #268

Merged
Xiddoc merged 1 commit into
masterfrom
claude/beetroot-sweep-optimize-fnq7r0
Jul 1, 2026
Merged

fix: api/cli/registry bridges + high-value codebase-audit fixes#268
Xiddoc merged 1 commit into
masterfrom
claude/beetroot-sweep-optimize-fnq7r0

Conversation

@Xiddoc

@Xiddoc Xiddoc commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Summary

Fourth sweep batch — 18 fixes: the api/cli/registry "bridge" issues that span multiple hub files (so they couldn't be parallelized in earlier batches), plus the highest-value findings from the 18-agent codebase audit (filed as #252#266). One bucket per file-disjoint area, each gate-verified, integrated cleanly.

Bridge issues (span api/cli/registry/settings/frida_download)

Codebase-audit fixes

Testing

Full local gate green: ruff, ruff format --check, mypy --strict (src+tests), pytest -p no:randomly at 100% line+branch coverage (1960 passed, 6 docker-daemon skips), shellcheck/shfmt, yamllint, actionlint, zizmor, codespell, deptry, uv lock --check, changelog lint.

🤖 Generated with Claude Code

https://claude.ai/code/session_01P1YsNMpGizhSDDPCBu5cut


Generated by Claude Code

Fourth sweep batch — the api/cli/registry "bridge" issues that span multiple
hub files, plus the highest-value findings from the parallel codebase audit.

bridges (span api/cli/registry/settings/frida_download):
- Frida-less ports config no longer KeyErrors ls/status/doctor/frida-addr (#158)
- ls/status reuse a single registry snapshot + memoized port index (#230)
- cross-instance port-collision check moved inside the registry lock (TOCTOU) (#183)
- apply() no longer clobbers a working frida-server before a re-fetch (#165)
- frida-server arch auto-detects from the host (aarch64 -> android-arm64) (#189)
- malformed BEETROOT_*_TIMEOUT maps to error/exit-1 instead of an import
  traceback that bricked even --help (#197)

codebase-audit fixes:
- install_frida detaches the daemon's stdio so it can't hang forever (#253)
- serial_is_available/_adb_unchecked run adb with a 5s timeout (#256)
- boot_cache memoizes the rootfs digest on (path,size,mtime) so warm up no
  longer re-hashes the multi-GB image every start (#254)
- rootfs bake pins --platform=linux/amd64 (#258); verify the Docker static
  bundle against a pinned sha256 before extract (#262, machinery; digest TBD)
- console.table markup-escapes cells/headers (#259)
- module downloads validate bytes vs Content-Length (#261)
- snapshot manifest written first + restore decompression-bomb cap (#265)
- e2e savevm cache stores the cache-key sidecar (#260); denylist trims
  whitespace (#263); docs main->master links (#266)

All covered by new tests; full gate green (ruff, mypy --strict src+tests,
pytest 100% line+branch (1960 passed), shellcheck/shfmt, yamllint, actionlint,
zizmor, codespell, deptry, uv lock, changelog lint).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01P1YsNMpGizhSDDPCBu5cut
@Xiddoc
Xiddoc merged commit b4eae90 into master Jul 1, 2026
17 checks passed
@Xiddoc
Xiddoc deleted the claude/beetroot-sweep-optimize-fnq7r0 branch July 1, 2026 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants