From adb3bce0f69074157348d40795739bdde4081d89 Mon Sep 17 00:00:00 2001 From: Sinabina Date: Tue, 21 Jul 2026 17:14:35 -0700 Subject: [PATCH] docs: require GSTACK PR liveness screenshot, drop changesets mention MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Every contribution must attach a screenshot with "GSTACK PR" typed live into a real surface (terminal, search bar, editor) — not overlaid or edited onto the image — to confirm a human on a real machine opened the PR. Remove the Changesets-migration reference from the CI-enforcement note since that migration is not planned. Co-Authored-By: Claude Opus 4.8 (1M context) --- .github/PULL_REQUEST_TEMPLATE.md | 9 +++++++++ CONTRIBUTING.md | 15 +++++++++++---- 2 files changed, 20 insertions(+), 4 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 17a8bcb75..0d631dac8 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -27,8 +27,17 @@ changed. --> - **Verified live by:** - **Did NOT test:** +## Liveness proof (required) + + + ## Checklist +- [ ] Liveness screenshot attached: `GSTACK PR` typed live into a real surface (not edited onto the image) - [ ] Commits are signed off (`git commit -s`) — DCO - [ ] This is not a generated-file-only diff (I edited the source/template and regenerated) - [ ] No ETHOS.md edits, and no changes to voice / founder perspective / YC references diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 88891443b..5ca77f914 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -107,6 +107,13 @@ holds us to internally: **prove it or don't say it.** "Pre-existing failure," submit it. Unsigned commits do not merge. 4. **A scope statement.** Three lines: what changed, how you verified it *live*, what you did not test. +5. **A liveness screenshot.** Attach a screenshot of your own machine with the + text `GSTACK PR` typed **live** into a real surface — your terminal prompt, a + shell command, your browser's address/search bar, an editor buffer, anything + genuinely on screen. It must be *typed into a live UI*, **not** drawn, + overlaid, or edited onto the image afterward. This proves a human on a real + machine opened this PR. A screenshot with `GSTACK PR` painted onto the pixels + instead of typed into a live surface is an automatic close. ### Auto-closed, no discussion @@ -130,10 +137,10 @@ A linked issue you are fixing, a failing test you make pass, a live transcript showing the improvement, a tight scope statement, and a signed commit. That PR gets reviewed the same day. -> **Coming with the Changesets migration:** the four requirements above will be -> enforced in CI — a required changeset (carrying the human "why"), a DCO check, -> and a generated-file-only-diff guard. Until then they are reviewer-enforced, -> and reviewers will hold the line. +> These requirements are reviewer-enforced. The DCO check and the +> generated-file-only-diff guard are the pieces we will move into CI; the human +> "why," the live evidence, and the liveness screenshot stay reviewer-judged. +> Reviewers hold the line — especially on the screenshot. ## Quick start