Skip to content

chore(deps): upgrade to adder 0.33.0#464

Merged
wolf31o2 merged 1 commit into
mainfrom
chore/adder-0.33.0
Nov 1, 2025
Merged

chore(deps): upgrade to adder 0.33.0#464
wolf31o2 merged 1 commit into
mainfrom
chore/adder-0.33.0

Conversation

@wolf31o2

@wolf31o2 wolf31o2 commented Oct 31, 2025

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Chores

    • Updated dependencies to the latest compatible versions for improved stability and compatibility.
  • Refactor

    • Streamlined internal event handling implementation for enhanced maintainability.

Signed-off-by: Chris Gianelloni <wolf31o2@blinklabs.io>
@wolf31o2
wolf31o2 requested a review from a team as a code owner October 31, 2025 20:20
@coderabbitai

coderabbitai Bot commented Oct 31, 2025

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

This pull request updates two main areas: dependency versions in go.mod are bumped (blinklabs-io/adder to v0.33.0 and SundaeSwap-finance/ogmigo/v6 to v6.2.0), and the indexer refactors event type handling to use generic event types instead of input_chainsync-specific types. Type assertions and context references in handleEventRollback and handleEventTransaction functions are updated to use the new event package types while maintaining existing control flow.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

  • Dependency version bumps are straightforward with no logic changes
  • Type refactoring follows a consistent, homogeneous pattern across functions
  • No new business logic or complex structural changes introduced

Areas requiring attention:

  • Verify that generic event types (event.RollbackEvent, event.TransactionEvent) maintain API compatibility with the previous input_chainsync-specific types
  • Confirm that event.TransactionContext properly replaces the previous context type in all affected code paths
  • Validate that dependency upgrades, particularly blinklabs-io/adder from v0.32.6 to v0.33.0, don't introduce breaking changes or new requirements

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "chore(deps): upgrade to adder 0.33.0" accurately describes a significant and substantive part of the changeset. The title explicitly references the adder dependency upgrade from v0.32.6 to v0.33.0, which is confirmed in the go.mod changes. The title is specific, concise, and clear—it uses concrete version information rather than vague language like "misc updates." While the PR also includes implementation changes in indexer.go related to event type refactoring, the title captures what appears to be the developer's intended primary focus based on the branch name and PR objectives. The title is therefore at least partially related to the changeset, referring to a real and significant aspect of the change.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/adder-0.33.0

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bda76f5 and 9b78a31.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (2)
  • go.mod (2 hunks)
  • internal/indexer/indexer.go (4 hunks)
🔇 Additional comments (6)
internal/indexer/indexer.go (4)

1-1: LGTM!

Copyright year update is appropriate.


195-195: LGTM!

Type assertion correctly updated to use event.RollbackEvent.


222-223: LGTM!

Type assertions correctly updated to use event.TransactionEvent and event.TransactionContext, aligning with the adder v0.33.0 API changes.


184-190: Event type migration is complete and verified.

All references to input_chainsync.RollbackEvent and input_chainsync.TransactionEvent have been successfully replaced with types from the event package. The code at lines 184–190 correctly uses event.RollbackEvent and event.TransactionEvent. The input_chainsync import remains valid and is used solely for ChainSync functionality, not for event types.

go.mod (2)

37-37: The ogmigo v6.2.0 release has been verified and exists.

The v6.2.0 tag exists in the repository, confirming the indirect dependency version is valid.


9-9: Let me also verify the indirect dependency bump:

Let me verify that ogmigo v6.2.0 is indeed a transitive dependency from the adder upgrade:

adder v0.33.0 verified—version exists and has no known security vulnerabilities.

The update is safe to merge.


Comment @coderabbitai help to get the list of available commands and usage tips.

@wolf31o2
wolf31o2 merged commit aafc37f into main Nov 1, 2025
11 checks passed
@wolf31o2
wolf31o2 deleted the chore/adder-0.33.0 branch November 1, 2025 16:03
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