@forge/lint 5.9.1-next.1 → 5.9.1-next.10

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 +75 -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 -33
  5. package/package.json +12 -12
package/CHANGELOG.md CHANGED
@@ -1,5 +1,80 @@
1
1
  # @forge/lint
2
2
 
3
+ ## 5.9.1-next.10
4
+
5
+ ### Patch Changes
6
+
7
+ - Updated dependencies [46daa27]
8
+ - @forge/manifest@10.1.0-next.8
9
+ - @forge/cli-shared@8.1.0-next.8
10
+
11
+ ## 5.9.1-next.9
12
+
13
+ ### Patch Changes
14
+
15
+ - c3810aa: Fix linter not triggering auto-fix for some cases of deprecated egress permissions
16
+
17
+ ## 5.9.1-next.8
18
+
19
+ ### Patch Changes
20
+
21
+ - Updated dependencies [634d7b9]
22
+ - Updated dependencies [620aea7]
23
+ - @forge/manifest@10.0.1-next.7
24
+ - @forge/cli-shared@8.1.0-next.7
25
+
26
+ ## 5.9.1-next.7
27
+
28
+ ### Patch Changes
29
+
30
+ - 5d07fbf: Update the default value for inScopeEUD of egress entries by the linter
31
+ - Updated dependencies [5d07fbf]
32
+ - @forge/manifest@10.0.1-next.6
33
+ - @forge/cli-shared@8.0.1-next.6
34
+
35
+ ## 5.9.1-next.6
36
+
37
+ ### Patch Changes
38
+
39
+ - Updated dependencies [f5ba3aa]
40
+ - @forge/csp@4.0.1-next.1
41
+
42
+ ## 5.9.1-next.5
43
+
44
+ ### Patch Changes
45
+
46
+ - Updated dependencies [d97a252]
47
+ - Updated dependencies [ca7e661]
48
+ - @forge/manifest@10.0.1-next.5
49
+ - @forge/cli-shared@8.0.1-next.5
50
+
51
+ ## 5.9.1-next.4
52
+
53
+ ### Patch Changes
54
+
55
+ - 195411c: patch dependencies
56
+ - Updated dependencies [195411c]
57
+ - @forge/cli-shared@8.0.1-next.4
58
+ - @forge/manifest@10.0.1-next.4
59
+ - @forge/egress@2.0.1-next.0
60
+ - @forge/csp@4.0.1-next.0
61
+
62
+ ## 5.9.1-next.3
63
+
64
+ ### Patch Changes
65
+
66
+ - Updated dependencies [1479d2d]
67
+ - @forge/manifest@10.0.1-next.3
68
+ - @forge/cli-shared@8.0.1-next.3
69
+
70
+ ## 5.9.1-next.2
71
+
72
+ ### Patch Changes
73
+
74
+ - Updated dependencies [5ab2c49]
75
+ - @forge/manifest@10.0.1-next.2
76
+ - @forge/cli-shared@8.0.1-next.2
77
+
3
78
  ## 5.9.1-next.1
4
79
 
5
80
  ### 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,CAyGtB,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,37 +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
- inScopeEUD: false
89
- });
90
- }
91
- else {
92
- valuesToOverride.push(egressValue);
93
- }
94
- config.external.fetch[key] = valuesToOverride;
95
- });
96
- });
97
- Object.entries(restExternal || {}).forEach(([key, values]) => {
98
- const valuesToOverride = [];
99
- values.forEach((egressValue) => {
100
- if (typeof egressValue === 'string') {
101
- valuesToOverride.push({
102
- address: egressValue,
103
- inScopeEUD: false
104
- });
105
- }
106
- else {
107
- valuesToOverride.push(egressValue);
108
- }
109
- config.external[key] = valuesToOverride;
110
- });
111
- });
119
+ applyEgressPermissionsMigration(config);
112
120
  }
113
121
  await state.configFile.writeToConfigFile('permissions', config);
114
122
  return state;
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@forge/lint",
3
- "version": "5.9.1-next.1",
3
+ "version": "5.9.1-next.10",
4
4
  "description": "Linting for forge apps",
5
5
  "main": "out/index.js",
6
6
  "license": "SEE LICENSE IN LICENSE.txt",
@@ -11,25 +11,25 @@
11
11
  },
12
12
  "devDependencies": {
13
13
  "@atlassian/xen-test-util": "^4.2.0",
14
- "@forge/api": "^6.0.0",
14
+ "@forge/api": "^6.0.1-next.0",
15
15
  "@types/array.prototype.flatmap": "^1.2.6",
16
16
  "@types/cross-spawn": "^6.0.6",
17
- "@types/eslint": "8.56.2",
18
- "@types/node-fetch": "^2.6.11",
17
+ "@types/eslint": "8.56.12",
18
+ "@types/node-fetch": "^2.6.12",
19
19
  "eslint-plugin-import": "^2.29.1"
20
20
  },
21
21
  "dependencies": {
22
- "@forge/cli-shared": "8.0.1-next.1",
23
- "@forge/csp": "4.0.0",
24
- "@forge/egress": "2.0.0",
25
- "@forge/manifest": "10.0.1-next.1",
22
+ "@forge/cli-shared": "8.1.0-next.8",
23
+ "@forge/csp": "4.0.1-next.1",
24
+ "@forge/egress": "2.0.1-next.0",
25
+ "@forge/manifest": "10.1.0-next.8",
26
26
  "@typescript-eslint/typescript-estree": "^5.62.0",
27
- "array.prototype.flatmap": "^1.3.2",
28
- "atlassian-openapi": "^1.0.18",
29
- "cross-spawn": "^7.0.5",
27
+ "array.prototype.flatmap": "^1.3.3",
28
+ "atlassian-openapi": "^1.0.21",
29
+ "cross-spawn": "^7.0.6",
30
30
  "node-fetch": "2.7.0"
31
31
  },
32
32
  "publishConfig": {
33
33
  "registry": "https://packages.atlassian.com/api/npm/npm-public/"
34
34
  }
35
- }
35
+ }