Skip to content

Optional create auth and gas tank service - #13

Merged
boqiu merged 1 commit into
mainfrom
optional-gastank-auth
Aug 6, 2026
Merged

Optional create auth and gas tank service#13
boqiu merged 1 commit into
mainfrom
optional-gastank-auth

Conversation

@boqiu

@boqiu boqiu commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

This change is Reviewable

Copilot AI 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.

Pull request overview

This PR updates service initialization and API routing to make the AccountAbstract (auth) and GasTank services optional based on configuration, and updates the example environment file accordingly.

Changes:

  • Make service.New only construct AccountAbstract and GasTankPaymaster when corresponding contract addresses are configured.
  • Conditionally register /aa/auth/* and /aa/gastank/* routes only when those services are enabled.
  • Update .env.example to document how to enable/disable these optional services (and adjust default rate-limit values).

Reviewed changes

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

File Description
service/service.go Builds Services with optional AccountAbstract and GasTank based on configured addresses.
api/route.go Registers AA/GasTank routes only when the services are non-nil; adjusts default rate limits.
.env.example Documents optional enabling via env vars; comments out GasTank address by default.

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

Comment thread service/service.go
Comment thread api/route.go
Comment thread api/route.go
@boqiu
boqiu merged commit 35ae615 into main Aug 6, 2026
2 checks passed
@boqiu
boqiu deleted the optional-gastank-auth branch August 6, 2026 09:25
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