Skip to content

fix(kimi): point kimi-web to international www.kimi.ai - #11045

Merged
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.50from
MeRezaRezaei:fix/kimi-web-international-domain
Aug 22, 2026
Merged

diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.50from
MeRezaRezaei:fix/kimi-web-international-domain

Conversation

@MeRezaRezaei

Copy link
Copy Markdown
Contributor

Summary

  • kimi.com is the China-region endpoint; from outside China it is not reachable, so the kimi-web consumer chat provider must default to the international www.kimi.ai (Connect-RPC API base URL + top-of-page website link).
  • The kimi-coding / kimi-coding-apikey partnership affiliate links intentionally stay on kimi.com (that plan is sold in the China region). Only kimi-web switches to kimi.ai.
  • Refreshes the stale comment in src/lib/providers/validation/webProvidersA.ts (kimi.com -> kimi.ai).
  • Updates the matching unit tests (executor-kimi-web, kimi-partner-aff-links, token-health-check-kimi, providerPageHeaderKimiPartnerLink).

Scope

8 files, 28 insertions / 38 deletions — kimi-web domain correction only, no unrelated changes.

Verification (all clean)

  • prettier --check on all 8 files ✅
  • eslint on all 8 files (exit 0) ✅
  • typecheck:core ✅ · typecheck:noimplicit:core ✅ · check:cycles (no cycles) ✅
  • Affected kimi tests pass: 3 node:test files + 1 vitest UI test ✅

Note

⚠️ base-red inherited: #9985 — release/v3.8.50 is red per the nightly release-green check. That is pre-existing and not introduced by this branch; not fixed here per project policy.

kimi.com is the China-region endpoint; from outside China it is not
reachable, so the kimi-web consumer chat provider must default to the
international www.kimi.ai (Connect-RPC API + top-of-page website link).
The kimi-coding / kimi-coding-apikey partnership aff links intentionally
stay on kimi.com (China region, where that plan is sold).

Also refreshes the stale comment in webProvidersA.ts (kimi.com -> kimi.ai)
and updates the matching unit tests.

base-red inherited: #9985 (release/v3.8.50 is red) - not introduced by this branch.
@diegosouzapw

Copy link
Copy Markdown
Owner

Thanks for this — the alignment work is clean and the kimi tests pass (25/25) on the head. One note to keep the fix coherent: since the chat executor already targets www.kimi.ai, the consistency sweep is incomplete in two live surfaces that still reference kimi.com:

  • src/lib/providers/validation/webProvidersA.ts:43 still probes https://www.kimi.com/api/user (and the Origin/Referer at :47-48 plus the error strings at :29/:38/:58/:72 say kimi.com). If www.kimi.ai is the international endpoint, the credential-validation probe and its messages should point there too (ideally reusing getKimiWebBaseUrl()/the KIMI_WEB_BASE_URL override so validation and chat agree).
  • open-sse/services/tokenExtractionConfig.ts:192-194 still lists the kimi.com login URL and .kimi.com cookie domain for the token-extraction guide.

[MANDATORY] Please update these so the whole kimi-web surface is on one domain, and record a short VPS live check (Hard Rule #18): after logging in at www.kimi.ai, confirm both the Connect-RPC chat endpoint and GET /api/user answer on www.kimi.ai rather than redirecting to kimi.com — the prior comments in #7531/#10944 asserted the opposite, so a live confirmation makes the flip trustworthy. Minor/optional: the kimi-web website field drops the aff=omniroute attribution (web-cookie.ts:326) — just confirm that removal is intended.

@diegosouzapw diegosouzapw added the needs-vps PR requires Hard Rule #18 VPS smoke test on 192.168.0.15 before merge label Aug 21, 2026
@diegosouzapw
diegosouzapw merged commit e15af18 into diegosouzapw:release/v3.8.50 Aug 22, 2026
9 of 16 checks passed
Zartharas pushed a commit to Zartharas/OmniRoute that referenced this pull request Aug 24, 2026
…11045)

Validated on the combined board over tip 80d931a: kimi suites green (executor-kimi-web, kimi-partner-aff-links, token-health-check-kimi 33 assertions), vitest providerPageHeaderKimiPartnerLink green, typecheck:core clean. Audited the diff: only kimi-web switches to the international www.kimi.ai (Connect-RPC base, website, auth hints); kimi-coding / kimi-coding-apikey affiliate links stay on kimi.com as intended — asserted by the updated tests. Owner approved merge without the VPS smoke. Thank you @MeRezaRezaei!
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…11045)

Validated on the combined board over tip 60fbb5b: kimi suites green (executor-kimi-web, kimi-partner-aff-links, token-health-check-kimi 33 assertions), vitest providerPageHeaderKimiPartnerLink green, typecheck:core clean. Audited the diff: only kimi-web switches to the international www.kimi.ai (Connect-RPC base, website, auth hints); kimi-coding / kimi-coding-apikey affiliate links stay on kimi.com as intended — asserted by the updated tests. Owner approved merge without the VPS smoke. Thank you @MeRezaRezaei!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-vps PR requires Hard Rule #18 VPS smoke test on 192.168.0.15 before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants