4a18b1cd07
Root cause: WorkerService.ClaimTask() was modifying the audit entry in memory but never persisting it to the database. This caused build tasks to remain stuck at "pending" status even after being claimed. Changes: - Add UpdateStatus method to port.BuildAudit interface - Implement UpdateStatus in postgres.BuildAuditRepository - Fix ClaimTask to call audit.UpdateStatus() to persist status - Add test coverage for audit update during task claim - Update all mock implementations Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
51 lines
1.7 KiB
Go
51 lines
1.7 KiB
Go
// Package port defines interface contracts for external adapters.
|
|
package port
|
|
|
|
import (
|
|
"context"
|
|
"time"
|
|
|
|
"github.com/orchard9/rdev/internal/domain"
|
|
)
|
|
|
|
// BuildAudit records build history for observability and debugging.
|
|
// Every build that passes through the system gets an audit entry,
|
|
// providing a complete history of what was requested, who executed it,
|
|
// and what the outcome was.
|
|
type BuildAudit interface {
|
|
// Record creates a new audit entry when a build starts.
|
|
Record(ctx context.Context, entry *domain.BuildAuditEntry) error
|
|
|
|
// Update modifies an existing entry when a build completes.
|
|
Update(ctx context.Context, taskID string, result *domain.BuildResult) error
|
|
|
|
// UpdateStatus updates the status and worker assignment when a task is claimed.
|
|
// This is called when a worker picks up a task to mark it as running.
|
|
UpdateStatus(ctx context.Context, taskID string, status domain.BuildStatus, workerID string) error
|
|
|
|
// Get retrieves a specific audit entry by task ID.
|
|
// Returns ErrBuildNotFound if the entry does not exist.
|
|
Get(ctx context.Context, taskID string) (*domain.BuildAuditEntry, error)
|
|
|
|
// List returns audit entries matching the filter.
|
|
List(ctx context.Context, filter BuildAuditFilter) ([]*domain.BuildAuditEntry, error)
|
|
}
|
|
|
|
// BuildAuditFilter specifies criteria for listing audit entries.
|
|
type BuildAuditFilter struct {
|
|
// ProjectID filters entries by project.
|
|
ProjectID string
|
|
|
|
// WorkerID filters entries by worker.
|
|
WorkerID string
|
|
|
|
// Status filters entries by build status. Nil means all statuses.
|
|
Status *domain.BuildStatus
|
|
|
|
// Since filters entries created after this time.
|
|
Since time.Time
|
|
|
|
// Limit is the maximum number of entries to return.
|
|
Limit int
|
|
}
|