Skip to content

Add loop tests (self-generated) + --agent flag for all commands - #8

Merged
5uck1ess merged 5 commits into
mainfrom
test-gen/c4b0da8ac338
Apr 3, 2026
Merged

Add loop tests (self-generated) + --agent flag for all commands#8
5uck1ess merged 5 commits into
mainfrom
test-gen/c4b0da8ac338

Conversation

@5uck1ess

@5uck1ess 5uck1ess commented Apr 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • 1408 lines of loop tests generated by devkit test-gen running on itself
  • --agent persistent flag on all loop commands (claude, codex, gemini)
  • Tri-review fixes: DRY flag, generic help text, resolveRunner tests
  • README overhaul: --agent docs, testing, CI/CD, updated roadmap/structure
  • CI/CD hardening: Go module caching, release tag-exists guard

Pre-flight

  • Branch: test-gen/c4b0da8ac338
  • Lint: go vet clean, gofmt clean
  • Tests: 4/4 packages passing (76+ tests), race detection clean
  • Security: no new user input paths, test-only exec.Command usage

Changelog

Features

  • --agent flag on all loop commands (persistent on rootCmd)

Tests

  • 55 loop tests (improve, feature, bugfix, refactor, testgen, review, dispatch)
  • 5 resolveRunner tests (found, case-insensitive, whitespace, not-found, no-agents)

Fixes

  • Generic help text ("Spawns an AI agent" vs "Spawns Claude")
  • gofmt formatting

Docs

  • README: --agent usage, testing section, CI/CD section, scrape command, roadmap updates

CI/CD

  • Go module caching via cache-dependency-path
  • Tag-exists guard prevents duplicate releases

5uck1ess added 5 commits April 3, 2026 11:41
Supports claude, codex, gemini. Shared resolveRunner() helper
shows available agents on error.
… text, resolveRunner tests

- Move --agent flag to rootCmd persistent flag (was duplicated in 6 files)
- Replace "Spawns Claude" with "Spawns an AI agent" in all help text
- Extract resolveRunnerFrom() for testability
- Add 5 tests covering: found, case-insensitive, whitespace, not-found, no-agents
- README: add --agent flag docs, testing section, CI/CD section, scrape command,
  mark harness as done in roadmap, update repo structure
- CI: add Go module caching via cache-dependency-path
- Release: skip tag creation if tag already exists
@5uck1ess
5uck1ess merged commit a3efe41 into main Apr 3, 2026
1 check passed
@5uck1ess
5uck1ess deleted the test-gen/c4b0da8ac338 branch April 3, 2026 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant