dart-lang/sdk
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

[messages] Remove utility to convert to camelCase.

Browse Source 
Now that diagnostic codes in `messages.yaml` files are required to be
`camelCase`, this utility isn't needed anymore.

Change-Id: I6a6a69646f93abf677abd46d53db4c17945c2c6f
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/467100
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Paul Berry <paulberry@google.com>
This commit is contained in:
Paul Berry
2025-12-09 14:16:14 -08:00
committed by Commit Queue
parent e6f054221c
commit 5501288b37
2 changed files with 0 additions and 89 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/analyzer_utilities/pubspec.yaml
-1
View File
@@ -21,6 +21,5 @@ dependencies:
# Use 'any' constraints here; we get our versions from the DEPS file.
dev_dependencies:
analyzer_plugin: any
lints: any
test_reflective_loader: any
pkg/analyzer_utilities/tool/messages/switch_to_camel_case.dart
-88
View File
@@ -1,88 +0,0 @@
// Copyright (c) 2025, the Dart project authors. Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
/// This is a temporary utility script that modifies error codes in
/// `messages.yaml` files, changing analyzer error codes from UPPER_SNAKE_CASE
/// to camelCase.
library;
import 'dart:io';
import 'package:analyzer_plugin/protocol/protocol_common.dart';
import 'package:analyzer_utilities/analyzer_messages.dart';
import 'package:analyzer_utilities/extensions/string.dart';
import 'package:analyzer_utilities/lint_messages.dart';
import 'package:analyzer_utilities/messages.dart';
import 'package:collection/collection.dart';
import 'package:source_span/source_span.dart';
import 'package:yaml/yaml.dart';
void main() {
Map<Uri, Converter> converters = {};
for (var message in [
...analyzerMessages,
...analysisServerMessages,
...feAnalyzerSharedMessages,
...lintMessages,
...frontEndMessages,
]) {
var sourceUrl = message.keySpan.sourceUrl;
(converters[sourceUrl!] ??= Converter(
File(sourceUrl.toFilePath()),
)).convertMessage(message);
}
for (var converter in converters.values) {
converter.apply();
}
}
class Converter {
final File file;
late final isAnalyzerFormat = switch (Uri.file(file.path).pathSegments) {
[..., '_fe_analyzer_shared', 'messages.yaml'] => false,
[..., 'analyzer', 'messages.yaml'] => true,
[..., 'analysis_server', 'messages.yaml'] => true,
[..., 'linter', 'messages.yaml'] => true,
_ => throw 'Unexpected path ${file.path}',
};
late final String content = file.readAsStringSync();
late final yaml = loadYamlNode(content) as YamlMap;
late final messageMaps = isAnalyzerFormat ? yaml.nodes.values : [yaml];
late final Map<String, YamlMap> messageNodesByName = {
for (var messageMap in messageMaps)
for (var MapEntry(:key, :value) in (messageMap as YamlMap).nodes.entries)
(key as YamlScalar).value as String: value as YamlMap,
};
final List<SourceEdit> edits = [];
Converter(this.file);
void apply() {
edits.sortBy((e) => -e.offset);
file.writeAsStringSync(SourceEdit.applySequence(content, edits));
}
void convertMessage(Message message) {
if (message is AnalyzerMessage && !message.keyString.isCamelCase) {
_edit(message.keySpan, message.keyString.toCamelCase());
} else if (message is CfeStyleMessage) {
_edit(message.keySpan, message.keyString.toSnakeCase().toCamelCase());
}
if (message is SharedMessage) {
var messageNode = yaml.nodes[message.keyString] as YamlMap;
_edit(
messageNode.nodes['analyzerCode']!.span,
message.analyzerCode.camelCaseName,
);
}
if (message.sharedName case var sharedName?) {
var messageNode = messageNodesByName[message.keyString] as YamlMap;
_edit(messageNode.nodes['sharedName']!.span, sharedName.camelCaseName);
}
}
void _edit(SourceSpan span, String newText) {
edits.add(SourceEdit(span.start.offset, span.length, newText));
}
}