fix the issue with lambda (#56)

Author: SandeepaHWP

examples/app.jac

@@ -152,4 +152,18 @@ cl{
             </p>
         </div>;
     }
+}
+
+walker get_profile {
+    can run with Root entry {
+        report 'ok';
+        tweets.sort(key=lambda t: s);
+        report 'not ok';
+    }
+}
+
+walker:pub get_all_profiles {
+    can run with Root entry {
+        report results;
+    }
 }

src/__tests__/inspectTokenScopes.test.ts

@@ -820,4 +820,64 @@ describe('inspectTokenScopesHandler - Location Based Tests', () => {
             expectToken(result, 153, 11, 14, 'div', ['source.jac', 'meta.jsx.html.jac', 'entity.name.tag.html.jsx.jac']);
         });
     });
+
+    describe('Walker get_profile with lambda in sort (lines 157-169)', () => {
+        test('walker keyword', () => {
+            // walker get_profile {
+            expectToken(result, 157, 1, 7, 'walker', ['source.jac', 'storage.type.class.jac']);
+        });
+
+        test('get_profile walker name', () => {
+            expectToken(result, 157, 8, 19, 'get_profile', ['source.jac', 'entity.name.type.class.jac']);
+        });
+
+        test('can keyword in ability', () => {
+            // can run with Root entry {
+            expectToken(result, 158, 5, 8, 'can', ['source.jac', 'storage.type.function.jac']);
+        });
+
+        test('run as ability name', () => {
+            expectToken(result, 158, 9, 12, 'run', ['source.jac', 'entity.name.function.jac']);
+        });
+
+        test('report keyword before lambda line', () => {
+            // report \'ok\';
+            expectToken(result, 159, 9, 15, 'report', ['source.jac', 'keyword.control.flow.jac']);
+        });
+
+        test('ok string literal', () => {
+            expectToken(result, 159, 17, 19, 'ok', ['source.jac', 'string.quoted.single.jac']);
+        });
+
+        test('lambda keyword inside sort function call', () => {
+            // tweets.sort(key=lambda t: s)
+            expectToken(result, 160, 25, 31, 'lambda', ['source.jac', 'storage.type.function.lambda.jac']);
+        });
+
+        test('report keyword after lambda line is correctly tokenized (regression)', () => {
+            // Critical regression test: before the lambda end-pattern fix, the lambda scope
+            // consumed the closing ) of sort(...), leaving function-arguments scope open and
+            // causing this line to be mis-tokenized.
+            expectToken(result, 161, 9, 15, 'report', ['source.jac', 'keyword.control.flow.jac']);
+        });
+
+        test('not ok string literal after lambda line (regression)', () => {
+            expectToken(result, 161, 17, 23, 'not ok', ['source.jac', 'string.quoted.single.jac']);
+        });
+
+        test('walker keyword for get_all_profiles', () => {
+            // walker:pub get_all_profiles {
+            expectToken(result, 165, 1, 7, 'walker', ['source.jac', 'storage.type.class.jac']);
+        });
+
+        test('pub modifier on walker', () => {
+            expectToken(result, 165, 8, 11, 'pub', ['source.jac', 'storage.modifier.declaration.jac']);
+        });
+
+        test('report results in get_all_profiles', () => {
+            // report results;
+            expectToken(result, 167, 9, 15, 'report', ['source.jac', 'keyword.control.flow.jac']);
+        });
+    });
+
 });

syntaxes/jac.tmLanguage.json

@@ -1822,7 +1822,7 @@
 				{
 					"name": "meta.lambda-function.jac",
 					"begin": "(?x)\n  \\b (lambda) \\b",
-					"end": "({)|(\\n)",
+					"end": "({)|(\\n)|(?=[),])",
 					"beginCaptures": {
 						"1": {
 							"name": "storage.type.function.lambda.jac"

tsconfig.tsbuildinfo

@@ -1 +1 @@
-{"root":["./src/constants.ts","./src/extension.ts","./src/utils.ts","./src/commands/index.ts","./src/commands/inspecttokenscopes.ts","./src/environment/manager.ts","./src/lsp/lsp_manager.ts","./src/test/platformhelper.ts","./src/test/runtest.ts","./src/test/suite/cleanup.integration.test.ts","./src/test/suite/environment.integration.test.ts","./src/test/suite/index.ts","./src/test/suite/lsp.integration.test.ts","./src/test/suite/testutils.ts","./src/utils/envdetection.ts","./src/utils/envversion.ts","./src/utils/jacrunner.ts","./src/visual_debugger/visdbg.ts","./src/webview/visualdebugger.ts"],"version":"5.9.3"}
+{"root":["./src/constants.ts","./src/extension.ts","./src/utils.ts","./src/commands/index.ts","./src/commands/inspectTokenScopes.ts","./src/environment/manager.ts","./src/lsp/lsp_manager.ts","./src/test/platformHelper.ts","./src/test/runTest.ts","./src/test/suite/cleanup.integration.test.ts","./src/test/suite/environment.integration.test.ts","./src/test/suite/index.ts","./src/test/suite/lsp.integration.test.ts","./src/test/suite/testUtils.ts","./src/utils/envDetection.ts","./src/utils/envVersion.ts","./src/utils/jacRunner.ts","./src/visual_debugger/visdbg.ts","./src/webview/visualDebugger.ts"],"version":"5.9.3"}