Skip to content

Replace archived Solana layouts with maintained first-party codecs - #57

Merged
SYMBaiEX merged 2 commits into
mainfrom
sym/pinned-eliza-source
Nov 1, 2026
Merged

SYMBaiEX merged 2 commits into
mainfrom
sym/pinned-eliza-source

Conversation

@SYMBaiEX

@SYMBaiEX SYMBaiEX commented Nov 1, 2026

Copy link
Copy Markdown
Owner

Replace the archived Solana layout utility with the approved first-party @doolittle/solana-layouts workspace. Real wallet/SPL consumers resolve its legacy API through a root-owned live link. Maintained Noble converters reject negative and overflowing unsigned amounts before changing the destination; public keys, booleans and exact 18-place decimals remain supported. Remove the old layout patch, bigint-buffer exclusion and backport package extension.

This retires one patch: four SDK patches remain. Pinned upstream source builds were evaluated and documented, but experimental archives are not installed or committed. The source migration is not complete.

Validation:

  • Full local nub run check:release passed: lint/types, 258 boundary tests, audits/builds, seven integration tests, six native desktop journeys and clean-HEAD packaged desktop smoke.
  • 13 focused wallet/SDK contracts passed, including actual SPL checked-transfer bytes and all unsigned widths/byte orders.
  • Actual wallet encoding passed in Electron's embedded Node 24.22.0.
  • Real linked-provider CLI coding passed. Desktop's first live attempt failed closed after the model supplied a truncated workspace path; a fresh isolated desktop task passed, with independent edit/test verification.
  • Documentation truth check passed after the documentation commit.

Two focused commits. No live wallet transaction was submitted. GitHub Actions were not used as the release gate.

Copilot AI balanced review requested due to automatic review settings November 1, 2026 20:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@coderabbitai

coderabbitai Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@SYMBaiEX
SYMBaiEX merged commit 408b0f5 into main Nov 1, 2026
4 checks passed
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.

2 participants