Skip to content

Add deterministic announcement demo workspace#1967

Open
klopez4212 wants to merge 3 commits into
mainfrom
kennylopez-announcement-demo
Open

Add deterministic announcement demo workspace#1967
klopez4212 wants to merge 3 commits into
mainfrom
kennylopez-announcement-demo

Conversation

@klopez4212

Copy link
Copy Markdown
Contributor

No description provided.

@klopez4212
klopez4212 requested a review from a team as a code owner July 16, 2026 16:44

@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: 3b032d4f1b

ℹ️ 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 desktop/src/main.tsx
Comment on lines +37 to +39
const isAnnouncementDemo =
url.searchParams.get("demo") === ANNOUNCEMENT_DEMO_QUERY_VALUE ||
import.meta.env.VITE_ANNOUNCEMENT_DEMO === "1";

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 Gate demo query to non-production builds

When a production/web build is opened with ?demo=announcement, this branch now treats it as the announcement demo, installs the E2E mock bridge, and then rewrites persistent localStorage like buzz-communities/buzz-active-community-id to the fake Honeycomb workspace. A shared link with that query can therefore switch real users away from their saved communities and load test-only mocks; the query-triggered path should be limited to import.meta.env.DEV or an explicit build-time demo flag.

Useful? React with 👍 / 👎.

@klopez4212
klopez4212 force-pushed the kennylopez-announcement-demo branch 2 times, most recently from e6938be to e1801db Compare July 17, 2026 13:46

@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: e1801db5b1

ℹ️ 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 ANNOUNCEMENT_DEMO.md
Comment on lines +19 to +20
The local demo relay also includes a running agent named **Scout**. Configure
OpenAI or Anthropic in the Agents screen, then `@mention` Scout in

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 Correct the demo agent name in the runbook

When someone follows this runbook and @mentions Scout, no demo agent will autocomplete or respond: the seeded announcement agents are Fizz, Honey, and Bumble in desktop/src/testing/announcementDemoFixtures.ts, and the e2e flow also mentions Fizz. Please update the instructions to match the seeded agent names or add a Scout agent fixture.

Useful? React with 👍 / 👎.

@klopez4212
klopez4212 force-pushed the kennylopez-announcement-demo branch from e1801db to d16d405 Compare July 17, 2026 15:20

@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: d16d405a64

ℹ️ 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 on lines +9316 to +9317
for (const channelId of channelIds) {
maybeStartAnnouncementDemoLiveConversation(channelId);

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 Start demo scripts only from the active channel

When the announcement demo loads, the app-level unread hook (useLiveChannelUpdates) opens live-* subscriptions for every channel, not just the timeline the user is viewing. Starting scripted conversations from every live subscription here means channels like #flight-path begin emitting messages a few seconds after startup while the user is still on Home/another channel, which can flip the seeded read state/unread dot and consume the staged reveal before the recorder opens that channel. Gate this trigger to the active timeline subscription or another explicit demo start signal.

Useful? React with 👍 / 👎.

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