fix: Detect calling agent via env vars and centralize agent config (backnotprop#418)

* fix: detect calling agent via env vars for correct badge display

Add prioritized agent detection chain: CODEX_THREAD_ID (codex) >
COPILOT_CLI (copilot-cli) > default (claude-code). All subcommands
use the shared detectedOrigin constant. Display name updated from
'Copilot CLI' to 'GitHub Copilot'. TypeScript origin union type
updated to include 'copilot-cli'.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* refactor: centralize agent config into @plannotator/shared/agents

- Create packages/shared/agents.ts as single source of truth for Origin type,
  agent display names (AGENT_CONFIG), getAgentName(), and getAgentBadge()
- Replace all inline origin type unions across 8 files with Origin import
- Type detectedOrigin and ServerOptions.origin as Origin for compile-time safety
- Add @plannotator/shared dependency to @plannotator/editor package.json
- Adding a new agent is now a one-file change to AGENT_CONFIG

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: restore gitContext in startReviewServer call

Accidentally removed alongside the origin refactor. Without it, local
code reviews lose branch detection, diff-type switching, and CWD resolution.

For provenance purposes, this commit was AI assisted.

---------

Co-authored-by: Yecats <Yecats@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Michael Ramos <mdramos8@gmail.com>

Author: 3 people

apps/hook/server/index.ts

@@ -67,6 +67,7 @@ import { registerSession, unregisterSession, listSessions } from "@plannotator/s
 import { openBrowser } from "@plannotator/server/browser";
 import { detectProjectName } from "@plannotator/server/project";
 import { planDenyFeedback } from "@plannotator/shared/feedback-templates";
+import type { Origin } from "@plannotator/shared/agents";
 import { findSessionLogsForCwd, resolveSessionLogByPpid, findSessionLogsByAncestorWalk, getLastRenderedMessage, type RenderedMessage } from "./session-log";
 import { findCodexRolloutByThreadId, getLastCodexMessage } from "./codex-session";
 import { findCopilotPlanContent, findCopilotSessionForCwd, getLastCopilotMessage } from "./copilot-session";
@@ -103,6 +104,13 @@ const shareBaseUrl = process.env.PLANNOTATOR_SHARE_URL || undefined;
 // Paste service URL for short URL sharing
 const pasteApiUrl = process.env.PLANNOTATOR_PASTE_URL || undefined;
 
+// Detect calling agent from environment variables set by agent runtimes.
+// Priority: Codex > Copilot CLI > Claude Code (default fallback)
+const detectedOrigin: Origin =
+  process.env.CODEX_THREAD_ID ? "codex" :
+  process.env.COPILOT_CLI ? "copilot-cli" :
+  "claude-code";
+
 if (args[0] === "sessions") {
   // ============================================
   // SESSION DISCOVERY MODE
@@ -215,7 +223,7 @@ if (args[0] === "sessions") {
     rawPatch,
     gitRef,
     error: diffError,
-    origin: "claude-code",
+    origin: detectedOrigin,
     diffType: isPRMode ? undefined : "uncommitted",
     gitContext,
     prMetadata,
@@ -337,7 +345,7 @@ if (args[0] === "sessions") {
   const server = await startAnnotateServer({
     markdown,
     filePath: absolutePath,
-    origin: "claude-code",
+    origin: detectedOrigin,
     mode: annotateMode,
     folderPath,
     sharingEnabled,
@@ -456,7 +464,7 @@ if (args[0] === "sessions") {
   const server = await startAnnotateServer({
     markdown: lastMessage.text,
     filePath: "last-message",
-    origin: isCodex ? "codex" : "claude-code",
+    origin: detectedOrigin,
     mode: "annotate-last",
     sharingEnabled,
     shareBaseUrl,
@@ -499,7 +507,7 @@ if (args[0] === "sessions") {
 
   const server = await startPlannotatorServer({
     plan: "",
-    origin: "claude-code",
+    origin: detectedOrigin,
     mode: "archive",
     sharingEnabled,
     shareBaseUrl,
@@ -704,7 +712,7 @@ if (args[0] === "sessions") {
   // Start the plan review server
   const server = await startPlannotatorServer({
     plan: planContent,
-    origin: "claude-code",
+    origin: detectedOrigin,
     permissionMode,
     sharingEnabled,
     shareBaseUrl,

bun.lock

@@ -1,4 +1,5 @@
 import React, { useState, useEffect, useMemo, useRef } from 'react';
+import { type Origin, getAgentName, getAgentBadge } from '@plannotator/shared/agents';
 import { parseMarkdownToBlocks, exportAnnotations, exportLinkedDocAnnotations, exportEditorAnnotations, extractFrontmatter, wrapFeedbackForAgent, Frontmatter } from '@plannotator/ui/utils/parser';
 import { Viewer, ViewerHandle } from '@plannotator/ui/components/Viewer';
 import { AnnotationPanel } from '@plannotator/ui/components/AnnotationPanel';
@@ -85,7 +86,7 @@ const App: React.FC = () => {
   });
   const [uiPrefs, setUiPrefs] = useState(() => getUIPreferences());
   const [isApiMode, setIsApiMode] = useState(false);
-  const [origin, setOrigin] = useState<'claude-code' | 'opencode' | 'pi' | 'codex' | null>(null);
+  const [origin, setOrigin] = useState<Origin | null>(null);
   const [gitUser, setGitUser] = useState<string | undefined>();
   const [isWSL, setIsWSL] = useState(false);
   const [globalAttachments, setGlobalAttachments] = useState<ImageAttachment[]>([]);
@@ -388,7 +389,7 @@ const App: React.FC = () => {
         if (!res.ok) throw new Error('Not in API mode');
         return res.json();
       })
-      .then((data: { plan: string; origin?: 'claude-code' | 'opencode' | 'pi' | 'codex'; mode?: 'annotate' | 'annotate-last' | 'annotate-folder' | 'archive'; filePath?: string; sharingEnabled?: boolean; shareBaseUrl?: string; pasteApiUrl?: string; repoInfo?: { display: string; branch?: string }; previousPlan?: string | null; versionInfo?: { version: number; totalVersions: number; project: string }; archivePlans?: ArchivedPlan[]; projectRoot?: string; isWSL?: boolean; serverConfig?: { displayName?: string; gitUser?: string } }) => {
+      .then((data: { plan: string; origin?: Origin; mode?: 'annotate' | 'annotate-last' | 'annotate-folder' | 'archive'; filePath?: string; sharingEnabled?: boolean; shareBaseUrl?: string; pasteApiUrl?: string; repoInfo?: { display: string; branch?: string }; previousPlan?: string | null; versionInfo?: { version: number; totalVersions: number; project: string }; archivePlans?: ArchivedPlan[]; projectRoot?: string; isWSL?: boolean; serverConfig?: { displayName?: string; gitUser?: string } }) => {
         // Initialize config store with server-provided values (config file > cookie > default)
         configStore.init(data.serverConfig);
         // gitUser drives the "Use git name" button in Settings; stays undefined (button hidden) when unavailable
@@ -986,14 +987,7 @@ const App: React.FC = () => {
     return () => document.removeEventListener('pointerdown', handleClickOutside);
   }, [showExportDropdown]);
 
-  const agentName = useMemo(() => {
-    if (origin === 'opencode') return 'OpenCode';
-    if (origin === 'claude-code') return 'Claude Code';
-    if (origin === 'copilot-cli') return 'Copilot CLI';
-    if (origin === 'pi') return 'Pi';
-    if (origin === 'codex') return 'Codex';
-    return 'Coding Agent';
-  }, [origin]);
+  const agentName = useMemo(() => getAgentName(origin), [origin]);
 
   const planMaxWidth = useMemo(() => {
     const widths: Record<PlanWidth, number> = { compact: 832, default: 1040, wide: 1280 };
@@ -1024,15 +1018,7 @@ const App: React.FC = () => {
               v{typeof __APP_VERSION__ !== 'undefined' ? __APP_VERSION__ : '0.0.0'}
             </a>
             {origin && (
-              <span className={`text-[10px] px-1.5 py-0.5 rounded font-medium hidden md:inline ${
-                origin === 'claude-code'
-                  ? 'bg-orange-500/15 text-orange-400'
-                  : origin === 'pi'
-                    ? 'bg-violet-500/15 text-violet-400'
-                    : origin === 'copilot-cli'
-                      ? 'bg-blue-500/15 text-blue-400'
-                      : 'bg-zinc-500/20 text-zinc-400'
-              }`}>
+              <span className={`text-[10px] px-1.5 py-0.5 rounded font-medium hidden md:inline ${getAgentBadge(origin)}`}>
                 {agentName}
               </span>
             )}

packages/editor/App.tsx

@@ -7,6 +7,7 @@
     "./styles": "./index.css"
   },
   "dependencies": {
+    "@plannotator/shared": "workspace:*",
     "@plannotator/ui": "workspace:*",
     "react": "^19.2.3",
     "react-dom": "^19.2.3",

packages/editor/package.json

@@ -1,4 +1,5 @@
 import React, { useState, useEffect, useCallback, useMemo } from 'react';
+import { type Origin, getAgentName } from '@plannotator/shared/agents';
 import { ThemeProvider, useTheme } from '@plannotator/ui/components/ThemeProvider';
 import { ModeToggle } from '@plannotator/ui/components/ModeToggle';
 import { ConfirmDialog } from '@plannotator/ui/components/ConfirmDialog';
@@ -43,7 +44,7 @@ interface DiffData {
   files: DiffFile[];
   rawPatch: string;
   gitRef: string;
-  origin?: 'opencode' | 'claude-code' | 'pi';
+  origin?: Origin;
   diffType?: string;
   gitContext?: GitContext;
   sharingEnabled?: boolean;
@@ -99,7 +100,7 @@ const ReviewApp: React.FC = () => {
   const [copyFeedback, setCopyFeedback] = useState<string | null>(null);
   const [viewedFiles, setViewedFiles] = useState<Set<string>>(new Set());
   const [hideViewedFiles, setHideViewedFiles] = useState(false);
-  const [origin, setOrigin] = useState<'opencode' | 'claude-code' | 'pi' | null>(null);
+  const [origin, setOrigin] = useState<Origin | null>(null);
   const [gitUser, setGitUser] = useState<string | undefined>();
   const [isWSL, setIsWSL] = useState(false);
   const [diffType, setDiffType] = useState<string>('uncommitted');
@@ -383,7 +384,7 @@ const ReviewApp: React.FC = () => {
       .then((data: {
         rawPatch: string;
         gitRef: string;
-        origin?: 'opencode' | 'claude-code' | 'pi';
+        origin?: Origin;
         diffType?: string;
         gitContext?: GitContext;
         sharingEnabled?: boolean;
@@ -1566,10 +1567,10 @@ const ReviewApp: React.FC = () => {
                 ? `Your approval was submitted to ${platformLabel}.`
                 : `Your feedback was submitted to ${platformLabel}.`
               : submitted === 'approved'
-                ? `${origin === 'claude-code' ? 'Claude Code' : origin === 'opencode' ? 'OpenCode' : origin === 'pi' ? 'Pi' : 'Your agent'} will proceed with the changes.`
-                : `${origin === 'claude-code' ? 'Claude Code' : origin === 'opencode' ? 'OpenCode' : origin === 'pi' ? 'Pi' : 'Your agent'} will address your review feedback.`
+                ? `${getAgentName(origin)} will proceed with the changes.`
+                : `${getAgentName(origin)} will address your review feedback.`
           }
-          agentLabel={origin === 'claude-code' ? 'Claude Code' : origin === 'opencode' ? 'OpenCode' : origin === 'pi' ? 'Pi' : 'Your agent'}
+          agentLabel={getAgentName(origin)}
         />
 
         {/* Update notification */}

packages/review-editor/App.tsx

@@ -13,6 +13,7 @@
 
 import { isRemoteSession, getServerPort } from "./remote";
 import { getRepoInfo } from "./repo";
+import type { Origin } from "@plannotator/shared/agents";
 import { handleImage, handleUpload, handleServerReady, handleDraftSave, handleDraftLoad, handleDraftDelete, handleFavicon } from "./shared-handlers";
 import { handleDoc, handleFileBrowserFiles } from "./reference-handlers";
 import { contentHash, deleteDraft } from "./draft";
@@ -35,7 +36,7 @@ export interface AnnotateServerOptions {
   /** HTML content to serve for the UI */
   htmlContent: string;
   /** Origin identifier for UI customization */
-  origin?: "opencode" | "claude-code" | "pi" | "codex";
+  origin?: Origin;
   /** UI mode: "annotate" for files, "annotate-last" for last agent message, "annotate-folder" for folders */
   mode?: "annotate" | "annotate-last" | "annotate-folder";
   /** Folder path when annotating a directory (used as projectRoot for file browser) */

packages/server/annotate.ts

@@ -9,6 +9,7 @@
  *   PLANNOTATOR_ORIGIN - Origin identifier ("claude-code" or "opencode")
  */
 
+import type { Origin } from "@plannotator/shared/agents";
 import { resolve } from "path";
 import { isRemoteSession, getServerPort } from "./remote";
 import { openEditorDiff } from "./ide";
@@ -58,7 +59,7 @@ export interface ServerOptions {
   /** The plan markdown content */
   plan: string;
   /** Origin identifier (e.g., "claude-code", "opencode") */
-  origin: string;
+  origin: Origin;
   /** HTML content to serve for the UI */
   htmlContent: string;
   /** Current permission mode to preserve (Claude Code only) */

packages/server/index.ts

@@ -10,6 +10,7 @@
  */
 
 import { isRemoteSession, getServerPort } from "./remote";
+import type { Origin } from "@plannotator/shared/agents";
 import { type DiffType, type GitContext, runGitDiff, getFileContentsForDiff, gitAddFile, gitResetFile, parseWorktreeDiffType, validateFilePath } from "./git";
 import { getRepoInfo } from "./repo";
 import { handleImage, handleUpload, handleAgents, handleServerReady, handleDraftSave, handleDraftLoad, handleDraftDelete, handleFavicon, type OpencodeClient } from "./shared-handlers";
@@ -39,7 +40,7 @@ export interface ReviewServerOptions {
   /** HTML content to serve for the UI */
   htmlContent: string;
   /** Origin identifier for UI customization */
-  origin?: "opencode" | "claude-code" | "pi";
+  origin?: Origin;
   /** Current diff type being displayed */
   diffType?: DiffType;
   /** Git context with branch info and available diff options */

packages/server/review.ts

@@ -0,0 +1,32 @@
+/**
+ * Centralized agent configuration — single source of truth for all supported agents.
+ *
+ * To add a new agent:
+ *   1. Add an entry to AGENT_CONFIG below (origin key, display name, badge CSS classes)
+ *   2. If detection is via environment variable, add it to the detection chain
+ *      in apps/hook/server/index.ts (detectedOrigin constant)
+ *   3. That's it — all UI components read from this config automatically
+ */
+
+export const AGENT_CONFIG = {
+  'claude-code': { name: 'Claude Code', badge: 'bg-orange-500/15 text-orange-400' },
+  'opencode':    { name: 'OpenCode',    badge: 'bg-emerald-500/15 text-emerald-400' },
+  'copilot-cli': { name: 'GitHub Copilot', badge: 'bg-blue-500/15 text-blue-400' },
+  'pi':          { name: 'Pi',          badge: 'bg-violet-500/15 text-violet-400' },
+  'codex':       { name: 'Codex',       badge: 'bg-purple-500/15 text-purple-400' },
+} as const;
+
+/** All recognized origin values. */
+export type Origin = keyof typeof AGENT_CONFIG;
+
+/** Resolve an origin to a human-readable agent name. */
+export function getAgentName(origin: Origin | null | undefined): string {
+  if (origin && origin in AGENT_CONFIG) return AGENT_CONFIG[origin as Origin].name;
+  return 'Coding Agent';
+}
+
+/** Resolve an origin to Tailwind badge classes. */
+export function getAgentBadge(origin: Origin | null | undefined): string {
+  if (origin && origin in AGENT_CONFIG) return AGENT_CONFIG[origin as Origin].badge;
+  return 'bg-zinc-500/20 text-zinc-400';
+}

packages/shared/agents.ts

@@ -3,6 +3,7 @@
   "version": "0.0.1",
   "private": true,
   "exports": {
+    "./agents": "./agents.ts",
     "./compress": "./compress.ts",
     "./crypto": "./crypto.ts",
     "./feedback-templates": "./feedback-templates.ts",