Skip to content

feat(code): use user GitHub integrations for cloud tasks#1951

Open
tatoalo wants to merge 2 commits intomainfrom
chore/cloud-agent/user-github-integrations
Open

feat(code): use user GitHub integrations for cloud tasks#1951
tatoalo wants to merge 2 commits intomainfrom
chore/cloud-agent/user-github-integrations

Conversation

@tatoalo
Copy link
Copy Markdown
Contributor

@tatoalo tatoalo commented Apr 30, 2026

Problem

cloud task creation and handoff should use the user's linked GitHub identity instead of relying on a local gh token or the project gh integration. This keeps PR authorship tied to the user who starts the task and works for Code-created and handoff-created cloud runs

Before:
gh_authorship_sad

After:
gh_authorship_happy

@tatoalo tatoalo self-assigned this Apr 30, 2026
@tatoalo tatoalo marked this pull request as ready for review April 30, 2026 11:53
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 30, 2026

Comments Outside Diff (1)

  1. apps/code/src/renderer/hooks/useIntegrations.ts, line 1051-1053 (link)

    P2 Superfluous useEffect — no-op on mount

    This effect resets requestedLimit to REPOSITORIES_PAGE_SIZE, but useState(REPOSITORIES_PAGE_SIZE) already initialises the state to that value. The effect fires once on mount and writes the same value that is already there, so it has no observable effect and can be removed.

    Prompt To Fix With AI
    This is a comment left during a code review.
    Path: apps/code/src/renderer/hooks/useIntegrations.ts
    Line: 1051-1053
    
    Comment:
    **Superfluous `useEffect` — no-op on mount**
    
    This effect resets `requestedLimit` to `REPOSITORIES_PAGE_SIZE`, but `useState(REPOSITORIES_PAGE_SIZE)` already initialises the state to that value. The effect fires once on mount and writes the same value that is already there, so it has no observable effect and can be removed.
    
    How can I resolve this? If you propose a fix, please make it concise.
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/hooks/useIntegrations.ts:1051-1053
**Superfluous `useEffect` — no-op on mount**

This effect resets `requestedLimit` to `REPOSITORIES_PAGE_SIZE`, but `useState(REPOSITORIES_PAGE_SIZE)` already initialises the state to that value. The effect fires once on mount and writes the same value that is already there, so it has no observable effect and can be removed.

Reviews (1): Last reviewed commit: "feat(code): use user github integrations..." | Re-trigger Greptile

@tatoalo tatoalo force-pushed the chore/cloud-agent/user-github-integrations branch from bacac97 to 4d65024 Compare April 30, 2026 13:10
@tatoalo tatoalo requested a review from a team April 30, 2026 16:43
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.

1 participant