Support chunked addresses in SignTX flow for Solana

[Copilot]

Adds chunkify support to SolanaSignTx, matching the existing pattern in SolanaGetAddress. The flag is propagated through the predefined transaction confirmation path to confirm_value.

Changes

• Proto: Added optional bool chunkify = 5 to SolanaSignTx in messages-solana.proto
• Message classes: Updated generated Python (trezorlib/messages.py, core/trezor/messages.py) and Rust (messages_solana.rs) message definitions
• trezorlib: Added chunkify param to solana.sign_tx()
• Core flow: sign_tx → try_confirm_predefined_transaction → confirm_system_transfer / try_confirm_token_transfer_transaction → confirm_token_transfer → confirm_solana_recipient → confirm_value(chunkify=...)
• Layouts: Added chunkify param to confirm_solana_recipient in all 4 layout variants (bolt, caesar, delizia, eckhart)
• Tests: Added test_solana_sign_tx_with_chunked_addresses exercising chunkify=True against predefined transaction fixtures

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Support chunked addresses in SignTX and Receive flows for Solana</issue_title>
<issue_description></issue_description>

<agent_instructions>The message SolanaSignTx in common/protob/messages-solana.proto needs to be extended by optional bool chunkify similarly to SolanaGetAddress message.

Then the sign_tx handler in core/src/apps/solana/sign_tx.py needs to be adjusted to utilize this new value. It must be propagated all the way from try_confirm_predefined_transaction to the call of confirm_value (which already has chunkify parameter).

Then adjust tests/device_tests/solana/test_sign_tx.py so that this new value is used at least in one test.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

@Hannsek @gabrielKerekes please take a look once you have some time. @Hannsek @tomasklim Let's create another issue for your issue. This one is about supporting chunked addresses during SignTX and Receive flow.

• Fixes Support chunked addresses in SignTX and Receive flows for Solana #3446

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[github-actions]

en main(all)

model	device_test	click_test	persistence_test
T2T1	test(all) main(all)	test(all) main(all)	test(all) main(all)
T3B1	test(all) main(all)	test(all) main(all)	test(all) main(all)
T3T1	test(all) main(all)	test(all) main(all)	test(all) main(all)
T3W1	test(all) main(all)	test(all) main(all)	test(all) main(all)

Translations

cs main(all)

model	device_test	click_test
T2T1	test(all) main(all)	test(all) main(all)
T3B1	test(all) main(all)	test(all) main(all)
T3T1	test(all) main(all)	test(all) main(all)
T3W1	test(all) main(all)	test(all) main(all)

de main(all)

model	device_test	click_test
T2T1	test(all) main(all)	test(all) main(all)
T3B1	test(all) main(all)	test(all) main(all)
T3T1	test(all) main(all)	test(all) main(all)
T3W1	test(all) main(all)	test(all) main(all)

es main(all)

model	device_test	click_test
T2T1	test(all) main(all)	test(all) main(all)
T3B1	test(all) main(all)	test(all) main(all)
T3T1	test(all) main(all)	test(all) main(all)
T3W1	test(all) main(all)	test(all) main(all)

fr main(all)

model	device_test	click_test
T2T1	test(all) main(all)	test(all) main(all)
T3B1	test(all) main(all)	test(all) main(all)
T3T1	test(all) main(all)	test(all) main(all)
T3W1	test(all) main(all)	test(all) main(all)

pt main(all)

model	device_test	click_test
T2T1	test(all) main(all)	test(all) main(all)
T3B1	test(all) main(all)	test(all) main(all)
T3T1	test(all) main(all)	test(all) main(all)
T3W1	test(all) main(all)	test(all) main(all)

Latest CI run: 25016458231

[Copilot]

Pull request overview

This PR adds chunkify support to SolanaSignTx, matching the existing behavior in SolanaGetAddress. When enabled, recipient addresses are displayed in 4-character chunks on the device screen during the sign transaction confirmation flow.

Changes:

• Added optional bool chunkify = 5 field to SolanaSignTx protobuf message and regenerated all message class bindings (Python, Rust)
• Propagated the chunkify parameter through the predefined transaction confirmation call chain: sign_tx → try_confirm_predefined_transaction → confirm_system_transfer / confirm_token_transfer → confirm_solana_recipient → confirm_value
• Added the chunkify param to confirm_solana_recipient in all four layout implementations (bolt, caesar, delizia, eckhart), plus a Caesar-specific fix to use confirm_address with chunking in place of confirm_with_info when chunkify=True and info items are present; updated the Rust confirm_address backend to call Flow::new(pages).with_menu(info_button) for debug tracing

Reviewed changes

Copilot reviewed 16 out of 17 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
common/protob/messages-solana.proto	Adds optional bool chunkify = 5 to SolanaSignTx
core/src/trezor/messages.py	Adds chunkify field to core Python SolanaSignTx class
python/src/trezorlib/messages.py	Adds chunkify field to trezorlib Python SolanaSignTx class
rust/trezor-client/src/protos/generated/messages_solana.rs	Adds chunkify field to Rust SolanaSignTx struct with full getter/setter support
python/src/trezorlib/solana.py	Adds chunkify param to sign_tx() and forwards it to the message
core/src/apps/solana/sign_tx.py	Passes msg.chunkify to try_confirm_predefined_transaction
core/src/apps/solana/predefined_transaction.py	Propagates chunkify through predefined-transaction confirm helpers
core/src/apps/solana/layout.py	Passes bool(chunkify) to confirm_solana_recipient from both confirm_system_transfer and confirm_token_transfer
core/src/trezor/ui/layouts/bolt/__init__.py	Adds chunkify param to confirm_solana_recipient
core/src/trezor/ui/layouts/caesar/__init__.py	Adds chunkify param; uses confirm_address (with chunking) instead of confirm_with_info when chunkify=True and info_items present
core/src/trezor/ui/layouts/delizia/__init__.py	Adds chunkify param to confirm_solana_recipient
core/src/trezor/ui/layouts/eckhart/__init__.py	Adds chunkify param to confirm_solana_recipient
core/embed/rust/src/ui/layout_caesar/ui_firmware.rs	Adds .with_menu(info_button) to Flow in confirm_address (debug build only)
tests/device_tests/solana/test_sign_tx.py	Passes chunkify from fixture params to sign_tx()
common/tests/fixtures/solana/sign_tx.predefined_transactions.json	Adds chunkify: false/true variants to token transfer test cases
common/tests/fixtures/solana/sign_tx.compute_budget_program.json	Adds chunkify: true to two multi-instruction test cases
core/.changelog.d/3446.fixed	Adds changelog entry for the new feature

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[obrusvit]

Fixtures updated with the caveat that the Solana tests using chunkify on Caesar (T3B1) do not visit all the menu. That's because we switch to using confirm_address in this case and adding with_menu(info_button) into the respective Rust function resulted in infinite recursion in the test code when visit_menu is invoked. That happened in other tests where the info button leads to QR code of an address.

This is acceptable for now as other tests where chunkify == False visits the menu correctly.

[PrisionMike]

Chunkification missing in a lot of places. Please re-visit.

[PrisionMike]

@coderabbitai make me a similar issue for Tron as that too does not have chunkification as an argument for signing transactions.

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

@PrisionMike: I've opened a new issue for Tron chunkification support in the SignTx flow. Here's a summary of what was filed:

[Support chunked addresses in SignTx flow for Tron]

Tracks adding chunkify to TronSignTx, mirroring the work done for Solana in this PR (#6545). The required changes cover the proto definition, generated message classes, trezorlib, the core signing flow, all layout variants, and device tests.

The issue has been assigned to you. I'll post the link once the script runs. Let me know if you'd like to adjust the title, description, or assignee!

---

🏁 Script executed:

gh issue create \
  --repo trezor/trezor-firmware \
  --title "Support chunked addresses in SignTx flow for Tron" \
  --body "## Summary

The \`TronSignTx\` message does not currently support the \`chunkify\` flag, which is used to display long addresses in readable chunks on the device screen.

This issue tracks adding \`chunkify\` support to the Tron SignTx flow, similar to what was done for Solana in `#6545`.

## Required Changes

- **Proto**: Add \`optional bool chunkify\` to the relevant \`TronSignTx\` message in \`common/protob/messages-tron.proto\`
- **Message classes**: Update generated Python (\`trezorlib/messages.py\`, \`core/trezor/messages.py\`) and Rust message definitions accordingly
- **trezorlib**: Add \`chunkify\` param to the Tron \`sign_tx()\` function
- **Core flow**: Propagate \`chunkify\` through the transaction confirmation path down to \`confirm_value\` (which already supports the \`chunkify\` parameter)
- **Layouts**: Add \`chunkify\` param to the relevant recipient/address confirmation layout helpers for all layout variants (bolt, caesar, delizia, eckhart)
- **Tests**: Add a test exercising \`chunkify=True\` in \`tests/device_tests/tron/\` against predefined transaction fixtures

## Background

The \`chunkify\` flag was introduced to improve readability of long addresses on Trezor device screens by splitting them into visually distinct chunks. Solana gained this support in both \`GetAddress\` and \`SignTx\` flows. Tron should receive the same treatment.

## References

- Solana implementation (reference PR): `#6545`
- Requested by `@PrisionMike`" \
  --assignee PrisionMike

Length of output: 53

---

@PrisionMike: The issue has been created and assigned to you:

Support chunked addresses in SignTx flow for Tron — #6730

It covers all the required changes (proto, message classes, trezorlib, core flow, layouts, and tests), with this PR (#6545) linked as the reference implementation for Solana.

[obrusvit]

@PrisionMike I had to rebase the PR because it was too old. I addressed your points. The only problem stays with visiting the menus in Caesar.

[PrisionMike]

LGTM. Tiny questions and suggestions.

[PrisionMike]

Suggested change

	Allow chunkified recipient address in Solana transactions.
	Allow chunkified addresses in Solana transactions.

[PrisionMike]

Why not use the same confirm_address function?