Fix video comment effect wrapping by klopez4212 · Pull Request #5748 · block/buzz

klopez4212 · 2026-08-13T07:32:34Z

What changed

render video-review timecode chips inside the first Markdown paragraph so comment text wraps naturally around them
reuse the canonical video-review chip treatment across the timeline, Inbox previews, and Inbox detail
preserve video-review context in Inbox so timestamp chips remain interactive

Why

Video comments now support Markdown-like effects, but non-player surfaces rendered the timestamp beside a separate text layout. That kept the chip and comment from sharing the same inline flow and made Inbox behavior inconsistent with the player.

Validation

pnpm --dir desktop check
100 focused Markdown, timecode, video-review, and Inbox unit tests
pnpm --dir desktop build:e2e
focused video-attachment.spec.ts Playwright scenario
pre-push desktop typecheck and 4,761-test desktop suite
native Builderlab staging with the configured profile

Focused timeline and Inbox snapshots will be attached in a PR comment.

Signed-off-by: kenny lopez <klopez4212@gmail.com>

klopez4212 · 2026-08-13T07:33:25Z

🤖 Visual validation

Timeline thread

The video-review timecode chip and Markdown comment share one inline text flow.

Inbox detail

Inbox uses the same chip treatment and wrapped comment layout.

klopez4212 · 2026-08-13T07:37:30Z

🤖 @codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5755c0e49a

ℹ️ 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".

Signed-off-by: kenny lopez <klopez4212@gmail.com>

klopez4212 · 2026-08-13T07:56:49Z

🤖 @codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6d5a694c9a

ℹ️ 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".

Signed-off-by: kenny lopez <klopez4212@gmail.com>

klopez4212 · 2026-08-13T08:12:39Z

🤖 @codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f36b4a2783

ℹ️ 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".

Signed-off-by: kenny lopez <klopez4212@gmail.com>

klopez4212 · 2026-08-13T08:24:02Z

🤖 @codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8fd9138138

ℹ️ 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".

Signed-off-by: kenny lopez <klopez4212@gmail.com>

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 950c663445

ℹ️ 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".

klopez4212 · 2026-08-13T08:41:22Z

🤖 @codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 950c663445

ℹ️ 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".

Signed-off-by: kenny lopez <klopez4212@gmail.com>

klopez4212 · 2026-08-13T08:58:51Z

🤖 @codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e5471397e0

ℹ️ 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".

Signed-off-by: kenny lopez <klopez4212@gmail.com>

klopez4212 · 2026-08-13T11:29:30Z

🤖 @codex review

chatgpt-codex-connector

💡 Codex Review

buzz/desktop/src/features/messages/lib/videoReviewContext.ts

Lines 235 to 237 in a0d641e

    
           export function buildVideoReviewPresentationByMessageId( 
        
             args: Parameters<typeof buildVideoReviewContextsByMessageId>[0], 
        
           ) {

Document the exported presentation builder

Add a doc comment describing this newly exported builder and the relationship between its context and comment-root maps; the repository requires documentation for every new public API.

AGENTS.md reference: AGENTS.md:L114-L117

buzz/desktop/src/features/messages/lib/videoReviewContext.ts

Lines 246 to 248 in a0d641e

    
           export type VideoReviewPresentation = ReturnType< 
        
             typeof buildVideoReviewPresentationByMessageId 
        
           >;

Document the exported presentation type

Add a doc comment describing this newly exported presentation type, or keep it module-private if it is not intended as API; the repository requires documentation for every new public API.

AGENTS.md reference: AGENTS.md:L114-L117

ℹ️ 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".

Signed-off-by: kenny lopez <klopez4212@gmail.com>

klopez4212 · 2026-08-13T11:44:55Z

🤖 @codex review

chatgpt-codex-connector · 2026-08-13T11:49:39Z

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 248a4ef883

ℹ️ 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".

Fix video comment effect wrapping

5755c0e

Signed-off-by: kenny lopez <klopez4212@gmail.com>

klopez4212 added a commit that referenced this pull request Aug 13, 2026

screenshots: PR #5748

0ee5e71

klopez4212 marked this pull request as ready for review August 13, 2026 07:37

klopez4212 requested a review from a team as a code owner August 13, 2026 07:37

chatgpt-codex-connector Bot reviewed Aug 13, 2026

View reviewed changes

Comment thread desktop/src/shared/ui/markdown.tsx Outdated

Comment thread desktop/src/shared/ui/VideoReviewTimecodeButton.tsx

Handle video timecodes before block Markdown

6d5a694

Signed-off-by: kenny lopez <klopez4212@gmail.com>

chatgpt-codex-connector Bot reviewed Aug 13, 2026

View reviewed changes

Comment thread desktop/src/shared/ui/VideoReviewCommentMarkdown.tsx Outdated

Comment thread desktop/src/features/home/ui/InboxDetailPane.tsx Outdated

Stabilize video comment rendering

f36b4a2

Signed-off-by: kenny lopez <klopez4212@gmail.com>

chatgpt-codex-connector Bot reviewed Aug 13, 2026

View reviewed changes

Comment thread desktop/src/shared/ui/markdown/types.ts

Document Markdown inline runtime

8fd9138

Signed-off-by: kenny lopez <klopez4212@gmail.com>

chatgpt-codex-connector Bot reviewed Aug 13, 2026

View reviewed changes

Comment thread desktop/src/features/home/ui/InboxMessageRow.tsx

Preserve Inbox extensionless video reviews

950c663

Signed-off-by: kenny lopez <klopez4212@gmail.com>

chatgpt-codex-connector Bot reviewed Aug 13, 2026

View reviewed changes

Comment thread desktop/src/features/home/ui/InboxListPane.tsx Outdated

chatgpt-codex-connector Bot reviewed Aug 13, 2026

View reviewed changes

Comment thread desktop/src/shared/lib/rehypeLeadingInlineContent.ts

Comment thread desktop/src/shared/lib/rehypeLeadingInlineContent.ts

Harden video review timecode placement

e547139

Signed-off-by: kenny lopez <klopez4212@gmail.com>

chatgpt-codex-connector Bot reviewed Aug 13, 2026

View reviewed changes

Comment thread desktop/src/features/home/ui/InboxListPane.tsx

Fix legacy video attachment detection

a0d641e

Signed-off-by: kenny lopez <klopez4212@gmail.com>

chatgpt-codex-connector Bot reviewed Aug 13, 2026

View reviewed changes

Comment thread desktop/src/features/messages/lib/videoReviewContext.ts Outdated

Parse video attachments from Markdown AST

248a4ef

Signed-off-by: kenny lopez <klopez4212@gmail.com>

	export function buildVideoReviewPresentationByMessageId(
	args: Parameters<typeof buildVideoReviewContextsByMessageId>[0],
	) {

	export type VideoReviewPresentation = ReturnType<
	typeof buildVideoReviewPresentationByMessageId
	>;

Uh oh!

Conversation

klopez4212 commented Aug 13, 2026

What changed

Why

Validation

Uh oh!

klopez4212 commented Aug 13, 2026

Timeline thread

Inbox detail

Uh oh!

klopez4212 commented Aug 13, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

Uh oh!

klopez4212 commented Aug 13, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

Uh oh!

klopez4212 commented Aug 13, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

klopez4212 commented Aug 13, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

klopez4212 commented Aug 13, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

Uh oh!

klopez4212 commented Aug 13, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

klopez4212 commented Aug 13, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

klopez4212 commented Aug 13, 2026

Uh oh!

chatgpt-codex-connector Bot commented Aug 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant