summaryrefslogblamecommitdiff
path: root/makima/src/db/repository.rs
blob: 2b595b58a69205f200971a5f4231d564c9e2eda2 (plain) (tree)
1
2
3
4
5
6
7
8
9
10
3400
3401
3402
3403
3404
3405
3406
3407
3408
3409
3410
3411
3412
3413
3414
3415
3416
3417
3418
3419
3420
3421
3422
3423
3424
3425
3426
3427
3428
3429
3430
3431
3432
3433
3434
3435
3436
3437
3438
3439
3440
3441
3442
3443
3444
3445
3446
3447
3448
3449
3450
3451
3452
3453
3454
3455
3456
3457
3458
3459
3460
3461
3462
3463
3464
3465
3466
3467
3468
3469
3470
3471
3472
3473
3474
3475
3476
3477
3478
3479
3480
3481
3482
3483
3484
3485
3486
3487
3488
3489
3490
3491
3492
3493
3494
3495
3496
3497
3498
3499
3500
3501
3502
3503
3504
3505
3506
3507
3508
3509
3510
3511
3512
3513
3514
3515
3516
3517
3518
3519
3520
3521
3522
3523
3524
3525
3526
3527
3528
3529
3530
3531
3532
3533
3534
3535
3536
3537
3538
3539
3540
3541
3542
3543
3544
3545
3546
3547
3548
3549
3550
3551
3552
3553
3554
3555
3556
3557
3558
3559
3560
3561
3562
3563
3564
3565
3566
3567
3568
3569
3570
3571
3572
3573
3574
3575
3576
3577
3578
3579
3580
3581
3582
3583
3584
3585
3586
3587
3588
3589
3590
3591
3592
3593
3594
3595
3596
3597
3598
3599
3600
3601
3602
3603
3604
3605
3606
3607
3608
3609
3610
3611
3612
3613
3614
3615
3616
3617
3618
3619
3620
3621
3622
3623
3624
3625
3626
3627
3628
3629
3630
3631
3632
3633
3634
3635
3636
3637
3638
3639
3640
3641
3642
3643
3644
3645
3646
3647
3648
3649
3650
3651
3652
3653
3654
3655
3656
3657
3658
3659
3660
3661
3662
3663
3664
3665
3666
3667
3668
3669
3670
3671
3672
3673
3674
3675
3676
3677
3678
3679
3680
3681
3682
3683
3684
3685
3686
3687
3688
3689
3690
3691
3692
3693
3694
3695
3696
3697
3698
3699
3700
3701
3702
3703
4279
4280
4281
4282
4283
4284
4285
4286
4287
4288
4289
4290
4291
4292
4293
4294
4295
4296
4297
4298
4299
4300
4301
4302
4303
4304
4305
4306
4307
4308
4309
4310
4311
4312
4313
4314
4315
4316
4317
4318
4319
4320
4321
4322
4323
4324
4325
4326
4327
4328
4329
4330
4331
4332
4333
4334
4335
4336
4337
4338
4339
4340
4341
4342
4343
4344
4345
4346
4347
4348
4349
4350
4351
4352
4353
4354
4355
4356
4357
4358
4359
4360
4361
4362
4363
4364
4365
4366
4367
4368
4369
4370
4371
4372
4373
4374
4375
4376
4377
4378
4379
4380
4381
4382
4383
4384
4385
4386
4387
4388
4389
4390
4391
4392
4393
4394
4395
4396
4397
4398
4399
4400
4401
4402
4403
4404
4405
4406
4407
4408
4409
4410
4411
4412
4413
4414
4415
4416
4417
4418
4419
4420
4421
4422
4423
4424
4425
4426
4427
4428
4429
4430
4431
4432
4433
4434
4435
4436
4437
4438
4439
4440
4441
4442
4443
4444
4445
4446
4447
4448
4449
4450
4451
4452
4453
4454
4455
4456
4457
4458
4459
4460
4461
4462
4463
4464
4465
4466
4467
4468
4469
4470
4471
4472
4473
4474
4475
4476
4477
4478
4479
4480
4481
4482
4483
4484
4485
4486
4487
4488
4489
4490
4491
4492
4493
4494
4495
4496
4497
4498
4499
4500
4501
4502
4503
4504
4505
4506
4507
4508
4509
4510
4511
4512
4513
4514
4515
4516
4517
4518
4519
4520
4521
4522
4523
4524
4525
4526
4527
4528
4529
4530
4531
4532
4533
4534
4535
4536
4537
4538
4539
4540
4541
4542
4543
4544
4545
4546
4547
4548
4549
4550
4551
4552
4553
4554
4555
4556
4557
4558
4559
4560
4561
4562
4563
4564
4565
4566
4567
4568
4569
4570
4571
4572
4573
4574
4575
4576
4577
4578
4579
4580
4581
4582
4583
4584
4585
4586
4587
4588
4589
4590
4591
4592
4593
4594
4595
4596
4597
4598
4599
4600
4601
4602
4603
4604
4605
4606
4607
4608
4609
4610
4611
4612
4613
4614
4615
4616
4617
4618
4619
4620
4621
4622
4623
4624
4625
4626
4627
4628
4629
5854
5855
5856
5857
5858
5859
5860
5861
5862
5863
5864
5865
5866
5867
5868
5869
5870
5871
5872
5873
5874
5875
5876
5877
5878
5879
5880
5881
5882
5883
5884
5885
5886
5887
5888
5889
5890
5891
5892
5893
5894
5895
5896
5897
5898
5899
5900
5901
5902
5903
5904
5905
5906
5907
5908
5909
5910
5911
5912
5913
5914
5915
5916
5917
5918
5919
5920
5921
5922
5923
5924
5925
5926
5927
5928
5929
5930
5931
5932
5933
5934
5935
5936
5937
5938
5939
5940
5941
5942
5943
5944
5945
5946
5947
5948
5949
5950
5951
5952
5953
5954
5955
5956
5957
5958
5959
5960
5961
5962
5963
5964
5965
5966
5967
5968
5969
5970
5971
5972
5973
5974
5975
5976
5977
5978
5979
5980
5981
5982
5983
5984
5985
5986
5987
5988
5989
5990
5991
5992
5993
5994
5995
5996
5997
5998
5999
6000
6001
6002
6003
6004
6005
6006
6007
6008
6009
6010
6011
6012
6013
6014
6015
6016
6017
6018
6019
6020
6021
6022
6023
6024
6025
6026
6027
6028
6029
6030
6031
6032
6033
6034
6035
6036
6037
6038
6039
6040
6041
6042
6043
6044
6045
6046
6047
6048
6049
6050
6051
6052
6053
6054
6055
6056
6057
6058
6059
6060
6061
6062
6063
6064
6065
6066
6067
6068
6069
6070
6071
6072
6073
6074
6075
6076
6077
6078
6079
6080
6081
6082
6083
6084
6085
6086
6087
6088
6089
6090
6091
6092
6093
6094
6095
6096
6097
6098
6099
6100
6101
6102
6103
6104
6105
6106
6107
6108
6109
6110
6111
6112
6113
6114
6115
6116
6117
6118
6119
6120
6121
6122
6123
6124
6125
6126
6127
6128
6129
6130
6131
6132
6133
6134
6135
6136
6137
6138
6139
6140
6141
6142
6143
6144
6145
6146
6147
6148
6149
6150
6151
6152
6153
6154
6155
6156
6157
6158
6159
6160
6161
6162
6163
6164
6165
6166
6167
6168
6169
6170
6171
6172
6173
6174
6175
6176
6177
6178
6179
6180
6181
6182
6183
6184
6185
6186
6187
6188
6189
6190
6191
6192
6193
6194
6195
6196
6197
6198
6199
6200
6201
6202
6203
6204
6205
6206
6207
6208
6209
6210
6211
6212
6213
6214
6215
6216
6217
6218
6219
6220
6221
6222
6223
6224
6225
6226
6227
6228
6229
6230
6231
6232
6233
6234
6235
6236
6237
6238
6239
6240
6241
6242
6243
6244
6245
6246
6247
6248
6249
6250
6251
6252
6253
6254
6255
6256
6257
6258
6259
6260
6261
6262
6263
6264


                                                    
                       


                 
                    

                                                                                              

                                                                                                  









                                                                                                   
  
 




































                                                                               





                                                           











                                                                                                      

                                                                                    
                                                                                             


                              

                                                                                  
                                                                                                                                                                                                      

           



                           
                     







                                                                                     
                                                                                                                                                                                                   
                  
                     


             



                         
                                               


                                                                          
                                                                                                                                                                                                   
                  


                                



                    





                                                                                          



                           
                                            





                                             









                                                         




                                                                                


                                                                    
 




                                                                          

                                                                                                         
                                                                                                                                                                                                          


                 












                                                

                                                                                                         
                                                                                                                                                                                                          


                 




















                                                                    






                                                                                
                     


             





                                  
                      
                                                                     
                                                                     




                        

                                                                                



                                                                                
                                                                                             






                                                                                    














                                                                            


                              


                                                                                                                                                                                                      


                   

                          




                           
                              











                                        
                                                                                                                                                                                                   













                                                                                                    
                                                                                                                                                                                                   









                                

































































                                                                             






































                                                                                                         
                                                                                                                                                                                                          


















                                                                                                         
                                                                                                                                                                                                          















































                                                                                



























































































































                                                                                                             
                             















                                                                                              












                                                                                           



                                                                                

                                                                                              
                                                                                                                     
                                                     
                                                                  




                                                          

                                                                                                       







                                                                                            
                                                                           

                                                                  
                                                                                                                           
                
                                                                                                     

                  
                                













                                                                                                       
                                                                                  
                                                                                  
                                                                                   
                                                                                  
         
                                                                                                     


                   
                      





                             
                            







                                     

                                     
                                           


















                                                                                     
                                         



                                                                                 
                                                                                    
                                        


                                                                                      
                                                                                                                   
                    
                                                     
                                                                            











                                                                                                     
                                                                                    
                                        


                                                                                      
                                                                                                                   
                    
                                                     








                                                   


















                                                              
                                                                 

                                                                               










                                                




                                         







                      



















                                                                                 






















































                                                                                    























































































































































                                                                                                  
                                                                                                                     
                                                     
                                                                                              












                                                                                                                                

                                                                                                       
 









                                                                                            
                                                                                                                           
                
                                                                                                     

                  
                                













                                                                                                       
                                                                                            
                                                                                  
                                                                                   
                                                                                  
         
                                                                                                          



                   
                      





                             
                            







                                     

                                     
                                           























                                                                                  
                                         






                                            
                                                                                    
                                        


                                                                                      
                                                                                                                   
                    
                                                     
                                                                                                
















                                                    
                                                                                    
                                        


                                                                                      
                                                                                                                   
                    
                                                     











































                                                                              
                                                                        

                                                                                 
                                                       













                                                                                   
                                                                                      

                                                             

















                                 
                              
                     









                                                                                   
                                                                                      
                                                


















                                           
                              
                     



















































































































































































































































































                                                                                                                   



















                                                 
































































                                                                                  


















                                                                          














                                                                                
                                                                                            
                                                


                                                                                              
                                                                                                                                   
                            
                                                             













                                                                               
                                                                                            
                                                


                                                                                              
                                                                                                                                   
                            
                                                             























































































































































































































































































                                                                                                             


















































































































                                                                                  




















                                                                                                              



                                                











































































































































































































































                                                                                                                         



                                                                                
                                                                                                




                                       
















                                                                                         
 








                                                                                              
 






                                                                                                  
 

                                                                            
 

                                                                  
                                                 
                                                                                    
                  

                                      


            
                                                               
                                                       
                                                     
                                                                 


                                                                     
 

                                  

                                                                                                                                                           





                           
                             
                
                          
                      
                     
                           
                            






























                                                                
                                                                            
                                                                                            





















                                                                                                     
                                                                            
                                                                                            








































                                                                                                     
                                                                                  
                                                                      
                                                                   
                                                                                     





                                                                     

                                                                                                                                                                    









                                 
                              
                          
                         
                               







                                                                     
                                                                                                                                                                    










                                           
                              
                          
                         
                               





































                                                                                  


                                                                                                 














































                                                                                            
































































































                                                                                        

































































































































































































































































































                                                                                                                                                                                                   
                                        


                                                                                      
                                                                                                                   











                                                     





                                                      


                                                                               









                                                                           
                                                                       








                                                                                      


















































































































































































































































                                                                                                                          















































































































































































































































































































                                                                                            
                                                                               

























































































































































                                                                                                                  























                                                                               






















































                                                                                    






























































































                                                                                                 




























                                                                                 


















































                                                                   





































































































































































                                                                                                                                      






























































































































































































































































































































































                                                                                                                               



























                                                                                











































































































                                                                                       



                                                                                





































































































































                                                                                                    















                                                                                

                                                                                                                





                           





























































































































































































































































                                                                                                          





                                                                              




                         



















































































































































                                                                                                                    

































































































































































































                                                                                                
                                                                              

































                                                                                        
                             









                                                               




















                                                                                      
                                                          




                                                                                  



                                                   




                                                                           

                                                                                                                                                           










                             
                           















































                                                                                  



                                                   










                                                              


                                                  











                             
                           


































































































































                                                                                              












                                                                                  


























































































































































































































































































































































































































                                                                                                                
//! Repository pattern for file database operations.

use chrono::Utc;
use serde::Deserialize;
use sqlx::PgPool;
use uuid::Uuid;

use super::models::{
    AddChainRepositoryRequest, AddContractDefinitionRequest, AddContractToChainRequest, Chain,
    ChainContract, ChainContractDefinition, ChainContractDetail, ChainDefinitionGraphNode,
    ChainDefinitionGraphResponse, ChainEditorContract, ChainEditorData, ChainEditorDeliverable,
    ChainEditorEdge, ChainEditorNode, ChainEditorTask, ChainEvent, ChainGraphEdge, ChainGraphNode,
    ChainGraphResponse, ChainRepository, ChainSummary, ChainWithContracts, CheckpointPatch,
    CheckpointPatchInfo, Contract, ContractChatConversation, ContractChatMessageRecord,
    ContractEvent, ContractRepository, ContractSummary, ContractTypeTemplateRecord,
    ConversationMessage, ConversationSnapshot, CreateChainRequest, CreateContractRequest,
    CreateFileRequest, CreateTaskRequest, CreateTemplateRequest, Daemon, DaemonTaskAssignment,
    DaemonWithCapacity, DeliverableDefinition, File, FileSummary, FileVersion, HistoryEvent,
    HistoryQueryFilters, MeshChatConversation, MeshChatMessageRecord, PhaseChangeResult,
    PhaseConfig, PhaseDefinition, SupervisorHeartbeatRecord, SupervisorState, Task, TaskCheckpoint,
    TaskEvent, TaskSummary, UpdateChainRequest, UpdateContractDefinitionRequest,
    UpdateContractRequest, UpdateFileRequest, UpdateTaskRequest, UpdateTemplateRequest,
};

/// Repository error types.
#[derive(Debug)]
pub enum RepositoryError {
    /// Database error
    Database(sqlx::Error),
    /// Version conflict (optimistic locking failure)
    VersionConflict {
        /// The version the client expected
        expected: i32,
        /// The actual current version in the database
        actual: i32,
    },
}

impl From<sqlx::Error> for RepositoryError {
    fn from(e: sqlx::Error) -> Self {
        RepositoryError::Database(e)
    }
}

impl std::fmt::Display for RepositoryError {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        match self {
            RepositoryError::Database(e) => write!(f, "Database error: {}", e),
            RepositoryError::VersionConflict { expected, actual } => {
                write!(
                    f,
                    "Version conflict: expected {}, actual {}",
                    expected, actual
                )
            }
        }
    }
}

impl std::error::Error for RepositoryError {}

/// Generate a default name based on current timestamp.
fn generate_default_name() -> String {
    let now = Utc::now();
    now.format("Recording - %b %d %Y %H:%M:%S").to_string()
}

/// Internal request for creating files without contract association (e.g., audio transcription).
/// User-facing file creation should use CreateFileRequest which requires contract_id.
pub struct InternalCreateFileRequest {
    pub name: Option<String>,
    pub description: Option<String>,
    pub transcript: Vec<super::models::TranscriptEntry>,
    pub location: Option<String>,
}

/// Create a new file record (internal use, no contract required).
/// For user-facing file creation, use create_file_for_owner which requires a contract.
pub async fn create_file(pool: &PgPool, req: InternalCreateFileRequest) -> Result<File, sqlx::Error> {
    let name = req.name.unwrap_or_else(generate_default_name);
    let transcript_json = serde_json::to_value(&req.transcript).unwrap_or_default();
    let body_json = serde_json::to_value::<Vec<super::models::BodyElement>>(vec![]).unwrap();

    sqlx::query_as::<_, File>(
        r#"
        INSERT INTO files (name, description, transcript, location, summary, body)
        VALUES ($1, $2, $3, $4, NULL, $5)
        RETURNING id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
        "#,
    )
    .bind(&name)
    .bind(&req.description)
    .bind(&transcript_json)
    .bind(&req.location)
    .bind(&body_json)
    .fetch_one(pool)
    .await
}

/// Get a file by ID.
pub async fn get_file(pool: &PgPool, id: Uuid) -> Result<Option<File>, sqlx::Error> {
    sqlx::query_as::<_, File>(
        r#"
        SELECT id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
        FROM files
        WHERE id = $1
        "#,
    )
    .bind(id)
    .fetch_optional(pool)
    .await
}

/// List all files, ordered by created_at DESC.
pub async fn list_files(pool: &PgPool) -> Result<Vec<File>, sqlx::Error> {
    sqlx::query_as::<_, File>(
        r#"
        SELECT id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
        FROM files
        ORDER BY created_at DESC
        "#,
    )
    .fetch_all(pool)
    .await
}

/// Update a file by ID with optimistic locking.
///
/// If `req.version` is provided, the update will only succeed if the current
/// version matches. Returns `RepositoryError::VersionConflict` if there's a mismatch.
///
/// If `req.version` is None (e.g., internal system updates), version checking is skipped.
pub async fn update_file(
    pool: &PgPool,
    id: Uuid,
    req: UpdateFileRequest,
) -> Result<Option<File>, RepositoryError> {
    // Get the existing file first
    let existing = get_file(pool, id).await?;
    let Some(existing) = existing else {
        return Ok(None);
    };

    // Check version if provided (optimistic locking)
    if let Some(expected_version) = req.version {
        if existing.version != expected_version {
            return Err(RepositoryError::VersionConflict {
                expected: expected_version,
                actual: existing.version,
            });
        }
    }

    // Apply updates
    let name = req.name.unwrap_or(existing.name);
    let description = req.description.or(existing.description);
    let transcript = req.transcript.unwrap_or(existing.transcript);
    let transcript_json = serde_json::to_value(&transcript).unwrap_or_default();
    let summary = req.summary.or(existing.summary);
    let body = req.body.unwrap_or(existing.body);
    let body_json = serde_json::to_value(&body).unwrap_or_default();

    // Update with version check in WHERE clause for race condition safety
    let result = if req.version.is_some() {
        sqlx::query_as::<_, File>(
            r#"
            UPDATE files
            SET name = $2, description = $3, transcript = $4, summary = $5, body = $6, updated_at = NOW()
            WHERE id = $1 AND version = $7
            RETURNING id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
            "#,
        )
        .bind(id)
        .bind(&name)
        .bind(&description)
        .bind(&transcript_json)
        .bind(&summary)
        .bind(&body_json)
        .bind(req.version.unwrap())
        .fetch_optional(pool)
        .await?
    } else {
        // No version check for internal updates
        sqlx::query_as::<_, File>(
            r#"
            UPDATE files
            SET name = $2, description = $3, transcript = $4, summary = $5, body = $6, updated_at = NOW()
            WHERE id = $1
            RETURNING id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
            "#,
        )
        .bind(id)
        .bind(&name)
        .bind(&description)
        .bind(&transcript_json)
        .bind(&summary)
        .bind(&body_json)
        .fetch_optional(pool)
        .await?
    };

    // If versioned update returned None, there was a race condition
    if result.is_none() && req.version.is_some() {
        // Re-fetch to get the actual version
        if let Some(current) = get_file(pool, id).await? {
            return Err(RepositoryError::VersionConflict {
                expected: req.version.unwrap(),
                actual: current.version,
            });
        }
    }

    Ok(result)
}

/// Delete a file by ID.
pub async fn delete_file(pool: &PgPool, id: Uuid) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM files
        WHERE id = $1
        "#,
    )
    .bind(id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Count total files.
pub async fn count_files(pool: &PgPool) -> Result<i64, sqlx::Error> {
    let result: (i64,) = sqlx::query_as("SELECT COUNT(*) FROM files")
        .fetch_one(pool)
        .await?;

    Ok(result.0)
}

// =============================================================================
// Owner-Scoped File Functions
// =============================================================================

/// Create a new file record for a specific owner.
/// Files must belong to a contract - the contract_id is required and the phase is looked up.
pub async fn create_file_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
    req: CreateFileRequest,
) -> Result<File, sqlx::Error> {
    let name = req.name.unwrap_or_else(generate_default_name);
    let transcript_json = serde_json::to_value(&req.transcript).unwrap_or_default();
    // Use body from request (may be empty or contain template elements)
    let body_json = serde_json::to_value(&req.body).unwrap_or_default();

    // Use provided contract_phase, or look up from contract's current phase
    let contract_phase: Option<String> = if req.contract_phase.is_some() {
        req.contract_phase
    } else {
        sqlx::query_scalar(
            "SELECT phase FROM contracts WHERE id = $1 AND owner_id = $2",
        )
        .bind(req.contract_id)
        .bind(owner_id)
        .fetch_optional(pool)
        .await?
    };

    sqlx::query_as::<_, File>(
        r#"
        INSERT INTO files (owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, repo_file_path)
        VALUES ($1, $2, $3, $4, $5, $6, $7, NULL, $8, $9)
        RETURNING id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
        "#,
    )
    .bind(owner_id)
    .bind(req.contract_id)
    .bind(&contract_phase)
    .bind(&name)
    .bind(&req.description)
    .bind(&transcript_json)
    .bind(&req.location)
    .bind(&body_json)
    .bind(&req.repo_file_path)
    .fetch_one(pool)
    .await
}

/// Get a file by ID, scoped to owner.
pub async fn get_file_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<Option<File>, sqlx::Error> {
    sqlx::query_as::<_, File>(
        r#"
        SELECT id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
        FROM files
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .fetch_optional(pool)
    .await
}

/// List all files for an owner, ordered by created_at DESC.
pub async fn list_files_for_owner(pool: &PgPool, owner_id: Uuid) -> Result<Vec<File>, sqlx::Error> {
    sqlx::query_as::<_, File>(
        r#"
        SELECT id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
        FROM files
        WHERE owner_id = $1
        ORDER BY created_at DESC
        "#,
    )
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Database row type for file summary with contract info
#[derive(Debug, sqlx::FromRow)]
struct FileSummaryRow {
    id: Uuid,
    contract_id: Option<Uuid>,
    contract_name: Option<String>,
    contract_phase: Option<String>,
    name: String,
    description: Option<String>,
    #[sqlx(json)]
    transcript: Vec<crate::db::models::TranscriptEntry>,
    version: i32,
    repo_file_path: Option<String>,
    repo_sync_status: Option<String>,
    created_at: chrono::DateTime<chrono::Utc>,
    updated_at: chrono::DateTime<chrono::Utc>,
}

/// List file summaries for an owner with contract info (joined).
pub async fn list_file_summaries_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
) -> Result<Vec<FileSummary>, sqlx::Error> {
    let rows = sqlx::query_as::<_, FileSummaryRow>(
        r#"
        SELECT
            f.id, f.contract_id, c.name as contract_name, f.contract_phase,
            f.name, f.description, f.transcript, f.version,
            f.repo_file_path, f.repo_sync_status, f.created_at, f.updated_at
        FROM files f
        LEFT JOIN contracts c ON f.contract_id = c.id
        WHERE f.owner_id = $1
        ORDER BY f.created_at DESC
        "#,
    )
    .bind(owner_id)
    .fetch_all(pool)
    .await?;

    Ok(rows
        .into_iter()
        .map(|row| {
            let duration = row
                .transcript
                .iter()
                .map(|t| t.end)
                .fold(0.0_f32, f32::max);
            FileSummary {
                id: row.id,
                contract_id: row.contract_id,
                contract_name: row.contract_name,
                contract_phase: row.contract_phase,
                name: row.name,
                description: row.description,
                transcript_count: row.transcript.len(),
                duration: if duration > 0.0 { Some(duration) } else { None },
                version: row.version,
                repo_file_path: row.repo_file_path,
                repo_sync_status: row.repo_sync_status,
                created_at: row.created_at,
                updated_at: row.updated_at,
            }
        })
        .collect())
}

/// Update a file by ID with optimistic locking, scoped to owner.
pub async fn update_file_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
    req: UpdateFileRequest,
) -> Result<Option<File>, RepositoryError> {
    // Get the existing file first (scoped to owner)
    let existing = get_file_for_owner(pool, id, owner_id).await?;
    let Some(existing) = existing else {
        return Ok(None);
    };

    // Check version if provided (optimistic locking)
    if let Some(expected_version) = req.version {
        if existing.version != expected_version {
            return Err(RepositoryError::VersionConflict {
                expected: expected_version,
                actual: existing.version,
            });
        }
    }

    // Apply updates
    let name = req.name.unwrap_or(existing.name);
    let description = req.description.or(existing.description);
    let transcript = req.transcript.unwrap_or(existing.transcript);
    let transcript_json = serde_json::to_value(&transcript).unwrap_or_default();
    let summary = req.summary.or(existing.summary);
    let body = req.body.unwrap_or(existing.body);
    let body_json = serde_json::to_value(&body).unwrap_or_default();

    // Update with version check in WHERE clause for race condition safety
    let result = if req.version.is_some() {
        sqlx::query_as::<_, File>(
            r#"
            UPDATE files
            SET name = $3, description = $4, transcript = $5, summary = $6, body = $7, updated_at = NOW()
            WHERE id = $1 AND owner_id = $2 AND version = $8
            RETURNING id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
            "#,
        )
        .bind(id)
        .bind(owner_id)
        .bind(&name)
        .bind(&description)
        .bind(&transcript_json)
        .bind(&summary)
        .bind(&body_json)
        .bind(req.version.unwrap())
        .fetch_optional(pool)
        .await?
    } else {
        // No version check for internal updates
        sqlx::query_as::<_, File>(
            r#"
            UPDATE files
            SET name = $3, description = $4, transcript = $5, summary = $6, body = $7, updated_at = NOW()
            WHERE id = $1 AND owner_id = $2
            RETURNING id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
            "#,
        )
        .bind(id)
        .bind(owner_id)
        .bind(&name)
        .bind(&description)
        .bind(&transcript_json)
        .bind(&summary)
        .bind(&body_json)
        .fetch_optional(pool)
        .await?
    };

    // If versioned update returned None, there was a race condition
    if result.is_none() && req.version.is_some() {
        // Re-fetch to get the actual version
        if let Some(current) = get_file_for_owner(pool, id, owner_id).await? {
            return Err(RepositoryError::VersionConflict {
                expected: req.version.unwrap(),
                actual: current.version,
            });
        }
    }

    Ok(result)
}

/// Delete a file by ID, scoped to owner.
pub async fn delete_file_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM files
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

// =============================================================================
// Version History Functions
// =============================================================================

/// Set the version source for the current transaction.
/// This is used by the trigger to record who made the change.
pub async fn set_version_source(pool: &PgPool, source: &str) -> Result<(), sqlx::Error> {
    sqlx::query(&format!("SET LOCAL app.version_source = '{}'", source))
        .execute(pool)
        .await?;
    Ok(())
}

/// Set the change description for the current transaction.
pub async fn set_change_description(pool: &PgPool, description: &str) -> Result<(), sqlx::Error> {
    // Escape single quotes for SQL
    let escaped = description.replace('\'', "''");
    sqlx::query(&format!("SET LOCAL app.change_description = '{}'", escaped))
        .execute(pool)
        .await?;
    Ok(())
}

/// List all versions of a file, ordered by version DESC.
pub async fn list_file_versions(pool: &PgPool, file_id: Uuid) -> Result<Vec<FileVersion>, sqlx::Error> {
    // First get the current version from the files table
    let current = get_file(pool, file_id).await?;

    let mut versions = sqlx::query_as::<_, FileVersion>(
        r#"
        SELECT id, file_id, version, name, description, summary, body, source, change_description, created_at
        FROM file_versions
        WHERE file_id = $1
        ORDER BY version DESC
        "#,
    )
    .bind(file_id)
    .fetch_all(pool)
    .await?;

    // Add the current version as the first entry if it exists
    if let Some(file) = current {
        let current_version = FileVersion {
            id: file.id,
            file_id: file.id,
            version: file.version,
            name: file.name,
            description: file.description,
            summary: file.summary,
            body: file.body,
            source: "user".to_string(), // Current version source
            change_description: None,
            created_at: file.updated_at,
        };
        versions.insert(0, current_version);
    }

    Ok(versions)
}

/// Get a specific version of a file.
pub async fn get_file_version(
    pool: &PgPool,
    file_id: Uuid,
    version: i32,
) -> Result<Option<FileVersion>, sqlx::Error> {
    // First check if this is the current version
    if let Some(file) = get_file(pool, file_id).await? {
        if file.version == version {
            return Ok(Some(FileVersion {
                id: file.id,
                file_id: file.id,
                version: file.version,
                name: file.name,
                description: file.description,
                summary: file.summary,
                body: file.body,
                source: "user".to_string(),
                change_description: None,
                created_at: file.updated_at,
            }));
        }
    }

    // Otherwise, look in the versions table
    sqlx::query_as::<_, FileVersion>(
        r#"
        SELECT id, file_id, version, name, description, summary, body, source, change_description, created_at
        FROM file_versions
        WHERE file_id = $1 AND version = $2
        "#,
    )
    .bind(file_id)
    .bind(version)
    .fetch_optional(pool)
    .await
}

/// Restore a file to a previous version.
/// This creates a new version with the content from the target version.
pub async fn restore_file_version(
    pool: &PgPool,
    file_id: Uuid,
    target_version: i32,
    current_version: i32,
) -> Result<Option<File>, RepositoryError> {
    // Get the target version content
    let target = get_file_version(pool, file_id, target_version).await?;
    let Some(target) = target else {
        return Ok(None);
    };

    // Set version source and description for the trigger
    set_version_source(pool, "system").await?;
    set_change_description(pool, &format!("Restored from version {}", target_version)).await?;

    // Update the file with the target version's content
    // This will trigger the save_file_version trigger to save the current state first
    let update_req = UpdateFileRequest {
        name: Some(target.name),
        description: target.description,
        transcript: None,
        summary: target.summary,
        body: Some(target.body),
        version: Some(current_version),
        repo_file_path: None,
    };

    update_file(pool, file_id, update_req).await
}

/// Count versions for a file.
pub async fn count_file_versions(pool: &PgPool, file_id: Uuid) -> Result<i64, sqlx::Error> {
    let result: (i64,) = sqlx::query_as(
        "SELECT COUNT(*) + 1 FROM file_versions WHERE file_id = $1", // +1 for current version
    )
    .bind(file_id)
    .fetch_one(pool)
    .await?;

    Ok(result.0)
}

// =============================================================================
// Task Functions
// =============================================================================

/// Create a new task.
///
/// If creating a subtask (parent_task_id is set) and repository settings are not provided,
/// the subtask will inherit repository_url, base_branch, target_branch, merge_mode,
/// and target_repo_path from the parent task. Depth is calculated from parent and limited
/// to max 1 (2 levels: orchestrator at depth 0, subtasks at depth 1).
///
/// NOTE: completion_action is NOT inherited - subtasks should not auto-merge unless
/// explicitly configured. The supervisor controls when completion steps happen.
///
/// Task spawning is now controlled by supervisors at the application level.
/// Depth is no longer constrained in the database.
pub async fn create_task(pool: &PgPool, req: CreateTaskRequest) -> Result<Task, sqlx::Error> {
    // Calculate depth and inherit settings from parent if applicable
    let (depth, contract_id, repo_url, base_branch, target_branch, merge_mode, target_repo_path, completion_action) =
        if let Some(parent_id) = req.parent_task_id {
            // Fetch parent task to get depth and inherit settings
            let parent = get_task(pool, parent_id).await?
                .ok_or_else(|| sqlx::Error::RowNotFound)?;

            let new_depth = parent.depth + 1;

            // Subtasks inherit contract_id from parent (or use request contract_id if parent has none)
            let contract_id = parent.contract_id.or(req.contract_id);

            // Inherit repo settings if not provided
            let repo_url = req.repository_url.clone().or(parent.repository_url);
            let base_branch = req.base_branch.clone().or(parent.base_branch);
            let target_branch = req.target_branch.clone().or(parent.target_branch);
            let merge_mode = req.merge_mode.clone().or(parent.merge_mode);
            let target_repo_path = req.target_repo_path.clone().or(parent.target_repo_path);
            // NOTE: completion_action is NOT inherited - subtasks should not auto-merge.
            // The supervisor integrates subtask work from their worktrees.
            let completion_action = req.completion_action.clone();

            (new_depth, contract_id, repo_url, base_branch, target_branch, merge_mode, target_repo_path, completion_action)
        } else {
            // Top-level task: depth 0, use contract_id from request (may be None for branched tasks)
            (
                0,
                req.contract_id,
                req.repository_url.clone(),
                req.base_branch.clone(),
                req.target_branch.clone(),
                req.merge_mode.clone(),
                req.target_repo_path.clone(),
                req.completion_action.clone(),
            )
        };

    let copy_files_json = req.copy_files.as_ref().map(|f| serde_json::to_value(f).unwrap_or_default());

    sqlx::query_as::<_, Task>(
        r#"
        INSERT INTO tasks (
            contract_id, parent_task_id, depth, name, description, plan, priority,
            is_supervisor, repository_url, base_branch, target_branch, merge_mode,
            target_repo_path, completion_action, continue_from_task_id, copy_files,
            branched_from_task_id, conversation_state, supervisor_worktree_task_id
        )
        VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, $14, $15, $16, $17, $18, $19)
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .bind(req.parent_task_id)
    .bind(depth)
    .bind(&req.name)
    .bind(&req.description)
    .bind(&req.plan)
    .bind(req.priority)
    .bind(req.is_supervisor)
    .bind(&repo_url)
    .bind(&base_branch)
    .bind(&target_branch)
    .bind(&merge_mode)
    .bind(&target_repo_path)
    .bind(&completion_action)
    .bind(&req.continue_from_task_id)
    .bind(&copy_files_json)
    .bind(&req.branched_from_task_id)
    .bind(&req.conversation_history)
    .bind(&req.supervisor_worktree_task_id)
    .fetch_one(pool)
    .await
}

/// Get a task by ID.
pub async fn get_task(pool: &PgPool, id: Uuid) -> Result<Option<Task>, sqlx::Error> {
    sqlx::query_as::<_, Task>(
        r#"
        SELECT *
        FROM tasks
        WHERE id = $1
        "#,
    )
    .bind(id)
    .fetch_optional(pool)
    .await
}

/// List all top-level tasks (no parent), ordered by created_at DESC.
/// Hidden tasks are excluded by default.
pub async fn list_tasks(pool: &PgPool) -> Result<Vec<TaskSummary>, sqlx::Error> {
    sqlx::query_as::<_, TaskSummary>(
        r#"
        SELECT
            t.id, t.contract_id, c.name as contract_name, c.phase as contract_phase,
            c.status as contract_status,
            t.parent_task_id, t.depth, t.name, t.status, t.priority,
            t.progress_summary,
            (SELECT COUNT(*) FROM tasks WHERE parent_task_id = t.id) as subtask_count,
            t.version, t.is_supervisor,             COALESCE(t.hidden, false) as hidden, t.created_at, t.updated_at
        FROM tasks t
        LEFT JOIN contracts c ON t.contract_id = c.id
        WHERE t.parent_task_id IS NULL AND COALESCE(t.hidden, false) = false
        ORDER BY t.priority DESC, t.created_at DESC
        "#,
    )
    .fetch_all(pool)
    .await
}

/// List subtasks of a parent task.
pub async fn list_subtasks(pool: &PgPool, parent_id: Uuid) -> Result<Vec<TaskSummary>, sqlx::Error> {
    sqlx::query_as::<_, TaskSummary>(
        r#"
        SELECT
            t.id, t.contract_id, c.name as contract_name, c.phase as contract_phase,
            c.status as contract_status,
            t.parent_task_id, t.depth, t.name, t.status, t.priority,
            t.progress_summary,
            (SELECT COUNT(*) FROM tasks WHERE parent_task_id = t.id) as subtask_count,
            t.version, t.is_supervisor,             COALESCE(t.hidden, false) as hidden, t.created_at, t.updated_at
        FROM tasks t
        LEFT JOIN contracts c ON t.contract_id = c.id
        WHERE t.parent_task_id = $1
        ORDER BY t.priority DESC, t.created_at DESC
        "#,
    )
    .bind(parent_id)
    .fetch_all(pool)
    .await
}

/// List all tasks in a contract (for supervisor tree view).
pub async fn list_tasks_by_contract(
    pool: &PgPool,
    contract_id: Uuid,
    owner_id: Uuid,
) -> Result<Vec<Task>, sqlx::Error> {
    sqlx::query_as::<_, Task>(
        r#"
        SELECT * FROM tasks
        WHERE contract_id = $1 AND owner_id = $2
        ORDER BY is_supervisor DESC, depth ASC, created_at ASC
        "#,
    )
    .bind(contract_id)
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Get pending tasks for a contract (non-supervisor tasks only).
/// Includes tasks that were interrupted (retry candidates).
/// Prioritizes interrupted tasks and excludes those that exceeded max_retries.
pub async fn get_pending_tasks_for_contract(
    pool: &PgPool,
    contract_id: Uuid,
    owner_id: Uuid,
) -> Result<Vec<Task>, sqlx::Error> {
    sqlx::query_as::<_, Task>(
        r#"
        SELECT * FROM tasks
        WHERE contract_id = $1 AND owner_id = $2
          AND status = 'pending'
          AND is_supervisor = false
          AND retry_count < max_retries
        ORDER BY
          interrupted_at DESC NULLS LAST,
          priority DESC,
          created_at ASC
        "#,
    )
    .bind(contract_id)
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Get all contracts that have pending tasks awaiting retry.
/// Returns tuples of (contract_id, owner_id) for contracts with retryable tasks.
pub async fn get_all_pending_task_contracts(
    pool: &PgPool,
) -> Result<Vec<(Uuid, Uuid)>, sqlx::Error> {
    sqlx::query_as::<_, (Uuid, Uuid)>(
        r#"
        SELECT DISTINCT contract_id, owner_id
        FROM tasks
        WHERE contract_id IS NOT NULL
          AND status = 'pending'
          AND is_supervisor = false
          AND retry_count < max_retries
        ORDER BY owner_id, contract_id
        "#,
    )
    .fetch_all(pool)
    .await
}

/// Mark a task as pending for retry after daemon failure.
/// Increments retry count and adds the failed daemon to exclusion list.
pub async fn mark_task_for_retry(
    pool: &PgPool,
    task_id: Uuid,
    failed_daemon_id: Uuid,
) -> Result<Option<Task>, sqlx::Error> {
    sqlx::query_as::<_, Task>(
        r#"
        UPDATE tasks
        SET status = 'pending',
            daemon_id = NULL,
            retry_count = retry_count + 1,
            failed_daemon_ids = array_append(COALESCE(failed_daemon_ids, '{}'), $2),
            last_active_daemon_id = $2,
            interrupted_at = NOW(),
            error_message = 'Daemon disconnected, awaiting retry',
            updated_at = NOW()
        WHERE id = $1
          AND retry_count < max_retries
        RETURNING *
        "#,
    )
    .bind(task_id)
    .bind(failed_daemon_id)
    .fetch_optional(pool)
    .await
}

/// Mark a task as permanently failed (exceeded retry limit).
pub async fn mark_task_permanently_failed(
    pool: &PgPool,
    task_id: Uuid,
    failed_daemon_id: Uuid,
) -> Result<(), sqlx::Error> {
    sqlx::query(
        r#"
        UPDATE tasks
        SET status = 'failed',
            daemon_id = NULL,
            retry_count = retry_count + 1,
            failed_daemon_ids = array_append(COALESCE(failed_daemon_ids, '{}'), $2),
            last_active_daemon_id = $2,
            error_message = 'Task failed: exceeded maximum retry attempts',
            updated_at = NOW()
        WHERE id = $1
        "#,
    )
    .bind(task_id)
    .bind(failed_daemon_id)
    .execute(pool)
    .await?;
    Ok(())
}

/// Update a task by ID with optimistic locking.
pub async fn update_task(
    pool: &PgPool,
    id: Uuid,
    req: UpdateTaskRequest,
) -> Result<Option<Task>, RepositoryError> {
    // Get the existing task first
    let existing = get_task(pool, id).await?;
    let Some(existing) = existing else {
        return Ok(None);
    };

    // Check version if provided (optimistic locking)
    if let Some(expected_version) = req.version {
        if existing.version != expected_version {
            return Err(RepositoryError::VersionConflict {
                expected: expected_version,
                actual: existing.version,
            });
        }
    }

    // Apply updates
    let name = req.name.unwrap_or(existing.name);
    let description = req.description.or(existing.description);
    let plan = req.plan.unwrap_or(existing.plan);
    let status = req.status.unwrap_or(existing.status);
    let priority = req.priority.unwrap_or(existing.priority);
    let progress_summary = req.progress_summary.or(existing.progress_summary);
    let last_output = req.last_output.or(existing.last_output);
    let error_message = req.error_message.or(existing.error_message);
    let merge_mode = req.merge_mode.or(existing.merge_mode);
    let pr_url = req.pr_url.or(existing.pr_url);
    let target_repo_path = req.target_repo_path.or(existing.target_repo_path);
    let completion_action = req.completion_action.or(existing.completion_action);
    // Handle clear_daemon_id: if true, set to NULL; otherwise use provided value or keep existing
    let daemon_id = if req.clear_daemon_id {
        None
    } else {
        req.daemon_id.or(existing.daemon_id)
    };

    // Update with version check in WHERE clause for race condition safety
    let result = if req.version.is_some() {
        sqlx::query_as::<_, Task>(
            r#"
            UPDATE tasks
            SET name = $2, description = $3, plan = $4, status = $5, priority = $6,
                progress_summary = $7, last_output = $8, error_message = $9,
                merge_mode = $10, pr_url = $11, daemon_id = $12,
                target_repo_path = $13, completion_action = $14, updated_at = NOW()
            WHERE id = $1 AND version = $15
            RETURNING *
            "#,
        )
        .bind(id)
        .bind(&name)
        .bind(&description)
        .bind(&plan)
        .bind(&status)
        .bind(priority)
        .bind(&progress_summary)
        .bind(&last_output)
        .bind(&error_message)
        .bind(&merge_mode)
        .bind(&pr_url)
        .bind(daemon_id)
        .bind(&target_repo_path)
        .bind(&completion_action)
        .bind(req.version.unwrap())
        .fetch_optional(pool)
        .await?
    } else {
        sqlx::query_as::<_, Task>(
            r#"
            UPDATE tasks
            SET name = $2, description = $3, plan = $4, status = $5, priority = $6,
                progress_summary = $7, last_output = $8, error_message = $9,
                merge_mode = $10, pr_url = $11, daemon_id = $12,
                target_repo_path = $13, completion_action = $14, updated_at = NOW()
            WHERE id = $1
            RETURNING *
            "#,
        )
        .bind(id)
        .bind(&name)
        .bind(&description)
        .bind(&plan)
        .bind(&status)
        .bind(priority)
        .bind(&progress_summary)
        .bind(&last_output)
        .bind(&error_message)
        .bind(&merge_mode)
        .bind(&pr_url)
        .bind(daemon_id)
        .bind(&target_repo_path)
        .bind(&completion_action)
        .fetch_optional(pool)
        .await?
    };

    // If versioned update returned None, there was a race condition
    if result.is_none() && req.version.is_some() {
        if let Some(current) = get_task(pool, id).await? {
            return Err(RepositoryError::VersionConflict {
                expected: req.version.unwrap(),
                actual: current.version,
            });
        }
    }

    Ok(result)
}

/// Delete a task by ID.
pub async fn delete_task(pool: &PgPool, id: Uuid) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM tasks
        WHERE id = $1
        "#,
    )
    .bind(id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Count total tasks.
pub async fn count_tasks(pool: &PgPool) -> Result<i64, sqlx::Error> {
    let result: (i64,) = sqlx::query_as(
        "SELECT COUNT(*) FROM tasks WHERE parent_task_id IS NULL",
    )
    .fetch_one(pool)
    .await?;

    Ok(result.0)
}

// =============================================================================
// Owner-Scoped Task Functions
// =============================================================================

/// Create a new task for a specific owner.
pub async fn create_task_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
    req: CreateTaskRequest,
) -> Result<Task, sqlx::Error> {
    // Calculate depth and inherit settings from parent if applicable
    let (depth, contract_id, repo_url, base_branch, target_branch, merge_mode, target_repo_path, completion_action) =
        if let Some(parent_id) = req.parent_task_id {
            // Fetch parent task to get depth and inherit settings (must belong to same owner)
            let parent = get_task_for_owner(pool, parent_id, owner_id).await?
                .ok_or_else(|| sqlx::Error::RowNotFound)?;

            let new_depth = parent.depth + 1;

            // Validate max depth
            if new_depth >= 2 {
                return Err(sqlx::Error::Protocol(format!(
                    "Maximum task depth exceeded. Cannot create subtask at depth {} (max is 1). Subtasks cannot have children.",
                    new_depth
                )));
            }

            // Subtasks inherit contract_id from parent (or use request contract_id if parent has none)
            let contract_id = parent.contract_id.or(req.contract_id);

            // Inherit repo settings if not provided
            let repo_url = req.repository_url.clone().or(parent.repository_url);
            let base_branch = req.base_branch.clone().or(parent.base_branch);
            let target_branch = req.target_branch.clone().or(parent.target_branch);
            let merge_mode = req.merge_mode.clone().or(parent.merge_mode);
            let target_repo_path = req.target_repo_path.clone().or(parent.target_repo_path);
            // NOTE: completion_action is NOT inherited - subtasks should not auto-merge.
            // The orchestrator integrates subtask work from their worktrees.
            let completion_action = req.completion_action.clone();

            (new_depth, contract_id, repo_url, base_branch, target_branch, merge_mode, target_repo_path, completion_action)
        } else {
            // Top-level task: depth 0, use contract_id from request (may be None for branched tasks)
            (
                0,
                req.contract_id,
                req.repository_url.clone(),
                req.base_branch.clone(),
                req.target_branch.clone(),
                req.merge_mode.clone(),
                req.target_repo_path.clone(),
                req.completion_action.clone(),
            )
        };

    let copy_files_json = req.copy_files.as_ref().map(|f| serde_json::to_value(f).unwrap_or_default());

    sqlx::query_as::<_, Task>(
        r#"
        INSERT INTO tasks (
            owner_id, contract_id, parent_task_id, depth, name, description, plan, priority,
            is_supervisor, repository_url, base_branch, target_branch, merge_mode,
            target_repo_path, completion_action, continue_from_task_id, copy_files,
            branched_from_task_id, conversation_state, supervisor_worktree_task_id
        )
        VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, $14, $15, $16, $17, $18, $19, $20)
        RETURNING *
        "#,
    )
    .bind(owner_id)
    .bind(contract_id)
    .bind(req.parent_task_id)
    .bind(depth)
    .bind(&req.name)
    .bind(&req.description)
    .bind(&req.plan)
    .bind(req.priority)
    .bind(req.is_supervisor)
    .bind(&repo_url)
    .bind(&base_branch)
    .bind(&target_branch)
    .bind(&merge_mode)
    .bind(&target_repo_path)
    .bind(&completion_action)
    .bind(&req.continue_from_task_id)
    .bind(&copy_files_json)
    .bind(&req.branched_from_task_id)
    .bind(&req.conversation_history)
    .bind(&req.supervisor_worktree_task_id)
    .fetch_one(pool)
    .await
}

/// Get a task by ID, scoped to owner.
pub async fn get_task_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<Option<Task>, sqlx::Error> {
    sqlx::query_as::<_, Task>(
        r#"
        SELECT *
        FROM tasks
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .fetch_optional(pool)
    .await
}

/// List all top-level tasks (no parent) for an owner, ordered by created_at DESC.
/// Hidden tasks are excluded by default.
pub async fn list_tasks_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
) -> Result<Vec<TaskSummary>, sqlx::Error> {
    sqlx::query_as::<_, TaskSummary>(
        r#"
        SELECT
            t.id, t.contract_id, c.name as contract_name, c.phase as contract_phase,
            c.status as contract_status,
            t.parent_task_id, t.depth, t.name, t.status, t.priority,
            t.progress_summary,
            (SELECT COUNT(*) FROM tasks WHERE parent_task_id = t.id) as subtask_count,
            t.version, t.is_supervisor,             COALESCE(t.hidden, false) as hidden, t.created_at, t.updated_at
        FROM tasks t
        LEFT JOIN contracts c ON t.contract_id = c.id
        WHERE t.owner_id = $1 AND t.parent_task_id IS NULL AND COALESCE(t.hidden, false) = false
        ORDER BY t.priority DESC, t.created_at DESC
        "#,
    )
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// List subtasks of a parent task, scoped to owner.
pub async fn list_subtasks_for_owner(
    pool: &PgPool,
    parent_id: Uuid,
    owner_id: Uuid,
) -> Result<Vec<TaskSummary>, sqlx::Error> {
    sqlx::query_as::<_, TaskSummary>(
        r#"
        SELECT
            t.id, t.contract_id, c.name as contract_name, c.phase as contract_phase,
            c.status as contract_status,
            t.parent_task_id, t.depth, t.name, t.status, t.priority,
            t.progress_summary,
            (SELECT COUNT(*) FROM tasks WHERE parent_task_id = t.id) as subtask_count,
            t.version, t.is_supervisor,             COALESCE(t.hidden, false) as hidden, t.created_at, t.updated_at
        FROM tasks t
        LEFT JOIN contracts c ON t.contract_id = c.id
        WHERE t.owner_id = $1 AND t.parent_task_id = $2
        ORDER BY t.priority DESC, t.created_at DESC
        "#,
    )
    .bind(owner_id)
    .bind(parent_id)
    .fetch_all(pool)
    .await
}

/// Update a task by ID with optimistic locking, scoped to owner.
pub async fn update_task_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
    req: UpdateTaskRequest,
) -> Result<Option<Task>, RepositoryError> {
    // Get the existing task first (scoped to owner)
    let existing = get_task_for_owner(pool, id, owner_id).await?;
    let Some(existing) = existing else {
        return Ok(None);
    };

    // Check version if provided (optimistic locking)
    if let Some(expected_version) = req.version {
        if existing.version != expected_version {
            return Err(RepositoryError::VersionConflict {
                expected: expected_version,
                actual: existing.version,
            });
        }
    }

    // Apply updates
    let name = req.name.unwrap_or(existing.name);
    let description = req.description.or(existing.description);
    let plan = req.plan.unwrap_or(existing.plan);
    let status = req.status.unwrap_or(existing.status);
    let priority = req.priority.unwrap_or(existing.priority);
    let progress_summary = req.progress_summary.or(existing.progress_summary);
    let last_output = req.last_output.or(existing.last_output);
    let error_message = req.error_message.or(existing.error_message);
    let merge_mode = req.merge_mode.or(existing.merge_mode);
    let pr_url = req.pr_url.or(existing.pr_url);
    let repository_url = req.repository_url.or(existing.repository_url);
    let target_repo_path = req.target_repo_path.or(existing.target_repo_path);
    let completion_action = req.completion_action.or(existing.completion_action);
    let hidden = req.hidden.unwrap_or(existing.hidden);
    let daemon_id = if req.clear_daemon_id {
        None
    } else {
        req.daemon_id.or(existing.daemon_id)
    };

    // Update with version check in WHERE clause for race condition safety
    let result = if req.version.is_some() {
        sqlx::query_as::<_, Task>(
            r#"
            UPDATE tasks
            SET name = $3, description = $4, plan = $5, status = $6, priority = $7,
                progress_summary = $8, last_output = $9, error_message = $10,
                merge_mode = $11, pr_url = $12, daemon_id = $13,
                target_repo_path = $14, completion_action = $15, repository_url = $16,
                hidden = $17, updated_at = NOW()
            WHERE id = $1 AND owner_id = $2 AND version = $18
            RETURNING *
            "#,
        )
        .bind(id)
        .bind(owner_id)
        .bind(&name)
        .bind(&description)
        .bind(&plan)
        .bind(&status)
        .bind(priority)
        .bind(&progress_summary)
        .bind(&last_output)
        .bind(&error_message)
        .bind(&merge_mode)
        .bind(&pr_url)
        .bind(daemon_id)
        .bind(&target_repo_path)
        .bind(&completion_action)
        .bind(&repository_url)
        .bind(hidden)
        .bind(req.version.unwrap())
        .fetch_optional(pool)
        .await?
    } else {
        sqlx::query_as::<_, Task>(
            r#"
            UPDATE tasks
            SET name = $3, description = $4, plan = $5, status = $6, priority = $7,
                progress_summary = $8, last_output = $9, error_message = $10,
                merge_mode = $11, pr_url = $12, daemon_id = $13,
                target_repo_path = $14, completion_action = $15, repository_url = $16,
                hidden = $17, updated_at = NOW()
            WHERE id = $1 AND owner_id = $2
            RETURNING *
            "#,
        )
        .bind(id)
        .bind(owner_id)
        .bind(&name)
        .bind(&description)
        .bind(&plan)
        .bind(&status)
        .bind(priority)
        .bind(&progress_summary)
        .bind(&last_output)
        .bind(&error_message)
        .bind(&merge_mode)
        .bind(&pr_url)
        .bind(daemon_id)
        .bind(&target_repo_path)
        .bind(&completion_action)
        .bind(&repository_url)
        .bind(hidden)
        .fetch_optional(pool)
        .await?
    };

    // If versioned update returned None, there was a race condition
    if result.is_none() && req.version.is_some() {
        if let Some(current) = get_task_for_owner(pool, id, owner_id).await? {
            return Err(RepositoryError::VersionConflict {
                expected: req.version.unwrap(),
                actual: current.version,
            });
        }
    }

    Ok(result)
}

/// Delete a task by ID, scoped to owner.
pub async fn delete_task_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM tasks
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Update task status and record event.
pub async fn update_task_status(
    pool: &PgPool,
    id: Uuid,
    new_status: &str,
    event_data: Option<serde_json::Value>,
) -> Result<Option<Task>, sqlx::Error> {
    // Get existing status
    let existing = get_task(pool, id).await?;
    let Some(existing) = existing else {
        return Ok(None);
    };

    let previous_status = existing.status.clone();

    // Update task status
    let task = sqlx::query_as::<_, Task>(
        r#"
        UPDATE tasks
        SET status = $2, updated_at = NOW(),
            started_at = CASE WHEN $2 = 'running' AND started_at IS NULL THEN NOW() ELSE started_at END,
            completed_at = CASE WHEN $2 IN ('done', 'failed', 'merged') THEN NOW() ELSE completed_at END
        WHERE id = $1
        RETURNING *
        "#,
    )
    .bind(id)
    .bind(new_status)
    .fetch_optional(pool)
    .await?;

    // Record event
    if task.is_some() {
        let _ = create_task_event(
            pool,
            id,
            "status_change",
            Some(&previous_status),
            Some(new_status),
            event_data,
        )
        .await;
    }

    Ok(task)
}

// =============================================================================
// Task Event Functions
// =============================================================================

/// Create a task event.
pub async fn create_task_event(
    pool: &PgPool,
    task_id: Uuid,
    event_type: &str,
    previous_status: Option<&str>,
    new_status: Option<&str>,
    event_data: Option<serde_json::Value>,
) -> Result<TaskEvent, sqlx::Error> {
    sqlx::query_as::<_, TaskEvent>(
        r#"
        INSERT INTO task_events (task_id, event_type, previous_status, new_status, event_data)
        VALUES ($1, $2, $3, $4, $5)
        RETURNING *
        "#,
    )
    .bind(task_id)
    .bind(event_type)
    .bind(previous_status)
    .bind(new_status)
    .bind(event_data)
    .fetch_one(pool)
    .await
}

/// List events for a task.
pub async fn list_task_events(
    pool: &PgPool,
    task_id: Uuid,
    limit: Option<i64>,
) -> Result<Vec<TaskEvent>, sqlx::Error> {
    let limit = limit.unwrap_or(100);
    sqlx::query_as::<_, TaskEvent>(
        r#"
        SELECT *
        FROM task_events
        WHERE task_id = $1
        ORDER BY created_at DESC
        LIMIT $2
        "#,
    )
    .bind(task_id)
    .bind(limit)
    .fetch_all(pool)
    .await
}

// =============================================================================
// Daemon Functions
// =============================================================================

/// Register a new daemon connection.
pub async fn register_daemon(
    pool: &PgPool,
    owner_id: Uuid,
    connection_id: &str,
    hostname: Option<&str>,
    machine_id: Option<&str>,
    max_concurrent_tasks: i32,
) -> Result<Daemon, sqlx::Error> {
    sqlx::query_as::<_, Daemon>(
        r#"
        INSERT INTO daemons (owner_id, connection_id, hostname, machine_id, max_concurrent_tasks)
        VALUES ($1, $2, $3, $4, $5)
        RETURNING *
        "#,
    )
    .bind(owner_id)
    .bind(connection_id)
    .bind(hostname)
    .bind(machine_id)
    .bind(max_concurrent_tasks)
    .fetch_one(pool)
    .await
}

/// Get a daemon by ID.
pub async fn get_daemon(pool: &PgPool, id: Uuid) -> Result<Option<Daemon>, sqlx::Error> {
    sqlx::query_as::<_, Daemon>(
        r#"
        SELECT *
        FROM daemons
        WHERE id = $1
        "#,
    )
    .bind(id)
    .fetch_optional(pool)
    .await
}

/// Get a daemon by connection ID.
pub async fn get_daemon_by_connection(
    pool: &PgPool,
    connection_id: &str,
) -> Result<Option<Daemon>, sqlx::Error> {
    sqlx::query_as::<_, Daemon>(
        r#"
        SELECT *
        FROM daemons
        WHERE connection_id = $1
        "#,
    )
    .bind(connection_id)
    .fetch_optional(pool)
    .await
}

/// List all daemons.
pub async fn list_daemons(pool: &PgPool) -> Result<Vec<Daemon>, sqlx::Error> {
    sqlx::query_as::<_, Daemon>(
        r#"
        SELECT *
        FROM daemons
        ORDER BY connected_at DESC
        "#,
    )
    .fetch_all(pool)
    .await
}

/// List daemons for a specific owner.
pub async fn list_daemons_for_owner(pool: &PgPool, owner_id: Uuid) -> Result<Vec<Daemon>, sqlx::Error> {
    sqlx::query_as::<_, Daemon>(
        r#"
        SELECT *
        FROM daemons
        WHERE owner_id = $1
        ORDER BY connected_at DESC
        "#,
    )
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Get a daemon by ID for a specific owner.
pub async fn get_daemon_for_owner(pool: &PgPool, id: Uuid, owner_id: Uuid) -> Result<Option<Daemon>, sqlx::Error> {
    sqlx::query_as::<_, Daemon>(
        r#"
        SELECT *
        FROM daemons
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .fetch_optional(pool)
    .await
}

/// Update daemon heartbeat.
pub async fn update_daemon_heartbeat(pool: &PgPool, id: Uuid) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        UPDATE daemons
        SET last_heartbeat_at = NOW(), status = 'connected'
        WHERE id = $1
        "#,
    )
    .bind(id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Update daemon status.
pub async fn update_daemon_status(
    pool: &PgPool,
    id: Uuid,
    status: &str,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        UPDATE daemons
        SET status = $2,
            disconnected_at = CASE WHEN $2 = 'disconnected' THEN NOW() ELSE disconnected_at END
        WHERE id = $1
        "#,
    )
    .bind(id)
    .bind(status)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Mark daemon as disconnected by connection_id.
pub async fn disconnect_daemon_by_connection(
    pool: &PgPool,
    connection_id: &str,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        UPDATE daemons
        SET status = 'disconnected',
            disconnected_at = NOW()
        WHERE connection_id = $1
        "#,
    )
    .bind(connection_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Update daemon task count.
pub async fn update_daemon_task_count(
    pool: &PgPool,
    id: Uuid,
    delta: i32,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        UPDATE daemons
        SET current_task_count = GREATEST(0, current_task_count + $2)
        WHERE id = $1
        "#,
    )
    .bind(id)
    .bind(delta)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Delete a daemon by ID.
pub async fn delete_daemon(pool: &PgPool, id: Uuid) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM daemons
        WHERE id = $1
        "#,
    )
    .bind(id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Delete a daemon by connection ID.
pub async fn delete_daemon_by_connection(
    pool: &PgPool,
    connection_id: &str,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM daemons
        WHERE connection_id = $1
        "#,
    )
    .bind(connection_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Count connected daemons.
pub async fn count_daemons(pool: &PgPool) -> Result<i64, sqlx::Error> {
    let result: (i64,) = sqlx::query_as(
        "SELECT COUNT(*) FROM daemons WHERE status = 'connected'",
    )
    .fetch_one(pool)
    .await?;

    Ok(result.0)
}

/// Delete stale daemons that haven't sent a heartbeat within the timeout.
/// Returns the number of deleted daemons.
pub async fn delete_stale_daemons(
    pool: &PgPool,
    timeout_seconds: i64,
) -> Result<u64, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM daemons
        WHERE last_heartbeat_at < NOW() - INTERVAL '1 second' * $1
        "#,
    )
    .bind(timeout_seconds)
    .execute(pool)
    .await?;

    Ok(result.rows_affected())
}

// =============================================================================
// Sibling Awareness Functions
// =============================================================================

/// List sibling tasks (tasks with the same parent, excluding the given task).
pub async fn list_sibling_tasks(
    pool: &PgPool,
    task_id: Uuid,
    parent_id: Option<Uuid>,
) -> Result<Vec<TaskSummary>, sqlx::Error> {
    match parent_id {
        Some(parent) => {
            sqlx::query_as::<_, TaskSummary>(
                r#"
                SELECT
                    t.id, t.contract_id, c.name as contract_name, c.phase as contract_phase,
                    c.status as contract_status,
                    t.parent_task_id, t.depth, t.name, t.status, t.priority,
                    t.progress_summary,
                    (SELECT COUNT(*) FROM tasks WHERE parent_task_id = t.id) as subtask_count,
                    t.version, t.is_supervisor,                     COALESCE(t.hidden, false) as hidden, t.created_at, t.updated_at
                FROM tasks t
                LEFT JOIN contracts c ON t.contract_id = c.id
                WHERE t.parent_task_id = $1 AND t.id != $2
                ORDER BY t.priority DESC, t.created_at DESC
                "#,
            )
            .bind(parent)
            .bind(task_id)
            .fetch_all(pool)
            .await
        }
        None => {
            // Top-level tasks (no parent) - siblings are other top-level tasks
            sqlx::query_as::<_, TaskSummary>(
                r#"
                SELECT
                    t.id, t.contract_id, c.name as contract_name, c.phase as contract_phase,
                    c.status as contract_status,
                    t.parent_task_id, t.depth, t.name, t.status, t.priority,
                    t.progress_summary,
                    (SELECT COUNT(*) FROM tasks WHERE parent_task_id = t.id) as subtask_count,
                    t.version, t.is_supervisor,                     COALESCE(t.hidden, false) as hidden, t.created_at, t.updated_at
                FROM tasks t
                LEFT JOIN contracts c ON t.contract_id = c.id
                WHERE t.parent_task_id IS NULL AND t.id != $1
                ORDER BY t.priority DESC, t.created_at DESC
                "#,
            )
            .bind(task_id)
            .fetch_all(pool)
            .await
        }
    }
}

/// Get running sibling tasks (for context injection).
pub async fn get_running_siblings(
    pool: &PgPool,
    owner_id: Uuid,
    task_id: Uuid,
    parent_id: Option<Uuid>,
) -> Result<Vec<Task>, sqlx::Error> {
    match parent_id {
        Some(parent) => {
            sqlx::query_as::<_, Task>(
                r#"
                SELECT *
                FROM tasks t
                WHERE t.owner_id = $1
                    AND t.parent_task_id = $2
                    AND t.id != $3
                    AND t.status = 'running'
                ORDER BY t.priority DESC
                "#,
            )
            .bind(owner_id)
            .bind(parent)
            .bind(task_id)
            .fetch_all(pool)
            .await
        }
        None => {
            sqlx::query_as::<_, Task>(
                r#"
                SELECT *
                FROM tasks t
                WHERE t.owner_id = $1
                    AND t.parent_task_id IS NULL
                    AND t.id != $2
                    AND t.status = 'running'
                ORDER BY t.priority DESC
                "#,
            )
            .bind(owner_id)
            .bind(task_id)
            .fetch_all(pool)
            .await
        }
    }
}

/// Get task with its siblings for context awareness.
pub async fn get_task_with_siblings(
    pool: &PgPool,
    id: Uuid,
) -> Result<Option<(Task, Vec<TaskSummary>)>, sqlx::Error> {
    let task = get_task(pool, id).await?;
    let Some(task) = task else {
        return Ok(None);
    };

    let siblings = list_sibling_tasks(pool, id, task.parent_task_id).await?;
    Ok(Some((task, siblings)))
}

// =============================================================================
// Task Output Persistence Functions
// =============================================================================

/// Save task output to the database.
/// This stores output in the task_events table with event_type='output'.
pub async fn save_task_output(
    pool: &PgPool,
    task_id: Uuid,
    message_type: &str,
    content: &str,
    tool_name: Option<&str>,
    tool_input: Option<serde_json::Value>,
    is_error: Option<bool>,
    cost_usd: Option<f64>,
    duration_ms: Option<u64>,
) -> Result<TaskEvent, sqlx::Error> {
    let event_data = serde_json::json!({
        "messageType": message_type,
        "content": content,
        "toolName": tool_name,
        "toolInput": tool_input,
        "isError": is_error,
        "costUsd": cost_usd,
        "durationMs": duration_ms,
    });

    create_task_event(pool, task_id, "output", None, None, Some(event_data)).await
}

/// Get task output from the database.
/// Retrieves all output events for a task, ordered by creation time.
pub async fn get_task_output(
    pool: &PgPool,
    task_id: Uuid,
    limit: Option<i64>,
) -> Result<Vec<TaskEvent>, sqlx::Error> {
    let limit = limit.unwrap_or(1000);
    sqlx::query_as::<_, TaskEvent>(
        r#"
        SELECT *
        FROM task_events
        WHERE task_id = $1 AND event_type = 'output'
        ORDER BY created_at ASC
        LIMIT $2
        "#,
    )
    .bind(task_id)
    .bind(limit)
    .fetch_all(pool)
    .await
}

/// Update task completion status with error message.
/// Sets the task status to 'done' or 'failed' and records completion time.
pub async fn complete_task(
    pool: &PgPool,
    task_id: Uuid,
    success: bool,
    error_message: Option<&str>,
) -> Result<Option<Task>, sqlx::Error> {
    let status = if success { "done" } else { "failed" };

    let task = sqlx::query_as::<_, Task>(
        r#"
        UPDATE tasks
        SET status = $2,
            error_message = COALESCE($3, error_message),
            completed_at = NOW(),
            updated_at = NOW()
        WHERE id = $1
        RETURNING *
        "#,
    )
    .bind(task_id)
    .bind(status)
    .bind(error_message)
    .fetch_optional(pool)
    .await?;

    // Record completion event
    if task.is_some() {
        let event_data = serde_json::json!({
            "success": success,
            "errorMessage": error_message,
        });
        let _ = create_task_event(
            pool,
            task_id,
            "complete",
            Some("running"),
            Some(status),
            Some(event_data),
        )
        .await;
    }

    Ok(task)
}

// =============================================================================
// Mesh Chat History Functions
// =============================================================================

/// Get or create the active conversation for an owner.
pub async fn get_or_create_active_conversation(
    pool: &PgPool,
    owner_id: Uuid,
) -> Result<MeshChatConversation, sqlx::Error> {
    // Try to get existing active conversation for this owner
    let existing = sqlx::query_as::<_, MeshChatConversation>(
        r#"
        SELECT *
        FROM mesh_chat_conversations
        WHERE is_active = true AND owner_id = $1
        LIMIT 1
        "#,
    )
    .bind(owner_id)
    .fetch_optional(pool)
    .await?;

    if let Some(conv) = existing {
        return Ok(conv);
    }

    // Create new conversation
    sqlx::query_as::<_, MeshChatConversation>(
        r#"
        INSERT INTO mesh_chat_conversations (owner_id, is_active)
        VALUES ($1, true)
        RETURNING *
        "#,
    )
    .bind(owner_id)
    .fetch_one(pool)
    .await
}

/// List messages for a conversation.
pub async fn list_chat_messages(
    pool: &PgPool,
    conversation_id: Uuid,
    limit: Option<i32>,
) -> Result<Vec<MeshChatMessageRecord>, sqlx::Error> {
    let limit = limit.unwrap_or(100);
    sqlx::query_as::<_, MeshChatMessageRecord>(
        r#"
        SELECT *
        FROM mesh_chat_messages
        WHERE conversation_id = $1
        ORDER BY created_at ASC
        LIMIT $2
        "#,
    )
    .bind(conversation_id)
    .bind(limit)
    .fetch_all(pool)
    .await
}

/// Add a message to a conversation.
#[allow(clippy::too_many_arguments)]
pub async fn add_chat_message(
    pool: &PgPool,
    conversation_id: Uuid,
    role: &str,
    content: &str,
    context_type: &str,
    context_task_id: Option<Uuid>,
    tool_calls: Option<serde_json::Value>,
    pending_questions: Option<serde_json::Value>,
) -> Result<MeshChatMessageRecord, sqlx::Error> {
    sqlx::query_as::<_, MeshChatMessageRecord>(
        r#"
        INSERT INTO mesh_chat_messages
        (conversation_id, role, content, context_type, context_task_id, tool_calls, pending_questions)
        VALUES ($1, $2, $3, $4, $5, $6, $7)
        RETURNING *
        "#,
    )
    .bind(conversation_id)
    .bind(role)
    .bind(content)
    .bind(context_type)
    .bind(context_task_id)
    .bind(tool_calls)
    .bind(pending_questions)
    .fetch_one(pool)
    .await
}

/// Clear conversation (archive existing and create new).
pub async fn clear_conversation(pool: &PgPool, owner_id: Uuid) -> Result<MeshChatConversation, sqlx::Error> {
    // Mark existing as inactive for this owner
    sqlx::query(
        r#"
        UPDATE mesh_chat_conversations
        SET is_active = false, updated_at = NOW()
        WHERE is_active = true AND owner_id = $1
        "#,
    )
    .bind(owner_id)
    .execute(pool)
    .await?;

    // Create new active conversation
    get_or_create_active_conversation(pool, owner_id).await
}

// =============================================================================
// Contract Chat History Functions
// =============================================================================

/// Get or create the active conversation for a contract.
pub async fn get_or_create_contract_conversation(
    pool: &PgPool,
    contract_id: Uuid,
    owner_id: Uuid,
) -> Result<ContractChatConversation, sqlx::Error> {
    // Try to get existing active conversation for this contract
    let existing = sqlx::query_as::<_, ContractChatConversation>(
        r#"
        SELECT *
        FROM contract_chat_conversations
        WHERE is_active = true AND contract_id = $1 AND owner_id = $2
        LIMIT 1
        "#,
    )
    .bind(contract_id)
    .bind(owner_id)
    .fetch_optional(pool)
    .await?;

    if let Some(conv) = existing {
        return Ok(conv);
    }

    // Create new conversation
    sqlx::query_as::<_, ContractChatConversation>(
        r#"
        INSERT INTO contract_chat_conversations (contract_id, owner_id, is_active)
        VALUES ($1, $2, true)
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .bind(owner_id)
    .fetch_one(pool)
    .await
}

/// List messages for a contract conversation.
pub async fn list_contract_chat_messages(
    pool: &PgPool,
    conversation_id: Uuid,
    limit: Option<i32>,
) -> Result<Vec<ContractChatMessageRecord>, sqlx::Error> {
    let limit = limit.unwrap_or(100);
    sqlx::query_as::<_, ContractChatMessageRecord>(
        r#"
        SELECT *
        FROM contract_chat_messages
        WHERE conversation_id = $1
        ORDER BY created_at ASC
        LIMIT $2
        "#,
    )
    .bind(conversation_id)
    .bind(limit)
    .fetch_all(pool)
    .await
}

/// Add a message to a contract conversation.
pub async fn add_contract_chat_message(
    pool: &PgPool,
    conversation_id: Uuid,
    role: &str,
    content: &str,
    tool_calls: Option<serde_json::Value>,
    pending_questions: Option<serde_json::Value>,
) -> Result<ContractChatMessageRecord, sqlx::Error> {
    sqlx::query_as::<_, ContractChatMessageRecord>(
        r#"
        INSERT INTO contract_chat_messages
        (conversation_id, role, content, tool_calls, pending_questions)
        VALUES ($1, $2, $3, $4, $5)
        RETURNING *
        "#,
    )
    .bind(conversation_id)
    .bind(role)
    .bind(content)
    .bind(tool_calls)
    .bind(pending_questions)
    .fetch_one(pool)
    .await
}

/// Clear contract conversation (archive existing and create new).
pub async fn clear_contract_conversation(
    pool: &PgPool,
    contract_id: Uuid,
    owner_id: Uuid,
) -> Result<ContractChatConversation, sqlx::Error> {
    // Mark existing as inactive for this contract
    sqlx::query(
        r#"
        UPDATE contract_chat_conversations
        SET is_active = false, updated_at = NOW()
        WHERE is_active = true AND contract_id = $1 AND owner_id = $2
        "#,
    )
    .bind(contract_id)
    .bind(owner_id)
    .execute(pool)
    .await?;

    // Create new active conversation
    get_or_create_contract_conversation(pool, contract_id, owner_id).await
}

// =============================================================================
// Contract Type Template Functions (Owner-Scoped)
// =============================================================================

/// Create a new contract type template for a specific owner.
pub async fn create_template_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
    req: CreateTemplateRequest,
) -> Result<ContractTypeTemplateRecord, sqlx::Error> {
    sqlx::query_as::<_, ContractTypeTemplateRecord>(
        r#"
        INSERT INTO contract_type_templates (owner_id, name, description, phases, default_phase, deliverables)
        VALUES ($1, $2, $3, $4, $5, $6)
        RETURNING *
        "#,
    )
    .bind(owner_id)
    .bind(&req.name)
    .bind(&req.description)
    .bind(serde_json::to_value(&req.phases).unwrap_or_default())
    .bind(&req.default_phase)
    .bind(match &req.deliverables {
        Some(d) => serde_json::to_value(d).ok(),
        None => None,
    })
    .fetch_one(pool)
    .await
}

/// Get a contract type template by ID, scoped to owner.
pub async fn get_template_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<Option<ContractTypeTemplateRecord>, sqlx::Error> {
    sqlx::query_as::<_, ContractTypeTemplateRecord>(
        r#"
        SELECT *
        FROM contract_type_templates
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .fetch_optional(pool)
    .await
}

/// Get a contract type template by ID (internal use, no owner scoping).
pub async fn get_template_by_id(
    pool: &PgPool,
    id: Uuid,
) -> Result<Option<ContractTypeTemplateRecord>, sqlx::Error> {
    sqlx::query_as::<_, ContractTypeTemplateRecord>(
        r#"
        SELECT *
        FROM contract_type_templates
        WHERE id = $1
        "#,
    )
    .bind(id)
    .fetch_optional(pool)
    .await
}

/// List all contract type templates for an owner, ordered by name.
pub async fn list_templates_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
) -> Result<Vec<ContractTypeTemplateRecord>, sqlx::Error> {
    sqlx::query_as::<_, ContractTypeTemplateRecord>(
        r#"
        SELECT *
        FROM contract_type_templates
        WHERE owner_id = $1
        ORDER BY name ASC
        "#,
    )
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Update a contract type template for an owner.
pub async fn update_template_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
    req: UpdateTemplateRequest,
) -> Result<Option<ContractTypeTemplateRecord>, RepositoryError> {
    // Build dynamic update query
    let mut query = String::from("UPDATE contract_type_templates SET updated_at = NOW()");
    let mut param_idx = 3; // $1 = id, $2 = owner_id

    if req.name.is_some() {
        query.push_str(&format!(", name = ${}", param_idx));
        param_idx += 1;
    }
    if req.description.is_some() {
        query.push_str(&format!(", description = ${}", param_idx));
        param_idx += 1;
    }
    if req.phases.is_some() {
        query.push_str(&format!(", phases = ${}", param_idx));
        param_idx += 1;
    }
    if req.default_phase.is_some() {
        query.push_str(&format!(", default_phase = ${}", param_idx));
        param_idx += 1;
    }
    if req.deliverables.is_some() {
        query.push_str(&format!(", deliverables = ${}", param_idx));
        param_idx += 1;
    }

    // Optimistic locking
    if req.version.is_some() {
        query.push_str(&format!(", version = version + 1 WHERE id = $1 AND owner_id = $2 AND version = ${}", param_idx));
    } else {
        query.push_str(", version = version + 1 WHERE id = $1 AND owner_id = $2");
    }
    query.push_str(" RETURNING *");

    let mut sql_query = sqlx::query_as::<_, ContractTypeTemplateRecord>(&query);
    sql_query = sql_query.bind(id).bind(owner_id);

    if let Some(ref name) = req.name {
        sql_query = sql_query.bind(name);
    }
    if let Some(ref description) = req.description {
        sql_query = sql_query.bind(description);
    }
    if let Some(ref phases) = req.phases {
        sql_query = sql_query.bind(serde_json::to_value(phases).unwrap_or_default());
    }
    if let Some(ref default_phase) = req.default_phase {
        sql_query = sql_query.bind(default_phase);
    }
    if let Some(ref deliverables) = req.deliverables {
        sql_query = sql_query.bind(serde_json::to_value(deliverables).unwrap_or_default());
    }
    if let Some(version) = req.version {
        sql_query = sql_query.bind(version);
    }

    match sql_query.fetch_optional(pool).await {
        Ok(result) => {
            if result.is_none() && req.version.is_some() {
                // Check if it's a version conflict
                if let Some(current) = get_template_for_owner(pool, id, owner_id).await? {
                    return Err(RepositoryError::VersionConflict {
                        expected: req.version.unwrap(),
                        actual: current.version,
                    });
                }
            }
            Ok(result)
        }
        Err(e) => Err(RepositoryError::Database(e)),
    }
}

/// Delete a contract type template for an owner.
pub async fn delete_template_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM contract_type_templates
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Helper function to build PhaseConfig from a template.
pub fn build_phase_config_from_template(template: &ContractTypeTemplateRecord) -> PhaseConfig {
    PhaseConfig {
        phases: template.phases.clone(),
        default_phase: template.default_phase.clone(),
        deliverables: template.deliverables.clone().unwrap_or_default(),
    }
}

/// Helper function to build PhaseConfig for built-in contract types.
pub fn build_phase_config_for_builtin(contract_type: &str) -> PhaseConfig {
    match contract_type {
        "simple" => PhaseConfig {
            phases: vec![
                PhaseDefinition { id: "plan".to_string(), name: "Plan".to_string(), order: 0 },
                PhaseDefinition { id: "execute".to_string(), name: "Execute".to_string(), order: 1 },
            ],
            default_phase: "plan".to_string(),
            deliverables: [
                ("plan".to_string(), vec![DeliverableDefinition {
                    id: "plan-document".to_string(),
                    name: "Plan".to_string(),
                    priority: "required".to_string(),
                }]),
                ("execute".to_string(), vec![DeliverableDefinition {
                    id: "pull-request".to_string(),
                    name: "Pull Request".to_string(),
                    priority: "required".to_string(),
                }]),
            ].into_iter().collect(),
        },
        "specification" => PhaseConfig {
            phases: vec![
                PhaseDefinition { id: "research".to_string(), name: "Research".to_string(), order: 0 },
                PhaseDefinition { id: "specify".to_string(), name: "Specify".to_string(), order: 1 },
                PhaseDefinition { id: "plan".to_string(), name: "Plan".to_string(), order: 2 },
                PhaseDefinition { id: "execute".to_string(), name: "Execute".to_string(), order: 3 },
                PhaseDefinition { id: "review".to_string(), name: "Review".to_string(), order: 4 },
            ],
            default_phase: "research".to_string(),
            deliverables: [
                ("research".to_string(), vec![DeliverableDefinition {
                    id: "research-notes".to_string(),
                    name: "Research Notes".to_string(),
                    priority: "required".to_string(),
                }]),
                ("specify".to_string(), vec![DeliverableDefinition {
                    id: "requirements-document".to_string(),
                    name: "Requirements Document".to_string(),
                    priority: "required".to_string(),
                }]),
                ("plan".to_string(), vec![DeliverableDefinition {
                    id: "plan-document".to_string(),
                    name: "Plan".to_string(),
                    priority: "required".to_string(),
                }]),
                ("execute".to_string(), vec![DeliverableDefinition {
                    id: "pull-request".to_string(),
                    name: "Pull Request".to_string(),
                    priority: "required".to_string(),
                }]),
                ("review".to_string(), vec![DeliverableDefinition {
                    id: "release-notes".to_string(),
                    name: "Release Notes".to_string(),
                    priority: "required".to_string(),
                }]),
            ].into_iter().collect(),
        },
        "execute" | _ => PhaseConfig {
            phases: vec![
                PhaseDefinition { id: "execute".to_string(), name: "Execute".to_string(), order: 0 },
            ],
            default_phase: "execute".to_string(),
            deliverables: std::collections::HashMap::new(),
        },
    }
}

// =============================================================================
// Contract Functions (Owner-Scoped)
// =============================================================================

/// Create a new contract for a specific owner.
/// Supports both built-in contract types (simple, specification, execute) and custom templates.
pub async fn create_contract_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
    req: CreateContractRequest,
) -> Result<Contract, sqlx::Error> {
    // Determine phase configuration based on template_id or contract_type
    let (phase_config, contract_type_str, default_phase): (PhaseConfig, String, String) =
        if let Some(template_id) = req.template_id {
            // Look up the custom template
            let template = get_template_by_id(pool, template_id)
                .await?
                .ok_or_else(|| {
                    sqlx::Error::Protocol(format!("Template not found: {}", template_id))
                })?;

            let config = build_phase_config_from_template(&template);
            let default = config.default_phase.clone();
            // For custom templates, store the template name as the contract_type
            (config, template.name.clone(), default)
        } else {
            // Use built-in contract type
            let contract_type = req.contract_type.as_deref().unwrap_or("simple");

            // Validate contract type
            let valid_types = ["simple", "specification", "execute"];
            if !valid_types.contains(&contract_type) {
                return Err(sqlx::Error::Protocol(format!(
                    "Invalid contract_type '{}'. Must be one of: {} or provide a template_id",
                    contract_type,
                    valid_types.join(", ")
                )));
            }

            let config = build_phase_config_for_builtin(contract_type);
            let default = config.default_phase.clone();
            (config, contract_type.to_string(), default)
        };

    // Get valid phase IDs from the configuration
    let valid_phase_ids: Vec<String> = phase_config.phases.iter().map(|p| p.id.clone()).collect();

    // Use provided initial_phase or default based on contract type/template
    let phase = req.initial_phase.as_deref().unwrap_or(&default_phase);

    // Validate the phase is valid for this contract type/template
    if !valid_phase_ids.contains(&phase.to_string()) {
        return Err(sqlx::Error::Protocol(format!(
            "Invalid initial_phase '{}' for contract type '{}'. Must be one of: {}",
            phase,
            contract_type_str,
            valid_phase_ids.join(", ")
        )));
    }

    let autonomous_loop = req.autonomous_loop.unwrap_or(false);
    let phase_guard = req.phase_guard.unwrap_or(false);
    let local_only = req.local_only.unwrap_or(false);
    let auto_merge_local = req.auto_merge_local.unwrap_or(false);

    // Serialize phase_config to JSON
    let phase_config_json = serde_json::to_value(&phase_config).ok();

    sqlx::query_as::<_, Contract>(
        r#"
        INSERT INTO contracts (owner_id, name, description, contract_type, phase, autonomous_loop, phase_guard, local_only, auto_merge_local, phase_config)
        VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10)
        RETURNING *
        "#,
    )
    .bind(owner_id)
    .bind(&req.name)
    .bind(&req.description)
    .bind(&contract_type_str)
    .bind(phase)
    .bind(autonomous_loop)
    .bind(phase_guard)
    .bind(local_only)
    .bind(auto_merge_local)
    .bind(phase_config_json)
    .fetch_one(pool)
    .await
}

/// Get a contract by ID, scoped to owner.
pub async fn get_contract_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<Option<Contract>, sqlx::Error> {
    sqlx::query_as::<_, Contract>(
        r#"
        SELECT *
        FROM contracts
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .fetch_optional(pool)
    .await
}

/// List all contracts for an owner, ordered by created_at DESC.
pub async fn list_contracts_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
) -> Result<Vec<ContractSummary>, sqlx::Error> {
    sqlx::query_as::<_, ContractSummary>(
        r#"
        SELECT
            c.id, c.name, c.description, c.contract_type, c.phase, c.status,
            c.supervisor_task_id, c.local_only, c.auto_merge_local, c.version, c.created_at,
            (SELECT COUNT(*) FROM files WHERE contract_id = c.id) as file_count,
            (SELECT COUNT(*) FROM tasks WHERE contract_id = c.id) as task_count,
            (SELECT COUNT(*) FROM contract_repositories WHERE contract_id = c.id) as repository_count
        FROM contracts c
        WHERE c.owner_id = $1
        ORDER BY c.created_at DESC
        "#,
    )
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Get contract summary by ID.
pub async fn get_contract_summary_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<Option<ContractSummary>, sqlx::Error> {
    sqlx::query_as::<_, ContractSummary>(
        r#"
        SELECT
            c.id, c.name, c.description, c.contract_type, c.phase, c.status,
            c.supervisor_task_id, c.local_only, c.auto_merge_local, c.version, c.created_at,
            (SELECT COUNT(*) FROM files WHERE contract_id = c.id) as file_count,
            (SELECT COUNT(*) FROM tasks WHERE contract_id = c.id) as task_count,
            (SELECT COUNT(*) FROM contract_repositories WHERE contract_id = c.id) as repository_count
        FROM contracts c
        WHERE c.id = $1 AND c.owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .fetch_optional(pool)
    .await
}

/// Update a contract by ID with optimistic locking, scoped to owner.
pub async fn update_contract_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
    req: UpdateContractRequest,
) -> Result<Option<Contract>, RepositoryError> {
    let existing = get_contract_for_owner(pool, id, owner_id).await?;
    let Some(existing) = existing else {
        return Ok(None);
    };

    // Check version if provided (optimistic locking)
    if let Some(expected_version) = req.version {
        if existing.version != expected_version {
            return Err(RepositoryError::VersionConflict {
                expected: expected_version,
                actual: existing.version,
            });
        }
    }

    // Apply updates
    let name = req.name.unwrap_or(existing.name);
    let description = req.description.or(existing.description);
    let phase = req.phase.unwrap_or(existing.phase);
    let status = req.status.unwrap_or(existing.status);
    let supervisor_task_id = req.supervisor_task_id.or(existing.supervisor_task_id);
    let autonomous_loop = req.autonomous_loop.unwrap_or(existing.autonomous_loop);
    let phase_guard = req.phase_guard.unwrap_or(existing.phase_guard);
    let local_only = req.local_only.unwrap_or(existing.local_only);
    let auto_merge_local = req.auto_merge_local.unwrap_or(existing.auto_merge_local);

    let result = if req.version.is_some() {
        sqlx::query_as::<_, Contract>(
            r#"
            UPDATE contracts
            SET name = $3, description = $4, phase = $5, status = $6,
                supervisor_task_id = $7, autonomous_loop = $8, phase_guard = $9, local_only = $10, auto_merge_local = $11, version = version + 1, updated_at = NOW()
            WHERE id = $1 AND owner_id = $2 AND version = $12
            RETURNING *
            "#,
        )
        .bind(id)
        .bind(owner_id)
        .bind(&name)
        .bind(&description)
        .bind(&phase)
        .bind(&status)
        .bind(supervisor_task_id)
        .bind(autonomous_loop)
        .bind(phase_guard)
        .bind(local_only)
        .bind(auto_merge_local)
        .bind(req.version.unwrap())
        .fetch_optional(pool)
        .await?
    } else {
        sqlx::query_as::<_, Contract>(
            r#"
            UPDATE contracts
            SET name = $3, description = $4, phase = $5, status = $6,
                supervisor_task_id = $7, autonomous_loop = $8, phase_guard = $9, local_only = $10, auto_merge_local = $11, version = version + 1, updated_at = NOW()
            WHERE id = $1 AND owner_id = $2
            RETURNING *
            "#,
        )
        .bind(id)
        .bind(owner_id)
        .bind(&name)
        .bind(&description)
        .bind(&phase)
        .bind(&status)
        .bind(supervisor_task_id)
        .bind(autonomous_loop)
        .bind(phase_guard)
        .bind(local_only)
        .bind(auto_merge_local)
        .fetch_optional(pool)
        .await?
    };

    // If versioned update returned None, there was a race condition
    if result.is_none() && req.version.is_some() {
        if let Some(current) = get_contract_for_owner(pool, id, owner_id).await? {
            return Err(RepositoryError::VersionConflict {
                expected: req.version.unwrap(),
                actual: current.version,
            });
        }
    }

    Ok(result)
}

/// Delete a contract by ID, scoped to owner.
pub async fn delete_contract_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM contracts
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Change contract phase and record event.
///
/// This is the simple version without version checking. Use `change_contract_phase_with_version`
/// for explicit version conflict detection.
pub async fn change_contract_phase_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
    new_phase: &str,
) -> Result<Option<Contract>, sqlx::Error> {
    // Get current phase
    let existing = get_contract_for_owner(pool, id, owner_id).await?;
    let Some(existing) = existing else {
        return Ok(None);
    };

    let previous_phase = existing.phase.clone();

    // Update phase
    let contract = sqlx::query_as::<_, Contract>(
        r#"
        UPDATE contracts
        SET phase = $3, version = version + 1, updated_at = NOW()
        WHERE id = $1 AND owner_id = $2
        RETURNING *
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .bind(new_phase)
    .fetch_optional(pool)
    .await?;

    // Record event
    if contract.is_some() {
        sqlx::query(
            r#"
            INSERT INTO contract_events (contract_id, event_type, previous_phase, new_phase)
            VALUES ($1, 'phase_change', $2, $3)
            "#,
        )
        .bind(id)
        .bind(&previous_phase)
        .bind(new_phase)
        .execute(pool)
        .await?;
    }

    Ok(contract)
}

/// Change contract phase with explicit version checking for conflict detection.
///
/// Uses `SELECT ... FOR UPDATE` to lock the row and prevent race conditions.
/// Returns `PhaseChangeResult::VersionConflict` if the expected version doesn't match.
pub async fn change_contract_phase_with_version(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
    new_phase: &str,
    expected_version: Option<i32>,
) -> Result<PhaseChangeResult, sqlx::Error> {
    // Start a transaction to ensure atomicity with row locking
    let mut tx = pool.begin().await?;

    // Lock the row with SELECT FOR UPDATE and get current state
    let existing: Option<Contract> = sqlx::query_as::<_, Contract>(
        r#"
        SELECT *
        FROM contracts
        WHERE id = $1 AND owner_id = $2
        FOR UPDATE
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .fetch_optional(&mut *tx)
    .await?;

    let Some(existing) = existing else {
        tx.rollback().await?;
        return Ok(PhaseChangeResult::NotFound);
    };

    // Check version if provided (optimistic locking)
    if let Some(expected) = expected_version {
        if existing.version != expected {
            tx.rollback().await?;
            return Ok(PhaseChangeResult::VersionConflict {
                expected,
                actual: existing.version,
                current_phase: existing.phase,
            });
        }
    }

    // Validate the phase transition is allowed
    let valid_phases = existing.valid_phase_ids();
    if !valid_phases.contains(&new_phase.to_string()) {
        tx.rollback().await?;
        return Ok(PhaseChangeResult::ValidationFailed {
            reason: format!(
                "Invalid phase '{}' for contract type '{}'",
                new_phase, existing.contract_type
            ),
            missing_requirements: vec![format!(
                "Phase must be one of: {}",
                valid_phases.join(", ")
            )],
        });
    }

    let previous_phase = existing.phase.clone();

    // Update phase with version increment
    let contract = sqlx::query_as::<_, Contract>(
        r#"
        UPDATE contracts
        SET phase = $3, version = version + 1, updated_at = NOW()
        WHERE id = $1 AND owner_id = $2
        RETURNING *
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .bind(new_phase)
    .fetch_one(&mut *tx)
    .await?;

    // Record event
    sqlx::query(
        r#"
        INSERT INTO contract_events (contract_id, event_type, previous_phase, new_phase)
        VALUES ($1, 'phase_change', $2, $3)
        "#,
    )
    .bind(id)
    .bind(&previous_phase)
    .bind(new_phase)
    .execute(&mut *tx)
    .await?;

    // Commit the transaction
    tx.commit().await?;

    Ok(PhaseChangeResult::Success(contract))
}

// =============================================================================
// Contract Repository Functions
// =============================================================================

/// List repositories for a contract.
pub async fn list_contract_repositories(
    pool: &PgPool,
    contract_id: Uuid,
) -> Result<Vec<ContractRepository>, sqlx::Error> {
    sqlx::query_as::<_, ContractRepository>(
        r#"
        SELECT *
        FROM contract_repositories
        WHERE contract_id = $1
        ORDER BY is_primary DESC, created_at ASC
        "#,
    )
    .bind(contract_id)
    .fetch_all(pool)
    .await
}

/// Add a remote repository to a contract.
pub async fn add_remote_repository(
    pool: &PgPool,
    contract_id: Uuid,
    name: &str,
    repository_url: &str,
    is_primary: bool,
) -> Result<ContractRepository, sqlx::Error> {
    // If is_primary, clear other primaries first
    if is_primary {
        sqlx::query(
            r#"
            UPDATE contract_repositories
            SET is_primary = false, updated_at = NOW()
            WHERE contract_id = $1 AND is_primary = true
            "#,
        )
        .bind(contract_id)
        .execute(pool)
        .await?;
    }

    sqlx::query_as::<_, ContractRepository>(
        r#"
        INSERT INTO contract_repositories (contract_id, name, repository_url, source_type, status, is_primary)
        VALUES ($1, $2, $3, 'remote', 'ready', $4)
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .bind(name)
    .bind(repository_url)
    .bind(is_primary)
    .fetch_one(pool)
    .await
}

/// Add a local repository to a contract.
pub async fn add_local_repository(
    pool: &PgPool,
    contract_id: Uuid,
    name: &str,
    local_path: &str,
    is_primary: bool,
) -> Result<ContractRepository, sqlx::Error> {
    // If is_primary, clear other primaries first
    if is_primary {
        sqlx::query(
            r#"
            UPDATE contract_repositories
            SET is_primary = false, updated_at = NOW()
            WHERE contract_id = $1 AND is_primary = true
            "#,
        )
        .bind(contract_id)
        .execute(pool)
        .await?;
    }

    sqlx::query_as::<_, ContractRepository>(
        r#"
        INSERT INTO contract_repositories (contract_id, name, local_path, source_type, status, is_primary)
        VALUES ($1, $2, $3, 'local', 'ready', $4)
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .bind(name)
    .bind(local_path)
    .bind(is_primary)
    .fetch_one(pool)
    .await
}

/// Create a managed repository (daemon will create it).
pub async fn create_managed_repository(
    pool: &PgPool,
    contract_id: Uuid,
    name: &str,
    is_primary: bool,
) -> Result<ContractRepository, sqlx::Error> {
    // If is_primary, clear other primaries first
    if is_primary {
        sqlx::query(
            r#"
            UPDATE contract_repositories
            SET is_primary = false, updated_at = NOW()
            WHERE contract_id = $1 AND is_primary = true
            "#,
        )
        .bind(contract_id)
        .execute(pool)
        .await?;
    }

    sqlx::query_as::<_, ContractRepository>(
        r#"
        INSERT INTO contract_repositories (contract_id, name, source_type, status, is_primary)
        VALUES ($1, $2, 'managed', 'pending', $3)
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .bind(name)
    .bind(is_primary)
    .fetch_one(pool)
    .await
}

/// Delete a repository from a contract.
pub async fn delete_contract_repository(
    pool: &PgPool,
    repo_id: Uuid,
    contract_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM contract_repositories
        WHERE id = $1 AND contract_id = $2
        "#,
    )
    .bind(repo_id)
    .bind(contract_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Set a repository as primary (and clear others).
pub async fn set_repository_primary(
    pool: &PgPool,
    repo_id: Uuid,
    contract_id: Uuid,
) -> Result<bool, sqlx::Error> {
    // Clear other primaries
    sqlx::query(
        r#"
        UPDATE contract_repositories
        SET is_primary = false, updated_at = NOW()
        WHERE contract_id = $1 AND is_primary = true
        "#,
    )
    .bind(contract_id)
    .execute(pool)
    .await?;

    // Set this one as primary
    let result = sqlx::query(
        r#"
        UPDATE contract_repositories
        SET is_primary = true, updated_at = NOW()
        WHERE id = $1 AND contract_id = $2
        "#,
    )
    .bind(repo_id)
    .bind(contract_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Update managed repository status (used by daemon).
pub async fn update_managed_repository_status(
    pool: &PgPool,
    repo_id: Uuid,
    status: &str,
    repository_url: Option<&str>,
) -> Result<Option<ContractRepository>, sqlx::Error> {
    sqlx::query_as::<_, ContractRepository>(
        r#"
        UPDATE contract_repositories
        SET status = $2, repository_url = COALESCE($3, repository_url), updated_at = NOW()
        WHERE id = $1
        RETURNING *
        "#,
    )
    .bind(repo_id)
    .bind(status)
    .bind(repository_url)
    .fetch_optional(pool)
    .await
}

// =============================================================================
// Contract Task Association Functions
// =============================================================================

/// Add a task to a contract.
pub async fn add_task_to_contract(
    pool: &PgPool,
    contract_id: Uuid,
    task_id: Uuid,
    owner_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        UPDATE tasks
        SET contract_id = $2, updated_at = NOW()
        WHERE id = $1 AND owner_id = $3
        "#,
    )
    .bind(task_id)
    .bind(contract_id)
    .bind(owner_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Remove a task from a contract.
pub async fn remove_task_from_contract(
    pool: &PgPool,
    contract_id: Uuid,
    task_id: Uuid,
    owner_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        UPDATE tasks
        SET contract_id = NULL, updated_at = NOW()
        WHERE id = $1 AND contract_id = $2 AND owner_id = $3
        "#,
    )
    .bind(task_id)
    .bind(contract_id)
    .bind(owner_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// List files in a contract.
pub async fn list_files_in_contract(
    pool: &PgPool,
    contract_id: Uuid,
    owner_id: Uuid,
) -> Result<Vec<FileSummary>, sqlx::Error> {
    // Use a manual query since FileSummary doesn't have a FromRow derive with all the computed fields
    let files = sqlx::query_as::<_, File>(
        r#"
        SELECT id, owner_id, contract_id, contract_phase, name, description, transcript, location, summary, body, version, repo_file_path, repo_synced_at, repo_sync_status, created_at, updated_at
        FROM files
        WHERE contract_id = $1 AND owner_id = $2
        ORDER BY created_at DESC
        "#,
    )
    .bind(contract_id)
    .bind(owner_id)
    .fetch_all(pool)
    .await?;

    Ok(files.into_iter().map(FileSummary::from).collect())
}

/// List tasks in a contract.
pub async fn list_tasks_in_contract(
    pool: &PgPool,
    contract_id: Uuid,
    owner_id: Uuid,
) -> Result<Vec<TaskSummary>, sqlx::Error> {
    sqlx::query_as::<_, TaskSummary>(
        r#"
        SELECT
            t.id, t.contract_id, c.name as contract_name, c.phase as contract_phase,
            c.status as contract_status,
            t.parent_task_id, t.depth, t.name, t.status, t.priority,
            t.progress_summary,
            (SELECT COUNT(*) FROM tasks WHERE parent_task_id = t.id) as subtask_count,
            t.version, t.is_supervisor,             COALESCE(t.hidden, false) as hidden, t.created_at, t.updated_at
        FROM tasks t
        LEFT JOIN contracts c ON t.contract_id = c.id
        WHERE t.contract_id = $1 AND t.owner_id = $2
        ORDER BY t.priority DESC, t.created_at DESC
        "#,
    )
    .bind(contract_id)
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Minimal task info for worktree cleanup operations.
#[derive(Debug, Clone, sqlx::FromRow)]
pub struct TaskWorktreeInfo {
    pub id: Uuid,
    pub daemon_id: Option<Uuid>,
    pub overlay_path: Option<String>,
    /// If set, this task shares the worktree of the specified supervisor task.
    /// Should NOT have its worktree deleted during cleanup.
    pub supervisor_worktree_task_id: Option<Uuid>,
}

/// List tasks in a contract with their daemon/worktree info.
/// Used for cleaning up worktrees when a contract is completed or deleted.
pub async fn list_contract_tasks_with_worktree_info(
    pool: &PgPool,
    contract_id: Uuid,
) -> Result<Vec<TaskWorktreeInfo>, sqlx::Error> {
    sqlx::query_as::<_, TaskWorktreeInfo>(
        r#"
        SELECT id, daemon_id, overlay_path, supervisor_worktree_task_id
        FROM tasks
        WHERE contract_id = $1 AND (daemon_id IS NOT NULL OR overlay_path IS NOT NULL)
        "#,
    )
    .bind(contract_id)
    .fetch_all(pool)
    .await
}

// =============================================================================
// Contract Events
// =============================================================================

/// List events for a contract.
pub async fn list_contract_events(
    pool: &PgPool,
    contract_id: Uuid,
) -> Result<Vec<ContractEvent>, sqlx::Error> {
    sqlx::query_as::<_, ContractEvent>(
        r#"
        SELECT *
        FROM contract_events
        WHERE contract_id = $1
        ORDER BY created_at DESC
        "#,
    )
    .bind(contract_id)
    .fetch_all(pool)
    .await
}

/// Record a contract event.
pub async fn record_contract_event(
    pool: &PgPool,
    contract_id: Uuid,
    event_type: &str,
    event_data: Option<serde_json::Value>,
) -> Result<ContractEvent, sqlx::Error> {
    sqlx::query_as::<_, ContractEvent>(
        r#"
        INSERT INTO contract_events (contract_id, event_type, event_data)
        VALUES ($1, $2, $3)
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .bind(event_type)
    .bind(event_data)
    .fetch_one(pool)
    .await
}

// ============================================================================
// Task Checkpoints
// ============================================================================

/// Create a checkpoint for a task.
pub async fn create_task_checkpoint(
    pool: &PgPool,
    task_id: Uuid,
    commit_sha: &str,
    branch_name: &str,
    message: &str,
    files_changed: Option<serde_json::Value>,
    lines_added: Option<i32>,
    lines_removed: Option<i32>,
) -> Result<TaskCheckpoint, sqlx::Error> {
    // Get current checkpoint count and increment
    let checkpoint_number: i32 = sqlx::query_scalar(
        "SELECT COALESCE(MAX(checkpoint_number), 0) + 1 FROM task_checkpoints WHERE task_id = $1",
    )
    .bind(task_id)
    .fetch_one(pool)
    .await?;

    // Update task's checkpoint tracking
    sqlx::query(
        r#"
        UPDATE tasks
        SET last_checkpoint_sha = $1,
            checkpoint_count = $2,
            checkpoint_message = $3,
            updated_at = NOW()
        WHERE id = $4
        "#,
    )
    .bind(commit_sha)
    .bind(checkpoint_number)
    .bind(message)
    .bind(task_id)
    .execute(pool)
    .await?;

    sqlx::query_as::<_, TaskCheckpoint>(
        r#"
        INSERT INTO task_checkpoints (
            task_id, checkpoint_number, commit_sha, branch_name, message,
            files_changed, lines_added, lines_removed
        )
        VALUES ($1, $2, $3, $4, $5, $6, $7, $8)
        RETURNING *
        "#,
    )
    .bind(task_id)
    .bind(checkpoint_number)
    .bind(commit_sha)
    .bind(branch_name)
    .bind(message)
    .bind(files_changed)
    .bind(lines_added)
    .bind(lines_removed)
    .fetch_one(pool)
    .await
}

/// Get a checkpoint by ID.
pub async fn get_task_checkpoint(
    pool: &PgPool,
    id: Uuid,
) -> Result<Option<TaskCheckpoint>, sqlx::Error> {
    sqlx::query_as::<_, TaskCheckpoint>("SELECT * FROM task_checkpoints WHERE id = $1")
        .bind(id)
        .fetch_optional(pool)
        .await
}

/// Get a checkpoint by commit SHA.
pub async fn get_task_checkpoint_by_sha(
    pool: &PgPool,
    commit_sha: &str,
) -> Result<Option<TaskCheckpoint>, sqlx::Error> {
    sqlx::query_as::<_, TaskCheckpoint>("SELECT * FROM task_checkpoints WHERE commit_sha = $1")
        .bind(commit_sha)
        .fetch_optional(pool)
        .await
}

/// List checkpoints for a task.
pub async fn list_task_checkpoints(
    pool: &PgPool,
    task_id: Uuid,
) -> Result<Vec<TaskCheckpoint>, sqlx::Error> {
    sqlx::query_as::<_, TaskCheckpoint>(
        "SELECT * FROM task_checkpoints WHERE task_id = $1 ORDER BY checkpoint_number DESC",
    )
    .bind(task_id)
    .fetch_all(pool)
    .await
}

// ============================================================================
// Supervisor State
// ============================================================================

/// Create or update supervisor state for a contract.
pub async fn upsert_supervisor_state(
    pool: &PgPool,
    contract_id: Uuid,
    task_id: Uuid,
    conversation_history: serde_json::Value,
    pending_task_ids: &[Uuid],
    phase: &str,
) -> Result<SupervisorState, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        INSERT INTO supervisor_states (contract_id, task_id, conversation_history, pending_task_ids, phase, last_activity)
        VALUES ($1, $2, $3, $4, $5, NOW())
        ON CONFLICT (contract_id) DO UPDATE SET
            task_id = EXCLUDED.task_id,
            conversation_history = EXCLUDED.conversation_history,
            pending_task_ids = EXCLUDED.pending_task_ids,
            phase = EXCLUDED.phase,
            last_activity = NOW(),
            updated_at = NOW()
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .bind(task_id)
    .bind(conversation_history)
    .bind(pending_task_ids)
    .bind(phase)
    .fetch_one(pool)
    .await
}

/// Get supervisor state for a contract.
pub async fn get_supervisor_state(
    pool: &PgPool,
    contract_id: Uuid,
) -> Result<Option<SupervisorState>, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>("SELECT * FROM supervisor_states WHERE contract_id = $1")
        .bind(contract_id)
        .fetch_optional(pool)
        .await
}

/// Get supervisor state by task ID.
pub async fn get_supervisor_state_by_task(
    pool: &PgPool,
    task_id: Uuid,
) -> Result<Option<SupervisorState>, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>("SELECT * FROM supervisor_states WHERE task_id = $1")
        .bind(task_id)
        .fetch_optional(pool)
        .await
}

/// Update supervisor conversation history.
pub async fn update_supervisor_conversation(
    pool: &PgPool,
    contract_id: Uuid,
    conversation_history: serde_json::Value,
) -> Result<SupervisorState, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        UPDATE supervisor_states
        SET conversation_history = $1,
            last_activity = NOW(),
            updated_at = NOW()
        WHERE contract_id = $2
        RETURNING *
        "#,
    )
    .bind(conversation_history)
    .bind(contract_id)
    .fetch_one(pool)
    .await
}

/// Update supervisor pending tasks.
pub async fn update_supervisor_pending_tasks(
    pool: &PgPool,
    contract_id: Uuid,
    pending_task_ids: &[Uuid],
) -> Result<SupervisorState, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        UPDATE supervisor_states
        SET pending_task_ids = $1,
            last_activity = NOW(),
            updated_at = NOW()
        WHERE contract_id = $2
        RETURNING *
        "#,
    )
    .bind(pending_task_ids)
    .bind(contract_id)
    .fetch_one(pool)
    .await
}

/// Update supervisor state with detailed activity tracking.
/// Called at key save points: LLM response, task spawn, question asked, phase change.
pub async fn update_supervisor_detailed_state(
    pool: &PgPool,
    contract_id: Uuid,
    state: &str,
    current_activity: Option<&str>,
    progress: i32,
    error_message: Option<&str>,
) -> Result<SupervisorState, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        UPDATE supervisor_states
        SET state = $1,
            current_activity = $2,
            progress = $3,
            error_message = $4,
            last_activity = NOW(),
            updated_at = NOW()
        WHERE contract_id = $5
        RETURNING *
        "#,
    )
    .bind(state)
    .bind(current_activity)
    .bind(progress)
    .bind(error_message)
    .bind(contract_id)
    .fetch_one(pool)
    .await
}

/// Add a spawned task ID to the supervisor's list.
pub async fn add_supervisor_spawned_task(
    pool: &PgPool,
    contract_id: Uuid,
    task_id: Uuid,
) -> Result<SupervisorState, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        UPDATE supervisor_states
        SET spawned_task_ids = array_append(spawned_task_ids, $1),
            last_activity = NOW(),
            updated_at = NOW()
        WHERE contract_id = $2
        RETURNING *
        "#,
    )
    .bind(task_id)
    .bind(contract_id)
    .fetch_one(pool)
    .await
}

/// Add a pending question to the supervisor state.
pub async fn add_supervisor_pending_question(
    pool: &PgPool,
    contract_id: Uuid,
    question: serde_json::Value,
) -> Result<SupervisorState, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        UPDATE supervisor_states
        SET pending_questions = pending_questions || $1::jsonb,
            state = 'waiting_for_user',
            last_activity = NOW(),
            updated_at = NOW()
        WHERE contract_id = $2
        RETURNING *
        "#,
    )
    .bind(question)
    .bind(contract_id)
    .fetch_one(pool)
    .await
}

/// Remove a pending question by ID.
pub async fn remove_supervisor_pending_question(
    pool: &PgPool,
    contract_id: Uuid,
    question_id: Uuid,
) -> Result<SupervisorState, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        UPDATE supervisor_states
        SET pending_questions = (
            SELECT COALESCE(jsonb_agg(elem), '[]'::jsonb)
            FROM jsonb_array_elements(pending_questions) elem
            WHERE (elem->>'id')::uuid != $1
        ),
            last_activity = NOW(),
            updated_at = NOW()
        WHERE contract_id = $2
        RETURNING *
        "#,
    )
    .bind(question_id)
    .bind(contract_id)
    .fetch_one(pool)
    .await
}

/// Comprehensive state save - used at major save points.
pub async fn save_supervisor_state_full(
    pool: &PgPool,
    contract_id: Uuid,
    task_id: Uuid,
    conversation_history: serde_json::Value,
    pending_task_ids: &[Uuid],
    phase: &str,
    state: &str,
    current_activity: Option<&str>,
    progress: i32,
    error_message: Option<&str>,
    spawned_task_ids: &[Uuid],
    pending_questions: serde_json::Value,
) -> Result<SupervisorState, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        INSERT INTO supervisor_states (
            contract_id, task_id, conversation_history, pending_task_ids, phase,
            state, current_activity, progress, error_message, spawned_task_ids,
            pending_questions, last_activity
        )
        VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, NOW())
        ON CONFLICT (contract_id) DO UPDATE SET
            task_id = EXCLUDED.task_id,
            conversation_history = EXCLUDED.conversation_history,
            pending_task_ids = EXCLUDED.pending_task_ids,
            phase = EXCLUDED.phase,
            state = EXCLUDED.state,
            current_activity = EXCLUDED.current_activity,
            progress = EXCLUDED.progress,
            error_message = EXCLUDED.error_message,
            spawned_task_ids = EXCLUDED.spawned_task_ids,
            pending_questions = EXCLUDED.pending_questions,
            last_activity = NOW(),
            updated_at = NOW()
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .bind(task_id)
    .bind(conversation_history)
    .bind(pending_task_ids)
    .bind(phase)
    .bind(state)
    .bind(current_activity)
    .bind(progress)
    .bind(error_message)
    .bind(spawned_task_ids)
    .bind(pending_questions)
    .fetch_one(pool)
    .await
}

/// Mark supervisor as restored from a crash/interruption.
pub async fn mark_supervisor_restored(
    pool: &PgPool,
    contract_id: Uuid,
    restoration_source: &str,
) -> Result<SupervisorState, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        UPDATE supervisor_states
        SET restoration_count = restoration_count + 1,
            last_restored_at = NOW(),
            restoration_source = $1,
            state = 'initializing',
            error_message = NULL,
            last_activity = NOW(),
            updated_at = NOW()
        WHERE contract_id = $2
        RETURNING *
        "#,
    )
    .bind(restoration_source)
    .bind(contract_id)
    .fetch_one(pool)
    .await
}

/// Get supervisors with pending questions (for re-delivery after restoration).
pub async fn get_supervisors_with_pending_questions(
    pool: &PgPool,
    owner_id: Uuid,
) -> Result<Vec<SupervisorState>, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        SELECT ss.*
        FROM supervisor_states ss
        JOIN contracts c ON c.id = ss.contract_id
        WHERE c.owner_id = $1
          AND ss.pending_questions != '[]'::jsonb
          AND c.status = 'active'
        ORDER BY ss.last_activity DESC
        "#,
    )
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Get supervisor state with full details for restoration.
/// Includes validation info.
pub async fn get_supervisor_state_for_restoration(
    pool: &PgPool,
    contract_id: Uuid,
) -> Result<Option<SupervisorState>, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        SELECT * FROM supervisor_states WHERE contract_id = $1
        "#,
    )
    .bind(contract_id)
    .fetch_optional(pool)
    .await
}

/// Validate spawned tasks are in expected states.
/// Returns map of task_id -> (status, updated_at).
pub async fn validate_spawned_tasks(
    pool: &PgPool,
    task_ids: &[Uuid],
) -> Result<std::collections::HashMap<Uuid, (String, chrono::DateTime<Utc>)>, sqlx::Error> {
    use sqlx::Row;

    let rows = sqlx::query(
        r#"
        SELECT id, status, updated_at
        FROM tasks
        WHERE id = ANY($1)
        "#,
    )
    .bind(task_ids)
    .fetch_all(pool)
    .await?;

    let mut result = std::collections::HashMap::new();
    for row in rows {
        let id: Uuid = row.get("id");
        let status: String = row.get("status");
        let updated_at: chrono::DateTime<Utc> = row.get("updated_at");
        result.insert(id, (status, updated_at));
    }
    Ok(result)
}

/// Update supervisor state when phase changes.
pub async fn update_supervisor_phase(
    pool: &PgPool,
    contract_id: Uuid,
    new_phase: &str,
) -> Result<SupervisorState, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        UPDATE supervisor_states
        SET phase = $1,
            state = 'working',
            current_activity = 'Phase changed to ' || $1,
            last_activity = NOW(),
            updated_at = NOW()
        WHERE contract_id = $2
        RETURNING *
        "#,
    )
    .bind(new_phase)
    .bind(contract_id)
    .fetch_one(pool)
    .await
}

/// Update supervisor state on heartbeat (lightweight update).
pub async fn update_supervisor_heartbeat_state(
    pool: &PgPool,
    contract_id: Uuid,
    state: &str,
    current_activity: Option<&str>,
    progress: i32,
    pending_task_ids: &[Uuid],
) -> Result<(), sqlx::Error> {
    sqlx::query(
        r#"
        UPDATE supervisor_states
        SET state = $1,
            current_activity = $2,
            progress = $3,
            pending_task_ids = $4,
            last_activity = NOW(),
            updated_at = NOW()
        WHERE contract_id = $5
        "#,
    )
    .bind(state)
    .bind(current_activity)
    .bind(progress)
    .bind(pending_task_ids)
    .bind(contract_id)
    .execute(pool)
    .await?;
    Ok(())
}

// ============================================================================
// Supervisor Heartbeats
// ============================================================================

/// Record a supervisor heartbeat.
/// This creates a historical record for monitoring and dead supervisor detection.
pub async fn create_supervisor_heartbeat(
    pool: &PgPool,
    supervisor_task_id: Uuid,
    contract_id: Uuid,
    state: &str,
    phase: &str,
    current_activity: Option<&str>,
    progress: i32,
    pending_task_ids: &[Uuid],
) -> Result<SupervisorHeartbeatRecord, sqlx::Error> {
    sqlx::query_as::<_, SupervisorHeartbeatRecord>(
        r#"
        INSERT INTO supervisor_heartbeats (
            supervisor_task_id, contract_id, state, phase, current_activity, progress, pending_task_ids, timestamp
        )
        VALUES ($1, $2, $3, $4, $5, $6, $7, NOW())
        RETURNING *
        "#,
    )
    .bind(supervisor_task_id)
    .bind(contract_id)
    .bind(state)
    .bind(phase)
    .bind(current_activity)
    .bind(progress)
    .bind(pending_task_ids)
    .fetch_one(pool)
    .await
}

/// Get the latest heartbeat for a supervisor task.
pub async fn get_latest_supervisor_heartbeat(
    pool: &PgPool,
    supervisor_task_id: Uuid,
) -> Result<Option<SupervisorHeartbeatRecord>, sqlx::Error> {
    sqlx::query_as::<_, SupervisorHeartbeatRecord>(
        r#"
        SELECT * FROM supervisor_heartbeats
        WHERE supervisor_task_id = $1
        ORDER BY timestamp DESC
        LIMIT 1
        "#,
    )
    .bind(supervisor_task_id)
    .fetch_optional(pool)
    .await
}

/// Get recent heartbeats for a supervisor task.
pub async fn get_supervisor_heartbeats(
    pool: &PgPool,
    supervisor_task_id: Uuid,
    limit: i64,
) -> Result<Vec<SupervisorHeartbeatRecord>, sqlx::Error> {
    sqlx::query_as::<_, SupervisorHeartbeatRecord>(
        r#"
        SELECT * FROM supervisor_heartbeats
        WHERE supervisor_task_id = $1
        ORDER BY timestamp DESC
        LIMIT $2
        "#,
    )
    .bind(supervisor_task_id)
    .bind(limit)
    .fetch_all(pool)
    .await
}

/// Get recent heartbeats for a contract.
pub async fn get_contract_supervisor_heartbeats(
    pool: &PgPool,
    contract_id: Uuid,
    limit: i64,
) -> Result<Vec<SupervisorHeartbeatRecord>, sqlx::Error> {
    sqlx::query_as::<_, SupervisorHeartbeatRecord>(
        r#"
        SELECT * FROM supervisor_heartbeats
        WHERE contract_id = $1
        ORDER BY timestamp DESC
        LIMIT $2
        "#,
    )
    .bind(contract_id)
    .bind(limit)
    .fetch_all(pool)
    .await
}

/// Delete old heartbeats beyond the TTL (24 hours by default).
/// Returns the number of deleted records.
pub async fn cleanup_old_heartbeats(
    pool: &PgPool,
    ttl_hours: i64,
) -> Result<u64, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM supervisor_heartbeats
        WHERE timestamp < NOW() - ($1 || ' hours')::INTERVAL
        "#,
    )
    .bind(ttl_hours.to_string())
    .execute(pool)
    .await?;

    Ok(result.rows_affected())
}

/// Find supervisors that have not sent a heartbeat within the timeout period.
/// Returns list of (supervisor_task_id, contract_id, last_heartbeat_timestamp).
pub async fn find_stale_supervisors(
    pool: &PgPool,
    timeout_seconds: i64,
) -> Result<Vec<(Uuid, Uuid, chrono::DateTime<Utc>)>, sqlx::Error> {
    let rows = sqlx::query(
        r#"
        WITH latest_heartbeats AS (
            SELECT DISTINCT ON (supervisor_task_id)
                supervisor_task_id,
                contract_id,
                timestamp
            FROM supervisor_heartbeats
            ORDER BY supervisor_task_id, timestamp DESC
        )
        SELECT
            lh.supervisor_task_id,
            lh.contract_id,
            lh.timestamp
        FROM latest_heartbeats lh
        JOIN tasks t ON t.id = lh.supervisor_task_id
        WHERE t.status = 'running'
          AND lh.timestamp < NOW() - ($1 || ' seconds')::INTERVAL
        "#,
    )
    .bind(timeout_seconds.to_string())
    .fetch_all(pool)
    .await?;

    let mut result = Vec::new();
    for row in rows {
        use sqlx::Row;
        let supervisor_task_id: Uuid = row.get("supervisor_task_id");
        let contract_id: Uuid = row.get("contract_id");
        let timestamp: chrono::DateTime<Utc> = row.get("timestamp");
        result.push((supervisor_task_id, contract_id, timestamp));
    }
    Ok(result)
}

// ============================================================================
// Contract Supervisor
// ============================================================================

/// Update contract's supervisor task ID.
pub async fn update_contract_supervisor(
    pool: &PgPool,
    contract_id: Uuid,
    supervisor_task_id: Uuid,
) -> Result<Contract, sqlx::Error> {
    sqlx::query_as::<_, Contract>(
        r#"
        UPDATE contracts
        SET supervisor_task_id = $1,
            updated_at = NOW()
        WHERE id = $2
        RETURNING *
        "#,
    )
    .bind(supervisor_task_id)
    .bind(contract_id)
    .fetch_one(pool)
    .await
}

/// Mark a deliverable as complete for a specific phase.
/// Uses JSONB operations to append the deliverable_id to the phase's array.
pub async fn mark_deliverable_complete(
    pool: &PgPool,
    contract_id: Uuid,
    phase: &str,
    deliverable_id: &str,
) -> Result<Contract, sqlx::Error> {
    // Use jsonb_set to add the deliverable to the phase's array
    // If the phase key doesn't exist, create an empty array first
    // COALESCE handles the case where the phase array doesn't exist yet
    sqlx::query_as::<_, Contract>(
        r#"
        UPDATE contracts
        SET completed_deliverables = jsonb_set(
            completed_deliverables,
            ARRAY[$2::text],
            COALESCE(completed_deliverables->$2, '[]'::jsonb) || to_jsonb($3::text),
            true
        ),
        updated_at = NOW()
        WHERE id = $1
        AND NOT (COALESCE(completed_deliverables->$2, '[]'::jsonb) ? $3)
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .bind(phase)
    .bind(deliverable_id)
    .fetch_one(pool)
    .await
}

/// Clear all completed deliverables for a specific phase.
/// Used when phase changes or deliverables need to be reset.
pub async fn clear_phase_deliverables(
    pool: &PgPool,
    contract_id: Uuid,
    phase: &str,
) -> Result<Contract, sqlx::Error> {
    sqlx::query_as::<_, Contract>(
        r#"
        UPDATE contracts
        SET completed_deliverables = completed_deliverables - $2,
            updated_at = NOW()
        WHERE id = $1
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .bind(phase)
    .fetch_one(pool)
    .await
}

/// Get the supervisor task for a contract.
pub async fn get_contract_supervisor_task(
    pool: &PgPool,
    contract_id: Uuid,
) -> Result<Option<Task>, sqlx::Error> {
    sqlx::query_as::<_, Task>(
        r#"
        SELECT t.* FROM tasks t
        JOIN contracts c ON c.supervisor_task_id = t.id
        WHERE c.id = $1
        "#,
    )
    .bind(contract_id)
    .fetch_optional(pool)
    .await
}

// ============================================================================
// Task Tree Queries
// ============================================================================

/// Get full task tree for a contract.
pub async fn get_contract_task_tree(
    pool: &PgPool,
    contract_id: Uuid,
) -> Result<Vec<Task>, sqlx::Error> {
    sqlx::query_as::<_, Task>(
        r#"
        WITH RECURSIVE task_tree AS (
            -- Base case: root tasks (no parent)
            SELECT * FROM tasks
            WHERE contract_id = $1 AND parent_task_id IS NULL
            UNION ALL
            -- Recursive case: children of current level
            SELECT t.* FROM tasks t
            JOIN task_tree tt ON t.parent_task_id = tt.id
        )
        SELECT * FROM task_tree
        ORDER BY depth, created_at
        "#,
    )
    .bind(contract_id)
    .fetch_all(pool)
    .await
}

/// Get task tree from a specific root task.
pub async fn get_task_tree(pool: &PgPool, root_task_id: Uuid) -> Result<Vec<Task>, sqlx::Error> {
    sqlx::query_as::<_, Task>(
        r#"
        WITH RECURSIVE task_tree AS (
            -- Base case: the root task
            SELECT * FROM tasks WHERE id = $1
            UNION ALL
            -- Recursive case: children of current level
            SELECT t.* FROM tasks t
            JOIN task_tree tt ON t.parent_task_id = tt.id
        )
        SELECT * FROM task_tree
        ORDER BY depth, created_at
        "#,
    )
    .bind(root_task_id)
    .fetch_all(pool)
    .await
}

// ============================================================================
// Daemon Selection
// ============================================================================

/// Get daemons with capacity info for selection.
pub async fn get_available_daemons(
    pool: &PgPool,
    owner_id: Uuid,
) -> Result<Vec<DaemonWithCapacity>, sqlx::Error> {
    sqlx::query_as::<_, DaemonWithCapacity>(
        r#"
        SELECT id, owner_id, connection_id, hostname, machine_id,
               max_concurrent_tasks, current_task_count,
               capacity_score, task_queue_length, supports_migration,
               status, last_heartbeat_at, connected_at
        FROM daemons
        WHERE owner_id = $1 AND status = 'connected'
        ORDER BY
            COALESCE(capacity_score, 100) DESC,
            (max_concurrent_tasks - current_task_count) DESC,
            COALESCE(task_queue_length, 0) ASC
        "#,
    )
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Get daemons with capacity info for selection, excluding specified daemon IDs.
/// Used for task retry to avoid reassigning to daemons that have already failed.
pub async fn get_available_daemons_excluding(
    pool: &PgPool,
    owner_id: Uuid,
    exclude_daemon_ids: &[Uuid],
) -> Result<Vec<DaemonWithCapacity>, sqlx::Error> {
    sqlx::query_as::<_, DaemonWithCapacity>(
        r#"
        SELECT id, owner_id, connection_id, hostname, machine_id,
               max_concurrent_tasks, current_task_count,
               capacity_score, task_queue_length, supports_migration,
               status, last_heartbeat_at, connected_at
        FROM daemons
        WHERE owner_id = $1
          AND status = 'connected'
          AND id != ALL($2)
        ORDER BY
            COALESCE(capacity_score, 100) DESC,
            (max_concurrent_tasks - current_task_count) DESC,
            COALESCE(task_queue_length, 0) ASC
        "#,
    )
    .bind(owner_id)
    .bind(exclude_daemon_ids)
    .fetch_all(pool)
    .await
}

/// Create a daemon task assignment.
pub async fn create_daemon_task_assignment(
    pool: &PgPool,
    daemon_id: Uuid,
    task_id: Uuid,
) -> Result<DaemonTaskAssignment, sqlx::Error> {
    sqlx::query_as::<_, DaemonTaskAssignment>(
        r#"
        INSERT INTO daemon_task_assignments (daemon_id, task_id)
        VALUES ($1, $2)
        RETURNING *
        "#,
    )
    .bind(daemon_id)
    .bind(task_id)
    .fetch_one(pool)
    .await
}

/// Update daemon task assignment status.
pub async fn update_daemon_task_assignment_status(
    pool: &PgPool,
    task_id: Uuid,
    status: &str,
) -> Result<DaemonTaskAssignment, sqlx::Error> {
    sqlx::query_as::<_, DaemonTaskAssignment>(
        r#"
        UPDATE daemon_task_assignments
        SET status = $1
        WHERE task_id = $2
        RETURNING *
        "#,
    )
    .bind(status)
    .bind(task_id)
    .fetch_one(pool)
    .await
}

/// Get daemon task assignment for a task.
pub async fn get_daemon_task_assignment(
    pool: &PgPool,
    task_id: Uuid,
) -> Result<Option<DaemonTaskAssignment>, sqlx::Error> {
    sqlx::query_as::<_, DaemonTaskAssignment>(
        "SELECT * FROM daemon_task_assignments WHERE task_id = $1",
    )
    .bind(task_id)
    .fetch_optional(pool)
    .await
}

// ============================================================================
// Repository History Functions
// ============================================================================

use super::models::RepositoryHistoryEntry;

/// List all repository history entries for an owner, ordered by use_count DESC, last_used_at DESC.
pub async fn list_repository_history_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
) -> Result<Vec<RepositoryHistoryEntry>, sqlx::Error> {
    sqlx::query_as::<_, RepositoryHistoryEntry>(
        r#"
        SELECT id, owner_id, name, repository_url, local_path, source_type, use_count, last_used_at, created_at
        FROM repository_history
        WHERE owner_id = $1
        ORDER BY use_count DESC, last_used_at DESC
        "#,
    )
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Get repository suggestions for an owner, optionally filtered by source type and query.
pub async fn get_repository_suggestions(
    pool: &PgPool,
    owner_id: Uuid,
    source_type: Option<&str>,
    query: Option<&str>,
    limit: i32,
) -> Result<Vec<RepositoryHistoryEntry>, sqlx::Error> {
    // Build query dynamically based on filters
    let mut sql = String::from(
        r#"
        SELECT id, owner_id, name, repository_url, local_path, source_type, use_count, last_used_at, created_at
        FROM repository_history
        WHERE owner_id = $1
        "#,
    );

    let mut param_idx = 2;

    if source_type.is_some() {
        sql.push_str(&format!(" AND source_type = ${}", param_idx));
        param_idx += 1;
    }

    if query.is_some() {
        sql.push_str(&format!(
            " AND (LOWER(name) LIKE ${} OR LOWER(COALESCE(repository_url, '')) LIKE ${} OR LOWER(COALESCE(local_path, '')) LIKE ${})",
            param_idx, param_idx, param_idx
        ));
        param_idx += 1;
    }

    sql.push_str(&format!(
        " ORDER BY use_count DESC, last_used_at DESC LIMIT ${}",
        param_idx
    ));

    // Build and execute query with the appropriate bindings
    let mut query_builder = sqlx::query_as::<_, RepositoryHistoryEntry>(&sql).bind(owner_id);

    if let Some(st) = source_type {
        query_builder = query_builder.bind(st);
    }

    if let Some(q) = query {
        let search_pattern = format!("%{}%", q.to_lowercase());
        query_builder = query_builder.bind(search_pattern);
    }

    query_builder = query_builder.bind(limit);

    query_builder.fetch_all(pool).await
}

/// Add or update a repository history entry.
/// If an entry with the same URL (for remote) or path (for local) already exists,
/// increment use_count and update last_used_at and name.
/// Otherwise, create a new entry.
pub async fn add_or_update_repository_history(
    pool: &PgPool,
    owner_id: Uuid,
    name: &str,
    repository_url: Option<&str>,
    local_path: Option<&str>,
    source_type: &str,
) -> Result<RepositoryHistoryEntry, sqlx::Error> {
    // Use UPSERT (INSERT ... ON CONFLICT)
    if source_type == "remote" {
        let url = repository_url.ok_or_else(|| {
            sqlx::Error::Protocol("repository_url required for remote type".to_string())
        })?;

        sqlx::query_as::<_, RepositoryHistoryEntry>(
            r#"
            INSERT INTO repository_history (owner_id, name, repository_url, local_path, source_type, use_count, last_used_at)
            VALUES ($1, $2, $3, NULL, $4, 1, NOW())
            ON CONFLICT (owner_id, repository_url) WHERE source_type = 'remote' AND repository_url IS NOT NULL
            DO UPDATE SET
                name = EXCLUDED.name,
                use_count = repository_history.use_count + 1,
                last_used_at = NOW()
            RETURNING id, owner_id, name, repository_url, local_path, source_type, use_count, last_used_at, created_at
            "#,
        )
        .bind(owner_id)
        .bind(name)
        .bind(url)
        .bind(source_type)
        .fetch_one(pool)
        .await
    } else if source_type == "local" {
        let path = local_path.ok_or_else(|| {
            sqlx::Error::Protocol("local_path required for local type".to_string())
        })?;

        sqlx::query_as::<_, RepositoryHistoryEntry>(
            r#"
            INSERT INTO repository_history (owner_id, name, repository_url, local_path, source_type, use_count, last_used_at)
            VALUES ($1, $2, NULL, $3, $4, 1, NOW())
            ON CONFLICT (owner_id, local_path) WHERE source_type = 'local' AND local_path IS NOT NULL
            DO UPDATE SET
                name = EXCLUDED.name,
                use_count = repository_history.use_count + 1,
                last_used_at = NOW()
            RETURNING id, owner_id, name, repository_url, local_path, source_type, use_count, last_used_at, created_at
            "#,
        )
        .bind(owner_id)
        .bind(name)
        .bind(path)
        .bind(source_type)
        .fetch_one(pool)
        .await
    } else {
        Err(sqlx::Error::Protocol(format!(
            "Invalid source_type: {}",
            source_type
        )))
    }
}

/// Delete a repository history entry.
/// Returns true if an entry was deleted, false if not found.
pub async fn delete_repository_history(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM repository_history
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

// ============================================================================
// Conversation Snapshots
// ============================================================================

/// Create a new conversation snapshot
pub async fn create_conversation_snapshot(
    pool: &PgPool,
    task_id: Uuid,
    checkpoint_id: Option<Uuid>,
    snapshot_type: &str,
    message_count: i32,
    conversation_state: serde_json::Value,
    metadata: Option<serde_json::Value>,
) -> Result<ConversationSnapshot, sqlx::Error> {
    sqlx::query_as::<_, ConversationSnapshot>(
        r#"
        INSERT INTO conversation_snapshots (task_id, checkpoint_id, snapshot_type, message_count, conversation_state, metadata)
        VALUES ($1, $2, $3, $4, $5, $6)
        RETURNING *
        "#
    )
    .bind(task_id)
    .bind(checkpoint_id)
    .bind(snapshot_type)
    .bind(message_count)
    .bind(conversation_state)
    .bind(metadata)
    .fetch_one(pool)
    .await
}

/// Get a conversation snapshot by ID
pub async fn get_conversation_snapshot(
    pool: &PgPool,
    id: Uuid,
) -> Result<Option<ConversationSnapshot>, sqlx::Error> {
    sqlx::query_as::<_, ConversationSnapshot>(
        "SELECT * FROM conversation_snapshots WHERE id = $1"
    )
    .bind(id)
    .fetch_optional(pool)
    .await
}

/// Get conversation snapshot at a specific checkpoint
pub async fn get_conversation_at_checkpoint(
    pool: &PgPool,
    checkpoint_id: Uuid,
) -> Result<Option<ConversationSnapshot>, sqlx::Error> {
    sqlx::query_as::<_, ConversationSnapshot>(
        "SELECT * FROM conversation_snapshots WHERE checkpoint_id = $1 ORDER BY created_at DESC LIMIT 1"
    )
    .bind(checkpoint_id)
    .fetch_optional(pool)
    .await
}

/// List conversation snapshots for a task
pub async fn list_conversation_snapshots(
    pool: &PgPool,
    task_id: Uuid,
    limit: Option<i32>,
) -> Result<Vec<ConversationSnapshot>, sqlx::Error> {
    let limit = limit.unwrap_or(100);
    sqlx::query_as::<_, ConversationSnapshot>(
        "SELECT * FROM conversation_snapshots WHERE task_id = $1 ORDER BY created_at DESC LIMIT $2"
    )
    .bind(task_id)
    .bind(limit)
    .fetch_all(pool)
    .await
}

/// Delete conversation snapshots older than retention period
pub async fn cleanup_old_snapshots(
    pool: &PgPool,
    retention_days: i32,
) -> Result<u64, sqlx::Error> {
    let result = sqlx::query(
        "DELETE FROM conversation_snapshots WHERE created_at < NOW() - INTERVAL '1 day' * $1"
    )
    .bind(retention_days)
    .execute(pool)
    .await?;
    Ok(result.rows_affected())
}

// ============================================================================
// History Events
// ============================================================================

/// Record a new history event
#[allow(clippy::too_many_arguments)]
pub async fn record_history_event(
    pool: &PgPool,
    owner_id: Uuid,
    contract_id: Option<Uuid>,
    task_id: Option<Uuid>,
    event_type: &str,
    event_subtype: Option<&str>,
    phase: Option<&str>,
    event_data: serde_json::Value,
) -> Result<HistoryEvent, sqlx::Error> {
    sqlx::query_as::<_, HistoryEvent>(
        r#"
        INSERT INTO history_events (owner_id, contract_id, task_id, event_type, event_subtype, phase, event_data)
        VALUES ($1, $2, $3, $4, $5, $6, $7)
        RETURNING *
        "#
    )
    .bind(owner_id)
    .bind(contract_id)
    .bind(task_id)
    .bind(event_type)
    .bind(event_subtype)
    .bind(phase)
    .bind(event_data)
    .fetch_one(pool)
    .await
}

/// Get contract history timeline
pub async fn get_contract_history(
    pool: &PgPool,
    contract_id: Uuid,
    owner_id: Uuid,
    filters: &HistoryQueryFilters,
) -> Result<(Vec<HistoryEvent>, i64), sqlx::Error> {
    let limit = filters.limit.unwrap_or(100);

    let mut query = String::from(
        "SELECT * FROM history_events WHERE contract_id = $1 AND owner_id = $2"
    );
    let mut count_query = String::from(
        "SELECT COUNT(*) FROM history_events WHERE contract_id = $1 AND owner_id = $2"
    );

    let mut param_count = 2;

    if filters.phase.is_some() {
        param_count += 1;
        query.push_str(&format!(" AND phase = ${}" , param_count));
        count_query.push_str(&format!(" AND phase = ${}", param_count));
    }

    if filters.from.is_some() {
        param_count += 1;
        query.push_str(&format!(" AND created_at >= ${}", param_count));
        count_query.push_str(&format!(" AND created_at >= ${}", param_count));
    }

    if filters.to.is_some() {
        param_count += 1;
        query.push_str(&format!(" AND created_at <= ${}", param_count));
        count_query.push_str(&format!(" AND created_at <= ${}", param_count));
    }

    query.push_str(" ORDER BY created_at DESC");
    query.push_str(&format!(" LIMIT {}", limit));

    // Build and execute the query dynamically
    let mut q = sqlx::query_as::<_, HistoryEvent>(&query)
        .bind(contract_id)
        .bind(owner_id);

    if let Some(ref phase) = filters.phase {
        q = q.bind(phase);
    }
    if let Some(ref from) = filters.from {
        q = q.bind(from);
    }
    if let Some(ref to) = filters.to {
        q = q.bind(to);
    }

    let events = q.fetch_all(pool).await?;

    // Get total count
    let mut cq = sqlx::query_scalar::<_, i64>(&count_query)
        .bind(contract_id)
        .bind(owner_id);

    if let Some(ref phase) = filters.phase {
        cq = cq.bind(phase);
    }
    if let Some(ref from) = filters.from {
        cq = cq.bind(from);
    }
    if let Some(ref to) = filters.to {
        cq = cq.bind(to);
    }

    let count = cq.fetch_one(pool).await?;

    Ok((events, count))
}

/// Get task history
pub async fn get_task_history(
    pool: &PgPool,
    task_id: Uuid,
    owner_id: Uuid,
    filters: &HistoryQueryFilters,
) -> Result<(Vec<HistoryEvent>, i64), sqlx::Error> {
    let limit = filters.limit.unwrap_or(100);

    let events = sqlx::query_as::<_, HistoryEvent>(
        r#"
        SELECT * FROM history_events
        WHERE task_id = $1 AND owner_id = $2
        ORDER BY created_at DESC
        LIMIT $3
        "#
    )
    .bind(task_id)
    .bind(owner_id)
    .bind(limit)
    .fetch_all(pool)
    .await?;

    let count: i64 = sqlx::query_scalar(
        "SELECT COUNT(*) FROM history_events WHERE task_id = $1 AND owner_id = $2"
    )
    .bind(task_id)
    .bind(owner_id)
    .fetch_one(pool)
    .await?;

    Ok((events, count))
}

/// Get unified timeline for an owner
pub async fn get_timeline(
    pool: &PgPool,
    owner_id: Uuid,
    filters: &HistoryQueryFilters,
) -> Result<(Vec<HistoryEvent>, i64), sqlx::Error> {
    let limit = filters.limit.unwrap_or(100);

    let events = sqlx::query_as::<_, HistoryEvent>(
        r#"
        SELECT * FROM history_events
        WHERE owner_id = $1
        ORDER BY created_at DESC
        LIMIT $2
        "#
    )
    .bind(owner_id)
    .bind(limit)
    .fetch_all(pool)
    .await?;

    let count: i64 = sqlx::query_scalar(
        "SELECT COUNT(*) FROM history_events WHERE owner_id = $1"
    )
    .bind(owner_id)
    .fetch_one(pool)
    .await?;

    Ok((events, count))
}

// ============================================================================
// Task Conversation Retrieval
// ============================================================================

// Helper struct for parsing task output events
#[derive(Debug, Deserialize)]
#[serde(rename_all = "camelCase")]
struct TaskOutputEvent {
    message_type: String,
    content: Option<String>,
    tool_name: Option<String>,
    tool_input: Option<serde_json::Value>,
    is_error: Option<bool>,
    cost_usd: Option<f32>,
}

/// Get task conversation messages (reconstructed from task_events)
pub async fn get_task_conversation(
    pool: &PgPool,
    task_id: Uuid,
    include_tool_calls: bool,
    include_tool_results: bool,
    limit: Option<i32>,
) -> Result<Vec<ConversationMessage>, sqlx::Error> {
    let limit = limit.unwrap_or(1000);

    // Get output events that represent conversation turns
    let events = sqlx::query_as::<_, TaskEvent>(
        r#"
        SELECT * FROM task_events
        WHERE task_id = $1 AND event_type = 'output'
        ORDER BY created_at ASC
        LIMIT $2
        "#
    )
    .bind(task_id)
    .bind(limit)
    .fetch_all(pool)
    .await?;

    // Convert task events to conversation messages
    let mut messages = Vec::new();
    for event in events {
        if let Some(data) = event.event_data {
            // Parse the event data to extract message info
            if let Ok(output) = serde_json::from_value::<TaskOutputEvent>(data.clone()) {
                let should_include = match output.message_type.as_str() {
                    "tool_use" => include_tool_calls,
                    "tool_result" => include_tool_results,
                    _ => true,
                };

                if should_include {
                    messages.push(ConversationMessage {
                        id: event.id.to_string(),
                        role: match output.message_type.as_str() {
                            "assistant" => "assistant".to_string(),
                            "tool_use" => "assistant".to_string(),
                            "tool_result" => "tool".to_string(),
                            "system" => "system".to_string(),
                            "error" => "system".to_string(),
                            _ => "user".to_string(),
                        },
                        content: output.content.unwrap_or_default(),
                        timestamp: event.created_at,
                        tool_calls: None,
                        tool_name: output.tool_name,
                        tool_input: output.tool_input,
                        tool_result: None,
                        is_error: output.is_error,
                        token_count: None,
                        cost_usd: output.cost_usd.map(|c| c as f64),
                    });
                }
            }
        }
    }

    Ok(messages)
}

/// Get supervisor conversation (from supervisor_states)
pub async fn get_supervisor_conversation_full(
    pool: &PgPool,
    contract_id: Uuid,
) -> Result<Option<SupervisorState>, sqlx::Error> {
    get_supervisor_state(pool, contract_id).await
}

// =============================================================================
// Anonymous Task Cleanup Functions
// =============================================================================

/// Delete stale anonymous tasks (tasks with contract_id = NULL) that:
/// - Are in a terminal state (done, failed, merged)
/// - Are older than the specified number of days
///
/// Returns the number of deleted tasks.
pub async fn cleanup_stale_anonymous_tasks(
    pool: &PgPool,
    max_age_days: i32,
) -> Result<i64, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM tasks
        WHERE contract_id IS NULL
          AND status IN ('done', 'failed', 'merged')
          AND created_at < NOW() - INTERVAL '1 day' * $1
        "#,
    )
    .bind(max_age_days)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() as i64)
}

// ============================================================================
// Checkpoint Patches (for task recovery)
// ============================================================================

/// Create a checkpoint patch for task recovery.
pub async fn create_checkpoint_patch(
    pool: &PgPool,
    task_id: Uuid,
    checkpoint_id: Option<Uuid>,
    base_commit_sha: &str,
    patch_data: &[u8],
    files_count: i32,
    ttl_hours: i64,
) -> Result<CheckpointPatch, sqlx::Error> {
    sqlx::query_as::<_, CheckpointPatch>(
        r#"
        INSERT INTO checkpoint_patches (
            task_id, checkpoint_id, base_commit_sha, patch_data,
            patch_size_bytes, files_count, expires_at
        )
        VALUES ($1, $2, $3, $4, $5, $6, NOW() + INTERVAL '1 hour' * $7)
        RETURNING *
        "#,
    )
    .bind(task_id)
    .bind(checkpoint_id)
    .bind(base_commit_sha)
    .bind(patch_data)
    .bind(patch_data.len() as i32)
    .bind(files_count)
    .bind(ttl_hours)
    .fetch_one(pool)
    .await
}

/// Get the latest checkpoint patch for a task.
pub async fn get_latest_checkpoint_patch(
    pool: &PgPool,
    task_id: Uuid,
) -> Result<Option<CheckpointPatch>, sqlx::Error> {
    sqlx::query_as::<_, CheckpointPatch>(
        r#"
        SELECT * FROM checkpoint_patches
        WHERE task_id = $1 AND expires_at > NOW()
        ORDER BY created_at DESC
        LIMIT 1
        "#,
    )
    .bind(task_id)
    .fetch_optional(pool)
    .await
}

/// Get a checkpoint patch by ID.
pub async fn get_checkpoint_patch(
    pool: &PgPool,
    id: Uuid,
) -> Result<Option<CheckpointPatch>, sqlx::Error> {
    sqlx::query_as::<_, CheckpointPatch>(
        "SELECT * FROM checkpoint_patches WHERE id = $1",
    )
    .bind(id)
    .fetch_optional(pool)
    .await
}

/// List all checkpoint patches for a task (without patch data for efficiency).
pub async fn list_checkpoint_patches(
    pool: &PgPool,
    task_id: Uuid,
) -> Result<Vec<CheckpointPatchInfo>, sqlx::Error> {
    sqlx::query_as::<_, CheckpointPatchInfo>(
        r#"
        SELECT id, task_id, checkpoint_id, base_commit_sha,
               patch_size_bytes, files_count, created_at, expires_at
        FROM checkpoint_patches
        WHERE task_id = $1
        ORDER BY created_at DESC
        "#,
    )
    .bind(task_id)
    .fetch_all(pool)
    .await
}

/// Delete expired checkpoint patches.
/// Returns the number of deleted patches.
pub async fn cleanup_expired_checkpoint_patches(
    pool: &PgPool,
) -> Result<i64, sqlx::Error> {
    let result = sqlx::query("DELETE FROM checkpoint_patches WHERE expires_at < NOW()")
        .execute(pool)
        .await?;
    Ok(result.rows_affected() as i64)
}

/// Delete all checkpoint patches for a task.
pub async fn delete_checkpoint_patches_for_task(
    pool: &PgPool,
    task_id: Uuid,
) -> Result<i64, sqlx::Error> {
    let result = sqlx::query("DELETE FROM checkpoint_patches WHERE task_id = $1")
        .bind(task_id)
        .execute(pool)
        .await?;
    Ok(result.rows_affected() as i64)
}

// =============================================================================
// Red Team Notifications
// =============================================================================
// =============================================================================
// Supervisor Status API Helpers
// =============================================================================

/// Get supervisor status for a contract.
/// Returns combined information from supervisor_states and tasks tables.
pub async fn get_supervisor_status(
    pool: &PgPool,
    contract_id: Uuid,
    owner_id: Uuid,
) -> Result<Option<SupervisorStatusInfo>, sqlx::Error> {
    // Query to get supervisor status by joining supervisor_states with tasks
    sqlx::query_as::<_, SupervisorStatusInfo>(
        r#"
        SELECT
            ss.task_id,
            COALESCE(t.status, 'unknown') as supervisor_state,
            ss.phase,
            t.progress_summary as current_activity,
            ss.pending_task_ids,
            ss.last_activity as last_heartbeat,
            t.status as task_status,
            t.daemon_id IS NOT NULL as is_running
        FROM supervisor_states ss
        JOIN tasks t ON t.id = ss.task_id
        WHERE ss.contract_id = $1
          AND t.owner_id = $2
        "#,
    )
    .bind(contract_id)
    .bind(owner_id)
    .fetch_optional(pool)
    .await
}

/// Internal struct to hold supervisor status query result
#[derive(Debug, Clone, sqlx::FromRow)]
pub struct SupervisorStatusInfo {
    pub task_id: Uuid,
    pub supervisor_state: String,
    pub phase: String,
    pub current_activity: Option<String>,
    #[sqlx(try_from = "Vec<Uuid>")]
    pub pending_task_ids: Vec<Uuid>,
    pub last_heartbeat: chrono::DateTime<chrono::Utc>,
    pub task_status: String,
    pub is_running: bool,
}

/// Get supervisor activity history from history_events table.
/// This provides a timeline of supervisor activities that can serve as "heartbeats".
pub async fn get_supervisor_activity_history(
    pool: &PgPool,
    contract_id: Uuid,
    limit: i32,
    offset: i32,
) -> Result<Vec<SupervisorActivityEntry>, sqlx::Error> {
    sqlx::query_as::<_, SupervisorActivityEntry>(
        r#"
        SELECT
            created_at as timestamp,
            COALESCE(event_subtype, 'activity') as state,
            event_data->>'activity' as activity,
            (event_data->>'progress')::INTEGER as progress,
            COALESCE(phase, 'unknown') as phase,
            CASE
                WHEN event_data->'pending_task_ids' IS NOT NULL
                THEN ARRAY(SELECT jsonb_array_elements_text(event_data->'pending_task_ids'))::UUID[]
                ELSE ARRAY[]::UUID[]
            END as pending_task_ids
        FROM history_events
        WHERE contract_id = $1
          AND event_type IN ('supervisor', 'phase', 'task')
        ORDER BY created_at DESC
        LIMIT $2 OFFSET $3
        "#,
    )
    .bind(contract_id)
    .bind(limit)
    .bind(offset)
    .fetch_all(pool)
    .await
}

/// Internal struct to hold supervisor activity entry
#[derive(Debug, Clone, sqlx::FromRow)]
pub struct SupervisorActivityEntry {
    pub timestamp: chrono::DateTime<chrono::Utc>,
    pub state: String,
    pub activity: Option<String>,
    pub progress: Option<i32>,
    pub phase: String,
    #[sqlx(try_from = "Vec<Uuid>")]
    pub pending_task_ids: Vec<Uuid>,
}

/// Count total supervisor activity history entries for a contract.
pub async fn count_supervisor_activity_history(
    pool: &PgPool,
    contract_id: Uuid,
) -> Result<i64, sqlx::Error> {
    let result: (i64,) = sqlx::query_as(
        r#"
        SELECT COUNT(*)
        FROM history_events
        WHERE contract_id = $1
          AND event_type IN ('supervisor', 'phase', 'task')
        "#,
    )
    .bind(contract_id)
    .fetch_one(pool)
    .await?;
    Ok(result.0)
}

/// Update supervisor state last_activity timestamp.
/// This acts as a "sync" operation to refresh the supervisor's heartbeat.
pub async fn sync_supervisor_state(
    pool: &PgPool,
    contract_id: Uuid,
) -> Result<Option<SupervisorState>, sqlx::Error> {
    sqlx::query_as::<_, SupervisorState>(
        r#"
        UPDATE supervisor_states
        SET last_activity = NOW(),
            updated_at = NOW()
        WHERE contract_id = $1
        RETURNING *
        "#,
    )
    .bind(contract_id)
    .fetch_optional(pool)
    .await
}

// =============================================================================
// Chain Operations (DAG of contracts for multi-contract orchestration)
// =============================================================================

/// Create a new chain for a specific owner.
pub async fn create_chain_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
    req: CreateChainRequest,
) -> Result<Chain, sqlx::Error> {
    let loop_enabled = req.loop_enabled.unwrap_or(false);
    let loop_max_iterations = req.loop_max_iterations.unwrap_or(10);

    sqlx::query_as::<_, Chain>(
        r#"
        INSERT INTO chains (owner_id, name, description, loop_enabled, loop_max_iterations, loop_progress_check)
        VALUES ($1, $2, $3, $4, $5, $6)
        RETURNING *
        "#,
    )
    .bind(owner_id)
    .bind(&req.name)
    .bind(&req.description)
    .bind(loop_enabled)
    .bind(loop_max_iterations)
    .bind(&req.loop_progress_check)
    .fetch_one(pool)
    .await
}

/// Get a chain by ID, scoped to owner.
pub async fn get_chain_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<Option<Chain>, sqlx::Error> {
    sqlx::query_as::<_, Chain>(
        r#"
        SELECT *
        FROM chains
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .fetch_optional(pool)
    .await
}

/// Get a chain by ID (no owner check - for internal use).
pub async fn get_chain(pool: &PgPool, id: Uuid) -> Result<Option<Chain>, sqlx::Error> {
    sqlx::query_as::<_, Chain>(
        r#"
        SELECT *
        FROM chains
        WHERE id = $1
        "#,
    )
    .bind(id)
    .fetch_optional(pool)
    .await
}

/// List chains for a specific owner.
pub async fn list_chains_for_owner(
    pool: &PgPool,
    owner_id: Uuid,
) -> Result<Vec<ChainSummary>, sqlx::Error> {
    sqlx::query_as::<_, ChainSummary>(
        r#"
        SELECT
            c.id,
            c.name,
            c.description,
            c.status,
            c.loop_enabled,
            c.loop_current_iteration,
            COUNT(DISTINCT cc.contract_id) as contract_count,
            COUNT(DISTINCT CASE WHEN con.status = 'completed' THEN cc.contract_id END) as completed_count,
            c.version,
            c.created_at
        FROM chains c
        LEFT JOIN chain_contracts cc ON cc.chain_id = c.id
        LEFT JOIN contracts con ON con.id = cc.contract_id
        WHERE c.owner_id = $1
        GROUP BY c.id
        ORDER BY c.created_at DESC
        "#,
    )
    .bind(owner_id)
    .fetch_all(pool)
    .await
}

/// Update a chain.
pub async fn update_chain_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
    req: UpdateChainRequest,
) -> Result<Chain, RepositoryError> {
    // First get current version if optimistic locking requested
    if let Some(expected_version) = req.version {
        let current: Option<(i32,)> = sqlx::query_as(
            "SELECT version FROM chains WHERE id = $1 AND owner_id = $2",
        )
        .bind(id)
        .bind(owner_id)
        .fetch_optional(pool)
        .await?;

        if let Some((actual_version,)) = current {
            if actual_version != expected_version {
                return Err(RepositoryError::VersionConflict {
                    expected: expected_version,
                    actual: actual_version,
                });
            }
        }
    }

    let result = sqlx::query_as::<_, Chain>(
        r#"
        UPDATE chains
        SET
            name = COALESCE($3, name),
            description = COALESCE($4, description),
            status = COALESCE($5, status),
            loop_enabled = COALESCE($6, loop_enabled),
            loop_max_iterations = COALESCE($7, loop_max_iterations),
            loop_progress_check = COALESCE($8, loop_progress_check),
            version = version + 1,
            updated_at = NOW()
        WHERE id = $1 AND owner_id = $2
        RETURNING *
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .bind(&req.name)
    .bind(&req.description)
    .bind(&req.status)
    .bind(req.loop_enabled)
    .bind(req.loop_max_iterations)
    .bind(&req.loop_progress_check)
    .fetch_one(pool)
    .await?;

    Ok(result)
}

/// Delete (archive) a chain.
pub async fn delete_chain_for_owner(
    pool: &PgPool,
    id: Uuid,
    owner_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        UPDATE chains
        SET status = 'archived', updated_at = NOW()
        WHERE id = $1 AND owner_id = $2
        "#,
    )
    .bind(id)
    .bind(owner_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Add a contract to a chain.
pub async fn add_contract_to_chain(
    pool: &PgPool,
    chain_id: Uuid,
    contract_id: Uuid,
    depends_on: Vec<Uuid>,
    order_index: i32,
    editor_x: Option<f64>,
    editor_y: Option<f64>,
) -> Result<ChainContract, sqlx::Error> {
    // Also update the contract's chain_id
    sqlx::query("UPDATE contracts SET chain_id = $1 WHERE id = $2")
        .bind(chain_id)
        .bind(contract_id)
        .execute(pool)
        .await?;

    sqlx::query_as::<_, ChainContract>(
        r#"
        INSERT INTO chain_contracts (chain_id, contract_id, depends_on, order_index, editor_x, editor_y)
        VALUES ($1, $2, $3, $4, $5, $6)
        ON CONFLICT (chain_id, contract_id) DO UPDATE SET
            depends_on = EXCLUDED.depends_on,
            order_index = EXCLUDED.order_index,
            editor_x = EXCLUDED.editor_x,
            editor_y = EXCLUDED.editor_y
        RETURNING *
        "#,
    )
    .bind(chain_id)
    .bind(contract_id)
    .bind(&depends_on)
    .bind(order_index)
    .bind(editor_x)
    .bind(editor_y)
    .fetch_one(pool)
    .await
}

/// Remove a contract from a chain.
pub async fn remove_contract_from_chain(
    pool: &PgPool,
    chain_id: Uuid,
    contract_id: Uuid,
) -> Result<bool, sqlx::Error> {
    // Clear the contract's chain_id
    sqlx::query("UPDATE contracts SET chain_id = NULL WHERE id = $1 AND chain_id = $2")
        .bind(contract_id)
        .bind(chain_id)
        .execute(pool)
        .await?;

    let result = sqlx::query(
        r#"
        DELETE FROM chain_contracts
        WHERE chain_id = $1 AND contract_id = $2
        "#,
    )
    .bind(chain_id)
    .bind(contract_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// List contracts in a chain with their details.
pub async fn list_chain_contracts(
    pool: &PgPool,
    chain_id: Uuid,
) -> Result<Vec<ChainContractDetail>, sqlx::Error> {
    sqlx::query_as::<_, ChainContractDetail>(
        r#"
        SELECT
            cc.id as chain_contract_id,
            cc.contract_id,
            c.name as contract_name,
            c.status as contract_status,
            c.phase as contract_phase,
            cc.depends_on,
            cc.order_index,
            cc.editor_x,
            cc.editor_y
        FROM chain_contracts cc
        JOIN contracts c ON c.id = cc.contract_id
        WHERE cc.chain_id = $1
        ORDER BY cc.order_index ASC
        "#,
    )
    .bind(chain_id)
    .fetch_all(pool)
    .await
}

/// Get chain with all contracts for detail view.
pub async fn get_chain_with_contracts(
    pool: &PgPool,
    chain_id: Uuid,
    owner_id: Uuid,
) -> Result<Option<ChainWithContracts>, sqlx::Error> {
    let chain = get_chain_for_owner(pool, chain_id, owner_id).await?;

    match chain {
        Some(chain) => {
            let contracts = list_chain_contracts(pool, chain_id).await?;
            let repositories = list_chain_repositories(pool, chain_id).await?;
            Ok(Some(ChainWithContracts {
                chain,
                contracts,
                repositories,
            }))
        }
        None => Ok(None),
    }
}

// =============================================================================
// Chain Repository Operations
// =============================================================================

/// List all repositories for a chain.
pub async fn list_chain_repositories(
    pool: &PgPool,
    chain_id: Uuid,
) -> Result<Vec<ChainRepository>, sqlx::Error> {
    sqlx::query_as::<_, ChainRepository>(
        r#"
        SELECT *
        FROM chain_repositories
        WHERE chain_id = $1
        ORDER BY is_primary DESC, created_at ASC
        "#,
    )
    .bind(chain_id)
    .fetch_all(pool)
    .await
}

/// Get a chain repository by ID.
pub async fn get_chain_repository(
    pool: &PgPool,
    chain_id: Uuid,
    repository_id: Uuid,
) -> Result<Option<ChainRepository>, sqlx::Error> {
    sqlx::query_as::<_, ChainRepository>(
        r#"
        SELECT *
        FROM chain_repositories
        WHERE id = $1 AND chain_id = $2
        "#,
    )
    .bind(repository_id)
    .bind(chain_id)
    .fetch_optional(pool)
    .await
}

/// Add a repository to a chain.
pub async fn add_chain_repository(
    pool: &PgPool,
    chain_id: Uuid,
    req: &AddChainRepositoryRequest,
) -> Result<ChainRepository, sqlx::Error> {
    // If is_primary, clear other primaries first
    if req.is_primary {
        sqlx::query(
            r#"
            UPDATE chain_repositories
            SET is_primary = false, updated_at = NOW()
            WHERE chain_id = $1 AND is_primary = true
            "#,
        )
        .bind(chain_id)
        .execute(pool)
        .await?;
    }

    sqlx::query_as::<_, ChainRepository>(
        r#"
        INSERT INTO chain_repositories (chain_id, name, repository_url, local_path, source_type, status, is_primary)
        VALUES ($1, $2, $3, $4, $5, 'ready', $6)
        RETURNING *
        "#,
    )
    .bind(chain_id)
    .bind(&req.name)
    .bind(&req.repository_url)
    .bind(&req.local_path)
    .bind(&req.source_type)
    .bind(req.is_primary)
    .fetch_one(pool)
    .await
}

/// Delete a repository from a chain.
pub async fn delete_chain_repository(
    pool: &PgPool,
    chain_id: Uuid,
    repository_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query(
        r#"
        DELETE FROM chain_repositories
        WHERE id = $1 AND chain_id = $2
        "#,
    )
    .bind(repository_id)
    .bind(chain_id)
    .execute(pool)
    .await?;

    Ok(result.rows_affected() > 0)
}

/// Set a repository as primary for a chain.
pub async fn set_chain_repository_primary(
    pool: &PgPool,
    chain_id: Uuid,
    repository_id: Uuid,
) -> Result<ChainRepository, sqlx::Error> {
    // Clear existing primary
    sqlx::query(
        r#"
        UPDATE chain_repositories
        SET is_primary = false, updated_at = NOW()
        WHERE chain_id = $1 AND is_primary = true
        "#,
    )
    .bind(chain_id)
    .execute(pool)
    .await?;

    // Set new primary
    sqlx::query_as::<_, ChainRepository>(
        r#"
        UPDATE chain_repositories
        SET is_primary = true, updated_at = NOW()
        WHERE id = $1 AND chain_id = $2
        RETURNING *
        "#,
    )
    .bind(repository_id)
    .bind(chain_id)
    .fetch_one(pool)
    .await
}

/// Get the primary repository for a chain.
pub async fn get_chain_primary_repository(
    pool: &PgPool,
    chain_id: Uuid,
) -> Result<Option<ChainRepository>, sqlx::Error> {
    sqlx::query_as::<_, ChainRepository>(
        r#"
        SELECT *
        FROM chain_repositories
        WHERE chain_id = $1 AND is_primary = true
        "#,
    )
    .bind(chain_id)
    .fetch_optional(pool)
    .await
}

/// Get chain graph structure for visualization.
pub async fn get_chain_graph(
    pool: &PgPool,
    chain_id: Uuid,
) -> Result<Option<ChainGraphResponse>, sqlx::Error> {
    let chain = get_chain(pool, chain_id).await?;

    match chain {
        Some(chain) => {
            let contracts = list_chain_contracts(pool, chain_id).await?;

            let nodes: Vec<ChainGraphNode> = contracts
                .iter()
                .map(|c| ChainGraphNode {
                    id: c.chain_contract_id,
                    contract_id: c.contract_id,
                    name: c.contract_name.clone(),
                    status: c.contract_status.clone(),
                    phase: c.contract_phase.clone(),
                    x: c.editor_x.unwrap_or(0.0),
                    y: c.editor_y.unwrap_or(0.0),
                })
                .collect();

            let mut edges: Vec<ChainGraphEdge> = Vec::new();
            for contract in &contracts {
                for dep_id in &contract.depends_on {
                    // Find the chain_contract_id for this dependency
                    if let Some(dep) = contracts.iter().find(|c| c.contract_id == *dep_id) {
                        edges.push(ChainGraphEdge {
                            from: dep.chain_contract_id,
                            to: contract.chain_contract_id,
                        });
                    }
                }
            }

            Ok(Some(ChainGraphResponse {
                chain_id: chain.id,
                chain_name: chain.name,
                chain_status: chain.status,
                nodes,
                edges,
            }))
        }
        None => Ok(None),
    }
}

/// Record a chain event.
pub async fn record_chain_event(
    pool: &PgPool,
    chain_id: Uuid,
    event_type: &str,
    contract_id: Option<Uuid>,
    event_data: Option<serde_json::Value>,
) -> Result<ChainEvent, sqlx::Error> {
    sqlx::query_as::<_, ChainEvent>(
        r#"
        INSERT INTO chain_events (chain_id, event_type, contract_id, event_data)
        VALUES ($1, $2, $3, $4)
        RETURNING *
        "#,
    )
    .bind(chain_id)
    .bind(event_type)
    .bind(contract_id)
    .bind(event_data)
    .fetch_one(pool)
    .await
}

/// List chain events.
pub async fn list_chain_events(
    pool: &PgPool,
    chain_id: Uuid,
) -> Result<Vec<ChainEvent>, sqlx::Error> {
    sqlx::query_as::<_, ChainEvent>(
        r#"
        SELECT *
        FROM chain_events
        WHERE chain_id = $1
        ORDER BY created_at DESC
        "#,
    )
    .bind(chain_id)
    .fetch_all(pool)
    .await
}

/// Increment chain loop iteration.
pub async fn increment_chain_loop(pool: &PgPool, chain_id: Uuid) -> Result<Chain, sqlx::Error> {
    sqlx::query_as::<_, Chain>(
        r#"
        UPDATE chains
        SET loop_current_iteration = COALESCE(loop_current_iteration, 0) + 1,
            updated_at = NOW()
        WHERE id = $1
        RETURNING *
        "#,
    )
    .bind(chain_id)
    .fetch_one(pool)
    .await
}

/// Mark a chain as completed.
pub async fn complete_chain(pool: &PgPool, chain_id: Uuid) -> Result<Chain, sqlx::Error> {
    sqlx::query_as::<_, Chain>(
        r#"
        UPDATE chains
        SET status = 'completed',
            updated_at = NOW()
        WHERE id = $1
        RETURNING *
        "#,
    )
    .bind(chain_id)
    .fetch_one(pool)
    .await
}

/// Get contracts in a chain that have no pending dependencies (ready to start).
/// Returns contracts where all depends_on contracts are completed.
pub async fn get_ready_chain_contracts(
    pool: &PgPool,
    chain_id: Uuid,
) -> Result<Vec<ChainContractDetail>, sqlx::Error> {
    sqlx::query_as::<_, ChainContractDetail>(
        r#"
        SELECT
            cc.id as chain_contract_id,
            cc.contract_id,
            c.name as contract_name,
            c.status as contract_status,
            c.phase as contract_phase,
            cc.depends_on,
            cc.order_index,
            cc.editor_x,
            cc.editor_y
        FROM chain_contracts cc
        JOIN contracts c ON c.id = cc.contract_id
        WHERE cc.chain_id = $1
          AND c.status = 'active'
          AND (
            -- No dependencies
            cc.depends_on IS NULL
            OR array_length(cc.depends_on, 1) IS NULL
            OR array_length(cc.depends_on, 1) = 0
            -- Or all dependencies completed
            OR NOT EXISTS (
                SELECT 1
                FROM unnest(cc.depends_on) AS dep_id
                JOIN contracts dep ON dep.id = dep_id
                WHERE dep.status != 'completed'
            )
          )
        ORDER BY cc.order_index ASC
        "#,
    )
    .bind(chain_id)
    .fetch_all(pool)
    .await
}

/// Check if all contracts in a chain are completed.
pub async fn is_chain_complete(pool: &PgPool, chain_id: Uuid) -> Result<bool, sqlx::Error> {
    let result: (i64,) = sqlx::query_as(
        r#"
        SELECT COUNT(*)
        FROM chain_contracts cc
        JOIN contracts c ON c.id = cc.contract_id
        WHERE cc.chain_id = $1
          AND c.status != 'completed'
        "#,
    )
    .bind(chain_id)
    .fetch_one(pool)
    .await?;

    Ok(result.0 == 0)
}

/// Get chain editor data for the GUI editor.
pub async fn get_chain_editor_data(
    pool: &PgPool,
    chain_id: Uuid,
    owner_id: Uuid,
) -> Result<Option<ChainEditorData>, sqlx::Error> {
    let chain = get_chain_for_owner(pool, chain_id, owner_id).await?;

    match chain {
        Some(chain) => {
            let contracts = list_chain_contracts(pool, chain_id).await?;
            let repositories = list_chain_repositories(pool, chain_id).await?;

            // Build nodes
            let nodes: Vec<ChainEditorNode> = contracts
                .iter()
                .map(|c| ChainEditorNode {
                    id: c.contract_id.to_string(),
                    x: c.editor_x.unwrap_or(0.0),
                    y: c.editor_y.unwrap_or(0.0),
                    contract: ChainEditorContract {
                        name: c.contract_name.clone(),
                        description: None, // Would need to join with full contract data
                        contract_type: "simple".to_string(),
                        phases: vec!["plan".to_string(), "execute".to_string()],
                        tasks: vec![],
                        deliverables: vec![],
                    },
                })
                .collect();

            // Build edges
            let edges: Vec<ChainEditorEdge> = contracts
                .iter()
                .flat_map(|c| {
                    c.depends_on.iter().map(move |dep_id| ChainEditorEdge {
                        from: dep_id.to_string(),
                        to: c.contract_id.to_string(),
                    })
                })
                .collect();

            Ok(Some(ChainEditorData {
                id: Some(chain.id),
                name: chain.name,
                description: chain.description,
                repositories,
                loop_enabled: chain.loop_enabled,
                loop_max_iterations: chain.loop_max_iterations,
                loop_progress_check: chain.loop_progress_check,
                nodes,
                edges,
            }))
        }
        None => Ok(None),
    }
}

// =============================================================================
// Chain Contract Definition Operations
// =============================================================================

/// Create a new contract definition in a chain.
pub async fn create_chain_contract_definition(
    pool: &PgPool,
    chain_id: Uuid,
    req: AddContractDefinitionRequest,
) -> Result<ChainContractDefinition, sqlx::Error> {
    // Get the next order index
    let max_order: Option<i32> = sqlx::query_scalar(
        "SELECT MAX(order_index) FROM chain_contract_definitions WHERE chain_id = $1",
    )
    .bind(chain_id)
    .fetch_one(pool)
    .await?;

    let order_index = max_order.unwrap_or(-1) + 1;

    // Convert tasks, deliverables, and validation to JSON
    let tasks_json = req.tasks.as_ref().map(|t| serde_json::to_value(t).unwrap());
    let deliverables_json = req
        .deliverables
        .as_ref()
        .map(|d| serde_json::to_value(d).unwrap());
    let validation_json = req
        .validation
        .as_ref()
        .map(|v| serde_json::to_value(v).unwrap());
    let depends_on_names: Vec<String> = req.depends_on.unwrap_or_default();

    sqlx::query_as::<_, ChainContractDefinition>(
        r#"
        INSERT INTO chain_contract_definitions
            (chain_id, name, description, contract_type, initial_phase, depends_on_names, tasks, deliverables, validation, editor_x, editor_y, order_index)
        VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12)
        RETURNING *
        "#,
    )
    .bind(chain_id)
    .bind(&req.name)
    .bind(&req.description)
    .bind(&req.contract_type)
    .bind(&req.initial_phase)
    .bind(&depends_on_names)
    .bind(&tasks_json)
    .bind(&deliverables_json)
    .bind(&validation_json)
    .bind(req.editor_x)
    .bind(req.editor_y)
    .bind(order_index)
    .fetch_one(pool)
    .await
}

/// List all contract definitions in a chain.
pub async fn list_chain_contract_definitions(
    pool: &PgPool,
    chain_id: Uuid,
) -> Result<Vec<ChainContractDefinition>, sqlx::Error> {
    sqlx::query_as::<_, ChainContractDefinition>(
        r#"
        SELECT * FROM chain_contract_definitions
        WHERE chain_id = $1
        ORDER BY order_index ASC
        "#,
    )
    .bind(chain_id)
    .fetch_all(pool)
    .await
}

/// Get a specific contract definition.
pub async fn get_chain_contract_definition(
    pool: &PgPool,
    definition_id: Uuid,
) -> Result<Option<ChainContractDefinition>, sqlx::Error> {
    sqlx::query_as::<_, ChainContractDefinition>(
        "SELECT * FROM chain_contract_definitions WHERE id = $1",
    )
    .bind(definition_id)
    .fetch_optional(pool)
    .await
}

/// Update a contract definition.
pub async fn update_chain_contract_definition(
    pool: &PgPool,
    definition_id: Uuid,
    req: UpdateContractDefinitionRequest,
) -> Result<ChainContractDefinition, sqlx::Error> {
    let tasks_json = req.tasks.as_ref().map(|t| serde_json::to_value(t).unwrap());
    let deliverables_json = req
        .deliverables
        .as_ref()
        .map(|d| serde_json::to_value(d).unwrap());
    let validation_json = req
        .validation
        .as_ref()
        .map(|v| serde_json::to_value(v).unwrap());

    sqlx::query_as::<_, ChainContractDefinition>(
        r#"
        UPDATE chain_contract_definitions SET
            name = COALESCE($2, name),
            description = COALESCE($3, description),
            contract_type = COALESCE($4, contract_type),
            initial_phase = COALESCE($5, initial_phase),
            depends_on_names = COALESCE($6, depends_on_names),
            tasks = COALESCE($7, tasks),
            deliverables = COALESCE($8, deliverables),
            validation = COALESCE($9, validation),
            editor_x = COALESCE($10, editor_x),
            editor_y = COALESCE($11, editor_y)
        WHERE id = $1
        RETURNING *
        "#,
    )
    .bind(definition_id)
    .bind(&req.name)
    .bind(&req.description)
    .bind(&req.contract_type)
    .bind(&req.initial_phase)
    .bind(&req.depends_on)
    .bind(&tasks_json)
    .bind(&deliverables_json)
    .bind(&validation_json)
    .bind(req.editor_x)
    .bind(req.editor_y)
    .fetch_one(pool)
    .await
}

/// Delete a contract definition.
pub async fn delete_chain_contract_definition(
    pool: &PgPool,
    definition_id: Uuid,
) -> Result<bool, sqlx::Error> {
    let result = sqlx::query("DELETE FROM chain_contract_definitions WHERE id = $1")
        .bind(definition_id)
        .execute(pool)
        .await?;
    Ok(result.rows_affected() > 0)
}

/// Get definitions that are ready to be instantiated (all dependencies are satisfied).
/// A definition is ready if all definitions it depends on have been instantiated as contracts
/// and those contracts have completed.
pub async fn get_ready_definitions(
    pool: &PgPool,
    chain_id: Uuid,
) -> Result<Vec<ChainContractDefinition>, sqlx::Error> {
    sqlx::query_as::<_, ChainContractDefinition>(
        r#"
        SELECT d.*
        FROM chain_contract_definitions d
        WHERE d.chain_id = $1
        -- Not already instantiated
        AND NOT EXISTS (
            SELECT 1 FROM chain_contracts cc
            WHERE cc.definition_id = d.id
        )
        -- All dependencies satisfied (either no deps, or all deps have completed contracts)
        AND (
            cardinality(d.depends_on_names) = 0
            OR NOT EXISTS (
                SELECT 1 FROM unnest(d.depends_on_names) AS dep_name
                WHERE NOT EXISTS (
                    SELECT 1 FROM chain_contract_definitions dep_def
                    JOIN chain_contracts cc ON cc.definition_id = dep_def.id
                    JOIN contracts c ON c.id = cc.contract_id
                    WHERE dep_def.chain_id = d.chain_id
                    AND dep_def.name = dep_name
                    AND c.status = 'completed'
                )
            )
        )
        ORDER BY d.order_index ASC
        "#,
    )
    .bind(chain_id)
    .fetch_all(pool)
    .await
}

/// Get the definition graph for visualization.
pub async fn get_chain_definition_graph(
    pool: &PgPool,
    chain_id: Uuid,
) -> Result<Option<ChainDefinitionGraphResponse>, sqlx::Error> {
    let chain = sqlx::query_as::<_, Chain>("SELECT * FROM chains WHERE id = $1")
        .bind(chain_id)
        .fetch_optional(pool)
        .await?;

    let Some(chain) = chain else {
        return Ok(None);
    };

    let definitions = list_chain_contract_definitions(pool, chain_id).await?;

    // Get instantiated contracts for each definition
    let chain_contracts = list_chain_contracts(pool, chain_id).await?;
    let instantiated: std::collections::HashMap<Uuid, &ChainContractDetail> = chain_contracts
        .iter()
        .filter_map(|cc| {
            // Find definition_id from cc - we need to query this
            // For now, match by name
            definitions
                .iter()
                .find(|d| d.name == cc.contract_name)
                .map(|d| (d.id, cc))
        })
        .collect();

    let nodes: Vec<ChainDefinitionGraphNode> = definitions
        .iter()
        .map(|d| {
            let cc = instantiated.get(&d.id);
            ChainDefinitionGraphNode {
                id: d.id,
                name: d.name.clone(),
                contract_type: d.contract_type.clone(),
                x: d.editor_x.unwrap_or(0.0),
                y: d.editor_y.unwrap_or(0.0),
                is_instantiated: cc.is_some(),
                contract_id: cc.map(|c| c.contract_id),
                contract_status: cc.map(|c| c.contract_status.clone()),
            }
        })
        .collect();

    // Build edges from depends_on_names
    let name_to_id: std::collections::HashMap<&str, Uuid> =
        definitions.iter().map(|d| (d.name.as_str(), d.id)).collect();

    let edges: Vec<ChainGraphEdge> = definitions
        .iter()
        .flat_map(|d| {
            let target_id = d.id;
            let name_to_id = &name_to_id;
            d.depends_on_names.iter().filter_map(move |dep_name| {
                name_to_id
                    .get(dep_name.as_str())
                    .map(|&from_id| ChainGraphEdge { from: from_id, to: target_id })
            })
        })
        .collect();

    Ok(Some(ChainDefinitionGraphResponse {
        chain_id: chain.id,
        chain_name: chain.name,
        chain_status: chain.status,
        nodes,
        edges,
    }))
}

/// Update chain status.
pub async fn update_chain_status(
    pool: &PgPool,
    chain_id: Uuid,
    status: &str,
) -> Result<(), sqlx::Error> {
    sqlx::query("UPDATE chains SET status = $2, updated_at = NOW() WHERE id = $1")
        .bind(chain_id)
        .bind(status)
        .execute(pool)
        .await?;
    Ok(())
}

// =============================================================================
// Chain Progression
// =============================================================================

/// Result of chain progression check
#[derive(Debug)]
pub struct ChainProgressionResult {
    /// Contracts created from ready definitions
    pub contracts_created: Vec<Uuid>,
    /// Whether all definitions are instantiated and completed (chain is done)
    pub chain_completed: bool,
}

/// Progress a chain by creating contracts from ready definitions.
///
/// This is called when a contract in the chain completes. It:
/// 1. Finds definitions whose dependencies are all satisfied (completed)
/// 2. Creates contracts from those definitions
/// 3. Links them to the chain
/// 4. Checks if chain is complete (all definitions instantiated and completed)
pub async fn progress_chain(
    pool: &PgPool,
    chain_id: Uuid,
    owner_id: Uuid,
) -> Result<ChainProgressionResult, sqlx::Error> {
    let mut contracts_created = Vec::new();

    // Get all definitions for this chain
    let definitions = list_chain_contract_definitions(pool, chain_id).await?;
    if definitions.is_empty() {
        return Ok(ChainProgressionResult {
            contracts_created: vec![],
            chain_completed: true,
        });
    }

    // Get existing chain contracts to know what's already instantiated
    let chain_contracts = list_chain_contracts(pool, chain_id).await?;

    // Build a map of definition name -> instantiated contract status
    let instantiated: std::collections::HashMap<String, Option<String>> = chain_contracts
        .iter()
        .map(|cc| (cc.contract_name.clone(), Some(cc.contract_status.clone())))
        .collect();

    // Find definitions that are ready to be instantiated:
    // - Not yet instantiated
    // - All dependencies are instantiated AND completed
    for def in &definitions {
        // Skip if already instantiated
        if instantiated.contains_key(&def.name) {
            continue;
        }

        // Check if all dependencies are completed
        let deps_satisfied = def.depends_on_names.iter().all(|dep_name| {
            instantiated
                .get(dep_name)
                .map(|status| status.as_deref() == Some("completed"))
                .unwrap_or(false)
        });

        // Root definitions (no dependencies) are always ready
        let is_root = def.depends_on_names.is_empty();

        if is_root || deps_satisfied {
            // Create contract from definition
            match create_contract_from_definition(pool, chain_id, owner_id, def).await {
                Ok(contract_id) => {
                    contracts_created.push(contract_id);
                    tracing::info!(
                        chain_id = %chain_id,
                        definition_name = %def.name,
                        contract_id = %contract_id,
                        "Created contract from chain definition"
                    );
                }
                Err(e) => {
                    tracing::error!(
                        chain_id = %chain_id,
                        definition_name = %def.name,
                        error = %e,
                        "Failed to create contract from chain definition"
                    );
                }
            }
        }
    }

    // Check if chain is complete (all definitions instantiated and completed)
    let updated_contracts = list_chain_contracts(pool, chain_id).await?;
    let all_instantiated = definitions.len() == updated_contracts.len();
    let all_completed = updated_contracts
        .iter()
        .all(|cc| cc.contract_status == "completed");
    let chain_completed = all_instantiated && all_completed;

    if chain_completed {
        update_chain_status(pool, chain_id, "completed").await?;
        tracing::info!(chain_id = %chain_id, "Chain completed - all contracts done");
    }

    Ok(ChainProgressionResult {
        contracts_created,
        chain_completed,
    })
}

/// Task definition parsed from JSON (matches chain YAML format)
#[derive(Debug, Clone, serde::Deserialize)]
struct ChainTaskDef {
    name: String,
    plan: String,
}

/// Validation config parsed from definition JSON
#[derive(Debug, Clone, serde::Deserialize)]
#[serde(rename_all = "camelCase")]
struct ValidationConfig {
    #[serde(default)]
    check_deliverables: bool,
    #[serde(default)]
    run_tests: bool,
    check_content: Option<String>,
    #[serde(default = "default_on_failure_str")]
    on_failure: String,
    #[serde(default = "default_max_retries_val")]
    max_retries: i32,
}

fn default_on_failure_str() -> String {
    "block".to_string()
}

fn default_max_retries_val() -> i32 {
    3
}

/// Generate a validation plan for a checkpoint contract.
fn generate_checkpoint_plan(
    def: &ChainContractDefinition,
    upstream_contracts: &[&ChainContractDetail],
    validation: &ValidationConfig,
) -> String {
    let upstream_names: Vec<&str> = upstream_contracts.iter().map(|c| c.contract_name.as_str()).collect();

    let mut plan = format!(
        r#"# Checkpoint Validation: {}

You are validating the outputs of upstream contracts before allowing downstream work to proceed.

## Upstream Contracts to Validate
{}

"#,
        def.name,
        upstream_names.iter().map(|n| format!("- {}", n)).collect::<Vec<_>>().join("\n")
    );

    // Add deliverables check section
    if validation.check_deliverables {
        plan.push_str(r#"## Deliverables Check
Verify that all required deliverables from upstream contracts exist and are properly completed.

Use the makima CLI to check contract status:
```bash
makima contract status <contract_id>
```

For each upstream contract, verify:
1. Contract status is "completed"
2. All required deliverables are marked as complete
3. Deliverable content exists and is not empty

"#);
    }

    // Add tests check section
    if validation.run_tests {
        plan.push_str(r#"## Tests Check
Run the test suite to verify the codebase is in a good state.

```bash
# Run tests appropriate for the project type
npm test       # for Node.js projects
cargo test     # for Rust projects
pytest         # for Python projects
go test ./...  # for Go projects
```

Verify:
1. All tests pass
2. No new test failures introduced
3. Test coverage is acceptable

"#);
    }

    // Add custom content check section
    if let Some(content_check) = &validation.check_content {
        plan.push_str(&format!(r#"## Custom Validation Criteria
{}

"#, content_check));
    }

    // Add validation result section
    plan.push_str(&format!(r#"## Reporting Results

After completing all validation checks, you must report the result:

**If ALL checks pass:**
Mark this checkpoint contract as completed using:
```bash
makima supervisor complete
```

**If ANY check fails (on_failure: "{}"):**
"#, validation.on_failure));

    match validation.on_failure.as_str() {
        "block" => plan.push_str(r#"
- Document the failure reason clearly
- Do NOT mark the contract as complete
- The chain will be blocked until issues are resolved manually
"#),
        "retry" => plan.push_str(&format!(r#"
- Document the failure reason
- Request retry of the failed upstream contract (max {} retries)
- Use: `makima supervisor ask "Upstream validation failed. Retry?" --choices "Yes,No"`
"#, validation.max_retries)),
        "warn" => plan.push_str(r#"
- Document the warning/issue found
- Mark the contract as complete anyway (downstream will proceed)
- Log the warning for visibility
"#),
        _ => plan.push_str(r#"
- Document the failure reason
- Do NOT mark the contract as complete
"#),
    }

    plan.push_str(r#"
## Begin Validation

Start by checking the status of each upstream contract, then proceed with the validation criteria above.
"#);

    plan
}

/// Create a contract from a chain definition.
async fn create_contract_from_definition(
    pool: &PgPool,
    chain_id: Uuid,
    owner_id: Uuid,
    def: &ChainContractDefinition,
) -> Result<Uuid, sqlx::Error> {
    // Get the existing contracts to find dependency info
    let existing_contracts = list_chain_contracts(pool, chain_id).await?;
    let name_to_contract: std::collections::HashMap<&str, &ChainContractDetail> = existing_contracts
        .iter()
        .map(|cc| (cc.contract_name.as_str(), cc))
        .collect();

    // Resolve dependency names to contract details
    let upstream_contracts: Vec<&ChainContractDetail> = def
        .depends_on_names
        .iter()
        .filter_map(|name| name_to_contract.get(name.as_str()).copied())
        .collect();

    // Create the contract request with basic fields
    let req = CreateContractRequest {
        name: def.name.clone(),
        description: def.description.clone(),
        contract_type: Some(def.contract_type.clone()),
        initial_phase: def.initial_phase.clone(),
        template_id: None,
        autonomous_loop: None,
        phase_guard: None,
        local_only: None,
        auto_merge_local: None,
    };

    // Create the contract
    let contract = create_contract_for_owner(pool, owner_id, req).await?;

    // For checkpoint contracts, generate a validation plan
    if def.contract_type == "checkpoint" {
        // Parse validation config
        let validation: ValidationConfig = def
            .validation
            .as_ref()
            .and_then(|v| serde_json::from_value(v.clone()).ok())
            .unwrap_or(ValidationConfig {
                check_deliverables: true,
                run_tests: false,
                check_content: None,
                on_failure: default_on_failure_str(),
                max_retries: default_max_retries_val(),
            });

        // Generate validation plan
        let validation_plan = generate_checkpoint_plan(def, &upstream_contracts, &validation);

        // Create a supervisor task with the validation plan
        let task_req = CreateTaskRequest {
            contract_id: Some(contract.id),
            name: format!("Validate: {}", def.name),
            description: Some("Checkpoint validation task".to_string()),
            plan: validation_plan,
            parent_task_id: None,
            is_supervisor: true, // Checkpoint uses supervisor task for validation
            priority: 0,
            repository_url: None,
            base_branch: None,
            target_branch: None,
            merge_mode: None,
            target_repo_path: None,
            completion_action: None,
            continue_from_task_id: None,
            copy_files: None,
            checkpoint_sha: None,
            branched_from_task_id: None,
            conversation_history: None,
            supervisor_worktree_task_id: None,
        };

        if let Err(e) = create_task_for_owner(pool, owner_id, task_req).await {
            tracing::warn!(
                contract_id = %contract.id,
                error = %e,
                "Failed to create validation task for checkpoint contract"
            );
        }

        // Set initial validation status
        sqlx::query(
            "UPDATE chain_contracts SET validation_status = 'pending' WHERE chain_id = $1 AND contract_id = $2",
        )
        .bind(chain_id)
        .bind(contract.id)
        .execute(pool)
        .await?;
    } else {
        // Parse and create tasks from definition for regular contracts
        if let Some(tasks_json) = &def.tasks {
            if let Ok(tasks) = serde_json::from_value::<Vec<ChainTaskDef>>(tasks_json.clone()) {
                for task_def in tasks {
                    let task_req = CreateTaskRequest {
                        contract_id: Some(contract.id),
                        name: task_def.name,
                        description: None,
                        plan: task_def.plan,
                        parent_task_id: None,
                        is_supervisor: false,
                        priority: 0,
                        repository_url: None,
                        base_branch: None,
                        target_branch: None,
                        merge_mode: None,
                        target_repo_path: None,
                        completion_action: None,
                        continue_from_task_id: None,
                        copy_files: None,
                        checkpoint_sha: None,
                        branched_from_task_id: None,
                        conversation_history: None,
                        supervisor_worktree_task_id: None,
                    };
                    if let Err(e) = create_task_for_owner(pool, owner_id, task_req).await {
                        tracing::warn!(
                            contract_id = %contract.id,
                            error = %e,
                            "Failed to create task from chain definition"
                        );
                    }
                }
            }
        }
    }

    // Resolve dependency names to contract IDs
    let depends_on: Vec<Uuid> = upstream_contracts.iter().map(|c| c.contract_id).collect();

    // Link contract to chain
    add_contract_to_chain(
        pool,
        chain_id,
        contract.id,
        depends_on,
        def.order_index,
        def.editor_x,
        def.editor_y,
    )
    .await?;

    // Update chain_contracts with definition_id link
    sqlx::query(
        "UPDATE chain_contracts SET definition_id = $1 WHERE chain_id = $2 AND contract_id = $3",
    )
    .bind(def.id)
    .bind(chain_id)
    .bind(contract.id)
    .execute(pool)
    .await?;

    Ok(contract.id)
}