feat: Remove quick mode from secure-release & reorganize dashboard workflows

## Secure Release Changes
- Remove "quick" mode entirely (use release-prep for quick checks)
- Modes now: "full" (DEFAULT with crew) and "standard" (fallback without crew)
- Add mode validation with ValueError for invalid modes
- Remove for_quick_check() class method
- CodeReviewWorkflow now always runs (not skipped in any mode)
- Update all tests to use valid modes only

## Dashboard Workflow Reorganization
- Reorganize 14 workflow buttons using Purpose Grouping
- Row 1: Code Review & Analysis (Review File | Run Analysis)
- Row 2: Pull Request Review (Review PR | empty)
- Row 3: Documentation (Manage Docs | Generate Docs) - side by side!
- Row 4: Code Quality (Generate Tests | Refactor Plan)
- Row 5: Security (Security Audit | Secure Release)
- Row 6: Performance & Health (Perf Audit | Check Health)
- Row 7: Prediction & Dependencies (Predict Bugs | Check Deps)
- Row 8: Release (Release Prep | empty)

## Tooltip Enhancements
- Add comprehensive tooltips to all Quick Action buttons
- Add comprehensive tooltips to all Workflow buttons
- Update secure-release tooltip to clarify always comprehensive

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>

Author: 2 people

src/empathy_os/workflows/secure_release.py

@@ -89,9 +89,11 @@ class SecureReleasePipeline:
     maximum coverage before release approval.
 
     Execution modes:
-    - "full": Run all workflows (SecurityAuditCrew + all workflows)
-    - "standard": Skip crew, run all workflows
-    - "quick": Only SecurityAuditWorkflow + ReleasePreparation
+    - "full": Run all workflows (SecurityAuditCrew + all workflows) [DEFAULT]
+    - "standard": Skip crew, run all workflows (fallback when crew unavailable)
+
+    Note: For quick release checks without full security audit, use the
+    ReleasePreparationWorkflow directly instead.
 
     Usage:
         pipeline = SecureReleasePipeline(mode="full")
@@ -113,7 +115,7 @@ class SecureReleasePipeline:
 
     def __init__(
         self,
-        mode: str = "full",  # "full", "standard", "quick"
+        mode: str = "full",  # "full" or "standard"
         use_crew: bool | None = None,  # Override mode's crew setting
         parallel_crew: bool = True,  # Run crew in parallel with first workflow
         crew_config: dict | None = None,
@@ -123,12 +125,16 @@ def __init__(
         Initialize secure release pipeline.
 
         Args:
-            mode: Execution mode ("full", "standard", "quick")
-            use_crew: Override crew setting (None uses mode default)
+            mode: Execution mode - "full" (with crew, DEFAULT) or "standard" (skip crew)
+            use_crew: Override crew setting (None uses mode default: full=True, standard=False)
             parallel_crew: Run SecurityAuditCrew in parallel with first workflow
             crew_config: Configuration for SecurityAuditCrew
             **kwargs: Additional arguments passed to child workflows
         """
+        # Validate mode
+        if mode not in ("full", "standard"):
+            raise ValueError(f"Invalid mode '{mode}'. Must be 'full' or 'standard'.")
+
         self.mode = mode
         self.use_crew = use_crew if use_crew is not None else (mode == "full")
         self.parallel_crew = parallel_crew
@@ -206,8 +212,8 @@ async def execute(
                     logger.warning("SecurityAuditCrew timed out")
                     warnings.append("SecurityAuditCrew timed out - results not included")
 
-            # Step 3: CodeReviewWorkflow (if not quick mode and diff provided)
-            if self.mode != "quick" and diff:
+            # Step 3: CodeReviewWorkflow (if diff provided)
+            if diff:
                 from .code_review import CodeReviewWorkflow
 
                 code_workflow = CodeReviewWorkflow(**self.kwargs)
@@ -438,14 +444,6 @@ def for_release(cls) -> "SecureReleasePipeline":
             crew_config={"scan_depth": "thorough"},
         )
 
-    @classmethod
-    def for_quick_check(cls) -> "SecureReleasePipeline":
-        """Create pipeline for quick security check."""
-        return cls(
-            mode="quick",
-            use_crew=False,
-        )
-
 
 def format_secure_release_report(result: SecureReleaseResult) -> str:
     """

tests/test_secure_release.py

@@ -4,14 +4,16 @@
 Tests the SecureReleasePipeline including:
 - SecureReleaseResult dataclass
 - SecureReleasePipeline initialization
-- Execution modes (full, standard, quick)
+- Execution modes (full, standard)
 - Go/No-Go decision logic
 - Report formatting
 
 Copyright 2025 Smart AI Memory, LLC
 Licensed under Fair Source 0.9
 """
 
+import pytest
+
 from src.empathy_os.workflows.secure_release import SecureReleasePipeline, SecureReleaseResult
 
 
@@ -164,14 +166,14 @@ def test_mode_standard(self):
         )
         assert result.mode == "standard"
 
-    def test_mode_quick(self):
-        """Test quick mode."""
+    def test_mode_invalid(self):
+        """Test result can store any mode string (for backwards compat)."""
         result = SecureReleaseResult(
             success=True,
             go_no_go="GO",
-            mode="quick",
+            mode="custom",
         )
-        assert result.mode == "quick"
+        assert result.mode == "custom"
 
 
 class TestSecureReleasePipelineInit:
@@ -192,15 +194,15 @@ def test_mode_standard(self):
         pipeline = SecureReleasePipeline(mode="standard")
         assert pipeline.mode == "standard"
 
-    def test_mode_quick(self):
-        """Test initialization with quick mode."""
-        pipeline = SecureReleasePipeline(mode="quick")
-        assert pipeline.mode == "quick"
+    def test_invalid_mode(self):
+        """Test initialization with invalid mode raises ValueError."""
+        with pytest.raises(ValueError, match="Invalid mode 'quick'"):
+            SecureReleasePipeline(mode="quick")
 
     def test_with_kwargs(self):
         """Test initialization with additional kwargs."""
-        pipeline = SecureReleasePipeline(mode="quick", provider="openai")
-        assert pipeline.mode == "quick"
+        pipeline = SecureReleasePipeline(mode="standard", provider="openai")
+        assert pipeline.mode == "standard"
         assert pipeline.kwargs.get("provider") == "openai"
 
 
@@ -209,14 +211,14 @@ class TestSecureReleasePipelineExecution:
 
     def test_execute_method_exists(self):
         """Test execute method exists."""
-        pipeline = SecureReleasePipeline(mode="quick")
+        pipeline = SecureReleasePipeline(mode="standard")
         assert hasattr(pipeline, "execute")
         assert callable(pipeline.execute)
 
     def test_pipeline_attributes(self):
         """Test pipeline has expected attributes."""
-        pipeline = SecureReleasePipeline(mode="quick")
-        assert pipeline.mode == "quick"
+        pipeline = SecureReleasePipeline(mode="standard")
+        assert pipeline.mode == "standard"
         assert hasattr(pipeline, "use_crew")
         assert hasattr(pipeline, "parallel_crew")
 
@@ -331,7 +333,7 @@ class TestSecureReleasePipelineIntegration:
 
     def test_pipeline_can_be_instantiated(self):
         """Test pipeline can be instantiated in all modes."""
-        for mode in ["full", "standard", "quick"]:
+        for mode in ["full", "standard"]:
             pipeline = SecureReleasePipeline(mode=mode)
             assert pipeline is not None
             assert pipeline.mode == mode

vscode-extension/src/panels/EmpathyDashboardPanel.ts

@@ -934,6 +934,7 @@ export class EmpathyDashboardProvider implements vscode.WebviewViewProvider {
             }
 
             // Special handling for code-review: send findings to CodeReviewPanel
+            let openedInPanel = false;
             if (workflowName === 'code-review' && success && stdout) {
                 try {
                     // Parse JSON output from CLI (uses --json flag)
@@ -964,6 +965,7 @@ export class EmpathyDashboardProvider implements vscode.WebviewViewProvider {
 
                         // Show the panel
                         await vscode.commands.executeCommand('empathy-code-review.focus');
+                        openedInPanel = true;
 
                         // Notify user
                         const findingCount = result.findings.length;
@@ -1009,9 +1011,10 @@ export class EmpathyDashboardProvider implements vscode.WebviewViewProvider {
                 data: {
                     workflow: workflowName,
                     status: success ? 'complete' : 'error',
-                    output: openedInEditor ? '(Report opened in editor)' : output,
+                    output: openedInEditor ? '(Report opened in editor)' : openedInPanel ? '(Results shown in Code Review panel)' : output,
                     error: error ? error.message : null,
-                    openedInEditor: openedInEditor
+                    openedInEditor: openedInEditor,
+                    openedInPanel: openedInPanel
                 }
             });
         });
@@ -2461,27 +2464,27 @@ export class EmpathyDashboardProvider implements vscode.WebviewViewProvider {
         <div class="card">
             <div class="card-title">Quick Actions</div>
             <div class="actions-grid workflow-grid">
-                <button class="action-btn workflow-btn" data-cmd="morning" data-title="Morning Briefing">
+                <button class="action-btn workflow-btn" data-cmd="morning" data-title="Morning Briefing" title="Daily project status summary with priorities and blockers">
                     <span class="action-icon">&#x2600;</span>
                     <span>Get Briefing</span>
                 </button>
-                <button class="action-btn workflow-btn" data-cmd="ship" data-title="Pre-Ship Check">
+                <button class="action-btn workflow-btn" data-cmd="ship" data-title="Pre-Ship Check" title="Pre-release quality gate: health, security, and changelog checks">
                     <span class="action-icon">&#x1F680;</span>
                     <span>Run Ship</span>
                 </button>
-                <button class="action-btn workflow-btn" data-cmd="fix-all" data-title="Fix All Issues">
+                <button class="action-btn workflow-btn" data-cmd="fix-all" data-title="Fix All Issues" title="Auto-fix linting, formatting, and safe code issues">
                     <span class="action-icon">&#x1F527;</span>
                     <span>Fix Issues</span>
                 </button>
-                <button class="action-btn workflow-btn" data-cmd="learn" data-title="Learn Patterns">
+                <button class="action-btn workflow-btn" data-cmd="learn" data-title="Learn Patterns" title="Analyze recent commits to learn debugging and refactoring patterns">
                     <span class="action-icon">&#x1F4DA;</span>
                     <span>Learn Patterns</span>
                 </button>
-                <button class="action-btn workflow-btn" data-cmd="run-tests" data-title="Run Tests">
+                <button class="action-btn workflow-btn" data-cmd="run-tests" data-title="Run Tests" title="Execute test suite and display results">
                     <span class="action-icon">&#x1F9EA;</span>
                     <span>Run Tests</span>
                 </button>
-                <button class="action-btn" data-cmd="initialize" data-title="Setup Wizard">
+                <button class="action-btn" data-cmd="initialize" data-title="Setup Wizard" title="First-time setup wizard for API keys and project config">
                     <span class="action-icon">&#x2699;</span>
                     <span>Setup</span>
                 </button>
@@ -2513,63 +2516,79 @@ export class EmpathyDashboardProvider implements vscode.WebviewViewProvider {
         <div class="card" style="margin-top: 12px">
             <div class="card-title">Workflows <span style="font-size: 10px; opacity: 0.6;">(Beta)</span></div>
             <div class="actions-grid workflow-grid">
-                <button class="action-btn workflow-btn" data-workflow="code-review">
+                <!-- Row 1: Code Review & Analysis -->
+                <button class="action-btn workflow-btn" data-workflow="code-review" title="Tiered code analysis with security, quality, and architecture review">
                     <span class="action-icon">&#x1F50D;</span>
                     <span>Review File</span>
                 </button>
-                <button class="action-btn workflow-btn" data-workflow="pro-review">
+                <button class="action-btn workflow-btn" data-workflow="pro-review" title="Advanced code analysis for diffs and pull requests">
                     <span class="action-icon">&#x2B50;</span>
                     <span>Run Analysis</span>
                 </button>
-                <button class="action-btn workflow-btn" data-workflow="doc-orchestrator">
+
+                <!-- Row 2: Pull Request Review -->
+                <button class="action-btn workflow-btn" data-workflow="pr-review" title="Comprehensive pull request review with diff analysis">
+                    <span class="action-icon">&#x1F50D;</span>
+                    <span>Review PR</span>
+                </button>
+                <!-- TODO: Add second PR-related workflow here -->
+
+                <!-- Row 3: Documentation -->
+                <button class="action-btn workflow-btn" data-workflow="doc-orchestrator" title="End-to-end documentation management: scout gaps, prioritize, generate">
                     <span class="action-icon">&#x1F4DA;</span>
                     <span>Manage Docs</span>
                 </button>
-                <button class="action-btn workflow-btn" data-workflow="bug-predict">
-                    <span class="action-icon">&#x1F41B;</span>
-                    <span>Predict Bugs</span>
-                </button>
-                <button class="action-btn workflow-btn" data-workflow="security-audit">
-                    <span class="action-icon">&#x1F512;</span>
-                    <span>Security Audit</span>
-                </button>
-                <button class="action-btn workflow-btn" data-workflow="perf-audit">
-                    <span class="action-icon">&#x26A1;</span>
-                    <span>Perf Audit</span>
+                <button class="action-btn workflow-btn" data-workflow="doc-gen" title="Cost-optimized documentation generation: outline → write → polish">
+                    <span class="action-icon">&#x1F4C4;</span>
+                    <span>Generate Docs</span>
                 </button>
-                <button class="action-btn workflow-btn" id="btn-test-gen-direct" data-workflow="test-gen">
+
+                <!-- Row 4: Code Quality -->
+                <button class="action-btn workflow-btn" id="btn-test-gen-direct" data-workflow="test-gen" title="Generate tests targeting areas with historical bugs and low coverage">
                     <span class="action-icon">&#x1F9EA;</span>
                     <span>Generate Tests</span>
                 </button>
-                <button class="action-btn workflow-btn" data-workflow="refactor-plan">
+                <button class="action-btn workflow-btn" data-workflow="refactor-plan" title="Prioritize tech debt based on code trajectory and maintenance impact">
                     <span class="action-icon">&#x1F3D7;</span>
                     <span>Refactor Plan</span>
                 </button>
-                <button class="action-btn workflow-btn" data-workflow="dependency-check">
-                    <span class="action-icon">&#x1F4E6;</span>
-                    <span>Check Deps</span>
+
+                <!-- Row 5: Security -->
+                <button class="action-btn workflow-btn" data-workflow="security-audit" title="OWASP-focused security scan with vulnerability assessment and remediation">
+                    <span class="action-icon">&#x1F512;</span>
+                    <span>Security Audit</span>
+                </button>
+                <button class="action-btn workflow-btn" data-workflow="secure-release" title="Full security pipeline: audit crew + OWASP scan + code review + release prep (always comprehensive)">
+                    <span class="action-icon">&#x1F510;</span>
+                    <span>Secure Release</span>
+                </button>
+
+                <!-- Row 6: Performance & Health -->
+                <button class="action-btn workflow-btn" data-workflow="perf-audit" title="Identify performance bottlenecks and optimization opportunities">
+                    <span class="action-icon">&#x26A1;</span>
+                    <span>Perf Audit</span>
                 </button>
-                <button class="action-btn workflow-btn" data-workflow="health-check" title="Run HealthCheckCrew for comprehensive 5-agent project health analysis">
+                <button class="action-btn workflow-btn" data-workflow="health-check" title="Project health diagnosis and fixing with 5-agent crew">
                     <span class="action-icon">&#x1FA7A;</span>
                     <span>Check Health</span>
                 </button>
-                <button class="action-btn workflow-btn" data-workflow="pr-review">
-                    <span class="action-icon">&#x1F50D;</span>
-                    <span>Review PR</span>
+
+                <!-- Row 7: Prediction & Dependencies -->
+                <button class="action-btn workflow-btn" data-workflow="bug-predict" title="Predict bugs by analyzing code against learned patterns and history">
+                    <span class="action-icon">&#x1F41B;</span>
+                    <span>Predict Bugs</span>
                 </button>
-                <!-- Hidden for v3.5.5 release - TODO: enable after workflow wizard is complete -->
-                <button class="action-btn workflow-btn" data-workflow="doc-gen">
-                    <span class="action-icon">&#x1F4C4;</span>
-                    <span>Generate Docs</span>
+                <button class="action-btn workflow-btn" data-workflow="dependency-check" title="Audit dependencies for security vulnerabilities and available updates">
+                    <span class="action-icon">&#x1F4E6;</span>
+                    <span>Check Deps</span>
                 </button>
-                <button class="action-btn workflow-btn" data-workflow="release-prep">
+
+                <!-- Row 8: Release -->
+                <button class="action-btn workflow-btn" data-workflow="release-prep" title="Pre-release quality gate with health, security, and changelog validation">
                     <span class="action-icon">&#x1F680;</span>
                     <span>Release Prep</span>
                 </button>
-                <button class="action-btn workflow-btn" data-workflow="secure-release">
-                    <span class="action-icon">&#x1F510;</span>
-                    <span>Secure Release</span>
-                </button>                <button class="action-btn workflow-btn new-workflow-btn" id="btn-new-workflow" title="Create a new workflow from template" style="display: none;">
+                <!-- TODO: Add second release-related workflow here -->                <button class="action-btn workflow-btn new-workflow-btn" id="btn-new-workflow" title="Create a new workflow from template" style="display: none;">
                     <span class="action-icon">&#x2795;</span>
                     <span>New Workflow</span>
                 </button>