# Complete retained Session review in Studio

## Traceability

- Spec ID: studio-inspector-session-parity
- Status: Implemented

## Intent

Finish the React-owned Session detail in Harness Studio so `#/sessions` preserves
the read-only review workflow already implemented by the standalone Harness
Inspector workbench. Studio continues to consume `HarnessInspectorReportV1` and
does not execute `workbench.js`; the standalone report remains the interaction
and visual contract.

## Acceptance Scenarios

- **AC-1:** Opening a retained Session renders the same notebook hierarchy as
  Inspector: numbered `In` and `Out` rows, an initially collapsed ordered
  Process trace, an evidence-bounded Outcome, unplaced evidence, commits outside
  observed Turn windows, and one overall Session activity summary.
- **AC-2:** The Session outline provides Turn/evidence navigation, expand and
  collapse controls, kind/tool/file filters, and observed Session facts. Tool
  steps resolve through the retained call ledger, adjacent calls keep observed
  order, and filters update the visible retained-call count.
- **AC-3:** Replay provides Events and Files indexes, previous/next and play/pause
  controls, 1x/2x/4x/8x speed selection, a sequence or observed-time rail, and
  keyboard J/L/Space navigation without executing or resuming the Session.
- **AC-4:** Opening Session detail creates a navigable Studio state. Close,
  Escape, and browser Back return to the workbench and restore focus to the
  initiating control; a copied Studio URL reopens the selected Session and
  Replay event when the report still contains them.
- **AC-5:** Wide, compact, and narrow layouts have no document-level horizontal
  overflow. At narrow width the outline remains reachable, controls retain
  visible keyboard focus, and the primary notebook decision path remains clear.
- **AC-6:** Focused model/component and browser tests exercise the behaviors
  above against retained ordered messages, tool calls, files, and Replay events.
  Browser verification checks console/page errors and screenshots.

## Non-goals

- Changing Inspector discovery, normalization, privacy filtering, correlation,
  or `HarnessInspectorReportV1` ownership.
- Executing Replay, resuming a native Session, mutating checkpoints, or writing
  back to host evidence stores.
- Removing the compact Catalog & Compare view or the standalone Inspector report.
- Claiming a patch, successful edit, verification result, or causal commit link
  when the retained report does not provide that evidence.

## Plan and Tasks

1. Add a pure retained-Session projection seam for ordered Turn calls, commit
   placement, unplaced evidence, filter counts, and Replay navigation.
2. Replace the simplified React Session detail with Inspector-compatible Trace,
   Outcome, outline, filtering, activity, and Replay components.
3. Synchronize Session/Replay selection with Studio navigation and restore focus
   on close without taking ownership from the outer Studio router.
4. Extend focused tests and the Sessions Playwright walkthrough, then run package,
   browser, link-graph, preview, and diff checks proportional to the change.

## Test and Review Evidence

- AC-1/AC-2/AC-3: focused model tests and Sessions browser assertions.
- AC-4: browser assertions for open, URL state, Back, Escape, and focus return.
- AC-5: 1440x900, 1024x768, and 390x844 screenshots plus overflow and focus
  checks.
- AC-6: console/page error collection, package tests, documentation link graph,
  preview health and Canvas module smoke, and `git diff --check`.
- Risk: Studio can accidentally overstate retained evidence. Copy and projections
  must preserve missing timestamps, unavailable responses, privacy redaction,
  and the read-only boundary used by the standalone workbench.

Implementation evidence (2026-08-24):

- `npm test -- --reporter=dot`: 103 files passed with 1,532 tests passed and one
  skipped repository-wide.
- `npm test -w @qoder-ai/harness-studio`: 43 files and 266 tests passed.
- `npm run test:browser -w @qoder-ai/harness-studio`: all 40 Playwright tests
  passed. The Sessions flow exercised ordered Process expansion, Outcome,
  filters, Replay, URL state, Escape/focus restoration, and 1440x900, 1024x768,
  and 390x844 screenshots without document overflow.
- The final focused Sessions Playwright run passed after Replay auto-follow,
  restart, legend, and tab-keyboard behavior were added.
- `npx vitest run test/skills-docs/doc-link-graph.test.mjs`: 8 tests passed after
  regenerating `docs/better-harness-doc-links.mmd`.
- Live in-app browser review used a 36-call retained Session, exercised Process
  expansion, tool filtering, Replay event navigation and URL state, and found no
  console warnings or errors. The repository Preview returned `ok` from
  `/health`, loaded `/canvas-module.js`, and `git diff --check` passed.