fix: triage all 50 review threads across 11 merged PRs (t147)

.agents/content.md

@@ -90,5 +90,3 @@ Integrate with WordPress workflow:
 - Product descriptions
 - Technical documentation
 - Marketing copy
-
-

.agents/tools/build-mcp/aidevops-plugin.md

@@ -415,7 +415,6 @@ Build the plugin when:
 1. OpenCode becomes dominant AI CLI tool
 2. Users request native plugin experience
 3. Hooks become essential (quality gates, etc.)
-4. Plugin hooks become essential (quality gates, etc.)
 
 ## References
 

TODO.md

@@ -63,15 +63,16 @@ Tasks with no open blockers - ready to work on. Use `/ready` to refresh this lis
   - [ ] t148.5 Add worker dispatch for fixing valid review feedback ~1.5h blocked-by:t148.4
   - [ ] t148.6 Add --skip-review-triage emergency bypass flag ~15m blocked-by:t148.4
   - Notes: 11 of last 50 merged PRs had CHANGES_REQUESTED. 50 unresolved review threads, 12 high/critical. Bot reviews post as COMMENTED not CHANGES_REQUESTED so reviewDecision stays NONE. Need to check unresolved threads directly, not just reviewDecision.
-- [ ] t147 Retroactive triage: 50 unresolved review threads across 11 merged PRs #quality #review ~4h (ai:3h test:30m read:30m) logged:2026-02-07 ref:GH#438
+- [x] t147 Retroactive triage: 50 unresolved review threads across 11 merged PRs #quality #review ~4h actual:3h (ai:3h) logged:2026-02-07 ref:GH#438 completed:2026-02-07
   - [x] t147.1 Triage PR #435 (4 threads, 2 high/critical) - fix $SUPERVISOR_DB bug ~30m blocked-by:none completed:2026-02-07
   - [x] t147.2 Triage PR #392 (6 threads, 4 high/critical) - stderr suppression ~45m blocked-by:none completed:2026-02-07
   - [x] t147.3 Triage PR #410 (9 threads, 3 high/critical) - VirusTotal error handling ~45m blocked-by:none started:2026-02-07 completed:2026-02-07
   - [x] t147.4 Triage PR #391 (1 thread, 1 critical) - schema-validator set -e ~15m blocked-by:none completed:2026-02-07
-  - [ ] t147.5 Triage PR #406 (3 threads, 1 high) - sed -i portability, attribution ~30m blocked-by:none
-    - Notes: BLOCKED by supervisor: FAILED: no_log_file  - [x] t147.6 Triage PR #403 (12 threads, 1 high) - voice AI unimplemented commands ~30m blocked-by:none completed:2026-02-07
-  - [ ] t147.7 Triage remaining PRs #418,#413,#412,#399,#394 (17 threads, 0 high) ~30m blocked-by:none
-  - Notes: For each thread: verify claim against code, fix real bugs, dismiss false positives with evidence reply. Priority: high/critical first. BLOCKED: FAILED: no_log_file
+  - [x] t147.5 Triage PR #406 (3 threads, 1 high) - sed -i portability, attribution ~30m blocked-by:none completed:2026-02-07
+    - Notes: Threads resolved on GitHub. sed -i fix tracked separately in t145.
+  - [x] t147.6 Triage PR #403 (12 threads, 1 high) - voice AI unimplemented commands ~30m blocked-by:none completed:2026-02-07
+  - [x] t147.7 Triage remaining PRs #418,#413,#412,#399,#394 (17 threads, 0 high) ~30m blocked-by:none completed:2026-02-07
+  - Notes: All 50 review threads across 11 PRs resolved. Valid bugs fixed in code, false positives dismissed with evidence. sed -i portability tracked in t145.
 - [x] t150 feat: supervisor self-healing - auto-create diagnostic subtask on failure/block #enhancement #supervisor #orchestration ~3h actual:2h (ai:2h) logged:2026-02-07 started:2026-02-07 completed:2026-02-07
   - [x] t150.1 Add create_diagnostic_subtask() function to supervisor-helper.sh ~1h completed:2026-02-07
   - [x] t150.2 Wire self-healing into pulse cycle blocked/failed handlers ~45m completed:2026-02-07

setup.sh

@@ -2041,8 +2041,9 @@ add_local_bin_to_path() {
     while IFS= read -r rc_file; do
         [[ -z "$rc_file" ]] && continue
 
-        # Create the rc file if it doesn't exist
+        # Create the rc file if it doesn't exist (ensure parent dir exists for fish etc.)
         if [[ ! -f "$rc_file" ]]; then
+            mkdir -p "$(dirname "$rc_file")"
             touch "$rc_file"
         fi
 
@@ -2160,7 +2161,7 @@ alias aws-helper './.agents/scripts/aws-helper.sh'
 ALIASES
 )
 
-    # Check if aliases already exist in any rc file
+    # Check if aliases already exist in any rc file (including fish config)
     local any_configured=false
     local rc_file
     while IFS= read -r rc_file; do
@@ -2170,6 +2171,13 @@ ALIASES
             break
         fi
     done < <(get_all_shell_rcs)
+    # Also check fish config (not included in get_all_shell_rcs on macOS)
+    if [[ "$any_configured" == "false" ]]; then
+        local fish_config="$HOME/.config/fish/config.fish"
+        if grep -q "# AI Assistant Server Access" "$fish_config" 2>/dev/null; then
+            any_configured=true
+        fi
+    fi
 
     if [[ "$any_configured" == "true" ]]; then
         print_info "Server Access aliases already configured - Skipping"
@@ -3787,14 +3795,6 @@ setup_opencode_plugins() {
 
     return 0
 }
-
-# Setup Oh-My-OpenCode Plugin (removed - no longer supported)
-# Kept as stub for backward compatibility with any callers
-setup_oh_my_opencode() {
-    # Removed - oh-my-opencode no longer supported
-    return 0
-}
-
 setup_seo_mcps() {
     print_info "Setting up SEO integrations..."