@halleyassist/rule-templater 0.0.3 → 0.0.5

package/README.md

@@ -27,8 +27,8 @@ const parsed = RuleTemplate.parse(template);
 const variables = parsed.extractVariables();
 console.log(variables);
 // [
-//   { name: 'EVENT_TYPE', filters: [] },
-//   { name: 'THRESHOLD', filters: [] }
+//   { name: 'EVENT_TYPE', filters: [], positions: [{ start: 8, end: 21 }] },
+//   { name: 'THRESHOLD', filters: [], positions: [{ start: 36, end: 48 }] }
 // ]
 
 // Validate that variables are provided correctly
@@ -57,8 +57,8 @@ const parsed = RuleTemplate.parse(template);
 // Extract variables
 const variables = parsed.extractVariables();
 // [
-//   { name: 'ACTION', filters: [] },
-//   { name: 'TIME', filters: [] }
+//   { name: 'ACTION', filters: [], positions: [{ start: 48, end: 57 }] },
+//   { name: 'TIME', filters: [], positions: [{ start: 142, end: 149 }] }
 // ]
 
 // Prepare with values
@@ -79,7 +79,7 @@ const template = 'EventIs(${EVENT_TYPE|upper})';
 
 const parsed = RuleTemplate.parse(template);
 const variables = parsed.extractVariables();
-// [{ name: 'EVENT_TYPE', filters: ['upper'] }]
+// [{ name: 'EVENT_TYPE', filters: ['upper'], positions: [{ start: 8, end: 27 }] }]
 
 // Prepare with filters applied
 const prepared = parsed.prepare({
@@ -154,6 +154,11 @@ Extracts all variables from the template using the AST.
 **Returns:** Array of objects with:
 - `name` (string): The variable name
 - `filters` (array): Array of filter names applied to the variable
+- `positions` (array): Array of position objects, each with:
+  - `start` (number): Zero-based start index of the variable in the template string
+  - `end` (number): Zero-based end index of the variable in the template string
+
+Note: If a variable appears multiple times in the template, all occurrences will be recorded in the `positions` array.
 
 ### `ruleTemplate.validate(variables)`
 

package/dist/rule-templater.browser.js

@@ -2032,7 +2032,7 @@ Object.defineProperty(exports, "ParsingError", { enumerable: true, get: function
 exports.Grammars = require("./Grammars");
 
 },{"./Grammars":5,"./Parser":6,"./ParsingError":7,"./TokenError":9}],11:[function(require,module,exports){
-module.exports=[{"name":"TEMPLATE_BEGIN","bnf":[["\"${\""]]},{"name":"TEMPLATE_END","bnf":[["\"}\""]]},{"name":"PIPE","bnf":[["\"|\""]]},{"name":"%IDENT[2]","bnf":[[/[A-Za-z0-9_]/]]},{"name":"IDENT","bnf":[[/[A-Za-z_]/,"%IDENT[2]*"]]},{"name":"DOT","bnf":[["\".\""]]},{"name":"template_value","bnf":[["TEMPLATE_BEGIN","WS*","template_expr","WS*","TEMPLATE_END"]]},{"name":"%template_expr[2]","bnf":[["WS*","template_pipe","WS*","template_filter_call"]],"fragment":true},{"name":"template_expr","bnf":[["template_path","%template_expr[2]*"]]},{"name":"template_pipe","bnf":[["PIPE"]]},{"name":"%template_path[2]","bnf":[["WS*","DOT","WS*","IDENT"]],"fragment":true},{"name":"template_path","bnf":[["IDENT","%template_path[2]*"]]},{"name":"%template_filter_call[2]","bnf":[["WS*","BEGIN_ARGUMENT","WS*","template_filter_args?","WS*","END_ARGUMENT"]],"fragment":true},{"name":"template_filter_call","bnf":[["template_filter_name","%template_filter_call[2]?"]]},{"name":"template_filter_name","bnf":[["IDENT"]]},{"name":"%template_filter_args[2]","bnf":[["WS*","\",\"","WS*","template_filter_arg"]],"fragment":true},{"name":"template_filter_args","bnf":[["template_filter_arg","%template_filter_args[2]*"]]},{"name":"template_filter_arg","bnf":[["value"],["template_value"]]},{"name":"number_atom","bnf":[["number"],["template_value"]]},{"name":"number_time_atom","bnf":[["number_time"],["template_value"]]},{"name":"tod_atom","bnf":[["number_tod"],["template_value"]]},{"name":"dow_atom","bnf":[["dow"],["template_value"]]}]
+module.exports=[{"name":"TEMPLATE_BEGIN","bnf":[["\"${\""]]},{"name":"TEMPLATE_END","bnf":[["\"}\""]]},{"name":"PIPE","bnf":[["\"|\""]]},{"name":"%IDENT[2]","bnf":[[/[A-Za-z0-9_]/]]},{"name":"IDENT","bnf":[[/[A-Za-z_]/,"%IDENT[2]*"]]},{"name":"DOT","bnf":[["\".\""]]},{"name":"template_value","bnf":[["TEMPLATE_BEGIN","WS*","template_expr","WS*","TEMPLATE_END"]]},{"name":"%template_expr[2]","bnf":[["WS*","template_pipe","WS*","template_filter_call"]],"fragment":true},{"name":"template_expr","bnf":[["template_path","%template_expr[2]*"]]},{"name":"template_pipe","bnf":[["PIPE"]]},{"name":"%template_path[2]","bnf":[["WS*","DOT","WS*","IDENT"]],"fragment":true},{"name":"template_path","bnf":[["IDENT","%template_path[2]*"]]},{"name":"%template_filter_call[2]","bnf":[["WS*","BEGIN_ARGUMENT","WS*","template_filter_args?","WS*","END_ARGUMENT"]],"fragment":true},{"name":"template_filter_call","bnf":[["template_filter_name","%template_filter_call[2]?"]]},{"name":"template_filter_name","bnf":[["IDENT"]]},{"name":"%template_filter_args[2]","bnf":[["WS*","\",\"","WS*","template_filter_arg"]],"fragment":true},{"name":"template_filter_args","bnf":[["template_filter_arg","%template_filter_args[2]*"]]},{"name":"template_filter_arg","bnf":[["value"],["template_value"]]},{"name":"number_atom","bnf":[["number"],["template_value"]]},{"name":"number_time_atom","bnf":[["number_time"],["template_value"]]},{"name":"tod_atom","bnf":[["number_tod"],["template_value"]]},{"name":"dow_atom","bnf":[["dow"],["template_value"]]},{"name":"between_time_only_atom","bnf":[["between_time_only"],["template_value"]]},{"name":"between_tod_only_atom","bnf":[["between_tod_only"],["template_value"]]}]
 },{}],12:[function(require,module,exports){
 module.exports = require('./RuleTemplater.production.js');
 },{"./RuleTemplater.production.js":13}],13:[function(require,module,exports){
@@ -2113,11 +2113,11 @@ class RuleTemplate {
 
     /**
      * Extract variables from the template using the AST
-     * @returns {Array} Array of {name, filters: []} objects
+     * @returns {Array} Array of {name, filters: [], positions: [{start, end}]} objects
      */
     extractVariables(){
         const variables = [];
-        const seen = new Set();
+        const variableMap = new Map();
         
         const traverse = (node) => {
             if (!node) return;
@@ -2126,9 +2126,24 @@ class RuleTemplate {
             if (node.type === 'template_value') {
                 // Extract the variable information
                 const varInfo = this._extractVariableFromNode(node);
-                if (varInfo && !seen.has(varInfo.name)) {
-                    seen.add(varInfo.name);
-                    variables.push(varInfo);
+                if (varInfo) {
+                    // Add position to existing variable or create new entry
+                    if (variableMap.has(varInfo.name)) {
+                        const existing = variableMap.get(varInfo.name);
+                        existing.positions.push({
+                            start: varInfo.start,
+                            end: varInfo.end
+                        });
+                    } else {
+                        variableMap.set(varInfo.name, {
+                            name: varInfo.name,
+                            filters: varInfo.filters,
+                            positions: [{
+                                start: varInfo.start,
+                                end: varInfo.end
+                            }]
+                        });
+                    }
                 }
             }
             
@@ -2141,6 +2156,12 @@ class RuleTemplate {
         };
         
         traverse(this.ast);
+        
+        // Convert map to array
+        for (const variable of variableMap.values()) {
+            variables.push(variable);
+        }
+        
         return variables;
     }
 
@@ -2172,7 +2193,11 @@ class RuleTemplate {
             }
         }
         
-        return { name, filters };
+        // Extract position information from the node
+        const start = node.start;
+        const end = node.end;
+        
+        return { name, filters, start, end };
     }
 
     /**

package/index.d.ts

@@ -1,6 +1,12 @@
+export interface VariablePosition {
+    start: number;
+    end: number;
+}
+
 export interface VariableInfo {
     name: string;
     filters: string[];
+    positions: VariablePosition[];
 }
 
 export interface VariableValue {
@@ -89,3 +95,24 @@ export default class RuleTemplate {
 export const ParserRules: any[];
 export const VariableTypes: string[];
 export const TemplateFilters: TemplateFiltersType;
+
+export interface IParsingErrorPosition {
+    offset: number;
+    line: number;
+    column: number;
+}
+
+export interface IFailureTreeNode {
+    name: string;
+    expected?: string | RegExp;
+    children?: IFailureTreeNode[];
+}
+
+export class ParsingError extends Error {
+    readonly position: IParsingErrorPosition;
+    readonly expected: string[];
+    readonly found: string;
+    readonly failureTree?: IFailureTreeNode[];
+    constructor(message: string, position: IParsingErrorPosition, expected: string[], found: string, failureTree?: IFailureTreeNode[]);
+    toString(): string;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@halleyassist/rule-templater",
-  "version": "0.0.3",
+  "version": "0.0.5",
   "description": "The grammar for HalleyAssist rules",
   "main": "src/RuleTemplater.production.js",
   "browser": "./dist/rule-templater.browser.js",

package/src/RuleTemplate.ebnf.js

@@ -27,6 +27,8 @@ const grammar = `
     number_time_atom     ::= number_time | template_value
     tod_atom             ::= number_tod | template_value
     dow_atom             ::= dow | template_value
+    between_time_only_atom ::= between_time_only | template_value
+    between_tod_only_atom  ::= between_tod_only | template_value
 `
 
 module.exports = Grammars.W3C.getRules(grammar);

package/src/RuleTemplate.production.ebnf.js

@@ -1 +1 @@
-module.exports=[{"name":"TEMPLATE_BEGIN","bnf":[["\"${\""]]},{"name":"TEMPLATE_END","bnf":[["\"}\""]]},{"name":"PIPE","bnf":[["\"|\""]]},{"name":"%IDENT[2]","bnf":[[/[A-Za-z0-9_]/]]},{"name":"IDENT","bnf":[[/[A-Za-z_]/,"%IDENT[2]*"]]},{"name":"DOT","bnf":[["\".\""]]},{"name":"template_value","bnf":[["TEMPLATE_BEGIN","WS*","template_expr","WS*","TEMPLATE_END"]]},{"name":"%template_expr[2]","bnf":[["WS*","template_pipe","WS*","template_filter_call"]],"fragment":true},{"name":"template_expr","bnf":[["template_path","%template_expr[2]*"]]},{"name":"template_pipe","bnf":[["PIPE"]]},{"name":"%template_path[2]","bnf":[["WS*","DOT","WS*","IDENT"]],"fragment":true},{"name":"template_path","bnf":[["IDENT","%template_path[2]*"]]},{"name":"%template_filter_call[2]","bnf":[["WS*","BEGIN_ARGUMENT","WS*","template_filter_args?","WS*","END_ARGUMENT"]],"fragment":true},{"name":"template_filter_call","bnf":[["template_filter_name","%template_filter_call[2]?"]]},{"name":"template_filter_name","bnf":[["IDENT"]]},{"name":"%template_filter_args[2]","bnf":[["WS*","\",\"","WS*","template_filter_arg"]],"fragment":true},{"name":"template_filter_args","bnf":[["template_filter_arg","%template_filter_args[2]*"]]},{"name":"template_filter_arg","bnf":[["value"],["template_value"]]},{"name":"number_atom","bnf":[["number"],["template_value"]]},{"name":"number_time_atom","bnf":[["number_time"],["template_value"]]},{"name":"tod_atom","bnf":[["number_tod"],["template_value"]]},{"name":"dow_atom","bnf":[["dow"],["template_value"]]}]
+module.exports=[{"name":"TEMPLATE_BEGIN","bnf":[["\"${\""]]},{"name":"TEMPLATE_END","bnf":[["\"}\""]]},{"name":"PIPE","bnf":[["\"|\""]]},{"name":"%IDENT[2]","bnf":[[/[A-Za-z0-9_]/]]},{"name":"IDENT","bnf":[[/[A-Za-z_]/,"%IDENT[2]*"]]},{"name":"DOT","bnf":[["\".\""]]},{"name":"template_value","bnf":[["TEMPLATE_BEGIN","WS*","template_expr","WS*","TEMPLATE_END"]]},{"name":"%template_expr[2]","bnf":[["WS*","template_pipe","WS*","template_filter_call"]],"fragment":true},{"name":"template_expr","bnf":[["template_path","%template_expr[2]*"]]},{"name":"template_pipe","bnf":[["PIPE"]]},{"name":"%template_path[2]","bnf":[["WS*","DOT","WS*","IDENT"]],"fragment":true},{"name":"template_path","bnf":[["IDENT","%template_path[2]*"]]},{"name":"%template_filter_call[2]","bnf":[["WS*","BEGIN_ARGUMENT","WS*","template_filter_args?","WS*","END_ARGUMENT"]],"fragment":true},{"name":"template_filter_call","bnf":[["template_filter_name","%template_filter_call[2]?"]]},{"name":"template_filter_name","bnf":[["IDENT"]]},{"name":"%template_filter_args[2]","bnf":[["WS*","\",\"","WS*","template_filter_arg"]],"fragment":true},{"name":"template_filter_args","bnf":[["template_filter_arg","%template_filter_args[2]*"]]},{"name":"template_filter_arg","bnf":[["value"],["template_value"]]},{"name":"number_atom","bnf":[["number"],["template_value"]]},{"name":"number_time_atom","bnf":[["number_time"],["template_value"]]},{"name":"tod_atom","bnf":[["number_tod"],["template_value"]]},{"name":"dow_atom","bnf":[["dow"],["template_value"]]},{"name":"between_time_only_atom","bnf":[["between_time_only"],["template_value"]]},{"name":"between_tod_only_atom","bnf":[["between_tod_only"],["template_value"]]}]

package/src/RuleTemplater.js

@@ -75,11 +75,11 @@ class RuleTemplate {
 
     /**
      * Extract variables from the template using the AST
-     * @returns {Array} Array of {name, filters: []} objects
+     * @returns {Array} Array of {name, filters: [], positions: [{start, end}]} objects
      */
     extractVariables(){
         const variables = [];
-        const seen = new Set();
+        const variableMap = new Map();
         
         const traverse = (node) => {
             if (!node) return;
@@ -88,9 +88,24 @@ class RuleTemplate {
             if (node.type === 'template_value') {
                 // Extract the variable information
                 const varInfo = this._extractVariableFromNode(node);
-                if (varInfo && !seen.has(varInfo.name)) {
-                    seen.add(varInfo.name);
-                    variables.push(varInfo);
+                if (varInfo) {
+                    // Add position to existing variable or create new entry
+                    if (variableMap.has(varInfo.name)) {
+                        const existing = variableMap.get(varInfo.name);
+                        existing.positions.push({
+                            start: varInfo.start,
+                            end: varInfo.end
+                        });
+                    } else {
+                        variableMap.set(varInfo.name, {
+                            name: varInfo.name,
+                            filters: varInfo.filters,
+                            positions: [{
+                                start: varInfo.start,
+                                end: varInfo.end
+                            }]
+                        });
+                    }
                 }
             }
             
@@ -103,6 +118,12 @@ class RuleTemplate {
         };
         
         traverse(this.ast);
+        
+        // Convert map to array
+        for (const variable of variableMap.values()) {
+            variables.push(variable);
+        }
+        
         return variables;
     }
 
@@ -134,7 +155,11 @@ class RuleTemplate {
             }
         }
         
-        return { name, filters };
+        // Extract position information from the node
+        const start = node.start;
+        const end = node.end;
+        
+        return { name, filters, start, end };
     }
 
     /**

package/src/RuleTemplater.production.js

@@ -75,11 +75,11 @@ class RuleTemplate {
 
     /**
      * Extract variables from the template using the AST
-     * @returns {Array} Array of {name, filters: []} objects
+     * @returns {Array} Array of {name, filters: [], positions: [{start, end}]} objects
      */
     extractVariables(){
         const variables = [];
-        const seen = new Set();
+        const variableMap = new Map();
         
         const traverse = (node) => {
             if (!node) return;
@@ -88,9 +88,24 @@ class RuleTemplate {
             if (node.type === 'template_value') {
                 // Extract the variable information
                 const varInfo = this._extractVariableFromNode(node);
-                if (varInfo && !seen.has(varInfo.name)) {
-                    seen.add(varInfo.name);
-                    variables.push(varInfo);
+                if (varInfo) {
+                    // Add position to existing variable or create new entry
+                    if (variableMap.has(varInfo.name)) {
+                        const existing = variableMap.get(varInfo.name);
+                        existing.positions.push({
+                            start: varInfo.start,
+                            end: varInfo.end
+                        });
+                    } else {
+                        variableMap.set(varInfo.name, {
+                            name: varInfo.name,
+                            filters: varInfo.filters,
+                            positions: [{
+                                start: varInfo.start,
+                                end: varInfo.end
+                            }]
+                        });
+                    }
                 }
             }
             
@@ -103,6 +118,12 @@ class RuleTemplate {
         };
         
         traverse(this.ast);
+        
+        // Convert map to array
+        for (const variable of variableMap.values()) {
+            variables.push(variable);
+        }
+        
         return variables;
     }
 
@@ -134,7 +155,11 @@ class RuleTemplate {
             }
         }
         
-        return { name, filters };
+        // Extract position information from the node
+        const start = node.start;
+        const end = node.end;
+        
+        return { name, filters, start, end };
     }
 
     /**