fix: avoid redundant inspector workflow refresh

Inspector workflow entry should initialize workflows only when the list is empty. The previous guard still refreshed unconditionally, so the check was dead code and repeated entries could rebuild already-loaded workflows.

Constraint: Issue #209 asks for the empty-workflow guard to short-circuit subsequent refreshes

Rejected: Refresh on every Inspector entry | preserves the dead-code guard and keeps unnecessary re-fetch behavior

Confidence: high

Scope-risk: narrow

Directive: Keep Inspector workflow initialization lazy unless a future change needs explicit reload behavior

Tested: env -u GOROOT go test ./internal/app -run TestInspectorEnsureWorkflows -count=1

Tested: env -u GOROOT make test

Tested: env -u GOROOT make build

Tested: env -u GOROOT go vet ./...

Tested: git diff --check

Related: #209

Author: YoungJinJung

internal/app/screen_inspector.go

@@ -79,9 +79,7 @@ func (im *inspectorModel) refreshWorkflows() {
 func (im *inspectorModel) ensureWorkflows() {
 	if len(im.workflows) == 0 {
 		im.refreshWorkflows()
-		return
 	}
-	im.refreshWorkflows()
 }
 
 func (im *inspectorModel) Enter(m *Model) {

internal/app/screen_inspector_test.go

@@ -3,8 +3,43 @@ package app
 import (
 	"testing"
 	"unicode/utf8"
+
+	"unic/internal/inspector"
 )
 
+func TestInspectorEnsureWorkflowsKeepsExistingWorkflows(t *testing.T) {
+	im := inspectorModel{
+		checklistPath: "new-checklist.yaml",
+		workflows: []inspector.Workflow{
+			{
+				Kind:        inspector.WorkflowSecurity,
+				Title:       "Existing Workflow",
+				Description: "already loaded",
+				Available:   true,
+			},
+		},
+	}
+
+	im.ensureWorkflows()
+
+	if len(im.workflows) != 1 {
+		t.Fatalf("expected existing workflow list to be preserved, got %d workflows", len(im.workflows))
+	}
+	if im.workflows[0].Title != "Existing Workflow" {
+		t.Fatalf("expected existing workflow to be preserved, got %#v", im.workflows[0])
+	}
+}
+
+func TestInspectorEnsureWorkflowsRefreshesEmptyWorkflows(t *testing.T) {
+	im := inspectorModel{}
+
+	im.ensureWorkflows()
+
+	if len(im.workflows) == 0 {
+		t.Fatal("expected empty workflow list to be populated")
+	}
+}
+
 func TestInspectorShortenHandlesUnicodeSafely(t *testing.T) {
 	tests := []struct {
 		name  string