feat: display methodology and analysis_intent in WebUI review sections

Add methodology and analysis_intent to ALLOWED_TARGET_SECTIONS (backend)
and COMMENTABLE_SECTIONS (frontend) so they appear as reviewable inline
comment targets in the design detail view. Closes #45.

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

Author: etoyama

CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+
+- Added methodology and analysis_intent as reviewable sections in WebUI inline comments
+
 ## [0.2.0] - 2026-03-11
 
 ### Added

frontend/e2e/design-detail.spec.ts

@@ -67,6 +67,54 @@ test("#5: overview panel displays design fields and metrics", async ({
   await expect(page.getByText("0.95")).toBeVisible();
 });
 
+// #5b: analysis_intent displays as text
+test("#5b: analysis_intent displays in overview panel", async ({ page }) => {
+  const designWithIntent = makeDesign({
+    id: "d-intent",
+    title: "Intent Design",
+    status: "in_review",
+    analysis_intent: "confirmatory",
+  });
+  await mockDesignList(page, [designWithIntent]);
+  await mockDesignDetail(page, designWithIntent);
+  await mockComments(page, designWithIntent.id, []);
+
+  await page.goto("/?tab=designs");
+  await page.getByText("Intent Design").click();
+
+  await expect(page.getByRole("tab", { name: /overview/i })).toBeVisible({
+    timeout: 5000,
+  });
+
+  await expect(page.getByText("Analysis Intent")).toBeVisible();
+  await expect(page.getByText("confirmatory")).toBeVisible();
+});
+
+// #5c: methodology displays as JsonTree when set
+test("#5c: methodology displays in overview panel when set", async ({
+  page,
+}) => {
+  const designWithMethodology = makeDesign({
+    id: "d-method",
+    title: "Methodology Design",
+    status: "in_review",
+    methodology: { approach: "regression", tool: "statsmodels" },
+  });
+  await mockDesignList(page, [designWithMethodology]);
+  await mockDesignDetail(page, designWithMethodology);
+  await mockComments(page, designWithMethodology.id, []);
+
+  await page.goto("/?tab=designs");
+  await page.getByText("Methodology Design").click();
+
+  await expect(page.getByRole("tab", { name: /overview/i })).toBeVisible({
+    timeout: 5000,
+  });
+
+  await expect(page.getByText("Methodology", { exact: true })).toBeVisible();
+  await expect(page.getByText("regression")).toBeVisible();
+});
+
 // #6: Workflow guide display — verify workflow guide shows for in_review design
 test("#6: workflow guide displays for in_review design", async ({ page }) => {
   await mockDesignList(page, [activeDesign]);
@@ -177,8 +225,8 @@ test.describe("Inline Review Comments", () => {
     await expect(page.getByRole("tab", { name: /overview/i })).toBeVisible({ timeout: 5000 });
     const buttons = page.getByTestId("comment-button");
     await expect(buttons.first()).toBeVisible({ timeout: 5000 });
-    // Should have 7 comment buttons (one per commentable section)
-    await expect(buttons).toHaveCount(7);
+    // Should have 9 comment buttons (one per commentable section)
+    await expect(buttons).toHaveCount(9);
   });
 
   test("comment buttons hidden on non-in_review design", async ({ page }) => {

frontend/e2e/fixtures/mock-data.ts

@@ -16,6 +16,8 @@ export interface Design {
   source_ids: string[];
   next_action: Record<string, unknown> | null;
   referenced_knowledge: Record<string, string[]>;
+  analysis_intent: string;
+  methodology: Record<string, unknown> | null;
   created_at: string;
   updated_at: string;
 }
@@ -35,6 +37,8 @@ export function makeDesign(overrides?: Partial<Design>): Design {
     source_ids: [],
     next_action: null,
     referenced_knowledge: {},
+    analysis_intent: "exploratory",
+    methodology: null,
     created_at: "2026-01-01T00:00:00",
     updated_at: "2026-01-01T00:00:00",
     ...overrides,

frontend/src/pages/design-detail/components/sections.ts

@@ -10,6 +10,8 @@ export interface CommentableSection {
 export const COMMENTABLE_SECTIONS: readonly CommentableSection[] = [
   { id: "hypothesis_statement", label: "Hypothesis Statement", type: "text" },
   { id: "hypothesis_background", label: "Hypothesis Background", type: "text" },
+  { id: "analysis_intent", label: "Analysis Intent", type: "text" },
+  { id: "methodology", label: "Methodology", type: "json" },
   { id: "metrics", label: "Metrics", type: "json" },
   { id: "explanatory", label: "Explanatory", type: "json" },
   { id: "chart", label: "Chart", type: "json" },

src/insight_blueprint/core/reviews.py

@@ -46,6 +46,8 @@
     "chart",
     "next_action",
     "referenced_knowledge",
+    "methodology",
+    "analysis_intent",
 }
 
 VALID_TRANSITIONS: dict[DesignStatus, set[DesignStatus]] = {

tests/test_reviews.py

@@ -906,6 +906,9 @@ class TestSaveReviewBatchTargetSectionValidation:
             "explanatory",
             "chart",
             "next_action",
+            "referenced_knowledge",
+            "methodology",
+            "analysis_intent",
         ],
     )
     def test_valid_target_sections_accepted(
@@ -914,7 +917,7 @@ def test_valid_target_sections_accepted(
         pending_design: AnalysisDesign,
         section: str,
     ) -> None:
-        """NFR-7: All 6 valid sections are accepted."""
+        """NFR-7: All 9 valid sections are accepted."""
         result = review_service.save_review_batch(
             pending_design.id,
             "supported",