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

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

Merged
tatoalo merged 3 commits into
mainfrom
chore/cloud-agent/user-github-integrations
May 1, 2026
Merged

tatoalo merged 3 commits into
mainfrom
chore/cloud-agent/user-github-integrations

Conversation

@tatoalo

@tatoalo tatoalo commented Apr 30, 2026 •

Copy link
Copy Markdown
Contributor

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

greptile-apps Bot commented Apr 30, 2026 •

Copy link
Copy Markdown
Contributor

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
@k11kirky k11kirky added the Create Release This will trigger a new release label May 1, 2026
@tatoalo
tatoalo merged commit c127528 into main May 1, 2026
15 checks passed
@tatoalo
tatoalo deleted the chore/cloud-agent/user-github-integrations branch May 1, 2026 16:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Create Release This will trigger a new release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants