Write go tests

Skill dayvidpham/aura-plugins/.claude/skills/write-go-tests

Use when adding or extending Go tests in strike-cli — *_test.go, table-driven cases, fixtures, httptest, tool Ask helpers, TUI harnesses, coverage gaps. Do not use for running tests only (use test-and-validate).From its SKILL.md

Install

npx -y skills add dayvidpham/aura-plugins --skill write-go-tests

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

2 stars2 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

SKILL.md

3.9 KB, as published. Nobody here has run it

Write Go tests (strike-cli)

Tests only. Never modify production code; if code is untestable or buggy, report it.

Conventions

Framework: stdlib testing only.
Placement: package foo tests beside source as foo_test.go (same package is fine and preferred for unexported helpers).
Style: table-driven where multiple inputs matter; plain functions for stateful sequences.
Isolation: t.TempDir(), t.Setenv, never touch real ~/.strike or network without httptest.
Assert observable behavior (return values, files written, events emitted, errors), not private call graphs.
Bug fixes: add at least one regression test that fails before the fix when feasible.

Helpers by domain

Tools (`internal/tool`)

func allowAll(dir string) *Context {
	return &Context{
		WorkDir: dir, // usually t.TempDir()
		Ask:     func(ctx context.Context, req AskRequest) error { return nil },
	}
}

Cover where relevant: happy path, invalid JSON args, permission rejection via Ask error, path relative/absolute, workspace sandbox escape attempts, filestate stale read after external change, truncation/output caps, deferred/toolsearch registration.

Tool surface is large (read/glob/grep/edit/write/apply_patch/bash + sandbox, task*, webfetch, todo*, memory_, issue_, notebook_edit, sleep, skill, question, plan_mode, phase_done, toolsearch, …). Prefer tests next to the module you touch; do not assume a fixed “six tools” list.

Permissions (`internal/permission`)

Evaluate last-match-wins across layered rulesets.
Service.Ask + Reply: once, always (session grant + sibling resolve), reject cascade, ctx cancel.

Protocol / session

Round-trip event types through protocol.Wrap → Envelope.Decode when adding kinds.
Session: Open → Append → Close → Replay; malformed line errors.

Providers

Prefer httptest.NewServer against internal/provider/base and concrete adapters.
Engine integration: internal/provider/echo (prompts starting with run emit bash tool calls).

Config / auth

Point HOME at t.TempDir() so GlobalPath/cache paths stay sandboxed.
Auth store must stay mode 0600.

TUI

Source lives under internal/tui/_src/ and is flattened into internal/tui via go generate ./internal/tui. Tests may live beside _src or as generated/package tests under internal/tui.

Reuse helpers from internal/tui/_src/app/app_test.go (same package after generate), including: updateApp, runAppCmd, runAllAppCmds, receiveAppOp, assertNoAppOp. Shared support also appears under internal/tui/_src/test/ (e.g. testsupport_test.go). Load skill tui-components before asserting chrome/theme behavior.

After writing

go test ./path/to/package/ -count=1 -v
# then tier gate — load test-and-validate

Priority gaps (fill these first when expanding coverage)

internal/tool — sandbox/workspace, filestate freshness, caps, defer/toolsearch
internal/engine — prune/compaction, interrupt, deferred tool re-promote, multi-tool
internal/permission — Evaluate + Service edge cases
internal/protocol + internal/session — new event kinds, replay
internal/question — multi-question ask/reply
internal/auth — OAuth/PKCE/device, resolve edge cases
internal/provider/* — SSE cancel, cache headers where applicable
internal/tui — keymap/default binds, modals, interrupt/esc paths

Platform notes

Compare canonical paths with filepath.EvalSymlinks when asserting absolute paths (macOS /var → /private/var).
Filesystem mtime granularity: sleep or content-change strategies when testing staleness.