Migrate data source from rules.json to messages.yaml #9

r0227n · 2025-03-19T09:30:02Z

Issue

close [Improve]: Migrate rules.json to messages.yaml #5

Overview (Required)

This pull request includes several changes to the tools/update_lint_rules package, focusing on updating the LintRule model, adding a new JSON converter for State, and modifying the AnalysisOptionsService to use the new State type. The most important changes include the introduction of the State type, the addition of new methods to the RuleState enum, and the updates to the AnalysisOptionsService to filter rules based on the new State type.

Updates to `LintRule` model:

Added nullable fields for categories and details, and replaced RuleState with the new State type in the Rule factory constructor.
Introduced the State type as a typedef for Map<RuleState, Since> and added the _StateJsonConverter class to handle JSON serialization and deserialization for the State type.

Enhancements to `RuleState` enum:

Added internal state to the RuleState enum.
Introduced active and inactive getter methods to determine the activity status of a rule state.

Changes to `AnalysisOptionsService`:

Added an extension method hasSupportedVersion to the State type to check if a rule state meets a minimum Dart SDK version.
Updated the filtering logic in AnalysisOptionsService to use the new State type and its hasSupportedVersion method.

Codebase simplification:

Removed unnecessary fields and methods related to description, incompatibles, sets, fixStatus, and since from the Rule class and its related classes and methods.
Updated the JSON serialization and deserialization logic in lint_rule.g.dart to reflect the changes in the Rule class.

Links

Screenshot

N/A

github-actions · 2025-03-19T09:30:15Z

Ready for review 🚀

blendthink

@r0227n
I've left some comments, so please take a look. 🙏

tools/update_lint_rules/lib/src/services/lint_rule_service.dart

tools/update_lint_rules/lib/src/models/lint_rule.dart

r0227n · 2025-03-21T09:26:55Z

@blendthink
Please re-review.
When comparing 3.6/all.yaml before and after the fix, package_api_docs and unsafe_html_attribute are

Before: not included
After modification: included

but we are aware that this is not a problem.

package_api_docs

Before: rules.json

https://github.com/dart-lang/sdk/blob/ae6da8b926f208bf87d2e11375be5c611c27ee1b/pkg/linter/tool/machine/rules.json#L1625-L1638
The value of state key was not output before the correction because it was removed.

After: message.yaml

https://github.com/dart-lang/sdk/blob/ae6da8b926f208bf87d2e11375be5c611c27ee1b/pkg/linter/messages.yaml#L7394-L7399

Because “removed” is 3.7 or later, it is now output in 3.6.

unsafe_html

Before modification: rules.json

Not output because it is not described in the rules.json.

After: message.yaml

https://github.com/dart-lang/sdk/blob/ae6da8b926f208bf87d2e11375be5c611c27ee1b/pkg/linter/messages.yaml#L13330-L13336

The output is output because it is described.

r0227n · 2025-04-02T04:37:55Z

@blendthink
Please re-review.🙇

… to JSON conversion process

…JSON conversion

…rule retrieval process

…ML to JSON within LintRuleService

Rename sourceRuleWithCategories to ruleWithCategories to make the code more concise while maintaining clarity about the variable's role in handling shared rule categories.

…dName and merge data after encoding.

tools/update_lint_rules/lib/src/mappers/lint_code_dto_mapper.dart

r0227n · 2025-04-16T01:41:54Z

@blendthink
Please re-review.🙇

blendthink

@r0227n
Please take a look and leave your comments or make any necessary changes.

tools/update_lint_rules/lib/src/mappers/rule_mapper.dart

tools/update_lint_rules/lib/src/services/analysis_options_service.dart

tools/update_lint_rules/test/src/mappers/rule_mapper_test.dart

…d sharedName null”.

tools/update_lint_rules/lib/src/services/lint_rule_service.dart

r0227n · 2025-04-18T03:14:10Z

@blendthink
Please re-review.🙇

https://github.com/yumemi-inc/flutter-yumemi-lints/pull/163/files

This is the latest diff as of 9ed9b39.
We would appreciate it if you could check it at the time of review.

r0227n · 2025-04-22T07:38:02Z

@blendthink
Added to CHANGELOG.md, “Reflect Status” and “remove incompatibles”.
Please re-review🙇

blendthink

LGTM!

github-actions bot assigned r0227n Mar 19, 2025

github-actions bot requested review from blendthink, K9i-0, morikann, trm11tkr, Yamasaki-pan961 and YumNumm March 19, 2025 09:30

blendthink reviewed Mar 19, 2025

View reviewed changes

tools/update_lint_rules/lib/src/services/lint_rule_service.dart Outdated Show resolved Hide resolved

tools/update_lint_rules/lib/src/services/lint_rule_service.dart Outdated Show resolved Hide resolved

tools/update_lint_rules/lib/src/models/lint_rule.dart Outdated Show resolved Hide resolved

r0227n and others added 19 commits April 10, 2025 12:33

feat: Change Rule property for yaml

4b904de

feat: Change information source from json to yaml

0a2c195

chor: Move convertToJsonFromYaml method into getRules method

c3f612c

fix: Add dependencies and remove ignore

755424d

fix: state and categories no longer output non-existent values.

032279a

chore: Minimized roop processing

17c63a1

add: convertToJsonFromYaml method test code

25b02fd

refactor: simplify convertToJsonFromYaml method and optimize YAML map…

a4a2a98

… to JSON conversion process

feat: add toJson extension method to YamlMap to simplify YAML map to …

cf2a1ea

…JSON conversion

fix: add exception handling for non-YamlMap entry values and improve …

2e51eff

…rule retrieval process

test: add unit tests for YamlMap's toJson method

67c16ff

refactor: remove convertToJsonFromYaml method and directly convert YA…

3e1ee91

…ML to JSON within LintRuleService

fix: typo ( exsistsCategories → existsCategories )

4ee2737

docs: add example YAML data to explain shared rule categories

67b2bde

refactor: rename variable to better reflect its purpose

aead3af

Rename sourceRuleWithCategories to ruleWithCategories to make the code more concise while maintaining clarity about the variable's role in handling shared rule categories.

docs: add comment explaining rule addition conditions

9188e2c

add: Create DTO for LintCode

d1be457

refactor: Changed to separate processing by presence/absence of Share…

6341baa

…dName and merge data after encoding.

refactor: Sort output results alphabetically

e6214a1

r0227n commented Apr 15, 2025

View reviewed changes

tools/update_lint_rules/lib/src/mappers/lint_code_dto_mapper.dart Outdated Show resolved Hide resolved

r0227n added 5 commits April 16, 2025 09:16

rename: Rename mapper class & method

ff06585

chore: Modified test cases due to function argument changes

7f37d0f

chore: Support for renaming staging leaks

a7c019a

add: Correction of forgetting to add required

973b4a1

chore: The “categories ? []” in the comments for the reason.

8ff0798

blendthink reviewed Apr 16, 2025

View reviewed changes

r0227n added 6 commits April 17, 2025 11:53

chore: Corrected test description

3fc666a

chore: rename comment

18ad3cf

chore: Fixed default values are assigned when categories is null.

c9b9fa1

remove: Remove test case that throws when categories is null.

a067364

remove: Remove the test case “DTO removes elements with categories an…

54d9340

…d sharedName null”.

chore: major version up :3.1.0 -> 4.0.0

e943f45

This was referenced Apr 17, 2025

[Improve]: Handle incompatibles property in update_lint_rules #15

Open

Sample diff creation changes for review yumemi-inc/flutter-yumemi-lints#163

Closed

r0227n added 2 commits April 18, 2025 11:29

fix: Fixed to be able to judge even in a mixed active/inactive state.

29892cb

add: Add _isNotFlutterOnlyRules to unreachable_from_main

9ed9b39

r0227n commented Apr 18, 2025

View reviewed changes

tools/update_lint_rules/lib/src/services/lint_rule_service.dart Show resolved Hide resolved

r0227n added 2 commits April 22, 2025 16:26

add: “Add Status” and “Remove incompatibles” to CHANGELOG

b88cb62

edit: Move Status changes to a smaller level

ab33133

blendthink approved these changes Apr 22, 2025

View reviewed changes

blendthink enabled auto-merge April 23, 2025 07:39

blendthink added this pull request to the merge queue Apr 23, 2025

Merged via the queue into main with commit 6fc2438 Apr 23, 2025
1 check passed

blendthink deleted the improve/update-source-rules branch April 23, 2025 07:40

blendthink mentioned this pull request Apr 23, 2025

Update lint rules yumemi-inc/flutter-yumemi-lints#165

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Migrate data source from rules.json to messages.yaml #9

Migrate data source from rules.json to messages.yaml #9

r0227n commented Mar 19, 2025 •

edited

Loading

github-actions bot commented Mar 19, 2025

blendthink left a comment

r0227n commented Mar 21, 2025 •

edited

Loading

r0227n commented Apr 2, 2025

r0227n commented Apr 16, 2025

blendthink left a comment

r0227n commented Apr 18, 2025

r0227n commented Apr 22, 2025

blendthink left a comment

Migrate data source from rules.json to messages.yaml #9

Migrate data source from rules.json to messages.yaml #9

Conversation

r0227n commented Mar 19, 2025 • edited Loading

Issue

Overview (Required)

Updates to LintRule model:

Enhancements to RuleState enum:

Changes to AnalysisOptionsService:

Codebase simplification:

Links

Screenshot

github-actions bot commented Mar 19, 2025

blendthink left a comment

Choose a reason for hiding this comment

r0227n commented Mar 21, 2025 • edited Loading

package_api_docs

Before: rules.json

After: message.yaml

unsafe_html

Before modification: rules.json

After: message.yaml

r0227n commented Apr 2, 2025

r0227n commented Apr 16, 2025

blendthink left a comment

Choose a reason for hiding this comment

r0227n commented Apr 18, 2025

r0227n commented Apr 22, 2025

blendthink left a comment

Choose a reason for hiding this comment

r0227n commented Mar 19, 2025 •

edited

Loading

Updates to `LintRule` model:

Enhancements to `RuleState` enum:

Changes to `AnalysisOptionsService`:

r0227n commented Mar 21, 2025 •

edited

Loading