feat(parser): add function boundary and parameter parsing

Author: loganch

src/parsers/MapParser.js

@@ -5,6 +5,7 @@ export default class MapParser {
  constructor(source) {
  this.source = source;
  this.lines = source.split('\n');
+ this.functions = []; // Add this
  }
 
  /**
@@ -69,4 +70,59 @@ export default class MapParser {
 
  return assignments;
  }
+
+ /**
+ * Parse function boundaries and parameters
+ * @returns {Array<{name: string, startLine: number, endLine: number, parameters: string[]}>}
+ */
+ parseFunctionBoundaries() {
+ this.functions = [];
+ const funcStartPattern = /^\s*Func\s+(\w+)\s*\((.*?)\)/i;
+ const funcEndPattern = /^\s*EndFunc/i;
+
+ let currentFunc = null;
+
+ this.lines.forEach((line, index) => {
+ const funcStart = line.match(funcStartPattern);
+ if (funcStart) {
+ // Parse parameters
+ const paramsStr = funcStart[2].trim();
+ const parameters = paramsStr
+ ? paramsStr.split(',').map(p => p.trim().split('=')[0].trim())
+ : [];
+
+ currentFunc = {
+ name: funcStart[1],
+ startLine: index,
+ endLine: -1,
+ parameters
+ };
+ return;
+ }
+
+ const funcEnd = line.match(funcEndPattern);
+ if (funcEnd && currentFunc) {
+ currentFunc.endLine = index;
+ this.functions.push(currentFunc);
+ currentFunc = null;
+ }
+ });
+
+ return this.functions;
+ }
+
+ /**
+ * Get the function containing a specific line
+ * @param {number} line - Line number
+ * @returns {object|null} Function object or null
+ */
+ getFunctionAtLine(line) {
+ if (this.functions.length === 0) {
+ this.parseFunctionBoundaries();
+ }
+
+ return this.functions.find(func =>
+ line >= func.startLine && line <= func.endLine
+ ) || null;
+ }
 }

test/parsers/MapParser.test.js

@@ -106,4 +106,71 @@ $mOther.key = "value"`;
  expect(assignments[0].key).toBe('name');
  });
  });
+
+ describe('parseFunctionBoundaries', () => {
+ it('should detect function start and end', () => {
+ const source = `Func MyFunction()
+ Local $x = 1
+EndFunc`;
+ const parser = new MapParser(source);
+ const functions = parser.parseFunctionBoundaries();
+
+ expect(functions).toHaveLength(1);
+ expect(functions[0]).toMatchObject({
+ name: 'MyFunction',
+ startLine: 0,
+ endLine: 2,
+ });
+ });
+
+ it('should detect multiple functions', () => {
+ const source = `Func First()
+EndFunc
+
+Func Second()
+ Local $x = 1
+EndFunc`;
+ const parser = new MapParser(source);
+ const functions = parser.parseFunctionBoundaries();
+
+ expect(functions).toHaveLength(2);
+ expect(functions[0].name).toBe('First');
+ expect(functions[1].name).toBe('Second');
+ });
+
+ it('should detect function parameters', () => {
+ const source = `Func ProcessUser($mUser, $name)
+EndFunc`;
+ const parser = new MapParser(source);
+ const functions = parser.parseFunctionBoundaries();
+
+ expect(functions).toHaveLength(1);
+ expect(functions[0].parameters).toEqual(['$mUser', '$name']);
+ });
+ });
+
+ describe('getFunctionAtLine', () => {
+ it('should return function containing the line', () => {
+ const source = `Func MyFunction()
+ Local $x = 1
+EndFunc`;
+ const parser = new MapParser(source);
+ parser.parseFunctionBoundaries();
+ const func = parser.getFunctionAtLine(1);
+
+ expect(func).toBeDefined();
+ expect(func.name).toBe('MyFunction');
+ });
+
+ it('should return null for line outside functions', () => {
+ const source = `Func MyFunction()
+EndFunc
+Local $x = 1`;
+ const parser = new MapParser(source);
+ parser.parseFunctionBoundaries();
+ const func = parser.getFunctionAtLine(2);
+
+ expect(func).toBeNull();
+ });
+ });
 });