Skip to content

Restore task action row (diff button) on channel task detail#3381

Merged
trunk-io[bot] merged 4 commits into
mainfrom
posthog-code/channel-task-diff-button
Jul 16, 2026
Merged

Restore task action row (diff button) on channel task detail#3381
trunk-io[bot] merged 4 commits into
mainfrom
posthog-code/channel-task-diff-button

Conversation

@k11kirky

Copy link
Copy Markdown
Contributor

Problem

Opening a task inside a channel (Bluebird Website space) no longer shows the diff button in the top right, so code changes made by the task can't be reviewed from there. Starting the same task from the "New task" tab (the /code space) still shows it.

The unified Bluebird chrome refactor (#3221) moved the task action row — branch selector, review-panel toggle (diff stats badge), handoff, skill buttons, task actions menu — from the code header bar into ContentHeader, which is intentionally mounted only outside /website. WebsiteLayout renders its own title bar with just the breadcrumb, so channel task details lost the whole row.

Changes

  • Extracted the task action row from ContentHeader into a reusable TaskHeaderActions component (pure move, no behavior change).
  • WebsiteLayout now renders TaskHeaderActions in its title bar when on a channel task detail route, restoring the diff button (and the rest of the row) for channel-filed tasks.

How did you test this?

  • Red-green: added WebsiteLayout.test.tsx asserting the action row renders on /website/$channelId/tasks/$taskId and not on other channel routes — failed before the fix, passes after.
  • Full @posthog/ui suite: 172 files / 1507 tests pass; pnpm --filter @posthog/ui typecheck and Biome clean.

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Created with PostHog Code

@trunk-io

trunk-io Bot commented Jul 13, 2026

Copy link
Copy Markdown

😎 Merged directly without going through the merge queue, as the queue was empty and the PR was up to date with the target branch - details.

@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit bc67d32.

@k11kirky
k11kirky marked this pull request as ready for review July 15, 2026 12:45
@greptile-apps

greptile-apps Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Security Review

The new action surface selects tasks by ID from an unfiltered user task list without validating the active channel. A mismatched channel/task URL can expose controls for a task outside the displayed channel.

Reviews (1): Last reviewed commit: "Restore task action row (diff button) on..." | Re-trigger Greptile

Comment thread packages/ui/src/features/canvas/components/WebsiteLayout.tsx Outdated
The unified Bluebird chrome moved the task action row (branch selector,
review-panel toggle, handoff, task actions) into ContentHeader, which is
mounted only outside /website — so tasks opened inside a channel lost the
diff button. Extract the row into TaskHeaderActions and mount it in
WebsiteLayout's title bar on channel task details.

Generated-By: PostHog Code
Task-Id: be6411d8-6b0f-429a-ad62-11312e69f7ac
@k11kirky
k11kirky force-pushed the posthog-code/channel-task-diff-button branch from 548a51d to 4d2f448 Compare July 15, 2026 13:36
k11kirky and others added 3 commits July 15, 2026 14:37
Generated-By: PostHog Code
Task-Id: 685b450e-680a-44fb-a565-c57e990e6a49
Generated-By: PostHog Code
Task-Id: 685b450e-680a-44fb-a565-c57e990e6a49

Copy link
Copy Markdown
Contributor Author

/trunk merge

@trunk-io
trunk-io Bot merged commit ed6cdfb into main Jul 16, 2026
31 checks passed
@trunk-io
trunk-io Bot deleted the posthog-code/channel-task-diff-button branch July 16, 2026 10:46
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.

2 participants