feat(taxonomy): rename thinking_correction to plan_correction (#1145)

Author: stateofkate

backend/api/services/blocks/analyzer/trajectory/trajectory_component_block.py

@@ -16,12 +16,14 @@ class ExploreTrajectoryBlockTaxonomy(str, enum.Enum):
     THINKING_RECALL = "thinking_recall"
     THINKING_UNDERSTAND = "thinking_understand"
     THINKING_HYPOTHESIZE = "thinking_hypothesize"
-    THINKING_CORRECTION = "thinking_correction"
 
 
 class ImplementTrajectoryBlockTaxonomy(str, enum.Enum):
-    # Ordered as the work happens: plan, build, correct, test, debug.
+    # Ordered as the work happens: plan, replan, build, correct, test, debug.
+    # PLAN_CORRECTION sits beside WRITING_PLAN so the model reads the two as a
+    # pair; the flat vocabulary below preserves this order in the prompt.
     WRITING_PLAN = "writing_plan"
+    PLAN_CORRECTION = "plan_correction"
     IMPLEMENTING = "implementing"
     IMPLEMENTING_CORRECTION = "implementing_correction"
     WRITING_TESTS = "writing_tests"

frontend/src/app/globals.css

@@ -228,7 +228,7 @@
     --tc-thinking-recall: #0f8fa8;
     --tc-thinking-understand: #1baf7a;
     --tc-thinking-hypothesize: #a184ea;
-    --tc-thinking-correction: #4a3aa7;
+    --tc-plan-correction: #4a3aa7;
     --tc-writing-plan: #008300;
     --tc-implementing: #b8860b;
     --tc-implementing-correction: #c62828;
@@ -296,7 +296,7 @@
     --tc-thinking-recall: #2aa5bd;
     --tc-thinking-understand: #199e70;
     --tc-thinking-hypothesize: #9085e9;
-    --tc-thinking-correction: #7a68d8;
+    --tc-plan-correction: #7a68d8;
     --tc-writing-plan: #14923f;
     --tc-implementing: #b5860e;
     --tc-implementing-correction: #e05252;

frontend/src/lib/trajectory-metrics.ts

@@ -96,8 +96,8 @@ const COMPONENT_COLOR_VARS: Record<string, string> = {
   thinking_recall: "var(--tc-thinking-recall)",
   thinking_understand: "var(--tc-thinking-understand)",
   thinking_hypothesize: "var(--tc-thinking-hypothesize)",
-  thinking_correction: "var(--tc-thinking-correction)",
   writing_plan: "var(--tc-writing-plan)",
+  plan_correction: "var(--tc-plan-correction)",
   implementing: "var(--tc-implementing)",
   implementing_correction: "var(--tc-implementing-correction)",
   writing_tests: "var(--tc-writing-tests)",
@@ -108,7 +108,8 @@ const COMPONENT_COLOR_VARS: Record<string, string> = {
   // Retired kinds, kept so stored summaries keep their fixed color rather than
   // falling through to appearance-order phase slots. Each shares the slot its
   // replacement took over; the two vocabularies never co-occur in one summary.
-  thinking_diagnose: "var(--tc-thinking-correction)",
+  thinking_correction: "var(--tc-plan-correction)",
+  thinking_diagnose: "var(--tc-plan-correction)",
   testing_custom_edge_cases: "var(--tc-testing-edge)",
   // Synthetic bucket for steps no component claims — always neutral gray.
   other: "var(--phase-other)",

frontend/src/lib/trajectory-segments.ts

@@ -10,15 +10,17 @@ const COMPONENT_LABELS: Record<TrajectoryComponentKind, string> = {
   thinking_recall: "Recalling",
   thinking_understand: "Understanding",
   thinking_hypothesize: "Hypothesizing",
-  thinking_correction: "Correcting course",
   writing_plan: "Planning",
+  plan_correction: "Correcting the plan",
   implementing: "Implementing",
   implementing_correction: "Correcting implementation",
   writing_tests: "Writing tests",
   testing_public: "Running public tests",
   testing_custom: "Running custom tests",
   testing_edge_cases: "Testing edge cases",
   debugging: "Debugging",
+  // Retired kinds, kept so stored summaries still render a real label.
+  thinking_correction: "Correcting course",
   thinking_diagnose: "Diagnosing",
   testing_custom_edge_cases: "Testing edge cases",
 };

frontend/src/lib/types.ts

@@ -703,8 +703,8 @@ export type TrajectoryComponentKind =
   | "thinking_recall"
   | "thinking_understand"
   | "thinking_hypothesize"
-  | "thinking_correction"
   | "writing_plan"
+  | "plan_correction"
   | "implementing"
   | "implementing_correction"
   | "writing_tests"
@@ -713,6 +713,7 @@ export type TrajectoryComponentKind =
   | "testing_edge_cases"
   | "debugging"
   // Retired from the backend enum, but stored summaries still carry them.
+  | "thinking_correction"
   | "thinking_diagnose"
   | "testing_custom_edge_cases";