@pgpmjs/ast 0.7.1 → 0.9.1

package/esm/plan-rename.js

@@ -1,28 +1,34 @@
-function escapeRe(str) {
-    return str.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-}
 /**
  * Rewrite change names in a pgpm.plan's content according to a rename map.
  * Textual and format-preserving: only the change-name token at the start of a
  * change line, plain (same-package) dependency refs inside `[...]`, and tag
  * lines' change field are rewritten; timestamps, planners, comments,
  * cross-package refs, and tag refs are untouched.
+ *
+ * Runs in a single linear pass: each line is scanned once and every
+ * whitespace/bracket-delimited token is looked up in the rename map directly,
+ * so the rewrite is O(content length) rather than O(lines × renames). A token
+ * qualifies when it is bounded on the left by start-of-line, whitespace or `[`
+ * and on the right by end-of-line, whitespace, `]` or `@` — which is exactly
+ * the change-name and plain-dependency positions. Cross-package refs
+ * (`pkg:change`) are left untouched because their change segment is preceded by
+ * `:`, and a trailing `@tag` suffix is preserved because the token ends at `@`.
  */
 export function renameInPlanContent(content, renames) {
-    const froms = Array.from(renames.keys()).sort((a, b) => b.length - a.length);
+    if (renames.size === 0)
+        return content;
+    // One regex, compiled once per call (not per line or per rename).
+    const tokenRe = /(^|[\s\[])([^\s\[\]@]+)(?=$|[\s\]@])/g;
     return content
         .split('\n')
         .map(line => {
         const trimmed = line.trim();
         if (!trimmed || trimmed.startsWith('#') || trimmed.startsWith('%'))
             return line;
-        let result = line;
-        for (const from of froms) {
-            const to = renames.get(from);
-            const re = new RegExp(`(^|[\\s\\[])${escapeRe(from)}(?=$|[\\s\\]@])`, 'g');
-            result = result.replace(re, `$1${to}`);
-        }
-        return result;
+        return line.replace(tokenRe, (match, pre, token) => {
+            const to = renames.get(token);
+            return to === undefined ? match : `${pre}${to}`;
+        });
     })
         .join('\n');
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pgpmjs/ast",
-  "version": "0.7.1",
+  "version": "0.9.1",
   "author": "Constructive <developers@constructive.io>",
   "description": "PGPM module AST — plan/control/SQL-header parsers and the lossless module read/write layer",
   "main": "index.js",
@@ -41,8 +41,8 @@
     "makage": "^0.3.0"
   },
   "dependencies": {
-    "@pgpmjs/env": "^2.35.1",
-    "@pgpmjs/types": "^2.43.1"
+    "@pgpmjs/env": "^2.37.0",
+    "@pgpmjs/types": "^2.45.0"
   },
-  "gitHead": "37e83eb66e9d13d0d4d8ac3b36b5ebe394b84a43"
+  "gitHead": "d85911699e8a50a9fdfeb7506195442525c1f238"
 }

package/plan-rename.d.ts

@@ -4,5 +4,14 @@
  * change line, plain (same-package) dependency refs inside `[...]`, and tag
  * lines' change field are rewritten; timestamps, planners, comments,
  * cross-package refs, and tag refs are untouched.
+ *
+ * Runs in a single linear pass: each line is scanned once and every
+ * whitespace/bracket-delimited token is looked up in the rename map directly,
+ * so the rewrite is O(content length) rather than O(lines × renames). A token
+ * qualifies when it is bounded on the left by start-of-line, whitespace or `[`
+ * and on the right by end-of-line, whitespace, `]` or `@` — which is exactly
+ * the change-name and plain-dependency positions. Cross-package refs
+ * (`pkg:change`) are left untouched because their change segment is preceded by
+ * `:`, and a trailing `@tag` suffix is preserved because the token ends at `@`.
  */
 export declare function renameInPlanContent(content: string, renames: Map<string, string>): string;

package/plan-rename.js

@@ -1,31 +1,37 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.renameInPlanContent = renameInPlanContent;
-function escapeRe(str) {
-    return str.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-}
 /**
  * Rewrite change names in a pgpm.plan's content according to a rename map.
  * Textual and format-preserving: only the change-name token at the start of a
  * change line, plain (same-package) dependency refs inside `[...]`, and tag
  * lines' change field are rewritten; timestamps, planners, comments,
  * cross-package refs, and tag refs are untouched.
+ *
+ * Runs in a single linear pass: each line is scanned once and every
+ * whitespace/bracket-delimited token is looked up in the rename map directly,
+ * so the rewrite is O(content length) rather than O(lines × renames). A token
+ * qualifies when it is bounded on the left by start-of-line, whitespace or `[`
+ * and on the right by end-of-line, whitespace, `]` or `@` — which is exactly
+ * the change-name and plain-dependency positions. Cross-package refs
+ * (`pkg:change`) are left untouched because their change segment is preceded by
+ * `:`, and a trailing `@tag` suffix is preserved because the token ends at `@`.
  */
 function renameInPlanContent(content, renames) {
-    const froms = Array.from(renames.keys()).sort((a, b) => b.length - a.length);
+    if (renames.size === 0)
+        return content;
+    // One regex, compiled once per call (not per line or per rename).
+    const tokenRe = /(^|[\s\[])([^\s\[\]@]+)(?=$|[\s\]@])/g;
     return content
         .split('\n')
         .map(line => {
         const trimmed = line.trim();
         if (!trimmed || trimmed.startsWith('#') || trimmed.startsWith('%'))
             return line;
-        let result = line;
-        for (const from of froms) {
-            const to = renames.get(from);
-            const re = new RegExp(`(^|[\\s\\[])${escapeRe(from)}(?=$|[\\s\\]@])`, 'g');
-            result = result.replace(re, `$1${to}`);
-        }
-        return result;
+        return line.replace(tokenRe, (match, pre, token) => {
+            const to = renames.get(token);
+            return to === undefined ? match : `${pre}${to}`;
+        });
     })
         .join('\n');
 }