Skip to content

Focus document launches in Atelier#271

Merged
samuelstroschein merged 5 commits into
mainfrom
codex/document-panel-startup
Jul 14, 2026
Merged

Focus document launches in Atelier#271
samuelstroschein merged 5 commits into
mainfrom
codex/document-panel-startup

Conversation

@samuelstroschein

@samuelstroschein samuelstroschein commented Jul 14, 2026

Copy link
Copy Markdown
Member

Summary

  • open normal folder workspaces with both Atelier side panels available by default
  • open direct and grouped document launches in a focused mode with both side panels collapsed
  • let users reopen the Files panel and cover persistent, standalone, grouped, and mixed launch paths
  • update the Atelier pin through Respect closed default panels atelier#32 and refresh its workspace dependencies
  • connect FlashType session persistence to Atelier’s host session-state store while retaining the legacy Lix-key fallback

Why

PR #269 left the document-focused panel behavior uncommitted. Direct file launches should prioritize the document, while folder launches should expose the full workspace. Advancing Atelier also moved shell state out of the shared Lix key, so FlashType needs to subscribe to the host session store to preserve transient documents across relaunches.

Impact

Users opening a file from Finder or grouped file arguments land directly in the document with a distraction-free layout. Folder launches retain visible Files and agent panels, and transient workspaces continue restoring their active document.

Dependency

Validation

  • pnpm exec vitest run electron/workspace.test.ts src/lib/atelier-document-state.test.ts src/lib/atelier-workspace-bridge.test.ts — 49 passed
  • pnpm run typecheck
  • focused Prettier and oxlint checks
  • pnpm run build
  • targeted Playwright rerun for relaunch, grouped standalone, and mixed launches — 3 passed

Note

Medium Risk
Changes workspace launch layout, session persistence path, and agent-turn review key layout; regressions could affect Finder opens, relaunch restore, or pending review badges, though coverage is broad in unit and e2e tests.

Overview
Adds document-focused launch mode: direct file and transient workspace targets get initialPanelMode: "document", which starts Atelier with both side panels collapsed; normal folder workspaces still open left and right by default.

Wires Atelier to a host memory session store (createMemorySessionStateStore) and updates the workspace bridge to persist open documents from that snapshot (subscribe + optional snapshot reads) instead of only watching the legacy atelier_ui_state Lix key. Document session helpers accept an injected UI state so relaunch/session checks stay accurate when the host owns shell state.

Agent turn review ranges move from one aggregated key to per-range keys under a prefix; readers merge values via agentTurnCommitRangesFromValues. The Files extension ignores stale watched: opens when a newer open request wins during slow imports.

Electron agent path probing runs a short script from env (avoids shells that reject long pasted commands) and finishes on marker or a brief post-exit drain. E2E gains shared workspace-session open-file assertions, dev __flashtypeE2E session state, multi-change review helpers, and collapsed-panel expectations for standalone launches.

Reviewed by Cursor Bugbot for commit bc17856. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor

cursor Bot commented Jul 14, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_7a9ac63e-6ef2-419e-b76e-a7199973864e)

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 14, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
flashtype-website bc17856 Commit Preview URL

Branch Preview URL
Jul 14 2026, 09:00 PM

@cursor

cursor Bot commented Jul 14, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_ced341ab-f7bd-4ce5-9e1b-f3f10eb93741)

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5d1fa3d294

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread electron/workspace.mjs
path: resolvedPath,
openFilePaths: uniqueWorkspaceRelativeFilePaths(openFilePaths),
name: path.basename(resolvedPath) || resolvedPath,
...(initialPanelMode ? { initialPanelMode } : {}),

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Preserve document mode for restored file sessions

For saved transient file workspaces, resolveWorkspaceSessionEntry still calls createEphemeralWorkspace(workspacePath, pendingOpenFilePaths) without the new mode argument, so a relaunch of a standalone/grouped file session with non-empty openFilePaths reopens with the normal left/right panels instead of the focused document layout used for the original file launch. Please propagate "document" when restoring session entries that have pending open files.

Useful? React with 👍 / 👎.

@blacksmith-sh

This comment has been minimized.

@cursor

cursor Bot commented Jul 14, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_b79ccf4e-535f-41ba-879a-90d1800cdb96)

@cursor

cursor Bot commented Jul 14, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_6cb564a5-8f33-4d8b-bbf0-b0a47f4ddd65)

@samuelstroschein
samuelstroschein merged commit fbb7064 into main Jul 14, 2026
5 checks passed
@samuelstroschein
samuelstroschein deleted the codex/document-panel-startup branch July 14, 2026 21:17
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