refactor: config_create(), main(), send_files() — reduce duplication and complexity (#149, #150, #151) #155
Closed
TapTap
wants to merge 1 commits from
fix/refactor-cli-config into main
pull from: fix/refactor-cli-config
merge into: TapTap:main
TapTap:dev
TapTap:evaluation-of-codebase-by-human
TapTap:integration/all-fixes-v2
TapTap:fix/features-v2
TapTap:fix/bugs-v2
TapTap:fix/quality-v2
TapTap:fix/security-v2
TapTap:main
TapTap:integration/all-fixes
TapTap:fix/test-skip-tls
TapTap:fix/rsync-parity-all
TapTap:fix/test-coverage-all
TapTap:fix/refactoring-all
TapTap:fix/bugs-all
TapTap:fix/security-all
TapTap:integration/2026-07-29-batch-fix
TapTap:fix/null-checks-cli
TapTap:merge-all-v2
TapTap:revert-pr-143-merge
TapTap:fix/protocol-improvements
TapTap:fix/cli-features
TapTap:fix/tests-and-docs
TapTap:fix/security-hardening
TapTap:fix/performance
TapTap:fix/memory-safety
TapTap:fix/test-coverage
TapTap:fix/integration-cleanup
TapTap:fix/refactoring
TapTap:ci/tier1-review-automation
TapTap:tls-transport
TapTap:add-gh-dependency
TapTap:add-readme
1 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
|
|
0021ca3fcd |
refactor: config_create(), main(), send_files() - reduce duplication and complexity
CI / lint (pull_request) Failing after 11s
CI / build-and-test (pull_request) Has been skipped
CI / sanitizers (address) (pull_request) Has been skipped
CI / sanitizers (undefined) (pull_request) Has been skipped
CI / fuzz-build (pull_request) Has been skipped
CI / coverage (pull_request) Has been skipped
CI / valgrind (pull_request) Has been skipped
- #150: Replace 11-parameter config_create() with config_create() that initializes to sensible defaults; callers set fields directly - #149: Extract validate_config() from main(); reduce main() from 325 to 281 lines by extracting validation logic into separate function - #151: Extract run_dry_run(), connect_to_server(), send_manifest(), and print_progress() shared helpers from send_files()/send_files_multithreaded() to eliminate code duplication |