-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
proof/dispute active & other tweaks/suggs [SLT-314] #3232
Changes from all commits
be9caf7
26f0396
b7c00cb
09a40ac
ace1887
caccd1b
7a0c7e7
911d778
b3883c0
7fa2c8e
dd9d3a8
76ad9ac
6f96108
ed49b1b
313a681
044cba7
61c2310
74e1690
b2dae77
5c3686c
9fe08f8
27b6d78
df0e800
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,46 @@ | |
All notable changes to this project will be documented in this file. | ||
See [Conventional Commits](https://conventionalcommits.org) for commit guidelines. | ||
|
||
## [0.3.7](https://github.com/synapsecns/sanguine/compare/@synapsecns/[email protected]...@synapsecns/[email protected]) (2024-10-05) | ||
|
||
**Note:** Version bump only for package @synapsecns/bridge-docs | ||
|
||
|
||
|
||
|
||
|
||
## [0.3.6](https://github.com/synapsecns/sanguine/compare/@synapsecns/[email protected]...@synapsecns/[email protected]) (2024-10-05) | ||
|
||
**Note:** Version bump only for package @synapsecns/bridge-docs | ||
|
||
|
||
|
||
|
||
|
||
## [0.3.5](https://github.com/synapsecns/sanguine/compare/@synapsecns/[email protected]...@synapsecns/[email protected]) (2024-10-04) | ||
|
||
**Note:** Version bump only for package @synapsecns/bridge-docs | ||
|
||
|
||
|
||
|
||
|
||
## [0.3.4](https://github.com/synapsecns/sanguine/compare/@synapsecns/[email protected]...@synapsecns/[email protected]) (2024-10-04) | ||
|
||
**Note:** Version bump only for package @synapsecns/bridge-docs | ||
|
||
|
||
|
||
|
||
|
||
## [0.3.3](https://github.com/synapsecns/sanguine/compare/@synapsecns/[email protected]...@synapsecns/[email protected]) (2024-10-04) | ||
|
||
**Note:** Version bump only for package @synapsecns/bridge-docs | ||
|
||
|
||
|
||
|
||
|
||
## [0.3.2](https://github.com/synapsecns/sanguine/compare/@synapsecns/[email protected]...@synapsecns/[email protected]) (2024-10-03) | ||
|
||
**Note:** Version bump only for package @synapsecns/bridge-docs | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,6 @@ | ||
{ | ||
"npmClient": "yarn", | ||
"useWorkspaces": true, | ||
"packages": [ | ||
"packages/*", | ||
"packages/rfq-indexer/*", | ||
"docs/*" | ||
], | ||
"packages": ["packages/*", "packages/rfq-indexer/*", "docs/*"], | ||
"version": "independent" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,22 @@ | |
All notable changes to this project will be documented in this file. | ||
See [Conventional Commits](https://conventionalcommits.org) for commit guidelines. | ||
|
||
## [0.3.10](https://github.com/synapsecns/sanguine/compare/@synapsecns/[email protected]...@synapsecns/[email protected]) (2024-10-05) | ||
|
||
**Note:** Version bump only for package @synapsecns/explorer-ui | ||
|
||
|
||
|
||
|
||
|
||
## [0.3.9](https://github.com/synapsecns/sanguine/compare/@synapsecns/[email protected]...@synapsecns/[email protected]) (2024-10-05) | ||
|
||
**Note:** Version bump only for package @synapsecns/explorer-ui | ||
|
||
|
||
|
||
|
||
|
||
## [0.3.8](https://github.com/synapsecns/sanguine/compare/@synapsecns/[email protected]...@synapsecns/[email protected]) (2024-10-01) | ||
|
||
**Note:** Version bump only for package @synapsecns/explorer-ui | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
import Image from 'next/image' | ||
import { TOKEN_HASH_MAP, AVWETH, USDC } from 'synapse-constants' | ||
|
||
import { TOKEN_HASH_MAP, AVWETH, USDC } from '@synapsecns/synapse-constants' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Missing Dependency: @synapsecns/synapse-constants The dependency 🔗 Analysis chainLGTM! Verify project dependencies. The update to use the scoped package Please ensure that the project's If the output is 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify the presence of @synapsecns/synapse-constants in package.json
# Test: Search for @synapsecns/synapse-constants in package.json
jq '.dependencies["@synapsecns/synapse-constants"]' package.json
Length of output: 71 |
||
import { getTokenAddressUrl } from '@urls' | ||
|
||
export const AssetImage = ({ tokenAddress, chainId, className }) => { | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,8 @@ | ||
import Link from 'next/link' | ||
import Image from 'next/image' | ||
import { CHAINS } from 'synapse-constants' | ||
import { QuestionMarkCircleIcon } from '@heroicons/react/outline' | ||
|
||
import { CHAINS } from '@synapsecns/synapse-constants' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification The new package This import change will fail because the package is not installed. Please add
🔗 Analysis chainApprove the import change and verify package installation. The update to the import statement from 'synapse-constants' to '@synapsecns/synapse-constants' looks good. This change likely reflects a move to a scoped package or a package name update. To ensure the change is correctly implemented:
Run the following script to find other files that might need updating: 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Find other files still using the old import statement
# Search for files still using the old import
echo "Files still using the old import:"
rg --type typescript "from 'synapse-constants'" -l
# Check if the new package is in package.json
echo "Checking package.json for @synapsecns/synapse-constants:"
jq '.dependencies["@synapsecns/synapse-constants"] // .devDependencies["@synapsecns/synapse-constants"] // "Not found"' package.json
Length of output: 294 Script: #!/bin/bash
# Description: Find other files still using the old import statement and verify the new package installation
# Search for files still using the old import
echo "Files still using the old import:"
rg --type typescript "from 'synapse-constants'" -l
# Check if the new package is in package.json
echo "Checking package.json for @synapsecns/synapse-constants:"
jq '.dependencies["@synapsecns/synapse-constants"] // .devDependencies["@synapsecns/synapse-constants"] // "Not found"' package.json
Length of output: 440 Script: #!/bin/bash
# Description: Find other files still using the old import statement and verify the new package installation
# Search for files still using the old import
echo "Files still using the old import:"
rg "from 'synapse-constants'" --glob '*.ts' --glob '*.tsx' -l
# Check if the new package is in package.json
echo "Checking package.json for @synapsecns/synapse-constants:"
jq '.dependencies["@synapsecns/synapse-constants"] // .devDependencies["@synapsecns/synapse-constants"] // "Not found"' package.json
# Additionally, check package-lock.json or yarn.lock for the package
if [ -f package-lock.json ]; then
echo "Checking package-lock.json for @synapsecns/synapse-constants:"
grep '"@synapsecns/synapse-constants":' package-lock.json || echo "Not found in package-lock.json"
elif [ -f yarn.lock ]; then
echo "Checking yarn.lock for @synapsecns/synapse-constants:"
grep '"@synapsecns/synapse-constants":' yarn.lock || echo "Not found in yarn.lock"
else
echo "No lock file found to check for @synapsecns/synapse-constants."
fi
Length of output: 694 |
||
import { getChainUrl, getExplorerTxUrl } from '@urls' | ||
|
||
const CHAINS_BY_ID = CHAINS.CHAINS_BY_ID | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
import { TOKEN_HASH_MAP, CHAINS } from 'synapse-constants' | ||
import { TOKEN_HASH_MAP, CHAINS } from '@synapsecns/synapse-constants' | ||
import { getCoinTextColor } from '@utils/styles/coins' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Issue Detected: Missing Dependency for The package Recommended Actions:
🔗 Analysis chainLGTM! Verify consistency across the project. The import path change from 'synapse-constants' to '@synapsecns/synapse-constants' looks correct. This change likely reflects a move to a scoped package, which is a good practice for better organization and avoiding naming conflicts. To ensure consistency across the project, please run the following script: Also, please verify that the '@synapsecns/synapse-constants' package is properly installed in your project's dependencies. You can check this by running: If you need any assistance with updating other files or installing the package, please let me know. Issues Detected: Import Path Change May Be Incomplete or Incorrect The package Please execute the above scripts and provide the outputs to further assess the situation. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Check for any remaining imports from 'synapse-constants'
# Test: Search for imports from 'synapse-constants'. Expect: No results.
rg --type typescript --type javascript "from 'synapse-constants'" -g '!node_modules'
# Test: Search for imports from '@synapsecns/synapse-constants'. Expect: Multiple results.
rg --type typescript --type javascript "from '@synapsecns/synapse-constants'" -g '!node_modules'
Length of output: 267 Script: #!/bin/bash
# Description: Check if @synapsecns/synapse-constants is installed
# Test: Look for @synapsecns/synapse-constants in package.json. Expect: The package to be listed.
jq '.dependencies["@synapsecns/synapse-constants"]' package.json
# Test: Check if the package is installed in node_modules. Expect: Directory to exist.
if [ -d "node_modules/@synapsecns/synapse-constants" ]; then
echo "Package is installed in node_modules"
else
echo "Package is not installed in node_modules"
fi
Length of output: 218 Script: #!/bin/bash
# Description: Check for any remaining imports from 'synapse-constants' and '@synapsecns/synapse-constants' without specifying file types.
# Test: Search for imports from 'synapse-constants'. Expect: No results.
rg "from ['\"]synapse-constants['\"]" -g '!node_modules'
# Test: Search for imports from '@synapsecns/synapse-constants'. Expect: Multiple results.
rg "from ['\"]@synapsecns/synapse-constants['\"]" -g '!node_modules'
Length of output: 2271 Script: #!/bin/bash
# Description: Check if @synapsecns/synapse-constants is listed in dependencies or devDependencies.
# Test: Look for @synapsecns/synapse-constants in dependencies and devDependencies.
jq '.dependencies["@synapsecns/synapse-constants"] // .devDependencies["@synapsecns/synapse-constants"]' package.json
Length of output: 124 |
||
import { getNetworkTextColor } from '@utils/styles/networks' | ||
import { Grid } from '@components/tailwind/Grid' | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,7 +2,7 @@ import _ from 'lodash' | |
import { useRouter } from 'next/router' | ||
import { useState, useEffect } from 'react' | ||
import { useSearchParams } from 'next/navigation' | ||
import { TOKEN_HASH_MAP } from 'synapse-constants' | ||
import { TOKEN_HASH_MAP } from '@synapsecns/synapse-constants' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification @synapsecns/synapse-constants is missing from package.json The import statement for TOKEN_HASH_MAP uses '@synapsecns/synapse-constants', but this package is not listed in package.json. Please add '@synapsecns/synapse-constants' to your dependencies to ensure the module is available and avoid potential import errors. 🔗 Analysis chainVerify the updated import path for TOKEN_HASH_MAP The import path for TOKEN_HASH_MAP has been changed from a relative path to an absolute path. While this change can improve code maintainability, please ensure that:
To verify the import, run the following script: 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Verify the import statement and its usage
# Check if the package is installed
if ! grep -q "@synapsecns/synapse-constants" package.json; then
echo "Warning: @synapsecns/synapse-constants is not listed in package.json"
fi
# Check for other occurrences of the old import
rg "from 'synapse-constants'" --type tsx --type ts
# Check for other occurrences of the new import
rg "from '@synapsecns/synapse-constants'" --type tsx --type ts
# Check the usage of TOKEN_HASH_MAP in this file
rg "TOKEN_HASH_MAP" packages/explorer-ui/pages/token/[tokenAddress].tsx
Length of output: 587 |
||
import { useQuery } from '@apollo/client' | ||
import { GET_BRIDGE_TRANSACTIONS_QUERY } from '@graphql/queries' | ||
import { TRANSACTIONS_PATH, getChainUrl } from '@urls' | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
import { CHAINS } from 'synapse-constants' | ||
import { CHAINS } from '@synapsecns/synapse-constants' | ||
import { useState, useEffect } from 'react' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification @synapsecns/synapse-constants package is not listed in package.json The import statement references 🔗 Analysis chainVerify the new package import and its contents. The import statement has been updated to use a scoped package
Run the following script to verify the package and its contents: 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify the existence and contents of the @synapsecns/synapse-constants package
# Test 1: Check if the package is installed
if npm list @synapsecns/synapse-constants | grep -q "@synapsecns/synapse-constants"; then
echo "Package @synapsecns/synapse-constants is installed."
else
echo "Package @synapsecns/synapse-constants is not installed."
fi
# Test 2: Check if CHAINS and CHAIN_ID_NAMES_REVERSE are exported
cat << EOF > temp_test.js
const { CHAINS } = require('@synapsecns/synapse-constants');
console.log(CHAINS && CHAINS.CHAIN_ID_NAMES_REVERSE ? 'CHAINS.CHAIN_ID_NAMES_REVERSE exists' : 'CHAINS.CHAIN_ID_NAMES_REVERSE does not exist');
EOF
node temp_test.js
rm temp_test.js
Length of output: 389 Script: #!/bin/bash
# Description: Verify if @synapsecns/synapse-constants is listed in package.json
# Check if jq is available
if command -v jq > /dev/null 2>&1; then
PACKAGE_EXISTS=$(jq '.dependencies["@synapsecns/synapse-constants"] // .devDependencies["@synapsecns/synapse-constants"]' package.json)
if [ "$PACKAGE_EXISTS" != "null" ]; then
echo "@synapsecns/synapse-constants is listed in package.json."
else
echo "@synapsecns/synapse-constants is not listed in package.json."
fi
else
# Fallback to grep if jq is not available
if grep -q '"@synapsecns/synapse-constants"' package.json; then
echo "@synapsecns/synapse-constants is listed in package.json."
else
echo "@synapsecns/synapse-constants is not listed in package.json."
fi
fi
Length of output: 314 |
||
import { useLazyQuery } from '@apollo/client' | ||
import { GET_BRIDGE_TRANSACTIONS_QUERY } from '@graphql/queries' | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
import { TOKEN_HASH_MAP } from 'synapse-constants' | ||
import { TOKEN_HASH_MAP } from '@synapsecns/synapse-constants' | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification @synapsecns/synapse-constants is not listed in Please add 🔗 Analysis chainApprove import statement update and verify project-wide consistency. The import statement has been updated to use a scoped package name To ensure consistency across the project, please run the following script: This script will help identify any inconsistencies in import statements and verify the package.json entry. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify the usage of @synapsecns/synapse-constants across the project
# Search for any remaining non-scoped 'synapse-constants' imports
echo "Checking for non-scoped 'synapse-constants' imports:"
rg --type typescript --type javascript "from\s+'synapse-constants'" -g '!node_modules'
# Search for the new scoped package imports
echo "Checking for new scoped '@synapsecns/synapse-constants' imports:"
rg --type typescript --type javascript "from\s+'@synapsecns/synapse-constants'" -g '!node_modules'
# Check package.json for the dependency
echo "Checking package.json for @synapsecns/synapse-constants:"
jq '.dependencies["@synapsecns/synapse-constants"] // .devDependencies["@synapsecns/synapse-constants"] // "Not found"' package.json
Length of output: 807 |
||
export const addressToSymbol = ({ tokenAddress, chainId }) => { | ||
if ( | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codebase verification
Google Analytics
trackingID
Not FoundThe
trackingID
for Google Analytics is not configured indocs/bridge/docusaurus.config.ts
. Please ensure that thetrackingID
is properly set up to enable analytics functionality.🔗 Analysis chain
Google Analytics plugin added successfully.
The addition of @docusaurus/plugin-google-gtag for Google Analytics integration is appropriate. The version "^3.5.2" aligns with other Docusaurus packages.
Please ensure that necessary privacy measures (such as user consent mechanisms) are in place before enabling Google Analytics. You may want to update your privacy policy accordingly.
To verify the integration, please run the following script:
It seems the previous verification script encountered errors. Please run the following corrected script to verify the Google Analytics configuration:
It appears that
docusaurus.config.js
does not exist. Please run the following script to locate the Docusaurus configuration file:Please run the following script to verify that the Google Analytics
trackingID
is properly configured indocs/bridge/docusaurus.config.ts
:It appears there was a syntax error in the previous script. Please run the following corrected script to verify that the Google Analytics
trackingID
is properly configured indocs/bridge/docusaurus.config.ts
:🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 312
Script:
Length of output: 124
Script:
Length of output: 81
Script:
Length of output: 93
Script:
Length of output: 89