Skip to content

feat(inbox): configurable base branch for inbox auto-PRs#2480

Merged
tatoalo merged 1 commit into
mainfrom
feat/cloud-agent/inbox-branch-switcher
Jun 8, 2026
Merged

feat(inbox): configurable base branch for inbox auto-PRs#2480
tatoalo merged 1 commit into
mainfrom
feat/cloud-agent/inbox-branch-switcher

Conversation

@tatoalo

@tatoalo tatoalo commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

adding configurable base branch for auto-PRs setting so inbox-driven cloud tasks can target a chosen branch (renderer side of #2470)

configurable_base

@tatoalo tatoalo self-assigned this Jun 4, 2026
@tatoalo tatoalo marked this pull request as ready for review June 4, 2026 09:15
@tatoalo tatoalo requested a review from a team June 4, 2026 09:16
@greptile-apps

greptile-apps Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor
Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
apps/code/src/renderer/features/inbox/hooks/useCreatePrReport.ts:107-111
The two-step lookup normalises only the *lookup* key, not the *stored* key. If the map entry was stored with uppercase letters (e.g. `MyOrg/MyRepo`) and `cloudRepository` arrives in lowercase, both lookups miss and the PR silently falls back to the repo default. Storing and looking up with a single canonical form (e.g. always `toLowerCase()`) would make the behaviour unambiguous. The current fallback is safe but the asymmetry could cause a confusing "setting has no effect" situation if casing diverges between the UI's source and the report's `cloudRepository`.

```suggestion
    const baseBranchOverrides = autonomyConfig?.autostart_base_branches ?? {};
    const baseBranch =
      baseBranchOverrides[cloudRepository] ??
      baseBranchOverrides[cloudRepository.toLowerCase()] ??
      baseBranchOverrides[
        Object.keys(baseBranchOverrides).find(
          (k) => k.toLowerCase() === cloudRepository.toLowerCase(),
        ) ?? ""
      ] ??
      null;
```

Reviews (1): Last reviewed commit: "feat(inbox): configurable base branch fo..." | Re-trigger Greptile

Comment thread apps/code/src/renderer/features/inbox/hooks/useCreatePrReport.ts Outdated
@tatoalo tatoalo force-pushed the feat/cloud-agent/inbox-branch-switcher branch 4 times, most recently from a1fe16a to a1ec463 Compare June 8, 2026 14:45
@tatoalo tatoalo force-pushed the feat/cloud-agent/inbox-branch-switcher branch from a1ec463 to 48d86da Compare June 8, 2026 14:46
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 48d86da.

@tatoalo tatoalo merged commit ff99183 into main Jun 8, 2026
19 checks passed
@tatoalo tatoalo deleted the feat/cloud-agent/inbox-branch-switcher branch June 8, 2026 15:45
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