Skip to content

chore(solana): resync devnet IDLs and regenerate clients - #43

Open
ariessa wants to merge 3 commits into
mainfrom
test/devnet-e2e
Open

chore(solana): resync devnet IDLs and regenerate clients#43
ariessa wants to merge 3 commits into
mainfrom
test/devnet-e2e

Conversation

@ariessa

@ariessa ariessa commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Note

High Risk
Large breaking alignment with on-chain layout and transaction account sets across create, fund, complete, reject, and multi-hook flows; incorrect PDA or vault wiring would fail or mis-route funds on mainnet/devnet.

Overview
Resyncs the Solana ACP SDK and generated Codama clients to the updated on-chain program: job identity, escrow vaults, global payment mint, and rent routing.

Job model: createJob no longer reads acp_state.job_counter; it picks a random per-client seed, derives the job PDA from ["job", client, seed], and passes seed in the instruction. resolveJobPda is public and cache keys include the client. Hook/router/subscription PDAs are seeded from the job account address instead of a numeric job id. getJobIdFromTxHash still returns the bigint seed by loading the job account after JobCreated; getJob maps id to seed and best-effort loads description from the creation tx logs.

Budget & vault: Per-job budgetMint is gone—setBudget/fund use acp_state.payment_token. The escrow vault is a program-derived token account created in fund() (client ATA create for the vault is dropped). Mint for funded paths comes from fetchVaultMint on that PDA.

Lifecycle instructions: complete/reject/claimRefund (and hook paths) pass sponsor from acp_state for reclaimed native rent instead of platformTreasury. Reject paths use sync getRejectInstruction so unfunded jobs omit vault correctly. Subscription completion no longer requires providerSigner (F-118). Router fund and submit always use sponsored ALT compression to stay under tx size limits with Alchemy prefunding.

acpAgent Solana paths resolve jobPda before fetchProposedTerms and hookRouterPda.

Reviewed by Cursor Bugbot for commit e6209d4. Bugbot is set up for automated code reviews on this repo. Configure here.

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