nspcc-dev/neo-go

AddBreakPoint to apply to all context with the same `prog`

Open

#3,675 opened on Nov 12, 2024

View on GitHubContributor guide
 (1 comment) (0 reactions) (0 assignees)Go (108 forks)auto 404
I2S4U3clienhancementhelp wantedvm

Repository metrics

Stars
 (132 stars)
PR merge metrics
 (PR metrics pending)

Description

Is your feature request related to a problem? Please describe.

I'm always frustrated when breakpoints set for a given prog (=smart contract) don't hit because they are registered for the current context and not globally for the smart contract.

Describe the solution you'd like

I'd like an AddBreakPoint on VM which allows to set a global breakpoint for a given contract (similar to the C# one).

Describe alternatives you've considered

Create AddBreakpoint / RemoveBreakPoint on Context.

Additional context

Imagine an IDE with NEO smart contract debugger support build on neo-go. The IDE tells what addresses to put a break point on, then runs the debugger waiting for a break point to hit. Currently it won't hit any break points except for those in the entry function.

Contributor guide

Research direction
Study how breakpoints are currently managed in neo go's VM and context. Look at the C# reference implementation in neo vm for global breakpoints. Propose changes to the AddBreakPoint method to allow setting breakpoints per contract ID rather than per context.
Tech stack
go
Domain
blockchain
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Go
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60