Skip to content

kyzn(deep): improve auto — health 98 → 98#3

Merged
bokiko merged 1 commit intomainfrom
kyzn/20260318-auto-20096b55
Mar 18, 2026
Merged

kyzn(deep): improve auto — health 98 → 98#3
bokiko merged 1 commit intomainfrom
kyzn/20260318-auto-20096b55

Conversation

@bokiko
Copy link
Owner

@bokiko bokiko commented Mar 18, 2026

kyzn Improvement Report

Health Score: 98 → 98 (→ 0)
Mode: deep | Focus: auto
Cost: $0.6552492000000001
Run ID: 20260318-011058-20096b55

Changes

 desktop/src/api_client.py  |  3 +++
 desktop/src/cli.py         |  2 ++
 desktop/src/ping_tester.py | 17 ++++++++++++++++-
 3 files changed, 21 insertions(+), 1 deletion(-)

Verification

  • Build: ✅ Passed
  • Tests: ✅ Passed

To approve: kyzn approve 20260318-011058-20096b55
To reject: kyzn reject 20260318-011058-20096b55 --reason "..."

Generated by kyzn — autonomous code improvement

Health: 98 → 98 (→0)
Cost: $0.6552492000000001
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 19eb2db615

ℹ️ 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".

Comment on lines +257 to +261
ping_avg=0.0, ping_min=0.0, ping_max=0.0,
jitter=0.0, packet_loss=100.0,
successful_pings=0, total_pings=ping_count,
raw_times=[],
region=server.get("region", ""),

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Don't misreport malformed server entries as timeouts

When test_server() raises on bad server metadata (for example, the API/defaults omit ip or location), this fallback turns that exception into packet_loss=100.0. Both the CLI and GUI render any packet_loss >= 100 as a normal timeout/unreachable server (desktop/src/cli.py:134 and desktop/src/gui.py:797-800), so a backend/config regression gets presented as network data and can change which region is recommended. Please preserve a distinct error state instead of classifying these exceptions as packet loss.

Useful? React with 👍 / 👎.

@bokiko bokiko merged commit 6ef9e9f into main Mar 18, 2026
1 check 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