microsoft / sarif-vscode-extension

SARIF Microsoft Visual Studio Code extension
MIT License
111 stars 50 forks source link

Analysis Steps improvements and other improvements. #445

Closed jeffersonking closed 2 years ago

jeffersonking commented 2 years ago

Note: I recommend reviewing by commit rather than the cumulatively.

The largest change is with Analysis Steps (Thread Flows). Previously flows were constrained to a single document. Now if you select a multi-document flow, and happen to have several documents open, you will see the decorations simultaneously light up. Indentation was also added to the Analysis Steps list to help visualize stack depth. Especially useful in long flows.

The next largest change better leveraging base URIs. This should offer an improved experience (less "Locate file..." dialogs) for one of our most common cases: where the Log contains base URIs and the user has a corresponding workspace open.

A few more changes are incoming, but I wanted to get a head start on the review process.