Replace Handlebars parser with Peggy grammar for @glimmer/syntax

.github/workflows/ci-jobs.yml

@@ -37,8 +37,6 @@ jobs:
         run: pnpm build:types
       - name: Check internal types
         run: pnpm type-check:internals
-      - name: Check @handlebars/parser types
-        run: pnpm type-check:handlebars
       - name: Check published types
         run: pnpm type-check:types
 
@@ -222,7 +220,7 @@ jobs:
           SHOULD_TRANSPILE_FOR_NODE: true
         run: pnpm build
       - name: test
-        run: pnpm test:node && pnpm --filter "@handlebars/parser" test
+        run: pnpm test:node
 
   blueprint-test:
     name: Blueprint Tests

.github/workflows/glimmer-syntax-prettier-smoke-test.yml

@@ -17,7 +17,6 @@ on:
       - "packages/@glimmer/interfaces/**"
       - "packages/@glimmer/util/**"
       - "packages/@glimmer/wire-format/**"
-      - "packages/@handlebars/parser/**"
   pull_request:
     paths:
       - ".github/workflows/glimmer-syntax-prettier-smoke-test.yml"
@@ -27,7 +26,6 @@ on:
       - "packages/@glimmer/interfaces/**"
       - "packages/@glimmer/util/**"
       - "packages/@glimmer/wire-format/**"
-      - "packages/@handlebars/parser/**"
   workflow_dispatch:
 
 permissions:

.prettierignore

@@ -7,8 +7,6 @@ docs/
 **/smoke-tests/
 **/types/
 **/type-tests/
-packages/@handlebars/**/*.l
-packages/@handlebars/**/*.yy
 CHANGELOG.md
 package.json
 pnpm-lock.yaml

eslint.config.mjs

@@ -28,8 +28,6 @@ export default [
       '**/type-tests/',
       'internal-docs/guides/**',
       'packages/@glimmer-workspace/**',
-      'packages/@handlebars/parser/lib/parser.js',
-      'packages/@handlebars/parser/src/**',
       'tracerbench-testing/',
     ],
   },
@@ -189,15 +187,6 @@ export default [
       'import/namespace': 'off',
     },
   },
-  {
-    files: ['packages/@handlebars/parser/spec/**/*.js'],
-
-    languageOptions: {
-      globals: {
-        ...globals.mocha,
-      },
-    },
-  },
   {
     files: [
       'packages/*/tests/**/*.[jt]s',

internal-docs/guides/development/build-constraints.md

@@ -137,7 +137,6 @@ The build system has specific rules for what gets inlined vs treated as external
 - TypeScript helper library (`tslib`)
 
 **Always External:**
-- `@handlebars/parser`
 - `simple-html-tokenizer`
 - `babel-plugin-debug-macros`
 - Other `@glimmer/*` packages (to avoid duplication)

package.json

@@ -55,7 +55,6 @@
     "test:browserstack": "node bin/run-browserstack-tests.js",
     "test:wip": "vite build --mode development --minify false && testem ci",
     "type-check:internals": "tsc --noEmit",
-    "type-check:handlebars": "tsc --noEmit --project packages/@handlebars/parser/tsconfig.json",
     "type-check:types": "tsc --noEmit --project type-tests",
     "type-check": "npm-run-all type-check:*"
   },

packages/@glimmer/syntax/lib/hbs-parser/exception.js

@@ -0,0 +1,18 @@
+export default class Exception extends Error {
+  constructor(message, node) {
+    let loc = node && node.loc;
+
+    if (loc) {
+      message += ' - ' + loc.start.line + ':' + loc.start.column;
+    }
+
+    super(message);
+
+    if (loc) {
+      this.lineNumber = loc.start.line;
+      this.endLineNumber = loc.end.line;
+      this.column = loc.start.column;
+      this.endColumn = loc.end.column;
+    }
+  }
+}

packages/@glimmer/syntax/lib/hbs-parser/index.js

@@ -0,0 +1 @@
+export { parse, parseWithoutProcessing } from './parse.js';

packages/@glimmer/syntax/lib/hbs-parser/parse.js

@@ -0,0 +1,18 @@
+import { rdParse } from './rd-parser.js';
+import WhitespaceControl from './whitespace-control.js';
+
+export function parseWithoutProcessing(input, options) {
+  // Just return if an already-compiled AST was passed in.
+  if (input.type === 'Program') {
+    return input;
+  }
+
+  return rdParse(input, options);
+}
+
+export function parse(input, options) {
+  let ast = parseWithoutProcessing(input, options);
+  let strip = new WhitespaceControl(options);
+
+  return strip.accept(ast);
+}