refactor: network impl & cli improvements

[zmh-program]

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Performance improvement
• Code refactoring (no functional changes)

Description

• Network
  • Refactored providers trait and network layer
  • Updated Clippy lint rules
  • Improved code organization and maintainability
• CLI
  • Enhanced progress bar styling and terminal UI
  • Added color support with customizable options
  • Streamlined CLI parameters and code cleanup
  • Fixed WASI build compatibility issues

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Updated (UTC)
✅ Deployment successful! View logs	bradar	84e023f	Jul 10 2025, 08:47 AM

[Copilot]

Pull Request Overview

This PR refactors the network implementation and enhances the CLI with customizable progress and credential support, while cleaning up legacy code and tests.

• Swapped set_github_token for a generic set_provider_credentials API and overhauled ProviderConfig/GitProvider traits.
• Streamlined CLI tests, progress bar styling, and removed trivial assertions and WASI incompatibilities.
• Removed the old core/net.rs implementation and server/mod.rs entry point, improving maintainability.

Reviewed Changes

Copilot reviewed 40 out of 46 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
tests/integration_tests.rs	Removed dummy assert!(true) calls in integration tests.
tests/cli_tests.rs	Updated CLI tests to use set_provider_credentials, removed trivial assertions and legacy async tests.
src/worker.rs	Simplified IntelligentFilter construction and silenced unused variable.
src/server/mod.rs	Deleted legacy analyze_url_server implementation.
src/net/traits.rs	Added full ProviderConfig struct and expanded GitProvider trait.
src/net/stream.rs	Introduced StreamReader and async tarball processing.
src/net/mod.rs	Rewrote RemoteAnalyzer with global/provider configs.
src/lib.rs	Re-exported RemoteAnalyzer from the new net module.
src/core/registry.rs	Derived Default on enums and removed manual impls.
src/core/net.rs	Removed outdated legacy network analyzer implementation.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

Comments suppressed due to low confidence (3)

tests/integration_tests.rs:145

• This test contains no assertions and will always pass. Consider adding checks (e.g., default timeout or config values) to validate the analyzer's state.

        let _default_analyzer = RemoteAnalyzer::default();

tests/cli_tests.rs:11

• No assertions follow this configuration change, so the test cannot catch regressions. Add assertions to verify timeout was applied correctly.

        analyzer.set_timeout(120);

[Copilot]

[nitpick] The computed _largest_file value is never used. Consider removing this unused computation or using it to avoid dead code.