Skip to content

fix: implemented copilot review suggestions#649

Merged
otsybizov merged 2 commits intomainnet-prodfrom
fix/envio-parity-review-suggestions
Apr 1, 2026
Merged

fix: implemented copilot review suggestions#649
otsybizov merged 2 commits intomainnet-prodfrom
fix/envio-parity-review-suggestions

Conversation

@otsybizov
Copy link
Copy Markdown
Contributor

🤖 Linear

Closes CONG-XXX

Copilot AI review requested due to automatic review settings April 1, 2026 17:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR appears to finalize migrating the packages/indexer package into the Yarn v3 monorepo (removing its pnpm lockfile) and updates the Envio subgraph reader to enrich hub deposit/invoice parsing with HubIntent status fetched from Envio.

Changes:

  • Add packages/indexer as a Yarn workspace and update dependency lockfiles accordingly.
  • Update the indexer README instructions from pnpm to yarn, and correct the chain name to “Unichain”.
  • Update EnvioReader to fetch HubIntents for deposits/invoices and propagate intent status into parsed results.

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
yarn.lock Adds lock entries for the new indexer workspace and its dependencies.
package.json Registers packages/indexer as a Yarn workspace.
packages/indexer/README.md Updates setup/run docs to Yarn and fixes “Unichain” naming.
packages/indexer/pnpm-lock.yaml Removes pnpm lockfile as part of Yarn workspace migration.
packages/adapters/subgraph/src/envio/reader.ts Enriches deposit/invoice parsing by fetching HubIntent status; refactors query calls; updates error logging.
Files not reviewed (1)
  • packages/indexer/pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/adapters/subgraph/src/envio/reader.ts
Comment thread packages/adapters/subgraph/src/envio/reader.ts
Comment thread packages/adapters/subgraph/src/envio/reader.ts
@otsybizov otsybizov merged commit 9f0652c into mainnet-prod Apr 1, 2026
23 checks passed
@otsybizov otsybizov deleted the fix/envio-parity-review-suggestions branch April 1, 2026 18:49
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.

3 participants