Skip to content

Add --test-timeout flag and fix CI node_modules symlink in diff#92

Merged
that-github-user merged 1 commit into
mainfrom
issue-58-test-timeout
Mar 28, 2026
Merged

Add --test-timeout flag and fix CI node_modules symlink in diff#92
that-github-user merged 1 commit into
mainfrom
issue-58-test-timeout

Conversation

@that-github-user

Copy link
Copy Markdown
Owner

Summary

  • --test-timeout <seconds> flag (default 120) for configurable test timeouts
  • Fix CI failure: exclude node_modules symlink from git add in getDiff/getDiffStats
  • 1 new test for timeout configuration

Generated by thinktank Opus — 5 agents, all pass, 71% convergence, Agent #2 recommended.

Change type

  • Bug fix
  • New feature

Related issue

Closes #58

How to test

npm test  # 80 tests pass
thinktank run --help  # shows --test-timeout flag

Breaking changes

  • This PR introduces breaking changes

🤖 Generated with thinktank (Opus)

- Configurable test timeout via --test-timeout <seconds> (default 120)
- Pass testTimeout through RunOptions → test runner
- Fix CI: exclude node_modules symlink from git add in getDiff/getDiffStats
  (symlink was showing as changed file, breaking worktree lifecycle tests)

Generated by thinktank Opus (5 agents, 71% convergence, Agent #2).
CI fix applied manually.

Closes #58

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

Make test timeout configurable via --test-timeout flag

1 participant