Solana svm#6
Merged
Merged
Conversation
Removes the analyze and context CLI subcommands, the /api/contract REST handlers, the Solidity Anchor fixtures and the Solidity guide chapters. Serve and Explore now reject any path detected as Solidity with a clear error pointing at the standalone ilold-evm repo.
Drops the 3133-line dual REPL (handle_input, print_result, slicing/trace helpers, Solidity prompt help) and keeps only the Solana code path: handle_solana_input, dispatch_solana, kv parser, scenario sync and prompt. The standalone fmt.rs / interactive.rs modules now have no callers and will be removed next.
Moves header_box and pad_right (the only two helpers still needed) into colors.rs, deletes fmt.rs (FlowTree/SliceResult renderers) and interactive.rs (TUI trace viewer), and removes the now-unused terminal_size/ratatui/crossterm dependencies and the ilold-core path dep from ilold-cli.
Removes the three /api/session/{function,trace,slice} routes (Solidity-only), the dual serve()/start_server() functions that built AppState from .sol paths, and the now-unused std::path::PathBuf import. The Solana entrypoints (serve_solana, start_solana_server) stay as the only public entry points; renaming to plain serve/start_server is deferred to a follow-up to keep this commit focused on the purge.
Drops SolidityState and the dual Backend enum, replaces unwrap_solidity/require_solidity helpers with direct solana() access, rewrites api/session.rs to dispatch every command through the Solana pipeline (1074 to 538 LOC), purges Solidity routes (/api/session/{state,timeline,sequence,trace/contract/func,slice/func/var,function/contract/func} and /api/contract/*), trims the WS handler to broadcast-only (search.rs deleted), and removes six Solidity integration tests. SourceSpan moves inline next to the Solana source endpoint.
Switches the kind prop default from solidity to solana in the 8 Svelte files that branch on it, rewrites the root +page.svelte to render only Solana programs (drops the Solidity contract grid), and pins the contract page detector to solana. The dead Solidity branches inside individual components stay until a follow-up cleanup; cross-check pipeline stays green.
Drops the ilold-core crate from the workspace and its 7 path dependents (ilold-web, ilold-render, plus solar 0.1.8 from workspace.dependencies). AccessLevel imports in ilold-render move from ilold-core to its real home in ilold-session-core. README, SUMMARY, introduction and getting-started are rewritten Solana-only, and images/diagram_solidity.png is removed. cargo test --workspace --release stays green.
Removes every file under crates/ilold-core/ now that the workspace member is gone and no live import references it.
… pages Rewrites the six primary pages (architecture, overview, api-endpoints, websocket, limitations, cross-cutting) as Solana-only and removes every Solidity link or comparative aside from the solana/* and roadmap/solana pages. mdbook build is clean; no broken cross-references remain.
Fixes FunctionSidebar.svelte::allRows which used the legacy $derived<T>(callback) form (Svelte 5 incompatible) and was called with parens, causing a runtime crash whenever the sidebar rendered. Switches to $derived.by, drops the now-impossible Solidity branch in contract/[name]/+page.svelte onMount, prunes contracts and relationships from the ProjectMap TS interface (the backend no longer returns them), and removes the orphaned petgraph workspace dependency.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.