Add bidirectional bandwidth diagnostics - #3
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f048762ee8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| Network bandwidth | ||
| Download 80.0 Mbps FAST Cloudflare, up to 2 MB | ||
| Upload 16.0 Mbps FAST Cloudflare, up to 1 MB zero-filled | ||
| Advice Bandwidth is sufficient for everyday AI-assisted development. |
There was a problem hiding this comment.
Remove bandwidth results from the --quick example
This example is introduced under $ ipcheck --quick, but --quick sets both RUN_BANDWIDTH=0 and RUN_UPLOAD=0, so it will only print skipped bandwidth entries rather than the shown 80/16 Mbps measurements and advice. This makes the primary quick-start output impossible to reproduce; use a normal invocation for this sample or omit the bandwidth section.
Useful? React with 👍 / 👎.
What changed
networkQualityreporting to upload throughputWhy
Latency alone does not tell developers whether repository downloads or large AI code-context uploads will be usable through a proxy. This adds clear, bounded, non-billable measurements for both directions.
Validation
npm testbash -n bin/ipcheck test/smoke.shgit diff --check