feat: add notes field to DesignPlan and field_agents config

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

agents/design_state.py

@@ -50,6 +50,7 @@ class DesignPlan(BaseModel):
     axis_recommendation: str = ""
     machining_notes: list[str] = Field(default_factory=list)
     confidence_score: float = 0.0
+    notes: str = ""
 
     @classmethod
     def from_state(cls, state: "DesignState", confidence_score: float) -> "DesignPlan":
@@ -86,6 +87,8 @@ class DesignPlan(BaseModel):
             lines.append(f"Axis: {self.axis_recommendation}")
         if self.machining_notes:
             lines.append(f"Machining Notes: {'; '.join(self.machining_notes)}")
+        if self.notes:
+            lines.append(f"User Notes: {self.notes}")
         lines.append("")
         lines.append("This plan has been reviewed and approved by the user.")
         lines.append("Generate the model according to these specifications.")

config.yaml

@@ -95,6 +95,14 @@ planning:
   approved_agents:
     - "cad"
     - "cnc"
+  field_agents:
+    material: engineering
+    dimensions: engineering
+    features: design
+    constraints: cnc
+    axis_recommendation: cnc
+    machining_notes: cnc
+    part_name: design
 
 gap_analysis:
   category_keywords:

config/settings.py

@@ -109,6 +109,15 @@ class PlanningConfig(BaseModel):
         "plan", "review", "ready", "show plan", "summarize", "what do we have",
     ])
     approved_agents: list[str] = Field(default_factory=lambda: ["cad", "cnc"])
+    field_agents: dict[str, str] = Field(default_factory=lambda: {
+        "material": "engineering",
+        "dimensions": "engineering",
+        "features": "design",
+        "constraints": "cnc",
+        "axis_recommendation": "cnc",
+        "machining_notes": "cnc",
+        "part_name": "design",
+    })
 
 
 class GapAnalysisConfig(BaseModel):

tests/test_design_state.py

@@ -126,6 +126,30 @@ class TestDesignPlan:
         assert "aluminum 6061" in rendered
         assert "No undercuts" in rendered
 
+    def test_plan_notes_default_empty(self):
+        plan = DesignPlan(part_name="test")
+        assert plan.notes == ""
+
+    def test_plan_notes_in_render(self):
+        plan = DesignPlan(
+            part_name="bracket",
+            material="aluminum",
+            notes="Check if 304 is overkill",
+        )
+        rendered = plan.render_approved()
+        assert "User Notes" in rendered
+        assert "Check if 304 is overkill" in rendered
+
+    def test_plan_notes_empty_not_in_render(self):
+        plan = DesignPlan(part_name="bracket", material="aluminum", notes="")
+        rendered = plan.render_approved()
+        assert "User Notes" not in rendered
+
+    def test_plan_from_state_no_notes(self):
+        state = DesignState(part_name="bracket", material="steel")
+        plan = DesignPlan.from_state(state, confidence_score=5.0)
+        assert plan.notes == ""
+
 
 class TestComputeScore:
     def test_empty_state_scores_zero(self):

tests/test_settings.py

@@ -65,6 +65,16 @@ class TestPlanningConfig:
         assert settings.planning.threshold == 8.0
 
 
+def test_planning_field_agents_defaults():
+    from config.settings import PlanningConfig
+    cfg = PlanningConfig()
+    assert cfg.field_agents["material"] == "engineering"
+    assert cfg.field_agents["features"] == "design"
+    assert cfg.field_agents["constraints"] == "cnc"
+    assert cfg.field_agents["axis_recommendation"] == "cnc"
+    assert cfg.field_agents["part_name"] == "design"
+
+
 class TestGapAnalysisConfig:
     def test_gap_analysis_defaults(self):
         from config.settings import Settings