(bug) No CI job runs against the merged tip of main or a release branch

Aperta Adatta ai principianti
  • Qualcosa non va?
#6,967 2 commenti 1 reazione 0 assegnatari Vedi su GitHub

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
72/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
github-actions, shell, typescript, yaml
Ambito
ci-cd, testing

Direzione di ricerca

Inizia da .github/workflows/12-check-unit-tests.yml e controlla i trigger, i filtri dei percorsi e il job di controllo dei tipi esistente; se necessario, confrontalo con altri workflow in .github/workflows/. Aggiungi la copertura per i push su main e release/** e assicurati che le modifiche a hosting/docker-compose/** siano coperte dalla CI. Il lavoro è completato quando i push sul commit di merge eseguono i controlli e le modifiche a Docker Compose non sono più escluse.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug report ci/cd devops

Found during release QA for v0.119.0.

What happens. No CI job in this repository type-checks or tests the merged tip of main or a release/** branch. Every workflow that checks application code triggers only on pull_request or workflow_dispatch. Nothing re-runs once a PR merges.

This is not theoretical: during this release, two PRs that were each individually green on their own PR checks broke the build once merged together. The merged tree failed a plain TypeScript compile with two "cannot find name" errors in the runner's turn-settlement code. That would have made every agent turn fail in production with a raw error message, and no CI signal ever caught it, because nothing ever checked the actual merged commit.

Where. .github/workflows/12-check-unit-tests.yml, which owns the pnpm run typecheck step across the web, SDK, API, services and clients packages, triggers only on pull_request and workflow_dispatch. Across the whole .github/workflows/ directory, the only workflows with a push trigger are the docs deploy, the website deploy, and the secret scan; none of them check application code. The concurrency group in 12-check-unit-tests.yml already falls back to github.ref_name, so it needs no other change to accept a push trigger.

A related gap in the same file: hosting/docker-compose/** is not in its path filters at all, so changes there get no CI coverage of any kind (no compose validation, no shellcheck), unlike hosting/kubernetes/helm/**, which is covered.

Suggested fix. Add a push trigger for main and release/** to .github/workflows/12-check-unit-tests.yml so a merge is actually verified, not just a preview of it. Separately, add path coverage (or a dedicated lightweight job) for hosting/docker-compose/**.

Lingua principale
TypeScript
Stelle
4.8k
Fork
690
Merge medio
23h 29m
PR unite (30g)
540

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Agenta-AI/agenta

Tutte le issue di Agenta-AI/agenta

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.