feat: require repository_url on tasks; fix UpdateTask to persist it; fix cascade-retry test race

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

1 files changed, 11 insertions, 8 deletions

diff --git a/internal/storage/db.go b/internal/storage/db.go
index ee5ee77..4ae0ab1 100644
--- a/internal/storage/db.go
+++ b/internal/storage/db.go
@@ -19,6 +19,10 @@ func Open(path string) (*DB, error) {
 	if err != nil {
 		return nil, fmt.Errorf("opening database: %w", err)
 	}
+	// SQLite only allows one concurrent writer. Limiting to one open connection
+	// prevents "database is locked" errors when multiple goroutines write
+	// simultaneously via database/sql's connection pool.
+	db.SetMaxOpenConns(1)
 	s := &DB{db: db}
 	if err := s.migrate(); err != nil {
 		db.Close()
@@ -334,9 +338,10 @@ func (s *DB) RejectTask(id, comment string) error {
 
 // TaskUpdate holds the fields that UpdateTask may change.
 type TaskUpdate struct {
-	Name        string
-	Description string
-	Config      task.AgentConfig
+	Name          string
+	Description   string
+	RepositoryURL string
+	Config        task.AgentConfig
 	Priority    task.Priority
 	TimeoutNS   int64
 	Retry       task.RetryConfig
@@ -375,13 +380,11 @@ func (s *DB) UpdateTask(id string, u TaskUpdate) error {
 	now := time.Now().UTC()
 	result, err := s.db.Exec(`
 		UPDATE tasks
-		SET name = ?, description = ?, config_json = ?, priority = ?, timeout_ns = ?,
+		SET name = ?, description = ?, repository_url = ?, config_json = ?, priority = ?, timeout_ns = ?,
 		    retry_json = ?, tags_json = ?, depends_on_json = ?, state = ?, updated_at = ?
 		WHERE id = ?`,
-		u.Name, u.Description, string(configJSON), string(u.Priority), u.TimeoutNS,
-		string(retryJSON), string(tagsJSON), string(depsJSON), string(task.StatePending), now,
-		id,
-	)
+		u.Name, u.Description, u.RepositoryURL, configJSON, string(u.Priority), u.TimeoutNS,
+		retryJSON, tagsJSON, depsJSON, string(task.StatePending), now, id)
 	if err != nil {
 		return err
 	}