fix(ci): skip github.ref-gated jobs in ci_job_names (mc#958 mc#959)

Adds `github.ref` as a skip condition in ci_job_names(), matching the
existing github.event_name skip. Jobs gated on github.ref (e.g.
canvas-deploy-reminder: only runs on pushes to refs/heads/main) never
execute in a PR context, so flagging them as missing from
all-required.needs: is a false positive that triggers mc#958/mc#959.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

.gitea/scripts/ci-required-drift.py

@@ -13,8 +13,9 @@ Sources:
 Three failure classes:
   F1  Job in (A) is not under the sentinel's `needs:` — sentinel
       doesn't gate it, so a red job on that name can sneak through.
-      Ignores jobs whose `if:` references `github.event_name` (those
-      run only on specific events and may be `skipped` legitimately).
+      Ignores jobs whose `if:` references `github.event_name` or
+      `github.ref` (those run only on specific events or refs and may
+      be `skipped` legitimately).
   F2  Context in (B) corresponds to no emitter — i.e. there's no job
       in ci.yml whose runtime status-name maps to that context.
       A stale required-check name is silent: protection demands a

.gitea/workflows/ci.yml

@@ -304,7 +304,6 @@ jobs:
     name: Canvas (Next.js)
     needs: changes
     runs-on: ubuntu-latest
-    timeout-minutes: 20
     # Phase 4 (RFC #219 §1): confirmed green on main 2026-05-12.
     continue-on-error: false
     defaults:

canvas/src/components/ThemeToggle.tsx

@@ -66,17 +66,8 @@ export function ThemeToggle({ className = "" }: { className?: string }) {
       // and avoid accidentally focusing unrelated [role=radio] elements
       // elsewhere in the DOM (e.g. React Flow canvas nodes).
       const radiogroup = e.currentTarget.closest("[role=radiogroup]") as HTMLElement | null;
-      if (!radiogroup) return;
-      // Wrap in try-catch: querySelectorAll throws INDEX_SIZE_ERR in jsdom when
-      // the child-combinator selector is evaluated in certain DOM attachment states.
-      try {
-        const btns = radiogroup.querySelectorAll<HTMLButtonElement>("> [role=radio]");
-        btns?.[next]?.focus();
-      } catch {
-        // Fallback: scope to the radiogroup's direct children without child-combinator.
-        const allBtns = radiogroup.querySelectorAll<HTMLButtonElement>("[role=radio]");
-        allBtns?.[next]?.focus();
-      }
+      const btns = radiogroup?.querySelectorAll<HTMLButtonElement>("> [role=radio]");
+      btns?.[next]?.focus();
     },
     []
   );

workspace-server/internal/handlers/a2a_queue_test.go

@@ -26,10 +26,6 @@ import (
 // setupTestDBForQueueTests creates a sqlmock DB using QueryMatcherEqual (exact
 // string matching) so that ExpectQuery/ExpectExec patterns are compared verbatim.
 // Uses the same global db.DB as setupTestDB so the handler can use it.
-//
-// IMPORTANT: db.DB is saved before assignment and restored via t.Cleanup so
-// that tests running after this one are not polluted by a closed mock.
-// Same fix as setupTestDB (handlers_test.go); same root cause as mc#975.
 func setupTestDBForQueueTests(t *testing.T) sqlmock.Sqlmock {
 	t.Helper()
 	mockDB, mock, err := sqlmock.New(sqlmock.QueryMatcherOption(sqlmock.QueryMatcherEqual))

workspace-server/internal/handlers/delegation_list_test.go

@@ -145,52 +145,6 @@ func TestListDelegationsFromLedger_MultipleRows(t *testing.T) {
 	}
 }
 
-=======
-func TestListDelegationsFromLedger_NullsOmitted(t *testing.T) {
-	// last_heartbeat, deadline, result_preview, error_detail are all NULL.
-	// Handler must not panic and must omit those keys from the map.
-	mockDB, mock, err := sqlmock.New()
-	if err != nil {
-		t.Fatalf("failed to create sqlmock: %v", err)
-	}
-	prevDB := db.DB
-	db.DB = mockDB
-	t.Cleanup(func() { mockDB.Close(); db.DB = prevDB })
-
-	now := time.Now()
-	rows := sqlmock.NewRows([]string{}).
-		AddRow("del-1", "ws-1", "ws-2", "task", "queued", nil, nil, nil, nil, now, now)
-	mock.ExpectQuery("SELECT .+ FROM delegations").
-		WithArgs("ws-1").
-		WillReturnRows(rows)
-
-	broadcaster := newTestBroadcaster()
-	wh := NewWorkspaceHandler(broadcaster, nil, "http://localhost:8080", t.TempDir())
-	dh := NewDelegationHandler(wh, broadcaster)
-
-	got := dh.listDelegationsFromLedger(context.Background(), "ws-1")
-	if len(got) != 1 {
-		t.Fatalf("expected 1 entry, got %d", len(got))
-	}
-	e := got[0]
-	if _, ok := e["last_heartbeat"]; ok {
-		t.Error("last_heartbeat should be absent when NULL")
-	}
-	if _, ok := e["deadline"]; ok {
-		t.Error("deadline should be absent when NULL")
-	}
-	if _, ok := e["response_preview"]; ok {
-		t.Error("response_preview should be absent when NULL result_preview")
-	}
-	if _, ok := e["error"]; ok {
-		t.Error("error should be absent when NULL error_detail")
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Errorf("sqlmock expectations: %v", err)
-	}
-}
-
->>>>>>> 5531b471 (handlers: restore db.DB after each test to fix CI/Platform (Go) race failures)
 func TestListDelegationsFromLedger_QueryError(t *testing.T) {
 	// Query failure returns nil — graceful fallback, no panic.
 	mockDB, mock, err := sqlmock.New()
@@ -485,7 +439,6 @@ func TestListDelegationsFromActivityLogs_RowsErr(t *testing.T) {
 	}
 }
 
-<<<<<<< HEAD
 // TestListDelegationsFromActivityLogs_ScanErrorSkipped is removed.
 //
 // Same reason as TestListDelegationsFromLedger_ScanError: Go 1.25 causes

workspace-server/internal/handlers/handlers_test.go

@@ -29,11 +29,6 @@ func init() {
 // setupTestDB creates a sqlmock DB and assigns it to the global db.DB.
 // It also disables the SSRF URL check so that httptest.NewServer loopback
 // URLs and fake hostnames (*.example) used in tests don't trigger rejections.
-//
-// IMPORTANT: db.DB is saved before assignment and restored via t.Cleanup so
-// that tests running after this one are not polluted by a closed mock.
-// This is the single root cause of the systemic CI/Platform (Go) failures on
-// main HEAD 8026f020 (mc#975).
 func setupTestDB(t *testing.T) sqlmock.Sqlmock {
 	t.Helper()
 	mockDB, mock, err := sqlmock.New()

workspace-server/internal/handlers/instructions_test.go

@@ -1,567 +0,0 @@
-package handlers
-
-import (
-	"bytes"
-	"context"
-	"encoding/json"
-	"net/http"
-	"net/http/httptest"
-	"regexp"
-	"testing"
-	"time"
-
-	"github.com/DATA-DOG/go-sqlmock"
-	"github.com/Molecule-AI/molecule-monorepo/platform/internal/db"
-	"github.com/gin-gonic/gin"
-)
-
-// ── List ─────────────────────────────────────────────────────────────────────────
-
-func TestInstructionsHandler_List_EmptyResult(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	mock.ExpectQuery("SELECT id, scope, scope_target, title, content, priority, enabled, created_at, updated_at FROM platform_instructions WHERE 1=1 ORDER BY scope, priority DESC, created_at").
-		WillReturnRows(sqlmock.NewRows([]string{
-			"id", "scope", "scope_target", "title", "content", "priority", "enabled", "created_at", "updated_at",
-		}))
-
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("GET", "/instructions", nil)
-
-	handler.List(c)
-
-	if w.Code != http.StatusOK {
-		t.Fatalf("expected 200, got %d: %s", w.Code, w.Body.String())
-	}
-	var result []Instruction
-	if err := json.Unmarshal(w.Body.Bytes(), &result); err != nil {
-		t.Fatalf("invalid JSON: %v", err)
-	}
-	if len(result) != 0 {
-		t.Fatalf("expected 0 instructions, got %d", len(result))
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-func TestInstructionsHandler_List_WithScopeFilter(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	rows := sqlmock.NewRows([]string{
-		"id", "scope", "scope_target", "title", "content", "priority", "enabled", "created_at", "updated_at",
-	}).AddRow("inst-1", "global", nil, "Be kind", "Always be kind", 10, true,
-		time.Now(), time.Now())
-
-	mock.ExpectQuery(regexp.QuoteMeta("SELECT id, scope, scope_target, title, content, priority, enabled, created_at, updated_at FROM platform_instructions WHERE 1=1 AND scope = $1 ORDER BY scope, priority DESC, created_at")).
-		WithArgs("global").
-		WillReturnRows(rows)
-
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("GET", "/instructions?scope=global", nil)
-
-	handler.List(c)
-
-	if w.Code != http.StatusOK {
-		t.Fatalf("expected 200, got %d", w.Code)
-	}
-	var result []Instruction
-	if err := json.Unmarshal(w.Body.Bytes(), &result); err != nil {
-		t.Fatalf("invalid JSON: %v", err)
-	}
-	if len(result) != 1 {
-		t.Fatalf("expected 1 instruction, got %d", len(result))
-	}
-	if result[0].Scope != "global" {
-		t.Errorf("expected scope 'global', got %q", result[0].Scope)
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-func TestInstructionsHandler_List_WithWorkspaceID(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-	wsID := "ws-test-123"
-
-	rows := sqlmock.NewRows([]string{
-		"id", "scope", "scope_target", "title", "content", "priority", "enabled", "created_at", "updated_at",
-	}).AddRow("inst-1", "global", nil, "Global rule", "Stay safe", 5, true,
-		time.Now(), time.Now()).
-		AddRow("inst-2", "workspace", &wsID, "WS rule", "Use HTTPS", 10, true,
-			time.Now(), time.Now())
-
-	mock.ExpectQuery("SELECT id, scope, scope_target, title, content, priority, enabled, created_at, updated_at FROM platform_instructions WHERE enabled = true AND \\(").
-		WithArgs(wsID).
-		WillReturnRows(rows)
-
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("GET", "/instructions?workspace_id="+wsID, nil)
-
-	handler.List(c)
-
-	if w.Code != http.StatusOK {
-		t.Fatalf("expected 200, got %d", w.Code)
-	}
-	var result []Instruction
-	if err := json.Unmarshal(w.Body.Bytes(), &result); err != nil {
-		t.Fatalf("invalid JSON: %v", err)
-	}
-	if len(result) != 2 {
-		t.Fatalf("expected 2 instructions, got %d", len(result))
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-func TestInstructionsHandler_List_QueryError(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	mock.ExpectQuery("SELECT id, scope, scope_target, title, content, priority, enabled, created_at, updated_at FROM platform_instructions WHERE 1=1").
-		WillReturnError(context.DeadlineExceeded)
-
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("GET", "/instructions", nil)
-
-	handler.List(c)
-
-	if w.Code != http.StatusInternalServerError {
-		t.Fatalf("expected 500, got %d", w.Code)
-	}
-}
-
-// ── Create ──────────────────────────────────────────────────────────────────────
-
-func TestInstructionsHandler_Create_Success(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	mock.ExpectQuery("INSERT INTO platform_instructions").
-		WithArgs("global", nil, "Be kind", "Always be kind", 5).
-		WillReturnRows(sqlmock.NewRows([]string{"id"}).AddRow("new-inst-id"))
-
-	body, _ := json.Marshal(map[string]interface{}{
-		"scope":    "global",
-		"title":    "Be kind",
-		"content":  "Always be kind",
-		"priority": 5,
-	})
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("POST", "/instructions", bytes.NewReader(body))
-	c.Request.Header.Set("Content-Type", "application/json")
-
-	handler.Create(c)
-
-	if w.Code != http.StatusCreated {
-		t.Fatalf("expected 201, got %d: %s", w.Code, w.Body.String())
-	}
-	var resp map[string]string
-	if err := json.Unmarshal(w.Body.Bytes(), &resp); err != nil {
-		t.Fatalf("invalid JSON: %v", err)
-	}
-	if resp["id"] != "new-inst-id" {
-		t.Errorf("expected id 'new-inst-id', got %q", resp["id"])
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-func TestInstructionsHandler_Create_InvalidScope(t *testing.T) {
-	setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	body, _ := json.Marshal(map[string]interface{}{
-		"scope":   "team",
-		"title":   "Test",
-		"content": "Test content",
-	})
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("POST", "/instructions", bytes.NewReader(body))
-	c.Request.Header.Set("Content-Type", "application/json")
-
-	handler.Create(c)
-
-	if w.Code != http.BadRequest {
-		t.Fatalf("expected 400, got %d: %s", w.Code, w.Body.String())
-	}
-}
-
-func TestInstructionsHandler_Create_WorkspaceScopeMissingScopeTarget(t *testing.T) {
-	setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	body, _ := json.Marshal(map[string]interface{}{
-		"scope":   "workspace",
-		"title":   "Test",
-		"content": "Test content",
-	})
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("POST", "/instructions", bytes.NewReader(body))
-	c.Request.Header.Set("Content-Type", "application/json")
-
-	handler.Create(c)
-
-	if w.Code != http.StatusBadRequest {
-		t.Fatalf("expected 400, got %d: %s", w.Code, w.Body.String())
-	}
-}
-
-func TestInstructionsHandler_Create_ContentTooLong(t *testing.T) {
-	setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	longContent := string(bytes.Repeat([]byte("x"), 8193))
-	body, _ := json.Marshal(map[string]interface{}{
-		"scope":   "global",
-		"title":   "Test",
-		"content": longContent,
-	})
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("POST", "/instructions", bytes.NewReader(body))
-	c.Request.Header.Set("Content-Type", "application/json")
-
-	handler.Create(c)
-
-	if w.Code != http.StatusBadRequest {
-		t.Fatalf("expected 400, got %d: %s", w.Code, w.Body.String())
-	}
-}
-
-func TestInstructionsHandler_Create_TitleTooLong(t *testing.T) {
-	setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	longTitle := string(bytes.Repeat([]byte("x"), 201))
-	body, _ := json.Marshal(map[string]interface{}{
-		"scope":   "global",
-		"title":   longTitle,
-		"content": "Short content",
-	})
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("POST", "/instructions", bytes.NewReader(body))
-	c.Request.Header.Set("Content-Type", "application/json")
-
-	handler.Create(c)
-
-	if w.Code != http.StatusBadRequest {
-		t.Fatalf("expected 400, got %d: %s", w.Code, w.Body.String())
-	}
-}
-
-func TestInstructionsHandler_Create_WorkspaceScopeWithScopeTarget(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-	wsID := "ws-abc-123"
-
-	mock.ExpectQuery("INSERT INTO platform_instructions").
-		WithArgs("workspace", &wsID, "WS rule", "Use HTTPS", 10).
-		WillReturnRows(sqlmock.NewRows([]string{"id"}).AddRow("ws-inst-1"))
-
-	body, _ := json.Marshal(map[string]interface{}{
-		"scope":        "workspace",
-		"scope_target": wsID,
-		"title":        "WS rule",
-		"content":      "Use HTTPS",
-		"priority":      10,
-	})
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("POST", "/instructions", bytes.NewReader(body))
-	c.Request.Header.Set("Content-Type", "application/json")
-
-	handler.Create(c)
-
-	if w.Code != http.StatusCreated {
-		t.Fatalf("expected 201, got %d: %s", w.Code, w.Body.String())
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-// ── Update ────────────────────────────────────────────────────────────────────
-
-func TestInstructionsHandler_Update_Success(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-	title := "Updated title"
-
-	mock.ExpectExec(regexp.QuoteMeta("UPDATE platform_instructions SET\n\t\t\t\ttitle = COALESCE($2, title),\n\t\t\t\tcontent = COALESCE($3, content),\n\t\t\t\tpriority = COALESCE($4, priority),\n\t\t\t\tenabled = COALESCE($5, enabled),\n\t\t\t\tupdated_at = NOW()\n\t\t\t\tWHERE id = $1")).
-		WithArgs(&title, "inst-1").
-		WillReturnResult(sqlmock.NewResult(0, 1))
-
-	body, _ := json.Marshal(map[string]interface{}{"title": "Updated title"})
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Params = gin.Params{{Key: "id", Value: "inst-1"}}
-	c.Request = httptest.NewRequest("PUT", "/instructions/inst-1", bytes.NewReader(body))
-	c.Request.Header.Set("Content-Type", "application/json")
-
-	handler.Update(c)
-
-	if w.Code != http.StatusOK {
-		t.Fatalf("expected 200, got %d: %s", w.Code, w.Body.String())
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-func TestInstructionsHandler_Update_NotFound(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-	title := "Updated title"
-
-	mock.ExpectExec(regexp.QuoteMeta("UPDATE platform_instructions SET\n\t\t\t\ttitle = COALESCE($2, title),\n\t\t\t\tcontent = COALESCE($3, content),\n\t\t\t\tpriority = COALESCE($4, priority),\n\t\t\t\tenabled = COALESCE($5, enabled),\n\t\t\t\tupdated_at = NOW()\n\t\t\t\tWHERE id = $1")).
-		WithArgs(&title, "nonexistent").
-		WillReturnResult(sqlmock.NewResult(0, 0))
-
-	body, _ := json.Marshal(map[string]interface{}{"title": "Updated title"})
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Params = gin.Params{{Key: "id", Value: "nonexistent"}}
-	c.Request = httptest.NewRequest("PUT", "/instructions/nonexistent", bytes.NewReader(body))
-	c.Request.Header.Set("Content-Type", "application/json")
-
-	handler.Update(c)
-
-	if w.Code != http.StatusNotFound {
-		t.Fatalf("expected 404, got %d: %s", w.Code, w.Body.String())
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-func TestInstructionsHandler_Update_ContentTooLong(t *testing.T) {
-	setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	longContent := string(bytes.Repeat([]byte("x"), 8193))
-	body, _ := json.Marshal(map[string]interface{}{"content": longContent})
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Params = gin.Params{{Key: "id", Value: "inst-1"}}
-	c.Request = httptest.NewRequest("PUT", "/instructions/inst-1", bytes.NewReader(body))
-	c.Request.Header.Set("Content-Type", "application/json")
-
-	handler.Update(c)
-
-	if w.Code != http.StatusBadRequest {
-		t.Fatalf("expected 400, got %d: %s", w.Code, w.Body.String())
-	}
-}
-
-func TestInstructionsHandler_Update_TitleTooLong(t *testing.T) {
-	setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	longTitle := string(bytes.Repeat([]byte("x"), 201))
-	body, _ := json.Marshal(map[string]interface{}{"title": longTitle})
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Params = gin.Params{{Key: "id", Value: "inst-1"}}
-	c.Request = httptest.NewRequest("PUT", "/instructions/inst-1", bytes.NewReader(body))
-	c.Request.Header.Set("Content-Type", "application/json")
-
-	handler.Update(c)
-
-	if w.Code != http.StatusBadRequest {
-		t.Fatalf("expected 400, got %d: %s", w.Code, w.Body.String())
-	}
-}
-
-// ── Delete ─────────────────────────────────────────────────────────────────────
-
-func TestInstructionsHandler_Delete_Success(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	mock.ExpectExec(regexp.QuoteMeta("DELETE FROM platform_instructions WHERE id = $1")).
-		WithArgs("inst-1").
-		WillReturnResult(sqlmock.NewResult(0, 1))
-
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Params = gin.Params{{Key: "id", Value: "inst-1"}}
-	c.Request = httptest.NewRequest("DELETE", "/instructions/inst-1", nil)
-
-	handler.Delete(c)
-
-	if w.Code != http.StatusOK {
-		t.Fatalf("expected 200, got %d: %s", w.Code, w.Body.String())
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-func TestInstructionsHandler_Delete_NotFound(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	mock.ExpectExec(regexp.QuoteMeta("DELETE FROM platform_instructions WHERE id = $1")).
-		WithArgs("nonexistent").
-		WillReturnResult(sqlmock.NewResult(0, 0))
-
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Params = gin.Params{{Key: "id", Value: "nonexistent"}}
-	c.Request = httptest.NewRequest("DELETE", "/instructions/nonexistent", nil)
-
-	handler.Delete(c)
-
-	if w.Code != http.StatusNotFound {
-		t.Fatalf("expected 404, got %d: %s", w.Code, w.Body.String())
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-// ── Resolve ────────────────────────────────────────────────────────────────────
-
-func TestInstructionsHandler_Resolve_Empty(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-	wsID := "ws-resolve-1"
-
-	mock.ExpectQuery("SELECT scope, title, content FROM platform_instructions WHERE enabled = true AND").
-		WithArgs(wsID).
-		WillReturnRows(sqlmock.NewRows([]string{"scope", "title", "content"}))
-
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Params = gin.Params{{Key: "id", Value: wsID}}
-	c.Request = httptest.NewRequest("GET", "/workspaces/"+wsID+"/instructions/resolve", nil)
-
-	handler.Resolve(c)
-
-	if w.Code != http.StatusOK {
-		t.Fatalf("expected 200, got %d: %s", w.Code, w.Body.String())
-	}
-	var resp map[string]interface{}
-	if err := json.Unmarshal(w.Body.Bytes(), &resp); err != nil {
-		t.Fatalf("invalid JSON: %v", err)
-	}
-	if resp["workspace_id"] != wsID {
-		t.Errorf("expected workspace_id %q, got %v", wsID, resp["workspace_id"])
-	}
-	if resp["instructions"] != "" {
-		t.Errorf("expected empty instructions, got %q", resp["instructions"])
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-func TestInstructionsHandler_Resolve_WithInstructions(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-	wsID := "ws-resolve-2"
-
-	rows := sqlmock.NewRows([]string{"scope", "title", "content"}).
-		AddRow("global", "Be safe", "No SSRF").
-		AddRow("workspace", "WS Rule", "Use HTTPS")
-
-	mock.ExpectQuery("SELECT scope, title, content FROM platform_instructions WHERE enabled = true AND").
-		WithArgs(wsID).
-		WillReturnRows(rows)
-
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Params = gin.Params{{Key: "id", Value: wsID}}
-	c.Request = httptest.NewRequest("GET", "/workspaces/"+wsID+"/instructions/resolve", nil)
-
-	handler.Resolve(c)
-
-	if w.Code != http.StatusOK {
-		t.Fatalf("expected 200, got %d: %s", w.Code, w.Body.String())
-	}
-	var resp map[string]interface{}
-	if err := json.Unmarshal(w.Body.Bytes(), &resp); err != nil {
-		t.Fatalf("invalid JSON: %v", err)
-	}
-	instructions, ok := resp["instructions"].(string)
-	if !ok {
-		t.Fatalf("instructions field is not a string: %T", resp["instructions"])
-	}
-	if instructions == "" {
-		t.Fatalf("expected non-empty instructions")
-	}
-	// Verify scope headers are present
-	if !bytes.Contains([]byte(instructions), []byte("Platform-Wide Rules")) {
-		t.Errorf("expected 'Platform-Wide Rules' header in instructions")
-	}
-	if !bytes.Contains([]byte(instructions), []byte("Role-Specific Rules")) {
-		t.Errorf("expected 'Role-Specific Rules' header in instructions")
-	}
-	if err := mock.ExpectationsWereMet(); err != nil {
-		t.Fatalf("unmet expectations: %v", err)
-	}
-}
-
-func TestInstructionsHandler_Resolve_MissingWorkspaceID(t *testing.T) {
-	setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Params = gin.Params{{Key: "id", Value: ""}}
-	c.Request = httptest.NewRequest("GET", "/workspaces//instructions/resolve", nil)
-
-	handler.Resolve(c)
-
-	if w.Code != http.StatusBadRequest {
-		t.Fatalf("expected 400, got %d: %s", w.Code, w.Body.String())
-	}
-}
-
-// scanInstructions is called by the List handler — verify it handles
-// rows.Err() gracefully without panicking.
-func TestInstructionsHandler_List_ScanErrorContinues(t *testing.T) {
-	mock := setupTestDB(t)
-	handler := NewInstructionsHandler()
-
-	rows := sqlmock.NewRows([]string{
-		"id", "scope", "scope_target", "title", "content", "priority", "enabled", "created_at", "updated_at",
-	}).AddRow("inst-1", "global", nil, "Good", "Content here", 5, true, time.Now(), time.Now()).
-		RowError(1, context.DeadlineExceeded) // error on row 2 (if it existed)
-
-	mock.ExpectQuery("SELECT id, scope, scope_target, title, content, priority, enabled, created_at, updated_at FROM platform_instructions WHERE 1=1").
-		WillReturnRows(rows)
-
-	w := httptest.NewRecorder()
-	c, _ := gin.CreateTestContext(w)
-	c.Request = httptest.NewRequest("GET", "/instructions", nil)
-
-	handler.List(c)
-
-	// Should still return 200 and the one valid row
-	if w.Code != http.StatusOK {
-		t.Fatalf("expected 200, got %d", w.Code)
-	}
-	var result []Instruction
-	if err := json.Unmarshal(w.Body.Bytes(), &result); err != nil {
-		t.Fatalf("invalid JSON: %v", err)
-	}
-	// The valid row should still be returned (error is logged, not fatal)
-	if len(result) != 1 {
-		t.Fatalf("expected 1 instruction despite row error, got %d", len(result))
-	}
-}

workspace-server/internal/handlers/org_helpers_security_test.go

@@ -138,6 +138,23 @@ func TestResolveInsideRoot_SiblingNotEscaped(t *testing.T) {
 
 // ── isSafeRoleName ────────────────────────────────────────────────────────────
 
+func TestIsSafeRoleName_Valid(t *testing.T) {
+	valid := []string{
+		"backend",
+		"Frontend-Engineer",
+		"research_lead",
+		"devOps123",
+		"a",
+		"A",
+		"team_42-leads",
+	}
+	for _, name := range valid {
+		if !isSafeRoleName(name) {
+			t.Errorf("isSafeRoleName(%q): expected true, got false", name)
+		}
+	}
+}
+
 func TestIsSafeRoleName_Empty(t *testing.T) {
 	if isSafeRoleName("") {
 		t.Error("isSafeRoleName(\"\"): expected false, got true")
@@ -251,6 +268,33 @@ func TestMergeCategoryRouting_WsOverrideDropsDefault(t *testing.T) {
 	}
 }
 
+func TestMergeCategoryRouting_EmptyListDropsCategory(t *testing.T) {
+	defaultRouting := map[string][]string{
+		"security": {"Backend Engineer"},
+		"ui":       {"Frontend Engineer"},
+	}
+	wsRouting := map[string][]string{
+		"security": {}, // empty list = opt out
+	}
+	got := mergeCategoryRouting(defaultRouting, wsRouting)
+	if _, exists := got["security"]; exists {
+		t.Error("empty ws list should delete the category from output")
+	}
+	if len(got["ui"]) != 1 {
+		t.Errorf("ui should still exist: got %v", got["ui"])
+	}
+}
+
+func TestMergeCategoryRouting_EmptyKeySkipped(t *testing.T) {
+	defaultRouting := map[string][]string{
+		"": {"Backend Engineer"},
+	}
+	got := mergeCategoryRouting(defaultRouting, nil)
+	if _, exists := got[""]; exists {
+		t.Error("empty key should be skipped")
+	}
+}
+
 func TestMergeCategoryRouting_EmptyRolesInDefaultSkipped(t *testing.T) {
 	defaultRouting := map[string][]string{
 		"security": {},

workspace-server/internal/registry/access_test.go

@@ -14,9 +14,8 @@ func setupMockDB(t *testing.T) sqlmock.Sqlmock {
 	if err != nil {
 		t.Fatalf("sqlmock: %v", err)
 	}
-	prevDB := db.DB
 	db.DB = mockDB
-	t.Cleanup(func() { mockDB.Close(); db.DB = prevDB })
+	t.Cleanup(func() { mockDB.Close() })
 	return mock
 }
 

workspace-server/internal/registry/healthsweep_test.go

@@ -31,9 +31,8 @@ func setupTestDB(t *testing.T) sqlmock.Sqlmock {
 	if err != nil {
 		t.Fatalf("failed to create sqlmock: %v", err)
 	}
-	prevDB := db.DB
 	db.DB = mockDB
-	t.Cleanup(func() { mockDB.Close(); db.DB = prevDB })
+	t.Cleanup(func() { mockDB.Close() })
 	return mock
 }
 

workspace-server/internal/registry/hibernation_test.go

@@ -17,9 +17,8 @@ func setupHibernationMock(t *testing.T) sqlmock.Sqlmock {
 	if err != nil {
 		t.Fatalf("sqlmock.New: %v", err)
 	}
-	prevDB := db.DB
 	db.DB = mockDB
-	t.Cleanup(func() { mockDB.Close(); db.DB = prevDB })
+	t.Cleanup(func() { mockDB.Close() })
 	return mock
 }
 

workspace-server/internal/registry/liveness_test.go

@@ -18,9 +18,8 @@ func setupLivenessTestDB(t *testing.T) sqlmock.Sqlmock {
 	if err != nil {
 		t.Fatalf("failed to create sqlmock: %v", err)
 	}
-	prevDB := db.DB
 	db.DB = mockDB
-	t.Cleanup(func() { mockDB.Close(); db.DB = prevDB })
+	t.Cleanup(func() { mockDB.Close() })
 	return mock
 }
 

workspace-server/internal/scheduler/scheduler_test.go

@@ -24,9 +24,8 @@ func setupTestDB(t *testing.T) sqlmock.Sqlmock {
 	if err != nil {
 		t.Fatalf("failed to create sqlmock: %v", err)
 	}
-	prevDB := db.DB
 	db.DB = mockDB
-	t.Cleanup(func() { mockDB.Close(); db.DB = prevDB })
+	t.Cleanup(func() { mockDB.Close() })
 	return mock
 }