Skip to content

repl: add customizable error handling#62188

Merged
nodejs-github-bot merged 2 commits intonodejs:mainfrom
addaleax:repl-handle-errors-fn
Mar 12, 2026
Merged

repl: add customizable error handling#62188
nodejs-github-bot merged 2 commits intonodejs:mainfrom
addaleax:repl-handle-errors-fn

Conversation

@addaleax
Copy link
Member

Whether or not an exception should be handled when it was thrown in the REPL's context but the REPL is already closed is something that will depend on the details of the specific REPL instance's use case.

Adding this option lets the REPL's creator control the details of this behavior.

This resolves a TODO recently added in bd3cba5.

@addaleax addaleax added the semver-minor PRs that contain new features and should be released in the next minor version. label Mar 10, 2026
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. repl Issues and PRs related to the REPL subsystem. labels Mar 10, 2026
Whether or not an exception should be handled when it
was thrown in the REPL's context but the REPL is already
closed is something that will depend on the details of
the specific REPL instance's use case.

Adding this option lets the REPL's creator control the
details of this behavior.

This resolves a TODO recently added in bd3cba5.
@addaleax addaleax force-pushed the repl-handle-errors-fn branch from 7aa0e38 to 64670a3 Compare March 10, 2026 16:18
@codecov
Copy link

codecov bot commented Mar 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.66%. Comparing base (ae228c1) to head (64980e1).
⚠️ Report is 13 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #62188   +/-   ##
=======================================
  Coverage   89.65%   89.66%           
=======================================
  Files         676      676           
  Lines      206546   206561   +15     
  Branches    39558    39552    -6     
=======================================
+ Hits       185179   185204   +25     
+ Misses      13485    13474   -11     
- Partials     7882     7883    +1     
Files with missing lines Coverage Δ
lib/repl.js 94.01% <100.00%> (+0.04%) ⬆️

... and 31 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.

@addaleax addaleax added the review wanted PRs that need reviews. label Mar 11, 2026
@addaleax addaleax added the request-ci Add this label to start a Jenkins CI on a PR. label Mar 11, 2026
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Mar 11, 2026
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Mar 11, 2026

@addaleax addaleax added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Mar 12, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 12, 2026
@nodejs-github-bot nodejs-github-bot merged commit 5e3951e into nodejs:main Mar 12, 2026
74 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 5e3951e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. needs-ci PRs that need a full CI run. repl Issues and PRs related to the REPL subsystem. review wanted PRs that need reviews. semver-minor PRs that contain new features and should be released in the next minor version.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants