autorepair: do not repair "out" in "South Carolina"

Fixes #695. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Regular_expressions/Word_boundary_assertion
tyrasd · Jun 12, 2024 · bee7a10 · bee7a10
1 parent be0f345
commit bee7a10
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/js/autorepair.ts b/js/autorepair.ts
@@ -140,11 +140,11 @@ export default function autorepair(q, lng) {
           "$1xml$3/*fixed by auto repair*/"
         );
       // 2. fix print statements: non meta output, overpass geometries
-      const prints = q.match(/(\.([^;.]+?)\s+)?(out[^:;"\]]*;)/g) || [];
+      const prints = q.match(/(\.([^;.]+?)\s+)?(\bout\b[^:;"\]]*;)/g) || [];
       for (const print_i of prints) {
         // eslint-disable-next-line prefer-const
         let [print, , out_set, out_statement] = print_i.match(
-          /(\.([^;.]+?)\s+)?(out[^:;"\]]*;)/
+          /(\.([^;.]+?)\s+)?(\bout[^:;"\]]*;)/
         );
         let new_print = print;
         let new_out_statement;

diff --git a/tests/test.autorepair.josm.ts b/tests/test.autorepair.josm.ts
@@ -121,6 +121,10 @@ describe("ide.autorepair.josm", () => {
         // more complex real world example
         inp: '/*bla*/\n[out:xml];\nway\n  ["amenity"]\n  ({{bbox}})\n->.foo;\n.foo out qt;',
         outp: '/*bla*/\n[out:xml];\nway\n  ["amenity"]\n  ({{bbox}})\n->.foo;\n.foo out meta qt;/*fixed by auto repair*/'
+      },
+      {
+        inp: "{{geocodeArea:South Carolina}}->.searchArea;",
+        outp: "{{geocodeArea:South Carolina}}->.searchArea;"
       }
     ];
     vi.spyOn(ide, "getQueryLang").mockImplementation(() => "OverpassQL");