fix(pr_context): CI base-ref resolution + workflow stderr visibility

pr_context now resolves the base ref for CI: tries the bare ref first
(works locally), falls back to origin/<ref> (works in detached-HEAD PR
checkouts). The codegraph-pr workflow passes origin/<base> explicitly
and surfaces stderr instead of hiding it behind a silent failure.

Validated by dogfooding the action on this PR — posted an accurate
review comment (2 files, blast radius, test gaps, reviewer).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: anvanster

.github/workflows/codegraph-pr.yml

@@ -35,12 +35,25 @@ jobs:
         run: |
           # Resolve the npm-installed binary for this platform
           BIN="$(npm root -g)/@astudioplus/codegraph-mcp/bin/codegraph-server-linux-x64"
-          chmod +x "$BIN"
-          "$BIN" \
-            --graph-only \
-            --run-tool codegraph_pr_context \
-            --tool-args "{\"baseBranch\":\"${{ github.base_ref }}\",\"format\":\"markdown\"}" \
-            > review.md 2>/dev/null || echo "CodeGraph review failed" > review.md
+          # stderr → log (visible in the Actions run); stdout → comment body
+          # Use origin/<base> — in a PR checkout the base branch exists as
+          # a remote-tracking ref, not a local branch. (codegraph 0.17.6+
+          # auto-resolves this, but origin/ works on all versions.)
+          if "$BIN" \
+              --graph-only \
+              --run-tool codegraph_pr_context \
+              --tool-args "{\"baseBranch\":\"origin/${{ github.base_ref }}\",\"format\":\"markdown\"}" \
+              > review.md 2> review.err; then
+            echo "CodeGraph review succeeded"
+          else
+            echo "::warning::CodeGraph review failed — see stderr below"
+            cat review.err
+            {
+              echo "## 🔍 CodeGraph PR Review"
+              echo ""
+              echo "Review could not be generated for this PR. <sub>(CodeGraph)</sub>"
+            } > review.md
+          fi
 
       - name: Post or update PR comment
         uses: actions/github-script@v7

crates/codegraph-server/src/mcp/server.rs

@@ -11,6 +11,8 @@
 /// happen in the Rust binary — the JS wrapper handles PostHog ingestion.
 ///
 /// Silently dropped if serialization fails (never blocks the server).
+/// The wrapper parses these lines from stderr; stdout stays reserved for
+/// the JSON-RPC channel.
 fn emit_tel(value: serde_json::Value) {
     if let Ok(json) = serde_json::to_string(&value) {
         eprintln!("TEL: {json}");
@@ -3339,6 +3341,36 @@ impl McpServer {
                     .map(|p| p.to_string_lossy().to_string())
                     .unwrap_or_else(|| ".".to_string());
 
+                // Resolve the base ref. In CI (GitHub Actions etc.) the
+                // checkout is a detached HEAD and the base branch only
+                // exists as a remote-tracking ref (origin/main), not a
+                // local branch. Try the bare ref first (works locally),
+                // then fall back to origin/<ref> (works in CI).
+                let base = {
+                    let probe = tokio::process::Command::new("git")
+                        .args(["rev-parse", "--verify", "--quiet", base])
+                        .current_dir(&workspace_root)
+                        .output()
+                        .await;
+                    let bare_ok = probe.map(|o| o.status.success()).unwrap_or(false);
+                    if bare_ok {
+                        base.to_string()
+                    } else {
+                        let remote = format!("origin/{}", base);
+                        let probe2 = tokio::process::Command::new("git")
+                            .args(["rev-parse", "--verify", "--quiet", &remote])
+                            .current_dir(&workspace_root)
+                            .output()
+                            .await;
+                        if probe2.map(|o| o.status.success()).unwrap_or(false) {
+                            remote
+                        } else {
+                            base.to_string() // let the diff surface the error
+                        }
+                    }
+                };
+                let base = base.as_str();
+
                 // ── Step 1: git diff --name-only for file list ──
                 let name_output = tokio::process::Command::new("git")
                     .args(["diff", "--name-only", &format!("{}...HEAD", base)])