Skip to content

feat: implement Figma workbench screens with UI test coverage - #27

Merged
johnnyhuy merged 1 commit into
mainfrom
implement-app-screens-from-figma
Oct 19, 2026
Merged

johnnyhuy merged 1 commit into
mainfrom
implement-app-screens-from-figma

Conversation

@johnnyhuy

Copy link
Copy Markdown
Contributor

Summary

Implement the Pidex Figma Workbench screens for new sessions, active conversations, Git changes, session history, and models/access. The shell uses the design's local icons and fonts, with workspace navigation, a command palette, file previews, and installed skills.

  • Connect session resume, streaming, stop/steer, fork, model selection, per-session drafts, rejected-prompt recovery, and Markdown export to the Pi RPC bridge.
  • Show real Git patches with file staging/unstaging and commits; validate workspace and archive paths before reading files.
  • Save workspace model/reasoning defaults and the device's reduced-motion preference. Runtime permissions and automatic worktrees remain explicitly unavailable.
  • Add unit and React component coverage for discovery, archives, previews, settings, composing, and keyboard navigation. Skip broken skill links without hiding sibling skills.
  • Run Electron UI tests using temporary Git workspaces, sessions, settings, and a deterministic Pi RPC fixture. The suite builds its own main/preload and starts its own renderer on a free port; PR CI needs no provider credentials and uploads screenshots and a Playwright trace.

Validation

  • mise run build: lint, typecheck, 51 unit tests, 22 component tests, production bundle.
  • mise run test:e2e: 10 Electron UI tests, covering all five screens, saved-session resume/fork, streaming/stop, real Git staging, persisted defaults, file/skill navigation, rejected prompts, the empty workspace picker, and minimum window dimensions.
  • Rechecked lint/typecheck after adding E2E diagnostics; git diff --cached --check passes.

Consequences

No migration or new runtime dependency. E2E tests exercise the actual Electron/main/preload/renderer stack with a mock Pi peer; live provider authentication and model responses are outside this suite. File previews are read-only and limited to text files up to 1 MB. Hunk-level Git actions, runtime-policy editing, and automatic worktree creation are not included.

Co-authored-by: OpenAI Codex <noreply@openai.com>
@johnnyhuy
johnnyhuy merged commit c061012 into main Oct 19, 2026
12 checks passed
@johnnyhuy
johnnyhuy deleted the implement-app-screens-from-figma branch October 19, 2026 09:14
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