[pull] main from aptos-labs:main#630
Merged
Merged
Conversation
* [move-flow] Add deps for aptos move replay integration * [move-flow] Scaffold replay_transaction tool module * [move-flow] Add network parser for replay tool * [move-flow] Define replay tool params and response types * [move-flow] Update Cargo.lock for added deps * [move-flow] Add abort-location formatter helper * [move-flow] Add MoveAbort details extractor * [move-flow] Replace wildcard match arm in abort_details_from * [move-flow] Add ExecutionFailure details extractor * [move-flow] Add transaction-status classifier * [move-flow] Add param validation helpers for replay tool * [move-flow] Add debugger factory helper * [move-flow] Add local-package overrides builder * [move-flow] Log source-mapping failures in local overrides * [move-flow] Wire up replay_transaction handler skeleton * [move-flow] Implement full replay execution path Replace the placeholder body of move_replay_transaction_impl with the real flow: fetch the transaction, reject non-user variants, optionally compile local overrides, run the VM in spawn_blocking, materialize the output, and build the JSON response. Mirrors aptos-move/cli/src/commands.rs::Replay. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * [move-flow] Add param-validation integration tests for replay tool * [move-flow] Document replay_transaction in CLAUDE.md * [move-flow] Update list_tools baseline for replay_transaction Regenerate baseline to include the new move_replay_transaction tool metadata exposed via the MCP list_tools endpoint. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * [move-flow] Apply cargo fmt Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> --------- Co-authored-by: zwxxb <zwxxb@users.noreply.github.com> Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
* [mono-move] Design doc for error system * Address feedback * Renames * Feedback * Rename * More feedback * Add note
The calibration script overwrites the TSV every time it runs, so it opens a PR for sub-percent run-to-run drift that no perf gate would have caught. PR #19646 was the latest example: an unrelated rust-setup failure triggered the calibration job, the script ran on healthy data, and produced a PR shifting every row by 0.1-0.5%. Now the script applies the same band check that single_node_performance.py uses to fail/pass a test, and only rewrites the TSV when at least one production row's new median falls outside its existing band. Move-e2e gets the same treatment using the band from aptos-move/e2e-benchmark/src/main.rs. Non-blocking executor types (NativeVM, AptosVMSpeculative, etc.) are excluded from the gate but still refreshed when a write happens for another reason. The band formula and NON_BLOCKING_EXECUTOR_TYPES live in single_node_performance_calibration.py; single_node_performance.py imports them so the trigger and the perf gate share one source of truth. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * [vm] Type formula to use checked access (#19798) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Co-authored-by: Calin Tataru <calintat@gmail.com>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )