nullkey
f2e8e3f56c
Survey of 10 mainstream CLIs (gh, lark, stripe, vercel, supabase, aws, azure, gcloud, openai/codex, github-copilot-cli) showed env-gated per-command --help blurbs are a Stripe-only pattern; gh uses env detect for telemetry only, and lark relies on installed agent Skills + MCP. Our cmd/mcp/serve already covers the dominant 2025/26 path, so internal/aiclient/ (136 LOC + 38 callsites) is net maintenance burden without precedent. - Drop internal/aiclient/ entirely (annotations + detect + tests) - Remove 38 SetAgentHelp callsites + agentAwareHelpFunc / SetHelpFunc wiring in cmd/root.go - Migrate 4 command-level rules to standard Long help (visible to all, not env-gated): doc upload mode mutex, kb edit at-least-one, kb pin idempotent, search chunks channel mutex - Rewrite AGENTS.md as a developer guide (gh-style 6 H2 / 167 lines): audience preamble + Build / Architecture / Command Structure / Testing / Code Style / Error Handling. Drops sections absent in surveyed projects (Commit & PR Conventions, Who Uses This CLI) - Clean 14 internal doc refs (ADR-N, spec §X, v0.X) in source comments and docs that pointed at docs/superpowers/ — that directory is local-only / uncommitted, so refs are dead for outside readers - Drop forward-looking "once v0.2 ships" from README
39 lines
1.1 KiB
Go
39 lines
1.1 KiB
Go
// Package compat probes server `/system/info` and decides client-server
|
|
// version skew level. Used by `weknora doctor`'s server_version check.
|
|
//
|
|
// 24h TTL state file cache, three-tier compat semantics
|
|
// (major-mismatch=hard / minor-mismatch=soft / equal=ok).
|
|
package compat
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"time"
|
|
|
|
sdk "github.com/Tencent/WeKnora/client"
|
|
)
|
|
|
|
// Info is the cached server version snapshot.
|
|
type Info struct {
|
|
ServerVersion string `yaml:"server_version"`
|
|
EngineInfo map[string]string `yaml:"engine_info,omitempty"`
|
|
ProbedAt time.Time `yaml:"probed_at"`
|
|
}
|
|
|
|
// ProbeService is the narrow SDK surface compat depends on; tests inject a fake.
|
|
type ProbeService interface {
|
|
GetSystemInfo(ctx context.Context) (*sdk.SystemInfo, error)
|
|
}
|
|
|
|
// Probe calls server /system/info and returns a fresh Info snapshot.
|
|
func Probe(ctx context.Context, c ProbeService) (*Info, error) {
|
|
resp, err := c.GetSystemInfo(ctx)
|
|
if err != nil {
|
|
return nil, fmt.Errorf("probe server: %w", err)
|
|
}
|
|
return &Info{
|
|
ServerVersion: resp.Version,
|
|
ProbedAt: time.Now(),
|
|
}, nil
|
|
}
|