FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

inspector: improve batch diagnostic channel subscriptions by legendecas · Pull Request #60009 · nodejs/node · GitHub

/ node Public

inspector: improve batch diagnostic channel subscriptions - #60009

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:inspector-register
Oct 6, 2025
Merged

inspector: improve batch diagnostic channel subscriptions#60009
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:inspector-register

Conversation

Copy link
Copy Markdown
Member

Improve batch diagnostic channel subscriptions in network inspections. This could prevent a channel from mismatches in subscribe/unsubscribe.

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/inspector

nodejs-github-bot added http2 Issues or PRs related to the http2 subsystem. inspector Issues and PRs related to the V8 inspector protocol needs-ci PRs that need a full CI run. labels Sep 25, 2025

codecov Bot commented Sep 25, 2025
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.47%. Comparing base (4dc19ec) to head (713dff3).
⚠️ Report is 97 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #60009      +/-   ##
==========================================
- Coverage   88.49%   88.47%   -0.02%     
==========================================
  Files         703      703              
  Lines      207529   207529              
  Branches    40011    40015       +4     
==========================================
- Hits       183643   183617      -26     
- Misses      15885    15899      +14     
- Partials     8001     8013      +12     
Files with missing lines Coverage Δ
lib/internal/inspector/network.js 95.95% <100.00%> (+1.22%) ⬆️
lib/internal/inspector/network_http.js 94.85% <100.00%> (-0.39%) ⬇️
lib/internal/inspector/network_http2.js 94.94% <100.00%> (-0.32%) ⬇️
lib/internal/inspector/network_undici.js 91.66% <100.00%> (-0.53%) ⬇️

... and 35 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 25, 2025
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 25, 2025

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

legendecas added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 6, 2025
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 6, 2025
nodejs-github-bot merged commit d9b8a35 into nodejs:main Oct 6, 2025
73 checks passed

Copy link
Copy Markdown
Collaborator

Landed in d9b8a35

legendecas deleted the inspector-register branch October 6, 2025 13:50
RafaelGSS pushed a commit that referenced this pull request Oct 7, 2025
PR-URL: #60009
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

http2 Issues or PRs related to the http2 subsystem. inspector Issues and PRs related to the V8 inspector protocol needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL