Maintain Open Session verification map - #387
open-session-os-tella-dev[bot] wants to merge 1 commit into
Conversation
Add live-tested coverage for Databases and Desk, and correct stale accessibility selectors for sessions and archived search. Co-authored-by: open-session-os-tella-dev[bot] <309495949+open-session-os-tella-dev[bot]@users.noreply.github.com>
🤖 OS review · request changes · quality 4/5 · risk lowSafe once the P2 below is fixed. The Desk persistence step relies on demo content that does not exist. 🟢 Risk low · recovery in minutes 1 inline comment below. Reviewed 🔁 Not merge-ready and no live session owns this branch — add the |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![open-session-os-tella-dev[bot]](https://avatars.githubusercontent.com/u/65127121?s=60&v=4){kind=small}
|
|
||
| - **Open Desk.** Open `/` at 1440x900, wait for button `Open the Desk`, capture the underlying page, and choose it. Wait for `dialog` named `Desk`. | ||
| - **Inspect text controls.** Capture a snapshot containing the Desk composer `Ask anything…`, suggested prompt buttons, `Clear chat`, `Open as a full session`, and `Minimise Desk`. Do not send a prompt merely to populate proof. | ||
| - **Minimize and reopen.** Choose `Minimise Desk`, require the dialog to disappear, then choose `Open the Desk` again. Existing seeded Desk content should remain because Desk is a durable session. |
There was a problem hiding this comment.
🟠 P2 — The demo cannot satisfy the seeded Desk persistence check
A fresh isolated run has no Desk fixture under src/server/demo; opening Desk calls ensureDeskSession, which creates a new empty session in src/server/desk.ts. Minimizing and reopening therefore only shows the same blank view, so the documented expectation of “existing seeded Desk content” cannot be observed and does not verify transcript durability. Either seed a synthetic Desk transcript or change this step to verify a stable session identity before and after reopening, for example by recording the full-session URL both times.
Summary
Verification
bun run checkverify-opensessionon desktop and phoneartifacts/verification/opensession/verify-20260914-060146-710829/locallyStarted by Daily verification skill maintenance in this OS session