Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vscode.debug API should provide the current thread/stack frame ids #63943

Closed
gregg-miskelly opened this issue Nov 28, 2018 · 34 comments · Fixed by #212190
Closed

vscode.debug API should provide the current thread/stack frame ids #63943

gregg-miskelly opened this issue Nov 28, 2018 · 34 comments · Fixed by #212190
Assignees
Labels
api api-finalization debug Debug viewlet, configurations, breakpoints, adapter issues feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-testplan
Milestone

Comments

@gregg-miskelly
Copy link
Member

gregg-miskelly commented Nov 28, 2018

Currently the vscode.debug API provides a way to send custom events to debug adapters. But it is missing one fairly important thing - there is no way to know what the current thread id / frame id is. So a debug adapter could maybe guess. But it feels like there should really be a way to query this.

@geajack
Copy link

geajack commented Feb 28, 2022

Hi @hediet, I noticed you self-assigned this recently. I was wondering, is there any kind of plan as to what an eventual API on this might look like? I also wanted to check that this feature request is the same one discussed on a different repo here. Is the idea here to allow access to the ID of the "current" stack frame in the sense of the one that the user has selected in the VS Code debugger view in the sidebar?

@hediet
Copy link
Member

hediet commented Mar 1, 2022

It is unclear if this ever gets implemented, but I have a personal interest in it, thus I assigned it to myself to keep track of it.

@elazarcoh
Copy link

elazarcoh commented Mar 3, 2022

@weinand Will you accept a PR for this?
I have the code here.
I tried to keep it smallest and coherent with existing code as possible.

@weinand
Copy link
Contributor

weinand commented Mar 3, 2022

@elazarcoh In principle we are willing to accept a PR. But since this is extension API it has to go through our extension API review process.
The first thing we need is an API proposal.

@elazarcoh
Copy link

is it something I can start? or it is something that is done only internally

@weinand
Copy link
Contributor

weinand commented Mar 3, 2022

@elazarcoh you can start just here.
Please study some existing API items with the "api-proposal" label to learn how API proposals look like.

@PaDebug
Copy link

PaDebug commented Mar 25, 2022

Hello,
I also need the feature (get the current user selected frame id) in order to update the windows of the debugger extension (especially for multi-core devices). Does this feature make progress? Is there any workaround to get the frame id?

@elazarcoh
Copy link

@PaDebug if you'd like to help, you can write the API proposal. That would be the first step.

@PaDebug
Copy link

PaDebug commented Mar 30, 2022

@elazarcoh What do I need to do to write the API proposal?

I would propose extending the debug namespace by an event which fires when the user selected a different stack frame in the call stack window. For example:

export namespace debug {

/**
 * An event which fires when the user selected a different stack frame in the call stack window.
 */
export const onDidChangeSelectedStackFrame: Event<StackFrame>;

}

The interface StackFrame consists of two properties: frameId and associated threadId. This would provide enough information to update the thread and stack frame in the Debug Adapter via custom request.

export interface StackFrame {

/**
 * Id of the associated thread (DAP id).
 */
readonly threadId: number;

/**
 * Id of the stack frame (DAP id).
 */
readonly frameId: number;

}

What do you think about this?

@elazarcoh
Copy link

@elazarcoh What do I need to do to write the API proposal?

I would propose extending the debug namespace by an event which fires when the user selected a different stack frame in the call stack window. For example:

export namespace debug {

/**
 * An event which fires when the user selected a different stack frame in the call stack window.
 */
export const onDidChangeSelectedStackFrame: Event<StackFrame>;

}

The interface StackFrame consists of two properties: frameId and associated threadId. This would provide enough information to update the thread and stack frame in the Debug Adapter via custom request.

export interface StackFrame {

/**
 * Id of the associated thread (DAP id).
 */
readonly threadId: number;

/**
 * Id of the stack frame (DAP id).
 */
readonly frameId: number;

}

What do you think about this?

@weinand will this do?

@haneefdm
Copy link
Contributor

If this is implemented, it needs to be specific to a session. As in threadId/frameId are not enough. We have to deal with multiple concurrent debug sessions. Needs DebugSession object or at least the sessionId.

Doesn't frameId encapsulate/include threadId from a DA's perspective?

@gregg-miskelly
Copy link
Member Author

Doesn't frameId encapsulate/include threadId from a DA's perspective?

Yes, but I think you want to expose both so that a VS Code extension could call any DAP API which either took a thread id or a frame id.

@roblourens
Copy link
Member

@gregg-miskelly @hediet could you tell me what the use case is? And does "current" mean "selected in the UI"?

@gregg-miskelly
Copy link
Member Author

Yes, "current" means the one selected in the UI.

I originally ran into this when I wanted to implement Set Next Statement in the C# extension before the VS Code UI added support for it. It has also come up when potentially wanting to support a disassembly window (though again, this was solved by building the feature in VS Code instead). My team doesn't have a current need for this, though I still think this is a good idea as this makes the extension API very limited in terms of building any kind of debugger feature. Without this feature, it isn't possible for an extension to make valid DAP requests for lots of feature areas, especially anything involving looking at variables/expression evaluation.

@geajack
Copy link

geajack commented Nov 5, 2022

For another use case, I ran into this when trying to develop a debugging extension for Python. One of the features was that you could type in a variable name, or more generally an expression, and you would get back a certain visualization of the value of that expression. In order to do this, the extension sends the expression over the DAP to have Python evaluate it and send the value back. However, the variable, or any variables in the expression have to actually be in scope for this to work.

If the user clicks back to a different stack frame in the UI, they can now view the values of variables in the built-in VSCode debugging panel. Since my extension also allows you to, in a sense, view the values of variables, the user expects to be be able to use it for any variables that they can see in the debugging panel. So I need to know when they switch stack frames to give them that experience. Right now my extension basically maintains a parallel list of stack frames in the UI, so if the user wants to climb back up the call stack and use my extension, they have to click the stack frame in both VSCode's built-in UI and in my extension's UI.

@hediet
Copy link
Member

hediet commented Nov 7, 2022

My use case is my debug visualizer extension. When the user selects a different stack frame than the top one, the watch windows and the visualizer window disagree.

@mblout
Copy link
Contributor

mblout commented Mar 28, 2023

I also have a use-case for having additional views participate in the debugging; being able to show a custom "registers" view that tracks with selection, for example. We also have some breakpoint functionality that changes if we're in an active session, and would leverage the active thread to 'scope' which threads for which it is active.
So, I'd want to both interrogate the current/selected thread/stack frame (in session context) and to listen for changes

daniel-wachira added a commit to trilogy-group/openvscode-server that referenced this issue Jul 6, 2023
* cli: add new control server messages for wsl (microsoft#180438)

* cli: add new control server messages for wsl

* support cwd in spawn

* use auth session id for sync when enabled by embedder (microsoft#180444)

* add plan-items to release notes section (microsoft#180445)

* Oops

* Render both debug frame indicator and breakpoint (microsoft#180448)

* clean up extension recommendations (microsoft#180450)

* clean up extension recommendations

* retain editor config extension

* Add API that exposes interactive session provider name (microsoft#180437)

* Add API that exposes interactive session provider name

* Update command label

* Suppress default text drop if there's also a uri list (microsoft#180452)

* Support markdown string

* Fix search result count again (microsoft#180453)

* Add comment

* Allow description map to be optional

* Add extension based tests

* Fix test

* Try

* Use custom node-gyp for SDL pipeline (microsoft#179723)

* Use custom node-gyp
* Avoid using relative cd
* Remove ia32 for Linux
* Bump gulp-atom-electron; unblocks downloading PDBs
* Add native-is-elevated to the scan

* Allow new codeblock actions to be shown at the top level of the toolbar (microsoft#180460)

* Make description optional when reading storage

* debug session: use queue to make sure debugee status get processed in correct order (microsoft#180410)

Similarly to fed69ad, this adds a queue to serialize the processing
of pause/continue events. Without this, if a pause and continue events
are sent really fast, the pause handler will take longer to process than
the continue handler and the state of the debugger UI will be paused
when the debuggee is actually running.

Fixes microsoft#180409

* No more raw IPC in the Issue Reporter (microsoft#180449)

This is the initial implementation of the issue reporter no longer having raw IPC calls in it.

I think there's plenty more to clean up, but this is a start...

cc @bpasero

cc @Tyriar if you wanna do this for the process explorer

* fix run selected text

* Fix Interactive Session widget role (microsoft#180463)

Fix Interactive Session widget

* cleanup save API (microsoft#180476)

* Revert "add accessibility verbosity settings for terminal and diff editor aria hints" (microsoft#180477)

* changed the weight to workbench contrib

* cancel code lens resolve requests when document is being edited (microsoft#180485)

https://github.com/microsoft/vscode-internalbacklog/issues/3884

* show commands only when having reply, proper type for different edit modes

* Fixes microsoft#3928 (microsoft#180404)

* Fixes microsoft#3928

* Fixes CI

* [debt] extract Session object which holds useful state for commands

* Fixes microsoft#153754. (microsoft#180490)

* Fixes microsoft#153754.

* Fixes microsoft#159479

* allow for default sash ratio in diff editor (side-by-side) and use it for inline diff'ing

* Fixes microsoft#173495 (microsoft#180492)

* Fix no tree view error when view is disposed during refresh (microsoft#180415)

Fixes microsoft/vscode-pull-request-github#4002

* add missing monaco.d.ts file

* Remove comments when a comment provider is unregistered (microsoft#180502)

Fixes microsoft/vscode-pull-request-github#4566

* Observed deriveds are now recomputed lazily. (microsoft#180487)

* Observed deriveds are now recomputed lazily.

* Fixes CI

* Fixes observable bug

* Fixes microsoft#173209 (microsoft#180505)

* Fixes accept next word bug

* Fixes microsoft#173209

* Don't dispose model in separate transactions, as deriveds are now computed lazily. (microsoft#180507)

* Fixes microsoft#180224 (microsoft#180511)

* in notebooks cancel inline chat sessions from sibling editors when starting a new session (microsoft#180509)

https://github.com/microsoft/vscode-internalbacklog/issues/3897

* extension host - allow to veto stopping (microsoft#179224) (microsoft#180513)

* changing the code so that view in chat is an additional button in the toolbar and the text appears below the toolbar

* adding the context key while waiting for the merge

* changing the css

* Improve selected row checkbox style (microsoft#180515)

Fixes microsoft#180506

* cleanup preview styles and keep status bar consistent (microsoft#180518)

* add `CTX_INTERACTIVE_EDITOR_LAST_RESPONSE_TYPE` and use that to control visiblity accept, cancel, and toggle inline action (microsoft#180517)

* add `CTX_INTERACTIVE_EDITOR_LAST_RESPONSE_TYPE` and use that to control visiblity accept, cancel, and toggle inline action

* fix compile errors

* cleaning the code

* cleaning the code

* Add command for expanding unresolved comments (microsoft#180521)

Fixes microsoft#169928

* adopt tunnel mutex name change (microsoft#180524)

* chore: update [email protected] (microsoft#180526)

* joh/passive barracuda (microsoft#180530)

* tweak inline diff editor options, make sure livePreview and notebooks is pretty

* tweak logging

* handle 403 and bail out immediately (microsoft#180547)

* Also wait for user data init

* wait only when init is required

* make sure rename disposes it `EditorStateCancellationTokenSource` so that its down-level context key is correctly updated (microsoft#180545)

this is part two of https://github.com/microsoft/vscode-internalbacklog/issues/3928

* Don't run chat codeblocks in task terminals (microsoft#180552)

* Add command/keybinding to change drop type (microsoft#180556)

* Add command/keybinding to change drop type

Adds a command/keybinding to toggle the post drop widget

* Also show keybinding

* Code review

* Fix drop widget having transparent background on hover (microsoft#180562)

We now overlay the button-secondaryHoverBackground over editor-widgetBackground

* Update distro

* Oops

* Iios

* Re microsoft#178715. Wait for notebook model to be ready for dispose (microsoft#180560)

* remove unnecessary ? (microsoft#180564)

* ensure focus is set correctly in output, enable keyboard scrolling for outputs

* report telemetry (microsoft#180567)

* Instrument Microsoft account type (microsoft#180573)

So we can see if folks are using MSA or AAD accounts. Also, this cleans up some dead code.

Fixes https://github.com/microsoft/vscode-internalbacklog/issues/3903

* Specify codeActionKinds (microsoft#180576)

Avoids extra calls and lets us show this info in the UI

* Also search injected text decorations tree for margin decorations (microsoft#180620)

* Also search injected text decorations tree for margin decorations

* Simplify fix

* quick access - allow a `Promise<FastAndSlowPicks<T>>` and adopt for commands (microsoft#180664)

* quick access - allow a `Promise<FastAndSlowPicks<T>>` and adopt for commands

* fix telemetry

* Change the way the InteractiveSessionModel is loaded into the widget (microsoft#180668)

* Change the way the InteractiveSessionModel is loaded into the widget
And the way viewState is managed. Fixes a bunch of general issues with this flow between views and editors

* Fix reloading editor after clear

* don't terminate IE session when observing outside edits (microsoft#180680)

Only when they earase everything cancel IE session, otherwise just live with it

* hide ghost text when invoking IE session (microsoft#180682)

https://github.com/microsoft/vscode-internalbacklog/issues/3573

* Fixes microsoft#180222 (microsoft#180681)

* Fixes microsoft#180538 (microsoft#180683)

* Fixes microsoft#180489 (microsoft#180684)

* add jsdoc comment for trackFocus (microsoft#180690)

* work in propgress, adding the fix for the feedback bug also

* Show hover message for diff revert arrow (microsoft#180378)

* fixes microsoft#177444 (microsoft#180704)

* prioritize images over text in accessiblity mode

* Fix microsoft#180588 (microsoft#180708)

* fix microsoft#180670

* cleaning the code

* split out event propagation handling

* using the toggle functionality instead

* pre saving the old data

* use editor options

* polishing the code

* remove true

* Allow negative numbers for `go to match` (microsoft#180479)

Fixes microsoft#180475

Also fixes the case where there are zero matches, which previously resulted in a non-functional quick input

* Pick up latest TS for building VS code (microsoft#180706)

* SCM - ActionBar rendering optimization (microsoft#180712)

* SCM - ActionBar rendering optimization

* address PR comments

---------

Co-authored-by: João Moreno <[email protected]>

* add some padding to MD message, restore right alignment of status label

* show accept/discard btns even when last response was a message response

* check the scroll height of the correct element

* Use readonly editor instead of an untitled buffer when showing environment changes

* Use view state to restore InteractiveSessionViewPane session by id (microsoft#180732)

* remove escape to container hotkey

* Organize Errors in GitHub Auth and make sure no double prompting happens (microsoft#180734)

* Organize Errors in GitHub Auth and make sure no double prompting happens

This mostly just moves some strings into variables... but this also fixes the GH Auth side of microsoft#180697 so you should only be asked once if you want to try a different way to log in.

* add comments

* Initial support for notebook CodeActions (microsoft#180740)

* support cmd codeActions on nb save

* add setting

* rm comment

* add log + add await command

* missed a space in setting description

* Improves observable docs

* fix class selection query

* fix message formatting

* fix: Double pasting in webview(using russian keyboard layout) (microsoft#161001) (microsoft#178120)

Co-authored-by: Matt Bierner <[email protected]>

* Use custom command to open JS doc links (microsoft#180737)

Fixes microsoft#162507

Prevents incorrect auto transform of the uri

* testing: tie test output correctly to runs, rework ui accordingly (microsoft#180746)

* testing: don't enqueue tests into tasks that are created

This is an old artifact that caused problems in the issue this fixes microsoft#180041

When we get a new 'task' from a test run request, we automatically
marked all tests as being enqueued with it. However, tests are now lazily
added to the test run states, so this would only affect tests that
_already_ had their state set to something (e.g. in another task). And
therefore it was also ineffective for its original purpose of marking
all included tests as being Queued, since extensions have had to do that
themselves anyway.

So just remove this code! Also, adjust priorities such that "skipped"
priorities are shown above unset ones.

* testing: tie test output correctly to runs, rework ui accordingly

Previously, test output was handled for an entire TestRunRequest into
a single stream. This didn't match what the public API implied, nor what
the real intentions were.

This makes output be stored correctly on a per-test run (called "tasks"
internally). It changes the order of the Test Results view so that tests
nested under their task, and also improves some of the tree handling
there to update nodes more specifically.

Clicking on the "terminal" button in the test view continues to show
output from the last test run request, but if there were multiple tasks,
the user is asked to choose which they want to view output for.

Finally I removed some old unused code that dealt with storing test
results on disk, which we no longer do, and moved to a simpler interface
instead.

Fixes microsoft#180041

* Apply buffer copy optimization to `updateOutput` oto (microsoft#180755)

For microsoft#168142

Follow up on 9fd77b4. Applies the same optimization to `updateOutput` as well

* Restore welcome message when restoring chat session (microsoft#180756)

* Change the shape of persisted sessions

* Also backup sessions that were not restored in this window

* Restore welcome message when restoring chat session

* add F20-F24 key support for keyboard shortcuts

* remove unused keyCodeOrd from key mappings list

* Feed 📓 error to interactive session. (microsoft#180766)

* Add command to show chat session history (microsoft#180770)

* testing: avoid including individual children in run at cursor if their parent is already included (microsoft#180763)

For #180760

* Add maximum number of persisted chat sessions (microsoft#180775)

* fix microsoft#180721

* build: quieter node module cache logging (microsoft#180709)

* Default Themes: Rename Experimental to Dark Modern (microsoft#180785)

Default Themes: Rename Experimental to Modern

* Merge user port changes with extension forwarded ports (microsoft#180694)

* Merge user port changes with extension forwards

* Actually add port to unrestoredExtensionTunnels

* nit

* context keys: fix: add serialize() support for ContextKeyNotRegexExpr

Fixes microsoft#180012

* update third party notices (microsoft#180788)

* Update ThirdPartyNotices.txt

* update distro

* new default theme: check setting when patching initial colors (microsoft#180687)

* new default theme: check setting when patching initial colors

* remove unneeded imports

* json/css/html: update dependencies (microsoft#180714)

* json/css/html: update dependencies

* update services

* add nls message to preserve link syntax (microsoft#180821)

* add lint rule to prevent type discrimination properties in API types (microsoft#180829)

re microsoft#63943 and other

* remove nb.codeactiononsave setting, add `nb.experimental` to command ID

* update wording

* cli: fix problem restarting tunnel with uppercase letter in name (microsoft#180881)

Fixes microsoft#180693

* Fix drop mime type wildcard regexp (microsoft#180883)

For microsoft#180878

* Disable breakpoint hint when a test loading icon is present (microsoft#180884)

* Add extra notes on `files` drop mime type (microsoft#180886)

Fixes microsoft#180878

* Fix enumItem labels (microsoft#180889)

`javascript.preferences.importModuleSpecifierEnding` and `typescript.preferences.importModuleSpecifierEnding` got out of sync here

* Fix markdown settings localization (microsoft#180891)

The syntax backtick hash str hash backtick was getting mangled.... but it probably should be extracted out so translators don't have the potential for changing it.

* Bump distro (microsoft#180895)

* Fix microsoft#180797. Fine tune edit mode switch. (microsoft#180900)

Fix microsoft#180797.

* Update terminal status even if it is already present

* Fix microsoft#180857. Empty cmd+f should not unset search keyword. (microsoft#180901)

* Fix microsoft#180861. Cell stays in editing mode if it is not modified by find. (microsoft#180902)

* API 💄 (microsoft#180911)

* more cursor overrides: style and blinking

fixes https://github.com/microsoft/vscode-internalbacklog/issues/3864

* making the css selectable by changing the css file

* make sure to restore diff editor focus after repositioning the zone widget

fixes https://github.com/microsoft/vscode-internalbacklog/issues/3964

* back to `computeHumanReadableDiff` diff

fixes https://github.com/microsoft/vscode-internalbacklog/issues/3962

* simplify code for demo (microsoft#180932)

* disable inline completions for inline chat input

fixes https://github.com/microsoft/vscode-internalbacklog/issues/3961

* disabe cmd/ctrl+z for "special undo" of inline chat

fixes https://github.com/microsoft/vscode-internalbacklog/issues/3966

* don't show release notes on web (microsoft#180937)

fixes microsoft#179536

* Fix initial comment expand state (microsoft#180941)

Fixes microsoft#169928

* prevent endless autoSend loop of inline chat (microsoft#180945)

fixes https://github.com/microsoft/vscode-internalbacklog/issues/3990

* "Show Next Change" doesn't show the right change (microsoft#180951)

Fixes microsoft#180744

* make sure EditModeStrategy#apply/cancel isn't called twice, keep reference onto editor model to prevent dispose before reset (microsoft#180952)

fixes https://github.com/microsoft/vscode-internalbacklog/issues/3963

* Enables experiments for diffEditor.diffAlgorithm (microsoft#180954)

* update distro (microsoft#180953)

* Fix typo in vscode.d.ts (microsoft#177377)

"of" -> "or"

Co-authored-by: Matt Bierner <[email protected]>

* fix multi range formatting (microsoft#180955)

fixes microsoft#178825

* update distro (microsoft#180957)

* Fix microsoft#177405. Treat error as plain text. (microsoft#180960)

* Fix microsoft#180290. Tweak wording accurately for install/enable missing extension. (microsoft#180961)

* Fix microsoft#163383 (microsoft#180963)

* Fix copying a suggested followup question in chat (microsoft#180965)

* add workspaceTrust to nb "codeaction" infra (microsoft#180968)

* Fix Issue Reporter API (microsoft#180971)

* Fix Issue Reporter API

CancellationTokens aren't hydrated over ProxyChannels so I've removed cancellation logic.

Additionally, there was a bad string compare that needed toLower when checking if an extension has a handler.

Additionally, added docs.

Fixes microsoft#180890
Fixes microsoft#180920
Fixes microsoft#180887

* remove import

* remove another import

* [json] update service (microsoft#180972)

* fix microsoft#180899

* Notification marker color turns grey when selected (microsoft#180973)

* Add mode to the notification (microsoft#180977)

ref microsoft#180803 (comment)

* Add overrideCategory back (microsoft#180975)

* Update getting started theme picker images to reflect new themes (microsoft#180976)

* Update getting started theme picker assets to reflect new themes

* Update theme names

* Fixes bug in observable utility

* Fixes microsoft#180792

* Fixes CI

* Update vscode.d.ts for clearSessionPreference (microsoft#180979)

Fixes microsoft#180860

* Use a colon instead of quotes to handle long queries better (microsoft#180980)

Fixes https://github.com/microsoft/vscode-internalbacklog/issues/4021

* fix microsoft#172465

* undo some changes

* Allow clicking to remove breakpoint even alongside non debug decorations (microsoft#180986)

* Load chat username/icon from current session, not persisted data (microsoft#180981)

* update distro (microsoft#180992)

* Avoid double encoding vscode.dev links (microsoft#181002)

* Fixes microsoft#181007 (microsoft#181008)

* Fixes microsoft#181009 (microsoft#181011)

* Fixes microsoft#181009

* Use isMeasurement: true for boolean

* Use correct key for hybrid port unforwarding (microsoft#181005)

* Use correct key for hybrid port unforwarding

* Use forwarded host for closing

* Delete key from this.autoForwarded

* Adds telemetry to understand how long the diff editor was visible to the user. (microsoft#181021)

* Adds telemetry to understand how long the diff editor was visible to the user.

* Uses StopWatch utility.

* Fixes CI

* 💄

* 💄

---------

Co-authored-by: Benjamin Pasero <[email protected]>

* enable live preview for last line of document. for that... (microsoft#181029)

- support `ordinal` and `showInHiddenAreas` for `IViewZone`
- adopt in zoneWidget
- adopt for inline chat widgets
- remove code that isn't needed anymore 👯‍♂️

fixes https://github.com/microsoft/vscode-internalbacklog/issues/4024

* fix microsoft#175014

* fix tab order for hidden status element when a message shows

fixes https://github.com/microsoft/vscode-internalbacklog/issues/4039

* don't use role or aria-label attributes so that contents get read out

fixes https://github.com/microsoft/vscode-internalbacklog/issues/4035

* Add a couple teammates to my-endgame notebook (microsoft#181059)

* Add a couple teammates to my-endgame notebook

friends :)

* Add @Yoyokrazy

* Add @paulacamargo25

* debug: bump js-debug (microsoft#181049)

Fixes microsoft#181047

* Revert "Fix: diff editor arrow click enables breakpoint " (microsoft#181040)

* Set glyph margin decoration width appropriately (microsoft#181071)

* Move opening default built-in walkthrough to after built-in walkthrough initialization (microsoft#181066)

* chore: bump distro (microsoft#181104)

* Unexpected theme name showing up in web (microsoft#181091)

* Unexpected theme name showing up in web

* incorperate feedback

* Install extension in the right server (microsoft#181110)

* Fixes `PieceTreeBase.equal` (microsoft#181125)

Fixes `PieceTreeBase.equal` (fixes microsoft/vscode-internalbacklog#4025)

* Revert "Fixes `PieceTreeBase.equal` (microsoft#181125)" (microsoft#181270)

This reverts commit 3862229.

* cli: tunnels can sporadically become unresponsive (microsoft#181286)

Last iteration I moved some RPC logic to use Tokios "codecs" to give
them cancellation safety. These operate on streams of input data.

While this worked at first, I failed to take into account that the byte
buffer we read from the stream could have _more_ data than just the
current message under load scenarios. We were discarding any extra data
from the subsequent message. In most cases caused the next message
"length" to be read from the middle of the next message, which usually
(when parsed as a u32) was some number of gigabytes, then causing the
connection to stall forever.

Fixes microsoft#181284

* Add temporary nodeJS walkthrough (microsoft#181442)

* bump version

* unc - adopt setting and handling of allow list (gitpod-io#5)

* unc - adopt setting and handling of allow list

* unc - set allow list on server too

* unc - pick our patched node.js for now

* bump electron

* unc - ignore sync is not needed with machine scope

* unc - use process set directly

* 🆙 22.5.1

* code web server initial commit

* Don't test the Feedback component

* enable prebuilds for all branches

* Add example guide for pre-installing vscode extensions with docker installation

* 💄

* Update deprecated example in README 

Option 'connection-secret' is deprecated: Use connection-token-file instead.

* Fix default image destroyed in merge

* Port the delayed signal patch to gp-code-1.78

* Fix build

* Remove access token requirement as we're running in a sandbox

* Revert "Remove access token requirement as we're running in a sandbox"

This reverts commit 8a7a90c.
Use the --without-connection-token arg instead

---------

Co-authored-by: Connor Peet <[email protected]>
Co-authored-by: Sandeep Somavarapu <[email protected]>
Co-authored-by: First <[email protected]>
Co-authored-by: Joyce Er <[email protected]>
Co-authored-by: Rob Lourens <[email protected]>
Co-authored-by: Matt Bierner <[email protected]>
Co-authored-by: Raymond Zhao <[email protected]>
Co-authored-by: Florent Revest <[email protected]>
Co-authored-by: Tyler James Leonhardt <[email protected]>
Co-authored-by: Megan Rogge <[email protected]>
Co-authored-by: Johannes Rieken <[email protected]>
Co-authored-by: Benjamin Pasero <[email protected]>
Co-authored-by: Alexandru Dima <[email protected]>
Co-authored-by: Aiday Marlen Kyzy <[email protected]>
Co-authored-by: Henning Dieterichs <[email protected]>
Co-authored-by: Alex Ross <[email protected]>
Co-authored-by: Martin Aeschlimann <[email protected]>
Co-authored-by: Robo <[email protected]>
Co-authored-by: Peng Lyu <[email protected]>
Co-authored-by: aamunger <[email protected]>
Co-authored-by: João Moreno <[email protected]>
Co-authored-by: meganrogge <[email protected]>
Co-authored-by: Ladislau Szomoru <[email protected]>
Co-authored-by: Michael Lively <[email protected]>
Co-authored-by: dongfang <[email protected]>
Co-authored-by: Ilia Pozdnyakov <[email protected]>
Co-authored-by: Ulugbek Abdullaev <[email protected]>
Co-authored-by: Lakshya A Agrawal <[email protected]>
Co-authored-by: Bhavya U <[email protected]>
Co-authored-by: David Dossett <[email protected]>
Co-authored-by: SteVen Batten <[email protected]>
Co-authored-by: Benjamin Pasero <[email protected]>
Co-authored-by: Filip Troníček <[email protected]>
Co-authored-by: Anton Kosyakov <[email protected]>
Co-authored-by: AXON <[email protected]>
Co-authored-by: Jean Pierre <[email protected]>
Co-authored-by: Bjarne Sievers <[email protected]>
@mblout
Copy link
Contributor

mblout commented Jul 13, 2023

Wanted to ask if there is anything I can do to make the discussions about promoting this proposed API easier?
Would a runnable demo help, like a vscode-mock-debug pull request?
Animated GIF or other form of doc/example? Thanks again!

@connor4312
Copy link
Member

This API has been finalized and will in tomorrow's Insiders and the next stable release. Please refer to https://github.com/microsoft/vscode/pull/212190/files#diff-0a75aed19c118603eb96332bc0b9c2d7867f4182346d16d18b7fc31b6ceeb321 for the final API.

connor4312 added a commit that referenced this issue May 7, 2024
@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels May 7, 2024
@microsoft microsoft locked and limited conversation to collaborators Jun 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api api-finalization debug Debug viewlet, configurations, breakpoints, adapter issues feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-testplan
Projects
None yet
Development

Successfully merging a pull request may close this issue.