Skip to content

Add SolanaErrors for MessagePacker#647

Merged
lorisleiva merged 1 commit intomainfrom
07-09-add_solanaerrors_for_messagepacker_
Jul 15, 2025
Merged

Add SolanaErrors for MessagePacker#647
lorisleiva merged 1 commit intomainfrom
07-09-add_solanaerrors_for_messagepacker_

Conversation

@lorisleiva
Copy link
Member

@lorisleiva lorisleiva commented Jul 9, 2025

This PR adds two new SolanaError codes that can be used by MessagePackers to indicate that:

  • The provided transaction message cannot be filled with any instructions.
  • The message packer is already done and the packMessageToCapacity function should no longer be called.

@changeset-bot
Copy link

changeset-bot bot commented Jul 9, 2025

⚠️ No Changeset found

Latest commit: e39dba7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@bundlemon
Copy link

bundlemon bot commented Jul 9, 2025

BundleMon

Files updated (7)
Status Path Size Limits
instruction-plans/dist/index.native.mjs
1.18KB (+240B +24.84%) -
instruction-plans/dist/index.node.mjs
1.18KB (+240B +24.87%) -
instruction-plans/dist/index.browser.mjs
1.18KB (+239B +24.69%) -
errors/dist/index.browser.mjs
15.01KB (+202B +1.33%) -
errors/dist/index.native.mjs
15.01KB (+202B +1.33%) -
errors/dist/index.node.mjs
15.03KB (+202B +1.33%) -
@solana/kit production bundle
kit/dist/index.production.min.js
34.59KB (+45B +0.13%) -
Unchanged files (123)
Status Path Size Limits
rpc-graphql/dist/index.browser.mjs
18.82KB -
rpc-graphql/dist/index.native.mjs
18.82KB -
rpc-graphql/dist/index.node.mjs
18.82KB -
transaction-messages/dist/index.browser.mjs
7.37KB -
transaction-messages/dist/index.native.mjs
7.36KB -
transaction-messages/dist/index.node.mjs
7.36KB -
codecs-data-structures/dist/index.native.mjs
4.77KB -
codecs-data-structures/dist/index.browser.mjs
4.77KB -
codecs-data-structures/dist/index.node.mjs
4.77KB -
webcrypto-ed25519-polyfill/dist/index.node.mj
s
3.57KB -
webcrypto-ed25519-polyfill/dist/index.browser
.mjs
3.56KB -
webcrypto-ed25519-polyfill/dist/index.native.
mjs
3.54KB -
rpc-subscriptions/dist/index.browser.mjs
3.38KB -
rpc-subscriptions/dist/index.node.mjs
3.34KB -
rpc-subscriptions/dist/index.native.mjs
3.31KB -
codecs-core/dist/index.browser.mjs
3.3KB -
codecs-core/dist/index.native.mjs
3.3KB -
codecs-core/dist/index.node.mjs
3.3KB -
addresses/dist/index.browser.mjs
2.93KB -
rpc-transformers/dist/index.browser.mjs
2.93KB -
rpc-transformers/dist/index.native.mjs
2.93KB -
addresses/dist/index.native.mjs
2.93KB -
addresses/dist/index.node.mjs
2.93KB -
rpc-transformers/dist/index.node.mjs
2.93KB -
signers/dist/index.browser.mjs
2.63KB -
signers/dist/index.native.mjs
2.63KB -
signers/dist/index.node.mjs
2.62KB -
codecs-strings/dist/index.browser.mjs
2.53KB -
kit/dist/index.browser.mjs
2.53KB -
kit/dist/index.native.mjs
2.53KB -
kit/dist/index.node.mjs
2.53KB -
codecs-strings/dist/index.node.mjs
2.48KB -
codecs-strings/dist/index.native.mjs
2.45KB -
transaction-confirmation/dist/index.node.mjs
2.39KB -
sysvars/dist/index.browser.mjs
2.35KB -
sysvars/dist/index.native.mjs
2.34KB -
transaction-confirmation/dist/index.native.mj
s
2.34KB -
sysvars/dist/index.node.mjs
2.34KB -
transaction-confirmation/dist/index.browser.m
js
2.34KB -
transactions/dist/index.browser.mjs
2.3KB -
transactions/dist/index.native.mjs
2.3KB -
transactions/dist/index.node.mjs
2.29KB -
rpc-subscriptions-spec/dist/index.node.mjs
2.18KB -
rpc-subscriptions-spec/dist/index.native.mjs
2.13KB -
rpc-subscriptions-spec/dist/index.browser.mjs
2.13KB -
keys/dist/index.browser.mjs
2.02KB -
keys/dist/index.native.mjs
2.02KB -
keys/dist/index.node.mjs
2.02KB -
codecs-numbers/dist/index.native.mjs
2.01KB -
codecs-numbers/dist/index.browser.mjs
2.01KB -
codecs-numbers/dist/index.node.mjs
2.01KB -
react/dist/index.native.mjs
1.99KB -
react/dist/index.browser.mjs
1.99KB -
react/dist/index.node.mjs
1.99KB -
rpc/dist/index.node.mjs
1.95KB -
rpc-transport-http/dist/index.browser.mjs
1.91KB -
rpc-transport-http/dist/index.native.mjs
1.91KB -
rpc/dist/index.native.mjs
1.8KB -
subscribable/dist/index.node.mjs
1.8KB -
rpc/dist/index.browser.mjs
1.8KB -
subscribable/dist/index.native.mjs
1.75KB -
subscribable/dist/index.browser.mjs
1.74KB -
rpc-transport-http/dist/index.node.mjs
1.73KB -
rpc-types/dist/index.browser.mjs
1.53KB -
rpc-types/dist/index.native.mjs
1.53KB -
rpc-types/dist/index.node.mjs
1.53KB -
rpc-subscriptions-channel-websocket/dist/inde
x.node.mjs
1.33KB -
rpc-subscriptions-channel-websocket/dist/inde
x.native.mjs
1.27KB -
rpc-subscriptions-channel-websocket/dist/inde
x.browser.mjs
1.26KB -
options/dist/index.browser.mjs
1.18KB -
options/dist/index.native.mjs
1.18KB -
options/dist/index.node.mjs
1.17KB -
accounts/dist/index.browser.mjs
1.13KB -
accounts/dist/index.native.mjs
1.12KB -
accounts/dist/index.node.mjs
1.12KB -
compat/dist/index.browser.mjs
971B -
compat/dist/index.native.mjs
970B -
compat/dist/index.node.mjs
968B -
rpc-spec-types/dist/index.browser.mjs
964B -
rpc-api/dist/index.browser.mjs
963B -
rpc-api/dist/index.native.mjs
962B -
rpc-spec-types/dist/index.native.mjs
962B -
rpc-api/dist/index.node.mjs
961B -
rpc-spec-types/dist/index.node.mjs
961B -
rpc-subscriptions-api/dist/index.native.mjs
870B -
rpc-subscriptions-api/dist/index.node.mjs
869B -
rpc-subscriptions-api/dist/index.browser.mjs
868B -
rpc-spec/dist/index.browser.mjs
852B -
rpc-spec/dist/index.native.mjs
851B -
rpc-spec/dist/index.node.mjs
850B -
promises/dist/index.browser.mjs
799B -
promises/dist/index.native.mjs
798B -
promises/dist/index.node.mjs
797B -
assertions/dist/index.browser.mjs
783B -
instructions/dist/index.browser.mjs
769B -
instructions/dist/index.native.mjs
768B -
instructions/dist/index.node.mjs
767B -
fast-stable-stringify/dist/index.browser.mjs
726B -
fast-stable-stringify/dist/index.native.mjs
725B -
assertions/dist/index.native.mjs
724B -
fast-stable-stringify/dist/index.node.mjs
724B -
assertions/dist/index.node.mjs
723B -
programs/dist/index.browser.mjs
329B -
programs/dist/index.native.mjs
327B -
programs/dist/index.node.mjs
325B -
event-target-impl/dist/index.node.mjs
230B -
functional/dist/index.browser.mjs
154B -
functional/dist/index.native.mjs
152B -
text-encoding-impl/dist/index.native.mjs
152B -
functional/dist/index.node.mjs
151B -
codecs/dist/index.browser.mjs
137B -
codecs/dist/index.native.mjs
136B -
codecs/dist/index.node.mjs
134B -
event-target-impl/dist/index.browser.mjs
133B -
ws-impl/dist/index.node.mjs
131B -
text-encoding-impl/dist/index.browser.mjs
122B -
text-encoding-impl/dist/index.node.mjs
119B -
ws-impl/dist/index.browser.mjs
113B -
crypto-impl/dist/index.node.mjs
111B -
crypto-impl/dist/index.browser.mjs
109B -
rpc-parsed-types/dist/index.browser.mjs
66B -
rpc-parsed-types/dist/index.native.mjs
65B -
rpc-parsed-types/dist/index.node.mjs
63B -

Total files change +1.34KB +0.38%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@lorisleiva lorisleiva marked this pull request as ready for review July 9, 2025 14:05
@github-actions
Copy link
Contributor

github-actions bot commented Jul 9, 2025

Documentation Preview: https://kit-docs-n2v8l4vo4-anza-tech.vercel.app

Copy link
Contributor

@steveluscher steveluscher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added via Giphy

Copy link
Contributor

@steveluscher steveluscher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

…actually.

@lorisleiva lorisleiva force-pushed the 07-09-add_solanaerrors_for_messagepacker_ branch from 57cce0d to 257cf55 Compare July 11, 2025 09:25
@lorisleiva lorisleiva force-pushed the 06-05-add_instructionplan_type_and_helpers branch from 624aeab to 30c6d1b Compare July 11, 2025 09:25
@lorisleiva lorisleiva force-pushed the 07-09-add_solanaerrors_for_messagepacker_ branch from 257cf55 to 3c41160 Compare July 11, 2025 16:20
@lorisleiva lorisleiva requested a review from steveluscher July 11, 2025 16:58
Copy link
Contributor

@steveluscher steveluscher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added via Giphy

@lorisleiva lorisleiva force-pushed the 07-09-add_solanaerrors_for_messagepacker_ branch from 3c41160 to 7402dc7 Compare July 14, 2025 12:55
@lorisleiva lorisleiva force-pushed the 06-05-add_instructionplan_type_and_helpers branch 2 times, most recently from 1db2aad to e227adf Compare July 14, 2025 13:50
@lorisleiva lorisleiva force-pushed the 07-09-add_solanaerrors_for_messagepacker_ branch 2 times, most recently from 8102db4 to b124099 Compare July 14, 2025 13:54
Copy link
Member Author

lorisleiva commented Jul 15, 2025

Merge activity

  • Jul 15, 11:09 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jul 15, 11:15 AM UTC: Graphite rebased this pull request as part of a merge.
  • Jul 15, 11:18 AM UTC: @lorisleiva merged this pull request with Graphite.

@lorisleiva lorisleiva changed the base branch from 06-05-add_instructionplan_type_and_helpers to graphite-base/647 July 15, 2025 11:11
@lorisleiva lorisleiva changed the base branch from graphite-base/647 to main July 15, 2025 11:14
@lorisleiva lorisleiva force-pushed the 07-09-add_solanaerrors_for_messagepacker_ branch from b124099 to e39dba7 Compare July 15, 2025 11:15
@socket-security
Copy link

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​changesets/​changelog-github@​0.5.11001006583100
Added@​graphql-tools/​schema@​10.0.231001007099100
Added@​eslint/​js@​9.31.01001009096100
Added@​eslint/​json@​0.13.010010010091100
Added@​changesets/​cli@​2.29.59710010092100
Added@​inquirer/​core@​10.1.1410010010094100

View full report

@lorisleiva lorisleiva merged commit 73c9757 into main Jul 15, 2025
13 checks passed
@lorisleiva lorisleiva deleted the 07-09-add_solanaerrors_for_messagepacker_ branch July 15, 2025 11:18
@github-actions
Copy link
Contributor

Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants