@forge/lint 5.7.6-next.3-experimental-bc58a34 → 5.8.0-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.
- package/CHANGELOG.md +52 -9
- package/out/lint/linters/permission-linter/permission-linter.d.ts.map +1 -1
- package/out/lint/linters/permission-linter/permission-linter.js +2 -1
- package/out/lint/linters/permission-linter/verifiers/storage-api-verifier.d.ts.map +1 -1
- package/out/lint/linters/permission-linter/verifiers/storage-api-verifier.js +2 -1
- package/out/lint/linters/utils.d.ts +2 -0
- package/out/lint/linters/utils.d.ts.map +1 -1
- package/out/lint/linters/utils.js +3 -1
- package/package.json +4 -4
package/CHANGELOG.md
CHANGED
|
@@ -1,17 +1,60 @@
|
|
|
1
1
|
# @forge/lint
|
|
2
2
|
|
|
3
|
-
## 5.
|
|
3
|
+
## 5.8.0-next.10
|
|
4
4
|
|
|
5
5
|
### Patch Changes
|
|
6
6
|
|
|
7
|
-
- Updated dependencies [
|
|
8
|
-
-
|
|
9
|
-
-
|
|
10
|
-
|
|
11
|
-
-
|
|
12
|
-
|
|
13
|
-
|
|
14
|
-
|
|
7
|
+
- Updated dependencies [a56ca61]
|
|
8
|
+
- @forge/manifest@9.3.0-next.6
|
|
9
|
+
- @forge/cli-shared@6.11.0-next.10
|
|
10
|
+
|
|
11
|
+
## 5.8.0-next.9
|
|
12
|
+
|
|
13
|
+
### Patch Changes
|
|
14
|
+
|
|
15
|
+
- Updated dependencies [f6ec33c]
|
|
16
|
+
- @forge/cli-shared@6.11.0-next.9
|
|
17
|
+
|
|
18
|
+
## 5.8.0-next.8
|
|
19
|
+
|
|
20
|
+
### Patch Changes
|
|
21
|
+
|
|
22
|
+
- Updated dependencies [6db64a3]
|
|
23
|
+
- @forge/cli-shared@6.11.0-next.8
|
|
24
|
+
|
|
25
|
+
## 5.8.0-next.7
|
|
26
|
+
|
|
27
|
+
### Patch Changes
|
|
28
|
+
|
|
29
|
+
- Updated dependencies [6444f47]
|
|
30
|
+
- @forge/cli-shared@6.11.0-next.7
|
|
31
|
+
|
|
32
|
+
## 5.8.0-next.6
|
|
33
|
+
|
|
34
|
+
### Patch Changes
|
|
35
|
+
|
|
36
|
+
- Updated dependencies [439551a]
|
|
37
|
+
- @forge/manifest@9.3.0-next.5
|
|
38
|
+
- @forge/cli-shared@6.11.0-next.6
|
|
39
|
+
|
|
40
|
+
## 5.8.0-next.5
|
|
41
|
+
|
|
42
|
+
### Patch Changes
|
|
43
|
+
|
|
44
|
+
- Updated dependencies [9a2ca12]
|
|
45
|
+
- @forge/cli-shared@6.11.0-next.5
|
|
46
|
+
|
|
47
|
+
## 5.8.0-next.4
|
|
48
|
+
|
|
49
|
+
### Minor Changes
|
|
50
|
+
|
|
51
|
+
- 43832a8: Allow scopes to be declared with offline user impersonation support
|
|
52
|
+
|
|
53
|
+
### Patch Changes
|
|
54
|
+
|
|
55
|
+
- Updated dependencies [43832a8]
|
|
56
|
+
- @forge/manifest@9.3.0-next.4
|
|
57
|
+
- @forge/cli-shared@6.11.0-next.4
|
|
15
58
|
|
|
16
59
|
## 5.7.6-next.3
|
|
17
60
|
|
|
@@ -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;
|
|
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;AAWrB,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,CAkEtB,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"}
|
|
@@ -9,6 +9,7 @@ const api_call_interface_1 = require("./api-call-interface");
|
|
|
9
9
|
const verifiers_1 = require("./verifiers");
|
|
10
10
|
const visitors_1 = require("./visitors");
|
|
11
11
|
const fs_1 = tslib_1.__importDefault(require("fs"));
|
|
12
|
+
const utils_1 = require("../utils");
|
|
12
13
|
exports.NON_FWD_SLASH_REGEX = '[^\\/\\s]*';
|
|
13
14
|
const CACHE_KEY = 'PERMISSIONS_LINTER';
|
|
14
15
|
const ONE_HOUR_MS = 1000 * 60 * 60;
|
|
@@ -38,7 +39,7 @@ const fixMissingPermissions = async (errors, warnings, state) => {
|
|
|
38
39
|
const missingExternalImage = (0, exports.findMissingPermissions)(errors, warnings, 'missingExternalImagesPermission', state);
|
|
39
40
|
const currentPermissions = (await state.configFile.readConfig())?.permissions ?? {};
|
|
40
41
|
const currentRemotes = (await state.configFile.readConfig())?.remotes ?? [];
|
|
41
|
-
const currentScopes = currentPermissions?.scopes ?? [];
|
|
42
|
+
const currentScopes = (0, utils_1.getAllScopeKeys)(currentPermissions?.scopes ?? []);
|
|
42
43
|
currentScopes.forEach((scope) => missingScopes.add(scope));
|
|
43
44
|
const currentExternalFetchBackend = currentPermissions?.external?.fetch?.backend ?? [];
|
|
44
45
|
currentExternalFetchBackend
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"storage-api-verifier.d.ts","sourceRoot":"","sources":["../../../../../src/lint/linters/permission-linter/verifiers/storage-api-verifier.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AACtE,OAAO,EAAE,OAAO,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAChE,OAAO,EAAE,iBAAiB,EAAE,qBAAqB,EAAE,MAAM,0BAA0B,CAAC;
|
|
1
|
+
{"version":3,"file":"storage-api-verifier.d.ts","sourceRoot":"","sources":["../../../../../src/lint/linters/permission-linter/verifiers/storage-api-verifier.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AACtE,OAAO,EAAE,OAAO,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAChE,OAAO,EAAE,iBAAiB,EAAE,qBAAqB,EAAE,MAAM,0BAA0B,CAAC;AAMpF,qBAAa,kBAAmB,SAAQ,qBAAsB,YAAW,iBAAiB,CAAC,OAAO,EAAE,CAAC;IACnG,SAAS,CAAC,YAAY,IAAI,SAAS;IAItB,OAAO,CAAC,QAAQ,EAAE,cAAc,EAAE,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC;CAe5E"}
|
|
@@ -4,13 +4,14 @@ exports.StorageAPIVerifier = void 0;
|
|
|
4
4
|
const linter_interface_1 = require("../../../linter-interface");
|
|
5
5
|
const verifier_interface_1 = require("../../verifier-interface");
|
|
6
6
|
const text_1 = require("../../../text");
|
|
7
|
+
const utils_1 = require("../../utils");
|
|
7
8
|
const STORAGE_PERMISSION = 'storage:app';
|
|
8
9
|
class StorageAPIVerifier extends verifier_interface_1.BaseLintIssueVerifier {
|
|
9
10
|
getLintClass() {
|
|
10
11
|
return linter_interface_1.LintClass.Error;
|
|
11
12
|
}
|
|
12
13
|
async process(apiCalls) {
|
|
13
|
-
if (this.manifest.permissions?.scopes
|
|
14
|
+
if ((0, utils_1.getAllScopeKeys)(this.manifest.permissions?.scopes ?? []).includes(STORAGE_PERMISSION)) {
|
|
14
15
|
return [];
|
|
15
16
|
}
|
|
16
17
|
return apiCalls.map((apiCall) => ({
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../src/lint/linters/utils.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,aAAa,eAAgB;IAAE,GAAG,EAAE,MAAM,CAAA;CAAE,WAMxD,CAAC"}
|
|
1
|
+
{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../src/lint/linters/utils.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,EAAE,MAAM,iBAAiB,CAAC;AAEzC,eAAO,MAAM,aAAa,eAAgB;IAAE,GAAG,EAAE,MAAM,CAAA;CAAE,WAMxD,CAAC;AAEF,eAAO,MAAM,eAAe,WAAY,MAAM,aAA2D,CAAC"}
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
"use strict";
|
|
2
2
|
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
-
exports.extractDomain = void 0;
|
|
3
|
+
exports.getAllScopeKeys = exports.extractDomain = void 0;
|
|
4
4
|
const extractDomain = (urlAstNode) => {
|
|
5
5
|
try {
|
|
6
6
|
return new URL(urlAstNode.url).hostname;
|
|
@@ -10,3 +10,5 @@ const extractDomain = (urlAstNode) => {
|
|
|
10
10
|
}
|
|
11
11
|
};
|
|
12
12
|
exports.extractDomain = extractDomain;
|
|
13
|
+
const getAllScopeKeys = (scopes) => (Array.isArray(scopes) ? scopes : Object.keys(scopes));
|
|
14
|
+
exports.getAllScopeKeys = getAllScopeKeys;
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@forge/lint",
|
|
3
|
-
"version": "5.
|
|
3
|
+
"version": "5.8.0-next.10",
|
|
4
4
|
"description": "Linting for forge apps",
|
|
5
5
|
"main": "out/index.js",
|
|
6
6
|
"license": "SEE LICENSE IN LICENSE.txt",
|
|
@@ -19,10 +19,10 @@
|
|
|
19
19
|
"eslint-plugin-import": "^2.29.1"
|
|
20
20
|
},
|
|
21
21
|
"dependencies": {
|
|
22
|
-
"@forge/cli-shared": "6.11.0-next.
|
|
23
|
-
"@forge/csp": "3.7.1
|
|
22
|
+
"@forge/cli-shared": "6.11.0-next.10",
|
|
23
|
+
"@forge/csp": "3.7.1",
|
|
24
24
|
"@forge/egress": "1.4.0",
|
|
25
|
-
"@forge/manifest": "9.3.0-next.
|
|
25
|
+
"@forge/manifest": "9.3.0-next.6",
|
|
26
26
|
"@typescript-eslint/typescript-estree": "^5.62.0",
|
|
27
27
|
"array.prototype.flatmap": "^1.3.2",
|
|
28
28
|
"atlassian-openapi": "^1.0.18",
|