Merge pull request #13 from hildjj/warning

Add warning if running in debug mode to prevent confusion when a peg$debugger statement is left in by mistake.

Author: hildjj

lib/index.js

@@ -184,7 +184,7 @@ function checkParserStarts(grammar, starts, modified, counts) {
           startRule,
           ...options,
         });
-        fail("Expected error but none received");
+        fail(`Expected error but none received for invalidInput: "${start.invalidInput}"`);
       } catch (er) {
         if (!(er instanceof grammar.SyntaxError)) {
           throw er;
@@ -424,9 +424,14 @@ export async function testPeggy(grammarUrl, starts, opts) {
     const withMap = src.toStringWithSourceMap();
     const map = Buffer.from(withMap.map.toString()).toString("base64");
     let code = withMap.code;
-    const sm = (opts && Object.prototype.hasOwnProperty.call(opts, "noMap"))
-      ? !opts.noMap
-      : starts.every(s => !s.options?.peg$debugger);
+    let sm = starts.every(s => !s.options?.peg$debugger);
+    if (opts && Object.prototype.hasOwnProperty.call(opts, "noMap")) {
+      sm = !opts.noMap;
+    } else {
+      if (!sm) {
+        console.error("WARNING: sourcemap disabled due to peg$debugger");
+      }
+    }
     const start = "//#"; // c8: THIS file is not mapped.
     if (sm) {
       code += `

package.json

@@ -40,7 +40,7 @@
     "typedoc": "0.26.11",
     "typescript": "5.6.3"
   },
-  "packageManager": "[email protected]",
+  "packageManager": "[email protected]+sha512.6e2baf77d06b9362294152c851c4f278ede37ab1eba3a55fda317a4a17b209f4dbb973fb250a77abc463a341fcb1f17f17cfa24091c4eb319cda0d9b84278387",
   "pnpm": {
     "overrides": {
       "@eslint/plugin-kit": "^0.2.3",

test/index.test.js

@@ -186,3 +186,19 @@ test("thows errors", async() => {
     },
   ]));
 });
+
+test("peg$debugger", async() => {
+  const old = console.error;
+  const stderr = [];
+  console.error = (...args) => stderr.push(args);
+  await testPeggy(MIN, [
+    {
+      validInput: "foo",
+      options: {
+        peg$debugger: true,
+      },
+    },
+  ]);
+  console.error = old;
+  deepEqual(stderr, [["WARNING: sourcemap disabled due to peg$debugger"]]);
+});