Make hooks Forgejo-native (dual-stack GitHub + Forgejo) #1

New issue

Closed

opened 2026-02-23 15:04:27 +00:00 by forgejo_admin · 0 comments

forgejo_admin commented

2026-02-23 15:04:27 +00:00

Contributor

Problem

All hooks use gh CLI which only works with GitHub repos. Forgejo repos (like basketball-api) bypass all enforcement.

Solution

Update hooks to detect whether a repo remote points to GitHub or Forgejo, and use the appropriate API.

Hooks to update

check-issue.sh — validate issues via Forgejo API when remote is Forgejo
remind-review-loop.sh — trigger for Forgejo PRs too
block-pr-merge.sh — intercept Forgejo PR merges
block-main-commits.sh — already git-native, verify it works
Create forgejo-helper.sh — shared functions for Forgejo API calls

Notion SOPs to update

SOP-2: GitHub Branch Protection — add Forgejo equivalent
SOP-11: Never Merge PRs Without Manual Approval — Forgejo context
SOP-12: PR Review-Fix Loop — Forgejo PR workflow

Related: PROJ-17 Basketball Program SaaS (first Forgejo-deployed service)

## Problem All hooks use `gh` CLI which only works with GitHub repos. Forgejo repos (like basketball-api) bypass all enforcement. ## Solution Update hooks to detect whether a repo remote points to GitHub or Forgejo, and use the appropriate API. ## Hooks to update - [ ] `check-issue.sh` — validate issues via Forgejo API when remote is Forgejo - [ ] `remind-review-loop.sh` — trigger for Forgejo PRs too - [ ] `block-pr-merge.sh` — intercept Forgejo PR merges - [ ] `block-main-commits.sh` — already git-native, verify it works - [ ] Create `forgejo-helper.sh` — shared functions for Forgejo API calls ## Notion SOPs to update - [ ] SOP-2: GitHub Branch Protection — add Forgejo equivalent - [ ] SOP-11: Never Merge PRs Without Manual Approval — Forgejo context - [ ] SOP-12: PR Review-Fix Loop — Forgejo PR workflow Related: PROJ-17 Basketball Program SaaS (first Forgejo-deployed service)