Skip to content

refactor(network): finalize forwarding removal and compress script - #22

Merged
LucaLin233 merged 5 commits into
mainfrom
luno/network-optimize-finalize-compression
Aug 19, 2026
Merged

refactor(network): finalize forwarding removal and compress script#22
LucaLin233 merged 5 commits into
mainfrom
luno/network-optimize-finalize-compression

Conversation

@LucaLin233

Copy link
Copy Markdown
Owner

Summary

  • Remove the completed forwarding/RA migration warning, read-only status rows, compatibility runtime capture, documentation, and migration-only tests.
  • Merge consecutive local declarations without changing initialization order or typed-array declarations.
  • Table-drive managed-file backup metadata and restore scope resolution while preserving paths, formats, ordering, and rollback behavior.
  • Share PID tracking, dependency checks, RTT output, and health-delta calculations.

Size

modules/network-optimize.sh: 3522 -> 3252 lines (-270). This starts the mechanical compression phase while keeping the remaining 3000-3100 target for a later isolated pass.

Validation

  • bash -n modules/network-optimize.sh
  • shellcheck --severity=warning modules/network-optimize.sh
  • shellcheck -x --severity=error linux_setup.sh modules/*.sh tools/*.sh tests/*.sh
  • bash tests/test-network-optimize.sh
  • bash tests/test-network-optimize-entry.sh
  • bash tests/test-network-optimize-verify.sh
  • bash tests/run.sh
  • no forwarding, RA, ip_forward, or accept_ra references remain in the network module, its tests, or README

Compatibility

CLI behavior, state paths and formats, initcwnd route handling, BBR/fq/buffer calculations, and restore previous/initial semantics remain unchanged. Existing historical backup files are not deleted or rewritten by this refactor.

@LucaLin233
LucaLin233 merged commit e3bcce9 into main Aug 19, 2026
2 checks passed
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.

1 participant