mirror of
https://github.com/router-for-me/CLIProxyAPI.git
synced 2026-06-23 23:54:09 +08:00
- Updated Antigravity Credits fallback to handle KV store unavailability as a service error. - Enhanced signature caching mechanisms with request-time KV access and sliding expiration. - Added and improved tests for KV client interactions, including error handling and expiration behaviors. - Introduced `CacheSignatureBestEffort` for non-critical signature caching and clarified function flows with required context. - Ensured consistent error reporting for missing or unavailable KV stores in various scenarios. - Replaced direct `homekv` calls with injectable KV client interfaces for `antigravity` and `codex_reasoning_replay` modules. - Improved error reporting and handling for KV operations, including `KVGet`, `KVSet`, `KVDel`, and `KVExpire`. - Introduced dedicated fake KV clients for expanded and granular test coverage. - Added new unit tests to validate KV client behaviors and error scenarios, ensuring robustness and sliding expiration functionality.
238 lines
9.0 KiB
Go
238 lines
9.0 KiB
Go
package helps
|
|
|
|
import (
|
|
"context"
|
|
"encoding/json"
|
|
"errors"
|
|
"net/http"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/router-for-me/CLIProxyAPI/v7/internal/config"
|
|
homekv "github.com/router-for-me/CLIProxyAPI/v7/internal/home"
|
|
cliproxyauth "github.com/router-for-me/CLIProxyAPI/v7/sdk/cliproxy/auth"
|
|
)
|
|
|
|
type fakeClaudeDeviceProfileKVClient struct {
|
|
values map[string][]byte
|
|
getErr error
|
|
setErr error
|
|
setNXErr error
|
|
expireErr error
|
|
setNXResult bool
|
|
getCount int
|
|
setCount int
|
|
setNXCount int
|
|
expireCount int
|
|
lastSetTTL time.Duration
|
|
lastSetNXTTL time.Duration
|
|
lastExpireTTL time.Duration
|
|
}
|
|
|
|
func newFakeClaudeDeviceProfileKVClient() *fakeClaudeDeviceProfileKVClient {
|
|
return &fakeClaudeDeviceProfileKVClient{
|
|
values: make(map[string][]byte),
|
|
setNXResult: true,
|
|
}
|
|
}
|
|
|
|
func (c *fakeClaudeDeviceProfileKVClient) KVGet(_ context.Context, key string) ([]byte, bool, error) {
|
|
c.getCount++
|
|
if c.getErr != nil {
|
|
return nil, false, c.getErr
|
|
}
|
|
value, ok := c.values[key]
|
|
if !ok {
|
|
return nil, false, nil
|
|
}
|
|
return append([]byte(nil), value...), true, nil
|
|
}
|
|
|
|
func (c *fakeClaudeDeviceProfileKVClient) KVSet(_ context.Context, key string, value []byte, opts homekv.KVSetOptions) (bool, error) {
|
|
c.setCount++
|
|
c.lastSetTTL = opts.EX
|
|
if c.setErr != nil {
|
|
return false, c.setErr
|
|
}
|
|
c.values[key] = append([]byte(nil), value...)
|
|
return true, nil
|
|
}
|
|
|
|
func (c *fakeClaudeDeviceProfileKVClient) KVSetNX(_ context.Context, key string, value []byte, ttl time.Duration) (bool, error) {
|
|
c.setNXCount++
|
|
c.lastSetNXTTL = ttl
|
|
if c.setNXErr != nil {
|
|
return false, c.setNXErr
|
|
}
|
|
if _, ok := c.values[key]; ok {
|
|
return false, nil
|
|
}
|
|
if c.setNXResult {
|
|
c.values[key] = append([]byte(nil), value...)
|
|
return true, nil
|
|
}
|
|
return false, nil
|
|
}
|
|
|
|
func (c *fakeClaudeDeviceProfileKVClient) KVExpire(_ context.Context, _ string, ttl time.Duration) (bool, error) {
|
|
c.expireCount++
|
|
c.lastExpireTTL = ttl
|
|
if c.expireErr != nil {
|
|
return false, c.expireErr
|
|
}
|
|
return true, nil
|
|
}
|
|
|
|
func useFakeClaudeDeviceProfileKVClient(t *testing.T, client *fakeClaudeDeviceProfileKVClient, homeMode bool, errClient error) {
|
|
t.Helper()
|
|
previous := currentClaudeDeviceProfileKVClient
|
|
currentClaudeDeviceProfileKVClient = func() (claudeDeviceProfileKVClient, bool, error) {
|
|
return client, homeMode, errClient
|
|
}
|
|
t.Cleanup(func() {
|
|
currentClaudeDeviceProfileKVClient = previous
|
|
})
|
|
}
|
|
|
|
func mustClaudeDeviceProfileJSON(t *testing.T, value claudeDeviceProfileKVValue) []byte {
|
|
t.Helper()
|
|
raw, errMarshal := json.Marshal(value)
|
|
if errMarshal != nil {
|
|
t.Fatalf("marshal device profile: %v", errMarshal)
|
|
}
|
|
return raw
|
|
}
|
|
|
|
func claudeDeviceHeaders(userAgent string) http.Header {
|
|
return http.Header{
|
|
"User-Agent": {userAgent},
|
|
"X-Stainless-Package-Version": {"0.80.0"},
|
|
"X-Stainless-Runtime-Version": {"v24.4.0"},
|
|
"X-Stainless-Os": {"Windows"},
|
|
"X-Stainless-Arch": {"x64"},
|
|
}
|
|
}
|
|
|
|
func TestResolveClaudeDeviceProfileRequiredHomeReadWithoutCandidate(t *testing.T) {
|
|
client := newFakeClaudeDeviceProfileKVClient()
|
|
auth := &cliproxyauth.Auth{ID: "auth-1"}
|
|
key := claudeDeviceProfileKVKey(auth, "api-key")
|
|
client.values[key] = mustClaudeDeviceProfileJSON(t, claudeDeviceProfileKVValue{
|
|
UserAgent: "claude-cli/2.2.0 (external, cli)",
|
|
PackageVersion: "0.80.0",
|
|
RuntimeVersion: "v24.4.0",
|
|
OS: "Windows",
|
|
Arch: "x64",
|
|
})
|
|
useFakeClaudeDeviceProfileKVClient(t, client, true, nil)
|
|
|
|
profile, errProfile := ResolveClaudeDeviceProfileRequired(context.Background(), auth, "api-key", nil, nil)
|
|
if errProfile != nil {
|
|
t.Fatalf("ResolveClaudeDeviceProfileRequired() error = %v", errProfile)
|
|
}
|
|
if profile.UserAgent != "claude-cli/2.2.0 (external, cli)" {
|
|
t.Fatalf("UserAgent = %q, want cached profile", profile.UserAgent)
|
|
}
|
|
if profile.OS != defaultClaudeFingerprintOS || profile.Arch != defaultClaudeFingerprintArch {
|
|
t.Fatalf("platform = %s/%s, want baseline pinned %s/%s", profile.OS, profile.Arch, defaultClaudeFingerprintOS, defaultClaudeFingerprintArch)
|
|
}
|
|
if client.expireCount != 1 || client.lastExpireTTL != claudeDeviceProfileTTL {
|
|
t.Fatalf("KVExpire count/ttl = %d/%v, want 1/%v", client.expireCount, client.lastExpireTTL, claudeDeviceProfileTTL)
|
|
}
|
|
}
|
|
|
|
func TestResolveClaudeDeviceProfileRequiredHomeCandidateLocksRereadsAndWrites(t *testing.T) {
|
|
client := newFakeClaudeDeviceProfileKVClient()
|
|
auth := &cliproxyauth.Auth{ID: "auth-1"}
|
|
useFakeClaudeDeviceProfileKVClient(t, client, true, nil)
|
|
|
|
profile, errProfile := ResolveClaudeDeviceProfileRequired(context.Background(), auth, "api-key", claudeDeviceHeaders("claude-cli/2.2.0 (external, cli)"), nil)
|
|
if errProfile != nil {
|
|
t.Fatalf("ResolveClaudeDeviceProfileRequired() error = %v", errProfile)
|
|
}
|
|
if profile.UserAgent != "claude-cli/2.2.0 (external, cli)" {
|
|
t.Fatalf("UserAgent = %q, want candidate", profile.UserAgent)
|
|
}
|
|
if client.setNXCount != 1 || client.lastSetNXTTL != claudeDeviceProfileLockTTL {
|
|
t.Fatalf("KVSetNX count/ttl = %d/%v, want 1/%v", client.setNXCount, client.lastSetNXTTL, claudeDeviceProfileLockTTL)
|
|
}
|
|
if client.getCount != 1 {
|
|
t.Fatalf("KVGet count = %d, want re-read after lock", client.getCount)
|
|
}
|
|
if client.setCount != 1 || client.lastSetTTL != claudeDeviceProfileTTL {
|
|
t.Fatalf("KVSet count/ttl = %d/%v, want 1/%v", client.setCount, client.lastSetTTL, claudeDeviceProfileTTL)
|
|
}
|
|
}
|
|
|
|
func TestResolveClaudeDeviceProfileRequiredHomeCandidateDoesNotDowngradeCachedProfile(t *testing.T) {
|
|
client := newFakeClaudeDeviceProfileKVClient()
|
|
auth := &cliproxyauth.Auth{ID: "auth-1"}
|
|
key := claudeDeviceProfileKVKey(auth, "api-key")
|
|
client.values[key] = mustClaudeDeviceProfileJSON(t, claudeDeviceProfileKVValue{
|
|
UserAgent: "claude-cli/2.4.0 (external, cli)",
|
|
PackageVersion: "0.90.0",
|
|
RuntimeVersion: "v24.5.0",
|
|
OS: "Windows",
|
|
Arch: "x64",
|
|
})
|
|
useFakeClaudeDeviceProfileKVClient(t, client, true, nil)
|
|
|
|
profile, errProfile := ResolveClaudeDeviceProfileRequired(context.Background(), auth, "api-key", claudeDeviceHeaders("claude-cli/2.3.0 (external, cli)"), nil)
|
|
if errProfile != nil {
|
|
t.Fatalf("ResolveClaudeDeviceProfileRequired() error = %v", errProfile)
|
|
}
|
|
if profile.UserAgent != "claude-cli/2.4.0 (external, cli)" {
|
|
t.Fatalf("UserAgent = %q, want higher cached profile", profile.UserAgent)
|
|
}
|
|
if client.setCount != 0 {
|
|
t.Fatalf("KVSet count = %d, want no downgrade write", client.setCount)
|
|
}
|
|
if client.expireCount != 1 {
|
|
t.Fatalf("KVExpire count = %d, want cached refresh", client.expireCount)
|
|
}
|
|
}
|
|
|
|
func TestResolveClaudeDeviceProfileRequiredHomeFailures(t *testing.T) {
|
|
for _, tc := range []struct {
|
|
name string
|
|
headers http.Header
|
|
client *fakeClaudeDeviceProfileKVClient
|
|
}{
|
|
{name: "read", client: &fakeClaudeDeviceProfileKVClient{values: make(map[string][]byte), getErr: errors.New("get failed")}},
|
|
{name: "lock", headers: claudeDeviceHeaders("claude-cli/2.2.0 (external, cli)"), client: &fakeClaudeDeviceProfileKVClient{values: make(map[string][]byte), setNXResult: true, setNXErr: errors.New("lock failed")}},
|
|
{name: "lock-miss", headers: claudeDeviceHeaders("claude-cli/2.2.0 (external, cli)"), client: &fakeClaudeDeviceProfileKVClient{values: make(map[string][]byte), setNXResult: false}},
|
|
{name: "reread", headers: claudeDeviceHeaders("claude-cli/2.2.0 (external, cli)"), client: &fakeClaudeDeviceProfileKVClient{values: make(map[string][]byte), setNXResult: true, getErr: errors.New("re-read failed")}},
|
|
{name: "write", headers: claudeDeviceHeaders("claude-cli/2.2.0 (external, cli)"), client: &fakeClaudeDeviceProfileKVClient{values: make(map[string][]byte), setNXResult: true, setErr: errors.New("write failed")}},
|
|
} {
|
|
t.Run(tc.name, func(t *testing.T) {
|
|
useFakeClaudeDeviceProfileKVClient(t, tc.client, true, nil)
|
|
if _, errProfile := ResolveClaudeDeviceProfileRequired(context.Background(), &cliproxyauth.Auth{ID: "auth-1"}, "api-key", tc.headers, nil); errProfile == nil {
|
|
t.Fatalf("ResolveClaudeDeviceProfileRequired() error = nil, want error")
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestResolveClaudeDeviceProfileRequiredNonHomeKeepsLocalCache(t *testing.T) {
|
|
ResetClaudeDeviceProfileCache()
|
|
client := newFakeClaudeDeviceProfileKVClient()
|
|
useFakeClaudeDeviceProfileKVClient(t, client, false, nil)
|
|
auth := &cliproxyauth.Auth{ID: "auth-1"}
|
|
cfg := &config.Config{}
|
|
|
|
first, errFirst := ResolveClaudeDeviceProfileRequired(context.Background(), auth, "api-key", claudeDeviceHeaders("claude-cli/2.2.0 (external, cli)"), cfg)
|
|
if errFirst != nil {
|
|
t.Fatalf("ResolveClaudeDeviceProfileRequired() first error = %v", errFirst)
|
|
}
|
|
second, errSecond := ResolveClaudeDeviceProfileRequired(context.Background(), auth, "api-key", nil, cfg)
|
|
if errSecond != nil {
|
|
t.Fatalf("ResolveClaudeDeviceProfileRequired() second error = %v", errSecond)
|
|
}
|
|
if second.UserAgent != first.UserAgent {
|
|
t.Fatalf("cached UserAgent = %q, want %q", second.UserAgent, first.UserAgent)
|
|
}
|
|
if client.getCount != 0 || client.setCount != 0 || client.setNXCount != 0 {
|
|
t.Fatalf("KV calls = get %d set %d setnx %d, want all zero", client.getCount, client.setCount, client.setNXCount)
|
|
}
|
|
}
|