Skip to content

Add explicit split payout approval#41

Open
FJAC-web wants to merge 1 commit into
skndash96:mainfrom
FJAC-web:codex/manual-split-payouts
Open

Add explicit split payout approval#41
FJAC-web wants to merge 1 commit into
skndash96:mainfrom
FJAC-web:codex/manual-split-payouts

Conversation

@FJAC-web

@FJAC-web FJAC-web commented Jun 8, 2026

Copy link
Copy Markdown

Summary

  • Add an optional splitPayouts approval payload so maintainers can explicitly distribute a locked bounty across multiple GitHub contributors.
  • Validate split recipients by unique GitHub username and exact cent-level total before sending any payout.
  • Resolve each recipient through their PR wallet tag, registered email fallback, or unclaimed payout flow, then persist payout/activity rows per recipient.
  • Add a maintainer UI for split rows prefilled from competing PR activity, while keeping the existing single-winner approval path unchanged.
  • Document the multi-contributor approval contract in public/LLM.md for agent-readable usage.

Validation

  • tsc --noEmit
  • focused ESLint on touched files
  • git diff --check

Fixes #12

@vercel

vercel Bot commented Jun 8, 2026

Copy link
Copy Markdown

@FJAC-web is attempting to deploy a commit to the skndash96's projects Team on Vercel.

A member of the Team first needs to authorize it.

@bountic-bot

bountic-bot Bot commented Jun 8, 2026

Copy link
Copy Markdown

⚡️ Bounty Competition Started

@FJAC-web has submitted a PR that references an issue with a bounty. Learn more about this bounty: https://bountic.vercel.app/b/skndash96/bountic/issues/12

When this PR is merged, the bounty will be locked and ready for payout approval.


Bountic: Autonomous USDC bounties for open source

@FJAC-web

FJAC-web commented Jun 8, 2026

Copy link
Copy Markdown
Author

Small note on the visible check: the Vercel failure is the standard fork authorization gate (A member of the Team first needs to authorize it), not a code/test failure from this branch.

Local validation completed before opening the PR:

  • tsc --noEmit
  • focused ESLint on the touched files
  • git diff --check

@FJAC-web

FJAC-web commented Jun 8, 2026

Copy link
Copy Markdown
Author

Additional local validation:

  • next build passes with the required env vars present (dummy local values used only for validation).
  • A plain next build without env compiles and typechecks, then fails during prerender because NEXT_PUBLIC_APP_URL, Supabase, and service keys are required by the app environment schema.
  • Full-repo eslint still reports pre-existing issues outside this branch (app/mock-checkout/[checkoutId]/page.tsx, app/page.tsx, plus warnings). Focused ESLint on this branch's touched files passes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: Multiple PR contributors payout distribution

1 participant