Skip to content

Use native Node.js standard APIs - #5933

Merged
Bharat Middha (bmiddha) merged 4 commits into
mainfrom
native-standard-apis
Aug 18, 2026
Merged

Use native Node.js standard APIs#5933
Bharat Middha (bmiddha) merged 4 commits into
mainfrom
native-standard-apis

Conversation

@bmiddha

@bmiddha Bharat Middha (bmiddha) commented Aug 18, 2026

Copy link
Copy Markdown
Member

Summary

  • Replace the package update timeout controller with AbortSignal.timeout()
  • Replace internal Text.replaceAll() calls with String.prototype.replaceAll()
  • Deprecate the compatibility Text.replaceAll() wrapper
  • Add timeout behavior coverage for package update checks

Stacked on #5931.

Validation

  • Built and tested the six affected packages
  • rush change --verify --no-fetch

@github-project-automation github-project-automation Bot moved this to Needs triage in Bug Triage Aug 18, 2026
Base automatically changed from es2022-baseline to main August 18, 2026 06:28
Bharat Middha (bmiddha) and others added 2 commits August 17, 2026 23:41
@bmiddha
refactor: use native Node.js APIs
8ea2d76 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 8ebd5bf2-c44b-42d5-be25-e7936d4b0a14
@bmiddha
refactor: deprecate Text.replaceAll
1223d19 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 8ebd5bf2-c44b-42d5-be25-e7936d4b0a14
iclanton
Ian Clanton-Thuon (iclanton) approved these changes Aug 18, 2026
View reviewed changes
Comment thread apps/api-extractor/src/analyzer/SourceFileLocationFormatter.ts Outdated
Comment thread apps/rundown/src/Rundown.ts Outdated
Comment thread common/changes/@rushstack/lockfile-explorer/native-standard-apis_2026-08-18-03-14-05.json Outdated
Comment thread common/changes/@rushstack/node-core-library/native-standard-apis_2026-08-18-03-14-05.json Outdated
Comment thread libraries/node-core-library/src/test/Executable.test.ts Outdated
Comment thread libraries/rush-lib/src/logic/installManager/RushInstallManager.ts Outdated
Comment thread libraries/node-core-library/src/test/Executable.test.ts Outdated
Comment thread common/changes/@rushstack/node-core-library/native-standard-apis_2026-08-18-03-14-05.json Outdated
Comment thread apps/rundown/src/Rundown.ts Outdated
Comment thread libraries/node-core-library/src/JsonFile.ts Outdated
@bmiddha
refactor: reuse path and newline helpers
4c88fa8 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 8ebd5bf2-c44b-42d5-be25-e7936d4b0a14
@bmiddha
Bharat Middha (bmiddha) enabled auto-merge (squash) August 18, 2026 17:51
@bmiddha
chore: remove implementation-only change entry
e488c02 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 8ebd5bf2-c44b-42d5-be25-e7936d4b0a14
@bmiddha
Bharat Middha (bmiddha) merged commit a2507e8 into main Aug 18, 2026
10 checks passed
@bmiddha
Bharat Middha (bmiddha) deleted the native-standard-apis branch August 18, 2026 18:36
@github-project-automation github-project-automation Bot moved this from Needs triage to Closed in Bug Triage Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iclanton Ian Clanton-Thuon (iclanton) iclanton approved these changes
@octogonz Pete Gonzalez (octogonz) Awaiting requested review from octogonz octogonz is a code owner
@apostolisms Apostolis Haitalis (apostolisms) Awaiting requested review from apostolisms apostolisms is a code owner
@dmichon-msft David Michon (dmichon-msft) Awaiting requested review from dmichon-msft dmichon-msft is a code owner
@jxanthony Jonathon Anthony (jxanthony) Awaiting requested review from jxanthony jxanthony is a code owner
@TheLarkInn Sean Larkin (TheLarkInn) Awaiting requested review from TheLarkInn

Assignees

No one assigned

Labels

None yet

Projects

Bug Triage
Status: Closed

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bmiddha @iclanton