Skip to content

fix(gas-station): allow init_stake_deposit discriminator — GHB-188#93

Merged
Gastonfoncea merged 1 commit into
mainfrom
gastonfoncea09/ghb-188-followup-stake-discriminator
May 17, 2026
Merged

fix(gas-station): allow init_stake_deposit discriminator — GHB-188#93
Gastonfoncea merged 1 commit into
mainfrom
gastonfoncea09/ghb-188-followup-stake-discriminator

Conversation

@Gastonfoncea
Copy link
Copy Markdown
Collaborator

Summary

  • Adds init_stake_deposit discriminator (d5cbd1d000e6846a) to the gas-station allowlist so /app/stake can actually submit the MCP-activation tx.
  • Bumps the sanity-test allowlist size assertion 4 → 5.

Why

The frontend at /app/stake builds an init_stake_deposit Anchor ix and routes it through the gas station (per the existing sponsor pattern for bounty-flow ixs). The validator's allowlist was the original GHB-176/180 set of 4 bounty-flow discriminators; the GHB-188 stake ix was never added, so prod rejects every stake with:

Transaction rejected by gas station:
disallowed_discriminator: escrow ix discriminator d5cbd1d000e6846a not in allowlist.

Reproduced live by Gaston during the post-merge smoke test.

Test plan

  • pnpm --filter @ghbounty/shared test — 54/54 pass locally (existing 49 + updated sanity test).
  • CI green on this PR.
  • Vercel preview deploy of gh-bounty-frontend succeeds.
  • Once merged, smoke-test the stake flow against the prod preview to confirm Activate MCP mints + flips mcp_status='active'.

🤖 Generated with Claude Code

The MCP-activation stake tx built by /app/stake (init_stake_deposit, Anchor
discriminator d5cbd1d000e6846a) was rejected by the gas-station validator
because the allowlist only contained the four bounty-flow ixs from
GHB-176/180. User-facing symptom in the browser:

  "Transaction rejected by gas station:
   disallowed_discriminator: escrow ix discriminator
   d5cbd1d000e6846a not in allowlist."

Add the discriminator (sourced from the IDL bytes
[213,203,209,208,0,230,132,106]) to ALLOWED_DISCRIMINATORS_HEX and bump
the sanity-test allowlist size assertion 4 → 5. No other validator rule
changes — init_stake_deposit fits the existing single-escrow-ix shape
(no bundled topup, no review fee), so the rest of the pipeline already
covers it.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Gastonfoncea Gastonfoncea requested a review from tomazzi14 as a code owner May 17, 2026 17:48
@vercel
Copy link
Copy Markdown

vercel Bot commented May 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
gh-bounty-frontend Ready Ready Preview, Comment May 17, 2026 5:50pm
ghbounty-mcp Ready Ready Preview, Comment May 17, 2026 5:50pm

@Gastonfoncea Gastonfoncea merged commit 49a087f into main May 17, 2026
5 checks passed
@Gastonfoncea Gastonfoncea deleted the gastonfoncea09/ghb-188-followup-stake-discriminator branch May 17, 2026 20:18
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.

1 participant