Skip to content

Expose recorded Workspace artifacts in the Demo Files panel - #775

Merged
luokerenx4 merged 2 commits into
devfrom
codex/demo-workspace-files
Jul 29, 2026
Merged

Expose recorded Workspace artifacts in the Demo Files panel#775
luokerenx4 merged 2 commits into
devfrom
codex/demo-workspace-files

Conversation

@luokerenx4

Copy link
Copy Markdown
Contributor

What changed

  • assign every readable demo artifact to its owning Workspace
  • build root and nested directory listings from that artifact index, including byte sizes and fresh demo-session timestamps
  • preserve the production directory contract by sorting folders first and rejecting parent-directory traversal
  • add handler coverage for the featured AAPL report, nested Chat notes, complete artifact indexing, and traversal rejection

Why

The featured AAPL Session says it saved the full earnings note into the Workspace, and the demo already serves that report through the file-read endpoint. However, the directory-listing handler always returned an empty array, so opening the same Workspace's Files panel showed empty. The same disconnect hid the research notes used by Tracked and scheduled-report fixtures.

User impact

Demo visitors can now browse the artifacts the recorded Sessions claim to create: the AAPL report appears in its Workspace root, Chat notes expose a navigable rotation/ directory, and clicking a listed file opens the existing viewer with its Session return path intact.

Validation

  • pnpm exec vitest run ui/src/demo/handlers/workspaces-files.spec.ts (4 passed)
  • npx tsc --noEmit
  • cd ui && npx tsc -b
  • pnpm test (358 files passed, 1 skipped; 3392 tests passed, 9 skipped)
  • pnpm -F open-alice-ui build:demo
  • real Chinese demo walkthrough: AAPL Session -> Files -> research-AAPL-q1.md; verified the report opens with sessionId=demo-session
  • real Chinese demo walkthrough: Chat Workspace -> Files -> rotation/; verified all three nested research notes are listed

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openalice-demo Ready Ready Preview, Comment Jul 29, 2026 5:39am

Request Review

@luokerenx4 luokerenx4 added area:demo Demo runtime, fixtures, handlers, or banner theme:demo Demo fidelity, fixtures, or simulated interactions workflow:parallel Autonomous parallel contribution; leave open for later acceptance labels Jul 29, 2026 — with ChatGPT Codex Connector
@luokerenx4
luokerenx4 marked this pull request as ready for review July 29, 2026 05:37
…iles

# Conflicts:
#	ui/src/demo/handlers/workspaces.ts
@luokerenx4
luokerenx4 merged commit 0353706 into dev Jul 29, 2026
4 of 5 checks passed
@luokerenx4
luokerenx4 deleted the codex/demo-workspace-files branch July 29, 2026 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:demo Demo runtime, fixtures, handlers, or banner theme:demo Demo fidelity, fixtures, or simulated interactions workflow:parallel Autonomous parallel contribution; leave open for later acceptance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant