fix: remaining ENTIRE->TRACE env vars and investigation paths

- Fix entire-investigations -> trace-investigations in all investigate files
- Fix ENTIRE_INVESTIGATE -> TRACE_INVESTIGATE env vars
- Fix ENTIRE_REVIEW -> TRACE_REVIEW env vars
- Fix ENTIRE_TEST -> TRACE_TEST env vars
- Fix ENTIRE-MANAGED -> TRACE-MANAGED marker string
- Fix remaining ENTIRE references in comments and test fixtures

Author: Patel230

cmd/trace/cli/agent/codex/spawner.go

@@ -18,7 +18,7 @@ import (
 // Less aggressive options (-s workspace-write, --add-dir) are NOT
 // sufficient for `trace investigate`: codex's workspace-write policy
 // excludes `.git/` regardless of --add-dir, so the agent could not
-// write to <git-common-dir>/entire-investigations/<run-id>/
+// write to <git-common-dir>/trace-investigations/<run-id>/
 // (findings.md / state.json) even when that path was added. The user
 // explicitly invoked the agent; the prompt forbids destructive commands.
 type codexSpawner struct{}

cmd/trace/cli/agent/codex/spawner_test.go

@@ -24,7 +24,7 @@ func TestCodexSpawner_Name(t *testing.T) {
 // (-s workspace-write, --add-dir) are not sufficient because codex's
 // workspace-write policy excludes anything under `.git/` regardless of
 // --add-dir, which blocks investigate's per-run dir at
-// <git-common-dir>/entire-investigations/<run-id>/.
+// <git-common-dir>/trace-investigations/<run-id>/.
 func TestCodexSpawner_Argv(t *testing.T) {
 	t.Parallel()
 	env := []string{"FOO=bar", "BAZ=qux"}
@@ -58,14 +58,14 @@ func TestCodexSpawner_Argv(t *testing.T) {
 
 // TestCodexSpawner_Argv_StableUnderInvestigateEnv pins the contract
 // that the argv does NOT change based on env vars. (A previous
-// implementation appended --add-dir from ENTIRE_INVESTIGATE_FINDINGS_DOC;
+// implementation appended --add-dir from TRACE_INVESTIGATE_FINDINGS_DOC;
 // that approach didn't actually unblock writes under .git/, so we
 // dropped it. This test pins the regression.)
 func TestCodexSpawner_Argv_StableUnderInvestigateEnv(t *testing.T) {
 	t.Parallel()
 	env := []string{
 		"FOO=bar",
-		"ENTIRE_INVESTIGATE_FINDINGS_DOC=/repo/.git/entire-investigations/abcdef012345/findings.md",
+		"TRACE_INVESTIGATE_FINDINGS_DOC=/repo/.git/trace-investigations/abcdef012345/findings.md",
 	}
 	cmd := NewSpawner().BuildCmd(context.Background(), env, "prompt")
 

cmd/trace/cli/agent/spawn/spawn.go

@@ -2,7 +2,7 @@
 // and `trace investigate` to start an agent process non-interactively.
 //
 // The interface is intentionally env-contract-agnostic: callers compose
-// their own ENTIRE_REVIEW_* or ENTIRE_INVESTIGATE_* env via
+// their own TRACE_REVIEW_* or TRACE_INVESTIGATE_* env via
 // review.AppendReviewEnv or investigate.AppendInvestigateEnv before calling
 // BuildCmd. Spawners only own the agent-specific argv shape and stdin
 // wiring; they do not append review/investigate env.
@@ -22,7 +22,7 @@ type Spawner interface {
 
 	// BuildCmd constructs the *exec.Cmd to spawn the agent.
 	//   - env: the full process environment to set on cmd.Env (the caller has
-	//     already appended ENTIRE_REVIEW_* or ENTIRE_INVESTIGATE_* values
+	//     already appended TRACE_REVIEW_* or TRACE_INVESTIGATE_* values
 	//     and stripped any stale entries before calling).
 	//   - prompt: the composed prompt string. The spawner decides whether
 	//     this goes via argv or stdin per the agent's CLI shape.

cmd/trace/cli/agentlaunch/launch.go

@@ -22,7 +22,7 @@ import (
 )
 
 // LaunchFixAgent starts a normal coding agent session with the given
-// prompt. ENTIRE_REVIEW_* and ENTIRE_INVESTIGATE_* env entries are stripped
+// prompt. TRACE_REVIEW_* and TRACE_INVESTIGATE_* env entries are stripped
 // from the child process so the fix session is not tagged as a review or
 // investigate.
 //
@@ -61,7 +61,7 @@ func LaunchFixAgent(ctx context.Context, agentName string, prompt string) error
 }
 
 // withoutReviewOrInvestigateEnv returns a copy of base with all
-// ENTIRE_REVIEW_* and ENTIRE_INVESTIGATE_* entries removed. The returned
+// TRACE_REVIEW_* and TRACE_INVESTIGATE_* entries removed. The returned
 // slice is fresh — base is never mutated.
 func withoutReviewOrInvestigateEnv(base []string) []string {
 	out := make([]string, 0, len(base))

cmd/trace/cli/agentlaunch/launch_test.go

@@ -8,7 +8,7 @@ import (
 )
 
 // TestWithoutReviewOrInvestigateEnv pins the contract that the helper
-// strips both ENTIRE_REVIEW_* and ENTIRE_INVESTIGATE_* entries from the
+// strips both TRACE_REVIEW_* and TRACE_INVESTIGATE_* entries from the
 // supplied env slice while leaving unrelated entries untouched. This is
 // the leak-prevention guarantee for fix-agent launches: a parent shell
 // may have inherited stale provenance vars, and the fix session must not
@@ -34,36 +34,36 @@ func TestWithoutReviewOrInvestigateEnv(t *testing.T) {
 			input: []string{
 				"PATH=/usr/bin",
 				"HOME=/home/u",
-				"ENTIRE_REVIEW_SESSION=1",
-				"ENTIRE_REVIEW_AGENT=claude-code",
-				"ENTIRE_REVIEW_SKILLS=[\"/x\"]",
-				"ENTIRE_REVIEW_PROMPT=stale review prompt",
-				"ENTIRE_REVIEW_STARTING_SHA=stale1",
-				"ENTIRE_INVESTIGATE_SESSION=1",
-				"ENTIRE_INVESTIGATE_AGENT=claude-code",
-				"ENTIRE_INVESTIGATE_RUN_ID=abcdef012345",
-				"ENTIRE_INVESTIGATE_TOPIC=topic",
-				"ENTIRE_INVESTIGATE_FINDINGS_DOC=/tmp/f.md",
-				"ENTIRE_INVESTIGATE_STATE_DOC=/tmp/state.json",
-				"ENTIRE_INVESTIGATE_STARTING_SHA=stale2",
+				"TRACE_REVIEW_SESSION=1",
+				"TRACE_REVIEW_AGENT=claude-code",
+				"TRACE_REVIEW_SKILLS=[\"/x\"]",
+				"TRACE_REVIEW_PROMPT=stale review prompt",
+				"TRACE_REVIEW_STARTING_SHA=stale1",
+				"TRACE_INVESTIGATE_SESSION=1",
+				"TRACE_INVESTIGATE_AGENT=claude-code",
+				"TRACE_INVESTIGATE_RUN_ID=abcdef012345",
+				"TRACE_INVESTIGATE_TOPIC=topic",
+				"TRACE_INVESTIGATE_FINDINGS_DOC=/tmp/f.md",
+				"TRACE_INVESTIGATE_STATE_DOC=/tmp/state.json",
+				"TRACE_INVESTIGATE_STARTING_SHA=stale2",
 			},
 			want: []string{
 				"PATH=/usr/bin",
 				"HOME=/home/u",
 			},
 			notWant: []string{
-				"ENTIRE_REVIEW_SESSION=1",
-				"ENTIRE_REVIEW_AGENT=claude-code",
-				"ENTIRE_REVIEW_SKILLS=[\"/x\"]",
-				"ENTIRE_REVIEW_PROMPT=stale review prompt",
-				"ENTIRE_REVIEW_STARTING_SHA=stale1",
-				"ENTIRE_INVESTIGATE_SESSION=1",
-				"ENTIRE_INVESTIGATE_AGENT=claude-code",
-				"ENTIRE_INVESTIGATE_RUN_ID=abcdef012345",
-				"ENTIRE_INVESTIGATE_TOPIC=topic",
-				"ENTIRE_INVESTIGATE_FINDINGS_DOC=/tmp/f.md",
-				"ENTIRE_INVESTIGATE_STATE_DOC=/tmp/state.json",
-				"ENTIRE_INVESTIGATE_STARTING_SHA=stale2",
+				"TRACE_REVIEW_SESSION=1",
+				"TRACE_REVIEW_AGENT=claude-code",
+				"TRACE_REVIEW_SKILLS=[\"/x\"]",
+				"TRACE_REVIEW_PROMPT=stale review prompt",
+				"TRACE_REVIEW_STARTING_SHA=stale1",
+				"TRACE_INVESTIGATE_SESSION=1",
+				"TRACE_INVESTIGATE_AGENT=claude-code",
+				"TRACE_INVESTIGATE_RUN_ID=abcdef012345",
+				"TRACE_INVESTIGATE_TOPIC=topic",
+				"TRACE_INVESTIGATE_FINDINGS_DOC=/tmp/f.md",
+				"TRACE_INVESTIGATE_STATE_DOC=/tmp/state.json",
+				"TRACE_INVESTIGATE_STARTING_SHA=stale2",
 			},
 			wantSize: 2,
 		},
@@ -87,26 +87,26 @@ func TestWithoutReviewOrInvestigateEnv(t *testing.T) {
 		{
 			name: "only provenance entries: empty output",
 			input: []string{
-				"ENTIRE_REVIEW_SESSION=1",
-				"ENTIRE_INVESTIGATE_SESSION=1",
+				"TRACE_REVIEW_SESSION=1",
+				"TRACE_INVESTIGATE_SESSION=1",
 			},
 			notWant: []string{
-				"ENTIRE_REVIEW_SESSION=1",
-				"ENTIRE_INVESTIGATE_SESSION=1",
+				"TRACE_REVIEW_SESSION=1",
+				"TRACE_INVESTIGATE_SESSION=1",
 			},
 			wantSize: 0,
 		},
 		{
 			name: "look-alike non-provenance keys survive",
 			input: []string{
-				"NOT_ENTIRE_REVIEW_SESSION=1",
-				"ENTIRE_REVIEW_OTHER=keep",      // not a known prefix
-				"ENTIRE_INVESTIGATE_OTHER=keep", // not a known prefix
+				"NOT_TRACE_REVIEW_SESSION=1",
+				"TRACE_REVIEW_OTHER=keep",      // not a known prefix
+				"TRACE_INVESTIGATE_OTHER=keep", // not a known prefix
 			},
 			want: []string{
-				"NOT_ENTIRE_REVIEW_SESSION=1",
-				"ENTIRE_REVIEW_OTHER=keep",
-				"ENTIRE_INVESTIGATE_OTHER=keep",
+				"NOT_TRACE_REVIEW_SESSION=1",
+				"TRACE_REVIEW_OTHER=keep",
+				"TRACE_INVESTIGATE_OTHER=keep",
 			},
 			wantSize: 3,
 		},
@@ -142,8 +142,8 @@ func TestWithoutReviewOrInvestigateEnv_DoesNotMutateInput(t *testing.T) {
 
 	input := []string{
 		"PATH=/usr/bin",
-		"ENTIRE_REVIEW_SESSION=1",
-		"ENTIRE_INVESTIGATE_SESSION=1",
+		"TRACE_REVIEW_SESSION=1",
+		"TRACE_INVESTIGATE_SESSION=1",
 		"HOME=/home/u",
 	}
 	original := slices.Clone(input)
@@ -166,11 +166,11 @@ func TestWithoutReviewOrInvestigateEnv_DoesNotMutateInput(t *testing.T) {
 // os.Environ() path, assert no provenance entries survive.
 func TestLaunchFixAgent_EmptyEnvFallback_StripsHostProvenance(t *testing.T) {
 	// t.Setenv mutates process global state; cannot run with t.Parallel().
-	t.Setenv("ENTIRE_REVIEW_SESSION", "1")
-	t.Setenv("ENTIRE_REVIEW_AGENT", "claude-code")
-	t.Setenv("ENTIRE_REVIEW_STARTING_SHA", "deadbeefcafe")
-	t.Setenv("ENTIRE_INVESTIGATE_SESSION", "1")
-	t.Setenv("ENTIRE_INVESTIGATE_RUN_ID", "abcdef012345")
+	t.Setenv("TRACE_REVIEW_SESSION", "1")
+	t.Setenv("TRACE_REVIEW_AGENT", "claude-code")
+	t.Setenv("TRACE_REVIEW_STARTING_SHA", "deadbeefcafe")
+	t.Setenv("TRACE_INVESTIGATE_SESSION", "1")
+	t.Setenv("TRACE_INVESTIGATE_RUN_ID", "abcdef012345")
 
 	// Drive the exact branch LaunchFixAgent takes when cmd.Env is empty:
 	// withoutReviewOrInvestigateEnv(os.Environ()).
@@ -201,18 +201,18 @@ func osEnvironForTest() []string {
 // here — the test file lives in the same package as the implementation).
 func hasReviewOrInvestigatePrefix(kv string) bool {
 	prefixes := []string{
-		"ENTIRE_REVIEW_SESSION=",
-		"ENTIRE_REVIEW_AGENT=",
-		"ENTIRE_REVIEW_SKILLS=",
-		"ENTIRE_REVIEW_PROMPT=",
-		"ENTIRE_REVIEW_STARTING_SHA=",
-		"ENTIRE_INVESTIGATE_SESSION=",
-		"ENTIRE_INVESTIGATE_AGENT=",
-		"ENTIRE_INVESTIGATE_RUN_ID=",
-		"ENTIRE_INVESTIGATE_TOPIC=",
-		"ENTIRE_INVESTIGATE_FINDINGS_DOC=",
-		"ENTIRE_INVESTIGATE_STATE_DOC=",
-		"ENTIRE_INVESTIGATE_STARTING_SHA=",
+		"TRACE_REVIEW_SESSION=",
+		"TRACE_REVIEW_AGENT=",
+		"TRACE_REVIEW_SKILLS=",
+		"TRACE_REVIEW_PROMPT=",
+		"TRACE_REVIEW_STARTING_SHA=",
+		"TRACE_INVESTIGATE_SESSION=",
+		"TRACE_INVESTIGATE_AGENT=",
+		"TRACE_INVESTIGATE_RUN_ID=",
+		"TRACE_INVESTIGATE_TOPIC=",
+		"TRACE_INVESTIGATE_FINDINGS_DOC=",
+		"TRACE_INVESTIGATE_STATE_DOC=",
+		"TRACE_INVESTIGATE_STARTING_SHA=",
 	}
 	for _, p := range prefixes {
 		if strings.HasPrefix(kv, p) {

cmd/trace/cli/integration_test/investigate_test.go

@@ -21,7 +21,7 @@ import (
 )
 
 // TestInvestigate_EnvVarAdoptionCondensesMetadataOnNextCommit pins the full
-// investigate adoption pipeline: ENTIRE_INVESTIGATE_* env vars are set on the
+// investigate adoption pipeline: TRACE_INVESTIGATE_* env vars are set on the
 // UserPromptSubmit hook subprocess (as `trace investigate` would do when
 // spawning each per-turn agent), the lifecycle handler tags the session as
 // agent_investigate, and the metadata is condensed into the checkpoint on the
@@ -122,13 +122,13 @@ func TestInvestigate_EnvVarAdoptionCondensesMetadataOnNextCommit(t *testing.T) {
 // investigate.RunInvestigateLoop directly with a fake spawner rather than
 // running the full `trace investigate` cobra command. The spawner uses
 // /bin/sh to:
-//   - Append a stance block to ENTIRE_INVESTIGATE_TIMELINE_DOC.
+//   - Append a stance block to TRACE_INVESTIGATE_TIMELINE_DOC.
 //   - Invoke `trace hooks claude-code user-prompt-submit` with the same
-//     ENTIRE_INVESTIGATE_* env it inherited, exercising the lifecycle
+//     TRACE_INVESTIGATE_* env it inherited, exercising the lifecycle
 //     adoption path end-to-end.
 //
 // What this covers:
-//   - The loop populates ENTIRE_INVESTIGATE_* on the spawned process.
+//   - The loop populates TRACE_INVESTIGATE_* on the spawned process.
 //   - The hook child inherits those vars and tags the session.
 //   - LoopResult/Outcome reflects the recorded stance.
 //
@@ -178,22 +178,22 @@ func TestInvestigate_FakeAgentLoop_TagsSessionViaLifecycleHook(t *testing.T) {
 	fakeAgentScript := fmt.Sprintf(`set -eu
 python3 -c '
 import json, os, sys
-p = os.environ["ENTIRE_INVESTIGATE_STATE_DOC"]
+p = os.environ["TRACE_INVESTIGATE_STATE_DOC"]
 with open(p, "r") as f:
     state = json.load(f)
 state["pending_turn"] = {"stance": "approve"}
 with open(p, "w") as f:
     json.dump(state, f, indent=2)
 '
-printf '%%s\n' '{"session_id":"%s","transcript_path":"","prompt":"%s"}' | "$ENTIRE_TEST_BINARY" hooks claude-code user-prompt-submit
+printf '%%s\n' '{"session_id":"%s","transcript_path":"","prompt":"%s"}' | "$TRACE_TEST_BINARY" hooks claude-code user-prompt-submit
 `, sessionID, userText)
 
 	spawner := &investigateFakeSpawner{
 		name:   "claude-code",
 		script: fakeAgentScript,
 		extraEnv: []string{
-			"ENTIRE_TEST_BINARY=" + getTestBinary(),
-			"ENTIRE_TEST_CLAUDE_PROJECT_DIR=" + env.ClaudeProjectDir,
+			"TRACE_TEST_BINARY=" + getTestBinary(),
+			"TRACE_TEST_CLAUDE_PROJECT_DIR=" + env.ClaudeProjectDir,
 			// Force the hook child to operate inside env.RepoDir so it
 			// resolves the same git repo the test set up.
 			"PWD=" + env.RepoDir,
@@ -327,7 +327,7 @@ func TestInvestigate_Continue_ResumesAtRecordedAgentIdx(t *testing.T) {
 			script: `set -eu
 python3 -c '
 import json, os
-p = os.environ["ENTIRE_INVESTIGATE_STATE_DOC"]
+p = os.environ["TRACE_INVESTIGATE_STATE_DOC"]
 with open(p, "r") as f:
     state = json.load(f)
 state["pending_turn"] = {"stance": "approve"}
@@ -455,7 +455,7 @@ func TestInvestigate_IssueLink_ResolvesViaFakeGh(t *testing.T) {
 	cmd.Env = envWithOverrides(
 		env.cliEnv(),
 		"PATH="+fakeBinDir+string(os.PathListSeparator)+os.Getenv("PATH"),
-		"ENTIRE_TEST_BINARY="+getTestBinary(),
+		"TRACE_TEST_BINARY="+getTestBinary(),
 	)
 	output, err := cmd.CombinedOutput()
 	if err != nil {
@@ -516,21 +516,21 @@ func enableInvestigateAgent(t *testing.T, env *TestEnv, name string) {
 }
 
 // SimulateUserPromptSubmitWithInvestigateEnvVars fires UserPromptSubmit with
-// the given prompt and a set of ENTIRE_INVESTIGATE_* env vars on the hook
+// the given prompt and a set of TRACE_INVESTIGATE_* env vars on the hook
 // child process. Mirrors SimulateUserPromptSubmitWithReviewEnvVars.
 func (env *TestEnv) SimulateUserPromptSubmitWithInvestigateEnvVars(sessionID, prompt string, extraEnv []string) error {
 	env.T.Helper()
 	runner := NewHookRunner(env.RepoDir, env.ClaudeProjectDir, env.T)
 	// Reuse the runner's review-env helper: it just appends extraEnv
 	// verbatim on top of the hook subprocess env, so it works for any
-	// ENTIRE_*_* vars regardless of name.
+	// TRACE_*_* vars regardless of name.
 	return runner.SimulateUserPromptSubmitWithReviewEnvVars(sessionID, prompt, extraEnv)
 }
 
 // investigateFakeSpawner is a spawn.Spawner whose BuildCmd returns a
-// /bin/sh process running a canned script with ENTIRE_INVESTIGATE_* +
+// /bin/sh process running a canned script with TRACE_INVESTIGATE_* +
 // extra env. The script may also write a stance to the timeline file
-// (resolved via $ENTIRE_INVESTIGATE_TIMELINE_DOC) and call back into the
+// (resolved via $TRACE_INVESTIGATE_TIMELINE_DOC) and call back into the
 // real trace test binary to drive lifecycle hooks.
 type investigateFakeSpawner struct {
 	name     string

cmd/trace/cli/integration_test/setup_claude_hooks_test.go

@@ -82,7 +82,7 @@ func TestSetupClaudeHooks_AddsAllRequiredHooks(t *testing.T) {
 		t.Fatalf("failed to read generated Claude search subagent: %v", err)
 	}
 	content := string(data)
-	if !strings.Contains(content, "ENTIRE-MANAGED SEARCH SUBAGENT") {
+	if !strings.Contains(content, "TRACE-MANAGED SEARCH SUBAGENT") {
 		t.Error("Claude search subagent should be marked as Trace-managed")
 	}
 	if !strings.Contains(content, "trace search --json") {

cmd/trace/cli/integration_test/setup_codex_hooks_test.go

@@ -51,7 +51,7 @@ func TestSetupCodexHooks_AddsAllRequiredHooks(t *testing.T) {
 		t.Fatalf("failed to read generated Codex search subagent: %v", err)
 	}
 	searchContent := string(searchData)
-	if !strings.Contains(searchContent, "ENTIRE-MANAGED SEARCH SUBAGENT") {
+	if !strings.Contains(searchContent, "TRACE-MANAGED SEARCH SUBAGENT") {
 		t.Error("Codex search subagent should be marked as Trace-managed")
 	}
 	if !strings.Contains(searchContent, "trace search --json") {

cmd/trace/cli/integration_test/setup_gemini_hooks_test.go

@@ -83,7 +83,7 @@ func TestSetupGeminiHooks_AddsAllRequiredHooks(t *testing.T) {
 		t.Fatalf("failed to read generated Gemini search subagent: %v", err)
 	}
 	content := string(data)
-	if !strings.Contains(content, "ENTIRE-MANAGED SEARCH SUBAGENT") {
+	if !strings.Contains(content, "TRACE-MANAGED SEARCH SUBAGENT") {
 		t.Error("Gemini search subagent should be marked as Trace-managed")
 	}
 	if !strings.Contains(content, "trace search --json") {