claudomator.git/internal/executor/executor_test.go, branch story/task-project-fk

fix: remove drain-lock circuit breaker that halted all executions after 3 consecutive failures

2026-04-03T08:39:32+00:00

Co-Authored-By: Claude Sonnet 4.6

fix: story stays PENDING when all subtasks complete but parent task stays QUEUED

2026-03-26T09:36:30+00:00

When a story is approved with pre-created subtasks, parent tasks are QUEUED but never run. Their subtasks complete, but: - maybeUnblockParent only handled BLOCKED parents, not QUEUED ones - checkStoryCompletion required ALL tasks (incl. subtasks) to be done Fixes: - maybeUnblockParent now also promotes QUEUED parents to READY when all subtasks are COMPLETED - checkStoryCompletion only checks top-level tasks (parent_task_id="") - RecoverStaleBlocked now also scans QUEUED parents on startup and triggers checkStoryCompletion if it promotes them - Add QUEUED→READY to valid state transitions (subtask delegation path) Co-Authored-By: Claude Sonnet 4.6

feat: graceful shutdown — drain workers before exit (default 3m timeout)

2026-03-26T09:14:14+00:00

- Add workerWg to Pool; Shutdown() closes workCh and waits for all in-flight execute/executeResume goroutines to finish - Signal handler now shuts down HTTP first, then drains the pool - ShutdownTimeout config field (toml: shutdown_timeout); default 3m - Tests: WaitsForWorkers and TimesOut Co-Authored-By: Claude Sonnet 4.6

fix: story branch push to bare repo; drain at 3 consecutive failures

2026-03-26T06:36:32+00:00

createStoryBranch was pushing to 'origin' which doesn't exist — branches never landed in the bare repo so agents couldn't clone them. Now uses the project's RemoteURL (bare repo path) directly for fetch and push. Raise drain threshold from 2 to 3 consecutive failures to reduce false positives from transient errors. Co-Authored-By: Claude Sonnet 4.6

test: add TestPool_DependsOn_NoDeadlock

2026-03-26T05:49:29+00:00

Co-Authored-By: Claude Sonnet 4.6

chore: replace all master branch references with main

2026-03-25T05:31:14+00:00

- executor.go: merge story branch to main before deploy - container.go: error messages reference git push origin main - api/stories.go: create story branch from origin/main (drop master fallback) - executor_test.go: test setup uses main branch Co-Authored-By: Claude Sonnet 4.6

feat: merge story branch to master before deploy, add doot project to registry

2026-03-24T23:01:22+00:00

- triggerStoryDeploy: fetch/checkout/merge --no-ff/push before running deploy script (ADR-007) - executor_test: TestPool_StoryDeploy_MergesStoryBranch proves merge happens - seed.go: add doot project with deploy script; wire claudomator deploy script Co-Authored-By: Claude Sonnet 4.6

Merge branch 'master' of /site/git.terst.org/repos/claudomator

2026-03-24T22:53:10+00:00

feat: validation result transitions story to REVIEW_READY or NEEDS_FIX (ADR-007)

2026-03-24T22:32:57+00:00

Add checkValidationResult which inspects the final task.State of a completed validation task and updates the story to REVIEW_READY (pass) or NEEDS_FIX (fail). Wire into handleRunResult so stories in VALIDATING state are dispatched to checkValidationResult instead of checkStoryCompletion, covering both success and FAILED terminal paths. Co-Authored-By: Claude Sonnet 4.6

feat: auto-create validation task on story DEPLOYED (ADR-007)

2026-03-24T22:24:40+00:00