Skip to content

fix(mcp-server.sj): delete console.log and console.error statements #11

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

Merged
merged 7 commits into from
Mar 23, 2025

Conversation

arthurgousset
Copy link
Member

@arthurgousset arthurgousset commented Mar 23, 2025

Description

  • fix(mcp-server.js): delete console.log and console.error
  • fix(index.js): delete console.log and console.error

Other changes

  • chore(package.json): bump patch version from 0.2.1 to 0.2.2

Tested

Yes, published beta version and tried to install in Cursor:

{
  "mcpServers": {
    "nodejs-debugger": {
      "command": "npx",
      "args": ["@hyperdrive-eng/[email protected]"]
    }
  }
}
image

Used on our test server:

node --inspect index.js
Debugger listening on ws://127.0.0.1:9229/6b913e98-c16a-48b0-bcd5-69afdd5acfcc
For help, see: https://nodejs.org/en/docs/inspector
Server running at http://localhost:3000/
Debugger attached.
image

Also tested local install with

npx ~/Documents/hyperdrive-eng/mcp-nodejs-debugger

Related issues

Backwards compatibility

Yes, fixes bug.

Documentation

Publish beta version with

npm publish --tag beta --access public

I mistakenly published 0.3.1 beta versions, but in the end I only released a patch update 0.2.2 because it's a backwards compatible bug fix.

@arthurgousset arthurgousset force-pushed the arthurgousset/fix/unexpected-token branch from 8990d85 to f95f0d2 Compare March 23, 2025 18:59
@arthurgousset arthurgousset changed the title fix(mcp-server.sj): delete console.log and console.error statements fix(mcp-server.sj): delete console.log and console.error statements Mar 23, 2025
@arthurgousset arthurgousset merged commit ab6c367 into main Mar 23, 2025
@arthurgousset arthurgousset deleted the arthurgousset/fix/unexpected-token branch March 23, 2025 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unexpected token 'D', "Debugger enabled" is not a valid JSON in Cursor IDE
1 participant