gmackie/ForgeGraph
1
0
Fork 0
Code Issues Pull requests 3 Projects Releases Packages Wiki Activity Actions

ci: native AI code review + fix Forgejo CI for Node 24 #1

Open
gmackie wants to merge 5 commits from ci/github-actions-and-coderabbit into main
pull from: ci/github-actions-and-coderabbit
merge into: gmackie:main
Conversation 0 Commits 5 Files changed 2 +91 -2
gmackie commented 2026-05-13 01:58:27 +00:00
Owner
Copy link

Summary

  • Add AI code review via Claude API as a Forgejo Action
  • Fix CI: Node 24, --no-frozen-lockfile for patchedDependencies
  • Remove GitHub Actions (Forgejo is primary)

The AI review:

  1. Gets PR diff
  2. Sends to Claude for security/correctness/performance review
  3. Posts review to ForgeGraph review system
  4. Posts comment on Forgejo PR
  5. Fails CI check if changes requested (acts as gate)

Setup needed

  • ANTHROPIC_API_KEY secret in Forgejo
  • FORGEJO_TOKEN secret (for posting PR comments)
## Summary - Add AI code review via Claude API as a Forgejo Action - Fix CI: Node 24, --no-frozen-lockfile for patchedDependencies - Remove GitHub Actions (Forgejo is primary) The AI review: 1. Gets PR diff 2. Sends to Claude for security/correctness/performance review 3. Posts review to ForgeGraph review system 4. Posts comment on Forgejo PR 5. Fails CI check if changes requested (acts as gate) ## Setup needed - `ANTHROPIC_API_KEY` secret in Forgejo - `FORGEJO_TOKEN` secret (for posting PR comments)
* fix(api): replace all db.query.findFirst RAW patterns with db.select

The Drizzle v1 relational query API with { RAW: } expressions crashes in
the Cloudflare Worker runtime. Migrated all 34 remaining API route files
(69 occurrences) to use db.select().from().where().limit(1) which works
reliably through Hyperdrive.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* feat: DB-driven public feed overlays, replace broken db.query patterns

Replace all hardcoded public app definitions with DB-driven overlays
seeded via the API. Fix workspace lookup in the feed pipeline that
silently failed in CF Workers (db.query.findFirst with RAW clause →
db.select). Expand seed script to cover all 24 user-facing apps across
platform, ventures, and experiments tiers.

Also includes: API route fixes replacing broken db.query.findFirst
patterns with db.select across deploy, apps, and admin endpoints;
mobile app simplification; and assorted API hardening.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
- CI workflow: typecheck, lint, test in parallel with ForgeGraph reporting
- Beta deploy: auto-deploys to beta.forgegraf.com on push to main
- Production deploy: manual workflow_dispatch with environment protection
- CodeRabbit: assertive review profile with path-specific instructions
  for API routes (no db.query pattern), schema, and tRPC routers

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The project requires Node ^24.14.0 and has patchedDependencies that cause
lockfile mismatch with --frozen-lockfile on fresh installs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
pnpm 10 enables frozen lockfile by default in CI environments even
without the explicit flag. Must pass --no-frozen-lockfile to allow
patchedDependencies resolution.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ForgeGraph's repository is registered as gmackie/forgegraph (Forgejo
identity), not the GitHub org name.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ci: add native AI code review via Forgejo Actions + fix CI
Some checks failed
AI Code Review / review (pull_request) Failing after 5s
Details
CI / ci (pull_request) Successful in 13m4s
Details
e9288184a2 
- Add ai-review.yml: Claude-powered PR review that posts to ForgeGraph's
  review system and Forgejo PR comments. Fails the check if changes are
  requested, acting as a CI gate.
- Fix ci.yml: bump Node to 24, use --no-frozen-lockfile for
  patchedDependencies compat
- Remove GitHub Actions workflows and CodeRabbit config — ForgeGraph uses
  Forgejo as the primary CI/CD system, not GitHub

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Some checks failed
AI Code Review / review (pull_request) Failing after 5s
Details
CI / ci (pull_request) Successful in 13m4s
Details
This pull request has changes conflicting with the target branch.
  • .forgejo/workflows/ai-review.yml
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin ci/github-actions-and-coderabbit:ci/github-actions-and-coderabbit
git switch ci/github-actions-and-coderabbit

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff ci/github-actions-and-coderabbit
git switch ci/github-actions-and-coderabbit
git rebase main
git switch main
git merge --ff-only ci/github-actions-and-coderabbit
git switch ci/github-actions-and-coderabbit
git rebase main
git switch main
git merge --no-ff ci/github-actions-and-coderabbit
git switch main
git merge --squash ci/github-actions-and-coderabbit
git switch main
git merge --ff-only ci/github-actions-and-coderabbit
git switch main
git merge ci/github-actions-and-coderabbit
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
gmackie/ForgeGraph!1
No description provided.