refactor(dotprompt-dart): rename dart handlebarrz package to handlebars_dart (#542)

Author: yesudeep

.github/labeler.yml

@@ -87,10 +87,10 @@ dotprompt-dart:
       - any-glob-to-any-file:
           - "dart/dotprompt/**"
 
-handlebarrz-dart:
+handlebars-dart:
   - changed-files:
       - any-glob-to-any-file:
-          - "dart/handlebarrz/**"
+          - "dart/handlebars_dart/**"
 
 js:
   - changed-files:
@@ -177,31 +177,31 @@ npm:
 
 # Automatically add labels to any PR also based on branch naming conventions.
 build:
-  - head-branch: ['^.*/build/.*', 'build']
+  - head-branch: ["^.*/build/.*", "build"]
 
 chore:
-  - head-branch: ['^.*/chore/.*', 'chore']
+  - head-branch: ["^.*/chore/.*", "chore"]
 
 ci:
-  - head-branch: ['^.*/ci/.*', 'ci']
+  - head-branch: ["^.*/ci/.*", "ci"]
 
 feature:
-  - head-branch: ['^.*/feat/.*', 'feature']
+  - head-branch: ["^.*/feat/.*", "feature"]
 
 fix:
-  - head-branch: ['^.*/fix/.*', 'fix']
+  - head-branch: ["^.*/fix/.*", "fix"]
 
 perf:
-  - head-branch: ['^.*/perf/.*', 'perf']
+  - head-branch: ["^.*/perf/.*", "perf"]
 
 refactor:
-  - head-branch: ['^.*/refactor/.*', 'refactor']
+  - head-branch: ["^.*/refactor/.*", "refactor"]
 
 style:
-  - head-branch: ['^.*/style/.*', 'style']
+  - head-branch: ["^.*/style/.*", "style"]
 
 test:
-  - head-branch: ['^.*/test/.*', 'test']
+  - head-branch: ["^.*/test/.*", "test"]
 
 # ==============================================================================
 # RELEASE PLEASE LABELS
@@ -210,48 +210,47 @@ test:
 # Branch pattern: release-please--<component>--<path>
 
 release:
-  - head-branch: ['^release-please--.*']
+  - head-branch: ["^release-please--.*"]
 
 # Component-specific release labels
 release/dotprompt:
-  - head-branch: ['^release-please--dotprompt.*']
+  - head-branch: ["^release-please--dotprompt.*"]
 
 release/dotpromptz:
-  - head-branch: ['^release-please--dotpromptz--.*']
+  - head-branch: ["^release-please--dotpromptz--.*"]
 
 release/dotpromptz-handlebars:
-  - head-branch: ['^release-please--dotpromptz-handlebars--.*']
+  - head-branch: ["^release-please--dotpromptz-handlebars--.*"]
 
 release/dotprompt-go:
-  - head-branch: ['^release-please--dotprompt-go.*']
+  - head-branch: ["^release-please--dotprompt-go.*"]
 
 release/dotprompt-rs:
-  - head-branch: ['^release-please--dotprompt-rs.*']
+  - head-branch: ["^release-please--dotprompt-rs.*"]
 
 release/dotprompt-java:
-  - head-branch: ['^release-please--dotprompt-java.*']
+  - head-branch: ["^release-please--dotprompt-java.*"]
 
 release/vscode:
-  - head-branch: ['^release-please--dotprompt-vscode.*']
+  - head-branch: ["^release-please--dotprompt-vscode.*"]
 
 release/vim:
-  - head-branch: ['^release-please--dotprompt-vim.*']
+  - head-branch: ["^release-please--dotprompt-vim.*"]
 
 release/emacs:
-  - head-branch: ['^release-please--dotprompt-emacs.*']
+  - head-branch: ["^release-please--dotprompt-emacs.*"]
 
 release/jetbrains:
-  - head-branch: ['^release-please--dotprompt-jetbrains.*']
+  - head-branch: ["^release-please--dotprompt-jetbrains.*"]
 
 release/monaco:
-  - head-branch: ['^release-please--dotprompt-monaco.*']
+  - head-branch: ["^release-please--dotprompt-monaco.*"]
 
 release/codemirror:
-  - head-branch: ['^release-please--dotprompt-codemirror.*']
+  - head-branch: ["^release-please--dotprompt-codemirror.*"]
 
 release/treesitter:
-  - head-branch: ['^release-please--tree-sitter-dotprompt.*']
+  - head-branch: ["^release-please--tree-sitter-dotprompt.*"]
 
 release/promptly:
-  - head-branch: ['^release-please--promptly.*']
-
+  - head-branch: ["^release-please--promptly.*"]

.github/workflows/dart.yml

@@ -142,7 +142,7 @@ jobs:
 
       - name: Run Dart tests via Bazel
         run: bazel test //dart/... --test_output=errors
-        continue-on-error: true  # Tests may need network access
+        continue-on-error: true # Tests may need network access
 
   antlr-grammar:
     needs: check-paths
@@ -156,8 +156,8 @@ jobs:
       - name: Setup Java (required for ANTLR)
         uses: actions/setup-java@v4
         with:
-          distribution: 'temurin'
-          java-version: '17'
+          distribution: "temurin"
+          java-version: "17"
 
       - name: Install ANTLR4
         run: |
@@ -180,10 +180,10 @@ jobs:
 
           # Compare with committed files (excluding headers)
           for file in HandlebarsLexer.dart HandlebarsParser.dart HandlebarsParserVisitor.dart HandlebarsParserBaseVisitor.dart; do
-            if [ -f "../../dart/handlebarrz/lib/src/antlr/$file" ]; then
+            if [ -f "../../dart/handlebars_dart/lib/src/antlr/$file" ]; then
               # Strip header comments and compare
               tail -n +22 "$file" > "/tmp/generated_$file"
-              tail -n +22 "../../dart/handlebarrz/lib/src/antlr/$file" > "/tmp/committed_$file"
+              tail -n +22 "../../dart/handlebars_dart/lib/src/antlr/$file" > "/tmp/committed_$file"
               if ! diff -q "/tmp/generated_$file" "/tmp/committed_$file" > /dev/null; then
                 echo "Warning: Generated $file differs from committed version"
                 echo "Run './scripts/generate_handlebars_parser' to update"

bazel/dart/defs.bzl

@@ -134,9 +134,9 @@ chmod +x $@
             "//dart/dotprompt:dotprompt",
             "//dart/dotprompt:pubspec.yaml",
             "//dart/dotprompt:test/spec_test.dart",
-            # Include handlebarrz as it's a path dependency
-            "//dart/handlebarrz:handlebarrz",
-            "//dart/handlebarrz:pubspec.yaml",
+            # Include handlebars_dart as it's a path dependency
+            "//dart/handlebars_dart:handlebars_dart",
+            "//dart/handlebars_dart:pubspec.yaml",
         ],
         tags = kwargs.pop("tags", []) + ["requires-network"],
         **kwargs

dart.MODULE.bazel

@@ -42,7 +42,7 @@ dart_deps.from_file(
     rules_dart_label = "@rules_dart//:defs.bzl",
 )
 dart_deps.from_file(
-    lock_file = "//dart/handlebarrz:pubspec.lock",
+    lock_file = "//dart/handlebars_dart:pubspec.lock",
     rules_dart_label = "@rules_dart//:defs.bzl",
 )
 use_repo(

dart/dotprompt/BUILD.bazel

@@ -35,7 +35,7 @@ dart_library(
         "@dart_deps_meta//:meta",
         "@dart_deps_path//:path",
         "@dart_deps_yaml//:yaml",
-        "@dotprompt//dart/handlebarrz",
+        "@dotprompt//dart/handlebars_dart",
     ],
 )
 
@@ -51,28 +51,28 @@ exports_files(["pubspec.lock"])
 
 dart_test(
     name = "dotprompt_test",
-    data = ["//dart/handlebarrz"],
+    data = ["//dart/handlebars_dart"],
     main = "test/dotprompt_test.dart",
     deps = [":dotprompt"],
 )
 
 dart_test(
     name = "parse_test",
-    data = ["//dart/handlebarrz"],
+    data = ["//dart/handlebars_dart"],
     main = "test/parse_test.dart",
     deps = [":dotprompt"],
 )
 
 dart_test(
     name = "picoschema_test",
-    data = ["//dart/handlebarrz"],
+    data = ["//dart/handlebars_dart"],
     main = "test/picoschema_test.dart",
     deps = [":dotprompt"],
 )
 
 dart_test(
     name = "types_test",
-    data = ["//dart/handlebarrz"],
+    data = ["//dart/handlebars_dart"],
     main = "test/types_test.dart",
     deps = [":dotprompt"],
 )
@@ -143,7 +143,7 @@ dart_test(
     name = "spec_test",
     data = [
         "//:MODULE.bazel",
-        "//dart/handlebarrz",
+        "//dart/handlebars_dart",
         "//spec",
     ],
     main = "test/spec_test.dart",

dart/dotprompt/lib/src/dotprompt.dart

@@ -53,7 +53,7 @@ library;
 import "dart:async";
 import "dart:convert";
 
-import "package:handlebarrz/handlebarrz.dart";
+import "package:handlebars_dart/handlebars_dart.dart";
 
 import "error.dart";
 import "helpers/media_helper.dart";

dart/dotprompt/pubspec.lock

@@ -201,10 +201,10 @@ packages:
       url: "https://pub.dev"
     source: hosted
     version: "2.3.2"
-  handlebarrz:
+  handlebars_dart:
     dependency: "direct main"
     description:
-      path: "../handlebarrz"
+      path: "../handlebars_dart"
       relative: true
     source: path
     version: "0.0.1"

dart/dotprompt/pubspec.yaml

@@ -34,7 +34,7 @@ dependencies:
   # Crypto for hashing
   crypto: ^3.0.6
   # Handlebars templating (our pure Dart implementation)
-  handlebarrz: ^0.0.1
+  handlebars_dart: ^0.0.1
   # JSON schema support
   json_schema: ^5.2.2
   # Meta annotations
@@ -46,8 +46,8 @@ dependencies:
 
 # Local development overrides (not included in published package).
 dependency_overrides:
-  handlebarrz:
-    path: ../handlebarrz
+  handlebars_dart:
+    path: ../handlebars_dart
 
 dev_dependencies:
   build_runner: ^2.4.13

dart/handlebarrz/BUILD.bazel dart/handlebars_dart/BUILD.bazeldart/handlebarrz/BUILD.bazel renamed to dart/handlebars_dart/BUILD.bazel

@@ -14,15 +14,15 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
-"""Handlebarrz - A pure Dart implementation of Handlebars template engine."""
+"""handlebars_dart - A pure Dart implementation of Handlebars template engine."""
 
 load("@rules_dart//:defs.bzl", "dart_library", "dart_pub_get", "dart_pub_publish", "dart_test")
 
 package(default_visibility = ["//visibility:public"])
 
-# Core handlebarrz library
+# Core handlebars_dart library
 dart_library(
-    name = "handlebarrz",
+    name = "handlebars_dart",
     srcs = glob(["lib/**/*.dart"]) + ["pubspec.yaml"],
     pubspec = "pubspec.yaml",
     deps = ["@dart_deps_antlr4//:antlr4"],
@@ -38,13 +38,13 @@ exports_files(["pubspec.lock"])
 dart_test(
     name = "antlr_parser_test",
     main = "test/antlr_parser_test.dart",
-    deps = [":handlebarrz"],
+    deps = [":handlebars_dart"],
 )
 
 dart_test(
     name = "handlebars_test",
     main = "test/handlebars_test.dart",
-    deps = [":handlebarrz"],
+    deps = [":handlebars_dart"],
 )
 
 dart_pub_get(