Przejdź do głównej zawartości

Code review i wdrożenie z Cursorem

Bezpieczne wdrażanie kodu napisanego w Cursorze polega na ułożeniu kontroli między diffem a produkcją: review przed pushem, Bugbot na pull requeście, deterministyczne bramki CI, zatwierdzanie według reguł tylko dla zmian niskiego ryzyka i monitorowany rollout. Recenzenci Cursora zgłaszają uwagi; o merge’u decydują testy, klasa ryzyka liczona w CI i wskazany z imienia właściciel.

Ta strona jest dla programistów i tech leadów, których agenci w Cursorze otwierają więcej pull requestów, niż zespół jest w stanie przeczytać. Schludne diffy ukrywają błędy: poluzowaną asercję, brakujące sprawdzenie uprawnień, migrację bez uzupełnienia danych. Strony poniżej zastępują czytanie linia po linii sprawdzalnymi bramkami.

Która strona o wdrażaniu z Cursorem odpowiada na twoje pytanie?

Dział zatytułowany „Która strona o wdrażaniu z Cursorem odpowiada na twoje pytanie?”
Twoje pytanieStrona
Który recenzent Cursora działa gdzie i jak zrobić triaż pull requesta?Code review w Cursorze
Jak dostroić reguły Bugbota i Autofix?Bugbot: wyuczone reguły i Autofix
Jak bezpiecznie pozwolić Cursorowi zatwierdzać pull requesty niskiego ryzyka?Rollouts oraz PR Routing & Approval
Jak zbudować pipeline GitHub Actions albo uruchomić w CI agenta tylko do odczytu?Pipeline’y CI/CD z Cursorem
Jak pisać konfigurację Terraform, Docker i Kubernetes?Infrastruktura jako kod z Cursorem

Bramka review jest tylko tak mocna jak testy, które za nią stoją; najpierw napisz je według strony testowanie i debugowanie z Cursorem.

W jakiej kolejności zmiana z Cursora powinna trafić na produkcję?

Dział zatytułowany „W jakiej kolejności zmiana z Cursora powinna trafić na produkcję?”

Najpierw uruchom tanie, prywatne kontrole.

  1. Przed pushem. Uruchom testy, sprawdzanie typów i linter, a potem jedno przejście review po gałęzi: Agent Review względem gałęzi głównej albo /review-bugbot. Checklistę review trzymaj w .cursor/BUGBOT.md: czytają ją Bugbot i Agent Review, a Bugbot pomija .cursor/rules/ (dokumentacja Cursora, nieoficjalne lustro na GitHubie odczytane 26 września 2026).

  2. Na pull requeście. Bugbot recenzuje diff, a w planach Teams i Enterprise Security Agents uruchamiają włączone przez ciebie kontrole. Przebieg Bugbota, który znajdzie problemy, domyślnie kończy się statusem neutral, a nie failure, więc wymaganie tego checka dowodzi tylko, że Bugbot się uruchomił. Przejście /review-bugbot przed pushem na tym samym diffie sprawia, że Bugbot pomija pull request; na ścieżkach high pomiń /review-bugbot przed pushem albo zmień diff. Komentarz bugbot run w pull requeście może ponownie uruchomić recenzję, ale dokumentacja Cursora nie mówi, że omija to pominięcie, więc sprawdź, czy check rzeczywiście się wykonał.

  3. W CI. Testy, typy, linter i check pakietu dowodów działają na commicie HEAD pull requesta, ale z definicjami workflow pobranymi z gałęzi bazowej, i wyliczają klasę ryzyka: low, standard albo high. Własny recenzent Cursora w CI dostaje narzędzia tylko do odczytu i żadnego tokenu z prawem zapisu.

  4. Przy zatwierdzaniu. PR Routing & Approval przydziela recenzentów według własności kodu i może zatwierdzić pull request low, który spełnia twoje kryteria. O merge’u decyduje ruleset GitHuba; CODEOWNERS przypisuje migracje, uwierzytelnianie, rozliczenia, publiczne API, testy i konfigurację CI do konkretnej, wskazanej osoby.

  5. Po merge’u. Zmiany standard wychodzą za flagą funkcji albo jako wdrożenie kanarkowe (canary); zmiany high dostają rollout etapowy. Promocję musi zatrzymać bramka twojego własnego systemu CD, gdy sygnału brakuje albo jest niejednoznaczny, a ścieżka revertu musi być przećwiczona. O funkcji Rollouts Cursora i teście akceptacyjnym, który musi przejść każdy monitor rolloutu, przeczytasz w Rollouts oraz PR Routing & Approval.

Skąd wiesz, że zmianę z Cursora można wdrożyć bez czytania każdej linii?

Dział zatytułowany „Skąd wiesz, że zmianę z Cursora można wdrożyć bez czytania każdej linii?”

Każda bramka produkuje dowód, który ma właściciela. Diff otwieraj tylko tam, gdzie wskazują dowody.

DowódCzego dowodziWłaściciel
Zielone CI na commicie HEAD: testy, typy, linter, pakiet dowodówDotychczasowe zachowanie działa; zmiana deklaruje, czego dotknęłaCI, skonfigurowane z gałęzi bazowej
Etykieta ryzyka wyliczona przez CI, nie przez agentaAgent może podnieść klasę, nigdy jej obniżyćWłaściciel .github/ w CODEOWNERS
Zatwierdzenie właściciela kodu na ścieżkach eskalacjiCzłowiek przyjął ryzyko tam, gdzie pomyłka kosztuje najwięcejWskazany właściciel kodu
Monitor rolloutu, który przeszedł test ze wstrzykniętą awariąProdukcja potwierdza zmianę albo szybko ją cofnieszDyżurny inżynier

Triaż opisuje code review PR-a agenta bez czytania każdej linii, a uzasadnienie strona dowody zamiast diffów. Odsetek celowo wstrzykniętych błędów, które wychwytują recenzenci, zmierzysz według strony o Bugbocie.

Co psuje się między review w Cursorze a działającym wdrożeniem?

Dział zatytułowany „Co psuje się między review w Cursorze a działającym wdrożeniem?”
ObjawPrzyczynaJak naprawić
Bugbot ignoruje twoją checklistę reviewBugbot nie czyta .cursor/rules/Przenieś ją do .cursor/BUGBOT.md; zostaw krótką regułę projektu, która do niej odsyła
Pull request z błędem przechodzi z zielonym BugbotemCheck zgłasza uwagi jako neutral i niczego nie blokujeTraktuj uwagi Bugbota jako jedną z przesłanek dla klasy standard, nie jako bramkę merge’u; decyzję zostaw testom i właścicielowi kodu. Jeśli to możliwe, włącz blokowanie na nierozwiązanych uwagach
Cursor zatwierdza pull request, który dotyka rozliczeńKlasa ryzyka pochodziła z opisu agentaWyliczaj klasę w CI na podstawie ścieżek; deklaracja agenta może ją tylko podnieść
Pull request kontrybutora osłabia własne reviewWorkflow albo prompt załadował się z gałęzi pull requestaŁaduj oba z gałęzi bazowej; zobacz tożsamość agentów i sekrety
Monitor rolloutu uznaje za zdrową zmianę, która zepsuła produkcjęSprawdzane sygnały nie miały telemetrii (luka w instrumentacji)Przed merge’em dodaj instrumentację tych sygnałów; w CD traktuj brak danych jak nieudany check
Rollback pada na produkcjiMigracja usunęła dane, których potrzebuje stary kodRozbij ją według wzorca expand and contract; stosuj progressive delivery

Używasz też innego agenta? Ten sam etap opisują strony przegląd i wdrożenie z Claude Code oraz przegląd i wdrożenie z Codexem.