Skip to content

[WIP] Prepare GoodWallet for GoodWidget integration - #57

Merged
L03TJ3 merged 13 commits into
mainfrom
copilot/prepare-goodwallet-for-goodwidget-integration
Aug 5, 2026
Merged

[WIP] Prepare GoodWallet for GoodWidget integration#57
L03TJ3 merged 13 commits into
mainfrom
copilot/prepare-goodwallet-for-goodwidget-integration

Conversation

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor
  • Add eth_getBalance and eth_call to WIDGET_PROVIDER_METHODS in policy.ts
  • Create src/widgets/icons/AiCreditsIcon.tsx (local SVG icon)
  • Add AI Credits defineWidget entry to registry.ts
  • Update WIDGETS array to include AI Credits widget (outside test guard)
  • Add registry tests for the AI Credits entry
  • Add vitest alias + fixture mock for @goodwidget/ai-credits-widget/register
  • Add package.json dependency pointing to local tarball
  • Document local tarball workflow in README + .gitignore

@vercel

vercel Bot commented Aug 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
good-wallet-v2 Error Error Aug 5, 2026 12:27pm

Request Review

- Add eth_getBalance and eth_call to WIDGET_PROVIDER_METHODS in policy.ts
- Create AiCreditsIcon.tsx (inline SVG spark/lightning icon)
- Add aiCreditsWidget defineWidget entry to registry.ts
- Include aiCreditsWidget in WIDGETS array (production build)
- Add AI Credits registry tests (presence, validation, dashboard actions)
- Add vitest alias mock for @goodwidget/ai-credits-widget/register
- Add aiCreditsWidgetRegisterMock fixture for unit tests
- Add @goodwidget/ai-credits-widget file: dep in package.json
- Document local tarball workflow in README
- Add local-packages/ to .gitignore

Co-authored-by: blueogin <43612769+blueogin@users.noreply.github.com>
@blueogin

blueogin commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

@copilot
can you rename the branch to copilot/feat/ai-credits-integration?

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

@copilot
can you rename the branch to copilot/feat/ai-credits-integration?

Branch renaming needs to be done directly on GitHub — I'm unable to rename remote branches from this environment. You can rename it via the GitHub UI: go to the repository's Branches page, find the current branch, and use the rename option to change it to copilot/feat/ai-credits-integration.

@blueogin blueogin closed this Aug 4, 2026
@blueogin
blueogin deleted the copilot/prepare-goodwallet-for-goodwidget-integration branch August 4, 2026 13:46
@blueogin
blueogin restored the copilot/prepare-goodwallet-for-goodwidget-integration branch August 4, 2026 13:47
@blueogin blueogin reopened this Aug 4, 2026
- Added environment variables for AI Credits backend URL and funding vault address in .env.example.
- Updated .gitignore to exclude specific local packages.
- Modified next.config.mjs to include new packages for AI Credits widget and set up webpack configuration for React Native compatibility.
- Enhanced AuthenticatedWidgetRoute to pass backend URL and funding vault address to the widget.
- Updated WalletConnectDialog styles for improved layout and responsiveness.
- Refactored widget properties to include backend URL and funding vault address in host properties.
- Added tests for new properties in hostProperties.test.ts.

Co-authored-by: blueogin <43612769+blueogin@users.noreply.github.com>
- Removed backendUrl and fundingVaultAddress from AuthenticatedWidgetRoute and MountedWidget components.
- Introduced elementProps to encapsulate backend URL and funding vault address for better property management.
- Updated hostProperties to handle elementProps, improving the assignment and clearing of widget properties.
- Adjusted tests to reflect changes in property handling and ensure proper functionality.

Co-authored-by: blueogin <43612769+blueogin@users.noreply.github.com>
@blueogin
blueogin marked this pull request as ready for review August 4, 2026 19:55
@blueogin

blueogin commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

@copilot
review this pr

Co-authored-by: blueogin <43612769+blueogin@users.noreply.github.com>
@blueogin

blueogin commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

@copilot
review this pr

…ip1193Provider

Co-authored-by: blueogin <43612769+blueogin@users.noreply.github.com>
sirpy
sirpy previously requested changes Aug 5, 2026
L03TJ3 and others added 3 commits August 5, 2026 21:22
- Removed the dialogOverlay class and replaced it with aboveWidgets for improved z-index management.
- Updated the WalletConnectDialog component to reflect the new class structure, ensuring proper overlay behavior.
- Removed the dependency on status for managing document body overflow.
- Ensured overflow is set to "hidden" when the dialog is mounted and reset to "auto" on unmount.
- Streamlined class name handling in the dialog container for improved readability.
@L03TJ3
L03TJ3 dismissed sirpy’s stale review August 5, 2026 15:31

Comments have been resolved

@L03TJ3
L03TJ3 merged commit 97dd6f1 into main Aug 5, 2026
2 of 3 checks passed
@L03TJ3
L03TJ3 deleted the copilot/prepare-goodwallet-for-goodwidget-integration branch August 5, 2026 15:32
L03TJ3 added a commit that referenced this pull request Aug 5, 2026
* Integration of superfluid widget

* fix: align widget support with fixes done in pr #57

* remove leftover nativesvg shim

* add: env feature-flag for superfluid widget
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.

[PLAN] Prepare GoodWallet for GoodWidget integration

4 participants