@forge/lint 5.1.1-next.5 → 5.1.1-next.6

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @forge/lint
 
+## 5.1.1-next.6
+
+### Patch Changes
+
+- Updated dependencies [670a419]
+  - @forge/manifest@7.3.0-next.5
+  - @forge/cli-shared@5.0.0-next.6
+
 ## 5.1.1-next.5
 
 ### Patch Changes

package/out/lint/abstract-linter.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.AbstractLinter = void 0;
 const cli_shared_1 = require("@forge/cli-shared");
 class AbstractLinter {
+    logger;
     constructor(logger) {
         this.logger = logger;
     }

package/out/lint/base-linter.js

@@ -4,13 +4,15 @@ const cli_shared_1 = require("@forge/cli-shared");
 const linter_interface_1 = require("./linter-interface");
 const abstract_linter_1 = require("./abstract-linter");
 class BaseLinter extends abstract_linter_1.AbstractLinter {
+    environment;
+    logger;
+    nodeVisitors = [];
+    matches = new Map();
+    verifiers = {};
     constructor(environment, logger) {
         super(logger);
         this.environment = environment;
         this.logger = logger;
-        this.nodeVisitors = [];
-        this.matches = new Map();
-        this.verifiers = {};
     }
     getFixer() {
         return undefined;

package/out/lint/lint.js

@@ -105,7 +105,7 @@ const lint = async (filesToLint, manifest, environment, logger, parseFunction =
                 }
             }
             return { ast: null, filepath: '' };
-        }))).filter((result) => !!(result === null || result === void 0 ? void 0 : result.ast));
+        }))).filter((result) => !!result?.ast);
         await Promise.all(linters.map((linter) => linter.bootstrap()));
         const results = await Promise.all(linters.map((linter) => linter.batchExecute(lintInputs))).then(cli_shared_1.flat);
         return results.map((result) => {

package/out/lint/linter-interface.js

@@ -7,11 +7,13 @@ var LintClass;
     LintClass["Warning"] = "warning";
 })(LintClass = exports.LintClass || (exports.LintClass = {}));
 class LintResult {
+    file;
+    fixer;
+    errors = [];
+    warnings = [];
     constructor(file, fixer) {
         this.file = file;
         this.fixer = fixer;
-        this.errors = [];
-        this.warnings = [];
     }
     add(rule) {
         switch (rule.class) {

package/out/lint/linters/dynamic-properties-linter/dynamic-properties-permissions-linter.js

@@ -9,6 +9,7 @@ const confluence_icon_visitor_1 = require("./visitors/confluence-icon-visitor");
 const permission_linter_1 = require("../permission-linter/permission-linter");
 const jira_icon_visitor_1 = require("./visitors/jira-icon-visitor");
 class DynamicPropertiesPermissionsLinter extends base_linter_1.default {
+    manifest;
     constructor(environment, manifest, logger) {
         super(environment, logger);
         this.manifest = manifest;

package/out/lint/linters/dynamic-properties-linter/verifiers/abstract-dynamic-properties-verifier.js

@@ -3,17 +3,16 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.AbstractDynamicPropertiesVerifier = void 0;
 const verifier_interface_1 = require("../../verifier-interface");
 class AbstractDynamicPropertiesVerifier extends verifier_interface_1.BaseLintIssueVerifier {
+    shouldRunCheck = false;
     constructor(environment, manifest) {
         super(environment, manifest);
-        this.shouldRunCheck = false;
         Object.keys(this.manifest.modules || {}).forEach((moduleType) => {
-            var _a, _b;
             switch (moduleType) {
                 case 'confluence:contentBylineItem':
                 case 'jira:issueGlance':
                 case 'jira:issueContext':
                     this.shouldRunCheck =
-                        ((_b = (_a = this.manifest.modules) === null || _a === void 0 ? void 0 : _a[moduleType]) === null || _b === void 0 ? void 0 : _b.find((module) => { var _a; return (_a = module.dynamicProperties) === null || _a === void 0 ? void 0 : _a.function; })) !==
+                        this.manifest.modules?.[moduleType]?.find((module) => module.dynamicProperties?.function) !==
                             undefined;
                     break;
             }

package/out/lint/linters/dynamic-properties-linter/verifiers/dynamic-properties-icon-verifier.js

@@ -7,10 +7,10 @@ const utils_1 = require("../../utils");
 const text_1 = require("../../../text");
 const abstract_dynamic_properties_verifier_1 = require("./abstract-dynamic-properties-verifier");
 class DynamicPropertiesIconVerifier extends abstract_dynamic_properties_verifier_1.AbstractDynamicPropertiesVerifier {
+    egressFilteringService;
     constructor(environment, manifest) {
-        var _a, _b, _c, _d;
         super(environment, manifest);
-        const allowList = (_d = (_c = (_b = (_a = this.manifest) === null || _a === void 0 ? void 0 : _a.permissions) === null || _b === void 0 ? void 0 : _b.external) === null || _c === void 0 ? void 0 : _c.images) !== null && _d !== void 0 ? _d : [];
+        const allowList = this.manifest?.permissions?.external?.images ?? [];
         const collectedUrls = allowList.filter((item) => typeof item === 'string');
         this.egressFilteringService = new egress_1.EgressFilteringService(collectedUrls);
     }

package/out/lint/linters/dynamic-properties-linter/visitors/abstract-icon-visitor.js

@@ -3,39 +3,36 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.AbstractIconVisitor = void 0;
 const typescript_estree_1 = require("@typescript-eslint/typescript-estree");
 class AbstractIconVisitor {
-    constructor() {
-        this.isBundledUri = (iconUri) => iconUri.startsWith('resource:') || iconUri.startsWith('data:image');
-        this.handleIconNode = (iconNode, callback) => {
-            if ((iconNode === null || iconNode === void 0 ? void 0 : iconNode.type) === typescript_estree_1.AST_NODE_TYPES.Literal && typeof (iconNode === null || iconNode === void 0 ? void 0 : iconNode.value) === 'string') {
-                if (iconNode.value.trim() !== '' && !this.isBundledUri(iconNode.value.trim())) {
-                    const iconUrl = this.transformArgsToIconUrl(iconNode);
-                    callback(iconUrl);
-                }
-            }
-        };
-        this.transformArgsToIconUrl = (iconNode) => ({
-            type: 'DYNAMIC_PROPERTIES_ICON',
-            url: iconNode.value,
-            line: iconNode.loc.start.line,
-            column: iconNode.loc.start.column
-        });
-    }
     visit(node, parent, callback) {
-        var _a, _b;
         switch (node.type) {
             case typescript_estree_1.AST_NODE_TYPES.ArrowFunctionExpression:
-                if (((_a = node.body) === null || _a === void 0 ? void 0 : _a.type) === typescript_estree_1.AST_NODE_TYPES.ObjectExpression) {
+                if (node.body?.type === typescript_estree_1.AST_NODE_TYPES.ObjectExpression) {
                     const iconDefinitionNode = this.getIconDefinitionNode(node.body);
-                    this.handleIconNode(iconDefinitionNode === null || iconDefinitionNode === void 0 ? void 0 : iconDefinitionNode.value, callback);
+                    this.handleIconNode(iconDefinitionNode?.value, callback);
                 }
                 break;
             case typescript_estree_1.AST_NODE_TYPES.ReturnStatement:
-                if (((_b = node.argument) === null || _b === void 0 ? void 0 : _b.type) === typescript_estree_1.AST_NODE_TYPES.ObjectExpression) {
+                if (node.argument?.type === typescript_estree_1.AST_NODE_TYPES.ObjectExpression) {
                     const iconDefinitionNode = this.getIconDefinitionNode(node.argument);
-                    this.handleIconNode(iconDefinitionNode === null || iconDefinitionNode === void 0 ? void 0 : iconDefinitionNode.value, callback);
+                    this.handleIconNode(iconDefinitionNode?.value, callback);
                 }
                 break;
         }
     }
+    isBundledUri = (iconUri) => iconUri.startsWith('resource:') || iconUri.startsWith('data:image');
+    handleIconNode = (iconNode, callback) => {
+        if (iconNode?.type === typescript_estree_1.AST_NODE_TYPES.Literal && typeof iconNode?.value === 'string') {
+            if (iconNode.value.trim() !== '' && !this.isBundledUri(iconNode.value.trim())) {
+                const iconUrl = this.transformArgsToIconUrl(iconNode);
+                callback(iconUrl);
+            }
+        }
+    };
+    transformArgsToIconUrl = (iconNode) => ({
+        type: 'DYNAMIC_PROPERTIES_ICON',
+        url: iconNode.value,
+        line: iconNode.loc.start.line,
+        column: iconNode.loc.start.column
+    });
 }
 exports.AbstractIconVisitor = AbstractIconVisitor;

package/out/lint/linters/dynamic-properties-linter/visitors/confluence-icon-visitor.js

@@ -4,16 +4,13 @@ exports.ConfluenceIconVisitor = void 0;
 const typescript_estree_1 = require("@typescript-eslint/typescript-estree");
 const abstract_icon_visitor_1 = require("./abstract-icon-visitor");
 class ConfluenceIconVisitor extends abstract_icon_visitor_1.AbstractIconVisitor {
-    constructor() {
-        super(...arguments);
-        this.isIconPropertyDefinition = (node) => {
-            return (node.type === typescript_estree_1.AST_NODE_TYPES.Property &&
-                ((node.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier && node.key.name === 'icon') ||
-                    (node.key.type === typescript_estree_1.AST_NODE_TYPES.Literal && node.key.value === 'icon')));
-        };
-    }
     getIconDefinitionNode(node) {
         return node.properties.find(this.isIconPropertyDefinition);
     }
+    isIconPropertyDefinition = (node) => {
+        return (node.type === typescript_estree_1.AST_NODE_TYPES.Property &&
+            ((node.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier && node.key.name === 'icon') ||
+                (node.key.type === typescript_estree_1.AST_NODE_TYPES.Literal && node.key.value === 'icon')));
+    };
 }
 exports.ConfluenceIconVisitor = ConfluenceIconVisitor;

package/out/lint/linters/dynamic-properties-linter/visitors/jira-icon-visitor.js

@@ -4,40 +4,37 @@ exports.JiraIconVisitor = void 0;
 const typescript_estree_1 = require("@typescript-eslint/typescript-estree");
 const abstract_icon_visitor_1 = require("./abstract-icon-visitor");
 class JiraIconVisitor extends abstract_icon_visitor_1.AbstractIconVisitor {
-    constructor() {
-        super(...arguments);
-        this.isStatusPropertyDefinition = (node) => {
-            return (node.type === typescript_estree_1.AST_NODE_TYPES.Property &&
-                ((node.key.type === typescript_estree_1.AST_NODE_TYPES.Literal && node.key.value === 'status') ||
-                    (node.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier && node.key.name === 'status')) &&
-                node.value.type === typescript_estree_1.AST_NODE_TYPES.ObjectExpression &&
-                node.value.properties.find((prop) => {
-                    return (prop.type === typescript_estree_1.AST_NODE_TYPES.Property &&
-                        ((prop.key.type === typescript_estree_1.AST_NODE_TYPES.Literal && prop.key.value === 'type') ||
-                            (prop.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier && prop.key.name === 'type')) &&
-                        ((prop.value.type === typescript_estree_1.AST_NODE_TYPES.Literal && prop.value.value === 'icon') ||
-                            (prop.value.type === typescript_estree_1.AST_NODE_TYPES.Identifier && prop.value.name === 'icon')));
-                }) !== undefined);
-        };
-        this.extractIconDefinition = (node) => {
-            if (!node) {
-                return undefined;
-            }
-            const valueDefinitionNode = node === null || node === void 0 ? void 0 : node.value.properties.find((prop) => {
-                return (prop.type === typescript_estree_1.AST_NODE_TYPES.Property &&
-                    ((prop.key.type === typescript_estree_1.AST_NODE_TYPES.Literal && prop.key.value === 'value') ||
-                        (prop.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier && prop.key.name === 'value')));
-            });
-            return valueDefinitionNode.value.properties.find((prop) => {
-                return (prop.type === typescript_estree_1.AST_NODE_TYPES.Property &&
-                    ((prop.key.type === typescript_estree_1.AST_NODE_TYPES.Literal && prop.key.value === 'url') ||
-                        (prop.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier && prop.key.name === 'url')));
-            });
-        };
-    }
     getIconDefinitionNode(node) {
         const statusPropertyNode = node.properties.find(this.isStatusPropertyDefinition);
         return this.extractIconDefinition(statusPropertyNode);
     }
+    isStatusPropertyDefinition = (node) => {
+        return (node.type === typescript_estree_1.AST_NODE_TYPES.Property &&
+            ((node.key.type === typescript_estree_1.AST_NODE_TYPES.Literal && node.key.value === 'status') ||
+                (node.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier && node.key.name === 'status')) &&
+            node.value.type === typescript_estree_1.AST_NODE_TYPES.ObjectExpression &&
+            node.value.properties.find((prop) => {
+                return (prop.type === typescript_estree_1.AST_NODE_TYPES.Property &&
+                    ((prop.key.type === typescript_estree_1.AST_NODE_TYPES.Literal && prop.key.value === 'type') ||
+                        (prop.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier && prop.key.name === 'type')) &&
+                    ((prop.value.type === typescript_estree_1.AST_NODE_TYPES.Literal && prop.value.value === 'icon') ||
+                        (prop.value.type === typescript_estree_1.AST_NODE_TYPES.Identifier && prop.value.name === 'icon')));
+            }) !== undefined);
+    };
+    extractIconDefinition = (node) => {
+        if (!node) {
+            return undefined;
+        }
+        const valueDefinitionNode = node?.value.properties.find((prop) => {
+            return (prop.type === typescript_estree_1.AST_NODE_TYPES.Property &&
+                ((prop.key.type === typescript_estree_1.AST_NODE_TYPES.Literal && prop.key.value === 'value') ||
+                    (prop.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier && prop.key.name === 'value')));
+        });
+        return valueDefinitionNode.value.properties.find((prop) => {
+            return (prop.type === typescript_estree_1.AST_NODE_TYPES.Property &&
+                ((prop.key.type === typescript_estree_1.AST_NODE_TYPES.Literal && prop.key.value === 'url') ||
+                    (prop.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier && prop.key.name === 'url')));
+        });
+    };
 }
 exports.JiraIconVisitor = JiraIconVisitor;

package/out/lint/linters/handler-linter/handler-linter.js

@@ -7,10 +7,10 @@ const base_linter_1 = tslib_1.__importDefault(require("../../base-linter"));
 const handlers_verifier_1 = require("./verifiers/handlers-verifier");
 const visitors_1 = require("./visitors");
 class HandlerLinter extends base_linter_1.default {
+    manifest;
     constructor(environment, manifest, logger) {
         super(environment, logger);
         this.manifest = manifest;
-        this.getHandlerFileRegex = (fileName) => new RegExp(`^src(\/|\\\\)${fileName}\.[tj](s|sx)$`);
     }
     async bootstrap() {
         this.nodeVisitors = [new visitors_1.ExportedFunctionNodeVisitor()];
@@ -19,20 +19,23 @@ class HandlerLinter extends base_linter_1.default {
             functionHandlers: new handlers_verifier_1.HandlersVerifier(humanReadableEnvironment, this.manifest)
         };
     }
+    getHandlerFileRegex = (fileName) => new RegExp(`^src(\/|\\\\)${fileName}\.[tj](s|sx)$`);
     getFunctionHandlers(manifest) {
-        var _a, _b, _c;
-        return (_c = (_b = (_a = manifest.modules) === null || _a === void 0 ? void 0 : _a.function) === null || _b === void 0 ? void 0 : _b.filter((funcs) => funcs.handler && funcs.handler.split('.').length === 2)) === null || _c === void 0 ? void 0 : _c.map((funcs) => {
+        return manifest.modules?.function
+            ?.filter((funcs) => funcs.handler && funcs.handler.split('.').length === 2)
+            ?.map((funcs) => {
             const [file, method] = funcs.handler.split('.');
             return {
                 file,
                 method,
                 key: funcs.key
             };
-        }).filter((handler) => handler.method && handler.file);
+        })
+            .filter((handler) => handler.method && handler.file);
     }
     setupMatchesMap(filepath) {
         const handlers = this.getFunctionHandlers(this.manifest);
-        const filteredHandlers = handlers === null || handlers === void 0 ? void 0 : handlers.filter((handler) => this.getHandlerFileRegex(handler.file).test(filepath));
+        const filteredHandlers = handlers?.filter((handler) => this.getHandlerFileRegex(handler.file).test(filepath));
         this.matches.set(filepath, {
             functionHandlers: {
                 expectedMethods: filteredHandlers || [],
@@ -42,7 +45,7 @@ class HandlerLinter extends base_linter_1.default {
     }
     filterLintInput(inputs) {
         const handlers = this.getFunctionHandlers(this.manifest);
-        return inputs.filter((input) => handlers === null || handlers === void 0 ? void 0 : handlers.find((handler) => this.getHandlerFileRegex(handler.file).test(input.filepath)));
+        return inputs.filter((input) => handlers?.find((handler) => this.getHandlerFileRegex(handler.file).test(input.filepath)));
     }
     addLintCriteriaMatch(exportedMethod, filepath) {
         const criteriaMatches = this.matches.get(filepath);

package/out/lint/linters/handler-linter/verifiers/handlers-verifier.js

@@ -9,8 +9,7 @@ class HandlersVerifier extends verifier_interface_1.BaseLintIssueVerifier {
         return linter_interface_1.LintClass.Warning;
     }
     async process(input) {
-        var _a, _b;
-        if (!((_b = (_a = this.manifest) === null || _a === void 0 ? void 0 : _a.modules) === null || _b === void 0 ? void 0 : _b.function)) {
+        if (!this.manifest?.modules?.function) {
             return [];
         }
         const missingMethods = input.expectedMethods.filter((func) => !input.exportedMethods.find((exportedMethod) => exportedMethod.method === func.method));

package/out/lint/linters/handler-linter/visitors/exported-function-node-visitor.js

@@ -23,21 +23,20 @@ class ExportedFunctionNodeVisitor {
     }
     visitSpecifiedNamedExports(node, specifier, callback) {
         if (specifier.type === typescript_estree_1.AST_NODE_TYPES.ExportSpecifier && specifier.local.type === typescript_estree_1.AST_NODE_TYPES.Identifier) {
-            callback(this.createExportedMethod(specifier.exported.name, Object.assign({}, node.loc.start)));
+            callback(this.createExportedMethod(specifier.exported.name, { ...node.loc.start }));
         }
     }
     visitInlineNamedExport(node, callback) {
-        var _a, _b, _c, _d;
-        if (((_a = node.declaration) === null || _a === void 0 ? void 0 : _a.type) === typescript_estree_1.AST_NODE_TYPES.VariableDeclaration) {
+        if (node.declaration?.type === typescript_estree_1.AST_NODE_TYPES.VariableDeclaration) {
             node.declaration.declarations.forEach((declarator) => {
                 if (declarator.id && declarator.id.type === typescript_estree_1.AST_NODE_TYPES.Identifier) {
-                    callback(this.createExportedMethod(declarator.id.name, Object.assign({}, declarator.id.loc.start)));
+                    callback(this.createExportedMethod(declarator.id.name, { ...declarator.id.loc.start }));
                 }
             });
         }
-        else if (((_b = node.declaration) === null || _b === void 0 ? void 0 : _b.type) === typescript_estree_1.AST_NODE_TYPES.FunctionDeclaration) {
-            if (((_c = node.declaration) === null || _c === void 0 ? void 0 : _c.id) && ((_d = node.declaration) === null || _d === void 0 ? void 0 : _d.id.type) === typescript_estree_1.AST_NODE_TYPES.Identifier) {
-                callback(this.createExportedMethod(node.declaration.id.name, Object.assign({}, node.declaration.id.loc.start)));
+        else if (node.declaration?.type === typescript_estree_1.AST_NODE_TYPES.FunctionDeclaration) {
+            if (node.declaration?.id && node.declaration?.id.type === typescript_estree_1.AST_NODE_TYPES.Identifier) {
+                callback(this.createExportedMethod(node.declaration.id.name, { ...node.declaration.id.loc.start }));
             }
         }
     }
@@ -48,21 +47,23 @@ class ExportedFunctionNodeVisitor {
     }
     visitMemberExpression(node, callback) {
         if (node.property.type === typescript_estree_1.AST_NODE_TYPES.Identifier && this.isExports(node.object)) {
-            callback(this.createExportedMethod(node.property.name, Object.assign({}, node.property.loc.start)));
+            callback(this.createExportedMethod(node.property.name, { ...node.property.loc.start }));
         }
     }
     tryAddObjectExpressionPropertiesExportedMethods(node, callback) {
-        var _a, _b;
-        if (((_a = node.right) === null || _a === void 0 ? void 0 : _a.type) === typescript_estree_1.AST_NODE_TYPES.ObjectExpression) {
-            (_b = node.right.properties) === null || _b === void 0 ? void 0 : _b.forEach((property) => {
+        if (node.right?.type === typescript_estree_1.AST_NODE_TYPES.ObjectExpression) {
+            node.right.properties?.forEach((property) => {
                 if (property.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier) {
-                    callback(this.createExportedMethod(property.key.name, Object.assign({}, property.key.loc.start)));
+                    callback(this.createExportedMethod(property.key.name, { ...property.key.loc.start }));
                 }
             });
         }
     }
     createExportedMethod(method, location) {
-        return Object.assign({ method }, location);
+        return {
+            method,
+            ...location
+        };
     }
     isExports(node) {
         return ((node.type === typescript_estree_1.AST_NODE_TYPES.MemberExpression &&
@@ -70,7 +71,7 @@ class ExportedFunctionNodeVisitor {
             node.object.name === 'module' &&
             node.property.type === typescript_estree_1.AST_NODE_TYPES.Identifier &&
             node.property.name === 'exports') ||
-            ((node === null || node === void 0 ? void 0 : node.type) === typescript_estree_1.AST_NODE_TYPES.Identifier && (node === null || node === void 0 ? void 0 : node.name) === 'exports'));
+            (node?.type === typescript_estree_1.AST_NODE_TYPES.Identifier && node?.name === 'exports'));
     }
 }
 exports.ExportedFunctionNodeVisitor = ExportedFunctionNodeVisitor;

package/out/lint/linters/manifest-linter/abstract-manifest-linter.js

@@ -5,14 +5,15 @@ const manifest_1 = require("@forge/manifest");
 const linter_interface_1 = require("../../linter-interface");
 const abstract_linter_1 = require("../../abstract-linter");
 class AbstractManifestLinter extends abstract_linter_1.AbstractLinter {
+    type;
+    processor;
     constructor(type, logger) {
         super(logger);
         this.type = type;
     }
     mapManifestResponse(results, fixer) {
-        var _a;
         const manifestLintResults = new linter_interface_1.LintResult(manifest_1.MANIFEST_FILE, fixer);
-        manifestLintResults.batchAdd(...((_a = this.getResultsForFixer(results).map(({ level, message, line, column, reference, metadata }) => {
+        manifestLintResults.batchAdd(...(this.getResultsForFixer(results).map(({ level, message, line, column, reference, metadata }) => {
             return {
                 class: level,
                 message,
@@ -21,7 +22,7 @@ class AbstractManifestLinter extends abstract_linter_1.AbstractLinter {
                 column,
                 metadata
             };
-        })) !== null && _a !== void 0 ? _a : []).sort((a, b) => a.line - b.line));
+        }) ?? []).sort((a, b) => a.line - b.line));
         return manifestLintResults;
     }
     getProcessor() {
@@ -42,8 +43,7 @@ class AbstractManifestLinter extends abstract_linter_1.AbstractLinter {
         return undefined;
     }
     getResultsForFixer(results) {
-        var _a;
-        return ((_a = results.errors) === null || _a === void 0 ? void 0 : _a.filter((error) => !error.metadata)) || [];
+        return results.errors?.filter((error) => !error.metadata) || [];
     }
 }
 exports.AbstractManifestLinter = AbstractManifestLinter;

package/out/lint/linters/manifest-linter/deprecated-csp-permissions-manifest-linter.js

@@ -12,8 +12,7 @@ class DeprecatedCspPermissionsManifestLinter extends abstract_manifest_linter_1.
         return permission_linter_1.fixMissingPermissions;
     }
     getResultsForFixer(results) {
-        var _a;
-        return ((_a = results.errors) === null || _a === void 0 ? void 0 : _a.filter((error) => { var _a; return (_a = error.metadata) === null || _a === void 0 ? void 0 : _a.missingContentStylePermission; })) || [];
+        return results.errors?.filter((error) => error.metadata?.missingContentStylePermission) || [];
     }
 }
 exports.DeprecatedCspPermissionsManifestLinter = DeprecatedCspPermissionsManifestLinter;

package/out/lint/linters/manifest-linter/permissions-manifest-linter.js

@@ -12,8 +12,7 @@ class PermissionsManifestLinter extends abstract_manifest_linter_1.AbstractManif
         return permission_linter_1.fixMissingPermissions;
     }
     getResultsForFixer(results) {
-        var _a;
-        return ((_a = results.errors) === null || _a === void 0 ? void 0 : _a.filter((error) => { var _a; return (_a = error.metadata) === null || _a === void 0 ? void 0 : _a.missingPermission; })) || [];
+        return results.errors?.filter((error) => error.metadata?.missingPermission) || [];
     }
 }
 exports.PermissionsManifestLinter = PermissionsManifestLinter;

package/out/lint/linters/manifest-linter/remote-compute-manifest-linter.js

@@ -12,8 +12,7 @@ class RemoteComputeManifestLinter extends abstract_manifest_linter_1.AbstractMan
         return permission_linter_1.fixMissingPermissions;
     }
     getResultsForFixer(results) {
-        var _a;
-        return ((_a = results.errors) === null || _a === void 0 ? void 0 : _a.filter((error) => { var _a; return (_a = error.metadata) === null || _a === void 0 ? void 0 : _a.missingPermission; })) || [];
+        return results.errors?.filter((error) => error.metadata?.missingPermission) || [];
     }
 }
 exports.RemoteComputeManifestLinter = RemoteComputeManifestLinter;

package/out/lint/linters/permission-linter/permission-linter.js

@@ -32,47 +32,70 @@ const findMissingPermissions = (errors, warnings, permissionsType, state) => {
 };
 exports.findMissingPermissions = findMissingPermissions;
 const fixMissingPermissions = async (errors, warnings, state) => {
-    var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o;
     const missingScopes = (0, exports.findMissingPermissions)(errors, warnings, 'missingPermission', state);
     const missingExternalFetchBackend = (0, exports.findMissingPermissions)(errors, warnings, 'missingExternalFetchPermission', state);
     const missingContentStyle = (0, exports.findMissingPermissions)(errors, warnings, 'missingContentStylePermission', state);
     const missingExternalImage = (0, exports.findMissingPermissions)(errors, warnings, 'missingExternalImagesPermission', state);
-    const currentPermissions = (_b = (_a = (await state.configFile.readConfig())) === null || _a === void 0 ? void 0 : _a.permissions) !== null && _b !== void 0 ? _b : {};
-    const currentRemotes = (_d = (_c = (await state.configFile.readConfig())) === null || _c === void 0 ? void 0 : _c.remotes) !== null && _d !== void 0 ? _d : [];
-    const currentScopes = (_e = currentPermissions === null || currentPermissions === void 0 ? void 0 : currentPermissions.scopes) !== null && _e !== void 0 ? _e : [];
+    const currentPermissions = (await state.configFile.readConfig())?.permissions ?? {};
+    const currentRemotes = (await state.configFile.readConfig())?.remotes ?? [];
+    const currentScopes = currentPermissions?.scopes ?? [];
     currentScopes.forEach((e) => missingScopes.add(e));
-    const currentExternalFetchBackend = (_h = (_g = (_f = currentPermissions === null || currentPermissions === void 0 ? void 0 : currentPermissions.external) === null || _f === void 0 ? void 0 : _f.fetch) === null || _g === void 0 ? void 0 : _g.backend) !== null && _h !== void 0 ? _h : [];
+    const currentExternalFetchBackend = currentPermissions?.external?.fetch?.backend ?? [];
     currentExternalFetchBackend
         .map((item) => {
-        var _a;
         if (typeof item === 'string')
             return item;
-        return (_a = currentRemotes.find((remote) => remote.key == item.remote)) === null || _a === void 0 ? void 0 : _a.baseUrl;
+        return currentRemotes.find((remote) => remote.key == item.remote)?.baseUrl;
     })
         .filter((item) => typeof item === 'string')
         .forEach((e) => missingExternalFetchBackend.add(e));
     const arrayMissingExternalFetchBackend = [...missingExternalFetchBackend];
-    const currentContentStyles = (_k = (_j = currentPermissions === null || currentPermissions === void 0 ? void 0 : currentPermissions.content) === null || _j === void 0 ? void 0 : _j.styles) !== null && _k !== void 0 ? _k : [];
+    const currentContentStyles = currentPermissions?.content?.styles ?? [];
     currentContentStyles.forEach((e) => missingContentStyle.add(e));
     const arrayMissingContentStyles = [...missingContentStyle];
-    const currentExternalImages = (_m = (_l = currentPermissions === null || currentPermissions === void 0 ? void 0 : currentPermissions.external) === null || _l === void 0 ? void 0 : _l.images) !== null && _m !== void 0 ? _m : [];
+    const currentExternalImages = currentPermissions?.external?.images ?? [];
     currentExternalImages.forEach((e) => missingExternalImage.add(e));
     const arrayMissingExternalImages = [...missingExternalImage];
-    const config = Object.assign(Object.assign({}, currentPermissions), { scopes: [...missingScopes] });
+    const config = {
+        ...currentPermissions,
+        scopes: [...missingScopes]
+    };
     if (arrayMissingExternalFetchBackend.length > 0) {
-        config.external = Object.assign(Object.assign({}, config.external), { fetch: Object.assign(Object.assign({}, (_o = config.external) === null || _o === void 0 ? void 0 : _o.fetch), { backend: arrayMissingExternalFetchBackend }) });
+        config.external = {
+            ...config.external,
+            fetch: {
+                ...config.external?.fetch,
+                backend: arrayMissingExternalFetchBackend
+            }
+        };
     }
     if (arrayMissingExternalImages.length > 0) {
-        config.external = Object.assign(Object.assign({}, config.external), { images: arrayMissingExternalImages });
+        config.external = {
+            ...config.external,
+            images: arrayMissingExternalImages
+        };
     }
     if (arrayMissingContentStyles.length > 0) {
-        config.content = Object.assign(Object.assign({}, config.content), { styles: arrayMissingContentStyles });
+        config.content = {
+            ...config.content,
+            styles: arrayMissingContentStyles
+        };
     }
     await state.configFile.writeToConfigFile('permissions', config);
     return state;
 };
 exports.fixMissingPermissions = fixMissingPermissions;
 class PermissionLinter extends base_linter_1.default {
+    manifest;
+    static JIRA_CACHE_KEY = 'permissions-jira-cache-key';
+    static JSM_CACHE_KEY = 'permissions-jsm-cache-key';
+    static JSW_CACHE_KEY = 'permissions-jsw-cache-key';
+    static CONFLUENCE_CACHE_KEY = 'permissions-confluence-cache-key';
+    static BITBUCKET_CACHE_KEY = 'permissions-bitbucket-cache-key';
+    jira;
+    confluence;
+    bitbucket;
+    cache;
     constructor(environment, manifest, logger) {
         super(environment, logger);
         this.manifest = manifest;
@@ -110,7 +133,11 @@ class PermissionLinter extends base_linter_1.default {
             ];
         }
         const [jiraSwagger, jsmSwagger, jswSwagger, confluenceSwagger, bitbucketSwagger] = await Promise.all(swaggerFiles);
-        this.jira = this.processPaths(Object.assign(Object.assign(Object.assign({}, jiraSwagger.paths), jsmSwagger.paths), jswSwagger.paths));
+        this.jira = this.processPaths({
+            ...jiraSwagger.paths,
+            ...jsmSwagger.paths,
+            ...jswSwagger.paths
+        });
         this.confluence = this.processPaths(confluenceSwagger.paths);
         this.bitbucket = this.processPaths(bitbucketSwagger.paths);
         this.nodeVisitors = [
@@ -174,12 +201,11 @@ class PermissionLinter extends base_linter_1.default {
         }
     }
     async getProductPaths(cacheKey, url) {
-        var _a, _b;
-        const cached = (_a = this.cache) === null || _a === void 0 ? void 0 : _a.get(cacheKey);
+        const cached = this.cache?.get(cacheKey);
         if (!cached) {
             const response = await (0, node_fetch_1.default)(url);
             const jsonBody = await response.json();
-            void ((_b = this.cache) === null || _b === void 0 ? void 0 : _b.set(cacheKey, jsonBody, 12 * ONE_HOUR_MS));
+            void this.cache?.set(cacheKey, jsonBody, 12 * ONE_HOUR_MS);
             return jsonBody;
         }
         return cached;
@@ -198,8 +224,3 @@ class PermissionLinter extends base_linter_1.default {
     }
 }
 exports.PermissionLinter = PermissionLinter;
-PermissionLinter.JIRA_CACHE_KEY = 'permissions-jira-cache-key';
-PermissionLinter.JSM_CACHE_KEY = 'permissions-jsm-cache-key';
-PermissionLinter.JSW_CACHE_KEY = 'permissions-jsw-cache-key';
-PermissionLinter.CONFLUENCE_CACHE_KEY = 'permissions-confluence-cache-key';
-PermissionLinter.BITBUCKET_CACHE_KEY = 'permissions-bitbucket-cache-key';

package/out/lint/linters/permission-linter/verifiers/content-property-verifier.js

@@ -88,8 +88,7 @@ class ContentPropertyVerifier extends verifier_interface_1.BaseLintIssueVerifier
         return lintErrors;
     }
     getMissingScopes({ context, method }) {
-        var _a, _b;
-        const existingScopes = (_b = (_a = this.manifest) === null || _a === void 0 ? void 0 : _a.permissions) === null || _b === void 0 ? void 0 : _b.scopes;
+        const existingScopes = this.manifest?.permissions?.scopes;
         return (0, manifest_1.getMissingScopes)(existingScopes, exports.CONTENT_PROPERTY_PERMISSIONS_MAP[context][method]);
     }
 }

package/out/lint/linters/permission-linter/verifiers/external-fetch-verifier.js

@@ -7,14 +7,14 @@ const verifier_interface_1 = require("../../verifier-interface");
 const text_1 = require("../../../text");
 const utils_1 = require("../../utils");
 class ExternalFetchVerifier extends verifier_interface_1.BaseLintIssueVerifier {
+    egressFilteringService;
     constructor(environment, manifest) {
-        var _a, _b, _c, _d, _e, _f, _g;
         super(environment, manifest);
-        const allowList = (_e = (_d = (_c = (_b = (_a = this.manifest) === null || _a === void 0 ? void 0 : _a.permissions) === null || _b === void 0 ? void 0 : _b.external) === null || _c === void 0 ? void 0 : _c.fetch) === null || _d === void 0 ? void 0 : _d.backend) !== null && _e !== void 0 ? _e : [];
-        const remoteList = (_g = (_f = this.manifest) === null || _f === void 0 ? void 0 : _f.remotes) !== null && _g !== void 0 ? _g : [];
+        const allowList = this.manifest?.permissions?.external?.fetch?.backend ?? [];
+        const remoteList = this.manifest?.remotes ?? [];
         const collectedUrls = allowList.filter((item) => typeof item === 'string');
         const collectedRemotes = allowList
-            .map((item) => { var _a; return typeof item !== 'string' && ((_a = remoteList.find((remoteItem) => remoteItem.key === item.remote)) === null || _a === void 0 ? void 0 : _a.baseUrl); })
+            .map((item) => typeof item !== 'string' && remoteList.find((remoteItem) => remoteItem.key === item.remote)?.baseUrl)
             .filter((x) => typeof x === 'string');
         this.egressFilteringService = new egress_1.EgressFilteringService(collectedUrls.concat(collectedRemotes));
     }

package/out/lint/linters/permission-linter/verifiers/image-url-verifier.js

@@ -7,10 +7,10 @@ const verifier_interface_1 = require("../../verifier-interface");
 const text_1 = require("../../../text");
 const utils_1 = require("../../utils");
 class ImageUrlVerifier extends verifier_interface_1.BaseLintIssueVerifier {
+    egressFilteringService;
     constructor(environment, manifest) {
-        var _a, _b, _c, _d;
         super(environment, manifest);
-        const allowList = (_d = (_c = (_b = (_a = this.manifest) === null || _a === void 0 ? void 0 : _a.permissions) === null || _b === void 0 ? void 0 : _b.external) === null || _c === void 0 ? void 0 : _c.images) !== null && _d !== void 0 ? _d : [];
+        const allowList = this.manifest?.permissions?.external?.images ?? [];
         const collectedUrls = allowList.filter((item) => typeof item === 'string');
         this.egressFilteringService = new egress_1.EgressFilteringService(collectedUrls);
     }

package/out/lint/linters/permission-linter/verifiers/product-verifier.js

@@ -9,6 +9,8 @@ const manifest_1 = require("@forge/manifest");
 const text_1 = require("../../../text");
 const array_prototype_flatmap_1 = tslib_1.__importDefault(require("array.prototype.flatmap"));
 class ProductVerifier extends verifier_interface_1.BaseLintIssueVerifier {
+    pathMap;
+    product;
     constructor(environment, manifest, pathMap, product) {
         super(environment, manifest);
         this.pathMap = pathMap;
@@ -18,7 +20,6 @@ class ProductVerifier extends verifier_interface_1.BaseLintIssueVerifier {
         return linter_interface_1.LintClass.Error;
     }
     async process(apiCalls) {
-        var _a;
         const rules = [];
         const lintCriteriaMatches = new Set();
         for (const apiCall of apiCalls) {
@@ -28,14 +29,14 @@ class ProductVerifier extends verifier_interface_1.BaseLintIssueVerifier {
             }
         }
         for (const apiCall of lintCriteriaMatches) {
-            if (!(apiCall === null || apiCall === void 0 ? void 0 : apiCall.regex)) {
+            if (!apiCall?.regex) {
                 continue;
             }
             const missingScopes = await this.getMissingPermissionScopes(apiCall);
             for (const permission of missingScopes) {
                 rules.push({
                     class: this.getLintClass(),
-                    message: text_1.messages.verifiers.product.message((0, cli_shared_1.capitalise)(this.product), apiCall.method.toUpperCase(), (_a = this.pathMap.get(apiCall.regex)) === null || _a === void 0 ? void 0 : _a.originalPath, permission),
+                    message: text_1.messages.verifiers.product.message((0, cli_shared_1.capitalise)(this.product), apiCall.method.toUpperCase(), this.pathMap.get(apiCall.regex)?.originalPath, permission),
                     reference: text_1.messages.verifiers.product.reference,
                     line: apiCall.line,
                     column: apiCall.column,
@@ -51,34 +52,36 @@ class ProductVerifier extends verifier_interface_1.BaseLintIssueVerifier {
         const pathWithoutQuery = apiCall.path.replace(/\?.+$/, '');
         for (const regex of this.pathMap.keys()) {
             if (regex.test(pathWithoutQuery)) {
-                return Object.assign(Object.assign({}, apiCall), { regex, path: pathWithoutQuery, method: apiCall.method.toLowerCase() });
+                return {
+                    ...apiCall,
+                    regex,
+                    path: pathWithoutQuery,
+                    method: apiCall.method.toLowerCase()
+                };
             }
         }
     }
     getMissingPermissionScopes(apiCall) {
-        var _a, _b;
         const current = this.getRequiredScopes(apiCall);
         const beta = this.getBetaScopes(apiCall);
-        return (0, manifest_1.getMissingScopes)((_b = (_a = this.manifest) === null || _a === void 0 ? void 0 : _a.permissions) === null || _b === void 0 ? void 0 : _b.scopes, { current, beta });
+        return (0, manifest_1.getMissingScopes)(this.manifest?.permissions?.scopes, { current, beta });
     }
     getRequiredScopes({ regex, method }) {
-        var _a, _b, _c, _d;
         if (!regex)
             return [];
-        const oAuth2Scopes = (_b = (_a = this.pathMap.get(regex)) === null || _a === void 0 ? void 0 : _a.methods[method]) === null || _b === void 0 ? void 0 : _b['x-atlassian-oauth2-scopes'];
+        const oAuth2Scopes = this.pathMap.get(regex)?.methods[method]?.['x-atlassian-oauth2-scopes'];
         if (oAuth2Scopes) {
             return (0, array_prototype_flatmap_1.default)(oAuth2Scopes.filter((scope) => scope.state === 'Current').map((scope) => scope.scopes), (x) => x);
         }
-        const swaggerEntries = (_d = (_c = this.pathMap.get(regex)) === null || _c === void 0 ? void 0 : _c.methods[method]) === null || _d === void 0 ? void 0 : _d.security;
+        const swaggerEntries = this.pathMap.get(regex)?.methods[method]?.security;
         if (!swaggerEntries)
             return [];
-        return swaggerEntries === null || swaggerEntries === void 0 ? void 0 : swaggerEntries.reduce((prev, curr) => [...prev, ...(curr['OAuth2'] || []), ...(curr['oAuthDefinitions'] || [])], []);
+        return swaggerEntries?.reduce((prev, curr) => [...prev, ...(curr['OAuth2'] || []), ...(curr['oAuthDefinitions'] || [])], []);
     }
     getBetaScopes({ regex, method }) {
-        var _a, _b;
         if (!regex)
             return [];
-        const oAuth2Scopes = (_b = (_a = this.pathMap.get(regex)) === null || _a === void 0 ? void 0 : _a.methods[method]) === null || _b === void 0 ? void 0 : _b['x-atlassian-oauth2-scopes'];
+        const oAuth2Scopes = this.pathMap.get(regex)?.methods[method]?.['x-atlassian-oauth2-scopes'];
         if (!oAuth2Scopes)
             return [];
         return (0, array_prototype_flatmap_1.default)(oAuth2Scopes.filter((scope) => scope.state === 'Beta').map((scope) => scope.scopes), (x) => x);

package/out/lint/linters/permission-linter/verifiers/storage-api-verifier.js

@@ -10,13 +10,18 @@ class StorageAPIVerifier extends verifier_interface_1.BaseLintIssueVerifier {
         return linter_interface_1.LintClass.Error;
     }
     async process(apiCalls) {
-        var _a, _b;
-        if ((_b = (_a = this.manifest.permissions) === null || _a === void 0 ? void 0 : _a.scopes) === null || _b === void 0 ? void 0 : _b.includes(STORAGE_PERMISSION)) {
+        if (this.manifest.permissions?.scopes?.includes(STORAGE_PERMISSION)) {
             return [];
         }
-        return apiCalls.map((apiCall) => (Object.assign(Object.assign({ class: this.getLintClass(), message: text_1.messages.verifiers.storage.message(STORAGE_PERMISSION), reference: text_1.messages.verifiers.storage.reference }, apiCall), { metadata: {
+        return apiCalls.map((apiCall) => ({
+            class: this.getLintClass(),
+            message: text_1.messages.verifiers.storage.message(STORAGE_PERMISSION),
+            reference: text_1.messages.verifiers.storage.reference,
+            ...apiCall,
+            metadata: {
                 missingPermission: STORAGE_PERMISSION
-            } })));
+            }
+        }));
     }
 }
 exports.StorageAPIVerifier = StorageAPIVerifier;

package/out/lint/linters/permission-linter/verifiers/ui-hook-verifier.js

@@ -24,13 +24,12 @@ class UIHookVerifier extends verifier_interface_1.BaseLintIssueVerifier {
         return linter_interface_1.LintClass.Error;
     }
     async process(apiCalls) {
-        var _a;
         const rules = [];
         for (const apiCall of apiCalls) {
             const requiredPermissions = FORGE_UI_HOOK_PERMISSIONS[apiCall.hookName];
             if (!requiredPermissions)
                 return rules;
-            const missingPermissions = await (0, manifest_1.getMissingScopes)((_a = this.manifest.permissions) === null || _a === void 0 ? void 0 : _a.scopes, requiredPermissions);
+            const missingPermissions = await (0, manifest_1.getMissingScopes)(this.manifest.permissions?.scopes, requiredPermissions);
             missingPermissions.forEach((perm) => rules.push(this.getHookRule(apiCall, perm)));
         }
         return rules;

package/out/lint/linters/permission-linter/visitors/external-fetch-call-visitor.js

@@ -5,20 +5,12 @@ const typescript_estree_1 = require("@typescript-eslint/typescript-estree");
 const url_1 = require("url");
 const api_call_interface_1 = require("../api-call-interface");
 class ExternalApiCallVisitor {
-    constructor() {
-        this.transformArgsToFetchCall = (endpointNode) => ({
-            type: api_call_interface_1.ApiCallTypes.EXTERNAL,
-            url: endpointNode.value,
-            line: endpointNode.loc.start.line,
-            column: endpointNode.loc.start.column
-        });
-    }
     visit(node, parent, callback) {
         switch (node.type) {
             case typescript_estree_1.AST_NODE_TYPES.CallExpression:
                 if (node.callee.type === typescript_estree_1.AST_NODE_TYPES.Identifier && node.callee.name === 'fetch') {
                     const [endpointNode] = node.arguments;
-                    if ((endpointNode === null || endpointNode === void 0 ? void 0 : endpointNode.type) === typescript_estree_1.AST_NODE_TYPES.Literal && typeof (endpointNode === null || endpointNode === void 0 ? void 0 : endpointNode.value) === 'string') {
+                    if (endpointNode?.type === typescript_estree_1.AST_NODE_TYPES.Literal && typeof endpointNode?.value === 'string') {
                         if (endpointNode.value.trim() !== '') {
                             const fetchCall = this.transformArgsToFetchCall(endpointNode);
                             callback(fetchCall);
@@ -29,7 +21,7 @@ class ExternalApiCallVisitor {
             case typescript_estree_1.AST_NODE_TYPES.MemberExpression:
                 if (node.property.type === typescript_estree_1.AST_NODE_TYPES.Identifier &&
                     node.property.name === 'fetch' &&
-                    (parent === null || parent === void 0 ? void 0 : parent.type) === typescript_estree_1.AST_NODE_TYPES.CallExpression) {
+                    parent?.type === typescript_estree_1.AST_NODE_TYPES.CallExpression) {
                     const [endpointNode] = parent.arguments;
                     if (this.endpointCanBeChecked(endpointNode)) {
                         const fetchCall = this.transformArgsToFetchCall(endpointNode);
@@ -45,15 +37,21 @@ class ExternalApiCallVisitor {
                 thunk();
                 return false;
             }
-            catch (_a) {
+            catch {
                 return true;
             }
         };
         const isPathOnlyUrl = (s) => throws(() => new url_1.URL(s)) && !throws(() => new url_1.URL(s, 'http://example'));
-        return ((endpointNode === null || endpointNode === void 0 ? void 0 : endpointNode.type) === typescript_estree_1.AST_NODE_TYPES.Literal &&
-            typeof (endpointNode === null || endpointNode === void 0 ? void 0 : endpointNode.value) === 'string' &&
+        return (endpointNode?.type === typescript_estree_1.AST_NODE_TYPES.Literal &&
+            typeof endpointNode?.value === 'string' &&
             endpointNode.value.trim() !== '' &&
             !isPathOnlyUrl(endpointNode.value));
     }
+    transformArgsToFetchCall = (endpointNode) => ({
+        type: api_call_interface_1.ApiCallTypes.EXTERNAL,
+        url: endpointNode.value,
+        line: endpointNode.loc.start.line,
+        column: endpointNode.loc.start.column
+    });
 }
 exports.ExternalApiCallVisitor = ExternalApiCallVisitor;

package/out/lint/linters/permission-linter/visitors/image-url-visitor.js

@@ -4,12 +4,13 @@ exports.ImageUrlVisitor = void 0;
 const typescript_estree_1 = require("@typescript-eslint/typescript-estree");
 const api_call_interface_1 = require("../api-call-interface");
 class ImageUrlVisitor {
+    static ABSOLUTE_URL_REGEX = /^(?:[a-z+]+:)?\/\//i;
     visit(node, _parent, callback) {
         const imageUrlNode = this.getImageUrlNode(node);
         if (!imageUrlNode) {
             return undefined;
         }
-        if ((imageUrlNode === null || imageUrlNode === void 0 ? void 0 : imageUrlNode.type) === typescript_estree_1.AST_NODE_TYPES.JSXAttribute) {
+        if (imageUrlNode?.type === typescript_estree_1.AST_NODE_TYPES.JSXAttribute) {
             const imageUrlAttribute = imageUrlNode.value;
             if (imageUrlAttribute &&
                 imageUrlAttribute.type === typescript_estree_1.AST_NODE_TYPES.Literal &&
@@ -54,4 +55,3 @@ class ImageUrlVisitor {
     }
 }
 exports.ImageUrlVisitor = ImageUrlVisitor;
-ImageUrlVisitor.ABSOLUTE_URL_REGEX = /^(?:[a-z+]+:)?\/\//i;

package/out/lint/linters/permission-linter/visitors/product-node-visitor.js

@@ -17,7 +17,7 @@ class ProductNodeVisitor {
             case typescript_estree_1.AST_NODE_TYPES.MemberExpression:
                 if (node.property.type === typescript_estree_1.AST_NODE_TYPES.Identifier &&
                     node.property.name.match(REQUEST_PRODUCT_REGEX) &&
-                    (parent === null || parent === void 0 ? void 0 : parent.type) === typescript_estree_1.AST_NODE_TYPES.CallExpression) {
+                    parent?.type === typescript_estree_1.AST_NODE_TYPES.CallExpression) {
                     const apiCallProblem = this.checkProductApiCallArgs(node.property.name, parent.arguments);
                     apiCallProblem && callback(apiCallProblem);
                 }
@@ -78,10 +78,10 @@ class ProductNodeVisitor {
         if (!endpointString) {
             return;
         }
-        const methodProperty = optionsNode === null || optionsNode === void 0 ? void 0 : optionsNode.properties.find((property) => property.type === typescript_estree_1.AST_NODE_TYPES.Property &&
+        const methodProperty = optionsNode?.properties.find((property) => property.type === typescript_estree_1.AST_NODE_TYPES.Property &&
             property.key.type === typescript_estree_1.AST_NODE_TYPES.Identifier &&
             property.key.name === 'method');
-        const methodValue = (methodProperty === null || methodProperty === void 0 ? void 0 : methodProperty.value.type) === typescript_estree_1.AST_NODE_TYPES.Literal && typeof methodProperty.value.value === 'string'
+        const methodValue = methodProperty?.value.type === typescript_estree_1.AST_NODE_TYPES.Literal && typeof methodProperty.value.value === 'string'
             ? methodProperty.value.value
             : 'GET';
         return {

package/out/lint/linters/permission-linter/visitors/storage-api-node-visitor.js

@@ -9,7 +9,10 @@ class StorageAPINodeVisitor {
             node.specifiers.forEach((specifier) => {
                 if (specifier.type === 'ImportSpecifier' && specifier.local.type === 'Identifier') {
                     if (specifier.loc && specifier.local.name === 'storage') {
-                        const storageApiCall = Object.assign({ type: api_call_interface_1.ApiCallTypes.STORAGE }, specifier.loc.start);
+                        const storageApiCall = {
+                            type: api_call_interface_1.ApiCallTypes.STORAGE,
+                            ...specifier.loc.start
+                        };
                         callback(storageApiCall);
                     }
                 }

package/out/lint/linters/permission-linter/visitors/ui-hook-node-visitor.js

@@ -8,7 +8,11 @@ class UIHookNodeVisitor {
         if (node.type === typescript_estree_1.AST_NODE_TYPES.ImportDeclaration) {
             node.specifiers.forEach((specifier) => {
                 if (specifier.loc && specifier.local.name.match(/^use(Space|Content|Issue)Property/)) {
-                    const hookApiCall = Object.assign({ type: api_call_interface_1.ApiCallTypes.HOOK, hookName: specifier.local.name }, specifier.loc.start);
+                    const hookApiCall = {
+                        type: api_call_interface_1.ApiCallTypes.HOOK,
+                        hookName: specifier.local.name,
+                        ...specifier.loc.start
+                    };
                     callback(hookApiCall);
                 }
             });

package/out/lint/linters/remote-linter/invoke-remote-linter.js

@@ -7,6 +7,7 @@ const invoke_remote_interface_1 = require("./invoke-remote-interface");
 const invoke_remote_verifier_1 = require("./verifiers/invoke-remote-verifier");
 const invoke_remote_call_visitor_1 = require("./visitors/invoke-remote-call-visitor");
 class InvokeRemoteLinter extends base_linter_1.default {
+    manifest;
     constructor(environment, manifest, logger) {
         super(environment, logger);
         this.manifest = manifest;

package/out/lint/linters/remote-linter/visitors/invoke-remote-call-visitor.js

@@ -4,13 +4,6 @@ exports.InvokeRemoteCallVisitor = void 0;
 const typescript_estree_1 = require("@typescript-eslint/typescript-estree");
 const invoke_remote_interface_1 = require("../invoke-remote-interface");
 class InvokeRemoteCallVisitor {
-    constructor() {
-        this.transformArgsToInvokeRemoteCall = (remoteKeyNode) => ({
-            remoteKey: remoteKeyNode.value,
-            line: remoteKeyNode.loc.start.line,
-            column: remoteKeyNode.loc.start.column
-        });
-    }
     visit(node, parent, callback) {
         switch (node.type) {
             case typescript_estree_1.AST_NODE_TYPES.ImportDeclaration:
@@ -38,7 +31,11 @@ class InvokeRemoteCallVisitor {
                 if (node.callee.type === typescript_estree_1.AST_NODE_TYPES.Identifier) {
                     const [remoteKeyNode] = node.arguments;
                     if (this.remoteKeyCanBeChecked(remoteKeyNode)) {
-                        const invokeRemoteCall = Object.assign(Object.assign({}, this.transformArgsToInvokeRemoteCall(remoteKeyNode)), { type: invoke_remote_interface_1.InvokeRemoteTypes.DIRECT_CALL, calleeName: node.callee.name });
+                        const invokeRemoteCall = {
+                            ...this.transformArgsToInvokeRemoteCall(remoteKeyNode),
+                            type: invoke_remote_interface_1.InvokeRemoteTypes.DIRECT_CALL,
+                            calleeName: node.callee.name
+                        };
                         callback(invokeRemoteCall);
                     }
                 }
@@ -47,10 +44,14 @@ class InvokeRemoteCallVisitor {
                 if (node.property.type === typescript_estree_1.AST_NODE_TYPES.Identifier &&
                     node.object.type === typescript_estree_1.AST_NODE_TYPES.Identifier &&
                     node.property.name === invoke_remote_interface_1.INVOKE_REMOTE_FUNCTION_NAME &&
-                    (parent === null || parent === void 0 ? void 0 : parent.type) === typescript_estree_1.AST_NODE_TYPES.CallExpression) {
+                    parent?.type === typescript_estree_1.AST_NODE_TYPES.CallExpression) {
                     const [remoteKeyNode] = parent.arguments;
                     if (this.remoteKeyCanBeChecked(remoteKeyNode)) {
-                        const invokeRemoteMemberCall = Object.assign(Object.assign({}, this.transformArgsToInvokeRemoteCall(remoteKeyNode)), { type: invoke_remote_interface_1.InvokeRemoteTypes.MEMBER_CALL, objectName: node.object.name });
+                        const invokeRemoteMemberCall = {
+                            ...this.transformArgsToInvokeRemoteCall(remoteKeyNode),
+                            type: invoke_remote_interface_1.InvokeRemoteTypes.MEMBER_CALL,
+                            objectName: node.object.name
+                        };
                         callback(invokeRemoteMemberCall);
                     }
                 }
@@ -58,9 +59,14 @@ class InvokeRemoteCallVisitor {
         }
     }
     remoteKeyCanBeChecked(remoteKeyNode) {
-        return ((remoteKeyNode === null || remoteKeyNode === void 0 ? void 0 : remoteKeyNode.type) === typescript_estree_1.AST_NODE_TYPES.Literal &&
-            typeof (remoteKeyNode === null || remoteKeyNode === void 0 ? void 0 : remoteKeyNode.value) === 'string' &&
+        return (remoteKeyNode?.type === typescript_estree_1.AST_NODE_TYPES.Literal &&
+            typeof remoteKeyNode?.value === 'string' &&
             remoteKeyNode.value.trim() !== '');
     }
+    transformArgsToInvokeRemoteCall = (remoteKeyNode) => ({
+        remoteKey: remoteKeyNode.value,
+        line: remoteKeyNode.loc.start.line,
+        column: remoteKeyNode.loc.start.column
+    });
 }
 exports.InvokeRemoteCallVisitor = InvokeRemoteCallVisitor;

package/out/lint/linters/verifier-interface.js

@@ -4,6 +4,8 @@ exports.BaseLintIssueVerifier = void 0;
 const linter_interface_1 = require("../linter-interface");
 const cli_shared_1 = require("@forge/cli-shared");
 class BaseLintIssueVerifier {
+    environment;
+    manifest;
     constructor(environment, manifest) {
         this.environment = environment;
         this.manifest = manifest;

package/out/parse/parser.js

@@ -10,12 +10,20 @@ const createParseOptions = (absoluteFilePath, fileSystemReader) => {
     const tsconfigOptions = fileSystemReader.fileExists(TSCONFIG_PATH)
         ? { tsconfigRootDir: process.cwd(), project: TSCONFIG_PATH }
         : {};
-    return Object.assign({ errorOnTypeScriptSyntacticAndSemanticIssues: true, loc: true, filePath: absoluteFilePath }, tsconfigOptions);
+    return {
+        errorOnTypeScriptSyntacticAndSemanticIssues: true,
+        loc: true,
+        filePath: absoluteFilePath,
+        ...tsconfigOptions
+    };
 };
 exports.createParseOptions = createParseOptions;
 const tsParser = async (code, filepath, parseOptions) => {
     const absoluteFilePath = path_1.default.resolve(process.cwd(), filepath);
-    const parseResult = (0, typescript_estree_1.parseAndGenerateServices)(code, Object.assign(Object.assign({}, (0, exports.createParseOptions)(absoluteFilePath, new cli_shared_1.FileSystemReader())), parseOptions));
+    const parseResult = (0, typescript_estree_1.parseAndGenerateServices)(code, {
+        ...(0, exports.createParseOptions)(absoluteFilePath, new cli_shared_1.FileSystemReader()),
+        ...parseOptions
+    });
     return {
         parsed: parseResult.ast,
         traverse: (options) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forge/lint",
-  "version": "5.1.1-next.5",
+  "version": "5.1.1-next.6",
   "description": "Linting for forge apps",
   "main": "out/index.js",
   "license": "UNLICENSED",
@@ -18,9 +18,9 @@
     "eslint-plugin-import": "^2.29.1"
   },
   "dependencies": {
-    "@forge/cli-shared": "5.0.0-next.5",
+    "@forge/cli-shared": "5.0.0-next.6",
     "@forge/egress": "1.2.13",
-    "@forge/manifest": "7.3.0-next.4",
+    "@forge/manifest": "7.3.0-next.5",
     "@typescript-eslint/typescript-estree": "^5.62.0",
     "array.prototype.flatmap": "^1.3.2",
     "atlassian-openapi": "^1.0.18",