Skip to content

refactor(@angular-devkit/core): deprecate stringToFileBuffer and fileBufferToString#33150

Merged
clydin merged 1 commit intoangular:mainfrom
clydin:refactor/deprecate-buffer-helpers
May 8, 2026
Merged

refactor(@angular-devkit/core): deprecate stringToFileBuffer and fileBufferToString#33150
clydin merged 1 commit intoangular:mainfrom
clydin:refactor/deprecate-buffer-helpers

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented May 8, 2026

DEPRECATED: stringToFileBuffer and fileBufferToString are deprecated. Use standard Web APIs (TextEncoder and TextDecoder) instead.

Internal usages within the repository have been removed and replaced with standard Web APIs. The public API golden file for @angular-devkit/core has been updated to reflect the deprecations.

@clydin clydin added the target: major This PR is targeted for the next major release label May 8, 2026
@angular-robot angular-robot Bot added detected: deprecation PR contains a commit with a deprecation area: @angular-devkit/core labels May 8, 2026
@clydin clydin marked this pull request as ready for review May 8, 2026 02:31
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request deprecates the stringToFileBuffer and fileBufferToString utility functions in favor of native TextEncoder and TextDecoder APIs. The changes involve updating numerous test files and builders across the repository to use these native interfaces directly. Feedback was provided regarding the createWorkspaceHost implementation, suggesting that TextEncoder and TextDecoder instances should be reused to avoid redundant allocations during file operations.

Comment thread packages/angular_devkit/core/src/workspace/host.ts
…BufferToString

DEPRECATED: `stringToFileBuffer` and `fileBufferToString` are deprecated. Use standard Web APIs (`TextEncoder` and `TextDecoder`) instead.

Internal usages within the repository have been removed and replaced with standard Web APIs. The public API golden file for `@angular-devkit/core` has been updated to reflect the deprecations.
@clydin clydin force-pushed the refactor/deprecate-buffer-helpers branch from 747a281 to 0ae5a9c Compare May 8, 2026 02:45
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label May 8, 2026
@clydin clydin requested a review from alan-agius4 May 8, 2026 02:45
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels May 8, 2026
@clydin clydin merged commit fd336d3 into angular:main May 8, 2026
32 of 36 checks passed
@clydin
Copy link
Copy Markdown
Member Author

clydin commented May 8, 2026

This PR was merged into the repository. The changes were merged into the following branches:

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

Labels

action: merge The PR is ready for merge by the caretaker area: @angular-devkit/core detected: deprecation PR contains a commit with a deprecation target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants