vscode-ext icon indicating copy to clipboard operation
vscode-ext copied to clipboard

Lightweight Scribble integration (arm / disarm / etc) under Integrations

Open kevinbluer opened this issue 2 years ago • 4 comments

Following #208, Scribble integration could be a nice additional candidate. High-level thoughts as follows...

  • Check to see if Scribble is installed and prompt to do so if not
  • Include ability to arm / disarm
  • Ability to check properties with MythX / Mythril / other

kevinbluer avatar Aug 16 '22 20:08 kevinbluer

At what level do we want to do this?

Is arm/disarm globally? Or are we wanting to do this specifically per file?

michaeljohnbennett avatar Oct 27 '22 22:10 michaeljohnbennett

I'm assuming when you say globally, it means to act on every contract. If so, I would say, ideally to have both options, on all files and individually. The scribble command itself acts on a single file, so this might be a more important use case.

acuarica avatar Oct 31 '22 11:10 acuarica

we need to know which contracts are armable too? Looking for scribble annotations?

michaeljohnbennett avatar Oct 31 '22 23:10 michaeljohnbennett

In a first pass, I would do something simpler, just an interface for its command, regardless of any annotations in the contract. We could parse Scribble's output to notify the user if any instrumentation actually happened.

In case we want to inform the user beforehand, we'll need some sort of Scribble's annotation parser.

acuarica avatar Nov 01 '22 13:11 acuarica