[Bug]: Generated PR bodies say "Not run" under Testing even when the thread ran the tests
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 62/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- tooling
Direzione di ricerca
Inizia da buildPrContentPrompt in apps/server/src/textGeneration/TextGenerationPrompts.ts e leggi il prompt che assembla quando non esiste un template di PR; poi traccia i suoi chiamanti per vedere quale contesto (trascrizione della thread, chiamate a strumenti, esecuzioni di test) è disponibile in quel punto di chiamata. Controlla se esistono test intorno alla generazione del corpo della PR/del prompt per capire dove vivrebbe un controllo di regression. "Done" significa che la sezione ## Testing riflette verifiche che sono effettivamente state eseguite, oppure è omessa/formulata in modo neutro quando l'autore non ha evidenze — nota che l'issue lascia la direzione esatta alla decisione dei maintainer, quindi conferma l'approccio preferito prima di modifiche importanti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Before submitting
- I searched existing issues and did not find a duplicate.
- I included enough detail to reproduce or investigate the problem.
Area
apps/server
Steps to reproduce
- Open a project whose repo has no PR template
- In a thread, have the agent make a change and run the tests, for example
bun test, and confirm they pass in the transcript. - Use the commit and create PR action.
- Read the generated PR body.
Expected behavior
The ## Testing section reflects the checks that actually ran in the thread. If the writer has no information about testing, it should leave the section out or say so neutrally, rather than stating that tests were not run.
Actual behavior
The Testing section almost always says "Not run." or "Not run in this change request.", even when the thread ran and passed the tests. Reviewers read that as a statement that the change wasn't tested.
This comes from buildPrContentPrompt in apps/server/src/textGeneration/TextGenerationPrompts.ts. When there's no template, the prompt requires:
- body must be markdown and include headings '## Summary' and '## Testing'
- under Testing, include bullet points with concrete checks or 'Not run' where appropriate
but the only inputs are the base and head branches, the commit summary, the diff stat and the diff patch. The writer never sees the thread or its tool calls, so it can't know about any checks. It has to fill a required section, and 'Not run' is the fallback the prompt offers, so it picks that.
A user reported the same thing in #264 ("it somehow always adds to my PR description, that the tests and ci did not run... even though the agent clearly said it was validated"). #264 was closed with PR template support, which only helps repos that have a template. The default path still produces this.
Possible directions, for maintainers to decide:
- Pass a short summary of the thread's verification commands and their exit status into the PR prompt.
- Make
## Testingoptional when the writer has no evidence, and drop'Not run'as the suggested fallback.
Impact
Minor bug or occasional failure
Version or commit
0.0.46-nightly.20261006.2752
Environment
Kubuntu 26.04, T3 Code Nightly desktop. The source control writer is GPT-6 Luna (low, fast), but the cause seems to be the prompt, not the model.
Logs or stack traces
Screenshots, recordings, or supporting files
No response
Workaround
- Add a PR template without a Testing section, or
- set custom writing instructions to leave out the Testing section, or
- have the agent open the PR itself by asking it to.
- Lingua principale
- TypeScript
- Stelle
- 24.8k
- Fork
- 6.4k
- Merge medio
- 13h 9m
- PR unite (30g)
- 271
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- 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 pingdotgg/t3code
-
[Bug]: Side-by-side diff squeezes the "N unmodified lines" divider into the line-number columnApertabug via-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
pingdotgg/t3code#17157 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: Local environment can't be turned back on once disabled (toggle hidden behind "No environment selected")Forse già presa Una pull request collegata a questa issue è aperta o già unita. Apertabug via-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
pingdotgg/t3code#17013 · 1 commento · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: T3 Connect doc says held webhooks are kept per environment, but the relay keeps one inbox per linkForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertadocumentation via-triage
Difficoltà 1/5 1-3 ore Idoneità per principianti 83/100
pingdotgg/t3code#16995 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: T3 Connect doc gives the relay webhook path as /v1/hooks/:environmentId/..., but the relay routes on :endpointKeyForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertadocumentation via-triage
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
pingdotgg/t3code#16988 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di pingdotgg/t3code
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
MystenLabs/MemWal#1163 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Mondriaan
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
knaw-huc/textannoviz#709 ·
I maintainer di solito rispondono entro 1 giorno
-
billion-context-pi
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
ranxianglei/billion-context#2521 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Add: YRF Music NepalApertastreams:add
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100