mirror of
https://github.com/Twigpine/zero.git
synced 2026-11-10 05:01:28 -10:00
* tui: migrate off the deprecated Bubble Tea mouse Type API (SA1019)
The mouse classification helpers checked both the current Button/Action pair and
the deprecated msg.Type / tea.Mouse* constants. Bubble Tea's parser always
populates Button+Action and only DERIVES Type from them, so the Type fallbacks
were pure redundancy (a left-button drag is Action==Motion, already covered by
mouseMotion). Drop them — clears all 11 SA1019 findings in internal/tui with no
behavior change. The left-drag test now asserts the Action==Motion path directly.
Gates: gofmt/vet/build(host+linux+windows)/test -race/staticcheck(SA1019 0)/
deadcode(no new) all pass.
* notify: wire the webhook sink behind an opt-in env var
The WebhookSink (Slack / generic JSON POST on turn completion) was fully
implemented, redaction-safe, and tested — but never attached to a Notifier, so
deadcode flagged its entire machinery (NewWebhookSink, Emit, text, redactLinks,
log, eventType, readSnippet) as unreachable.
Wire it via a small, testable helper, MaybeAddWebhookSink(n, env, logf):
- reads ZERO_NOTIFY_WEBHOOK_URL (+ optional ZERO_NOTIFY_WEBHOOK_SUMMARY)
- no-op when the URL is unset/blank, so it is safe to call unconditionally
- sourcing the URL from the environment keeps the secret token out of any
on-disk config file
Attached at both notifier construction sites:
- headless exec: failed deliveries log to stderr (never stdout); the sink
redacts before logging
- TUI: logf=nil so a delivery failure can't corrupt the alt-screen
The sink stays gated by the existing Mode/focus policy (verified by
TestNotifierOffSuppressesSinks), so a webhook only fires when notifications are
enabled (e.g. --notify both) — no change to default behavior. Opt-in and
fail-closed: with the env var unset nothing is attached.
deadcode: 100 -> 93 unreachable funcs (the 7 WebhookSink entries become
reachable); no new dead code. gofmt/vet/build(host+linux+windows)/test -race/
staticcheck(no new)/govulncheck(0) all pass.
* swarm: add opt-in recurring-spawn scheduler (wakeup + daily)
Adds a dependency-free Scheduler to internal/swarm so a member can be spawned on
a recurring schedule, plus a swarm_schedule tool to drive it.
- Schedule is interval-based ("wakeup", every >= 1s) with an optional first
delay and a MaxRuns cap; a daily "HH:MM" ("cron") time is expressed as a 24h
interval whose first delay lands on the next occurrence.
- Each fire spawns a fresh member through the existing Swarm.Spawn path, so
members inherit the recorded policy and are bounded by the team slot cap +
queue. Non-overlapping by default: a fire is skipped while the job's previous
spawn is still running, preventing queue pile-up.
- Scheduler is lazily created (Swarm.Scheduler()), opt-in (nothing runs until a
job is added), and tied to the Swarm's base context — Swarm.Close() stops the
scheduler first so no spawn fires during shutdown.
- swarm_schedule tool: action add|list|cancel; prompt-gated like swarm_spawn;
add validates schedule + agent type up front (fail fast).
The timing loop takes an injectable ticker, so the tests drive fires
deterministically with no real time (fire/skip/cap/cancel/close, plus pure
nextDailyDelay/parseClock/swarmInt). All swarm tests pass 3x under -race.
Gates: gofmt/vet/build(host+linux+windows)/test/staticcheck(clean)/govulncheck(0)/
deadcode(no new) all pass.
* daemon/remote: SessionLink + git-bundle upload over the bridge
Adds an opt-in way to ship a local repo's git history to a remote daemon and
link a remote working tree to it — without a shared filesystem or a network git
remote. The daemon core (protocol/server) is untouched; all changes live in the
remote package plus a CLI hook.
Wire:
- The auth handshake gains an optional Mode field ("" / "session" => a daemon
session as before; "bundle" => a one-shot git-bundle upload). Unknown modes,
or bundle mode when uploads are disabled, are denied (fail closed).
- Bundle transfer reuses the daemon frame codec: a header frame (link id +
exact size), then the bundle bytes as KindData frames, then a result frame.
Server (Bridge):
- BridgeOptions.BundleDir enables uploads (empty => refused). A received bundle
is size-capped (default 256 MiB), staged to a temp file, `git bundle verify`-d,
then `git clone`-d into a staging dir and atomically renamed over
<BundleDir>/<link-id>. Link ids are sanitized to a single path component and
containment is re-checked, so an upload can never escape the bundle dir.
Client:
- UploadRepoBundle creates `git bundle create --all` of a work tree, hashes it
(sha256), streams it over an authenticated bundle-mode TLS connection, and
returns a SessionLink {address, link id, remote path, bundle sha}. SessionLink
persists as a 0600 atomic JSON file (token never stored) with Save/Load/Validate.
CLI:
- `daemon serve-remote --bundle-dir <d>` enables uploads.
- `daemon link --remote <addr> --repo <dir> --id <name> [--out <file>]` uploads
and prints the remote path (+ a ready run command); `daemon link --show <file>`
prints a saved link.
Tests: git bundle create/verify/clone roundtrip; link-id sanitization + path
containment; SessionLink save/load/validate (0600); and a full bridge upload E2E
over real TLS (extract + content check + re-upload-replaces + disabled + bad
token). All remote tests pass 2x under -race; CLI smoke verified via the binary.
Gates: gofmt/vet/build(host+linux+windows)/test/staticcheck(no new)/
govulncheck(0)/deadcode(no new) all pass.
* mcp: unify OAuth token storage onto the shared oauth store
MCP server tokens lived in their own mcp-oauth-tokens.json with a bespoke
codec/lock, separate from the provider-login store (internal/oauth). The oauth
store already reserved the "mcp:" key namespace for exactly this. Unify them.
- mcp.TokenStore now delegates to internal/oauth.Store, keying each server's
token as "mcp:<server>". Its public API (StoredToken/TokenStatus/Save/Load/
Delete/Status/FilePath) is unchanged, so callers (network client, CLI) are
untouched. The bespoke read/write/lock code is removed in favor of the shared,
cross-process-locked, atomic store.
- Transparent, non-destructive migration on first construction: a legacy
mcp-oauth-tokens.json is imported into the unified store (only entries not
already present — a newer unified token is never overwritten), then renamed to
a ".migrated" backup so it is imported at most once and stays recoverable.
A missing / unreadable / unknown-schema legacy file is left untouched and
never blocks startup. An explicit FilePath suppresses the default-path
migration so tests/overrides can't touch the real user config.
- Server names that can't form a valid namespaced key (oauth.ValidateKey) are
rejected on save and skipped during migration. Added oauth.Store.FilePath().
The token format was already identical (access/refresh/type/scopes/expires), so
this is a key rename, not a format change. Existing TokenStore tests pass
unchanged (they exercise the API); added migration + namespacing + newer-wins
tests. Local run check: `zero mcp oauth status` migrates a seeded legacy file
into oauth-tokens.json under mcp:demo, renames the legacy to .migrated, and leaks
no token material.
Gates: gofmt/vet/build(host+linux+windows)/test -race/staticcheck(clean)/
govulncheck(0)/deadcode(no new) all pass.
* oauth: add an opt-in OS keyring storage backend
Introduces a pluggable storage backend for the unified oauth token store and a
dependency-free OS keyring implementation, selected with
ZERO_OAUTH_STORAGE=keyring (default stays the 0600 file).
- internal/keyring: a small, third-party-dep-free secret store backed by the OS
tooling — `security` on macOS, `secret-tool` (libsecret) on Linux; other
platforms report unsupported. On Linux the secret is passed via stdin (never
the argv); on macOS `security` takes it as an argument (briefly visible in the
process list) — documented, and a reason to prefer the file backend there.
A runner seam makes the per-OS command logic fully testable without a real
keychain.
- internal/oauth: Store now persists its JSON blob through a blobStore backend.
fileBlob preserves the exact prior behavior (0600, atomic write, cross-process
lock file). keyringBlob stores the blob as one base64 entry (keeps multi-line
JSON a single control-char-free value) via a KeyringClient. NewStore resolves
the backend from StoreOptions.Storage or ZERO_OAUTH_STORAGE; unknown storage
and an unavailable keyring fail closed with a clear error.
Because both `zero auth` and the (now unified) MCP token store construct their
store through oauth.NewStore, the keyring backend is honored by both with no
call-site changes. Added oauth.Store.FilePath() returns a "keyring:" identifier
for that backend.
No new module dependency (go.mod unchanged). Tests: keyring round-trip/not-found/
unsupported/missing-binary/stdin-not-argv via an injected runner; store keyring
round-trip + base64-at-rest + storage selection; all existing file-backend tests
pass unchanged. Local run check (macOS): validated the exact `security`
add/find/delete shapes and the not-found exit code (44) against the real binary,
and `mcp oauth status` with ZERO_OAUTH_STORAGE=keyring reads the real keychain
cleanly.
Gates: gofmt/vet/build(host+linux+windows)/test -race/staticcheck(no new)/
govulncheck(0)/deadcode(no new) all pass.
* all: prune unused unexported functions (dead-code sweep)
Removes 15 unexported functions/types/consts that are unreachable even from
tests (every one flagged by staticcheck U1000), clearing the U1000 noise the v16
audit surfaced. Conservative by design: only unexported, test-unused symbols are
removed — exported internal API (which may be an intentional surface or used only
by tests) is deliberately left in place.
Removed:
- agenteval: validateExpectedChangedFiles
- providers/gemini: const providerName
- tui: stripMarkdownInline, removeTrailingAtToken,
noopModelSwitchCompactionPolicy (+ BeforeModelSwitch), deleteComposerLineBefore,
deleteComposerLineAfter, deleteCompletedFileMentionBefore, mcpViewStatus,
mcpServerLines, fitCommandOutput, (*commandPicker).clearQuery,
tuiTheme.onPanel2, indentText
No behavior change (the symbols had no callers). staticcheck U1000 is now empty;
deadcode drops accordingly with no genuinely-new unreachable functions (verified
by a name-based diff vs origin/main). gofmt/vet/build(host+linux+windows)/full
test all pass.
* Address review feedback on the audit backlog
Resolve CodeRabbit review comments on this PR:
- keyring isNotFound: match the specific not-found exit codes (macOS `security`
44 = errSecItemNotFound; `secret-tool` 1) instead of treating any non-zero
exit as "no entry", so a real failure (locked keychain, permissions) surfaces
instead of being masked as absent.
- swarm scheduler daily_at: recompute each fire as the next local HH:MM rather
than adding a fixed 24h, so a wall-clock daily schedule does not drift across
DST. Adds Daily/Hour/Minute to Schedule and a clock to the Scheduler.
- swarm scheduler: reject Add when the scheduler context is already canceled
(not just when closed); re-check cancellation after a tick so a tick+cancel
race can't spawn one extra member after Cancel/Close.
- swarm schedule_tool swarmInt: reject non-integer / non-finite JSON numbers
(max_runs=1.9 errors instead of silently truncating to 1).
- mcp migrateLegacy: resolve a relative LegacyPath to absolute before the
same-file guard so it can't be bypassed by a relative spelling.
- oauth keyring store: serialize the keyring read-modify-write with a
cross-process lock file (best-effort, beside the config dir) so concurrent
processes don't clobber each other's tokens; previously withLock was a no-op.
Adds tests for the non-not-found keyring error, the daily delay recompute, the
Add-after-cancel guard, and non-integer swarmInt.
* swarm: make nextDailyDelay roll the day, not add 24h (DST)
CodeRabbit follow-up: the run loop already recomputes a daily job's delay each
cycle, but nextDailyDelay itself rolled "passed today" to the next occurrence
with Add(24*time.Hour). On a DST changeover day (23h/25h) that fires an hour
early/late. Use time.Date(..., day+1, hour, minute, ...) so Go normalizes the
date and applies the correct local offset, holding the wall-clock HH:MM across
DST. Adds an America/New_York spring-forward test (skips if tzdata is absent).
194 lines
6.0 KiB
Go
194 lines
6.0 KiB
Go
package remote
|
|
|
|
import (
|
|
"context"
|
|
"os"
|
|
"os/exec"
|
|
"path/filepath"
|
|
"runtime"
|
|
"testing"
|
|
)
|
|
|
|
// initTestRepo creates a temp git work tree with one committed file and returns
|
|
// its path. It sets a deterministic identity so it does not depend on global git
|
|
// config.
|
|
func initTestRepo(t *testing.T, file, content string) string {
|
|
t.Helper()
|
|
dir := t.TempDir()
|
|
run := func(args ...string) {
|
|
cmd := exec.Command("git", args...)
|
|
cmd.Dir = dir
|
|
cmd.Env = append(os.Environ(),
|
|
"GIT_AUTHOR_NAME=t", "[email protected]",
|
|
"GIT_COMMITTER_NAME=t", "[email protected]")
|
|
if out, err := cmd.CombinedOutput(); err != nil {
|
|
t.Fatalf("git %v: %v\n%s", args, err, out)
|
|
}
|
|
}
|
|
run("init", "-q")
|
|
run("config", "user.email", "[email protected]")
|
|
run("config", "user.name", "t")
|
|
if err := os.WriteFile(filepath.Join(dir, file), []byte(content), 0o644); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
run("add", ".")
|
|
run("commit", "-q", "-m", "init")
|
|
return dir
|
|
}
|
|
|
|
func TestGitBundleRoundTrip(t *testing.T) {
|
|
repo := initTestRepo(t, "a.txt", "content")
|
|
ctx := context.Background()
|
|
bundle := filepath.Join(t.TempDir(), "x.bundle")
|
|
if err := gitBundleCreate(ctx, repo, bundle); err != nil {
|
|
t.Fatalf("create: %v", err)
|
|
}
|
|
if err := gitBundleVerify(ctx, bundle); err != nil {
|
|
t.Fatalf("verify: %v", err)
|
|
}
|
|
dest := filepath.Join(t.TempDir(), "clone")
|
|
if err := gitClone(ctx, bundle, dest); err != nil {
|
|
t.Fatalf("clone: %v", err)
|
|
}
|
|
if _, err := os.Stat(filepath.Join(dest, "a.txt")); err != nil {
|
|
t.Fatalf("cloned tree missing file: %v", err)
|
|
}
|
|
}
|
|
|
|
func TestIsGitWorktree(t *testing.T) {
|
|
repo := initTestRepo(t, "f", "x")
|
|
if !isGitWorktree(repo) {
|
|
t.Fatal("a git repo should be detected")
|
|
}
|
|
if isGitWorktree(t.TempDir()) {
|
|
t.Fatal("a plain dir should not be detected as a git repo")
|
|
}
|
|
}
|
|
|
|
func TestSanitizeLinkID(t *testing.T) {
|
|
for _, ok := range []string{"proj", "proj-1", "a_b.c", "ABC123"} {
|
|
if _, err := sanitizeLinkID(ok); err != nil {
|
|
t.Fatalf("sanitizeLinkID(%q) unexpected error: %v", ok, err)
|
|
}
|
|
}
|
|
for _, bad := range []string{"", " ", ".", "..", "a/b", "a\\b", "a b", "a$b", string(make([]byte, 200))} {
|
|
if _, err := sanitizeLinkID(bad); err == nil {
|
|
t.Fatalf("sanitizeLinkID(%q) should error", bad)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestWithinDir(t *testing.T) {
|
|
root := t.TempDir()
|
|
if !withinDir(root, filepath.Join(root, "child")) {
|
|
t.Fatal("child should be within root")
|
|
}
|
|
if withinDir(root, filepath.Dir(root)) {
|
|
t.Fatal("parent should not be within root")
|
|
}
|
|
}
|
|
|
|
func TestSessionLinkSaveLoad(t *testing.T) {
|
|
path := filepath.Join(t.TempDir(), "link.json")
|
|
link := SessionLink{Address: "host:9000", ServerName: "host", LinkID: "proj-1", RemotePath: "/bundles/proj-1", BundleSHA256: "deadbeef"}
|
|
if err := link.Save(path); err != nil {
|
|
t.Fatalf("Save: %v", err)
|
|
}
|
|
if runtime.GOOS != "windows" {
|
|
info, err := os.Stat(path)
|
|
if err != nil {
|
|
t.Fatalf("stat: %v", err)
|
|
}
|
|
if perm := info.Mode().Perm(); perm != 0o600 {
|
|
t.Fatalf("link file perm = %v, want 0600", perm)
|
|
}
|
|
}
|
|
got, err := LoadSessionLink(path)
|
|
if err != nil {
|
|
t.Fatalf("Load: %v", err)
|
|
}
|
|
if *got != link {
|
|
t.Fatalf("roundtrip mismatch: %+v != %+v", *got, link)
|
|
}
|
|
}
|
|
|
|
func TestSessionLinkValidate(t *testing.T) {
|
|
for _, bad := range []SessionLink{
|
|
{LinkID: "p", RemotePath: "/r"}, // no address
|
|
{Address: "h:1", RemotePath: "/r"}, // no link id
|
|
{Address: "h:1", LinkID: "p"}, // no remote path
|
|
} {
|
|
if err := bad.Validate(); err == nil {
|
|
t.Fatalf("Validate(%+v) should error", bad)
|
|
}
|
|
if err := bad.Save(filepath.Join(t.TempDir(), "x.json")); err == nil {
|
|
t.Fatalf("Save of invalid link %+v should error", bad)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestUploadRepoBundleRejectsNonRepo(t *testing.T) {
|
|
// A non-git dir is rejected before any dial, so the bogus address is never used.
|
|
_, err := UploadRepoBundle(RemoteConfig{Address: "127.0.0.1:1", Token: "t"}, t.TempDir(), "p")
|
|
if err == nil {
|
|
t.Fatal("a non-git dir should be rejected")
|
|
}
|
|
}
|
|
|
|
func TestBridgeBundleUploadRoundTrip(t *testing.T) {
|
|
srv := newBridgeServer(t, staticLauncher())
|
|
auth, _ := NewTokenAuthenticator("tok")
|
|
bundleRoot := t.TempDir()
|
|
addr, ca := startBridge(t, srv, BridgeOptions{Authenticator: auth, BundleDir: bundleRoot})
|
|
|
|
repo := initTestRepo(t, "hello.txt", "hi there")
|
|
link, err := UploadRepoBundle(RemoteConfig{Address: addr, Token: "tok", CACertFile: ca}, repo, "proj-1")
|
|
if err != nil {
|
|
t.Fatalf("UploadRepoBundle: %v", err)
|
|
}
|
|
wantPath := filepath.Join(bundleRoot, "proj-1")
|
|
if link.RemotePath != wantPath {
|
|
t.Fatalf("remote path = %q, want %q", link.RemotePath, wantPath)
|
|
}
|
|
if link.BundleSHA256 == "" {
|
|
t.Fatal("link should carry a bundle sha256")
|
|
}
|
|
// The extracted work tree should contain the committed file.
|
|
data, err := os.ReadFile(filepath.Join(link.RemotePath, "hello.txt"))
|
|
if err != nil {
|
|
t.Fatalf("extracted tree missing file: %v", err)
|
|
}
|
|
if string(data) != "hi there" {
|
|
t.Fatalf("extracted file content = %q", data)
|
|
}
|
|
|
|
// A second upload to the same link id replaces the prior extraction.
|
|
if _, err := UploadRepoBundle(RemoteConfig{Address: addr, Token: "tok", CACertFile: ca}, repo, "proj-1"); err != nil {
|
|
t.Fatalf("re-upload: %v", err)
|
|
}
|
|
}
|
|
|
|
func TestBridgeBundleDisabledByDefault(t *testing.T) {
|
|
srv := newBridgeServer(t, staticLauncher())
|
|
auth, _ := NewTokenAuthenticator("tok")
|
|
addr, ca := startBridge(t, srv, BridgeOptions{Authenticator: auth, AuthFailDelay: -1}) // no BundleDir
|
|
|
|
repo := initTestRepo(t, "f", "x")
|
|
_, err := UploadRepoBundle(RemoteConfig{Address: addr, Token: "tok", CACertFile: ca}, repo, "p")
|
|
if err == nil {
|
|
t.Fatal("bundle upload must be refused when --bundle-dir is unset")
|
|
}
|
|
}
|
|
|
|
func TestBridgeBundleRejectsBadToken(t *testing.T) {
|
|
srv := newBridgeServer(t, staticLauncher())
|
|
auth, _ := NewTokenAuthenticator("correct")
|
|
addr, ca := startBridge(t, srv, BridgeOptions{Authenticator: auth, BundleDir: t.TempDir(), AuthFailDelay: -1})
|
|
|
|
repo := initTestRepo(t, "f", "x")
|
|
_, err := UploadRepoBundle(RemoteConfig{Address: addr, Token: "wrong", CACertFile: ca}, repo, "p")
|
|
if err == nil {
|
|
t.Fatal("bundle upload with a bad token must be refused")
|
|
}
|
|
}
|