@appland/scanner 1.87.0 → 1.87.1

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [@appland/scanner-v1.87.1](https://github.com/getappmap/appmap-js/compare/@appland/scanner-v1.87.0...@appland/scanner-v1.87.1) (2024-07-25)
+
+
+### Bug Fixes
+
+* conditionally create sql_warnings.txt ([737f59f](https://github.com/getappmap/appmap-js/commit/737f59f323b3c9300bc6985cb179fa7264c8f7fd))
+
 # [@appland/scanner-v1.87.0](https://github.com/getappmap/appmap-js/compare/@appland/scanner-v1.86.0...@appland/scanner-v1.87.0) (2024-04-30)
 
 

package/built/sqlWarning.js

@@ -17,12 +17,22 @@ function writeErrorToFile(error) {
     return __awaiter(this, void 0, void 0, function* () {
         const flags = SqlParseErrorFileOpened ? 'a' : 'w';
         SqlParseErrorFileOpened = true;
-        (0, promises_1.open)(SqlParseErrorFileName, flags).then((handle) => {
-            handle.write([error.toString(), ''].join('\n')).finally(handle.close.bind(handle));
-        });
+        const msg = [String(error), ''].join('\n');
+        try {
+            (0, promises_1.open)(SqlParseErrorFileName, flags).then((handle) => {
+                handle.write([error.toString(), ''].join('\n')).finally(handle.close.bind(handle));
+            });
+        }
+        catch (e) {
+            console.warn(e);
+            console.warn(`SQL Error: ${msg}`);
+        }
     });
 }
 function sqlWarning(parseError) {
+    if (!process.env['APPMAP_SQL_WARNING']) {
+        return;
+    }
     if (!SqlErrors.has(parseError.sql)) {
         writeErrorToFile(parseError);
         SqlErrors.add(parseError.sql);

package/built/telemetry.js

@@ -368,12 +368,14 @@ class GitProperties {
     }
 }
 const gitCache = new Map();
+const noCacheList = ['clearCache'];
 // GitProperties is available externally as Git.
 // This export provides a simple caching layer around GitProperties to avoid
 // excessive shelling out to git.
 exports.Git = new Proxy(GitProperties, {
     get(target, prop) {
-        if (typeof target[prop] === 'function') {
+        if (!noCacheList.includes(prop.toString()) &&
+            typeof target[prop] === 'function') {
             return new Proxy(target[prop], {
                 apply(target, thisArg, argArray) {
                     const cacheKey = `${prop.toString()}(${JSON.stringify(argArray)})`;
@@ -382,12 +384,7 @@ exports.Git = new Proxy(GitProperties, {
                     }
                     /* eslint-disable-next-line @typescript-eslint/ban-types */
                     const result = Reflect.apply(target, thisArg, argArray);
-                    if (result instanceof Promise) {
-                        return result.then((r) => {
-                            gitCache.set(cacheKey, r);
-                            return r;
-                        });
-                    }
+                    gitCache.set(cacheKey, result);
                     return result;
                 },
             });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@appland/scanner",
-  "version": "1.87.0",
+  "version": "1.87.1",
   "description": "Analyze AppMaps for code flaws",
   "bin": "built/cli.js",
   "main": "built/index.js",
@@ -19,7 +19,7 @@
     "doc-up-to-date": "git diff --exit-code doc/",
     "lint": "eslint src --ext .ts",
     "ci": "yarn lint && yarn build && yarn schema-up-to-date && yarn doc-up-to-date && yarn test",
-    "test": "jest --filter=./test/testFilter.js --detectOpenHandles",
+    "test": "yarn jest",
     "jest": "jest --filter=./test/testFilter.js --detectOpenHandles",
     "semantic-release": "semantic-release",
     "watch": "node bin/preBuild.js && tsc -p tsconfig.build.json --watch"