coderabbitai[bot]
commented
1 week ago Walkthrough
The changes in this pull request primarily involve enhancements to error handling and validation across various components of the codebase. Key modifications include updates to command handling logic in the botmd package, the introduction of a new BytesToArray function in the core package, and extensive updates to validation logic in the REST API routes. Additionally, several Solidity files across the contracts have had their pragma versions updated, and various packages have received version bumps in their package.json files.
Changes
| File Path | Change Summary |
|---|---|
contrib/opbot/botmd/commands.go |
Updated command handling logic, enhanced error handling in traceCommand, modified rfqRefund method, and changed makeFastBridge signature. |
core/bytes.go |
Added new function BytesToArray for converting byte slices to fixed-length arrays with error handling. |
packages/contracts-rfq/CHANGELOG.md |
Updated changelog for version 0.12.1, noting version bump only. |
packages/contracts-rfq/contracts/interfaces/IAdmin.sol |
Updated Solidity version from ^0.8.20 to ^0.8.4. |
packages/contracts-rfq/contracts/interfaces/IFastBridge.sol |
Updated Solidity version from ^0.8.20 to ^0.8.4. |
packages/contracts-rfq/contracts/interfaces/IFastBridgeV2.sol |
Updated Solidity version from ^0.8.20 to ^0.8.4, added new structures and enums for enhanced functionality. |
packages/contracts-rfq/contracts/interfaces/IFastBridgeV2Errors.sol |
Updated Solidity version from ^0.8.20 to ^0.8.4. |
packages/contracts-rfq/contracts/interfaces/IMulticallTarget.sol |
Updated Solidity version from ^0.8.20 to ^0.8.4. |
packages/contracts-rfq/contracts/interfaces/IZapRecipient.sol |
Updated Solidity version from ^0.8.20 to ^0.8.4. |
packages/contracts-rfq/contracts/libs/BridgeTransactionV2.sol |
Updated Solidity version from ^0.8.20 to ^0.8.4. |
packages/contracts-rfq/contracts/libs/Errors.sol |
Updated Solidity version from ^0.8.20 to ^0.8.4. |
packages/contracts-rfq/contracts/utils/MulticallTarget.sol |
Updated Solidity version from ^0.8.20 to ^0.8.4. |
packages/contracts-rfq/package.json |
Updated version from 0.12.0 to 0.12.1. |
packages/contracts-rfq/test/MulticallTarget.t.sol |
Updated Solidity version from ^0.8.0 to ^0.8.4. |
packages/rest-api/CHANGELOG.md |
Updated changelog for version bumps from 1.8.5 to 1.8.7. |
packages/rest-api/package.json |
Updated version from 1.8.4 to 1.8.7, updated dependency @synapsecns/sdk-router to ^0.11.6. |
packages/rest-api/src/app.ts |
Added new CORS middleware to handle cross-origin requests. |
packages/rest-api/src/routes/bridgeRoute.ts |
Enhanced validation logic for the /bridge route, added checks for token decimals. |
packages/rest-api/src/routes/bridgeTxInfoRoute.ts |
Enhanced validation logic for the amount parameter in bridge transaction info. |
packages/rest-api/src/routes/swapRoute.ts |
Enhanced validation logic for the swap route, added checks for token decimals. |
packages/rest-api/src/routes/swapTxInfoRoute.ts |
Enhanced validation logic for the swap transaction info route. |
packages/rest-api/src/validations/validateDecimals.ts |
Added new function validateDecimals for validating decimal precision of amounts. |
packages/sdk-router/CHANGELOG.md |
Updated changelog for version 0.11.6, noting version bump only. |
packages/sdk-router/package.json |
Updated version from 0.11.5 to 0.11.6. |
packages/sdk-router/src/rfq/api.integration.test.ts |
Added retry mechanism for flaky tests. |
packages/sdk-router/src/rfq/api.test.ts |
Enhanced error handling in tests for getAllQuotes function. |
packages/sdk-router/src/sdk.test.ts |
Updated fetch mock and added retry configuration for flaky tests. |
packages/synapse-interface/CHANGELOG.md |
Updated changelog for version bumps from 0.40.17 to 0.40.18. |
packages/synapse-interface/components/layouts/LandingPageWrapper/index.tsx |
Updated endDate for AnnouncementBanner component. |
packages/synapse-interface/package.json |
Updated version from 0.40.16 to 0.40.18, updated dependency @synapsecns/sdk-router to ^0.11.6. |
packages/widget/CHANGELOG.md |
Updated changelog for version 0.9.5, noting version bump only. |
packages/widget/package.json |
Updated version from 0.9.4 to 0.9.5, updated dependency @synapsecns/sdk-router to ^0.11.6. |
Possibly related PRs
- #3156: Enhances validation logic in
bridgeRoute.ts, relevant to command handling. - #3165: Modifies
bridgeControllerfor better bridge fee formatting, related to transaction processing. - #3180: Adds
validateRouteExistsin REST API, relevant to command handling logic. - #3247: Introduces
validSwapvalidation in swap route, related to error handling improvements. - #3378: Modifications to validation logic for
/bridgeroute enhance error handling. - #3379: Cleanup of console errors in SDK tests supports overall API robustness.
- #3381: Relaxation of pragmas in contracts may improve integration compatibility.
Suggested labels
M-deps, size/xs
Suggested reviewers
- trajan0x
- ChiTimesChi
- bigboydiamonds
🐰 In a world of code, we hop and play,
With fixes and features, brightening the day.
From commands to routes, we tidy and mend,
Each change a new journey, around every bend.
So let’s celebrate progress, both big and small,
For every little tweak makes our code better for all! 🐇✨
[!WARNING] There were issues while running some tools. Please review the errors and either fix the tool’s configuration or disable the tool if it’s a critical failure.
🔧 eslint
> If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.packages/synapse-interface/components/layouts/LandingPageWrapper/index.tsx
Oops! Something went wrong! :( ESLint: 8.57.1 Error: Failed to load parser '@babel/eslint-parser' declared in 'packages/synapse-interface/.eslintrc.js': Cannot find module '@babel/core/package.json' Require stack: - /node_modules/@babel/eslint-parser/lib/parse.cjs - /node_modules/@babel/eslint-parser/lib/index.cjs - /node_modules/@eslint/eslintrc/dist/eslintrc.cjs at Module._resolveFilename (node:internal/modules/cjs/loader:1248:15) at Function.resolve (node:internal/modules/helpers:145:19) at Object.(/node_modules/@babel/eslint-parser/lib/parse.cjs:14:19) at Module._compile (node:internal/modules/cjs/loader:1546:14) at Module._extensions..js (node:internal/modules/cjs/loader:1691:10) at Module.load (node:internal/modules/cjs/loader:1317:32) at Module._load (node:internal/modules/cjs/loader:1127:12) at TracingChannel.traceSync (node:diagnostics_channel:315:14) at wrapModuleLoad (node:internal/modules/cjs/loader:217:24) at Module.require (node:internal/modules/cjs/loader:1339:12) 🔧 golangci-lint
level=warning msg="[config_reader] The configuration option `run.skip-files` is deprecated, please use `issues.exclude-files`." level=warning msg="[config_reader] The configuration option `run.skip-dirs` is deprecated, please use `issues.exclude-dirs`." level=warning msg="[config_reader] The configuration option `run.skip-dirs-use-default` is deprecated, please use `issues.exclude-dirs-use-default`." level=warning msg="[lintersdb] The linter \"maligned\" is deprecated (step 2) and deactivated. It should be removed from the list of disabled linters. https://golangci-lint.run/product/roadmap/#linter-deprecation-cycle" level=warning msg="[lintersdb] The linter \"exhaustivestruct\" is deprecated (step 2) and deactivated. It should be removed from the list of disabled linters. https://golangci-lint.run/product/roadmap/#linter-deprecation-cycle" level=warning msg="[lintersdb] The linter \"ifshort\" is deprecated (step 2) and deactivated. It should be removed from the list of disabled linters. https://golangci-lint.run/product/roadmap/#linter-deprecation-cycle" level=warning msg="[lintersdb] The linter \"interfacer\" is deprecated (step 2) and deactivated. It should be removed from the list of disabled linters. https://golangci-lint.run/product/roadmap/#linter-deprecation-cycle" level=warning msg="[lintersdb] The linter \"nosnakecase\" is deprecated (step 2) and deactivated. It should be removed from the list of disabled linters. https://golangci-lint.run/product/roadmap/#linter-deprecation-cycle" level=warning msg="[lintersdb] The name \"goerr113\" is deprecated. The linter has been renamed to: err113." level=warning msg="The linter 'execinquery' is deprecated (since v1.58.0) due to: The repository of the linter has been archived by the owner. " level=warning msg="The linter 'exportloopref' is deprecated (since v1.60.2) due to: Since Go1.22 (loopvar) this linter is no longer relevant. Replaced by copyloopvar."
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
cloudflare-workers-and-pages[bot]
null 822975d6123a35c754a4c149819f10ed71079a87: synapse-interface preview link
Summary by CodeRabbit
Release Notes
New Features
Bug Fixes
Documentation
Chores