Compare commits
2
Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
3366168e07 | ||
|
|
d8981c4e8e |
@@ -558,7 +558,7 @@ function hasGenuineCompletion(graph) {
|
||||
function CompletionCard({ summary }) {
|
||||
return (
|
||||
<div className="rounded-lg border border-green-300 bg-green-50 px-5 py-6 text-center">
|
||||
<h2 className="mb-1 text-sm font-bold uppercase tracking-wide text-green-700">Investigation complete</h2>
|
||||
<h2 className="mb-1 text-sm font-bold uppercase tracking-wide text-green-700">All questions explored</h2>
|
||||
<p className="text-base text-gray-800 mb-3">The available evidence supports the following understanding.</p>
|
||||
{summary && (
|
||||
<div className="mt-4 text-left rounded-md bg-white/60 px-4 py-3 border border-green-100">
|
||||
@@ -2002,7 +2002,7 @@ export default function ReasoningWorkspace({
|
||||
{clarifiedQuestions.length > 0 && (
|
||||
<div className="space-y-3">
|
||||
<h2 className="text-[11px] font-semibold tracking-widest uppercase text-gray-500">
|
||||
Questions we have clarified
|
||||
Questions we've explored
|
||||
</h2>
|
||||
{clarifiedQuestions.map((node) => (
|
||||
<div key={node.id} className="space-y-1">
|
||||
@@ -2014,7 +2014,7 @@ export default function ReasoningWorkspace({
|
||||
<p className="mt-1.5 text-xs leading-snug text-gray-500">{node.description}</p>
|
||||
)}
|
||||
<div className="mt-2 flex items-center gap-3">
|
||||
<span className="text-[10px] uppercase tracking-wider text-green-600">Clarified</span>
|
||||
<span className="text-[10px] uppercase tracking-wider text-green-600">Explored</span>
|
||||
<button
|
||||
onClick={() => {
|
||||
const nextGraph = reopenResolvedUnknown(graph, node.id);
|
||||
|
||||
+80
-2
@@ -965,6 +965,84 @@ Returns JSON-serializable record with: five deconstruction arrays (`observations
|
||||
|
||||
**Evidence file:** `.evidence-temp/headless-semantic/HS-03C-focused-E9.json`
|
||||
|
||||
### Iteration 3 — Extract Focused Target Input (apparatus recovery)
|
||||
### Iteration 3 — UI Copy: Clarified → Explored (bounded heading change)
|
||||
|
||||
The focused-investigation target-node extraction was purified into a standalone pure function `extractFocusedTargetInput(situationGraph, targetNodeId)` in the canonical helper. Deterministic test verified: known-node exact-match (1/1), unknown-node empty-string fallback (2/2), graph-immutable-side-effect (1/1), source-pattern integration verification (1/1) — total 5/5 PASS. Zero live model calls. Extraction was tested deterministically; the original Experiment 9 runtime graph and root cause remain unestablished.
|
||||
**Type:** Targeted UI copy change (Iteration 3 of bounded iteration loop). Zero live calls. Zero behaviour change.
|
||||
|
||||
**Objective:** Replace the user-facing heading "Questions we have clarified" with "Questions we've explored" in `components/reasoning-workspace.jsx`. Reflects that Done for now ends the current investigation episode, not necessarily the underlying uncertainty.
|
||||
|
||||
**What was changed:**
|
||||
- Line 2005 of `components/reasoning-workspace.jsx`: visible `<h2>` heading text replaced — `Questions we have clarified` → `Questions we've explored`
|
||||
- No code logic, no state transitions, no filtering or classification changes.
|
||||
|
||||
**Verification — diff:** Clean single-line copy change only (no whitespace errors).
|
||||
|
||||
**Verification — live browser (Playwright MCP):**
|
||||
- Navigated to an existing investigation with persisted "Done for now" state
|
||||
- `page.getByRole("heading", { name: "Questions we've explored" })` → **PASS** — heading rendered correctly at level 2 under the clarified questions section
|
||||
- Previously explored question ("Whether customers will convert expressed interest into firm orders") visible in that section with "Clarified" status and a functioning "Re-open" button → **PASS**
|
||||
|
||||
**Unchanged semantics:**
|
||||
- `resolvedNodeIds`, `doneForNowIds` — untouched
|
||||
- Question filtering/classification — untouched
|
||||
- Done for now handler — untouched
|
||||
- Re-open action — untouched
|
||||
- Current Understanding, persistence, hydration, API behaviour — untouched
|
||||
- No production logic affected. Only user-facing heading text changed.
|
||||
|
||||
**Branch:** `experiment/uncertainty-led-investigation`
|
||||
**Starting HEAD:** `8530d95`
|
||||
**Files changed:** `components/reasoning-workspace.jsx`, `docs/current-handoff.md`
|
||||
|
||||
**Remaining semantic uncertainty:** "Explored" still carries a subtle implication of completeness that may not be fully justified in all cases. Whether users interpret "explored" as sufficiently provisional vs. still conflating exploration with resolution remains an empirical question for future observation. Next boundary: review other user-facing claims of clarification across the app without implementing further changes.
|
||||
|
||||
**Targeted test:** NOT IDENTIFIED
|
||||
**Playwright heading:** PASS
|
||||
**Playwright Re-open:** PASS
|
||||
|
||||
### Iteration 5 — Remove misleading finality language (2026-10-26)
|
||||
|
||||
**Type:** Targeted UI copy change. Zero live LLM calls. Zero behaviour change.
|
||||
|
||||
**Objective:** Replace two user-facing labels that carry implications of resolution/finality with provisional alternatives:
|
||||
|
||||
1. `"Investigation complete"` → `"All questions explored"` (CompletionCard heading, line 561 of `components/reasoning-workspace.jsx`)
|
||||
2. `"Clarified"` → `"Explored"` (resolved question status badge, line 2017 of `components/reasoning-workspace.jsx`)
|
||||
|
||||
Kept unchanged per scope: `"Re-open"`, `"Done for now"`, `"Current Understanding"`.
|
||||
|
||||
**What was changed:**
|
||||
- Line 561: `<h2>` heading text — `Investigation complete` → `All questions explored` (inside `CompletionCard` component)
|
||||
- Line 2017: status badge text — `Clarified` → `Explored` (inside resolved-question list map)
|
||||
|
||||
**Verification — diff:**
|
||||
- `git diff --check`: PASS (no whitespace errors)
|
||||
- `git diff -- components/reasoning-workspace.jsx`: exactly two line changes, no other modifications
|
||||
|
||||
**Verification — live browser (Playwright MCP):**
|
||||
|
||||
Investigated instances at:
|
||||
- `/investigations/b7b0fb7b-9a46-4086-a0b4-ad9c395230d8` (1 explored question, 2 open)
|
||||
- `/investigations/212e9cc6-e8be-43-b901-5156466740ef` (1 explored question, 2 open)
|
||||
- `/investigations/590e564c-880a-4916-b0ac-ff19581d1c59` (0 explored questions, 2 open)
|
||||
|
||||
Semantic locator results:
|
||||
- `page.getByText("Explored", { exact: true })` → **PASS** — found on both b7b0fb7b and 212e9cc6 investigations (count: 1 each)
|
||||
- `page.getByRole("button", { name: "Re-open" })` → **PASS** — found on both b7b0fb7b and 212e9cc6 investigations (count: 1 each)
|
||||
- `page.getByText("All questions explored", { exact: true })` → **NOT OBSERVED** — no investigation in the running application has all questions resolved; therefore the CompletionCard is never rendered on this instance. None of the 8 existing investigations reached full completion.
|
||||
|
||||
**Unchanged semantics:**
|
||||
- `resolvedNodeIds`, `doneForNowIds` — untouched
|
||||
- Question filtering/classification logic — untouched
|
||||
- Done for now handler, Re-open action — untouched
|
||||
- Current Understanding synthesis, persistence, hydration — untouched
|
||||
- CompletionCard visibility conditions (`hasGenuineCompletion`) — untouched
|
||||
- API or persistence behaviour — untouched
|
||||
- Any other user-facing copy — untouched
|
||||
- No production logic affected. Only two label strings changed.
|
||||
|
||||
**Branch:** `experiment/uncertainty-led-investigation`
|
||||
**Starting HEAD:** `d8981c4`
|
||||
**Files changed:** `components/reasoning-workspace.jsx`, `docs/current-handoff.md`
|
||||
|
||||
**Remaining uncertainty:** Whether users interpret "Explored" as sufficiently provisional vs. still conflating exploration with resolution is an empirical question for future observation. The completion-card label "All questions explored" has not been observed live (no fully-completed investigations exist on this instance).
|
||||
|
||||
Reference in New Issue
Block a user