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

Node debug: auto attach mode is not remembered #53586

Closed
egamma opened this issue Jul 5, 2018 · 9 comments
Closed

Node debug: auto attach mode is not remembered #53586

egamma opened this issue Jul 5, 2018 · 9 comments
Assignees
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues feature-request Request for new features or functionality release-notes Release notes issues verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@egamma
Copy link
Member

egamma commented Jul 5, 2018

  • enable auto attach using the Toggle Auto Attach command
  • reload the window

-> the auto attach statusbar item is gone.

I was wondering whether the auto attach mode would not better be a setting, then I can define per workspace when I want to have the feature enabled.

@weinand
Copy link
Contributor

weinand commented Jul 5, 2018

@egamma The reason why we don't remember this as a setting is because that would require to activate the node-debug extensions on startup (which we are reluctant to do).

@weinand weinand added debug Debug viewlet, configurations, breakpoints, adapter issues feature-request Request for new features or functionality labels Jul 5, 2018
@weinand
Copy link
Contributor

weinand commented Jul 6, 2018

see #53640

@egamma
Copy link
Member Author

egamma commented Jul 6, 2018

closing as a duplicate of #53640

@egamma egamma closed this as completed Jul 6, 2018
@weinand
Copy link
Contributor

weinand commented Jul 12, 2018

I've reopened this issue to track the fixes for the following problems:

  • the auto-attach state is now a setting with 3 values: 'disabled', 'on', 'off'. The value 'disabled' hides it from the status bar.
  • if the setting is created via the "Toggle Auto Attach", it will be stored in workspace settings.
  • restarting remembers the setting's value.

@weinand weinand reopened this Jul 12, 2018
@weinand weinand added this to the July 2018 milestone Jul 12, 2018
@weinand
Copy link
Contributor

weinand commented Jul 12, 2018

I've created #54183 to address the issue with the settings API.

@weinand weinand added the verification-needed Verification of issue is requested label Jul 29, 2018
@weinand
Copy link
Contributor

weinand commented Jul 29, 2018

Verification info:

  • the auto-attach state is now a setting with 3 values: 'disabled', 'on', 'off'. The value 'disabled' hides it from the status bar.
  • if the setting is created via the "Toggle Auto Attach", it will be stored in workspace settings.
  • restarting remembers the setting's value.

Verify that

  • the auto-attach setting works as described,
  • auto-attach setting can be used in user and workspace settings,
  • and is persisted across debug sessions.

@JacksonKearl JacksonKearl added the verification-found Issue verification failed label Jul 31, 2018
@JacksonKearl
Copy link
Contributor

I'm not able to get auto-attach to work on win10 with node v8.11.2 on latest insiders.

I have index.js:

let a = () => setTimeout(() => {
    console.log('ping')
    a();
}, 500);
a();

And when I run node --inspect index.js in either bash or powershell, the program executes but code does not attach

@roblourens roblourens reopened this Aug 1, 2018
@weinand
Copy link
Contributor

weinand commented Aug 2, 2018

auto-attach failing on Windows issue is unrelated to this issue.
I've created issue #55672 for this.

@weinand weinand closed this as completed Aug 2, 2018
@weinand weinand added verification-needed Verification of issue is requested verification-found Issue verification failed release-notes Release notes issues verified Verification succeeded and removed verification-found Issue verification failed verification-needed Verification of issue is requested labels Aug 2, 2018
@weinand
Copy link
Contributor

weinand commented Aug 6, 2018

The "verification needed" is incorrect because found issue #55672 is a new independent issue (that has been verified already).

@vscodebot vscodebot bot locked and limited conversation to collaborators Sep 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues feature-request Request for new features or functionality release-notes Release notes issues verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants