@snowbridge/contract-types 0.3.3 → 0.4.1-beta.1

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.
@@ -1,4832 +1,10 @@
1
-  WARN  Issue while reading "/home/runner/work/snowbridge/snowbridge/web/.npmrc". Failed to replace env in config: ${NPM_AUTH_TOKEN}
2
1
 
3
- > @snowbridge/contract-types@0.3.0 build /home/runner/work/snowbridge/snowbridge/web/packages/contract-types
2
+ > @snowbridge/contract-types@0.4.0 build /home/runner/work/snowbridge/snowbridge/web/packages/contract-types
4
3
  > rm -rf src && rm -rf dist && cd ../../../contracts && forge build && cd ../web/packages/contract-types && pnpm typechain && tsc --build --force
5
4
 
6
5
  No files changed, compilation skipped
7
- warning[incorrect-shift]: the order of args in a shift operation is incorrect
8
- --> src/utils/SparseBitmap.sol:12:20
9
- |
10
- 12 | uint256 mask = 1 << (index & 0xff);
11
- | ^^^^^^^^^^^^^^^^^^^
12
- |
13
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#incorrect-shift
14
6
 
15
- note[unsafe-cheatcode]: usage of unsafe cheatcodes that can perform dangerous operations
16
- --> test/MMRProof.t.sol:35:26
17
- |
18
- 35 | fixtureData = vm.readFile(path).parseRaw("");
19
- | ^^^^^^^^
20
- |
21
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-cheatcode
22
-
23
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
24
- --> src/utils/Uint16Array.sol:67:20
25
- |
26
- 67 | uint8 inside = uint8(index) & 0x0F;
27
- | ^^^^^^^^^^^^
28
- |
29
- = note: consider disabling this lint if you're certain the cast is safe
30
-
31
- // casting to 'uint8' is safe because [explain why]
32
- // forge-lint: disable-next-line(unsafe-typecast)
33
-
34
-
35
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
36
-
37
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
38
- --> src/utils/Uint16Array.sol:86:20
39
- |
40
- 86 | uint8 inside = uint8(index) & 0x0F;
41
- | ^^^^^^^^^^^^
42
- |
43
- = note: consider disabling this lint if you're certain the cast is safe
44
-
45
- // casting to 'uint8' is safe because [explain why]
46
- // forge-lint: disable-next-line(unsafe-typecast)
47
-
48
-
49
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
50
-
51
- note[screaming-snake-case-immutable]: immutables should use SCREAMING_SNAKE_CASE
52
- --> src/Token.sol:17:30
53
- |
54
- 17 | address public immutable gateway;
55
- | ^^^^^^^ help: consider using: `GATEWAY`
56
- |
57
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-immutable
58
-
59
- note[screaming-snake-case-immutable]: immutables should use SCREAMING_SNAKE_CASE
60
- --> src/Token.sol:18:28
61
- |
62
- 18 | uint8 public immutable decimals;
63
- | ^^^^^^^^ help: consider using: `DECIMALS`
64
- |
65
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-immutable
66
-
67
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
68
- --> test/Token.t.sol:4:8
69
- |
70
- 4 | import "forge-std/Test.sol";
71
- | ^^^^^^^^^^^^^^^^^^^^
72
- |
73
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
74
-
75
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
76
- --> test/Token.t.sol:5:8
77
- |
78
- 5 | import "forge-std/console.sol";
79
- | ^^^^^^^^^^^^^^^^^^^^^^^
80
- |
81
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
82
-
83
- warning[incorrect-shift]: the order of args in a shift operation is incorrect
84
- --> src/utils/SparseBitmap.sol:18:20
85
- |
86
- 18 | uint256 mask = 1 << (index & 0xff);
87
- | ^^^^^^^^^^^^^^^^^^^
88
- |
89
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#incorrect-shift
90
-
91
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
92
- --> test/mocks/BeefyClientMock.sol:6:8
93
- |
94
- 6 | import "forge-std/console.sol";
95
- | ^^^^^^^^^^^^^^^^^^^^^^^
96
- |
97
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
98
-
99
- note[unwrapped-modifier-logic]: wrap modifier logic to reduce code size
100
- --> src/Token.sol:37:5
101
- |
102
- 37 | / modifier onlyGateway() {
103
- 38 | | if (msg.sender != gateway) {
104
- 39 | | revert Unauthorized();
105
- ... |
106
- 42 | | }
107
- | |_____^
108
- |
109
- help: wrap modifier logic to reduce code size
110
- |
111
- 37 ~ modifier onlyGateway() {
112
- 38 + _onlyGateway();
113
- 39 + _;
114
- 40 + }
115
- 41 +
116
- 42 + function _onlyGateway() internal {
117
- 43 + if (msg.sender != gateway) {
118
- 44 + revert Unauthorized();
119
- 45 + }
120
- 46 + }
121
- |
122
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unwrapped-modifier-logic
123
-
124
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
125
- --> src/TokenLib.sol:76:26
126
- |
127
- 76 | bytes32 digest = keccak256(
128
- | __________________________^
129
- 77 | | abi.encodePacked(
130
- 78 | | hex"1901",
131
- 79 | | _domainSeparator(tokenName),
132
- ... |
133
- 86 | | );
134
- | |_________^
135
- |
136
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
137
-
138
- warning[incorrect-shift]: the order of args in a shift operation is incorrect
139
- --> src/utils/Math.sol:82:60
140
- |
141
- 82 | return result + (unsignedRoundsUp(rounding) && 1 << result < value ? 1 : 0);
142
- | ^^^^^^^^^^^
143
- |
144
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#incorrect-shift
145
-
146
- note[unused-import]: unused imports should be removed
147
- --> test/MMRProof.t.sol:5:9
148
- |
149
- 5 | import {console} from "forge-std/console.sol";
150
- | ^^^^^^^
151
- |
152
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
153
-
154
- note[mixed-case-function]: function names should use mixedCase
155
- --> test/mocks/MockGateway.sol:26:14
156
- |
157
- 26 | function v1_handleAgentExecute_public(bytes calldata params) external {
158
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleAgentExecutePublic`
159
- |
160
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
161
-
162
- note[mixed-case-function]: function names should use mixedCase
163
- --> test/mocks/MockGateway.sol:30:14
164
- |
165
- 30 | function v1_handleUpgrade_public(bytes calldata params) external {
166
- | ^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleUpgradePublic`
167
- |
168
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
169
-
170
- note[mixed-case-function]: function names should use mixedCase
171
- --> test/mocks/MockGateway.sol:34:14
172
- |
173
- 34 | function v1_handleSetOperatingMode_public(bytes calldata params) external {
174
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleSetOperatingModePublic`
175
- |
176
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
177
-
178
- note[mixed-case-function]: function names should use mixedCase
179
- --> test/mocks/BeefyClientMock.sol:29:14
180
- |
181
- 29 | function encodeCommitment_public(Commitment calldata commitment)
182
- | ^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `encodeCommitmentPublic`
183
- |
184
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
185
-
186
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
187
- --> src/utils/Bitfield.sol:181:34
188
- |
189
- 181 | return self[element].bit(uint8(index)) == 1;
190
- | ^^^^^^^^^^^^
191
- |
192
- = note: consider disabling this lint if you're certain the cast is safe
193
-
194
- // casting to 'uint8' is safe because [explain why]
195
- // forge-lint: disable-next-line(unsafe-typecast)
196
-
197
-
198
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
199
-
200
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
201
- --> src/utils/Bitfield.sol:186:46
202
- |
203
- 186 | self[element] = self[element].setBit(uint8(index));
204
- | ^^^^^^^^^^^^
205
- |
206
- = note: consider disabling this lint if you're certain the cast is safe
207
-
208
- // casting to 'uint8' is safe because [explain why]
209
- // forge-lint: disable-next-line(unsafe-typecast)
210
-
211
-
212
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
213
-
214
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
215
- --> src/utils/Bitfield.sol:191:48
216
- |
217
- 191 | self[element] = self[element].clearBit(uint8(index));
218
- | ^^^^^^^^^^^^
219
- |
220
- = note: consider disabling this lint if you're certain the cast is safe
221
-
222
- // casting to 'uint8' is safe because [explain why]
223
- // forge-lint: disable-next-line(unsafe-typecast)
224
-
225
-
226
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
227
-
228
- note[mixed-case-function]: function names should use mixedCase
229
- --> test/mocks/BeefyClientMock.sol:45:14
230
- |
231
- 45 | function initialize_public(
232
- | ^^^^^^^^^^^^^^^^^ help: consider using: `initializePublic`
233
- |
234
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
235
-
236
- note[mixed-case-variable]: mutable variables should use mixedCase
237
- --> src/storage/CoreStorage.sol:17:17
238
- |
239
- 17 | uint256 __gap;
240
- | ^^^^^ help: consider using: `_gap`
241
- |
242
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
243
-
244
- note[unused-import]: unused imports should be removed
245
- --> src/l2-integration/SnowbridgeL1Adaptor.sol:7:21
246
- |
247
- 7 | import {ISpokePool, IMessageHandler} from "./interfaces/ISpokePool.sol";
248
- | ^^^^^^^^^^^^^^^
249
- |
250
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
251
-
252
- note[unused-import]: unused imports should be removed
253
- --> src/l2-integration/SnowbridgeL1Adaptor.sol:8:24
254
- |
255
- 8 | import {DepositParams, Instructions, Call} from "./Types.sol";
256
- | ^^^^^^^^^^^^
257
- |
258
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
259
-
260
- note[unused-import]: unused imports should be removed
261
- --> src/l2-integration/SnowbridgeL1Adaptor.sol:8:38
262
- |
263
- 8 | import {DepositParams, Instructions, Call} from "./Types.sol";
264
- | ^^^^
265
- |
266
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
267
-
268
- note[mixed-case-function]: function names should use mixedCase
269
- --> test/mocks/MockGateway.sol:38:14
270
- |
271
- 38 | function v1_handleSetTokenTransferFees_public(bytes calldata params) external {
272
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleSetTokenTransferFeesPublic`
273
- |
274
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
275
-
276
- note[mixed-case-variable]: mutable variables should use mixedCase
277
- --> test/Token.t.sol:249:17
278
- |
279
- 249 | bytes32 PERMIT_TYPEHASH = keccak256(
280
- | ^^^^^^^^^^^^^^^ help: consider using: `permitTypehash`
281
- |
282
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
283
-
284
- note[unsafe-cheatcode]: usage of unsafe cheatcodes that can perform dangerous operations
285
- --> test/Bitfield.t.sol:21:16
286
- |
287
- 21 | vm.readFile(string.concat(vm.projectRoot(), "/test/data/beefy-validator-set.json"));
288
- | ^^^^^^^^
289
- |
290
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-cheatcode
291
-
292
- note[mixed-case-function]: function names should use mixedCase
293
- --> test/mocks/MockGateway.sol:42:14
294
- |
295
- 42 | function v1_handleSetPricingParameters_public(bytes calldata params) external {
296
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleSetPricingParametersPublic`
297
- |
298
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
299
-
300
- note[mixed-case-variable]: mutable variables should use mixedCase
301
- --> src/interfaces/IGatewayBase.sol:34:50
302
- |
303
- 34 | event ForeignTokenRegistered(bytes32 indexed tokenID, address token);
304
- | ^^^^^^^ help: consider using: `tokenId`
305
- |
306
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
307
-
308
- note[mixed-case-function]: function names should use mixedCase
309
- --> test/mocks/BeefyClientMock.sol:89:14
310
- |
311
- 89 | function computeNumRequiredSignatures_public(
312
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `computeNumRequiredSignaturesPublic`
313
- |
314
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
315
-
316
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
317
- --> src/TokenLib.sol:99:16
318
- |
319
- 99 | return keccak256(
320
- | ________________^
321
- 100 | | abi.encode(
322
- 101 | | DOMAIN_TYPEHASH,
323
- 102 | | keccak256(bytes(name)),
324
- ... |
325
- 107 | | );
326
- | |_________^
327
- |
328
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
329
-
330
- note[unsafe-cheatcode]: usage of unsafe cheatcodes that can perform dangerous operations
331
- --> test/Bitfield.t.sol:131:16
332
- |
333
- 131 | vm.readFile(string.concat(vm.projectRoot(), "/test/data/beefy-validator-set.json"));
334
- | ^^^^^^^^
335
- |
336
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-cheatcode
337
-
338
- note[mixed-case-variable]: mutable variables should use mixedCase
339
- --> test/Token.t.sol:286:17
340
- |
341
- 286 | bytes32 PERMIT_TYPEHASH = keccak256(
342
- | ^^^^^^^^^^^^^^^ help: consider using: `permitTypehash`
343
- |
344
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
345
-
346
- note[mixed-case-variable]: mutable variables should use mixedCase
347
- --> src/storage/AssetsStorage.sol:13:16
348
- |
349
- 13 | ParaID assetHubParaID;
350
- | ^^^^^^^^^^^^^^ help: consider using: `assetHubParaId`
351
- |
352
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
353
-
354
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
355
- --> test/GatewayV1.t.sol:68:8
356
- |
357
- 68 | import "./mocks/HighGasToken.sol";
358
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^
359
- |
360
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
361
-
362
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
363
- --> test/GatewayV1.t.sol:69:8
364
- |
365
- 69 | import "./mocks/FeeOnTransferToken.sol";
366
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
367
- |
368
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
369
-
370
- note[mixed-case-variable]: mutable variables should use mixedCase
371
- --> src/Functions.sol:31:34
372
- |
373
- 31 | function ensureAgent(bytes32 agentID) internal view returns (address agent) {
374
- | ^^^^^^^ help: consider using: `agentId`
375
- |
376
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
377
-
378
- note[mixed-case-variable]: mutable variables should use mixedCase
379
- --> src/Functions.sol:39:38
380
- |
381
- 39 | function ensureChannel(ChannelID channelID) internal view returns (Channel storage ch) {
382
- | ^^^^^^^^^ help: consider using: `channelId`
383
- |
384
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
385
-
386
- note[mixed-case-variable]: mutable variables should use mixedCase
387
- --> src/types/Common.sol:14:13
388
- |
389
- 14 | bytes32 foreignID;
390
- | ^^^^^^^^^ help: consider using: `foreignId`
391
- |
392
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
393
-
394
- note[mixed-case-variable]: mutable variables should use mixedCase
395
- --> test/GatewayV1.t.sol:75:27
396
- |
397
- 75 | ChannelID indexed channelID, uint64 nonce, bytes32 indexed messageID, bool success
398
- | ^^^^^^^^^ help: consider using: `channelId`
399
- |
400
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
401
-
402
- note[mixed-case-variable]: mutable variables should use mixedCase
403
- --> test/GatewayV1.t.sol:75:68
404
- |
405
- 75 | ChannelID indexed channelID, uint64 nonce, bytes32 indexed messageID, bool success
406
- | ^^^^^^^^^ help: consider using: `messageId`
407
- |
408
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
409
-
410
- note[mixed-case-variable]: mutable variables should use mixedCase
411
- --> test/GatewayV1.t.sol:78:19
412
- |
413
- 78 | ParaID public bridgeHubParaID = ParaID.wrap(1013);
414
- | ^^^^^^^^^^^^^^^ help: consider using: `bridgeHubParaId`
415
- |
416
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
417
-
418
- note[mixed-case-variable]: mutable variables should use mixedCase
419
- --> test/GatewayV1.t.sol:80:20
420
- |
421
- 80 | bytes32 public bridgeHubAgentID =
422
- | ^^^^^^^^^^^^^^^^ help: consider using: `bridgeHubAgentId`
423
- |
424
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
425
-
426
- note[mixed-case-variable]: mutable variables should use mixedCase
427
- --> test/GatewayV1.t.sol:84:19
428
- |
429
- 84 | ParaID public assetHubParaID = ParaID.wrap(1000);
430
- | ^^^^^^^^^^^^^^ help: consider using: `assetHubParaId`
431
- |
432
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
433
-
434
- note[mixed-case-variable]: mutable variables should use mixedCase
435
- --> test/GatewayV1.t.sol:85:20
436
- |
437
- 85 | bytes32 public assetHubAgentID =
438
- | ^^^^^^^^^^^^^^^ help: consider using: `assetHubAgentId`
439
- |
440
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
441
-
442
- note[mixed-case-variable]: mutable variables should use mixedCase
443
- --> test/Token.t.sol:317:17
444
- |
445
- 317 | bytes32 PERMIT_TYPEHASH = keccak256(
446
- | ^^^^^^^^^^^^^^^ help: consider using: `permitTypehash`
447
- |
448
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
449
-
450
- note[mixed-case-variable]: mutable variables should use mixedCase
451
- --> src/Functions.sol:122:17
452
- |
453
- 122 | bytes32 foreignTokenID,
454
- | ^^^^^^^^^^^^^^ help: consider using: `foreignTokenId`
455
- |
456
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
457
-
458
- note[mixed-case-variable]: mutable variables should use mixedCase
459
- --> test/Token.t.sol:558:17
460
- |
461
- 558 | bytes32 PERMIT_TYPEHASH = keccak256(
462
- | ^^^^^^^^^^^^^^^ help: consider using: `permitTypehash`
463
- |
464
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
465
-
466
- note[mixed-case-variable]: mutable variables should use mixedCase
467
- --> test/Token.t.sol:573:17
468
- |
469
- 573 | bytes32 DOMAIN_SEPARATOR = token.DOMAIN_SEPARATOR();
470
- | ^^^^^^^^^^^^^^^^ help: consider using: `domainSeparator`
471
- |
472
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
473
-
474
- note[mixed-case-variable]: mutable variables should use mixedCase
475
- --> src/Functions.sol:142:39
476
- |
477
- 142 | function mintForeignToken(bytes32 foreignTokenID, address recipient, uint128 amount)
478
- | ^^^^^^^^^^^^^^ help: consider using: `foreignTokenId`
479
- |
480
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
481
-
482
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
483
- --> src/interfaces/IERC20Metadata.sol:7:8
484
- |
485
- 7 | import "./IERC20.sol";
486
- | ^^^^^^^^^^^^^^
487
- |
488
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
489
-
490
- note[unsafe-cheatcode]: usage of unsafe cheatcodes that can perform dangerous operations
491
- --> test/Bitfield.t.sol:150:16
492
- |
493
- 150 | vm.readFile(string.concat(vm.projectRoot(), "/test/data/beefy-validator-set.json"));
494
- | ^^^^^^^^
495
- |
496
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-cheatcode
497
-
498
- note[mixed-case-variable]: mutable variables should use mixedCase
499
- --> test/GatewayV1.t.sol:107:20
500
- |
501
- 107 | bytes32 public messageID = keccak256("cabbage");
502
- | ^^^^^^^^^ help: consider using: `messageId`
503
- |
504
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
505
-
506
- note[mixed-case-variable]: mutable variables should use mixedCase
507
- --> test/Token.t.sol:684:17
508
- |
509
- 684 | bytes32 PERMIT_TYPEHASH = keccak256(
510
- | ^^^^^^^^^^^^^^^ help: consider using: `permitTypehash`
511
- |
512
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
513
-
514
- note[mixed-case-function]: function names should use mixedCase
515
- --> test/mocks/BeefyClientMock.sol:97:14
516
- |
517
- 97 | function computeQuorum_public(uint256 numValidators) public pure returns (uint256) {
518
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using: `computeQuorumPublic`
519
- |
520
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
521
-
522
- note[mixed-case-function]: function names should use mixedCase
523
- --> test/mocks/BeefyClientMock.sol:101:14
524
- |
525
- 101 | function computeMaxRequiredSignatures_public(uint256 numValidators)
526
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `computeMaxRequiredSignaturesPublic`
527
- |
528
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
529
-
530
- note[unused-import]: unused imports should be removed
531
- --> src/types/Common.sol:5:9
532
- |
533
- 5 | import {UD60x18} from "prb/math/src/UD60x18.sol";
534
- | ^^^^^^^
535
- |
536
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
537
-
538
- note[mixed-case-variable]: mutable variables should use mixedCase
539
- --> src/Functions.sol:154:44
540
- |
541
- 154 | function _ensureTokenAddressOf(bytes32 tokenID) internal view returns (address) {
542
- | ^^^^^^^ help: consider using: `tokenId`
543
- |
544
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
545
-
546
- note[unused-import]: unused imports should be removed
547
- --> test/Bitfield.t.sol:5:9
548
- |
549
- 5 | import {console} from "forge-std/console.sol";
550
- | ^^^^^^^
551
- |
552
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
553
-
554
- note[mixed-case-function]: function names should use mixedCase
555
- --> test/mocks/MockGateway.sol:46:14
556
- |
557
- 46 | function v1_handleUnlockNativeToken_public(bytes calldata params) external {
558
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleUnlockNativeTokenPublic`
559
- |
560
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
561
-
562
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
563
- --> test/SparseBitmap.t.sol:4:8
564
- |
565
- 4 | import "forge-std/Test.sol";
566
- | ^^^^^^^^^^^^^^^^^^^^
567
- |
568
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
569
-
570
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
571
- --> test/SparseBitmap.t.sol:5:8
572
- |
573
- 5 | import "forge-std/console.sol";
574
- | ^^^^^^^^^^^^^^^^^^^^^^^
575
- |
576
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
577
-
578
- note[mixed-case-function]: function names should use mixedCase
579
- --> test/mocks/MockGateway.sol:50:14
580
- |
581
- 50 | function v1_handleRegisterForeignToken_public(bytes calldata params) external {
582
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleRegisterForeignTokenPublic`
583
- |
584
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
585
-
586
- note[mixed-case-function]: function names should use mixedCase
587
- --> test/mocks/BeefyClientMock.sol:113:14
588
- |
589
- 113 | function createTicketID_public(address relayer, bytes32 commitmentHash)
590
- | ^^^^^^^^^^^^^^^^^^^^^ help: consider using: `createTicketIdPublic`
591
- |
592
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
593
-
594
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
595
- --> src/utils/ScaleCodec.sol:114:37
596
- |
597
- 114 | return abi.encodePacked(uint8(value << 2));
598
- | ^^^^^^^^^^^^^^^^^
599
- |
600
- = note: consider disabling this lint if you're certain the cast is safe
601
-
602
- // casting to 'uint8' is safe because [explain why]
603
- // forge-lint: disable-next-line(unsafe-typecast)
604
-
605
-
606
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
607
-
608
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
609
- --> src/utils/ScaleCodec.sol:131:37
610
- |
611
- 131 | return abi.encodePacked(uint8(value << 2));
612
- | ^^^^^^^^^^^^^^^^^
613
- |
614
- = note: consider disabling this lint if you're certain the cast is safe
615
-
616
- // casting to 'uint8' is safe because [explain why]
617
- // forge-lint: disable-next-line(unsafe-typecast)
618
-
619
-
620
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
621
-
622
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
623
- --> src/utils/ScaleCodec.sol:138:30
624
- |
625
- 138 | uint16 encoded = uint16(value << 2) | 0x01;
626
- | ^^^^^^^^^^^^^^^^^^
627
- |
628
- = note: consider disabling this lint if you're certain the cast is safe
629
-
630
- // casting to 'uint16' is safe because [explain why]
631
- // forge-lint: disable-next-line(unsafe-typecast)
632
-
633
-
634
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
635
-
636
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
637
- --> src/utils/ScaleCodec.sol:147:31
638
- |
639
- 147 | uint32 encoded = (uint32(value) << 2) | 0x02;
640
- | ^^^^^^^^^^^^^
641
- |
642
- = note: consider disabling this lint if you're certain the cast is safe
643
-
644
- // casting to 'uint32' is safe because [explain why]
645
- // forge-lint: disable-next-line(unsafe-typecast)
646
-
647
-
648
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
649
-
650
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
651
- --> src/utils/ScaleCodec.sol:180:29
652
- |
653
- 180 | buf[i] = bytes1(uint8(current & 0xFF));
654
- | ^^^^^^^^^^^^^^^^^^^^^
655
- |
656
- = note: consider disabling this lint if you're certain the cast is safe
657
-
658
- // casting to 'uint8' is safe because [explain why]
659
- // forge-lint: disable-next-line(unsafe-typecast)
660
-
661
-
662
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
663
-
664
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
665
- --> src/utils/ScaleCodec.sol:199:33
666
- |
667
- 199 | return encodeCompactU32(uint32(value));
668
- | ^^^^^^^^^^^^^
669
- |
670
- = note: consider disabling this lint if you're certain the cast is safe
671
-
672
- // casting to 'uint32' is safe because [explain why]
673
- // forge-lint: disable-next-line(unsafe-typecast)
674
-
675
-
676
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
677
-
678
- note[mixed-case-function]: function names should use mixedCase
679
- --> src/SubstrateTypes.sol:19:14
680
- |
681
- 19 | function H160(address account) internal pure returns (bytes memory) {
682
- | ^^^^ help: consider using: `h160`
683
- |
684
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
685
-
686
- note[mixed-case-function]: function names should use mixedCase
687
- --> src/SubstrateTypes.sol:23:14
688
- |
689
- 23 | function VecU8(bytes memory input) internal pure returns (bytes memory) {
690
- | ^^^^^ help: consider using: `vecU8`
691
- |
692
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
693
-
694
- note[mixed-case-function]: function names should use mixedCase
695
- --> src/SubstrateTypes.sol:33:14
696
- |
697
- 33 | function None() internal pure returns (bytes memory) {
698
- | ^^^^ help: consider using: `none`
699
- |
700
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
701
-
702
- note[mixed-case-function]: function names should use mixedCase
703
- --> src/SubstrateTypes.sol:42:14
704
- |
705
- 42 | function RegisterToken(address token, uint128 fee) internal view returns (bytes memory) {
706
- | ^^^^^^^^^^^^^ help: consider using: `registerToken`
707
- |
708
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
709
-
710
- note[mixed-case-function]: function names should use mixedCase
711
- --> src/SubstrateTypes.sol:57:14
712
- |
713
- 57 | function SendTokenToAssetHubAddress32(
714
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendTokenToAssetHubAddress32`
715
- |
716
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
717
-
718
- note[mixed-case-function]: function names should use mixedCase
719
- --> src/SubstrateTypes.sol:76:14
720
- |
721
- 76 | function SendTokenToAddress32(
722
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendTokenToAddress32`
723
- |
724
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
725
-
726
- note[mixed-case-variable]: mutable variables should use mixedCase
727
- --> src/SubstrateTypes.sol:78:16
728
- |
729
- 78 | ParaID paraID,
730
- | ^^^^^^ help: consider using: `paraId`
731
- |
732
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
733
-
734
- note[mixed-case-function]: function names should use mixedCase
735
- --> src/SubstrateTypes.sol:99:14
736
- |
737
- 99 | function SendTokenToAddress20(
738
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendTokenToAddress20`
739
- |
740
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
741
-
742
- note[unused-import]: unused imports should be removed
743
- --> test/Token.t.sol:10:9
744
- |
745
- 10 | import {TokenLib} from "../src/TokenLib.sol";
746
- | ^^^^^^^^
747
- |
748
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
749
-
750
- note[mixed-case-function]: function names should use mixedCase
751
- --> test/mocks/MockGateway.sol:54:14
752
- |
753
- 54 | function v1_handleMintForeignToken_public(ChannelID channelID, bytes calldata params)
754
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleMintForeignTokenPublic`
755
- |
756
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
757
-
758
- note[mixed-case-variable]: mutable variables should use mixedCase
759
- --> test/BeefyClient.t.sol:42:12
760
- |
761
- 42 | bytes2 mmrRootID = bytes2("mh");
762
- | ^^^^^^^^^ help: consider using: `mmrRootId`
763
- |
764
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
765
-
766
- note[unused-import]: unused imports should be removed
767
- --> test/mocks/BeefyClientMock.sol:5:9
768
- |
769
- 5 | import {Uint16Array, createUint16Array} from "../../src/utils/Uint16Array.sol";
770
- | ^^^^^^^^^^^
771
- |
772
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
773
-
774
- note[mixed-case-variable]: mutable variables should use mixedCase
775
- --> src/SubstrateTypes.sol:101:16
776
- |
777
- 101 | ParaID paraID,
778
- | ^^^^^^ help: consider using: `paraId`
779
- |
780
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
781
-
782
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
783
- --> test/ScaleCodec.t.sol:4:8
784
- |
785
- 4 | import "forge-std/Test.sol";
786
- | ^^^^^^^^^^^^^^^^^^^^
787
- |
788
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
789
-
790
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
791
- --> test/ScaleCodec.t.sol:5:8
792
- |
793
- 5 | import "forge-std/console.sol";
794
- | ^^^^^^^^^^^^^^^^^^^^^^^
795
- |
796
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
797
-
798
- note[unsafe-cheatcode]: usage of unsafe cheatcodes that can perform dangerous operations
799
- --> test/BeefyClient.t.sol:59:47
800
- |
801
- 59 | string memory beefyCommitmentRaw = vm.readFile(beefyCommitmentFile);
802
- | ^^^^^^^^
803
- |
804
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-cheatcode
805
-
806
- note[unsafe-cheatcode]: usage of unsafe cheatcodes that can perform dangerous operations
807
- --> test/BeefyClient.t.sol:75:49
808
- |
809
- 75 | string memory beefyValidatorSetRaw = vm.readFile(beefyValidatorSetFile);
810
- | ^^^^^^^^
811
- |
812
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-cheatcode
813
-
814
- note[unsafe-cheatcode]: usage of unsafe cheatcodes that can perform dangerous operations
815
- --> test/BeefyClient.t.sol:98:42
816
- |
817
- 98 | string memory finalProofRaw = vm.readFile(finalProofFile);
818
- | ^^^^^^^^
819
- |
820
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-cheatcode
821
-
822
- note[unsafe-cheatcode]: usage of unsafe cheatcodes that can perform dangerous operations
823
- --> test/BeefyClient.t.sol:103:47
824
- |
825
- 103 | string memory fiatShamirProofRaw = vm.readFile(fiatShamirProofFile);
826
- | ^^^^^^^^
827
- |
828
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-cheatcode
829
-
830
- note[unused-import]: unused imports should be removed
831
- --> test/mocks/ReantrantAttacker.sol:7:9
832
- |
833
- 7 | import {console} from "forge-std/console.sol";
834
- | ^^^^^^^
835
- |
836
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
837
-
838
- note[mixed-case-variable]: mutable variables should use mixedCase
839
- --> test/mocks/MockGateway.sol:54:57
840
- |
841
- 54 | function v1_handleMintForeignToken_public(ChannelID channelID, bytes calldata params)
842
- | ^^^^^^^^^ help: consider using: `channelId`
843
- |
844
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
845
-
846
- note[mixed-case-variable]: mutable variables should use mixedCase
847
- --> test/GatewayV1.t.sol:124:20
848
- |
849
- 124 | bytes32 public dotTokenID;
850
- | ^^^^^^^^^^ help: consider using: `dotTokenId`
851
- |
852
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
853
-
854
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
855
- --> test/Agent.t.sol:4:8
856
- |
857
- 4 | import "forge-std/Test.sol";
858
- | ^^^^^^^^^^^^^^^^^^^^
859
- |
860
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
861
-
862
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
863
- --> test/Agent.t.sol:5:8
864
- |
865
- 5 | import "forge-std/console.sol";
866
- | ^^^^^^^^^^^^^^^^^^^^^^^
867
- |
868
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
869
-
870
- note[mixed-case-function]: function names should use mixedCase
871
- --> test/BeefyClient.t.sol:163:14
872
- |
873
- 163 | function decodeMMRLeaf(string memory beefyCommitmentRaw) internal {
874
- | ^^^^^^^^^^^^^ help: consider using: `decodeMmrLeaf`
875
- |
876
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
877
-
878
- note[mixed-case-variable]: mutable variables should use mixedCase
879
- --> test/GatewayV1.t.sol:177:17
880
- |
881
- 177 | bytes32 agentID,
882
- | ^^^^^^^ help: consider using: `agentId`
883
- |
884
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
885
-
886
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
887
- --> test/Verification.t.sol:4:8
888
- |
889
- 4 | import "openzeppelin/utils/Strings.sol";
890
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
891
- |
892
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
893
-
894
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
895
- --> test/Verification.t.sol:5:8
896
- |
897
- 5 | import "forge-std/Test.sol";
898
- | ^^^^^^^^^^^^^^^^^^^^
899
- |
900
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
901
-
902
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
903
- --> test/Verification.t.sol:6:8
904
- |
905
- 6 | import "forge-std/console.sol";
906
- | ^^^^^^^^^^^^^^^^^^^^^^^
907
- |
908
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
909
-
910
- note[mixed-case-variable]: mutable variables should use mixedCase
911
- --> test/Verification.t.sol:18:19
912
- |
913
- 18 | bytes4 public encodedParachainID;
914
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `encodedParachainId`
915
- |
916
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
917
-
918
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
919
- --> test/Token.t.sol:63:9
920
- |
921
- 63 | token.transfer(receiver, transferAmount);
922
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
923
- |
924
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
925
-
926
- note[mixed-case-variable]: mutable variables should use mixedCase
927
- --> test/BeefyClient.t.sol:167:16
928
- |
929
- 167 | uint64 nextAuthoritySetID =
930
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `nextAuthoritySetId`
931
- |
932
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
933
-
934
- note[screaming-snake-case-immutable]: immutables should use SCREAMING_SNAKE_CASE
935
- --> test/mocks/FeeOnTransferToken.sol:8:30
936
- |
937
- 8 | uint256 public immutable feeBps;
938
- | ^^^^^^ help: consider using: `FEE_BPS`
939
- |
940
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-immutable
941
-
942
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
943
- --> test/FiatShamirPaddingBitsGrind.t.sol:2:8
944
- |
945
- 2 | import "forge-std/Test.sol";
946
- | ^^^^^^^^^^^^^^^^^^^^
947
- |
948
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
949
-
950
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
951
- --> test/Token.t.sol:102:9
952
- |
953
- 102 | token.transferFrom(owner, receiver, transferAmount);
954
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
955
- |
956
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
957
-
958
- note[mixed-case-function]: function names should use mixedCase
959
- --> src/SubstrateTypes.sol:121:14
960
- |
961
- 121 | function SendForeignTokenToAssetHubAddress32(
962
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendForeignTokenToAssetHubAddress32`
963
- |
964
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
965
-
966
- note[mixed-case-variable]: mutable variables should use mixedCase
967
- --> src/SubstrateTypes.sol:122:17
968
- |
969
- 122 | bytes32 tokenID,
970
- | ^^^^^^^ help: consider using: `tokenId`
971
- |
972
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
973
-
974
- note[mixed-case-function]: function names should use mixedCase
975
- --> src/SubstrateTypes.sol:140:14
976
- |
977
- 140 | function SendForeignTokenToAddress32(
978
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendForeignTokenToAddress32`
979
- |
980
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
981
-
982
- note[mixed-case-variable]: mutable variables should use mixedCase
983
- --> src/SubstrateTypes.sol:141:17
984
- |
985
- 141 | bytes32 tokenID,
986
- | ^^^^^^^ help: consider using: `tokenId`
987
- |
988
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
989
-
990
- note[mixed-case-variable]: mutable variables should use mixedCase
991
- --> src/SubstrateTypes.sol:142:16
992
- |
993
- 142 | ParaID paraID,
994
- | ^^^^^^ help: consider using: `paraId`
995
- |
996
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
997
-
998
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
999
- --> test/Token.t.sol:140:9
1000
- |
1001
- 140 | token.transferFrom(owner, receiver, transferAmount);
1002
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1003
- |
1004
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1005
-
1006
- note[mixed-case-variable]: mutable variables should use mixedCase
1007
- --> test/GatewayV1.t.sol:190:17
1008
- |
1009
- 190 | bytes32 agentID,
1010
- | ^^^^^^^ help: consider using: `agentId`
1011
- |
1012
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1013
-
1014
- note[mixed-case-function]: function names should use mixedCase
1015
- --> test/mocks/MockGateway.sol:64:14
1016
- |
1017
- 64 | function prank_registerNativeToken(address token) external {
1018
- | ^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `prankRegisterNativeToken`
1019
- |
1020
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1021
-
1022
- note[mixed-case-variable]: mutable variables should use mixedCase
1023
- --> test/FiatShamirPaddingBitsGrind.t.sol:48:17
1024
- |
1025
- 48 | bytes32 maliciousMMRRoot = keccak256("attacker-chosen-mmr-root");
1026
- | ^^^^^^^^^^^^^^^^ help: consider using: `maliciousMmrRoot`
1027
- |
1028
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1029
-
1030
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1031
- --> test/Token.t.sol:165:9
1032
- |
1033
- 165 | token.transferFrom(owner, address(0), amount);
1034
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1035
- |
1036
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1037
-
1038
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1039
- --> test/Token.t.sol:196:9
1040
- |
1041
- 196 | token.transferFrom(owner, receiver, transferAmount);
1042
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1043
- |
1044
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1045
-
1046
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1047
- --> test/Token.t.sol:205:9
1048
- |
1049
- 205 | token.transferFrom(owner, receiver, allowanceAmount);
1050
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1051
- |
1052
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1053
-
1054
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1055
- --> test/Token.t.sol:230:9
1056
- |
1057
- 230 | token.transferFrom(owner, receiver, allowanceAmount);
1058
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1059
- |
1060
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1061
-
1062
- note[mixed-case-variable]: mutable variables should use mixedCase
1063
- --> test/mocks/VerificationWrapper.sol:8:16
1064
- |
1065
- 8 | bytes4 encodedParachainID,
1066
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `encodedParachainId`
1067
- |
1068
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1069
-
1070
- note[mixed-case-function]: function names should use mixedCase
1071
- --> test/mocks/MockGateway.sol:68:14
1072
- |
1073
- 68 | function prank_registerForeignToken(
1074
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `prankRegisterForeignToken`
1075
- |
1076
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1077
-
1078
- note[mixed-case-variable]: mutable variables should use mixedCase
1079
- --> test/mocks/MockGateway.sol:69:17
1080
- |
1081
- 69 | bytes32 foreignTokenID,
1082
- | ^^^^^^^^^^^^^^ help: consider using: `foreignTokenId`
1083
- |
1084
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1085
-
1086
- note[unused-import]: unused imports should be removed
1087
- --> test/BeefyClient.t.sol:11:9
1088
- |
1089
- 11 | import {ScaleCodec} from "../src/utils/ScaleCodec.sol";
1090
- | ^^^^^^^^^^
1091
- |
1092
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1093
-
1094
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
1095
- --> test/GatewayV2.t.sol:118:8
1096
- |
1097
- 118 | import "./mocks/FeeOnTransferToken.sol";
1098
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1099
- |
1100
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
1101
-
1102
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
1103
- --> test/FiatShamirPaddingBitsGrind.t.sol:30:54
1104
- |
1105
- 30 | BeefyClient.ValidatorSet({id: 1, length: uint128(N), root: vsetRoot});
1106
- | ^^^^^^^^^^
1107
- |
1108
- = note: consider disabling this lint if you're certain the cast is safe
1109
-
1110
- // casting to 'uint128' is safe because [explain why]
1111
- // forge-lint: disable-next-line(unsafe-typecast)
1112
-
1113
-
1114
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
1115
-
1116
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
1117
- --> test/FiatShamirPaddingBitsGrind.t.sol:32:54
1118
- |
1119
- 32 | BeefyClient.ValidatorSet({id: 2, length: uint128(N), root: vsetRoot});
1120
- | ^^^^^^^^^^
1121
- |
1122
- = note: consider disabling this lint if you're certain the cast is safe
1123
-
1124
- // casting to 'uint128' is safe because [explain why]
1125
- // forge-lint: disable-next-line(unsafe-typecast)
1126
-
1127
-
1128
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
1129
-
1130
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
1131
- --> test/Uint16Array.t.sol:4:8
1132
- |
1133
- 4 | import "forge-std/Test.sol";
1134
- | ^^^^^^^^^^^^^^^^^^^^
1135
- |
1136
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
1137
-
1138
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
1139
- --> test/Uint16Array.t.sol:5:8
1140
- |
1141
- 5 | import "forge-std/console.sol";
1142
- | ^^^^^^^^^^^^^^^^^^^^^^^
1143
- |
1144
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
1145
-
1146
- note[mixed-case-function]: function names should use mixedCase
1147
- --> src/SubstrateTypes.sol:163:14
1148
- |
1149
- 163 | function SendForeignTokenToAddress20(
1150
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendForeignTokenToAddress20`
1151
- |
1152
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1153
-
1154
- note[mixed-case-variable]: mutable variables should use mixedCase
1155
- --> src/SubstrateTypes.sol:164:17
1156
- |
1157
- 164 | bytes32 tokenID,
1158
- | ^^^^^^^ help: consider using: `tokenId`
1159
- |
1160
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1161
-
1162
- note[mixed-case-variable]: mutable variables should use mixedCase
1163
- --> src/SubstrateTypes.sol:165:16
1164
- |
1165
- 165 | ParaID paraID,
1166
- | ^^^^^^ help: consider using: `paraId`
1167
- |
1168
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1169
-
1170
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1171
- --> test/Token.t.sol:503:9
1172
- |
1173
- 503 | token.transferFrom(owner, pool, swapAmount);
1174
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1175
- |
1176
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1177
-
1178
- note[mixed-case-function]: function names should use mixedCase
1179
- --> test/mocks/MockGateway.sol:113:14
1180
- |
1181
- 113 | function exposed_v1_transactionBaseGas() external pure returns (uint256) {
1182
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `exposedV1TransactionBaseGas`
1183
- |
1184
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1185
-
1186
- note[mixed-case-function]: function names should use mixedCase
1187
- --> test/mocks/MockGateway.sol:118:14
1188
- |
1189
- 118 | function exposed_dispatchCommand(CommandV2 calldata cmd, bytes32 origin)
1190
- | ^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `exposedDispatchCommand`
1191
- |
1192
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1193
-
1194
- note[mixed-case-variable]: mutable variables should use mixedCase
1195
- --> test/Uint16Array.t.sol:98:16
1196
- |
1197
- 98 | uint16 new_value = counters.get(index);
1198
- | ^^^^^^^^^ help: consider using: `newValue`
1199
- |
1200
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1201
-
1202
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
1203
- --> test/BeefyClient.t.sol:42:24
1204
- |
1205
- 42 | bytes2 mmrRootID = bytes2("mh");
1206
- | ^^^^^^^^^^^^
1207
- |
1208
- = note: consider disabling this lint if you're certain the cast is safe
1209
-
1210
- // casting to 'bytes2' is safe because [explain why]
1211
- // forge-lint: disable-next-line(unsafe-typecast)
1212
-
1213
-
1214
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
1215
-
1216
- note[mixed-case-variable]: mutable variables should use mixedCase
1217
- --> test/GatewayV1.t.sol:614:16
1218
- |
1219
- 614 | ParaID paraID = ParaID.wrap(1000);
1220
- | ^^^^^^ help: consider using: `paraId`
1221
- |
1222
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1223
-
1224
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
1225
- --> test/utils/ForkTestFixtures.sol:33:24
1226
- |
1227
- 33 | Vm public constant vm = Vm(0x7109709ECfa91a80626fF3989D68f67F5b1DD12D);
1228
- | ^^ help: consider using: `VM`
1229
- |
1230
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
1231
-
1232
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1233
- --> test/Token.t.sol:528:9
1234
- |
1235
- 528 | token.transferFrom(owner, pool, firstSwapAmount);
1236
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1237
- |
1238
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1239
-
1240
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1241
- --> test/Token.t.sol:533:9
1242
- |
1243
- 533 | token.transferFrom(owner, pool, secondSwapAmount);
1244
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1245
- |
1246
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1247
-
1248
- note[mixed-case-variable]: mutable variables should use mixedCase
1249
- --> src/BeefyClient.sol:73:16
1250
- |
1251
- 73 | uint64 validatorSetID;
1252
- | ^^^^^^^^^^^^^^ help: consider using: `validatorSetId`
1253
- |
1254
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1255
-
1256
- note[mixed-case-variable]: mutable variables should use mixedCase
1257
- --> src/BeefyClient.sol:85:16
1258
- |
1259
- 85 | bytes2 payloadID;
1260
- | ^^^^^^^^^ help: consider using: `payloadId`
1261
- |
1262
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1263
-
1264
- note[mixed-case-variable]: mutable variables should use mixedCase
1265
- --> test/GatewayV2.t.sol:142:20
1266
- |
1267
- 142 | bytes32 public dotTokenID;
1268
- | ^^^^^^^^^^ help: consider using: `dotTokenId`
1269
- |
1270
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1271
-
1272
- note[mixed-case-variable]: mutable variables should use mixedCase
1273
- --> test/GatewayV1.t.sol:639:16
1274
- |
1275
- 639 | ParaID paraID = ParaID.wrap(1000);
1276
- | ^^^^^^ help: consider using: `paraId`
1277
- |
1278
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1279
-
1280
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
1281
- --> test/ForkUpgrade202509.t.sol:34:31
1282
- |
1283
- 34 | address constant internal assetHubAgent = 0xd803472c47a87D7B63E888DE53f03B4191B846a8;
1284
- | ^^^^^^^^^^^^^ help: consider using: `ASSET_HUB_AGENT`
1285
- |
1286
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
1287
-
1288
- note[unused-import]: unused imports should be removed
1289
- --> test/mocks/MockGateway.sol:7:20
1290
- |
1291
- 7 | import {ChannelID, ParaID, OperatingMode} from "../../src/Types.sol";
1292
- | ^^^^^^
1293
- |
1294
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1295
-
1296
- note[unused-import]: unused imports should be removed
1297
- --> test/mocks/MockGateway.sol:7:28
1298
- |
1299
- 7 | import {ChannelID, ParaID, OperatingMode} from "../../src/Types.sol";
1300
- | ^^^^^^^^^^^^^
1301
- |
1302
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1303
-
1304
- note[unused-import]: unused imports should be removed
1305
- --> test/mocks/MockGateway.sol:11:9
1306
- |
1307
- 11 | import {IInitializable} from "../../src/interfaces/IInitializable.sol";
1308
- | ^^^^^^^^^^^^^^
1309
- |
1310
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1311
-
1312
- note[unused-import]: unused imports should be removed
1313
- --> test/mocks/MockGateway.sol:13:9
1314
- |
1315
- 13 | import {UD60x18} from "prb/math/src/UD60x18.sol";
1316
- | ^^^^^^^
1317
- |
1318
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1319
-
1320
- note[unused-import]: unused imports should be removed
1321
- --> test/GatewayV2.t.sol:5:9
1322
- |
1323
- 5 | import {Strings} from "openzeppelin/utils/Strings.sol";
1324
- | ^^^^^^^
1325
- |
1326
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1327
-
1328
- note[unused-import]: unused imports should be removed
1329
- --> test/GatewayV2.t.sol:6:9
1330
- |
1331
- 6 | import {console} from "forge-std/console.sol";
1332
- | ^^^^^^^
1333
- |
1334
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1335
-
1336
- note[unused-import]: unused imports should be removed
1337
- --> test/GatewayV2.t.sol:8:9
1338
- |
1339
- 8 | import {BeefyClient} from "../src/BeefyClient.sol";
1340
- | ^^^^^^^^^^^
1341
- |
1342
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1343
-
1344
- note[unused-import]: unused imports should be removed
1345
- --> test/GatewayV2.t.sol:12:9
1346
- |
1347
- 12 | import {IInitializable} from "../src/interfaces/IInitializable.sol";
1348
- | ^^^^^^^^^^^^^^
1349
- |
1350
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1351
-
1352
- note[unused-import]: unused imports should be removed
1353
- --> test/GatewayV2.t.sol:13:9
1354
- |
1355
- 13 | import {IUpgradable} from "../src/interfaces/IUpgradable.sol";
1356
- | ^^^^^^^^^^^
1357
- |
1358
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1359
-
1360
- note[unused-import]: unused imports should be removed
1361
- --> test/GatewayV2.t.sol:14:9
1362
- |
1363
- 14 | import {Gateway} from "../src/Gateway.sol";
1364
- | ^^^^^^^
1365
- |
1366
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1367
-
1368
- note[unused-import]: unused imports should be removed
1369
- --> test/GatewayV2.t.sol:16:9
1370
- |
1371
- 16 | import {MockGatewayV2} from "./mocks/MockGatewayV2.sol";
1372
- | ^^^^^^^^^^^^^
1373
- |
1374
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1375
-
1376
- note[unused-import]: unused imports should be removed
1377
- --> test/GatewayV2.t.sol:21:9
1378
- |
1379
- 21 | import {Agent} from "../src/Agent.sol";
1380
- | ^^^^^
1381
- |
1382
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1383
-
1384
- note[unused-import]: unused imports should be removed
1385
- --> test/GatewayV2.t.sol:23:9
1386
- |
1387
- 23 | import {SubstrateTypes} from "./../src/SubstrateTypes.sol";
1388
- | ^^^^^^^^^^^^^^
1389
- |
1390
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1391
-
1392
- note[unused-import]: unused imports should be removed
1393
- --> test/GatewayV2.t.sol:24:24
1394
- |
1395
- 24 | import {OperatingMode, ParaID, CommandV2, CommandKind, InboundMessageV2} from "../src/Types.sol";
1396
- | ^^^^^^
1397
- |
1398
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1399
-
1400
- note[unused-import]: unused imports should be removed
1401
- --> test/GatewayV2.t.sol:26:9
1402
- |
1403
- 26 | import {NativeTransferFailed} from "../src/utils/SafeTransfer.sol";
1404
- | ^^^^^^^^^^^^^^^^^^^^
1405
- |
1406
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1407
-
1408
- note[unused-import]: unused imports should be removed
1409
- --> test/GatewayV2.t.sol:27:9
1410
- |
1411
- 27 | import {PricingStorage} from "../src/storage/PricingStorage.sol";
1412
- | ^^^^^^^^^^^^^^
1413
- |
1414
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1415
-
1416
- note[unused-import]: unused imports should be removed
1417
- --> test/GatewayV2.t.sol:29:9
1418
- |
1419
- 29 | import {TokenLib} from "../src/TokenLib.sol";
1420
- | ^^^^^^^^
1421
- |
1422
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1423
-
1424
- note[unused-import]: unused imports should be removed
1425
- --> test/GatewayV2.t.sol:46:5
1426
- |
1427
- 46 | Xcm,
1428
- | ^^^
1429
- |
1430
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1431
-
1432
- note[unused-import]: unused imports should be removed
1433
- --> test/GatewayV2.t.sol:51:5
1434
- |
1435
- 51 | AgentExecuteCommand,
1436
- | ^^^^^^^^^^^^^^^^^^^
1437
- |
1438
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1439
-
1440
- note[unused-import]: unused imports should be removed
1441
- --> test/GatewayV2.t.sol:52:5
1442
- |
1443
- 52 | InboundMessage,
1444
- | ^^^^^^^^^^^^^^
1445
- |
1446
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1447
-
1448
- note[unused-import]: unused imports should be removed
1449
- --> test/GatewayV2.t.sol:54:5
1450
- |
1451
- 54 | ParaID,
1452
- | ^^^^^^
1453
- |
1454
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1455
-
1456
- note[unused-import]: unused imports should be removed
1457
- --> test/GatewayV2.t.sol:55:5
1458
- |
1459
- 55 | Command
1460
- | ^^^^^^^
1461
- |
1462
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1463
-
1464
- note[unused-import]: unused imports should be removed
1465
- --> test/GatewayV2.t.sol:59:9
1466
- |
1467
- 59 | import {UD60x18, ud60x18, convert} from "prb/math/src/UD60x18.sol";
1468
- | ^^^^^^^
1469
- |
1470
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1471
-
1472
- note[unused-import]: unused imports should be removed
1473
- --> test/GatewayV2.t.sol:59:27
1474
- |
1475
- 59 | import {UD60x18, ud60x18, convert} from "prb/math/src/UD60x18.sol";
1476
- | ^^^^^^^
1477
- |
1478
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1479
-
1480
- note[unused-import]: unused imports should be removed
1481
- --> test/mocks/MockGateway.sol:17:9
1482
- |
1483
- 17 | import {AgentExecutor} from "../../src/AgentExecutor.sol";
1484
- | ^^^^^^^^^^^^^
1485
- |
1486
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1487
-
1488
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1489
- --> test/Token.t.sol:588:9
1490
- |
1491
- 588 | token.transferFrom(userAddress, pool, permitAmount);
1492
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1493
- |
1494
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1495
-
1496
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1497
- --> test/Token.t.sol:619:13
1498
- |
1499
- 619 | token.transferFrom(owner, recipients[i], transferAmount);
1500
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1501
- |
1502
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1503
-
1504
- note[pascal-case-struct]: structs should use PascalCase
1505
- --> src/BeefyClient.sol:125:12
1506
- |
1507
- 125 | struct MMRLeaf {
1508
- | ^^^^^^^ help: consider using: `MmrLeaf`
1509
- |
1510
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#pascal-case-struct
1511
-
1512
- note[mixed-case-variable]: mutable variables should use mixedCase
1513
- --> src/BeefyClient.sol:133:16
1514
- |
1515
- 133 | uint64 nextAuthoritySetID;
1516
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `nextAuthoritySetId`
1517
- |
1518
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1519
-
1520
- note[unsafe-cheatcode]: usage of unsafe cheatcodes that can perform dangerous operations
1521
- --> test/utils/ForkTestFixtures.sol:73:33
1522
- |
1523
- 73 | string memory data = vm.readFile(string.concat(vm.projectRoot(), fixturePath));
1524
- | ^^^^^^^^
1525
- |
1526
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-cheatcode
1527
-
1528
- note[named-struct-fields]: prefer initializing structs with named fields
1529
- --> test/GatewayV2.t.sol:258:13
1530
- |
1531
- 258 | RegisterForeignTokenParams(id, name, symbol, decimals);
1532
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using named fields: `RegisterForeignTokenParams({ foreignTokenID: id, name: name, symbol: symbol, decimals: decimals })`
1533
- |
1534
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
1535
-
1536
- note[mixed-case-variable]: mutable variables should use mixedCase
1537
- --> test/GatewayV1.t.sol:665:16
1538
- |
1539
- 665 | ParaID paraID = ParaID.wrap(1000);
1540
- | ^^^^^^ help: consider using: `paraId`
1541
- |
1542
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1543
-
1544
- note[mixed-case-variable]: mutable variables should use mixedCase
1545
- --> src/BeefyClient.sol:171:20
1546
- |
1547
- 171 | bytes32 public latestMMRRoot;
1548
- | ^^^^^^^^^^^^^ help: consider using: `latestMmrRoot`
1549
- |
1550
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1551
-
1552
- note[mixed-case-variable]: mutable variables should use mixedCase
1553
- --> test/ForkUpgrade202509.t.sol:152:16
1554
- |
1555
- 152 | ParaID paraID = ParaID.wrap(1000);
1556
- | ^^^^^^ help: consider using: `paraId`
1557
- |
1558
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1559
-
1560
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
1561
- --> test/BeefyClient.t.sol:550:47
1562
- |
1563
- 550 | _payload[0] = BeefyClient.PayloadItem(bytes2("ab"), hex"000102");
1564
- | ^^^^^^^^^^^^
1565
- |
1566
- = note: consider disabling this lint if you're certain the cast is safe
1567
-
1568
- // casting to 'bytes2' is safe because [explain why]
1569
- // forge-lint: disable-next-line(unsafe-typecast)
1570
-
1571
-
1572
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
1573
-
1574
- note[named-struct-fields]: prefer initializing structs with named fields
1575
- --> test/GatewayV2.t.sol:272:48
1576
- |
1577
- 272 | MintForeignTokenParams memory params = MintForeignTokenParams(id, recipient, amount);
1578
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using named fields: `MintForeignTokenParams({ foreignTokenID: id, recipient: recipient, amount: amount })`
1579
- |
1580
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
1581
-
1582
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1583
- --> test/Token.t.sol:649:9
1584
- |
1585
- 649 | token.transferFrom(owner, pool, firstAmount);
1586
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1587
- |
1588
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1589
-
1590
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1591
- --> test/Token.t.sol:658:9
1592
- |
1593
- 658 | token.transferFrom(owner, pool, secondAmount);
1594
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1595
- |
1596
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1597
-
1598
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1599
- --> test/Token.t.sol:705:9
1600
- |
1601
- 705 | token.transferFrom(user, pool, amount1);
1602
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1603
- |
1604
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1605
-
1606
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
1607
- --> test/Token.t.sol:706:9
1608
- |
1609
- 706 | token.transferFrom(user, pool2, amount2);
1610
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1611
- |
1612
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
1613
-
1614
- note[mixed-case-variable]: mutable variables should use mixedCase
1615
- --> test/utils/ForkTestFixtures.sol:132:15
1616
- |
1617
- 132 | bytes consensusEngineID;
1618
- | ^^^^^^^^^^^^^^^^^ help: consider using: `consensusEngineId`
1619
- |
1620
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1621
-
1622
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
1623
- --> test/GatewayV2.t.sol:979:75
1624
- |
1625
- 979 | token: address(0), recipient: address(recipient), amount: uint128(amt)
1626
- | ^^^^^^^^^^^^
1627
- |
1628
- = note: consider disabling this lint if you're certain the cast is safe
1629
-
1630
- // casting to 'uint128' is safe because [explain why]
1631
- // forge-lint: disable-next-line(unsafe-typecast)
1632
-
1633
-
1634
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
1635
-
1636
- note[mixed-case-function]: function names should use mixedCase
1637
- --> test/SubstrateTypes.t.sol:10:14
1638
- |
1639
- 10 | function H160(address account) external pure returns (bytes memory) {
1640
- | ^^^^ help: consider using: `h160`
1641
- |
1642
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1643
-
1644
- note[mixed-case-function]: function names should use mixedCase
1645
- --> test/SubstrateTypes.t.sol:14:14
1646
- |
1647
- 14 | function VecU8(bytes calldata input) external pure returns (bytes memory) {
1648
- | ^^^^^ help: consider using: `vecU8`
1649
- |
1650
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1651
-
1652
- note[mixed-case-function]: function names should use mixedCase
1653
- --> test/SubstrateTypes.t.sol:18:14
1654
- |
1655
- 18 | function None() external pure returns (bytes memory) {
1656
- | ^^^^ help: consider using: `none`
1657
- |
1658
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1659
-
1660
- note[mixed-case-function]: function names should use mixedCase
1661
- --> test/SubstrateTypes.t.sol:22:14
1662
- |
1663
- 22 | function RegisterToken(address token, uint128 fee) external view returns (bytes memory) {
1664
- | ^^^^^^^^^^^^^ help: consider using: `registerToken`
1665
- |
1666
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1667
-
1668
- note[mixed-case-function]: function names should use mixedCase
1669
- --> test/SubstrateTypes.t.sol:26:14
1670
- |
1671
- 26 | function SendTokenToAssetHubAddress32(
1672
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendTokenToAssetHubAddress32`
1673
- |
1674
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1675
-
1676
- note[mixed-case-function]: function names should use mixedCase
1677
- --> test/SubstrateTypes.t.sol:35:14
1678
- |
1679
- 35 | function SendTokenToAddress32(
1680
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendTokenToAddress32`
1681
- |
1682
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1683
-
1684
- note[mixed-case-function]: function names should use mixedCase
1685
- --> test/SubstrateTypes.t.sol:48:14
1686
- |
1687
- 48 | function SendTokenToAddress20(
1688
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendTokenToAddress20`
1689
- |
1690
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1691
-
1692
- note[mixed-case-variable]: mutable variables should use mixedCase
1693
- --> test/GatewayV1.t.sol:682:16
1694
- |
1695
- 682 | ParaID paraID = ParaID.wrap(1000);
1696
- | ^^^^^^ help: consider using: `paraId`
1697
- |
1698
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1699
-
1700
- note[mixed-case-variable]: mutable variables should use mixedCase
1701
- --> scripts/l2-integration/across/test/TestSnowbridgeL1Adaptor.s.sol:37:17
1702
- |
1703
- 37 | address USDC_ADDRESS;
1704
- | ^^^^^^^^^^^^ help: consider using: `usdcAddress`
1705
- |
1706
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1707
-
1708
- note[unused-import]: unused imports should be removed
1709
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorWeth.s.sol:4:17
1710
- |
1711
- 4 | import {Script, console} from "forge-std/Script.sol";
1712
- | ^^^^^^^
1713
- |
1714
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1715
-
1716
- note[unused-import]: unused imports should be removed
1717
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorWeth.s.sol:9:9
1718
- |
1719
- 9 | import {ISpokePool} from "../../../../src/l2-integration/interfaces/ISpokePool.sol";
1720
- | ^^^^^^^^^^
1721
- |
1722
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1723
-
1724
- note[unused-import]: unused imports should be removed
1725
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorWeth.s.sol:10:36
1726
- |
1727
- 10 | import {DepositParams, SendParams, SwapParams} from "../../../../src/l2-integration/Types.sol";
1728
- | ^^^^^^^^^^
1729
- |
1730
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1731
-
1732
- note[unused-import]: unused imports should be removed
1733
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorWeth.s.sol:13:5
1734
- |
1735
- 13 | BASE_CHAIN_ID as SEPOLIA_BASE_CHAIN_ID,
1736
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1737
- |
1738
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1739
-
1740
- note[unused-import]: unused imports should be removed
1741
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorWeth.s.sol:20:5
1742
- |
1743
- 20 | BASE_CHAIN_ID as MAINNET_BASE_CHAIN_ID,
1744
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
1745
- |
1746
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1747
-
1748
- note[mixed-case-function]: function names should use mixedCase
1749
- --> test/SubstrateTypes.t.sol:61:14
1750
- |
1751
- 61 | function SendForeignTokenToAssetHubAddress32(
1752
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendForeignTokenToAssetHubAddress32`
1753
- |
1754
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1755
-
1756
- note[mixed-case-variable]: mutable variables should use mixedCase
1757
- --> test/SubstrateTypes.t.sol:62:17
1758
- |
1759
- 62 | bytes32 tokenID,
1760
- | ^^^^^^^ help: consider using: `tokenId`
1761
- |
1762
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1763
-
1764
- note[mixed-case-function]: function names should use mixedCase
1765
- --> test/SubstrateTypes.t.sol:72:14
1766
- |
1767
- 72 | function SendForeignTokenToAddress32(
1768
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendForeignTokenToAddress32`
1769
- |
1770
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1771
-
1772
- note[mixed-case-variable]: mutable variables should use mixedCase
1773
- --> test/SubstrateTypes.t.sol:73:17
1774
- |
1775
- 73 | bytes32 tokenID,
1776
- | ^^^^^^^ help: consider using: `tokenId`
1777
- |
1778
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1779
-
1780
- note[screaming-snake-case-immutable]: immutables should use SCREAMING_SNAKE_CASE
1781
- --> src/BeefyClient.sol:203:30
1782
- |
1783
- 203 | uint256 public immutable randaoCommitDelay;
1784
- | ^^^^^^^^^^^^^^^^^ help: consider using: `RANDAO_COMMIT_DELAY`
1785
- |
1786
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-immutable
1787
-
1788
- note[screaming-snake-case-immutable]: immutables should use SCREAMING_SNAKE_CASE
1789
- --> src/BeefyClient.sol:211:30
1790
- |
1791
- 211 | uint256 public immutable randaoCommitExpiration;
1792
- | ^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `RANDAO_COMMIT_EXPIRATION`
1793
- |
1794
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-immutable
1795
-
1796
- note[screaming-snake-case-immutable]: immutables should use SCREAMING_SNAKE_CASE
1797
- --> src/BeefyClient.sol:217:30
1798
- |
1799
- 217 | uint256 public immutable minNumRequiredSignatures;
1800
- | ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `MIN_NUM_REQUIRED_SIGNATURES`
1801
- |
1802
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-immutable
1803
-
1804
- note[screaming-snake-case-immutable]: immutables should use SCREAMING_SNAKE_CASE
1805
- --> src/BeefyClient.sol:224:30
1806
- |
1807
- 224 | uint256 public immutable fiatShamirRequiredSignatures;
1808
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `FIAT_SHAMIR_REQUIRED_SIGNATURES`
1809
- |
1810
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-immutable
1811
-
1812
- note[unused-import]: unused imports should be removed
1813
- --> test/utils/ForkTestFixtures.sol:5:9
1814
- |
1815
- 5 | import {Test} from "forge-std/Test.sol";
1816
- | ^^^^
1817
- |
1818
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1819
-
1820
- note[unused-import]: unused imports should be removed
1821
- --> test/utils/ForkTestFixtures.sol:7:9
1822
- |
1823
- 7 | import {console} from "forge-std/console.sol";
1824
- | ^^^^^^^
1825
- |
1826
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1827
-
1828
- note[unused-import]: unused imports should be removed
1829
- --> test/utils/ForkTestFixtures.sol:9:9
1830
- |
1831
- 9 | import {IUpgradable} from "../../src/interfaces/IUpgradable.sol";
1832
- | ^^^^^^^^^^^
1833
- |
1834
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1835
-
1836
- note[unused-import]: unused imports should be removed
1837
- --> test/utils/ForkTestFixtures.sol:12:5
1838
- |
1839
- 12 | UpgradeParams,
1840
- | ^^^^^^^^^^^^^
1841
- |
1842
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1843
-
1844
- note[unused-import]: unused imports should be removed
1845
- --> test/utils/ForkTestFixtures.sol:13:5
1846
- |
1847
- 13 | SetOperatingModeParams,
1848
- | ^^^^^^^^^^^^^^^^^^^^^^
1849
- |
1850
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1851
-
1852
- note[unused-import]: unused imports should be removed
1853
- --> test/utils/ForkTestFixtures.sol:14:5
1854
- |
1855
- 14 | OperatingMode,
1856
- | ^^^^^^^^^^^^^
1857
- |
1858
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1859
-
1860
- note[unused-import]: unused imports should be removed
1861
- --> test/utils/ForkTestFixtures.sol:15:5
1862
- |
1863
- 15 | RegisterForeignTokenParams,
1864
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^
1865
- |
1866
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1867
-
1868
- note[unused-import]: unused imports should be removed
1869
- --> test/utils/ForkTestFixtures.sol:17:5
1870
- |
1871
- 17 | ParaID,
1872
- | ^^^^^^
1873
- |
1874
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1875
-
1876
- note[unused-import]: unused imports should be removed
1877
- --> test/utils/ForkTestFixtures.sol:18:5
1878
- |
1879
- 18 | OperatingMode,
1880
- | ^^^^^^^^^^^^^
1881
- |
1882
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1883
-
1884
- note[unused-import]: unused imports should be removed
1885
- --> test/utils/ForkTestFixtures.sol:21:5
1886
- |
1887
- 21 | TokenInfo
1888
- | ^^^^^^^^^
1889
- |
1890
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
1891
-
1892
- note[mixed-case-function]: function names should use mixedCase
1893
- --> test/SubstrateTypes.t.sol:85:14
1894
- |
1895
- 85 | function SendForeignTokenToAddress20(
1896
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `sendForeignTokenToAddress20`
1897
- |
1898
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
1899
-
1900
- note[mixed-case-variable]: mutable variables should use mixedCase
1901
- --> test/SubstrateTypes.t.sol:86:17
1902
- |
1903
- 86 | bytes32 tokenID,
1904
- | ^^^^^^^ help: consider using: `tokenId`
1905
- |
1906
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1907
-
1908
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
1909
- --> test/GatewayV2.t.sol:1003:74
1910
- |
1911
- 1003 | token: address(token), recipient: address(this), amount: uint128(tAmt)
1912
- | ^^^^^^^^^^^^^
1913
- |
1914
- = note: consider disabling this lint if you're certain the cast is safe
1915
-
1916
- // casting to 'uint128' is safe because [explain why]
1917
- // forge-lint: disable-next-line(unsafe-typecast)
1918
-
1919
-
1920
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
1921
-
1922
- note[mixed-case-variable]: mutable variables should use mixedCase
1923
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL1Adaptor.s.sol:24:17
1924
- |
1925
- 24 | address SPOKE_POOL_ADDRESS;
1926
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `spokePoolAddress`
1927
- |
1928
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1929
-
1930
- note[mixed-case-variable]: mutable variables should use mixedCase
1931
- --> scripts/l2-integration/across/test/TestSnowbridgeL1Adaptor.s.sol:38:17
1932
- |
1933
- 38 | address BASE_USDC_ADDRESS;
1934
- | ^^^^^^^^^^^^^^^^^ help: consider using: `baseUsdcAddress`
1935
- |
1936
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1937
-
1938
- note[mixed-case-variable]: mutable variables should use mixedCase
1939
- --> scripts/l2-integration/across/test/TestSnowbridgeL1Adaptor.s.sol:39:17
1940
- |
1941
- 39 | uint256 BASE_CHAIN_ID;
1942
- | ^^^^^^^^^^^^^ help: consider using: `baseChainId`
1943
- |
1944
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1945
-
1946
- note[mixed-case-variable]: mutable variables should use mixedCase
1947
- --> scripts/l2-integration/across/test/TestSnowbridgeL1Adaptor.s.sol:40:16
1948
- |
1949
- 40 | uint32 TIME_BUFFER;
1950
- | ^^^^^^^^^^^ help: consider using: `timeBuffer`
1951
- |
1952
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1953
-
1954
- note[mixed-case-variable]: mutable variables should use mixedCase
1955
- --> test/ForkUpgrade202509.t.sol:281:16
1956
- |
1957
- 281 | ParaID paraID = ParaID.wrap(1000);
1958
- | ^^^^^^ help: consider using: `paraId`
1959
- |
1960
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1961
-
1962
- note[mixed-case-variable]: mutable variables should use mixedCase
1963
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL1Adaptor.s.sol:25:17
1964
- |
1965
- 25 | address BASE_MULTI_CALL_HANDLER_ADDRESS;
1966
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `baseMultiCallHandlerAddress`
1967
- |
1968
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1969
-
1970
- note[mixed-case-variable]: mutable variables should use mixedCase
1971
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL1Adaptor.s.sol:26:17
1972
- |
1973
- 26 | address WETH9_ADDRESS;
1974
- | ^^^^^^^^^^^^^ help: consider using: `weth9Address`
1975
- |
1976
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1977
-
1978
- note[mixed-case-variable]: mutable variables should use mixedCase
1979
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL1Adaptor.s.sol:27:17
1980
- |
1981
- 27 | address BASE_WETH9_ADDRESS;
1982
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `baseWeth9Address`
1983
- |
1984
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1985
-
1986
- note[mixed-case-variable]: mutable variables should use mixedCase
1987
- --> test/ForkUpgrade202509.t.sol:311:16
1988
- |
1989
- 311 | ParaID paraID = ParaID.wrap(1000);
1990
- | ^^^^^^ help: consider using: `paraId`
1991
- |
1992
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
1993
-
1994
- note[mixed-case-variable]: mutable variables should use mixedCase
1995
- --> src/BeefyClient.sol:349:17
1996
- |
1997
- 349 | bytes32 ticketID = createTicketID(msg.sender, commitmentHash);
1998
- | ^^^^^^^^ help: consider using: `ticketId`
1999
- |
2000
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2001
-
2002
- note[mixed-case-variable]: mutable variables should use mixedCase
2003
- --> src/BeefyClient.sol:394:17
2004
- |
2005
- 394 | bytes32 ticketID = createTicketID(msg.sender, commitmentHash);
2006
- | ^^^^^^^^ help: consider using: `ticketId`
2007
- |
2008
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2009
-
2010
- note[mixed-case-variable]: mutable variables should use mixedCase
2011
- --> src/BeefyClient.sol:397:14
2012
- |
2013
- 397 | bool is_next_session = false;
2014
- | ^^^^^^^^^^^^^^^ help: consider using: `isNextSession`
2015
- |
2016
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2017
-
2018
- note[mixed-case-variable]: mutable variables should use mixedCase
2019
- --> src/BeefyClient.sol:412:17
2020
- |
2021
- 412 | bytes32 newMMRRoot = ensureProvidesMMRRoot(commitment);
2022
- | ^^^^^^^^^^ help: consider using: `newMmrRoot`
2023
- |
2024
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2025
-
2026
- note[mixed-case-variable]: mutable variables should use mixedCase
2027
- --> test/SubstrateTypes.t.sol:223:17
2028
- |
2029
- 223 | bytes32 tokenID = keccak256("tokenid");
2030
- | ^^^^^^^ help: consider using: `tokenId`
2031
- |
2032
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2033
-
2034
- note[mixed-case-variable]: mutable variables should use mixedCase
2035
- --> test/SubstrateTypes.t.sol:246:17
2036
- |
2037
- 246 | bytes32 tokenID = keccak256("t2");
2038
- | ^^^^^^^ help: consider using: `tokenId`
2039
- |
2040
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2041
-
2042
- note[mixed-case-variable]: mutable variables should use mixedCase
2043
- --> test/SubstrateTypes.t.sol:273:17
2044
- |
2045
- 273 | bytes32 tokenID = keccak256("t3");
2046
- | ^^^^^^^ help: consider using: `tokenId`
2047
- |
2048
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2049
-
2050
- note[mixed-case-variable]: mutable variables should use mixedCase
2051
- --> test/GatewayV1.t.sol:1068:20
2052
- |
2053
- 1068 | bytes32 public expectChannelIDBytes =
2054
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using: `expectChannelIdBytes`
2055
- |
2056
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2057
-
2058
- note[mixed-case-variable]: mutable variables should use mixedCase
2059
- --> test/GatewayV1.t.sol:1072:16
2060
- |
2061
- 1072 | ParaID para_id = ParaID.wrap(1000);
2062
- | ^^^^^^^ help: consider using: `paraId`
2063
- |
2064
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2065
-
2066
- note[mixed-case-variable]: mutable variables should use mixedCase
2067
- --> test/GatewayV1.t.sol:1073:19
2068
- |
2069
- 1073 | ChannelID channel_id = para_id.into();
2070
- | ^^^^^^^^^^ help: consider using: `channelId`
2071
- |
2072
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2073
-
2074
- note[mixed-case-function]: function names should use mixedCase
2075
- --> src/BeefyClient.sol:444:14
2076
- |
2077
- 444 | function verifyMMRLeafProof(bytes32 leafHash, bytes32[] calldata proof, uint256 proofOrder)
2078
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `verifyMmrLeafProof`
2079
- |
2080
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
2081
-
2082
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
2083
- --> test/GatewayV2.t.sol:1067:20
2084
- |
2085
- 1067 | m.origin = bytes32("x");
2086
- | ^^^^^^^^^^^^
2087
- |
2088
- = note: consider disabling this lint if you're certain the cast is safe
2089
-
2090
- // casting to 'bytes32' is safe because [explain why]
2091
- // forge-lint: disable-next-line(unsafe-typecast)
2092
-
2093
-
2094
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
2095
-
2096
- note[mixed-case-variable]: mutable variables should use mixedCase
2097
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL2Adaptor.s.sol:34:17
2098
- |
2099
- 34 | address BASE_SPOKE_POOL_ADDRESS;
2100
- | ^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `baseSpokePoolAddress`
2101
- |
2102
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2103
-
2104
- note[mixed-case-variable]: mutable variables should use mixedCase
2105
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL2Adaptor.s.sol:35:17
2106
- |
2107
- 35 | address MULTI_CALL_HANDLER_ADDRESS;
2108
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `multiCallHandlerAddress`
2109
- |
2110
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2111
-
2112
- note[mixed-case-variable]: mutable variables should use mixedCase
2113
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL2Adaptor.s.sol:36:17
2114
- |
2115
- 36 | address GATEWAY_V2_ADDRESS;
2116
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `gatewayV2Address`
2117
- |
2118
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2119
-
2120
- note[mixed-case-variable]: mutable variables should use mixedCase
2121
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL2Adaptor.s.sol:37:17
2122
- |
2123
- 37 | address WETH9_ADDRESS;
2124
- | ^^^^^^^^^^^^^ help: consider using: `weth9Address`
2125
- |
2126
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2127
-
2128
- note[mixed-case-variable]: mutable variables should use mixedCase
2129
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL2Adaptor.s.sol:38:17
2130
- |
2131
- 38 | address BASE_WETH9_ADDRESS;
2132
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `baseWeth9Address`
2133
- |
2134
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2135
-
2136
- note[mixed-case-variable]: mutable variables should use mixedCase
2137
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL2Adaptor.s.sol:39:17
2138
- |
2139
- 39 | address UNISWAP_ROUTER_ADDRESS;
2140
- | ^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `uniswapRouterAddress`
2141
- |
2142
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2143
-
2144
- note[unused-import]: unused imports should be removed
2145
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL2Adaptor.s.sol:7:5
2146
- |
2147
- 7 | SPOKE_POOL as SEPOLIA_SPOKE_POOL,
2148
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2149
- |
2150
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2151
-
2152
- note[unused-import]: unused imports should be removed
2153
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL2Adaptor.s.sol:10:5
2154
- |
2155
- 10 | BASE_MULTI_CALL_HANDLER as SEPOLIA_BASE_MULTI_CALL_HANDLER,
2156
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2157
- |
2158
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2159
-
2160
- note[unused-import]: unused imports should be removed
2161
- --> scripts/l2-integration/across/test/TestSnowbridgeL2Adaptor.s.sol:4:17
2162
- |
2163
- 4 | import {Script, console} from "forge-std/Script.sol";
2164
- | ^^^^^^^
2165
- |
2166
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2167
-
2168
- note[unused-import]: unused imports should be removed
2169
- --> scripts/l2-integration/across/test/TestSnowbridgeL2Adaptor.s.sol:6:9
2170
- |
2171
- 6 | import {WETH9} from "canonical-weth/WETH9.sol";
2172
- | ^^^^^
2173
- |
2174
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2175
-
2176
- note[unused-import]: unused imports should be removed
2177
- --> scripts/l2-integration/across/test/TestSnowbridgeL2Adaptor.s.sol:9:9
2178
- |
2179
- 9 | import {ISpokePool} from "../../../../src/l2-integration/interfaces/ISpokePool.sol";
2180
- | ^^^^^^^^^^
2181
- |
2182
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2183
-
2184
- note[unused-import]: unused imports should be removed
2185
- --> scripts/l2-integration/across/test/TestSnowbridgeL2Adaptor.s.sol:18:5
2186
- |
2187
- 18 | BASE_CHAIN_ID as SEPOLIA_BASE_CHAIN_ID,
2188
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2189
- |
2190
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2191
-
2192
- note[unused-import]: unused imports should be removed
2193
- --> scripts/l2-integration/across/test/TestSnowbridgeL2Adaptor.s.sol:20:5
2194
- |
2195
- 20 | BASE_WETH9 as SEPOLIA_BASE_WETH9,
2196
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2197
- |
2198
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2199
-
2200
- note[unused-import]: unused imports should be removed
2201
- --> scripts/l2-integration/across/test/TestSnowbridgeL2Adaptor.s.sol:30:5
2202
- |
2203
- 30 | BASE_CHAIN_ID as MAINNET_BASE_CHAIN_ID,
2204
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2205
- |
2206
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2207
-
2208
- note[unused-import]: unused imports should be removed
2209
- --> scripts/l2-integration/across/test/TestSnowbridgeL2Adaptor.s.sol:32:5
2210
- |
2211
- 32 | BASE_WETH9 as MAINNET_BASE_WETH9,
2212
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2213
- |
2214
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2215
-
2216
- note[unused-import]: unused imports should be removed
2217
- --> scripts/l2-integration/across/test/TestSnowbridgeL1Adaptor.s.sol:4:17
2218
- |
2219
- 4 | import {Script, console} from "forge-std/Script.sol";
2220
- | ^^^^^^^
2221
- |
2222
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2223
-
2224
- note[unused-import]: unused imports should be removed
2225
- --> scripts/l2-integration/across/test/TestSnowbridgeL1Adaptor.s.sol:8:9
2226
- |
2227
- 8 | import {ISpokePool} from "../../../../src/l2-integration/interfaces/ISpokePool.sol";
2228
- | ^^^^^^^^^^
2229
- |
2230
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2231
-
2232
- note[unused-import]: unused imports should be removed
2233
- --> scripts/upgrade/DeployGateway.sol:9:9
2234
- |
2235
- 9 | import {ParaID} from "../../src/Types.sol";
2236
- | ^^^^^^
2237
- |
2238
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2239
-
2240
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
2241
- --> test/BeefyClientAdvanced.t.sol:3:8
2242
- |
2243
- 3 | import "forge-std/Test.sol";
2244
- | ^^^^^^^^^^^^^^^^^^^^
2245
- |
2246
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
2247
-
2248
- note[unused-import]: unused imports should be removed
2249
- --> test/ForkUpgrade202509.t.sol:4:9
2250
- |
2251
- 4 | import {Vm} from "forge-std/Vm.sol";
2252
- | ^^
2253
- |
2254
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2255
-
2256
- note[unused-import]: unused imports should be removed
2257
- --> test/ForkUpgrade202509.t.sol:6:9
2258
- |
2259
- 6 | import {stdJson} from "forge-std/StdJson.sol";
2260
- | ^^^^^^^
2261
- |
2262
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2263
-
2264
- note[unused-import]: unused imports should be removed
2265
- --> test/ForkUpgrade202509.t.sol:7:9
2266
- |
2267
- 7 | import {console} from "forge-std/console.sol";
2268
- | ^^^^^^^
2269
- |
2270
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2271
-
2272
- note[unused-import]: unused imports should be removed
2273
- --> test/ForkUpgrade202509.t.sol:8:18
2274
- |
2275
- 8 | import {UD60x18, ud60x18, unwrap} from "prb/math/src/UD60x18.sol";
2276
- | ^^^^^^^
2277
- |
2278
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2279
-
2280
- note[unused-import]: unused imports should be removed
2281
- --> test/ForkUpgrade202509.t.sol:16:24
2282
- |
2283
- 16 | import {UpgradeParams, SetOperatingModeParams, OperatingMode, RegisterForeignTokenParams} from "../src/v1/Types.sol";
2284
- | ^^^^^^^^^^^^^^^^^^^^^^
2285
- |
2286
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2287
-
2288
- note[unused-import]: unused imports should be removed
2289
- --> test/ForkUpgrade202509.t.sol:16:48
2290
- |
2291
- 16 | import {UpgradeParams, SetOperatingModeParams, OperatingMode, RegisterForeignTokenParams} from "../src/v1/Types.sol";
2292
- | ^^^^^^^^^^^^^
2293
- |
2294
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2295
-
2296
- note[unused-import]: unused imports should be removed
2297
- --> test/ForkUpgrade202509.t.sol:16:63
2298
- |
2299
- 16 | import {UpgradeParams, SetOperatingModeParams, OperatingMode, RegisterForeignTokenParams} from "../src/v1/Types.sol";
2300
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^
2301
- |
2302
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2303
-
2304
- note[unused-import]: unused imports should be removed
2305
- --> test/ForkUpgrade202509.t.sol:17:28
2306
- |
2307
- 17 | import {ChannelID, ParaID, OperatingMode, InboundMessage, Command, TokenInfo, MintForeignTokenParams} from "../src/v1/Types.sol";
2308
- | ^^^^^^^^^^^^^
2309
- |
2310
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2311
-
2312
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
2313
- --> scripts/l2-integration/across/test/TestUniswapQuoter.s.sol:4:8
2314
- |
2315
- 4 | import "forge-std/Script.sol";
2316
- | ^^^^^^^^^^^^^^^^^^^^^^
2317
- |
2318
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
2319
-
2320
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
2321
- --> scripts/l2-integration/across/test/TestUniswapQuoter.s.sol:5:8
2322
- |
2323
- 5 | import "forge-std/console.sol";
2324
- | ^^^^^^^^^^^^^^^^^^^^^^^
2325
- |
2326
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
2327
-
2328
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
2329
- --> scripts/l2-integration/across/test/TestUniswapQuoter.s.sol:6:8
2330
- |
2331
- 6 | import "openzeppelin/token/ERC20/IERC20.sol";
2332
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2333
- |
2334
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
2335
-
2336
- note[mixed-case-variable]: mutable variables should use mixedCase
2337
- --> scripts/l2-integration/across/test/TestUniswapQuoter.s.sol:27:16
2338
- |
2339
- 27 | uint24 POOL_FEE = 500; // 0.05%
2340
- | ^^^^^^^^ help: consider using: `poolFee`
2341
- |
2342
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2343
-
2344
- note[mixed-case-variable]: mutable variables should use mixedCase
2345
- --> test/GatewayV1.t.sol:1367:16
2346
- |
2347
- 1367 | ParaID paraID = ParaID.wrap(1000);
2348
- | ^^^^^^ help: consider using: `paraId`
2349
- |
2350
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2351
-
2352
- note[unused-import]: unused imports should be removed
2353
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL2Adaptor.s.sol:16:5
2354
- |
2355
- 16 | SPOKE_POOL as MAINNET_SPOKE_POOL,
2356
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2357
- |
2358
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2359
-
2360
- note[unused-import]: unused imports should be removed
2361
- --> scripts/l2-integration/across/deploy/DeploySnowbridgeL2Adaptor.s.sol:19:5
2362
- |
2363
- 19 | BASE_MULTI_CALL_HANDLER as MAINNET_BASE_MULTI_CALL_HANDLER,
2364
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2365
- |
2366
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2367
-
2368
- note[unused-import]: unused imports should be removed
2369
- --> test/ForkUpgrade202509.t.sol:17:68
2370
- |
2371
- 17 | import {ChannelID, ParaID, OperatingMode, InboundMessage, Command, TokenInfo, MintForeignTokenParams} from "../src/v1/Types.sol";
2372
- | ^^^^^^^^^
2373
- |
2374
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2375
-
2376
- note[unused-import]: unused imports should be removed
2377
- --> test/GatewayV1.t.sol:5:9
2378
- |
2379
- 5 | import {Strings} from "openzeppelin/utils/Strings.sol";
2380
- | ^^^^^^^
2381
- |
2382
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2383
-
2384
- note[unused-import]: unused imports should be removed
2385
- --> test/GatewayV1.t.sol:6:9
2386
- |
2387
- 6 | import {console} from "forge-std/console.sol";
2388
- | ^^^^^^^
2389
- |
2390
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2391
-
2392
- note[unused-import]: unused imports should be removed
2393
- --> test/GatewayV1.t.sol:8:9
2394
- |
2395
- 8 | import {BeefyClient} from "../src/BeefyClient.sol";
2396
- | ^^^^^^^^^^^
2397
- |
2398
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2399
-
2400
- note[unused-import]: unused imports should be removed
2401
- --> test/GatewayV1.t.sol:11:9
2402
- |
2403
- 11 | import {IInitializable} from "../src/interfaces/IInitializable.sol";
2404
- | ^^^^^^^^^^^^^^
2405
- |
2406
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2407
-
2408
- note[unused-import]: unused imports should be removed
2409
- --> test/GatewayV1.t.sol:20:9
2410
- |
2411
- 20 | import {Agent} from "../src/Agent.sol";
2412
- | ^^^^^
2413
- |
2414
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2415
-
2416
- note[unused-import]: unused imports should be removed
2417
- --> test/GatewayV1.t.sol:22:9
2418
- |
2419
- 22 | import {SubstrateTypes} from "./../src/SubstrateTypes.sol";
2420
- | ^^^^^^^^^^^^^^
2421
- |
2422
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2423
-
2424
- note[unused-import]: unused imports should be removed
2425
- --> test/GatewayV1.t.sol:24:5
2426
- |
2427
- 24 | Channel,
2428
- | ^^^^^^^
2429
- |
2430
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2431
-
2432
- note[mixed-case-variable]: mutable variables should use mixedCase
2433
- --> src/BeefyClient.sol:536:14
2434
- |
2435
- 536 | bool is_next_session = false;
2436
- | ^^^^^^^^^^^^^^^ help: consider using: `isNextSession`
2437
- |
2438
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2439
-
2440
- note[mixed-case-variable]: mutable variables should use mixedCase
2441
- --> src/BeefyClient.sol:555:17
2442
- |
2443
- 555 | bytes32 newMMRRoot = ensureProvidesMMRRoot(commitment);
2444
- | ^^^^^^^^^^ help: consider using: `newMmrRoot`
2445
- |
2446
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2447
-
2448
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
2449
- --> test/GatewayV2.t.sol:1145:23
2450
- |
2451
- 1145 | msgv.origin = bytes32("orig");
2452
- | ^^^^^^^^^^^^^^^
2453
- |
2454
- = note: consider disabling this lint if you're certain the cast is safe
2455
-
2456
- // casting to 'bytes32' is safe because [explain why]
2457
- // forge-lint: disable-next-line(unsafe-typecast)
2458
-
2459
-
2460
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
2461
-
2462
- note[mixed-case-variable]: mutable variables should use mixedCase
2463
- --> src/v1/Handlers.sol:120:41
2464
- |
2465
- 120 | function mintForeignToken(ChannelID channelID, bytes calldata data) external {
2466
- | ^^^^^^^^^ help: consider using: `channelId`
2467
- |
2468
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2469
-
2470
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
2471
- --> test/Math.t.sol:4:8
2472
- |
2473
- 4 | import "forge-std/Test.sol";
2474
- | ^^^^^^^^^^^^^^^^^^^^
2475
- |
2476
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
2477
-
2478
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
2479
- --> test/Math.t.sol:5:8
2480
- |
2481
- 5 | import "forge-std/console.sol";
2482
- | ^^^^^^^^^^^^^^^^^^^^^^^
2483
- |
2484
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
2485
-
2486
- note[named-struct-fields]: prefer initializing structs with named fields
2487
- --> test/ForkUpgrade202509.t.sol:83:13
2488
- |
2489
- 83 | / InboundMessage(
2490
- 84 | | GOVERNANCE_CHANNEL,
2491
- 85 | | nonce + 1,
2492
- 86 | | Command.Upgrade,
2493
- ... |
2494
- 91 | | keccak256("message-id")
2495
- 92 | | ),
2496
- | |_____________^ help: consider using named fields: `InboundMessage({ channelID: GOVERNANCE_CHANNEL, nonce: nonce + 1, command: Command.Upgrade, params: abi.encode(params), maxDispatchGas: 100_000, maxFeePerGas: block.basefee, reward: 0, id: keccak256("message-id") })`
2497
- |
2498
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
2499
-
2500
- note[unused-import]: unused imports should be removed
2501
- --> test/GatewayV1.t.sol:33:9
2502
- |
2503
- 33 | import {PricingStorage} from "../src/storage/PricingStorage.sol";
2504
- | ^^^^^^^^^^^^^^
2505
- |
2506
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2507
-
2508
- note[unused-import]: unused imports should be removed
2509
- --> test/GatewayV1.t.sol:35:9
2510
- |
2511
- 35 | import {TokenLib} from "../src/TokenLib.sol";
2512
- | ^^^^^^^^
2513
- |
2514
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2515
-
2516
- note[mixed-case-variable]: mutable variables should use mixedCase
2517
- --> test/BeefyClientAdvanced.t.sol:30:13
2518
- |
2519
- 30 | uint256 validator0PK;
2520
- | ^^^^^^^^^^^^ help: consider using: `validator0Pk`
2521
- |
2522
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2523
-
2524
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
2525
- --> test/BeefyClientAdvanced.t.sol:34:22
2526
- |
2527
- 34 | bytes32 constant MMRRoot =
2528
- | ^^^^^^^ help: consider using: `MMR_ROOT`
2529
- |
2530
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
2531
-
2532
- note[unused-import]: unused imports should be removed
2533
- --> scripts/l2-integration/across/test/TestSnowbridgeL1Adaptor.s.sol:9:24
2534
- |
2535
- 9 | import {DepositParams, SendParams} from "../../../../src/l2-integration/Types.sol";
2536
- | ^^^^^^^^^^
2537
- |
2538
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2539
-
2540
- note[unused-import]: unused imports should be removed
2541
- --> scripts/l2-integration/across/test/TestSnowbridgeL1Adaptor.s.sol:14:5
2542
- |
2543
- 14 | CHAIN_ID as SEPOLIA_CHAIN_ID,
2544
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2545
- |
2546
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2547
-
2548
- note[unused-import]: unused imports should be removed
2549
- --> scripts/l2-integration/across/test/TestSnowbridgeL1Adaptor.s.sol:21:5
2550
- |
2551
- 21 | CHAIN_ID as MAINNET_CHAIN_ID,
2552
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2553
- |
2554
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2555
-
2556
- note[unused-import]: unused imports should be removed
2557
- --> src/v1/Handlers.sol:13:9
2558
- |
2559
- 13 | import {AgentExecutor} from "../AgentExecutor.sol";
2560
- | ^^^^^^^^^^^^^
2561
- |
2562
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2563
-
2564
- note[mixed-case-function]: function names should use mixedCase
2565
- --> src/BeefyClient.sol:586:14
2566
- |
2567
- 586 | function createTicketID(address account, bytes32 commitmentHash)
2568
- | ^^^^^^^^^^^^^^ help: consider using: `createTicketId`
2569
- |
2570
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
2571
-
2572
- note[unused-import]: unused imports should be removed
2573
- --> scripts/l2-integration/across/test/TestUniswapQuoter.s.sol:7:9
2574
- |
2575
- 7 | import {WETH9} from "canonical-weth/WETH9.sol";
2576
- | ^^^^^
2577
- |
2578
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2579
-
2580
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
2581
- --> test/ForkUpgrade202509.t.sol:165:67
2582
- |
2583
- 165 | emit IGatewayV1.OutboundMessageAccepted(paraID.into(), 1, bytes32('0x'), hex"");
2584
- | ^^^^^^^^^^^^^
2585
- |
2586
- = note: consider disabling this lint if you're certain the cast is safe
2587
-
2588
- // casting to 'bytes32' is safe because [explain why]
2589
- // forge-lint: disable-next-line(unsafe-typecast)
2590
-
2591
-
2592
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
2593
-
2594
- note[named-struct-fields]: prefer initializing structs with named fields
2595
- --> test/ForkUpgrade202509.t.sol:202:13
2596
- |
2597
- 202 | / InboundMessage(
2598
- 203 | | GOVERNANCE_CHANNEL,
2599
- 204 | | nonce + 1,
2600
- 205 | | Command.Upgrade,
2601
- ... |
2602
- 210 | | keccak256("message-id")
2603
- 211 | | ),
2604
- | |_____________^ help: consider using named fields: `InboundMessage({ channelID: GOVERNANCE_CHANNEL, nonce: nonce + 1, command: Command.Upgrade, params: abi.encode(params), maxDispatchGas: 100_000, maxFeePerGas: block.basefee, reward: 0, id: keccak256("message-id") })`
2605
- |
2606
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
2607
-
2608
- note[mixed-case-variable]: mutable variables should use mixedCase
2609
- --> src/BeefyClient.sol:643:17
2610
- |
2611
- 643 | bytes32 ticketID,
2612
- | ^^^^^^^^ help: consider using: `ticketId`
2613
- |
2614
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2615
-
2616
- note[mixed-case-variable]: mutable variables should use mixedCase
2617
- --> src/v1/IGateway.sol:20:27
2618
- |
2619
- 20 | ChannelID indexed channelID, uint64 nonce, bytes32 indexed messageID, bool success
2620
- | ^^^^^^^^^ help: consider using: `channelId`
2621
- |
2622
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2623
-
2624
- note[mixed-case-variable]: mutable variables should use mixedCase
2625
- --> src/v1/IGateway.sol:20:68
2626
- |
2627
- 20 | ChannelID indexed channelID, uint64 nonce, bytes32 indexed messageID, bool success
2628
- | ^^^^^^^^^ help: consider using: `messageId`
2629
- |
2630
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2631
-
2632
- note[mixed-case-variable]: mutable variables should use mixedCase
2633
- --> src/v1/IGateway.sol:25:27
2634
- |
2635
- 25 | ChannelID indexed channelID, uint64 nonce, bytes32 indexed messageID, bytes payload
2636
- | ^^^^^^^^^ help: consider using: `channelId`
2637
- |
2638
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2639
-
2640
- note[mixed-case-variable]: mutable variables should use mixedCase
2641
- --> src/v1/IGateway.sol:25:68
2642
- |
2643
- 25 | ChannelID indexed channelID, uint64 nonce, bytes32 indexed messageID, bytes payload
2644
- | ^^^^^^^^^ help: consider using: `messageId`
2645
- |
2646
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2647
-
2648
- note[mixed-case-variable]: mutable variables should use mixedCase
2649
- --> src/v1/IGateway.sol:32:47
2650
- |
2651
- 32 | event AgentFundsWithdrawn(bytes32 indexed agentID, address indexed recipient, uint256 amount);
2652
- | ^^^^^^^ help: consider using: `agentId`
2653
- |
2654
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2655
-
2656
- note[mixed-case-variable]: mutable variables should use mixedCase
2657
- --> src/v1/IGateway.sol:42:30
2658
- |
2659
- 42 | function agentOf(bytes32 agentID) external view returns (address);
2660
- | ^^^^^^^ help: consider using: `agentId`
2661
- |
2662
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2663
-
2664
- note[mixed-case-variable]: mutable variables should use mixedCase
2665
- --> src/v1/IGateway.sol:44:47
2666
- |
2667
- 44 | function channelOperatingModeOf(ChannelID channelID) external view returns (OperatingMode);
2668
- | ^^^^^^^^^ help: consider using: `channelId`
2669
- |
2670
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2671
-
2672
- note[mixed-case-variable]: mutable variables should use mixedCase
2673
- --> src/v1/IGateway.sol:46:40
2674
- |
2675
- 46 | function channelNoncesOf(ChannelID channelID) external view returns (uint64, uint64);
2676
- | ^^^^^^^^^ help: consider using: `channelId`
2677
- |
2678
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2679
-
2680
- note[mixed-case-function]: function names should use mixedCase
2681
- --> src/v1/IGateway.sol:86:14
2682
- |
2683
- 86 | function queryForeignTokenID(address token) external view returns (bytes32);
2684
- | ^^^^^^^^^^^^^^^^^^^ help: consider using: `queryForeignTokenId`
2685
- |
2686
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
2687
-
2688
- note[unused-import]: unused imports should be removed
2689
- --> scripts/l2-integration/across/test/TestUniswapQuoter.s.sol:9:9
2690
- |
2691
- 9 | import {ISwapRouter} from "../interfaces/ISwapRouter.sol";
2692
- | ^^^^^^^^^^^
2693
- |
2694
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2695
-
2696
- note[named-struct-fields]: prefer initializing structs with named fields
2697
- --> test/GatewayV1.t.sol:255:17
2698
- |
2699
- 255 | / InboundMessage(
2700
- 256 | | assetHubParaID.into(),
2701
- 257 | | 1,
2702
- 258 | | command,
2703
- ... |
2704
- 263 | | messageID
2705
- 264 | | ),
2706
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: command, params: params, maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
2707
- |
2708
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
2709
-
2710
- note[mixed-case-function]: function names should use mixedCase
2711
- --> src/BeefyClient.sol:764:14
2712
- |
2713
- 764 | function ensureProvidesMMRRoot(Commitment calldata commitment)
2714
- | ^^^^^^^^^^^^^^^^^^^^^ help: consider using: `ensureProvidesMmrRoot`
2715
- |
2716
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
2717
-
2718
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
2719
- --> test/ForkUpgrade202509.t.sol:324:67
2720
- |
2721
- 324 | emit IGatewayV1.OutboundMessageAccepted(paraID.into(), 1, bytes32("0x"), hex"");
2722
- | ^^^^^^^^^^^^^
2723
- |
2724
- = note: consider disabling this lint if you're certain the cast is safe
2725
-
2726
- // casting to 'bytes32' is safe because [explain why]
2727
- // forge-lint: disable-next-line(unsafe-typecast)
2728
-
2729
-
2730
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
2731
-
2732
- note[mixed-case-variable]: mutable variables should use mixedCase
2733
- --> src/v1/IGateway.sol:117:37
2734
- |
2735
- 117 | function tokenAddressOf(bytes32 tokenID) external view returns (address);
2736
- | ^^^^^^^ help: consider using: `tokenId`
2737
- |
2738
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2739
-
2740
- note[mixed-case-variable]: mutable variables should use mixedCase
2741
- --> scripts/l2-integration/across/test/TestSnowbridgeL1AdaptorNativeEther.s.sol:28:17
2742
- |
2743
- 28 | uint256 BASE_CHAIN_ID;
2744
- | ^^^^^^^^^^^^^ help: consider using: `baseChainId`
2745
- |
2746
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2747
-
2748
- note[mixed-case-variable]: mutable variables should use mixedCase
2749
- --> test/BeefyClientAdvanced.t.sol:90:17
2750
- |
2751
- 90 | bytes32 ticketID1 = beefyClient.createTicketID_public(honestRelayer1, commitmentHash);
2752
- | ^^^^^^^^^ help: consider using: `ticketId1`
2753
- |
2754
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2755
-
2756
- note[unsafe-cheatcode]: usage of unsafe cheatcodes that can perform dangerous operations
2757
- --> scripts/DeployLocal.sol:36:47
2758
- |
2759
- 36 | string memory beefyCheckpointRaw = vm.readFile(beefyCheckpointFile);
2760
- | ^^^^^^^^
2761
- |
2762
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-cheatcode
2763
-
2764
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
2765
- --> scripts/l2-integration/across/test/TestSnowbridgeL1Adaptor.s.sol:65:9
2766
- |
2767
- 65 | IERC20(params.inputToken).transfer(l1SnowbridgeAdaptor, params.inputAmount);
2768
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2769
- |
2770
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
2771
-
2772
- note[named-struct-fields]: prefer initializing structs with named fields
2773
- --> test/GatewayV1.t.sol:297:17
2774
- |
2775
- 297 | / InboundMessage(
2776
- 298 | | assetHubParaID.into(),
2777
- 299 | | 1,
2778
- 300 | | command,
2779
- ... |
2780
- 305 | | messageID
2781
- 306 | | ),
2782
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: command, params: params, maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
2783
- |
2784
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
2785
-
2786
- note[named-struct-fields]: prefer initializing structs with named fields
2787
- --> test/GatewayV1.t.sol:334:17
2788
- |
2789
- 334 | / InboundMessage(
2790
- 335 | | assetHubParaID.into(),
2791
- 336 | | 1,
2792
- 337 | | command,
2793
- ... |
2794
- 342 | | messageID
2795
- 343 | | ),
2796
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: command, params: params, maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
2797
- |
2798
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
2799
-
2800
- note[named-struct-fields]: prefer initializing structs with named fields
2801
- --> test/GatewayV1.t.sol:369:17
2802
- |
2803
- 369 | / InboundMessage(
2804
- 370 | | assetHubParaID.into(),
2805
- 371 | | 1,
2806
- 372 | | command,
2807
- ... |
2808
- 377 | | messageID
2809
- 378 | | ),
2810
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: command, params: params, maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
2811
- |
2812
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
2813
-
2814
- note[named-struct-fields]: prefer initializing structs with named fields
2815
- --> test/GatewayV1.t.sol:397:17
2816
- |
2817
- 397 | / InboundMessage(
2818
- 398 | | assetHubParaID.into(),
2819
- 399 | | 1,
2820
- 400 | | command,
2821
- ... |
2822
- 405 | | messageID
2823
- 406 | | ),
2824
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: command, params: params, maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
2825
- |
2826
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
2827
-
2828
- note[named-struct-fields]: prefer initializing structs with named fields
2829
- --> test/GatewayV1.t.sol:416:17
2830
- |
2831
- 416 | / InboundMessage(
2832
- 417 | | assetHubParaID.into(),
2833
- 418 | | 1,
2834
- 419 | | command,
2835
- ... |
2836
- 424 | | messageID
2837
- 425 | | ),
2838
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: command, params: params, maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
2839
- |
2840
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
2841
-
2842
- note[mixed-case-function]: function names should use mixedCase
2843
- --> src/BeefyClient.sol:809:14
2844
- |
2845
- 809 | function encodeMMRLeaf(MMRLeaf calldata leaf) internal pure returns (bytes memory) {
2846
- | ^^^^^^^^^^^^^ help: consider using: `encodeMmrLeaf`
2847
- |
2848
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
2849
-
2850
- note[unused-import]: unused imports should be removed
2851
- --> scripts/DeployLocal.sol:17:9
2852
- |
2853
- 17 | import {UD60x18, ud60x18} from "prb/math/src/UD60x18.sol";
2854
- | ^^^^^^^
2855
- |
2856
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2857
-
2858
- note[mixed-case-function]: function names should use mixedCase
2859
- --> src/v1/Types.sol:13:10
2860
- |
2861
- 13 | function ParaIDEq(ParaID a, ParaID b) pure returns (bool) {
2862
- | ^^^^^^^^ help: consider using: `paraIdEq`
2863
- |
2864
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
2865
-
2866
- note[mixed-case-function]: function names should use mixedCase
2867
- --> src/v1/Types.sol:17:10
2868
- |
2869
- 17 | function ParaIDNe(ParaID a, ParaID b) pure returns (bool) {
2870
- | ^^^^^^^^ help: consider using: `paraIdNe`
2871
- |
2872
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
2873
-
2874
- note[mixed-case-variable]: mutable variables should use mixedCase
2875
- --> src/BeefyClient.sol:843:17
2876
- |
2877
- 843 | bytes32 ticketID,
2878
- | ^^^^^^^^ help: consider using: `ticketId`
2879
- |
2880
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2881
-
2882
- note[unused-import]: unused imports should be removed
2883
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:4:17
2884
- |
2885
- 4 | import {Script, console} from "forge-std/Script.sol";
2886
- | ^^^^^^^
2887
- |
2888
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2889
-
2890
- note[unused-import]: unused imports should be removed
2891
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:5:9
2892
- |
2893
- 5 | import {IERC20} from "openzeppelin/token/ERC20/IERC20.sol";
2894
- | ^^^^^^
2895
- |
2896
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2897
-
2898
- note[unused-import]: unused imports should be removed
2899
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:6:9
2900
- |
2901
- 6 | import {WETH9} from "canonical-weth/WETH9.sol";
2902
- | ^^^^^
2903
- |
2904
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2905
-
2906
- note[unused-import]: unused imports should be removed
2907
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:9:9
2908
- |
2909
- 9 | import {ISpokePool} from "../../../../src/l2-integration/interfaces/ISpokePool.sol";
2910
- | ^^^^^^^^^^
2911
- |
2912
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2913
-
2914
- note[unused-import]: unused imports should be removed
2915
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:10:36
2916
- |
2917
- 10 | import {DepositParams, SendParams, SwapParams} from "../../../../src/l2-integration/Types.sol";
2918
- | ^^^^^^^^^^
2919
- |
2920
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2921
-
2922
- note[unused-import]: unused imports should be removed
2923
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:13:5
2924
- |
2925
- 13 | BASE_CHAIN_ID as SEPOLIA_BASE_CHAIN_ID,
2926
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2927
- |
2928
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2929
-
2930
- note[unused-import]: unused imports should be removed
2931
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:14:5
2932
- |
2933
- 14 | WETH9 as SEPOLIA_WETH9,
2934
- | ^^^^^^^^^^^^^^^^^^^^^^
2935
- |
2936
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2937
-
2938
- note[unused-import]: unused imports should be removed
2939
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:15:5
2940
- |
2941
- 15 | BASE_WETH9 as SEPOLIA_BASE_WETH9,
2942
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2943
- |
2944
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2945
-
2946
- note[unused-import]: unused imports should be removed
2947
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:20:5
2948
- |
2949
- 20 | BASE_CHAIN_ID as MAINNET_BASE_CHAIN_ID,
2950
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2951
- |
2952
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2953
-
2954
- note[unused-import]: unused imports should be removed
2955
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:21:5
2956
- |
2957
- 21 | WETH9 as MAINNET_WETH9,
2958
- | ^^^^^^^^^^^^^^^^^^^^^^
2959
- |
2960
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2961
-
2962
- note[unused-import]: unused imports should be removed
2963
- --> scripts/l2-integration/across/test/TestSnowbridgeL2AdaptorNativeEther.s.sol:22:5
2964
- |
2965
- 22 | BASE_WETH9 as MAINNET_BASE_WETH9,
2966
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2967
- |
2968
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
2969
-
2970
- note[mixed-case-variable]: mutable variables should use mixedCase
2971
- --> test/BeefyClientAdvanced.t.sol:108:17
2972
- |
2973
- 108 | bytes32 ticketID2 = beefyClient.createTicketID_public(honestRelayer2, commitmentHash);
2974
- | ^^^^^^^^^ help: consider using: `ticketId2`
2975
- |
2976
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2977
-
2978
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
2979
- --> src/BeefyClient.sol:196:42
2980
- |
2981
- 196 | bytes2 public constant MMR_ROOT_ID = bytes2("mh");
2982
- | ^^^^^^^^^^^^
2983
- |
2984
- = note: consider disabling this lint if you're certain the cast is safe
2985
-
2986
- // casting to 'bytes2' is safe because [explain why]
2987
- // forge-lint: disable-next-line(unsafe-typecast)
2988
-
2989
-
2990
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
2991
-
2992
- note[mixed-case-variable]: mutable variables should use mixedCase
2993
- --> scripts/l2-integration/across/test/TestSnowbridgeL1AdaptorNativeEther.s.sol:29:16
2994
- |
2995
- 29 | uint32 TIME_BUFFER;
2996
- | ^^^^^^^^^^^ help: consider using: `timeBuffer`
2997
- |
2998
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
2999
-
3000
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
3001
- --> scripts/DeployLocal.sol:92:9
3002
- |
3003
- 92 | weth.transfer(user, 10 ether);
3004
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
3005
- |
3006
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
3007
-
3008
- note[mixed-case-variable]: mutable variables should use mixedCase
3009
- --> test/BeefyClientAdvanced.t.sol:147:17
3010
- |
3011
- 147 | bytes32 PREVRANDAO = bytes32(
3012
- | ^^^^^^^^^^ help: consider using: `prevrandao`
3013
- |
3014
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3015
-
3016
- note[mixed-case-variable]: mutable variables should use mixedCase
3017
- --> src/v1/Types.sol:21:22
3018
- |
3019
- 21 | function into(ParaID paraID) pure returns (ChannelID) {
3020
- | ^^^^^^ help: consider using: `paraId`
3021
- |
3022
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3023
-
3024
- note[mixed-case-function]: function names should use mixedCase
3025
- --> src/v1/Types.sol:29:10
3026
- |
3027
- 29 | function ChannelIDEq(ChannelID a, ChannelID b) pure returns (bool) {
3028
- | ^^^^^^^^^^^ help: consider using: `channelIdEq`
3029
- |
3030
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
3031
-
3032
- note[mixed-case-function]: function names should use mixedCase
3033
- --> src/v1/Types.sol:33:10
3034
- |
3035
- 33 | function ChannelIDNe(ChannelID a, ChannelID b) pure returns (bool) {
3036
- | ^^^^^^^^^^^ help: consider using: `channelIdNe`
3037
- |
3038
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
3039
-
3040
- note[mixed-case-variable]: mutable variables should use mixedCase
3041
- --> src/v1/Types.sol:53:15
3042
- |
3043
- 53 | ChannelID channelID;
3044
- | ^^^^^^^^^ help: consider using: `channelId`
3045
- |
3046
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3047
-
3048
- note[mixed-case-variable]: mutable variables should use mixedCase
3049
- --> test/BeefyClientAdvanced.t.sol:346:58
3050
- |
3051
- 346 | function _buildCommitment(uint32 blockNumber, uint64 validatorSetID, bytes32 mmrRoot)
3052
- | ^^^^^^^^^^^^^^ help: consider using: `validatorSetId`
3053
- |
3054
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3055
-
3056
- note[named-struct-fields]: prefer initializing structs with named fields
3057
- --> test/GatewayV1.t.sol:442:17
3058
- |
3059
- 442 | / InboundMessage(
3060
- 443 | | ParaID.wrap(42).into(),
3061
- 444 | | 1,
3062
- 445 | | command,
3063
- ... |
3064
- 450 | | messageID
3065
- 451 | | ),
3066
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: ParaID.wrap(42).into(), nonce: 1, command: command, params: params, maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
3067
- |
3068
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3069
-
3070
- note[named-struct-fields]: prefer initializing structs with named fields
3071
- --> test/GatewayV1.t.sol:470:17
3072
- |
3073
- 470 | / InboundMessage(
3074
- 471 | | assetHubParaID.into(),
3075
- 472 | | 1,
3076
- 473 | | command,
3077
- ... |
3078
- 478 | | messageID
3079
- 479 | | ),
3080
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: command, params: params, maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
3081
- |
3082
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3083
-
3084
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
3085
- --> src/BeefyClient.sol:311:34
3086
- |
3087
- 311 | bytes32 commitmentHash = keccak256(encodeCommitment(commitment));
3088
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
3089
- |
3090
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
3091
-
3092
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
3093
- --> src/BeefyClient.sol:393:34
3094
- |
3095
- 393 | bytes32 commitmentHash = keccak256(encodeCommitment(commitment));
3096
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
3097
- |
3098
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
3099
-
3100
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
3101
- --> scripts/l2-integration/across/test/TestUniswap.s.sol:4:8
3102
- |
3103
- 4 | import "forge-std/Script.sol";
3104
- | ^^^^^^^^^^^^^^^^^^^^^^
3105
- |
3106
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
3107
-
3108
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
3109
- --> scripts/l2-integration/across/test/TestUniswap.s.sol:5:8
3110
- |
3111
- 5 | import "forge-std/console.sol";
3112
- | ^^^^^^^^^^^^^^^^^^^^^^^
3113
- |
3114
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
3115
-
3116
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
3117
- --> scripts/l2-integration/across/test/TestUniswap.s.sol:6:8
3118
- |
3119
- 6 | import "openzeppelin/token/ERC20/IERC20.sol";
3120
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
3121
- |
3122
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
3123
-
3124
- note[mixed-case-function]: function names should use mixedCase
3125
- --> src/v2/Calls.sol:139:14
3126
- |
3127
- 139 | function _handleAssetERC20(address token, uint128 amount) internal returns (Asset memory) {
3128
- | ^^^^^^^^^^^^^^^^^ help: consider using: `_handleAssetErc20`
3129
- |
3130
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
3131
-
3132
- note[named-struct-fields]: prefer initializing structs with named fields
3133
- --> test/Math.t.sol:20:13
3134
- |
3135
- 20 | Log2Test(0, 0),
3136
- | ^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 0, input: 0 })`
3137
- |
3138
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3139
-
3140
- note[named-struct-fields]: prefer initializing structs with named fields
3141
- --> test/Math.t.sol:21:13
3142
- |
3143
- 21 | Log2Test(0, 1),
3144
- | ^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 0, input: 1 })`
3145
- |
3146
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3147
-
3148
- note[named-struct-fields]: prefer initializing structs with named fields
3149
- --> test/Math.t.sol:22:13
3150
- |
3151
- 22 | Log2Test(1, 2),
3152
- | ^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 1, input: 2 })`
3153
- |
3154
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3155
-
3156
- note[named-struct-fields]: prefer initializing structs with named fields
3157
- --> test/Math.t.sol:23:13
3158
- |
3159
- 23 | Log2Test(2, 3),
3160
- | ^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 2, input: 3 })`
3161
- |
3162
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3163
-
3164
- note[named-struct-fields]: prefer initializing structs with named fields
3165
- --> test/Math.t.sol:24:13
3166
- |
3167
- 24 | Log2Test(2, 4),
3168
- | ^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 2, input: 4 })`
3169
- |
3170
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3171
-
3172
- note[named-struct-fields]: prefer initializing structs with named fields
3173
- --> test/Math.t.sol:25:13
3174
- |
3175
- 25 | Log2Test(3, 5),
3176
- | ^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 3, input: 5 })`
3177
- |
3178
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3179
-
3180
- note[named-struct-fields]: prefer initializing structs with named fields
3181
- --> test/Math.t.sol:26:13
3182
- |
3183
- 26 | Log2Test(3, 6),
3184
- | ^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 3, input: 6 })`
3185
- |
3186
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3187
-
3188
- note[named-struct-fields]: prefer initializing structs with named fields
3189
- --> test/Math.t.sol:27:13
3190
- |
3191
- 27 | Log2Test(3, 8),
3192
- | ^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 3, input: 8 })`
3193
- |
3194
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3195
-
3196
- note[unused-import]: unused imports should be removed
3197
- --> scripts/l2-integration/across/test/TestSnowbridgeL1AdaptorNativeEther.s.sol:4:17
3198
- |
3199
- 4 | import {Script, console} from "forge-std/Script.sol";
3200
- | ^^^^^^^
3201
- |
3202
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3203
-
3204
- note[unused-import]: unused imports should be removed
3205
- --> scripts/l2-integration/across/test/TestSnowbridgeL1AdaptorNativeEther.s.sol:5:9
3206
- |
3207
- 5 | import {IERC20} from "openzeppelin/token/ERC20/IERC20.sol";
3208
- | ^^^^^^
3209
- |
3210
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3211
-
3212
- note[unused-import]: unused imports should be removed
3213
- --> scripts/l2-integration/across/test/TestSnowbridgeL1AdaptorNativeEther.s.sol:8:9
3214
- |
3215
- 8 | import {ISpokePool} from "../../../../src/l2-integration/interfaces/ISpokePool.sol";
3216
- | ^^^^^^^^^^
3217
- |
3218
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3219
-
3220
- note[unused-import]: unused imports should be removed
3221
- --> scripts/l2-integration/across/test/TestSnowbridgeL1AdaptorNativeEther.s.sol:9:24
3222
- |
3223
- 9 | import {DepositParams, SendParams} from "../../../../src/l2-integration/Types.sol";
3224
- | ^^^^^^^^^^
3225
- |
3226
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3227
-
3228
- note[named-struct-fields]: prefer initializing structs with named fields
3229
- --> test/GatewayV1.t.sol:524:17
3230
- |
3231
- 524 | / InboundMessage(
3232
- 525 | | assetHubParaID.into(),
3233
- 526 | | 1,
3234
- 527 | | command,
3235
- ... |
3236
- 532 | | messageID
3237
- 533 | | ),
3238
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: command, params: params, maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
3239
- |
3240
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3241
-
3242
- note[named-struct-fields]: prefer initializing structs with named fields
3243
- --> test/GatewayV1.t.sol:567:17
3244
- |
3245
- 567 | / InboundMessage(
3246
- 568 | | assetHubParaID.into(),
3247
- 569 | | 1,
3248
- 570 | | command,
3249
- ... |
3250
- 575 | | messageID
3251
- 576 | | ),
3252
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: command, params: params, maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
3253
- |
3254
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3255
-
3256
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
3257
- --> src/BeefyClient.sol:510:34
3258
- |
3259
- 510 | bytes32 commitmentHash = keccak256(encodeCommitment(commitment));
3260
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
3261
- |
3262
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
3263
-
3264
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
3265
- --> src/BeefyClient.sol:557:34
3266
- |
3267
- 557 | bytes32 commitmentHash = keccak256(encodeCommitment(commitment));
3268
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
3269
- |
3270
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
3271
-
3272
- note[mixed-case-variable]: mutable variables should use mixedCase
3273
- --> src/v1/Types.sol:108:13
3274
- |
3275
- 108 | bytes32 agentID;
3276
- | ^^^^^^^ help: consider using: `agentId`
3277
- |
3278
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3279
-
3280
- note[mixed-case-variable]: mutable variables should use mixedCase
3281
- --> scripts/l2-integration/across/test/TestUniswap.s.sol:19:17
3282
- |
3283
- 19 | address USDC = 0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48;
3284
- | ^^^^ help: consider using: `usdc`
3285
- |
3286
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3287
-
3288
- note[mixed-case-variable]: mutable variables should use mixedCase
3289
- --> scripts/l2-integration/across/test/TestUniswap.s.sol:21:17
3290
- |
3291
- 21 | address WETH = 0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2;
3292
- | ^^^^ help: consider using: `weth`
3293
- |
3294
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3295
-
3296
- note[mixed-case-variable]: mutable variables should use mixedCase
3297
- --> scripts/l2-integration/across/test/TestUniswap.s.sol:23:16
3298
- |
3299
- 23 | uint24 POOL_FEE = 500; // 0.05%
3300
- | ^^^^^^^^ help: consider using: `poolFee`
3301
- |
3302
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3303
-
3304
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
3305
- --> src/upgrade/Gateway202601.sepolia.sol:5:8
3306
- |
3307
- 5 | import "../Gateway.sol";
3308
- | ^^^^^^^^^^^^^^^^
3309
- |
3310
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
3311
-
3312
- note[mixed-case-variable]: mutable variables should use mixedCase
3313
- --> src/v1/Types.sol:153:13
3314
- |
3315
- 153 | bytes32 agentID;
3316
- | ^^^^^^^ help: consider using: `agentId`
3317
- |
3318
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3319
-
3320
- note[mixed-case-variable]: mutable variables should use mixedCase
3321
- --> src/v1/Types.sol:165:13
3322
- |
3323
- 165 | bytes32 foreignTokenID;
3324
- | ^^^^^^^^^^^^^^ help: consider using: `foreignTokenId`
3325
- |
3326
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3327
-
3328
- note[mixed-case-variable]: mutable variables should use mixedCase
3329
- --> src/v1/Types.sol:177:13
3330
- |
3331
- 177 | bytes32 foreignTokenID;
3332
- | ^^^^^^^^^^^^^^ help: consider using: `foreignTokenId`
3333
- |
3334
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3335
-
3336
- note[unused-import]: unused imports should be removed
3337
- --> src/v1/Types.sol:5:9
3338
- |
3339
- 5 | import {TokenInfo, OperatingMode} from "../types/Common.sol";
3340
- | ^^^^^^^^^
3341
- |
3342
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3343
-
3344
- note[unused-import]: unused imports should be removed
3345
- --> src/v1/Types.sol:6:9
3346
- |
3347
- 6 | import {MultiAddress} from "./MultiAddress.sol";
3348
- | ^^^^^^^^^^^^
3349
- |
3350
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3351
-
3352
- note[named-struct-fields]: prefer initializing structs with named fields
3353
- --> test/GatewayV1.t.sol:742:17
3354
- |
3355
- 742 | / InboundMessage(
3356
- 743 | | assetHubParaID.into(),
3357
- 744 | | 1,
3358
- 745 | | CommandV1.Upgrade,
3359
- ... |
3360
- 750 | | messageID
3361
- 751 | | ),
3362
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: CommandV1.Upgrade, params: abi.encode(params), maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
3363
- |
3364
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3365
-
3366
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
3367
- --> src/BeefyClient.sol:755:32
3368
- |
3369
- 755 | bytes32 bitFieldHash = keccak256(abi.encodePacked(bitfield));
3370
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
3371
- |
3372
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
3373
-
3374
- note[unaliased-plain-import]: use named imports '{A, B}' or alias 'import ".." as X'
3375
- --> src/upgrade/Gateway202509.sol:5:8
3376
- |
3377
- 5 | import "../Gateway.sol";
3378
- | ^^^^^^^^^^^^^^^^
3379
- |
3380
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unaliased-plain-import
3381
-
3382
- note[named-struct-fields]: prefer initializing structs with named fields
3383
- --> test/Math.t.sol:28:13
3384
- |
3385
- 28 | Log2Test(4, 9),
3386
- | ^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 4, input: 9 })`
3387
- |
3388
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3389
-
3390
- note[named-struct-fields]: prefer initializing structs with named fields
3391
- --> test/Math.t.sol:29:13
3392
- |
3393
- 29 | Log2Test(4, 12),
3394
- | ^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 4, input: 12 })`
3395
- |
3396
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3397
-
3398
- note[named-struct-fields]: prefer initializing structs with named fields
3399
- --> test/Math.t.sol:30:13
3400
- |
3401
- 30 | Log2Test(4, 16),
3402
- | ^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 4, input: 16 })`
3403
- |
3404
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3405
-
3406
- note[named-struct-fields]: prefer initializing structs with named fields
3407
- --> test/Math.t.sol:31:13
3408
- |
3409
- 31 | Log2Test(5, 17),
3410
- | ^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 5, input: 17 })`
3411
- |
3412
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3413
-
3414
- note[named-struct-fields]: prefer initializing structs with named fields
3415
- --> test/Math.t.sol:32:13
3416
- |
3417
- 32 | Log2Test(5, 24),
3418
- | ^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 5, input: 24 })`
3419
- |
3420
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3421
-
3422
- note[named-struct-fields]: prefer initializing structs with named fields
3423
- --> test/Math.t.sol:33:13
3424
- |
3425
- 33 | Log2Test(5, 32),
3426
- | ^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 5, input: 32 })`
3427
- |
3428
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3429
-
3430
- note[named-struct-fields]: prefer initializing structs with named fields
3431
- --> test/Math.t.sol:34:13
3432
- |
3433
- 34 | Log2Test(6, 33),
3434
- | ^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 6, input: 33 })`
3435
- |
3436
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3437
-
3438
- note[named-struct-fields]: prefer initializing structs with named fields
3439
- --> test/Math.t.sol:35:13
3440
- |
3441
- 35 | Log2Test(6, 48),
3442
- | ^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 6, input: 48 })`
3443
- |
3444
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3445
-
3446
- note[named-struct-fields]: prefer initializing structs with named fields
3447
- --> test/Math.t.sol:36:13
3448
- |
3449
- 36 | Log2Test(6, 64),
3450
- | ^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 6, input: 64 })`
3451
- |
3452
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3453
-
3454
- note[mixed-case-variable]: mutable variables should use mixedCase
3455
- --> src/Agent.sol:19:25
3456
- |
3457
- 19 | constructor(bytes32 agentID) {
3458
- | ^^^^^^^ help: consider using: `agentId`
3459
- |
3460
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3461
-
3462
- note[unused-import]: unused imports should be removed
3463
- --> test/BeefyClientAdvanced.t.sol:9:9
3464
- |
3465
- 9 | import {SubstrateMerkleProof} from "../src/utils/SubstrateMerkleProof.sol";
3466
- | ^^^^^^^^^^^^^^^^^^^^
3467
- |
3468
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3469
-
3470
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
3471
- --> test/BeefyClientAdvanced.t.sol:33:35
3472
- |
3473
- 33 | bytes2 constant MMR_ROOT_ID = bytes2("mh");
3474
- | ^^^^^^^^^^^^
3475
- |
3476
- = note: consider disabling this lint if you're certain the cast is safe
3477
-
3478
- // casting to 'bytes2' is safe because [explain why]
3479
- // forge-lint: disable-next-line(unsafe-typecast)
3480
-
3481
-
3482
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
3483
-
3484
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
3485
- --> test/BeefyClientAdvanced.t.sol:58:60
3486
- |
3487
- 58 | BeefyClient.ValidatorSet({id: VSET_ID, length: uint128(VSET_LEN), root: vsetRoot});
3488
- | ^^^^^^^^^^^^^^^^^
3489
- |
3490
- = note: consider disabling this lint if you're certain the cast is safe
3491
-
3492
- // casting to 'uint128' is safe because [explain why]
3493
- // forge-lint: disable-next-line(unsafe-typecast)
3494
-
3495
-
3496
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
3497
-
3498
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
3499
- --> test/BeefyClientAdvanced.t.sol:60:64
3500
- |
3501
- 60 | BeefyClient.ValidatorSet({id: VSET_ID + 1, length: uint128(VSET_LEN), root: vsetRoot});
3502
- | ^^^^^^^^^^^^^^^^^
3503
- |
3504
- = note: consider disabling this lint if you're certain the cast is safe
3505
-
3506
- // casting to 'uint128' is safe because [explain why]
3507
- // forge-lint: disable-next-line(unsafe-typecast)
3508
-
3509
-
3510
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
3511
-
3512
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
3513
- --> test/BeefyClientAdvanced.t.sol:114:34
3514
- |
3515
- 114 | assertLe(nRequiredAfter, uint32(quorum), "N must be capped at quorum");
3516
- | ^^^^^^^^^^^^^^
3517
- |
3518
- = note: consider disabling this lint if you're certain the cast is safe
3519
-
3520
- // casting to 'uint32' is safe because [explain why]
3521
- // forge-lint: disable-next-line(unsafe-typecast)
3522
-
3523
-
3524
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
3525
-
3526
- note[unused-import]: unused imports should be removed
3527
- --> src/v2/Calls.sol:29:9
3528
- |
3529
- 29 | import {UD60x18, ud60x18, convert} from "prb/math/src/UD60x18.sol";
3530
- | ^^^^^^^
3531
- |
3532
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3533
-
3534
- note[unused-import]: unused imports should be removed
3535
- --> src/v2/Calls.sol:29:18
3536
- |
3537
- 29 | import {UD60x18, ud60x18, convert} from "prb/math/src/UD60x18.sol";
3538
- | ^^^^^^^
3539
- |
3540
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3541
-
3542
- note[unused-import]: unused imports should be removed
3543
- --> src/v2/Calls.sol:29:27
3544
- |
3545
- 29 | import {UD60x18, ud60x18, convert} from "prb/math/src/UD60x18.sol";
3546
- | ^^^^^^^
3547
- |
3548
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3549
-
3550
- note[named-struct-fields]: prefer initializing structs with named fields
3551
- --> test/Math.t.sol:37:13
3552
- |
3553
- 37 | Log2Test(7, 65),
3554
- | ^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 7, input: 65 })`
3555
- |
3556
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3557
-
3558
- note[named-struct-fields]: prefer initializing structs with named fields
3559
- --> test/Math.t.sol:38:13
3560
- |
3561
- 38 | Log2Test(7, 96),
3562
- | ^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 7, input: 96 })`
3563
- |
3564
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3565
-
3566
- note[named-struct-fields]: prefer initializing structs with named fields
3567
- --> test/Math.t.sol:39:13
3568
- |
3569
- 39 | Log2Test(7, 128),
3570
- | ^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 7, input: 128 })`
3571
- |
3572
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3573
-
3574
- note[named-struct-fields]: prefer initializing structs with named fields
3575
- --> test/Math.t.sol:40:13
3576
- |
3577
- 40 | Log2Test(8, 129),
3578
- | ^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 8, input: 129 })`
3579
- |
3580
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3581
-
3582
- note[named-struct-fields]: prefer initializing structs with named fields
3583
- --> test/Math.t.sol:41:13
3584
- |
3585
- 41 | Log2Test(8, 192),
3586
- | ^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 8, input: 192 })`
3587
- |
3588
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3589
-
3590
- note[named-struct-fields]: prefer initializing structs with named fields
3591
- --> test/Math.t.sol:42:13
3592
- |
3593
- 42 | Log2Test(8, 256),
3594
- | ^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 8, input: 256 })`
3595
- |
3596
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3597
-
3598
- note[named-struct-fields]: prefer initializing structs with named fields
3599
- --> test/Math.t.sol:43:13
3600
- |
3601
- 43 | Log2Test(9, 257),
3602
- | ^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 9, input: 257 })`
3603
- |
3604
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3605
-
3606
- note[named-struct-fields]: prefer initializing structs with named fields
3607
- --> test/Math.t.sol:44:13
3608
- |
3609
- 44 | Log2Test(9, 384),
3610
- | ^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 9, input: 384 })`
3611
- |
3612
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3613
-
3614
- note[named-struct-fields]: prefer initializing structs with named fields
3615
- --> test/Math.t.sol:45:13
3616
- |
3617
- 45 | Log2Test(9, 512),
3618
- | ^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 9, input: 512 })`
3619
- |
3620
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3621
-
3622
- note[named-struct-fields]: prefer initializing structs with named fields
3623
- --> test/Math.t.sol:46:13
3624
- |
3625
- 46 | Log2Test(10, 513),
3626
- | ^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 10, input: 513 })`
3627
- |
3628
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3629
-
3630
- note[named-struct-fields]: prefer initializing structs with named fields
3631
- --> test/Math.t.sol:47:13
3632
- |
3633
- 47 | Log2Test(10, 768),
3634
- | ^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 10, input: 768 })`
3635
- |
3636
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3637
-
3638
- note[named-struct-fields]: prefer initializing structs with named fields
3639
- --> test/Math.t.sol:48:13
3640
- |
3641
- 48 | Log2Test(10, 1024),
3642
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 10, input: 1024 })`
3643
- |
3644
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3645
-
3646
- note[named-struct-fields]: prefer initializing structs with named fields
3647
- --> test/Math.t.sol:49:13
3648
- |
3649
- 49 | Log2Test(11, 1025),
3650
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 11, input: 1025 })`
3651
- |
3652
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3653
-
3654
- note[named-struct-fields]: prefer initializing structs with named fields
3655
- --> test/Math.t.sol:50:13
3656
- |
3657
- 50 | Log2Test(11, 1536),
3658
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 11, input: 1536 })`
3659
- |
3660
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3661
-
3662
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
3663
- --> src/BeefyClient.sol:835:30
3664
- |
3665
- 835 | bytes32 hashedLeaf = keccak256(abi.encodePacked(account));
3666
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
3667
- |
3668
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
3669
-
3670
- note[unused-import]: unused imports should be removed
3671
- --> src/Types.sol:5:9
3672
- |
3673
- 5 | import {TokenInfo, OperatingMode} from "./types/Common.sol";
3674
- | ^^^^^^^^^
3675
- |
3676
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3677
-
3678
- note[unused-import]: unused imports should be removed
3679
- --> src/Types.sol:5:20
3680
- |
3681
- 5 | import {TokenInfo, OperatingMode} from "./types/Common.sol";
3682
- | ^^^^^^^^^^^^^
3683
- |
3684
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3685
-
3686
- note[unused-import]: unused imports should be removed
3687
- --> src/Types.sol:6:9
3688
- |
3689
- 6 | import {UD60x18} from "prb/math/src/UD60x18.sol";
3690
- | ^^^^^^^
3691
- |
3692
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3693
-
3694
- note[unused-import]: unused imports should be removed
3695
- --> src/Types.sol:8:5
3696
- |
3697
- 8 | ParaID,
3698
- | ^^^^^^
3699
- |
3700
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3701
-
3702
- note[unused-import]: unused imports should be removed
3703
- --> src/Types.sol:9:5
3704
- |
3705
- 9 | ChannelID,
3706
- | ^^^^^^^^^
3707
- |
3708
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3709
-
3710
- note[unused-import]: unused imports should be removed
3711
- --> src/Types.sol:10:5
3712
- |
3713
- 10 | Channel,
3714
- | ^^^^^^^
3715
- |
3716
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3717
-
3718
- note[unused-import]: unused imports should be removed
3719
- --> src/Types.sol:11:5
3720
- |
3721
- 11 | InboundMessage as InboundMessageV1,
3722
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
3723
- |
3724
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3725
-
3726
- note[unused-import]: unused imports should be removed
3727
- --> src/Types.sol:12:5
3728
- |
3729
- 12 | Command as CommandV1,
3730
- | ^^^^^^^^^^^^^^^^^^^^
3731
- |
3732
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3733
-
3734
- note[unused-import]: unused imports should be removed
3735
- --> src/Types.sol:13:5
3736
- |
3737
- 13 | MultiAddress
3738
- | ^^^^^^^^^^^^
3739
- |
3740
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3741
-
3742
- note[unused-import]: unused imports should be removed
3743
- --> src/Types.sol:15:9
3744
- |
3745
- 15 | import {CallsV1} from "./v1/Calls.sol";
3746
- | ^^^^^^^
3747
- |
3748
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3749
-
3750
- note[unused-import]: unused imports should be removed
3751
- --> src/Types.sol:16:9
3752
- |
3753
- 16 | import {HandlersV1} from "./v1/Handlers.sol";
3754
- | ^^^^^^^^^^
3755
- |
3756
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3757
-
3758
- note[unused-import]: unused imports should be removed
3759
- --> src/Types.sol:17:9
3760
- |
3761
- 17 | import {IGatewayV1} from "./v1/IGateway.sol";
3762
- | ^^^^^^^^^^
3763
- |
3764
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3765
-
3766
- note[unused-import]: unused imports should be removed
3767
- --> src/Types.sol:20:5
3768
- |
3769
- 20 | InboundMessage as InboundMessageV2, Command as CommandV2, CommandKind
3770
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
3771
- |
3772
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3773
-
3774
- note[unused-import]: unused imports should be removed
3775
- --> src/Types.sol:20:41
3776
- |
3777
- 20 | InboundMessage as InboundMessageV2, Command as CommandV2, CommandKind
3778
- | ^^^^^^^^^^^^^^^^^^^^
3779
- |
3780
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3781
-
3782
- note[unused-import]: unused imports should be removed
3783
- --> src/Types.sol:20:63
3784
- |
3785
- 20 | InboundMessage as InboundMessageV2, Command as CommandV2, CommandKind
3786
- | ^^^^^^^^^^^
3787
- |
3788
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3789
-
3790
- note[unused-import]: unused imports should be removed
3791
- --> src/Types.sol:22:9
3792
- |
3793
- 22 | import {CallsV2} from "./v2/Calls.sol";
3794
- | ^^^^^^^
3795
- |
3796
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3797
-
3798
- note[unused-import]: unused imports should be removed
3799
- --> src/Types.sol:23:9
3800
- |
3801
- 23 | import {HandlersV2} from "./v2/Handlers.sol";
3802
- | ^^^^^^^^^^
3803
- |
3804
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3805
-
3806
- note[unused-import]: unused imports should be removed
3807
- --> src/Types.sol:24:9
3808
- |
3809
- 24 | import {IGatewayV2} from "./v2/IGateway.sol";
3810
- | ^^^^^^^^^^
3811
- |
3812
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
3813
-
3814
- note[named-struct-fields]: prefer initializing structs with named fields
3815
- --> test/GatewayV1.t.sol:805:17
3816
- |
3817
- 805 | / InboundMessage(
3818
- 806 | | assetHubParaID.into(),
3819
- 807 | | 1,
3820
- 808 | | CommandV1.SetOperatingMode,
3821
- ... |
3822
- 813 | | messageID
3823
- 814 | | ),
3824
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: CommandV1.SetOperatingMode, params: abi.encode(params), maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
3825
- |
3826
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
3827
-
3828
- note[pascal-case-struct]: structs should use PascalCase
3829
- --> src/Verification.sol:24:12
3830
- |
3831
- 24 | struct MMRLeafPartial {
3832
- | ^^^^^^^^^^^^^^ help: consider using: `MmrLeafPartial`
3833
- |
3834
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#pascal-case-struct
3835
-
3836
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
3837
- --> src/v2/Types.sol:28:20
3838
- |
3839
- 28 | uint8 constant Upgrade = 0;
3840
- | ^^^^^^^ help: consider using: `UPGRADE`
3841
- |
3842
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
3843
-
3844
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
3845
- --> src/v2/Types.sol:30:20
3846
- |
3847
- 30 | uint8 constant SetOperatingMode = 1;
3848
- | ^^^^^^^^^^^^^^^^ help: consider using: `SET_OPERATING_MODE`
3849
- |
3850
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
3851
-
3852
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
3853
- --> src/v2/Types.sol:32:20
3854
- |
3855
- 32 | uint8 constant UnlockNativeToken = 2;
3856
- | ^^^^^^^^^^^^^^^^^ help: consider using: `UNLOCK_NATIVE_TOKEN`
3857
- |
3858
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
3859
-
3860
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
3861
- --> src/v2/Types.sol:34:20
3862
- |
3863
- 34 | uint8 constant RegisterForeignToken = 3;
3864
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using: `REGISTER_FOREIGN_TOKEN`
3865
- |
3866
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
3867
-
3868
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
3869
- --> src/v2/Types.sol:36:20
3870
- |
3871
- 36 | uint8 constant MintForeignToken = 4;
3872
- | ^^^^^^^^^^^^^^^^ help: consider using: `MINT_FOREIGN_TOKEN`
3873
- |
3874
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
3875
-
3876
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
3877
- --> src/v2/Types.sol:38:20
3878
- |
3879
- 38 | uint8 constant CallContract = 5;
3880
- | ^^^^^^^^^^^^ help: consider using: `CALL_CONTRACT`
3881
- |
3882
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
3883
-
3884
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
3885
- --> src/v2/Types.sol:68:20
3886
- |
3887
- 68 | uint8 constant Raw = 0;
3888
- | ^^^ help: consider using: `RAW`
3889
- |
3890
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
3891
-
3892
- note[mixed-case-variable]: mutable variables should use mixedCase
3893
- --> src/Verification.sol:28:16
3894
- |
3895
- 28 | uint64 nextAuthoritySetID;
3896
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `nextAuthoritySetId`
3897
- |
3898
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3899
-
3900
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
3901
- --> test/BeefyClientAdvanced.t.sol:300:36
3902
- |
3903
- 300 | leaf.nextAuthoritySetLen = uint32(VSET_LEN);
3904
- | ^^^^^^^^^^^^^^^^
3905
- |
3906
- = note: consider disabling this lint if you're certain the cast is safe
3907
-
3908
- // casting to 'uint32' is safe because [explain why]
3909
- // forge-lint: disable-next-line(unsafe-typecast)
3910
-
3911
-
3912
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
3913
-
3914
- note[mixed-case-variable]: mutable variables should use mixedCase
3915
- --> src/v2/IGateway.sol:22:30
3916
- |
3917
- 22 | function agentOf(bytes32 agentID) external view returns (address);
3918
- | ^^^^^^^ help: consider using: `agentId`
3919
- |
3920
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3921
-
3922
- note[mixed-case-variable]: mutable variables should use mixedCase
3923
- --> src/v2/IGateway.sol:29:32
3924
- |
3925
- 29 | event AgentCreated(bytes32 agentID, address agent);
3926
- | ^^^^^^^ help: consider using: `agentId`
3927
- |
3928
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3929
-
3930
- note[mixed-case-function]: function names should use mixedCase
3931
- --> src/v2/IGateway.sol:51:14
3932
- |
3933
- 51 | function v2_submit(
3934
- | ^^^^^^^^^ help: consider using: `v2Submit`
3935
- |
3936
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
3937
-
3938
- note[mixed-case-function]: function names should use mixedCase
3939
- --> src/v2/IGateway.sol:81:14
3940
- |
3941
- 81 | function v2_sendMessage(
3942
- | ^^^^^^^^^^^^^^ help: consider using: `v2SendMessage`
3943
- |
3944
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
3945
-
3946
- note[mixed-case-variable]: mutable variables should use mixedCase
3947
- --> src/v1/Calls.sol:159:40
3948
- |
3949
- 159 | function channelNoncesOf(ChannelID channelID) external view returns (uint64, uint64) {
3950
- | ^^^^^^^^^ help: consider using: `channelId`
3951
- |
3952
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3953
-
3954
- note[mixed-case-variable]: mutable variables should use mixedCase
3955
- --> src/v1/Calls.sol:164:47
3956
- |
3957
- 164 | function channelOperatingModeOf(ChannelID channelID) external view returns (OperatingMode) {
3958
- | ^^^^^^^^^ help: consider using: `channelId`
3959
- |
3960
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3961
-
3962
- note[mixed-case-variable]: mutable variables should use mixedCase
3963
- --> src/v1/Calls.sol:170:37
3964
- |
3965
- 170 | function tokenAddressOf(bytes32 tokenID) external view returns (address) {
3966
- | ^^^^^^^ help: consider using: `tokenId`
3967
- |
3968
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
3969
-
3970
- note[mixed-case-function]: function names should use mixedCase
3971
- --> src/v2/IGateway.sol:96:14
3972
- |
3973
- 96 | function v2_registerToken(
3974
- | ^^^^^^^^^^^^^^^^ help: consider using: `v2RegisterToken`
3975
- |
3976
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
3977
-
3978
- note[mixed-case-function]: function names should use mixedCase
3979
- --> src/v2/IGateway.sol:105:14
3980
- |
3981
- 105 | function v2_createAgent(bytes32 id) external;
3982
- | ^^^^^^^^^^^^^^ help: consider using: `v2CreateAgent`
3983
- |
3984
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
3985
-
3986
- note[mixed-case-function]: function names should use mixedCase
3987
- --> src/v2/IGateway.sol:108:14
3988
- |
3989
- 108 | function v2_outboundNonce() external view returns (uint64);
3990
- | ^^^^^^^^^^^^^^^^ help: consider using: `v2OutboundNonce`
3991
- |
3992
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
3993
-
3994
- note[mixed-case-function]: function names should use mixedCase
3995
- --> src/v2/IGateway.sol:111:14
3996
- |
3997
- 111 | function v2_isDispatched(uint64 nonce) external view returns (bool);
3998
- | ^^^^^^^^^^^^^^^ help: consider using: `v2IsDispatched`
3999
- |
4000
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4001
-
4002
- note[mixed-case-variable]: mutable variables should use mixedCase
4003
- --> src/v1/Calls.sol:211:19
4004
- |
4005
- 211 | ChannelID channelID = ticket.dest.into();
4006
- | ^^^^^^^^^ help: consider using: `channelId`
4007
- |
4008
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4009
-
4010
- note[mixed-case-variable]: mutable variables should use mixedCase
4011
- --> src/v1/Calls.sol:242:17
4012
- |
4013
- 242 | bytes32 messageID = keccak256(abi.encodePacked(channelID, channel.outboundNonce));
4014
- | ^^^^^^^^^ help: consider using: `messageId`
4015
- |
4016
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4017
-
4018
- note[unused-import]: unused imports should be removed
4019
- --> src/Initializer.sol:5:9
4020
- |
4021
- 5 | import {AgentExecutor} from "./AgentExecutor.sol";
4022
- | ^^^^^^^^^^^^^
4023
- |
4024
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4025
-
4026
- note[unused-import]: unused imports should be removed
4027
- --> src/Initializer.sol:7:24
4028
- |
4029
- 7 | import {OperatingMode, ParaID, TokenInfo, Channel, ChannelID} from "./Types.sol";
4030
- | ^^^^^^
4031
- |
4032
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4033
-
4034
- note[unused-import]: unused imports should be removed
4035
- --> src/Initializer.sol:7:52
4036
- |
4037
- 7 | import {OperatingMode, ParaID, TokenInfo, Channel, ChannelID} from "./Types.sol";
4038
- | ^^^^^^^^^
4039
- |
4040
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4041
-
4042
- note[unused-import]: unused imports should be removed
4043
- --> src/Initializer.sol:16:18
4044
- |
4045
- 16 | import {UD60x18, ud60x18, convert} from "prb/math/src/UD60x18.sol";
4046
- | ^^^^^^^
4047
- |
4048
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4049
-
4050
- note[unused-import]: unused imports should be removed
4051
- --> src/Initializer.sol:16:27
4052
- |
4053
- 16 | import {UD60x18, ud60x18, convert} from "prb/math/src/UD60x18.sol";
4054
- | ^^^^^^^
4055
- |
4056
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4057
-
4058
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
4059
- --> src/v2/Types.sol:70:20
4060
- |
4061
- 70 | uint8 constant CreateAsset = 1;
4062
- | ^^^^^^^^^^^ help: consider using: `CREATE_ASSET`
4063
- |
4064
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
4065
-
4066
- note[mixed-case-function]: function names should use mixedCase
4067
- --> src/v2/Types.sol:81:10
4068
- |
4069
- 81 | function makeRawXCM(bytes memory xcm) pure returns (Xcm memory) {
4070
- | ^^^^^^^^^^ help: consider using: `makeRawXcm`
4071
- |
4072
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4073
-
4074
- note[mixed-case-function]: function names should use mixedCase
4075
- --> src/v2/Types.sol:85:10
4076
- |
4077
- 85 | function makeCreateAssetXCM(address token, Network network) pure returns (Xcm memory) {
4078
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `makeCreateAssetXcm`
4079
- |
4080
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4081
-
4082
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
4083
- --> src/v2/Types.sol:98:20
4084
- |
4085
- 98 | uint8 constant NativeTokenERC20 = 0;
4086
- | ^^^^^^^^^^^^^^^^ help: consider using: `NATIVE_TOKEN_ERC20`
4087
- |
4088
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
4089
-
4090
- note[screaming-snake-case-const]: constants should use SCREAMING_SNAKE_CASE
4091
- --> src/v2/Types.sol:99:20
4092
- |
4093
- 99 | uint8 constant ForeignTokenERC20 = 1;
4094
- | ^^^^^^^^^^^^^^^^^ help: consider using: `FOREIGN_TOKEN_ERC20`
4095
- |
4096
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#screaming-snake-case-const
4097
-
4098
- note[pascal-case-struct]: structs should use PascalCase
4099
- --> src/v2/Types.sol:103:8
4100
- |
4101
- 103 | struct AsNativeTokenERC20 {
4102
- | ^^^^^^^^^^^^^^^^^^ help: consider using: `AsNativeTokenErc20`
4103
- |
4104
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#pascal-case-struct
4105
-
4106
- note[pascal-case-struct]: structs should use PascalCase
4107
- --> src/v2/Types.sol:109:8
4108
- |
4109
- 109 | struct AsForeignTokenERC20 {
4110
- | ^^^^^^^^^^^^^^^^^^^ help: consider using: `AsForeignTokenErc20`
4111
- |
4112
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#pascal-case-struct
4113
-
4114
- note[mixed-case-variable]: mutable variables should use mixedCase
4115
- --> src/v2/Types.sol:110:13
4116
- |
4117
- 110 | bytes32 foreignID;
4118
- | ^^^^^^^^^ help: consider using: `foreignId`
4119
- |
4120
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4121
-
4122
- note[mixed-case-variable]: mutable variables should use mixedCase
4123
- --> src/v2/Types.sol:121:35
4124
- |
4125
- 121 | function makeForeignAsset(bytes32 foreignID, uint128 amount) pure returns (Asset memory) {
4126
- | ^^^^^^^^^ help: consider using: `foreignId`
4127
- |
4128
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4129
-
4130
- note[mixed-case-variable]: mutable variables should use mixedCase
4131
- --> src/Gateway.sol:249:47
4132
- |
4133
- 249 | function channelOperatingModeOf(ChannelID channelID) external view returns (OperatingMode) {
4134
- | ^^^^^^^^^ help: consider using: `channelId`
4135
- |
4136
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4137
-
4138
- note[mixed-case-variable]: mutable variables should use mixedCase
4139
- --> src/Gateway.sol:253:40
4140
- |
4141
- 253 | function channelNoncesOf(ChannelID channelID) external view returns (uint64, uint64) {
4142
- | ^^^^^^^^^ help: consider using: `channelId`
4143
- |
4144
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4145
-
4146
- note[mixed-case-variable]: mutable variables should use mixedCase
4147
- --> src/Gateway.sol:257:30
4148
- |
4149
- 257 | function agentOf(bytes32 agentID)
4150
- | ^^^^^^^ help: consider using: `agentId`
4151
- |
4152
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4153
-
4154
- note[mixed-case-function]: function names should use mixedCase
4155
- --> src/Gateway.sol:287:14
4156
- |
4157
- 287 | function queryForeignTokenID(address token) external view returns (bytes32) {
4158
- | ^^^^^^^^^^^^^^^^^^^ help: consider using: `queryForeignTokenId`
4159
- |
4160
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4161
-
4162
- note[named-struct-fields]: prefer initializing structs with named fields
4163
- --> test/GatewayV1.t.sol:1044:17
4164
- |
4165
- 1044 | / InboundMessage(
4166
- 1045 | | assetHubParaID.into(),
4167
- 1046 | | 1,
4168
- 1047 | | CommandV1.SetTokenTransferFees,
4169
- ... |
4170
- 1058 | | messageID
4171
- 1059 | | ),
4172
- | |_________________^
4173
- |
4174
- help: consider using named fields
4175
- |
4176
- 1044 ~ InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: CommandV1.SetTokenTransferFees, params: abi.encode(
4177
- 1045 + SetTokenTransferFeesParams({
4178
- 1046 + assetHubCreateAssetFee: createTokenFee * 2,
4179
- 1047 + registerTokenFee: registerTokenFee,
4180
- 1048 + assetHubReserveTransferFee: sendTokenFee * 3
4181
- 1049 + })
4182
- 1050 ~ ), maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID }),
4183
- |
4184
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4185
-
4186
- note[named-struct-fields]: prefer initializing structs with named fields
4187
- --> test/Math.t.sol:51:13
4188
- |
4189
- 51 | Log2Test(11, 2048),
4190
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 11, input: 2048 })`
4191
- |
4192
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4193
-
4194
- note[named-struct-fields]: prefer initializing structs with named fields
4195
- --> test/Math.t.sol:52:13
4196
- |
4197
- 52 | Log2Test(12, 2049),
4198
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 12, input: 2049 })`
4199
- |
4200
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4201
-
4202
- note[named-struct-fields]: prefer initializing structs with named fields
4203
- --> test/Math.t.sol:53:13
4204
- |
4205
- 53 | Log2Test(12, 3072),
4206
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 12, input: 3072 })`
4207
- |
4208
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4209
-
4210
- note[named-struct-fields]: prefer initializing structs with named fields
4211
- --> test/Math.t.sol:54:13
4212
- |
4213
- 54 | Log2Test(12, 4096),
4214
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 12, input: 4096 })`
4215
- |
4216
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4217
-
4218
- note[mixed-case-variable]: mutable variables should use mixedCase
4219
- --> src/v1/Calls.sol:355:17
4220
- |
4221
- 355 | bytes32 foreignID,
4222
- | ^^^^^^^^^ help: consider using: `foreignId`
4223
- |
4224
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4225
-
4226
- note[mixed-case-variable]: mutable variables should use mixedCase
4227
- --> src/Verification.sol:51:16
4228
- |
4229
- 51 | bytes4 consensusEngineID;
4230
- | ^^^^^^^^^^^^^^^^^ help: consider using: `consensusEngineId`
4231
- |
4232
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4233
-
4234
- note[named-struct-fields]: prefer initializing structs with named fields
4235
- --> test/GatewayV1.t.sol:1083:17
4236
- |
4237
- 1083 | / InboundMessage(
4238
- 1084 | | assetHubParaID.into(),
4239
- 1085 | | 1,
4240
- 1086 | | CommandV1.SetPricingParameters,
4241
- ... |
4242
- 1097 | | messageID
4243
- 1098 | | ),
4244
- | |_________________^
4245
- |
4246
- help: consider using named fields
4247
- |
4248
- 1083 ~ InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: CommandV1.SetPricingParameters, params: abi.encode(
4249
- 1084 + SetPricingParametersParams({
4250
- 1085 + exchangeRate: exchangeRate.mul(convert(2)),
4251
- 1086 + multiplier: multiplier.mul(convert(2)),
4252
- 1087 + deliveryCost: outboundFee
4253
- 1088 + })
4254
- 1089 ~ ), maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID }),
4255
- |
4256
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4257
-
4258
- note[mixed-case-variable]: mutable variables should use mixedCase
4259
- --> src/v2/Types.sol:159:13
4260
- |
4261
- 159 | bytes32 foreignTokenID;
4262
- | ^^^^^^^^^^^^^^ help: consider using: `foreignTokenId`
4263
- |
4264
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4265
-
4266
- note[mixed-case-variable]: mutable variables should use mixedCase
4267
- --> src/v2/Types.sol:171:13
4268
- |
4269
- 171 | bytes32 foreignTokenID;
4270
- | ^^^^^^^^^^^^^^ help: consider using: `foreignTokenId`
4271
- |
4272
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4273
-
4274
- note[mixed-case-variable]: mutable variables should use mixedCase
4275
- --> src/Gateway.sol:324:37
4276
- |
4277
- 324 | function tokenAddressOf(bytes32 tokenID) external view returns (address) {
4278
- | ^^^^^^^ help: consider using: `tokenId`
4279
- |
4280
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4281
-
4282
- note[mixed-case-function]: function names should use mixedCase
4283
- --> src/Gateway.sol:333:14
4284
- |
4285
- 333 | function v1_handleAgentExecute(bytes calldata data) external onlySelf {
4286
- | ^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleAgentExecute`
4287
- |
4288
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4289
-
4290
- note[mixed-case-function]: function names should use mixedCase
4291
- --> src/Gateway.sol:338:14
4292
- |
4293
- 338 | function v1_handleUpgrade(bytes calldata data) external onlySelf {
4294
- | ^^^^^^^^^^^^^^^^ help: consider using: `v1HandleUpgrade`
4295
- |
4296
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4297
-
4298
- note[mixed-case-function]: function names should use mixedCase
4299
- --> src/Gateway.sol:343:14
4300
- |
4301
- 343 | function v1_handleSetOperatingMode(bytes calldata data) external onlySelf {
4302
- | ^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleSetOperatingMode`
4303
- |
4304
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4305
-
4306
- note[mixed-case-function]: function names should use mixedCase
4307
- --> src/Gateway.sol:348:14
4308
- |
4309
- 348 | function v1_handleSetTokenTransferFees(bytes calldata data) external onlySelf {
4310
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleSetTokenTransferFees`
4311
- |
4312
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4313
-
4314
- note[mixed-case-function]: function names should use mixedCase
4315
- --> src/Gateway.sol:353:14
4316
- |
4317
- 353 | function v1_handleSetPricingParameters(bytes calldata data) external onlySelf {
4318
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleSetPricingParameters`
4319
- |
4320
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4321
-
4322
- note[mixed-case-function]: function names should use mixedCase
4323
- --> src/Gateway.sol:358:14
4324
- |
4325
- 358 | function v1_handleUnlockNativeToken(bytes calldata data) external onlySelf {
4326
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleUnlockNativeToken`
4327
- |
4328
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4329
-
4330
- note[mixed-case-function]: function names should use mixedCase
4331
- --> src/Gateway.sol:363:14
4332
- |
4333
- 363 | function v1_handleRegisterForeignToken(bytes calldata data) external onlySelf {
4334
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleRegisterForeignToken`
4335
- |
4336
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4337
-
4338
- note[mixed-case-function]: function names should use mixedCase
4339
- --> src/Gateway.sol:368:14
4340
- |
4341
- 368 | function v1_handleMintForeignToken(ChannelID channelID, bytes calldata data)
4342
- | ^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1HandleMintForeignToken`
4343
- |
4344
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4345
-
4346
- note[mixed-case-variable]: mutable variables should use mixedCase
4347
- --> src/Gateway.sol:368:50
4348
- |
4349
- 368 | function v1_handleMintForeignToken(ChannelID channelID, bytes calldata data)
4350
- | ^^^^^^^^^ help: consider using: `channelId`
4351
- |
4352
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4353
-
4354
- note[mixed-case-function]: function names should use mixedCase
4355
- --> src/Gateway.sol:394:14
4356
- |
4357
- 394 | function v1_transactionBaseGas() internal pure returns (uint256) {
4358
- | ^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v1TransactionBaseGas`
4359
- |
4360
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4361
-
4362
- note[mixed-case-function]: function names should use mixedCase
4363
- --> src/Gateway.sol:410:14
4364
- |
4365
- 410 | function v2_submit(
4366
- | ^^^^^^^^^ help: consider using: `v2Submit`
4367
- |
4368
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4369
-
4370
- note[unused-import]: unused imports should be removed
4371
- --> src/v2/Handlers.sol:13:9
4372
- |
4373
- 13 | import {IGatewayV2} from "./IGateway.sol";
4374
- | ^^^^^^^^^^
4375
- |
4376
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4377
-
4378
- note[named-struct-fields]: prefer initializing structs with named fields
4379
- --> test/Math.t.sol:55:13
4380
- |
4381
- 55 | Log2Test(13, 4097),
4382
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 13, input: 4097 })`
4383
- |
4384
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4385
-
4386
- note[named-struct-fields]: prefer initializing structs with named fields
4387
- --> test/Math.t.sol:56:13
4388
- |
4389
- 56 | Log2Test(13, 6144),
4390
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 13, input: 6144 })`
4391
- |
4392
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4393
-
4394
- note[named-struct-fields]: prefer initializing structs with named fields
4395
- --> test/Math.t.sol:57:13
4396
- |
4397
- 57 | Log2Test(13, 8192),
4398
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 13, input: 8192 })`
4399
- |
4400
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4401
-
4402
- note[named-struct-fields]: prefer initializing structs with named fields
4403
- --> test/Math.t.sol:58:13
4404
- |
4405
- 58 | Log2Test(14, 8193),
4406
- | ^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 14, input: 8193 })`
4407
- |
4408
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4409
-
4410
- note[named-struct-fields]: prefer initializing structs with named fields
4411
- --> test/Math.t.sol:59:13
4412
- |
4413
- 59 | Log2Test(14, 12_288),
4414
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 14, input: 12_288 })`
4415
- |
4416
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4417
-
4418
- note[named-struct-fields]: prefer initializing structs with named fields
4419
- --> test/Math.t.sol:60:13
4420
- |
4421
- 60 | Log2Test(14, 16_384),
4422
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 14, input: 16_384 })`
4423
- |
4424
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4425
-
4426
- note[mixed-case-function]: function names should use mixedCase
4427
- --> src/Gateway.sol:444:14
4428
- |
4429
- 444 | function v2_outboundNonce() external view returns (uint64) {
4430
- | ^^^^^^^^^^^^^^^^ help: consider using: `v2OutboundNonce`
4431
- |
4432
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4433
-
4434
- note[mixed-case-function]: function names should use mixedCase
4435
- --> src/Gateway.sol:448:14
4436
- |
4437
- 448 | function v2_isDispatched(uint64 nonce) external view returns (bool) {
4438
- | ^^^^^^^^^^^^^^^ help: consider using: `v2IsDispatched`
4439
- |
4440
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4441
-
4442
- note[mixed-case-function]: function names should use mixedCase
4443
- --> src/Gateway.sol:453:14
4444
- |
4445
- 453 | function v2_sendMessage(
4446
- | ^^^^^^^^^^^^^^ help: consider using: `v2SendMessage`
4447
- |
4448
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4449
-
4450
- note[named-struct-fields]: prefer initializing structs with named fields
4451
- --> test/Math.t.sol:61:13
4452
- |
4453
- 61 | Log2Test(15, 16_385),
4454
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 15, input: 16_385 })`
4455
- |
4456
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4457
-
4458
- note[named-struct-fields]: prefer initializing structs with named fields
4459
- --> test/Math.t.sol:62:13
4460
- |
4461
- 62 | Log2Test(15, 24_576),
4462
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 15, input: 24_576 })`
4463
- |
4464
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4465
-
4466
- note[named-struct-fields]: prefer initializing structs with named fields
4467
- --> test/Math.t.sol:63:13
4468
- |
4469
- 63 | Log2Test(15, 32_768),
4470
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 15, input: 32_768 })`
4471
- |
4472
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4473
-
4474
- note[named-struct-fields]: prefer initializing structs with named fields
4475
- --> test/Math.t.sol:64:13
4476
- |
4477
- 64 | Log2Test(16, 32_769),
4478
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 16, input: 32_769 })`
4479
- |
4480
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4481
-
4482
- note[named-struct-fields]: prefer initializing structs with named fields
4483
- --> test/Math.t.sol:65:13
4484
- |
4485
- 65 | Log2Test(16, 49_152),
4486
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 16, input: 49_152 })`
4487
- |
4488
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4489
-
4490
- note[named-struct-fields]: prefer initializing structs with named fields
4491
- --> test/Math.t.sol:66:13
4492
- |
4493
- 66 | Log2Test(16, 65_535)
4494
- | ^^^^^^^^^^^^^^^^^^^^ help: consider using named fields: `Log2Test({ result: 16, input: 65_535 })`
4495
- |
4496
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4497
-
4498
- note[unused-import]: unused imports should be removed
4499
- --> src/v1/Calls.sol:14:9
4500
- |
4501
- 14 | import {AgentExecutor} from "../AgentExecutor.sol";
4502
- | ^^^^^^^^^^^^^
4503
- |
4504
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4505
-
4506
- note[unused-import]: unused imports should be removed
4507
- --> src/v1/Calls.sol:15:9
4508
- |
4509
- 15 | import {Agent} from "../Agent.sol";
4510
- | ^^^^^
4511
- |
4512
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4513
-
4514
- note[unused-import]: unused imports should be removed
4515
- --> src/v1/Calls.sol:16:9
4516
- |
4517
- 16 | import {Call} from "../utils/Call.sol";
4518
- | ^^^^
4519
- |
4520
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4521
-
4522
- note[unused-import]: unused imports should be removed
4523
- --> src/v1/Calls.sol:25:5
4524
- |
4525
- 25 | AgentExecuteCommand,
4526
- | ^^^^^^^^^^^^^^^^^^^
4527
- |
4528
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4529
-
4530
- note[unused-import]: unused imports should be removed
4531
- --> src/v1/Calls.sol:31:18
4532
- |
4533
- 31 | import {UD60x18, ud60x18, convert} from "prb/math/src/UD60x18.sol";
4534
- | ^^^^^^^
4535
- |
4536
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4537
-
4538
- note[mixed-case-variable]: mutable variables should use mixedCase
4539
- --> src/Verification.sol:108:16
4540
- |
4541
- 108 | bytes4 encodedParaID,
4542
- | ^^^^^^^^^^^^^ help: consider using: `encodedParaId`
4543
- |
4544
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4545
-
4546
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
4547
- --> test/GatewayV1.t.sol:1147:9
4548
- |
4549
- 1147 | token.transfer(address(assetHubAgent), 200);
4550
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
4551
- |
4552
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
4553
-
4554
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
4555
- --> src/v1/Calls.sol:242:29
4556
- |
4557
- 242 | bytes32 messageID = keccak256(abi.encodePacked(channelID, channel.outboundNonce));
4558
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
4559
- |
4560
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
4561
-
4562
- note[named-struct-fields]: prefer initializing structs with named fields
4563
- --> test/GatewayV1.t.sol:1167:17
4564
- |
4565
- 1167 | / InboundMessage(
4566
- 1168 | | assetHubParaID.into(),
4567
- 1169 | | 1,
4568
- 1170 | | CommandV1.RegisterForeignToken,
4569
- ... |
4570
- 1175 | | messageID
4571
- 1176 | | ),
4572
- | |_________________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: CommandV1.RegisterForeignToken, params: abi.encode(params), maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
4573
- |
4574
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4575
-
4576
- note[mixed-case-function]: function names should use mixedCase
4577
- --> src/Gateway.sol:464:14
4578
- |
4579
- 464 | function v2_registerToken(
4580
- | ^^^^^^^^^^^^^^^^ help: consider using: `v2RegisterToken`
4581
- |
4582
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4583
-
4584
- note[mixed-case-function]: function names should use mixedCase
4585
- --> src/Gateway.sol:475:14
4586
- |
4587
- 475 | function v2_createAgent(bytes32 id) external {
4588
- | ^^^^^^^^^^^^^^ help: consider using: `v2CreateAgent`
4589
- |
4590
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4591
-
4592
- note[mixed-case-function]: function names should use mixedCase
4593
- --> src/Gateway.sol:484:14
4594
- |
4595
- 484 | function v2_handleUpgrade(bytes calldata data) external onlySelf {
4596
- | ^^^^^^^^^^^^^^^^ help: consider using: `v2HandleUpgrade`
4597
- |
4598
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4599
-
4600
- note[mixed-case-function]: function names should use mixedCase
4601
- --> src/Gateway.sol:489:14
4602
- |
4603
- 489 | function v2_handleSetOperatingMode(bytes calldata data) external onlySelf {
4604
- | ^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v2HandleSetOperatingMode`
4605
- |
4606
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4607
-
4608
- note[mixed-case-function]: function names should use mixedCase
4609
- --> src/Gateway.sol:494:14
4610
- |
4611
- 494 | function v2_handleUnlockNativeToken(bytes calldata data) external onlySelf {
4612
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v2HandleUnlockNativeToken`
4613
- |
4614
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4615
-
4616
- note[mixed-case-function]: function names should use mixedCase
4617
- --> src/Gateway.sol:499:14
4618
- |
4619
- 499 | function v2_handleRegisterForeignToken(bytes calldata data) external onlySelf {
4620
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v2HandleRegisterForeignToken`
4621
- |
4622
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4623
-
4624
- note[mixed-case-function]: function names should use mixedCase
4625
- --> src/Gateway.sol:504:14
4626
- |
4627
- 504 | function v2_handleMintForeignToken(bytes calldata data) external onlySelf {
4628
- | ^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v2HandleMintForeignToken`
4629
- |
4630
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4631
-
4632
- note[mixed-case-function]: function names should use mixedCase
4633
- --> src/Gateway.sol:509:14
4634
- |
4635
- 509 | function v2_handleCallContract(bytes32 origin, bytes calldata data) external onlySelf {
4636
- | ^^^^^^^^^^^^^^^^^^^^^ help: consider using: `v2HandleCallContract`
4637
- |
4638
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4639
-
4640
- note[mixed-case-function]: function names should use mixedCase
4641
- --> src/Gateway.sol:566:14
4642
- |
4643
- 566 | function v2_dispatch(InboundMessageV2 calldata message) internal returns (bool) {
4644
- | ^^^^^^^^^^^ help: consider using: `v2Dispatch`
4645
- |
4646
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4647
-
4648
- warning[erc20-unchecked-transfer]: ERC20 'transfer' and 'transferFrom' calls should check the return value
4649
- --> test/GatewayV1.t.sol:1334:9
4650
- |
4651
- 1334 | token.transfer(address(assetHubAgent), 200);
4652
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
4653
- |
4654
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#erc20-unchecked-transfer
4655
-
4656
- note[unused-import]: unused imports should be removed
4657
- --> src/Gateway.sol:8:9
4658
- |
4659
- 8 | import {AgentExecutor} from "./AgentExecutor.sol";
4660
- | ^^^^^^^^^^^^^
4661
- |
4662
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4663
-
4664
- note[unused-import]: unused imports should be removed
4665
- --> src/Gateway.sol:29:9
4666
- |
4667
- 29 | import {Upgrade} from "./Upgrade.sol";
4668
- | ^^^^^^^
4669
- |
4670
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4671
-
4672
- note[unused-import]: unused imports should be removed
4673
- --> src/Gateway.sol:41:9
4674
- |
4675
- 41 | import {PricingStorage} from "./storage/PricingStorage.sol";
4676
- | ^^^^^^^^^^^^^^
4677
- |
4678
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4679
-
4680
- note[unused-import]: unused imports should be removed
4681
- --> src/Gateway.sol:44:18
4682
- |
4683
- 44 | import {UD60x18, ud60x18, convert} from "prb/math/src/UD60x18.sol";
4684
- | ^^^^^^^
4685
- |
4686
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4687
-
4688
- note[unused-import]: unused imports should be removed
4689
- --> src/Gateway.sol:44:27
4690
- |
4691
- 44 | import {UD60x18, ud60x18, convert} from "prb/math/src/UD60x18.sol";
4692
- | ^^^^^^^
4693
- |
4694
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4695
-
4696
- note[unwrapped-modifier-logic]: wrap modifier logic to reduce code size
4697
- --> src/Gateway.sol:57:5
4698
- |
4699
- 57 | / modifier onlySelf() {
4700
- 58 | | if (msg.sender != address(this)) {
4701
- 59 | | revert IGatewayBase.Unauthorized();
4702
- ... |
4703
- 62 | | }
4704
- | |_____^
4705
- |
4706
- help: wrap modifier logic to reduce code size
4707
- |
4708
- 57 ~ modifier onlySelf() {
4709
- 58 + _onlySelf();
4710
- 59 + _;
4711
- 60 + }
4712
- 61 +
4713
- 62 + function _onlySelf() internal {
4714
- 63 + if (msg.sender != address(this)) {
4715
- 64 + revert IGatewayBase.Unauthorized();
4716
- 65 + }
4717
- 66 + }
4718
- |
4719
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unwrapped-modifier-logic
4720
-
4721
- note[named-struct-fields]: prefer initializing structs with named fields
4722
- --> test/GatewayV1.t.sol:1413:13
4723
- |
4724
- 1413 | / InboundMessage(
4725
- 1414 | | assetHubParaID.into(),
4726
- 1415 | | 1,
4727
- 1416 | | CommandV1.UnlockNativeToken,
4728
- ... |
4729
- 1421 | | messageID
4730
- 1422 | | ),
4731
- | |_____________^ help: consider using named fields: `InboundMessage({ channelID: assetHubParaID.into(), nonce: 1, command: CommandV1.UnlockNativeToken, params: abi.encode(params), maxDispatchGas: maxDispatchGas, maxFeePerGas: maxRefund, reward: reward, id: messageID })`
4732
- |
4733
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#named-struct-fields
4734
-
4735
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
4736
- --> src/Gateway.sol:158:28
4737
- |
4738
- 158 | bytes32 leafHash = keccak256(abi.encode(message));
4739
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
4740
- |
4741
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
4742
-
4743
- note[mixed-case-variable]: mutable variables should use mixedCase
4744
- --> src/Verification.sol:205:53
4745
- |
4746
- 205 | function createParachainHeaderMerkleLeaf(bytes4 encodedParaID, ParachainHeader calldata header)
4747
- | ^^^^^^^^^^^^^ help: consider using: `encodedParaId`
4748
- |
4749
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4750
-
4751
- note[mixed-case-variable]: mutable variables should use mixedCase
4752
- --> src/Verification.sol:214:43
4753
- |
4754
- 214 | function createParachainHeader(bytes4 encodedParaID, ParachainHeader calldata header)
4755
- | ^^^^^^^^^^^^^ help: consider using: `encodedParaId`
4756
- |
4757
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-variable
4758
-
4759
- note[mixed-case-function]: function names should use mixedCase
4760
- --> src/Verification.sol:243:14
4761
- |
4762
- 243 | function createMMRLeaf(MMRLeafPartial memory leaf, bytes32 parachainHeadsRoot)
4763
- | ^^^^^^^^^^^^^ help: consider using: `createMmrLeaf`
4764
- |
4765
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#mixed-case-function
4766
-
4767
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
4768
- --> src/Gateway.sol:422:28
4769
- |
4770
- 422 | bytes32 leafHash = keccak256(abi.encode(message));
4771
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
4772
- |
4773
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
4774
-
4775
- note[unused-import]: unused imports should be removed
4776
- --> src/Verification.sol:8:9
4777
- |
4778
- 8 | import {SubstrateTypes} from "./SubstrateTypes.sol";
4779
- | ^^^^^^^^^^^^^^
4780
- |
4781
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unused-import
4782
-
4783
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
4784
- --> src/Verification.sol:193:24
4785
- |
4786
- 193 | bytes1(uint8(DIGEST_ITEM_OTHER)),
4787
- | ^^^^^^^^^^^^^^^^^^^^^^^^
4788
- |
4789
- = note: consider disabling this lint if you're certain the cast is safe
4790
-
4791
- // casting to 'uint8' is safe because [explain why]
4792
- // forge-lint: disable-next-line(unsafe-typecast)
4793
-
4794
-
4795
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
4796
-
4797
- warning[unsafe-typecast]: typecasts that can truncate values should be checked
4798
- --> src/Verification.sol:198:40
4799
- |
4800
- 198 | return bytes.concat(bytes1(uint8(DIGEST_ITEM_RUNTIME_ENVIRONMENT_UPDATED)));
4801
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
4802
- |
4803
- = note: consider disabling this lint if you're certain the cast is safe
4804
-
4805
- // casting to 'uint8' is safe because [explain why]
4806
- // forge-lint: disable-next-line(unsafe-typecast)
4807
-
4808
-
4809
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#unsafe-typecast
4810
-
4811
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
4812
- --> src/Verification.sol:211:16
4813
- |
4814
- 211 | return keccak256(createParachainHeader(encodedParaID, header));
4815
- | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
4816
- |
4817
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
4818
-
4819
- note[asm-keccak256]: use of inefficient hashing mechanism; consider using inline assembly
4820
- --> src/Verification.sol:257:16
4821
- |
4822
- 257 | return keccak256(encodedLeaf);
4823
- | ^^^^^^^^^^^^^^^^^^^^^^
4824
- |
4825
- = help: https://book.getfoundry.sh/reference/forge/forge-lint#asm-keccak256
4826
-
4827
-  WARN  Issue while reading "/home/runner/work/snowbridge/snowbridge/web/.npmrc". Failed to replace env in config: ${NPM_AUTH_TOKEN}
4828
-
4829
- > @snowbridge/contract-types@0.3.0 typechain /home/runner/work/snowbridge/snowbridge/web/packages/contract-types
7
+ > @snowbridge/contract-types@0.4.0 typechain /home/runner/work/snowbridge/snowbridge/web/packages/contract-types
4830
8
  > typechain --target ethers-v6 '../../../contracts/out/?(IERC20.sol|IERC20Metadata.sol|IGateway.sol|BeefyClient.sol|WETH9.sol|SnowbridgeL1Adaptor.sol|SnowbridgeL2Adaptor.sol|ISwapQuoter.sol|ISwapRouter.sol|ISwapLegacyRouter.sol)/!(*.abi).json' --out-dir src
4831
9
 
4832
10
  Successfully generated 27 typings!