Merge branch 'main' into feat/job-zitfpv4v

This commit is contained in:
Janpeter Visser 2026-05-01 12:06:16 +02:00
commit 566fbf46bf
2 changed files with 42 additions and 2 deletions

View file

@ -1161,9 +1161,14 @@ UI klikt 'Voer uit'
`enqueueClaudeJobAction` weigert een tweede enqueue als er al een job bestaat met `status IN (QUEUED, CLAIMED, RUNNING)`. Teruggestuurde fout bevat het bestaande `jobId` zodat de UI ernaar kan linken.
### Auto-promote task naar DONE
### Auto-promote task-status op job-overgangen
Wanneer een `claude_job` op `DONE` komt, vuurt de Postgres-trigger `claude_job_status_to_task` (zie `prisma/migrations/20260501110000_sync_task_status_from_claude_job`) en zet de bijbehorende task ook op `DONE`. Werkt voor INSERT (direct als DONE aangemaakt) en UPDATE (transitie naar DONE). Idempotent: skip wanneer de task al DONE is. De bestaande `notify_task_change`-trigger op `tasks` vuurt dan automatisch de pg_notify zodat de Solo-paneel-UI synct — geen extra plumbing in de SSE-handler nodig.
Twee Postgres-triggers houden `task.status` in sync met `claude_job.status` zodat de Solo-kaart altijd in de juiste kolom staat:
- **`claude_job_claim_to_task`** (`prisma/migrations/20260501130000_promote_task_to_in_progress_on_claim`): bij INSERT met status `CLAIMED|RUNNING` of UPDATE OF status naar `CLAIMED|RUNNING`, promoot de bijbehorende task van `TO_DO` naar `IN_PROGRESS`. Forceert niet vanuit andere status — handmatige overrides (REVIEW, DONE) blijven staan.
- **`claude_job_status_to_task`** (`prisma/migrations/20260501110000_sync_task_status_from_claude_job`): bij DONE zet de task ook op `DONE`. Idempotent: skip wanneer task al DONE is.
De bestaande `notify_task_change`-trigger op `tasks` vuurt automatisch de pg_notify naar `/api/realtime/solo` zodat de UI direct synct — geen extra plumbing in de SSE-handler nodig.
### Hybride-ready

View file

@ -0,0 +1,35 @@
-- Promoot task naar IN_PROGRESS zodra een ClaudeJob CLAIMED of RUNNING wordt.
--
-- Aanvulling op de DONE-trigger uit
-- 20260501110000_sync_task_status_from_claude_job. Daar handelen we de
-- afronding af; hier handelen we de aanvang. Zonder deze trigger blijft
-- een task in de "Te doen"-kolom staan terwijl de agent er al mee bezig
-- is — verwarrend voor de gebruiker.
--
-- Voorwaarde: task moet nog op TO_DO staan. We forceren niet vanuit
-- IN_PROGRESS/REVIEW/DONE terug — die zijn handmatig gezet door de
-- gebruiker en hebben prioriteit.
--
-- De bestaande notify_task_change-trigger op tasks vuurt automatisch de
-- pg_notify naar /api/realtime/solo zodat de UI synct.
CREATE OR REPLACE FUNCTION promote_task_on_claude_job_claim() RETURNS trigger AS $$
BEGIN
IF NEW.status IN ('CLAIMED', 'RUNNING') THEN
IF (TG_OP = 'INSERT')
OR (TG_OP = 'UPDATE' AND OLD.status IS DISTINCT FROM NEW.status)
THEN
UPDATE tasks
SET status = 'IN_PROGRESS', updated_at = now()
WHERE id = NEW.task_id AND status = 'TO_DO';
END IF;
END IF;
RETURN NEW;
END;
$$ LANGUAGE plpgsql;
DROP TRIGGER IF EXISTS claude_job_claim_to_task ON claude_jobs;
CREATE TRIGGER claude_job_claim_to_task
AFTER INSERT OR UPDATE OF status ON claude_jobs
FOR EACH ROW
EXECUTE FUNCTION promote_task_on_claude_job_claim();