Fix: global namespace resolution, handler exception semantics, record property init, unit test alignment

src/Api/Data/IssueRepository.cs

@@ -152,7 +152,9 @@ public static IssueEntity FromDomain(Issue issue)
 			Status = issue.Status.ToString(),
 			CreatedAt = issue.CreatedAt,
 			UpdatedAt = issue.UpdatedAt,
-			IsArchived = false,
+			IsArchived = issue.IsArchived,
+			ArchivedBy = issue.ArchivedBy,
+			ArchivedAt = issue.ArchivedAt,
 			Labels = issue.Labels?.Select(l => new LabelEntity { Name = l.Name, Color = l.Color }).ToList()
 		};
 	}

src/Api/Handlers/DeleteIssueHandler.cs

@@ -1,6 +1,7 @@
 using FluentValidation;
 using IssueManager.Api.Data;
 using IssueManager.Shared.Validators;
+using global::Shared.Exceptions;
 
 namespace IssueManager.Api.Handlers;
 
@@ -33,7 +34,21 @@ public async Task<bool> Handle(DeleteIssueCommand command, CancellationToken can
 			throw new ValidationException(validationResult.Errors);
 		}
 
-		// Archive the issue (soft-delete)
-		return await _repository.ArchiveAsync(command.Id, cancellationToken);
+		// Get the existing issue
+		var existingIssue = await _repository.GetByIdAsync(command.Id, cancellationToken);
+		if (existingIssue is null)
+		{
+			throw new NotFoundException($"Issue with ID '{command.Id}' was not found.");
+		}
+
+		// If already archived, this is idempotent - return success without updating
+		if (existingIssue.IsArchived)
+		{
+			return true;
+		}
+
+		// Archive the issue via the dedicated archive operation
+		await _repository.ArchiveAsync(command.Id, cancellationToken);
+		return true;
 	}
 }

src/Api/Handlers/UpdateIssueHandler.cs

@@ -1,7 +1,8 @@
 using FluentValidation;
 using IssueManager.Api.Data;
-using IssueManager.Shared.Domain;
+using global::Shared.Domain;
 using IssueManager.Shared.Validators;
+using global::Shared.Exceptions;
 
 namespace IssueManager.Api.Handlers;
 
@@ -25,7 +26,7 @@ public UpdateIssueHandler(IIssueRepository repository, UpdateIssueValidator vali
 	/// <summary>
 	/// Handles the update of an existing issue.
 	/// </summary>
-	public async Task<Issue?> Handle(UpdateIssueCommand command, CancellationToken cancellationToken = default)
+	public async Task<Issue> Handle(UpdateIssueCommand command, CancellationToken cancellationToken = default)
 	{
 		// Validate the command
 		var validationResult = await _validator.ValidateAsync(command, cancellationToken);
@@ -38,13 +39,24 @@ public UpdateIssueHandler(IIssueRepository repository, UpdateIssueValidator vali
 		var existingIssue = await _repository.GetByIdAsync(command.Id, cancellationToken);
 		if (existingIssue is null)
 		{
-			return null;
+			throw new NotFoundException($"Issue with ID '{command.Id}' was not found.");
+		}
+
+		// Cannot update an archived issue
+		if (existingIssue.IsArchived)
+		{
+			throw new ConflictException($"Issue with ID '{command.Id}' is archived and cannot be updated.");
 		}
 
 		// Update the issue using the domain method
 		var updatedIssue = existingIssue.Update(command.Title, command.Description);
 
 		// Persist the updated issue
-		return await _repository.UpdateAsync(updatedIssue, cancellationToken);
+		var result = await _repository.UpdateAsync(updatedIssue, cancellationToken);
+		if (result is null)
+		{
+			throw new NotFoundException($"Issue with ID '{command.Id}' could not be updated.");
+		}
+		return result;
 	}
 }

src/Api/Program.cs

@@ -2,6 +2,7 @@
 using IssueManager.Api.Data;
 using IssueManager.Api.Handlers;
 using IssueManager.Shared.Validators;
+using global::Shared.Validators;
 using IssueManager.Shared.Domain.DTOs;
 using static IssueManager.Api.Handlers.GetIssueHandler;
 
@@ -46,6 +47,18 @@
 			.ToDictionary(g => g.Key, g => g.Select(e => e.ErrorMessage).ToArray());
 		await context.Response.WriteAsJsonAsync(new { title = "Validation failed", errors });
 	}
+	else if (ex is global::Shared.Exceptions.NotFoundException notFoundEx)
+	{
+		context.Response.StatusCode = StatusCodes.Status404NotFound;
+		context.Response.ContentType = "application/problem+json";
+		await context.Response.WriteAsJsonAsync(new { title = "Not Found", detail = notFoundEx.Message });
+	}
+	else if (ex is global::Shared.Exceptions.ConflictException conflictEx)
+	{
+		context.Response.StatusCode = StatusCodes.Status409Conflict;
+		context.Response.ContentType = "application/problem+json";
+		await context.Response.WriteAsJsonAsync(new { title = "Conflict", detail = conflictEx.Message });
+	}
 	else
 	{
 		context.Response.StatusCode = StatusCodes.Status500InternalServerError;
@@ -56,7 +69,7 @@
 app.MapOpenApi();
 
 // Issue API Endpoints
 var issuesApi = app.MapGroup("/api/v1/issues")
 	.WithTags("Issues")
 	.WithOpenApi();
 
@@ -100,20 +113,21 @@
 {
 	var commandWithId = command with { Id = id };
 	var result = await handler.Handle(commandWithId);
-	return result is not null ? Results.Ok(result) : Results.NotFound();
+	return Results.Ok(result);
 })
 .WithName("UpdateIssue")
 .WithSummary("Update an existing issue")
 .Produces(StatusCodes.Status200OK)
 .Produces(StatusCodes.Status400BadRequest)
-.Produces(StatusCodes.Status404NotFound);
+.Produces(StatusCodes.Status404NotFound)
+.Produces(StatusCodes.Status409Conflict);
 
 // Delete Issue (soft-delete)
 issuesApi.MapDelete("{id}", async (string id, DeleteIssueHandler handler) =>
 {
 	var command = new DeleteIssueCommand { Id = id };
-	var result = await handler.Handle(command);
-	return result ? Results.NoContent() : Results.NotFound();
+	await handler.Handle(command);
+	return Results.NoContent();
 })
 .WithName("DeleteIssue")
 .WithSummary("Delete (archive) an issue")

src/Shared/Domain/Issue.cs

@@ -36,22 +36,22 @@ public record Issue(
 	/// <summary>
 	/// Gets the detailed description of the issue.
 	/// </summary>
-	public string? Description { get; init; }
+	public string? Description { get; init; } = Description;
 
 	/// <summary>
 	/// Gets the current status of the issue.
 	/// </summary>
-	public IssueStatus Status { get; init; }
+	public IssueStatus Status { get; init; } = Status;
 
 	/// <summary>
 	/// Gets the timestamp when the issue was created.
 	/// </summary>
-	public DateTime CreatedAt { get; init; }
+	public DateTime CreatedAt { get; init; } = CreatedAt;
 
 	/// <summary>
 	/// Gets the timestamp when the issue was last updated.
 	/// </summary>
-	public DateTime UpdatedAt { get; init; }
+	public DateTime UpdatedAt { get; init; } = UpdatedAt;
 
 	/// <summary>
 	/// Gets the collection of labels attached to the issue.

src/Web/Components/IssueForm.razor

@@ -1,4 +1,4 @@
-@using Shared.Domain
+@using global::Shared.Domain
 @namespace IssueManager.Web.Components
 
 <div class="form-container">

src/Web/_Imports.razor

@@ -7,3 +7,4 @@
 @using IssueManager.Web.Layout
 @using IssueManager.Web.Pages
 @using IssueManager.Web.Components
+@using global::Shared.Domain