Skip to content

feat: Add Arc support#3424

Merged
pxrl merged 6 commits into
masterfrom
dijanin-brat/addArc
Jun 16, 2026
Merged

feat: Add Arc support#3424
pxrl merged 6 commits into
masterfrom
dijanin-brat/addArc

Conversation

@dijanin-brat

Copy link
Copy Markdown
Contributor

No description provided.

@dijanin-brat dijanin-brat self-assigned this May 27, 2026
@dijanin-brat

Copy link
Copy Markdown
Contributor Author

@codex can you review this PR that adds spport for Arc chain.
Keep in mind that the Arc's native token is USDC and that there are 2 interfaces for it, one as regular ERC20 token that has 6 decimals and the other one when you look at it as a native token (for example for getBalance method) where it has 18 decimals. Focus on the part where those 2 things can cause a problem:

  • gas estimations needs to be in 18 decimals
  • fills and deposits needs to be in 6 decimals
  • for monitor i think that its basically the same thing because we need the balance that we have on a specific address (its important that if we query with getBalance we div it with 18 decimals and if we query for balance with balanceOf then 6 decimals)
  • for refiller its the same, we just need to be consistent when toping up something, its not super important if we do it with 18 or 6 decimals

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1ac46de709

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/clients/ProfitClient.ts Outdated
Comment thread src/monitor/Monitor.ts Outdated
@dijanin-brat dijanin-brat force-pushed the dijanin-brat/addArc branch from 0fbaa47 to 675fe3a Compare June 15, 2026 11:05
@dijanin-brat dijanin-brat marked this pull request as ready for review June 15, 2026 14:24

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f3ae365c75

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/common/Constants.ts
Comment thread src/common/Constants.ts Outdated
Comment thread src/clients/ProfitClient.ts Outdated
Comment thread src/common/Constants.ts Outdated
Comment thread src/common/Constants.ts Outdated
@pxrl pxrl merged commit b01ebe7 into master Jun 16, 2026
5 checks passed
@pxrl pxrl deleted the dijanin-brat/addArc branch June 16, 2026 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants