Skip to content
This repository was archived by the owner on Aug 6, 2026. It is now read-only.

fix(tasks): infer repository from pull request prompts - #3748

Merged
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/infer-repository-from-pr-prompts
Jul 23, 2026
Merged

fix(tasks): infer repository from pull request prompts#3748
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/infer-repository-from-pr-prompts

Conversation

@tatoalo

@tatoalo tatoalo commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Problem

A new-task deep link without an explicit repository can open against the desktop app’s previously selected repository, even when the structured prompt identifies a pull request elsewhere.

Why: Pull request actions should start in the repository identified by the selected pull request without making ambiguous choices for multi-repository prompts.

Changes

Infer the repository when all structured pull request references point to one repository. Explicit repository parameters remain authoritative, and mixed-repository prompts remain unselected.

Generated-By: PostHog Code
Task-Id: 9fc61684-6e88-477c-bcf8-df5d08d42675
@tatoalo tatoalo self-assigned this Jul 23, 2026
@trunk-io

trunk-io Bot commented Jul 23, 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

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 16d357a.

@tatoalo tatoalo added the Stamphog This will request an autostamp by stamphog on small changes label Jul 23, 2026
@tatoalo
tatoalo marked this pull request as ready for review July 23, 2026 10:27
@greptile-apps

greptile-apps Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(tasks): infer repository from pull r..." | Re-trigger Greptile

@github-actions github-actions 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.

Contained, well-tested fix to a deep-link prompt-parsing helper in core; not risky territory (no auth, billing, API contract, or infra surface), diff matches description, and explicit repo params remain authoritative.

  • 👍 on the PR from greptile-apps[bot].
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 31L, 1F substantive, 79L/2F incl. docs/generated/snapshots — within ceiling
tier T1-agent / T1b-small (79L, 2F, single-area, fix)
stamphog 2.0.0b3 .stamphog/policy.yml @ 5bbe70c · reviewed head 16d357a

@tatoalo
tatoalo enabled auto-merge (squash) July 23, 2026 10:38
@trunk-io
trunk-io Bot merged commit 0a4e8f7 into main Jul 23, 2026
40 checks passed
@trunk-io
trunk-io Bot deleted the posthog-code/infer-repository-from-pr-prompts branch July 23, 2026 10:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants