@forge/lint 5.9.1-next.8 → 5.9.1-next.9-experimental-effab31

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (5) hide show
  1. package/CHANGELOG.md +31 -0
  2. package/out/lint/linters/permission-linter/permission-linter.d.ts +1 -0
  3. package/out/lint/linters/permission-linter/permission-linter.d.ts.map +1 -1
  4. package/out/lint/linters/permission-linter/permission-linter.js +41 -31
  5. package/package.json +6 -6
package/CHANGELOG.md CHANGED
@@ -1,5 +1,36 @@
1
1
  # @forge/lint
2
2
 
3
+ ## 5.9.1-next.9-experimental-effab31
4
+
5
+ ### Patch Changes
6
+
7
+ - 5d07fbf: Update the default value for inScopeEUD of egress entries by the linter
8
+ - c3810aa: Fix linter not triggering auto-fix for some cases of deprecated egress permissions
9
+ - 195411c: patch dependencies
10
+ - Updated dependencies [effab31]
11
+ - Updated dependencies [5d07fbf]
12
+ - Updated dependencies [4b15e26]
13
+ - Updated dependencies [634d7b9]
14
+ - Updated dependencies [620aea7]
15
+ - Updated dependencies [25a3da2]
16
+ - Updated dependencies [b9574d4]
17
+ - Updated dependencies [195411c]
18
+ - Updated dependencies [f5ba3aa]
19
+ - Updated dependencies [d97a252]
20
+ - Updated dependencies [1479d2d]
21
+ - Updated dependencies [ca7e661]
22
+ - Updated dependencies [5ab2c49]
23
+ - @forge/cli-shared@8.1.0-next.7-experimental-effab31
24
+ - @forge/manifest@10.0.1-next.7-experimental-effab31
25
+ - @forge/csp@4.0.1-next.1-experimental-effab31
26
+ - @forge/egress@2.0.1-next.0-experimental-effab31
27
+
28
+ ## 5.9.1-next.9
29
+
30
+ ### Patch Changes
31
+
32
+ - c3810aa: Fix linter not triggering auto-fix for some cases of deprecated egress permissions
33
+
3
34
  ## 5.9.1-next.8
4
35
 
5
36
  ### Patch Changes
package/out/lint/linters/permission-linter/permission-linter.d.ts CHANGED
@@ -10,6 +10,7 @@ interface SwaggerFormat {
10
10
  }
11
11
  export declare const NON_FWD_SLASH_REGEX = "[^\\/\\s]*";
12
12
  export declare const findMissingPermissions: (errors: LintResultRule[], warnings: LintResultRule[], permissionsType: 'missingPermission' | 'missingExternalFetchPermission' | 'missingContentStylePermission' | 'missingExternalImagesPermission', state?: LintFixState) => Set<string>;
13
+ export declare const deprecatedEgressPermissions: (warnings: LintResultRule[], state?: LintFixState) => boolean;
13
14
  export declare const fixMissingPermissions: (errors: LintResultRule[], warnings: LintResultRule[], state: LintFixState) => Promise<LintFixState>;
14
15
  interface PermissionLintCriteriaMatches {
15
16
  [key: string]: ApiCall[];
package/out/lint/linters/permission-linter/permission-linter.d.ts.map CHANGED
@@ -1 +1 @@
1
- {"version":3,"file":"permission-linter.d.ts","sourceRoot":"","sources":["../../../../src/lint/linters/permission-linter/permission-linter.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,cAAc,IAAI,QAAQ,EAAe,MAAM,iBAAiB,CAAC;AAE/E,OAAO,UAAU,MAAM,mBAAmB,CAAC;AAC3C,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAClF,OAAO,EACL,OAAO,EAEP,eAAe,EACf,QAAQ,EACR,cAAc,EACd,cAAc,EACd,aAAa,EACd,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EAML,WAAW,EAGZ,MAAM,aAAa,CAAC;AAYrB,UAAU,aAAa;IACrB,KAAK,EAAE;QACL,CAAC,IAAI,EAAE,MAAM,GAAG,WAAW,CAAC;KAC7B,CAAC;CACH;AAED,eAAO,MAAM,mBAAmB,eAAe,CAAC;AAKhD,eAAO,MAAM,sBAAsB,WACzB,cAAc,EAAE,YACd,cAAc,EAAE,mBAEtB,mBAAmB,GACnB,gCAAgC,GAChC,+BAA+B,GAC/B,iCAAiC,UAC7B,YAAY,KACnB,IAAI,MAAM,CAkBZ,CAAC;AAEF,eAAO,MAAM,qBAAqB,WACxB,cAAc,EAAE,YACd,cAAc,EAAE,SACnB,YAAY,KAClB,QAAQ,YAAY,CAuGtB,CAAC;AAEF,UAAU,6BAA6B;IACrC,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,EAAE,CAAC;IACzB,UAAU,EAAE,cAAc,EAAE,CAAC;IAC7B,IAAI,EAAE,cAAc,EAAE,CAAC;IACvB,OAAO,EAAE,cAAc,EAAE,CAAC;IAC1B,MAAM,EAAE,aAAa,EAAE,CAAC;IACxB,QAAQ,EAAE,eAAe,EAAE,CAAC;IAC5B,KAAK,EAAE,QAAQ,EAAE,CAAC;CACnB;AAED,qBAAa,gBAAiB,SAAQ,UAAU,CAAC,OAAO,EAAE,6BAA6B,EAAE,OAAO,EAAE,CAAC;IAe/F,OAAO,CAAC,QAAQ;IAdlB,MAAM,CAAC,cAAc,SAAgC;IACrD,MAAM,CAAC,aAAa,SAA+B;IACnD,MAAM,CAAC,aAAa,SAA+B;IACnD,MAAM,CAAC,oBAAoB,SAAsC;IACjE,MAAM,CAAC,uBAAuB,SAAyC;IACvE,MAAM,CAAC,mBAAmB,SAAqC;IAE/D,OAAO,CAAC,IAAI,CAAC,CAA0B;IACvC,OAAO,CAAC,UAAU,CAAC,CAA0B;IAC7C,OAAO,CAAC,SAAS,CAAC,CAA0B;IAC5C,OAAO,CAAC,KAAK,CAAC,CAAa;gBAGzB,WAAW,EAAE,MAAM,EACX,QAAQ,EAAE,QAAQ,EAC1B,MAAM,EAAE,UAAU;IAKP,SAAS,IAAI,OAAO,CAAC,IAAI,CAAC;IA2FvC,SAAS,CAAC,eAAe,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI;IAcjD,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;IAIb,SAAS,CAAC,oBAAoB,CAAC,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,GAAG,IAAI;IAsB3D,eAAe,CAAC,QAAQ,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,aAAa,CAAC;IAWtE,mBAAmB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,aAAa,CAAC;IAKrE,OAAO,CAAC,YAAY;CAarB"}
1
+ {"version":3,"file":"permission-linter.d.ts","sourceRoot":"","sources":["../../../../src/lint/linters/permission-linter/permission-linter.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,cAAc,IAAI,QAAQ,EAAe,MAAM,iBAAiB,CAAC;AAE/E,OAAO,UAAU,MAAM,mBAAmB,CAAC;AAC3C,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAClF,OAAO,EACL,OAAO,EAEP,eAAe,EACf,QAAQ,EACR,cAAc,EACd,cAAc,EACd,aAAa,EACd,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EAML,WAAW,EAGZ,MAAM,aAAa,CAAC;AAYrB,UAAU,aAAa;IACrB,KAAK,EAAE;QACL,CAAC,IAAI,EAAE,MAAM,GAAG,WAAW,CAAC;KAC7B,CAAC;CACH;AAED,eAAO,MAAM,mBAAmB,eAAe,CAAC;AAKhD,eAAO,MAAM,sBAAsB,WACzB,cAAc,EAAE,YACd,cAAc,EAAE,mBAEtB,mBAAmB,GACnB,gCAAgC,GAChC,+BAA+B,GAC/B,iCAAiC,UAC7B,YAAY,KACnB,IAAI,MAAM,CAkBZ,CAAC;AAEF,eAAO,MAAM,2BAA2B,aAAc,cAAc,EAAE,UAAU,YAAY,KAAG,OAI9F,CAAC;AAoCF,eAAO,MAAM,qBAAqB,WACxB,cAAc,EAAE,YACd,cAAc,EAAE,SACnB,YAAY,KAClB,QAAQ,YAAY,CAyEtB,CAAC;AAEF,UAAU,6BAA6B;IACrC,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,EAAE,CAAC;IACzB,UAAU,EAAE,cAAc,EAAE,CAAC;IAC7B,IAAI,EAAE,cAAc,EAAE,CAAC;IACvB,OAAO,EAAE,cAAc,EAAE,CAAC;IAC1B,MAAM,EAAE,aAAa,EAAE,CAAC;IACxB,QAAQ,EAAE,eAAe,EAAE,CAAC;IAC5B,KAAK,EAAE,QAAQ,EAAE,CAAC;CACnB;AAED,qBAAa,gBAAiB,SAAQ,UAAU,CAAC,OAAO,EAAE,6BAA6B,EAAE,OAAO,EAAE,CAAC;IAe/F,OAAO,CAAC,QAAQ;IAdlB,MAAM,CAAC,cAAc,SAAgC;IACrD,MAAM,CAAC,aAAa,SAA+B;IACnD,MAAM,CAAC,aAAa,SAA+B;IACnD,MAAM,CAAC,oBAAoB,SAAsC;IACjE,MAAM,CAAC,uBAAuB,SAAyC;IACvE,MAAM,CAAC,mBAAmB,SAAqC;IAE/D,OAAO,CAAC,IAAI,CAAC,CAA0B;IACvC,OAAO,CAAC,UAAU,CAAC,CAA0B;IAC7C,OAAO,CAAC,SAAS,CAAC,CAA0B;IAC5C,OAAO,CAAC,KAAK,CAAC,CAAa;gBAGzB,WAAW,EAAE,MAAM,EACX,QAAQ,EAAE,QAAQ,EAC1B,MAAM,EAAE,UAAU;IAKP,SAAS,IAAI,OAAO,CAAC,IAAI,CAAC;IA2FvC,SAAS,CAAC,eAAe,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI;IAcjD,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;IAIb,SAAS,CAAC,oBAAoB,CAAC,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,GAAG,IAAI;IAsB3D,eAAe,CAAC,QAAQ,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,aAAa,CAAC;IAWtE,mBAAmB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,aAAa,CAAC;IAKrE,OAAO,CAAC,YAAY;CAarB"}
package/out/lint/linters/permission-linter/permission-linter.js CHANGED
@@ -1,6 +1,6 @@
1
1
  "use strict";
2
2
  Object.defineProperty(exports, "__esModule", { value: true });
3
- exports.PermissionLinter = exports.fixMissingPermissions = exports.findMissingPermissions = exports.NON_FWD_SLASH_REGEX = void 0;
3
+ exports.PermissionLinter = exports.fixMissingPermissions = exports.deprecatedEgressPermissions = exports.findMissingPermissions = exports.NON_FWD_SLASH_REGEX = void 0;
4
4
  const tslib_1 = require("tslib");
5
5
  const cli_shared_1 = require("@forge/cli-shared");
6
6
  const node_fetch_1 = tslib_1.__importDefault(require("node-fetch"));
@@ -32,12 +32,50 @@ const findMissingPermissions = (errors, warnings, permissionsType, state) => {
32
32
  return missingPermissions;
33
33
  };
34
34
  exports.findMissingPermissions = findMissingPermissions;
35
+ const deprecatedEgressPermissions = (warnings, state) => {
36
+ const hasDeprecated = warnings.some((warning) => warning.metadata?.hasDeprecatedEgressPermissions);
37
+ if (hasDeprecated && state)
38
+ state.warningsFixed++;
39
+ return hasDeprecated;
40
+ };
41
+ exports.deprecatedEgressPermissions = deprecatedEgressPermissions;
42
+ const applyEgressPermissionsMigration = (config) => {
43
+ const { fetch: originalFetch, ...restExternal } = config.external ?? {};
44
+ Object.entries(originalFetch || {}).forEach(([key, values]) => {
45
+ const valuesToOverride = [];
46
+ values.forEach((egressValue) => {
47
+ if (typeof egressValue === 'string') {
48
+ valuesToOverride.push({
49
+ address: egressValue
50
+ });
51
+ }
52
+ else {
53
+ valuesToOverride.push(egressValue);
54
+ }
55
+ config.external.fetch[key] = valuesToOverride;
56
+ });
57
+ });
58
+ Object.entries(restExternal || {}).forEach(([key, values]) => {
59
+ const valuesToOverride = [];
60
+ values.forEach((egressValue) => {
61
+ if (typeof egressValue === 'string') {
62
+ valuesToOverride.push({
63
+ address: egressValue
64
+ });
65
+ }
66
+ else {
67
+ valuesToOverride.push(egressValue);
68
+ }
69
+ config.external[key] = valuesToOverride;
70
+ });
71
+ });
72
+ };
35
73
  const fixMissingPermissions = async (errors, warnings, state) => {
36
74
  const missingScopes = (0, exports.findMissingPermissions)(errors, warnings, 'missingPermission', state);
37
75
  const missingExternalFetchBackend = (0, exports.findMissingPermissions)(errors, warnings, 'missingExternalFetchPermission', state);
38
76
  const missingContentStyle = (0, exports.findMissingPermissions)(errors, warnings, 'missingContentStylePermission', state);
39
77
  const missingExternalImage = (0, exports.findMissingPermissions)(errors, warnings, 'missingExternalImagesPermission', state);
40
- const hasDeprecatedEgressPermissions = errors.some((error) => error.metadata?.hasDeprecatedEgressPermissions) ||
78
+ const hasDeprecatedEgressPermissions = (0, exports.deprecatedEgressPermissions)(warnings, state) ||
41
79
  missingExternalImage.size > 0 ||
42
80
  missingExternalFetchBackend.size > 0;
43
81
  const currentPermissions = (await state.configFile.readConfig())?.permissions ?? {};
@@ -78,35 +116,7 @@ const fixMissingPermissions = async (errors, warnings, state) => {
78
116
  };
79
117
  }
80
118
  if (hasDeprecatedEgressPermissions) {
81
- const { fetch: originalFetch, ...restExternal } = config.external ?? {};
82
- Object.entries(originalFetch || {}).forEach(([key, values]) => {
83
- const valuesToOverride = [];
84
- values.forEach((egressValue) => {
85
- if (typeof egressValue === 'string') {
86
- valuesToOverride.push({
87
- address: egressValue
88
- });
89
- }
90
- else {
91
- valuesToOverride.push(egressValue);
92
- }
93
- config.external.fetch[key] = valuesToOverride;
94
- });
95
- });
96
- Object.entries(restExternal || {}).forEach(([key, values]) => {
97
- const valuesToOverride = [];
98
- values.forEach((egressValue) => {
99
- if (typeof egressValue === 'string') {
100
- valuesToOverride.push({
101
- address: egressValue
102
- });
103
- }
104
- else {
105
- valuesToOverride.push(egressValue);
106
- }
107
- config.external[key] = valuesToOverride;
108
- });
109
- });
119
+ applyEgressPermissionsMigration(config);
110
120
  }
111
121
  await state.configFile.writeToConfigFile('permissions', config);
112
122
  return state;
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@forge/lint",
3
- "version": "5.9.1-next.8",
3
+ "version": "5.9.1-next.9-experimental-effab31",
4
4
  "description": "Linting for forge apps",
5
5
  "main": "out/index.js",
6
6
  "license": "SEE LICENSE IN LICENSE.txt",
@@ -11,7 +11,7 @@
11
11
  },
12
12
  "devDependencies": {
13
13
  "@atlassian/xen-test-util": "^4.2.0",
14
- "@forge/api": "^6.0.1-next.0",
14
+ "@forge/api": "^6.0.1-next.0-experimental-effab31",
15
15
  "@types/array.prototype.flatmap": "^1.2.6",
16
16
  "@types/cross-spawn": "^6.0.6",
17
17
  "@types/eslint": "8.56.12",
@@ -19,10 +19,10 @@
19
19
  "eslint-plugin-import": "^2.29.1"
20
20
  },
21
21
  "dependencies": {
22
- "@forge/cli-shared": "8.1.0-next.7",
23
- "@forge/csp": "4.0.1-next.1",
24
- "@forge/egress": "2.0.1-next.0",
25
- "@forge/manifest": "10.0.1-next.7",
22
+ "@forge/cli-shared": "8.1.0-next.7-experimental-effab31",
23
+ "@forge/csp": "4.0.1-next.1-experimental-effab31",
24
+ "@forge/egress": "2.0.1-next.0-experimental-effab31",
25
+ "@forge/manifest": "10.0.1-next.7-experimental-effab31",
26
26
  "@typescript-eslint/typescript-estree": "^5.62.0",
27
27
  "array.prototype.flatmap": "^1.3.3",
28
28
  "atlassian-openapi": "^1.0.21",