CalvinBackup/gstack
1
0
Fork 0
mirror of https://github.com/garrytan/gstack.git synced 2026-05-02 11:45:20 +02:00
Code Issues Packages Projects Releases Wiki Activity

feat: add reverted QA commit detection to /ship

Browse Source 
During pre-landing review, /ship now checks for reverted fix(qa):
commits in the branch history and recommends /debug for systematic
investigation. Informational only — does not block shipping.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Garry Tan
2026-03-18 11:09:19 -07:00
parent b9d734ad91
commit 21daf76723
2 changed files with 16 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ship/SKILL.md
+8
View File
@@ -727,6 +727,14 @@ Substitute: TIMESTAMP = ISO 8601 datetime, STATUS = "clean" if 0 findings or "is
Save the review output — it goes into the PR body in Step 8.
**Reverted QA fix detection:** Check if the branch history contains reverted QA commits:
```bash
git log origin/<base>..HEAD --oneline | grep -i 'revert.*fix(qa)'
```
If reverted QA commits are found, note in review output:
"This branch has reverted QA fixes — the underlying bugs may still be present. Consider running `/debug` on the affected areas before shipping."
This is INFORMATIONAL only — does not block shipping.
---
## Step 3.75: Address Greptile review comments (if PR exists)
ship/SKILL.md.tmpl
+8
View File
@@ -361,6 +361,14 @@ Review the diff for structural issues that tests don't catch.
Save the review output — it goes into the PR body in Step 8.
**Reverted QA fix detection:** Check if the branch history contains reverted QA commits:
```bash
git log origin/<base>..HEAD --oneline | grep -i 'revert.*fix(qa)'
```
If reverted QA commits are found, note in review output:
"This branch has reverted QA fixes — the underlying bugs may still be present. Consider running `/debug` on the affected areas before shipping."
This is INFORMATIONAL only — does not block shipping.
---
## Step 3.75: Address Greptile review comments (if PR exists)