Commit Graph

16 Commits

Author SHA1 Message Date
Connor Peet
66744e3b25 debug: enable js-debug to auto attach (#95807)
* debug: enable js-debug to auto attach

This modifies the debug-auto-launch extension to trigger js-debug as
outlined in https://github.com/microsoft/vscode/issues/88599#issuecomment-617242405

Since we now have four states, I moved the previous combinational logic
to a `transitions` map, which is more clear and reliable. The state
changes are also now a queue (in the form of a promise chain) which
should avoid race conditions.

There's some subtlety around how we cached the "ipcAddress" and know
that environment variables are set. The core desire is being able to
send a command to js-debug to set the environment variables only if they
haven't previously been set--otherwise, reused
the cached ones and the address.

This process (in `getIpcAddress`) would be vastly simpler if extensions
could read the environment variables that others provide, though there
may be security considerations since secrets are sometimes stashed
(though I could technically implement this today by manually creating
and terminal and running the appropriate `echo $FOO` command).

This seems to work fairly well in my testing. Fixes #88599.

* fix typo

* clear js-debug environment variables when disabling auto attach
2020-04-23 09:49:29 -07:00
Benjamin Pasero
ca22de4b17 debt - bump node.d.ts dependency in extensions to 12.x (part of #82514) 2019-10-28 08:11:47 +01:00
Aurélien Pupier
7a5bca9faf Add License field to package.jsons #68423 (#68771)
it should avoid to have "warning XXX: No license field" during yarn
build

Signed-off-by: Aurélien Pupier <apupier@redhat.com>
2019-06-06 11:44:50 +02:00
Benjamin Pasero
d78a75973c update @types/node (#74881) 2019-06-05 10:20:01 +02:00
Benjamin Pasero
d7977c880f debt - use @types/node: 10 for all extensions 2019-06-05 09:32:36 +02:00
Matt Bierner
dd3e2d94f8 Remove use strict directives in extension code
We compile with alwaysStrict so these directives are not needed
2018-10-02 16:02:44 -07:00
Matt Bierner
52d3961c2b Update more extensions to use shared config 2018-10-02 10:24:19 -07:00
Joao Moreno
707787c5b8 update yarn.lock files to use integrity 2018-09-30 16:34:43 +02:00
Alex Dima
5bae3f69d9 Use webpack also for small built-in extensions 2018-09-20 14:17:45 +02:00
Alex Dima
34e7a220c8 Avoid shipping unnecessary files 2018-09-19 19:53:46 +02:00
Johannes Rieken
fc2b0a8ec1 add missing vscode-nls updates 2018-09-06 15:12:28 +02:00
Andre Weinand
1ae8d0168e auto-attach cleanup 2018-08-25 00:21:24 +02:00
Andre Weinand
14595011f1 move some auto-attach code back to node-debug 2018-08-22 01:08:52 +02:00
Andre Weinand
edfda964e0 use terminal.processId for auto-attach; fixes #55918 2018-08-07 12:28:30 +02:00
Andre Weinand
47cf06001c detect 'winpty-agent.exe'; fixes #55672 2018-08-02 17:37:02 +02:00
Andre Weinand
bf7ac9201e move auto-attach into new built-in extension; fixes #53586 2018-07-12 16:41:03 +02:00