Fix activity audit trail not recording board mutations

backend/src/Taskdeck.Api/Extensions/ApplicationServiceRegistration.cs

@@ -30,6 +30,7 @@ public static IServiceCollection AddApplicationServices(this IServiceCollection
         services.AddScoped<ICaptureService, CaptureService>();
         services.AddScoped<ICaptureTriageService, CaptureTriageService>();
         services.AddScoped<HistoryService>();
+        services.AddScoped<IHistoryService>(sp => sp.GetRequiredService<HistoryService>());
         services.AddScoped<IAutomationProposalService, AutomationProposalService>();
         services.AddScoped<IAutomationPolicyEngine, AutomationPolicyEngine>();
         services.AddScoped<IAutomationPlannerService, AutomationPlannerService>();

backend/src/Taskdeck.Application/Services/BoardService.cs

@@ -2,6 +2,7 @@
 using Taskdeck.Application.Interfaces;
 using Taskdeck.Domain.Common;
 using Taskdeck.Domain.Entities;
+using Taskdeck.Domain.Enums;
 using Taskdeck.Domain.Exceptions;
 
 namespace Taskdeck.Application.Services;
@@ -11,20 +12,30 @@ public class BoardService
     private readonly IUnitOfWork _unitOfWork;
     private readonly IAuthorizationService? _authorizationService;
     private readonly IBoardRealtimeNotifier _realtimeNotifier;
+    private readonly IHistoryService? _historyService;
 
     public BoardService(IUnitOfWork unitOfWork)
-        : this(unitOfWork, authorizationService: null, realtimeNotifier: null)
+        : this(unitOfWork, authorizationService: null, realtimeNotifier: null, historyService: null)
     {
     }
 
     public BoardService(
         IUnitOfWork unitOfWork,
         IAuthorizationService? authorizationService,
-        IBoardRealtimeNotifier? realtimeNotifier = null)
+        IBoardRealtimeNotifier? realtimeNotifier = null,
+        IHistoryService? historyService = null)
     {
         _unitOfWork = unitOfWork;
         _authorizationService = authorizationService;
         _realtimeNotifier = realtimeNotifier ?? NoOpBoardRealtimeNotifier.Instance;
+        _historyService = historyService;
+    }
+
+    private async Task SafeLogAsync(string entityType, Guid entityId, AuditAction action, Guid? userId = null, string? changes = null)
+    {
+        if (_historyService == null) return;
+        try { await _historyService.LogActionAsync(entityType, entityId, action, userId, changes); }
+        catch (Exception) { /* Audit is secondary — never crash the mutation */ }
     }
 
     public async Task<Result<BoardDto>> CreateBoardAsync(CreateBoardDto dto, Guid actingUserId, CancellationToken cancellationToken = default)
@@ -139,6 +150,7 @@ private async Task<Result<BoardDto>> CreateBoardInternalAsync(CreateBoardDto dto
             await _realtimeNotifier.NotifyBoardMutationAsync(
                 new BoardRealtimeEvent(board.Id, "board", "created", board.Id, DateTimeOffset.UtcNow),
                 cancellationToken);
+            await SafeLogAsync("board", board.Id, AuditAction.Created, ownerId, $"name={board.Name}");
 
             return Result.Success(MapToDto(board));
         }
@@ -171,6 +183,12 @@ private async Task<Result<BoardDto>> UpdateBoardInternalAsync(Guid id, UpdateBoa
             await _realtimeNotifier.NotifyBoardMutationAsync(
                 new BoardRealtimeEvent(board.Id, "board", "updated", board.Id, DateTimeOffset.UtcNow),
                 cancellationToken);
+            if (dto.IsArchived == true)
+                await SafeLogAsync("board", board.Id, AuditAction.Archived, changes: $"name={board.Name}");
+            else if (dto.IsArchived == false)
+                await SafeLogAsync("board", board.Id, AuditAction.Updated, changes: $"unarchived; name={board.Name}");
+            else
+                await SafeLogAsync("board", board.Id, AuditAction.Updated, changes: $"name={board.Name}");
             return Result.Success(MapToDto(board));
         }
         catch (DomainException ex)
@@ -199,6 +217,7 @@ private async Task<Result> DeleteBoardInternalAsync(Guid id, CancellationToken c
         await _realtimeNotifier.NotifyBoardMutationAsync(
             new BoardRealtimeEvent(board.Id, "board", "archived", board.Id, DateTimeOffset.UtcNow),
             cancellationToken);
+        await SafeLogAsync("board", board.Id, AuditAction.Archived, changes: $"name={board.Name}");
         return Result.Success();
     }
 

backend/src/Taskdeck.Application/Services/CardService.cs

@@ -11,16 +11,25 @@ public class CardService
 {
     private readonly IUnitOfWork _unitOfWork;
     private readonly IBoardRealtimeNotifier _realtimeNotifier;
+    private readonly IHistoryService? _historyService;
 
     public CardService(IUnitOfWork unitOfWork)
-        : this(unitOfWork, realtimeNotifier: null)
+        : this(unitOfWork, realtimeNotifier: null, historyService: null)
     {
     }
 
-    public CardService(IUnitOfWork unitOfWork, IBoardRealtimeNotifier? realtimeNotifier = null)
+    public CardService(IUnitOfWork unitOfWork, IBoardRealtimeNotifier? realtimeNotifier = null, IHistoryService? historyService = null)
     {
         _unitOfWork = unitOfWork;
         _realtimeNotifier = realtimeNotifier ?? NoOpBoardRealtimeNotifier.Instance;
+        _historyService = historyService;
+    }
+
+    private async Task SafeLogAsync(string entityType, Guid entityId, AuditAction action, Guid? userId = null, string? changes = null)
+    {
+        if (_historyService == null) return;
+        try { await _historyService.LogActionAsync(entityType, entityId, action, userId, changes); }
+        catch (Exception) { /* Audit is secondary — never crash the mutation */ }
     }
 
     public async Task<Result<CardDto>> CreateCardAsync(CreateCardDto dto, CancellationToken cancellationToken = default)
@@ -74,6 +83,7 @@ public async Task<Result<CardDto>> CreateCardAsync(
             await _realtimeNotifier.NotifyBoardMutationAsync(
                 new BoardRealtimeEvent(card.BoardId, "card", "created", card.Id, DateTimeOffset.UtcNow),
                 cancellationToken);
+            await SafeLogAsync("card", card.Id, AuditAction.Created, changes: $"title={card.Title}");
 
             var createdCard = await _unitOfWork.Cards.GetByIdWithLabelsAsync(card.Id, cancellationToken);
             return Result.Success(MapToDto(createdCard!));
@@ -134,6 +144,7 @@ public async Task<Result<CardDto>> UpdateCardAsync(
             await _realtimeNotifier.NotifyBoardMutationAsync(
                 new BoardRealtimeEvent(card.BoardId, "card", "updated", card.Id, DateTimeOffset.UtcNow),
                 cancellationToken);
+            await SafeLogAsync("card", card.Id, AuditAction.Updated, actorUserId);
 
             var updatedCard = await _unitOfWork.Cards.GetByIdWithLabelsAsync(id, cancellationToken);
             return Result.Success(MapToDto(updatedCard!));
@@ -213,6 +224,7 @@ public async Task<Result<CardDto>> MoveCardAsync(Guid id, MoveCardDto dto, Cance
             await _realtimeNotifier.NotifyBoardMutationAsync(
                 new BoardRealtimeEvent(card.BoardId, "card", "moved", card.Id, DateTimeOffset.UtcNow),
                 cancellationToken);
+            await SafeLogAsync("card", card.Id, AuditAction.Moved, changes: $"target_column={dto.TargetColumnId}; position={dto.TargetPosition}");
 
             var movedCard = await _unitOfWork.Cards.GetByIdWithLabelsAsync(id, cancellationToken);
             return Result.Success(MapToDto(movedCard!));
@@ -299,6 +311,7 @@ public async Task<Result> DeleteCardAsync(Guid id, CancellationToken cancellatio
         await _realtimeNotifier.NotifyBoardMutationAsync(
             new BoardRealtimeEvent(card.BoardId, "card", "deleted", card.Id, DateTimeOffset.UtcNow),
             cancellationToken);
+        await SafeLogAsync("card", card.Id, AuditAction.Deleted, changes: $"title={card.Title}");
 
         return Result.Success();
     }

backend/src/Taskdeck.Application/Services/ColumnService.cs

@@ -2,6 +2,7 @@
 using Taskdeck.Application.Interfaces;
 using Taskdeck.Domain.Common;
 using Taskdeck.Domain.Entities;
+using Taskdeck.Domain.Enums;
 using Taskdeck.Domain.Exceptions;
 
 namespace Taskdeck.Application.Services;
@@ -10,16 +11,25 @@ public class ColumnService
 {
     private readonly IUnitOfWork _unitOfWork;
     private readonly IBoardRealtimeNotifier _realtimeNotifier;
+    private readonly IHistoryService? _historyService;
 
     public ColumnService(IUnitOfWork unitOfWork)
-        : this(unitOfWork, realtimeNotifier: null)
+        : this(unitOfWork, realtimeNotifier: null, historyService: null)
     {
     }
 
-    public ColumnService(IUnitOfWork unitOfWork, IBoardRealtimeNotifier? realtimeNotifier = null)
+    public ColumnService(IUnitOfWork unitOfWork, IBoardRealtimeNotifier? realtimeNotifier = null, IHistoryService? historyService = null)
     {
         _unitOfWork = unitOfWork;
         _realtimeNotifier = realtimeNotifier ?? NoOpBoardRealtimeNotifier.Instance;
+        _historyService = historyService;
+    }
+
+    private async Task SafeLogAsync(string entityType, Guid entityId, AuditAction action, Guid? userId = null, string? changes = null)
+    {
+        if (_historyService == null) return;
+        try { await _historyService.LogActionAsync(entityType, entityId, action, userId, changes); }
+        catch (Exception) { /* Audit is secondary — never crash the mutation */ }
     }
 
     public async Task<Result<ColumnDto>> CreateColumnAsync(CreateColumnDto dto, CancellationToken cancellationToken = default)
@@ -45,6 +55,7 @@ public async Task<Result<ColumnDto>> CreateColumnAsync(CreateColumnDto dto, Canc
             await _realtimeNotifier.NotifyBoardMutationAsync(
                 new BoardRealtimeEvent(column.BoardId, "column", "created", column.Id, DateTimeOffset.UtcNow),
                 cancellationToken);
+            await SafeLogAsync("column", column.Id, AuditAction.Created, changes: $"name={column.Name}");
 
             return Result.Success(MapToDto(column));
         }
@@ -67,6 +78,7 @@ public async Task<Result<ColumnDto>> UpdateColumnAsync(Guid id, UpdateColumnDto
             await _realtimeNotifier.NotifyBoardMutationAsync(
                 new BoardRealtimeEvent(column.BoardId, "column", "updated", column.Id, DateTimeOffset.UtcNow),
                 cancellationToken);
+            await SafeLogAsync("column", column.Id, AuditAction.Updated);
 
             return Result.Success(MapToDto(column));
         }
@@ -105,6 +117,7 @@ public async Task<Result> DeleteColumnAsync(Guid id, CancellationToken cancellat
         await _realtimeNotifier.NotifyBoardMutationAsync(
             new BoardRealtimeEvent(column.BoardId, "column", "deleted", column.Id, DateTimeOffset.UtcNow),
             cancellationToken);
+        await SafeLogAsync("column", column.Id, AuditAction.Deleted, changes: $"name={column.Name}");
 
         return Result.Success();
     }
@@ -166,6 +179,7 @@ public async Task<Result<IEnumerable<ColumnDto>>> ReorderColumnsAsync(Guid board
             await _realtimeNotifier.NotifyBoardMutationAsync(
                 new BoardRealtimeEvent(boardId, "column", "reordered", null, DateTimeOffset.UtcNow),
                 cancellationToken);
+            await SafeLogAsync("column", boardId, AuditAction.Updated, changes: $"reordered; count={dto.ColumnIds.Count}");
 
             // Return reordered columns
             var reorderedColumns = dto.ColumnIds.Select(id => MapToDto(columnDict[id]));

backend/src/Taskdeck.Application/Services/HistoryService.cs

@@ -70,6 +70,12 @@ public async Task<Result> LogActionAsync(string entityType, Guid entityId, Audit
         {
             return Result.Failure(ex.ErrorCode, ex.Message);
         }
+        catch (Exception)
+        {
+            // Audit logging is secondary to the mutation — never let infrastructure
+            // failures (e.g. DB full, concurrency) crash the calling operation.
+            return Result.Failure(ErrorCodes.UnexpectedError, $"Failed to persist audit log for {entityType}/{entityId}/{action}");
+        }
     }
 
     private static AuditLogDto MapToDto(AuditLog log)

backend/src/Taskdeck.Application/Services/IHistoryService.cs

@@ -6,7 +6,6 @@ namespace Taskdeck.Application.Services;
 
 /// <summary>
 /// Service interface for audit log and history operations.
-/// SCAFFOLDING: Implementation pending.
 /// </summary>
 public interface IHistoryService
 {

backend/src/Taskdeck.Application/Services/LabelService.cs

@@ -2,6 +2,7 @@
 using Taskdeck.Application.Interfaces;
 using Taskdeck.Domain.Common;
 using Taskdeck.Domain.Entities;
+using Taskdeck.Domain.Enums;
 using Taskdeck.Domain.Exceptions;
 
 namespace Taskdeck.Application.Services;
@@ -10,16 +11,25 @@ public class LabelService
 {
     private readonly IUnitOfWork _unitOfWork;
     private readonly IBoardRealtimeNotifier _realtimeNotifier;
+    private readonly IHistoryService? _historyService;
 
     public LabelService(IUnitOfWork unitOfWork)
-        : this(unitOfWork, realtimeNotifier: null)
+        : this(unitOfWork, realtimeNotifier: null, historyService: null)
     {
     }
 
-    public LabelService(IUnitOfWork unitOfWork, IBoardRealtimeNotifier? realtimeNotifier = null)
+    public LabelService(IUnitOfWork unitOfWork, IBoardRealtimeNotifier? realtimeNotifier = null, IHistoryService? historyService = null)
     {
         _unitOfWork = unitOfWork;
         _realtimeNotifier = realtimeNotifier ?? NoOpBoardRealtimeNotifier.Instance;
+        _historyService = historyService;
+    }
+
+    private async Task SafeLogAsync(string entityType, Guid entityId, AuditAction action, Guid? userId = null, string? changes = null)
+    {
+        if (_historyService == null) return;
+        try { await _historyService.LogActionAsync(entityType, entityId, action, userId, changes); }
+        catch (Exception) { /* Audit is secondary — never crash the mutation */ }
     }
 
     public async Task<Result<LabelDto>> CreateLabelAsync(CreateLabelDto dto, CancellationToken cancellationToken = default)
@@ -36,6 +46,7 @@ public async Task<Result<LabelDto>> CreateLabelAsync(CreateLabelDto dto, Cancell
             await _realtimeNotifier.NotifyBoardMutationAsync(
                 new BoardRealtimeEvent(label.BoardId, "label", "created", label.Id, DateTimeOffset.UtcNow),
                 cancellationToken);
+            await SafeLogAsync("label", label.Id, AuditAction.Created, changes: $"name={label.Name}");
 
             return Result.Success(MapToDto(label));
         }
@@ -58,6 +69,7 @@ public async Task<Result<LabelDto>> UpdateLabelAsync(Guid id, UpdateLabelDto dto
             await _realtimeNotifier.NotifyBoardMutationAsync(
                 new BoardRealtimeEvent(label.BoardId, "label", "updated", label.Id, DateTimeOffset.UtcNow),
                 cancellationToken);
+            await SafeLogAsync("label", label.Id, AuditAction.Updated);
 
             return Result.Success(MapToDto(label));
         }
@@ -93,6 +105,7 @@ public async Task<Result> DeleteLabelAsync(Guid id, CancellationToken cancellati
         await _realtimeNotifier.NotifyBoardMutationAsync(
             new BoardRealtimeEvent(label.BoardId, "label", "deleted", label.Id, DateTimeOffset.UtcNow),
             cancellationToken);
+        await SafeLogAsync("label", label.Id, AuditAction.Deleted, changes: $"name={label.Name}");
 
         return Result.Success();
     }