fix(solana/spl_token): migrate to shared core::InstructionView, drop data clone

[shahan-khatchadourian-anchorage]

Summary

• Removes the private InstructionView struct from spl_token/mod.rs that clones context.data().to_vec() into a Vec<u8> on every instruction — a per-instruction heap allocation with no architectural justification.
• Switches to the shared crate::core::InstructionView for program ID and account resolution.
• Accesses instruction bytes via context.data() directly (a zero-copy borrow), consistent with all other presets in the codebase.
• Fixes the stale None => "unknown" fallback arm (now "unresolved(oob:N)" matching the canonical form in core::InstructionView).

Closes #380

Test plan

• cargo test -p visualsign-solana — 328 passed, 0 failed, 3 ignored
• make -C src lint — clippy clean with -D warnings
• cargo fmt --all -- --check — clean
• Diff is mechanical: import change, struct deletion, &instruction.data -> context.data() substitutions, and the rustfmt reflow those substitutions imply (the shorter argument lets 17 create_text_field("Raw Data", ...) calls fit on one line)

Note on formatting coverage

The rustfmt reflow above is invisible to CI. presets/mod.rs declares preset modules through include!(concat!(env!("OUT_DIR"), "/generated_presets_mod.rs")), so rustfmt cannot traverse into presets/*/ — cargo fmt --all formats presets/mod.rs and nothing beneath it. The whole preset tree is therefore unformatted as far as the make generated + clean-tree gate is concerned. Worth a follow-up issue; out of scope here.

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR updates the spl_token Solana preset to use the shared crate::core::InstructionView (introduced in #367) and removes the preset’s private InstructionView that cloned instruction bytes per instruction, switching all byte access to a zero-copy borrow via context.data().

Changes:

• Replaces the private spl_token::InstructionView with the shared core::InstructionView for program/account display resolution.
• Eliminates per-instruction heap allocation by removing context.data().to_vec() and decoding directly from context.data().
• Aligns unresolved-account fallback behavior with the canonical core::InstructionView placeholder format.

---

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