/update: the bot is refused on contributors' forks (worked on the owner's fork), and it doesn't log why
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 65/100
Direzione di ricerca
Start with the /update workflow code and run 39116506769 to find where the update-branch API response is handled; add failure status and message logging without secrets. Reproduce the request on a contributor fork and compare it with the successful maintainer call. Update CONTRIBUTING.md and docs/AGENT_WORKFLOW.md to describe the confirmed behavior; done when the cause is confirmed and the docs match.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Follow-up to #472 (#464).
During its pre-merge test, /update could update a branch in itsskofficial/Sentient, the repo owner's own fork. On the first real use, on #407 from syedmustafan/Sentient with "Allow edits by maintainers" on, PUT /pulls/{n}/update-branch with the workflow token was refused. The fallback worked as designed: the button hint plus the needs update label. A maintainer then ran the same API call with their own credentials and it succeeded.
So the built-in GITHUB_TOKEN probably can't push to forks owned by other people. It only worked on the admin's own fork. The run log (run 39116506769) also doesn't show the status or message GitHub returned, so this can't be confirmed from the log.
To do:
- Log the update-branch response status and message (no secrets) whenever it fails.
- Confirm the cause on a contributor fork, and update CONTRIBUTING.md and docs/AGENT_WORKFLOW.md to match. The likely result is that
/updatemerges main on same-repo branches only, and on forks it offers the button and labels the PR for a maintainer. That is fine with our policy, because maintainers handle labelled PRs. - Only if the owner wants it: a GitHub App with contents:write could update fork branches. That needs a setup decision, so don't add new secrets without asking.
- Lingua principale
- Python
- Stelle
- 702
- Fork
- 113
- Merge medio
- 5h 6m
- PR unite (30g)
- 172
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di existence-master/Sentient
-
Telegram/Discord: tapping "Local only" in the /model list should ask too, and "/model Local only no" should cancelForse già presa @bernalalexis-try l’ha presa oggi. Apertaarea: channels enhancement good first issue priority: P2 size: S
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
existence-master/Sentient#457 · 2 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
area: desktop bug good first issue priority: P2 size: S
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
existence-master/Sentient#454 ·
I maintainer di solito rispondono entro 1 giorno
-
area: desktop bug priority: P2 size: S
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
existence-master/Sentient#440 ·
I maintainer di solito rispondono entro 1 giorno
-
area: desktop bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
existence-master/Sentient#426 ·
I maintainer di solito rispondono entro 1 giorno
-
area: memory bug flaky
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
existence-master/Sentient#422 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di existence-master/Sentient
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
mohitmishra786/readprism#322 ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
API-key credentials accept a blank key: empty via the API, spaces-only via the builder's Add API key dialogForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
Significant-Gravitas/AutoGPT#15727 ·
I maintainer di solito rispondono entro 1 giorno