Skip to content

Add live Experiment pages to the Console - #1270

Merged
JacobStephens2 merged 2 commits into
issue-977from
issue-1020
Oct 24, 2026
Merged

JacobStephens2 merged 2 commits into
issue-977from
issue-1020

Conversation

@JacobStephens2

@JacobStephens2 JacobStephens2 commented Oct 24, 2026 •

Copy link
Copy Markdown
Owner

Summary

Console
├── /experiments          Running / Needs you / Done, newest first
└── /experiments/<id>     live Trials, Verdict, rechecks, Arms and Bar
    └── Trial / Eval task Peek → Scores, reasons, attempts and session evidence

Builds layout E and the Inbox groups from #1020. Both pages use the CLI's retained Experiment view; rechecks stay in the CLI. Adds refs, title, end time, resume command, running Trials and a confined session inventory without changing Experiment identity or the Bar. Updates the manual and hand-written site pages.

Closes #1020.

Evidence

  • Before: /experiments returned 404; the CLI omitted the task inventory needed to show queued Trials.
    After: cargo test --test console experiments:: passes both route/CLI parity tests, including time-zone ordering, legacy logs becoming Session traces and session confinement.
  • Before: both Spec-review reproductions failed: paused Trials exposed 0 session links; resolving the final recheck left 0 visible task rows.
    After: both exact reproduction commands pass. Their assertions are retained in console/tests/experiments.test.cjs, which passes at 390 px and desktop widths, including live updates, Peeks, copying and filtering.
  • cargo test --test experiments: 66 passed, 1 ignored. npm test --prefix console: 68 passed, 1 skipped. npm test --prefix site: 44 passed. Phone screenshots inspected.
  • cargo clippy --all-targets -- -D warnings and cargo fmt --check pass. scripts/affected-tests issue-977 passes: 44 site tests and 2,565 Rust tests (58 skipped). /usr/bin/time -p scripts/affected-tests issue-977: 2,750.62 s wall time, including compilation and build-directory lock waiting; Rust test execution was 2,111.78 s.
  • CI is green on head 46642243; release artifact builds also pass.

Test seams

  • thirdshift experiments [id] --json and the paired Console HTTP pages over retained Experiment artifacts.
  • Real-browser Console interactions: live Trial updates, Peeks, rechecks, session evidence and 390 px layout.

Review

Independent Standards and Spec reviews use issue-977 as the fixed point. Addressed both Standards judgement calls and both reproduced Spec findings; both reviewers checked the corrections. Each axis read all 29 changed files, including the generated manual. Changed files left unread: none. Generated-output exclusions: none.

Reports are retained in this Session at .thirdshift-review-UjYEfX/standards.md and .thirdshift-review-UjYEfX/spec.md, each with its final files-read list.

Unaddressed findings

  • Standards: none.
  • Spec: none.
  • Security: none.

Security review outcome

No unaddressed introduced findings.

Merge Danger

Door: two-way

Read-only Console pages and additive, backward-compatible Experiment metadata; retained Trials, Scores, identity and Bar are preserved.

Blast Radius: Console

Also extends the experiments CLI's saved view and session selection.

Built with codex · gpt-6.1-sol · xhigh

@JacobStephens2
JacobStephens2 merged commit 3e82362 into issue-977 Oct 24, 2026
18 checks passed
@JacobStephens2
JacobStephens2 deleted the issue-1020 branch October 24, 2026 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant