@forge/lint 5.18.1-next.1 → 5.19.0-next.0

package/CHANGELOG.md

@@ -1,5 +1,30 @@
 # @forge/lint
 
+## 5.19.0-next.0
+
+### Minor Changes
+
+- d471144: fix lint
+- 56e38e9: Added support for new v2 url format and validation
+
+### Patch Changes
+
+- Updated dependencies [a99c9f1]
+- Updated dependencies [372861e]
+- Updated dependencies [56e38e9]
+- Updated dependencies [aed9912]
+  - @forge/manifest@12.7.0-next.0
+  - @forge/cli-shared@8.21.1-next.0
+
+## 5.18.1
+
+### Patch Changes
+
+- Updated dependencies [7f9ae91]
+- Updated dependencies [e7fa544]
+  - @forge/cli-shared@8.21.0
+  - @forge/csp@5.8.0
+
 ## 5.18.1-next.1
 
 ### Patch Changes

package/out/lint/linters/manifest-linter/full-manifest-linter.d.ts

@@ -1,7 +1,8 @@
 import { AbstractManifestLinter } from './abstract-manifest-linter';
-import { LintLogger } from '../../linter-interface';
+import { LintLogger, LintResultRule, LintFixState } from '../../linter-interface';
 import { StatsigService } from '@forge/cli-shared';
 export declare class FullManifestLinter extends AbstractManifestLinter {
     constructor(logger: LintLogger, statsigService: StatsigService);
+    protected getFixer(): ((errors: LintResultRule[], warnings: LintResultRule[], state: LintFixState) => Promise<LintFixState>) | undefined;
 }
 //# sourceMappingURL=full-manifest-linter.d.ts.map

package/out/lint/linters/manifest-linter/full-manifest-linter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"full-manifest-linter.d.ts","sourceRoot":"","sources":["../../../../src/lint/linters/manifest-linter/full-manifest-linter.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,sBAAsB,EAAE,MAAM,4BAA4B,CAAC;AACpE,OAAO,EAAE,UAAU,EAAE,MAAM,wBAAwB,CAAC;AACpD,OAAO,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAC;AAEnD,qBAAa,kBAAmB,SAAQ,sBAAsB;gBAChD,MAAM,EAAE,UAAU,EAAE,cAAc,EAAE,cAAc;CAG/D"}
+{"version":3,"file":"full-manifest-linter.d.ts","sourceRoot":"","sources":["../../../../src/lint/linters/manifest-linter/full-manifest-linter.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,sBAAsB,EAAE,MAAM,4BAA4B,CAAC;AACpE,OAAO,EAAE,UAAU,EAAE,cAAc,EAAE,YAAY,EAAE,MAAM,wBAAwB,CAAC;AAClF,OAAO,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAC;AAGnD,qBAAa,kBAAmB,SAAQ,sBAAsB;gBAChD,MAAM,EAAE,UAAU,EAAE,cAAc,EAAE,cAAc;IAI9D,SAAS,CAAC,QAAQ,IACd,CAAC,CAAC,MAAM,EAAE,cAAc,EAAE,EAAE,QAAQ,EAAE,cAAc,EAAE,EAAE,KAAK,EAAE,YAAY,KAAK,OAAO,CAAC,YAAY,CAAC,CAAC,GACtG,SAAS;CAGd"}

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

@@ -3,9 +3,13 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.FullManifestLinter = void 0;
 const manifest_1 = require("@forge/manifest");
 const abstract_manifest_linter_1 = require("./abstract-manifest-linter");
+const webtrigger_url_format_manifest_linter_1 = require("./webtrigger-url-format-manifest-linter");
 class FullManifestLinter extends abstract_manifest_linter_1.AbstractManifestLinter {
     constructor(logger, statsigService) {
         super(manifest_1.ValidationTypes.FULL, logger, statsigService);
     }
+    getFixer() {
+        return webtrigger_url_format_manifest_linter_1.fixInvalidUrlFormat;
+    }
 }
 exports.FullManifestLinter = FullManifestLinter;

package/out/lint/linters/manifest-linter/webtrigger-url-format-manifest-linter.d.ts

@@ -0,0 +1,3 @@
+import { LintResultRule, LintFixState } from '../../linter-interface';
+export declare const fixInvalidUrlFormat: (_errors: LintResultRule[], _warnings: LintResultRule[], state: LintFixState) => Promise<LintFixState>;
+//# sourceMappingURL=webtrigger-url-format-manifest-linter.d.ts.map

package/out/lint/linters/manifest-linter/webtrigger-url-format-manifest-linter.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"webtrigger-url-format-manifest-linter.d.ts","sourceRoot":"","sources":["../../../../src/lint/linters/manifest-linter/webtrigger-url-format-manifest-linter.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,cAAc,EAAE,YAAY,EAAE,MAAM,wBAAwB,CAAC;AAItE,eAAO,MAAM,mBAAmB,YACrB,cAAc,EAAE,aACd,cAAc,EAAE,SACpB,YAAY,KAClB,QAAQ,YAAY,CA+BtB,CAAC"}

package/out/lint/linters/manifest-linter/webtrigger-url-format-manifest-linter.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.fixInvalidUrlFormat = void 0;
+const VALID_URL_FORMATS = ['v1', 'v2'];
+const fixInvalidUrlFormat = async (_errors, _warnings, state) => {
+    const currentModules = (await state.configFile.readConfig())?.modules ?? {};
+    const webtriggers = currentModules.webtrigger;
+    if (!webtriggers || webtriggers.length === 0) {
+        return state;
+    }
+    const invalidEntries = webtriggers.filter((wt) => typeof wt.urlFormat === 'string' && !VALID_URL_FORMATS.includes(wt.urlFormat));
+    if (invalidEntries.length === 0) {
+        return state;
+    }
+    const fixedWebtriggers = webtriggers.map((wt) => invalidEntries.includes(wt) ? { ...wt, urlFormat: 'v2' } : wt);
+    const newModules = { ...currentModules, webtrigger: fixedWebtriggers };
+    await state.configFile.writeToConfigFile('modules', newModules);
+    state.errorsFixed += invalidEntries.length;
+    return state;
+};
+exports.fixInvalidUrlFormat = fixInvalidUrlFormat;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forge/lint",
-  "version": "5.18.1-next.1",
+  "version": "5.19.0-next.0",
   "description": "Linting for forge apps",
   "main": "out/index.js",
   "license": "SEE LICENSE IN LICENSE.txt",
@@ -11,7 +11,7 @@
   },
   "devDependencies": {
     "@atlassian/xen-test-util": "^4.2.0",
-    "@forge/api": "^7.1.4",
+    "@forge/api": "^7.2.0-next.0",
     "@types/array.prototype.flatmap": "^1.2.6",
     "@types/cross-spawn": "^6.0.6",
     "@types/eslint": "8.56.12",
@@ -19,10 +19,10 @@
     "eslint-plugin-import": "^2.29.1"
   },
   "dependencies": {
-    "@forge/cli-shared": "8.21.0-next.0",
-    "@forge/csp": "5.8.0-next.0",
+    "@forge/cli-shared": "8.21.1-next.0",
+    "@forge/csp": "5.8.0",
     "@forge/egress": "2.3.2",
-    "@forge/manifest": "12.6.0",
+    "@forge/manifest": "12.7.0-next.0",
     "@typescript-eslint/typescript-estree": "^5.62.0",
     "array.prototype.flatmap": "^1.3.3",
     "@atlassian/atlassian-openapi": "^1.0.6",