Skip to content

Add pre-flight validation before spawning agents#91

Merged
that-github-user merged 1 commit intomainfrom
issue-54-preflight
Mar 28, 2026
Merged

Add pre-flight validation before spawning agents#91
that-github-user merged 1 commit intomainfrom
issue-54-preflight

Conversation

@that-github-user
Copy link
Copy Markdown
Owner

Summary

  • preflightValidation() checks git repo + test command safety before worktree creation
  • Fails fast with actionable error messages instead of cryptic failures mid-run
  • 7 new tests: git repo check, valid test cmd, shell operator rejection, empty cmd

Generated by thinktank Opus — 5 agents, all pass, Agent #5 recommended (+88/-3).

Change type

  • New feature
  • Bug fix

Related issue

Closes #54

How to test

npm test  # 79 tests pass
cd /tmp && thinktank run "task"  # "Not a git repository" error

Breaking changes

  • This PR introduces breaking changes

🤖 Generated with thinktank (Opus)

Check git repo accessibility and test command safety BEFORE creating
worktrees. preflightValidation() returns actionable error messages.
7 new tests covering git check, valid/invalid test commands.

Generated by thinktank Opus (5 agents, all pass, Agent #5 recommended).

Closes #54

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@that-github-user
Copy link
Copy Markdown
Owner Author

Self-review: Preflight runs before worktree creation. Uses existing validateTestCommand. Exports preflightValidation for testing. 79 tests pass, lint clean.

@that-github-user that-github-user merged commit ffeb65c into main Mar 28, 2026
@that-github-user that-github-user deleted the issue-54-preflight branch March 28, 2026 19:00
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.

Add pre-flight validation before spawning agents

1 participant