Skip to content

fix(desktop): archive tasks from focused editors - #79643

Merged
trunk-io[bot] merged 1 commit into
masterfrom
posthog-code/fix-task-archive-shortcut
Aug 7, 2026
Merged

fix(desktop): archive tasks from focused editors#79643
trunk-io[bot] merged 1 commit into
masterfrom
posthog-code/fix-task-archive-shortcut

Conversation

@MattPua

@MattPua MattPua commented Aug 7, 2026

Copy link
Copy Markdown
Member

Problem

  • Task users could press the archive shortcut while focused on text input and see no action.
  • The task-detail handler ignored focused inputs and rich-text editors.

Changes

  • The archive shortcut now works when an input or editor has focus.
  • No visual change.

How did you test this code?

  • The desktop UI test suite completed without test failures.
  • The full desktop production build completed after restoring workspace dependencies.
  • Formatting passed for the changed file.
  • Package typecheck remains blocked by existing errors in billing, canvas, loops, and chat-thread files.

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

Automatic notifications

  • Publish to changelog?

Docs update

  • None. This fixes existing shortcut behavior without changing a documented workflow.

🤖 Agent context

Autonomy: Human-driven (agent-assisted)

  • Authored with PostHog Code for a user-reported archive shortcut failure.
  • Skills invoked: /writing-tests and /writing-pr-descriptions.
  • Extended the existing global shortcut behavior to focused task editors.

Created with PostHog Code

Generated-By: PostHog Code
Task-Id: cbf77652-78d0-46a8-a4b8-e09db430e140
@MattPua MattPua self-assigned this Aug 7, 2026
@trunk-io

trunk-io Bot commented Aug 7, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

@github-actions github-actions Bot added the feature/desktop Feature Tag: Desktop label Aug 7, 2026
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 91259a1.

@MattPua
MattPua marked this pull request as ready for review August 7, 2026 14:17
@MattPua MattPua added the stamphog Request AI approval (no full review) label Aug 7, 2026
@greptile-apps

greptile-apps Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(desktop): archive tasks from focused..." | Re-trigger Greptile

@trunk-io

trunk-io Bot commented Aug 7, 2026

Copy link
Copy Markdown

Static BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@stamphog stamphog 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.

Trivial, contained keyboard-shortcut fix in desktop UI, matches PR description, no risky territory, and author has STRONG familiarity with these exact lines.

  • Author wrote 100% of the modified lines and has 1 merged PRs in these paths (familiarity STRONG).
  • 👍 on the PR from greptile-apps[bot], hex-security-app[bot].
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 6L, 1F substantive — within ceiling
tier T1-agent / T1a-trivial (6L, 1F, single-area, fix)
stamphog 2.0.0b4 .stamphog/policy.yml @ bc0f7d3 · reviewed head 91259a1

@trunk-io
trunk-io Bot merged commit 8df24a2 into master Aug 7, 2026
322 of 323 checks passed
@trunk-io
trunk-io Bot deleted the posthog-code/fix-task-archive-shortcut branch August 7, 2026 15:25
@deployment-status-posthog

deployment-status-posthog Bot commented Aug 7, 2026

Copy link
Copy Markdown

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-08-07 16:13 UTC Run
prod-us ✅ Deployed 2026-08-07 16:33 UTC Run
prod-eu ✅ Deployed 2026-08-07 16:31 UTC Run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature/desktop Feature Tag: Desktop stamphog Request AI approval (no full review)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant