Skip to content

Conversation

clydin
Copy link
Member

@clydin clydin commented Aug 1, 2025

This commit introduces a number of significant improvements to the core utility functions, with a primary focus on enhancing the robustness and developer experience of JSONFile.

The JSONFile class now automatically detects and preserves file indentation, uses native private fields for better encapsulation, and includes a new delete() method for a more intuitive API.

Additionally, the readAndParseJson and parseJson functions are now generic for improved type-safety. The assertIsError utility is more robust, and a type-related bug in workspace configuration validation has been fixed. All related utilities have been updated with comprehensive JSDoc comments.

This commit introduces a number of significant improvements to the core utility functions, with a primary focus on enhancing the robustness and developer experience of `JSONFile`.

The `JSONFile` class now automatically detects and preserves file indentation, uses native private fields for better encapsulation, and includes a new `delete()` method for a more intuitive API.

Additionally, the `readAndParseJson` and `parseJson` functions are now generic for improved type-safety. The `assertIsError` utility is more robust, and a type-related bug in workspace configuration validation has been fixed. All related utilities have been updated with comprehensive JSDoc comments.
@clydin clydin added the target: minor This PR is targeted for the next minor release label Aug 1, 2025
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Aug 1, 2025
@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 Aug 11, 2025
@alan-agius4 alan-agius4 merged commit 659e5bd into angular:main Aug 11, 2025
32 checks passed
@clydin clydin deleted the cli/json-file-refactor branch August 11, 2025 13:16
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/cli target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants