Skip to content

feat: add transaction context crate - #19

Open
bmuddha wants to merge 1 commit into
transaction-viewfrom
transaction-context
Open

feat: add transaction context crate#19
bmuddha wants to merge 1 commit into
transaction-viewfrom
transaction-context

Conversation

@bmuddha

@bmuddha bmuddha commented May 28, 2026

Copy link
Copy Markdown
Collaborator

What changed

Customized the imported solana-transaction-context baseline for the engine's
account representation and patched the crate into the workspace.

Why

VM handlers need to remap account data on writes through a mapped region instead
of copying through a serialized input buffer.

Closes #10.

Impact

  • Reshapes TransactionContext, InstructionContext, and
    TransactionAccounts around AccountRef and AccountRefMut views.
  • Keeps accounts in UnsafeCell behind explicit borrow counters so borrows stay
    local to the context.
  • Returns keyed accounts, return data, touched-account count, and resize deltas
    through ExecutionRecord.

Reviewer notes

Borrow counters are released on drop; outstanding references must be gone before
a context is deconstructed. The mapped-account divergences are documented in
solana/README.md.

Follow-up

program-runtime and svm consume this customized context upstack.

@coderabbitai

coderabbitai Bot commented May 28, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Free

Run ID: 92fa47c9-1c28-4bd7-b07a-f462ef424103

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@bmuddha
bmuddha force-pushed the transaction-context branch from c4029f9 to e4a023f Compare May 29, 2026 09:08
@bmuddha
bmuddha force-pushed the transaction-context branch 2 times, most recently from 27ca11d to ff1197d Compare June 1, 2026 15:29
@bmuddha
bmuddha force-pushed the solana-account branch 2 times, most recently from 9f791a8 to aab5a4e Compare June 2, 2026 18:26
@bmuddha
bmuddha force-pushed the transaction-context branch 2 times, most recently from 7f5babf to b1bb9fd Compare June 5, 2026 12:10
@bmuddha
bmuddha force-pushed the transaction-context branch from b1bb9fd to f7c2bac Compare June 16, 2026 10:08
@bmuddha
bmuddha force-pushed the transaction-context branch from f7c2bac to 881f449 Compare June 22, 2026 19:14
@bmuddha
bmuddha force-pushed the solana-account branch 2 times, most recently from 2bad847 to a7a8ce1 Compare June 23, 2026 14:17
@bmuddha
bmuddha force-pushed the transaction-context branch from 881f449 to 59ad79e Compare June 23, 2026 14:17
@bmuddha
bmuddha force-pushed the transaction-context branch from 16b19e7 to 6e052c1 Compare July 15, 2026 19:44
@bmuddha
bmuddha force-pushed the transaction-context branch 3 times, most recently from 801df8d to f8d3173 Compare July 17, 2026 14:55
@bmuddha
bmuddha force-pushed the transaction-context branch from f8d3173 to 0ef24b5 Compare July 21, 2026 19:46
@bmuddha
bmuddha force-pushed the transaction-context branch from 0ef24b5 to a1c175f Compare July 26, 2026 13:14
@bmuddha
bmuddha force-pushed the transaction-context branch from a1c175f to cd9fd68 Compare July 27, 2026 13:19
@bmuddha
bmuddha force-pushed the solana-account branch 2 times, most recently from 3b5ca59 to a87302d Compare July 27, 2026 13:32
@bmuddha
bmuddha force-pushed the transaction-context branch from cd9fd68 to 80378f1 Compare July 27, 2026 13:32
@bmuddha
bmuddha force-pushed the transaction-context branch from 80378f1 to e041528 Compare July 27, 2026 14:17
@bmuddha
bmuddha force-pushed the transaction-context branch from e041528 to e7b59e5 Compare July 28, 2026 14:25
@bmuddha
bmuddha force-pushed the transaction-context branch from e7b59e5 to f78da53 Compare July 28, 2026 20:16
@bmuddha
bmuddha changed the base branch from solana-account to graphite-base/19 July 30, 2026 17:51
@bmuddha
bmuddha force-pushed the graphite-base/19 branch from 700128f to de564a1 Compare July 30, 2026 17:51
@bmuddha
bmuddha force-pushed the transaction-context branch from f78da53 to 7125534 Compare July 30, 2026 17:51
@bmuddha
bmuddha changed the base branch from graphite-base/19 to transaction-view July 30, 2026 17:51
@bmuddha
bmuddha force-pushed the transaction-view branch from de564a1 to acf36a7 Compare July 30, 2026 18:20
@bmuddha
bmuddha force-pushed the transaction-context branch from 7125534 to 2e0b538 Compare July 30, 2026 18:20
@bmuddha
bmuddha force-pushed the transaction-view branch from acf36a7 to e7813be Compare July 31, 2026 09:55
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