Prepare AI Credits Widget for GoodWallet integration by Copilot · Pull Request #134 · GoodDollar/GoodWidget

Copilot · 2026-07-29T13:14:36Z

AI Credits now accepts GoodWallet’s injected EIP-1193 provider and theme overrides consistently in React and Custom Element hosts. Production configuration fails closed rather than silently activating mock clients.

Host integration
- Forward Custom Element provider, config, theme overrides, and default theme to AiCreditsWidget.
- Preserve existing payment success/error events.
Production safety
- Add an unavailable backend client for missing production/staging backend configuration.
- Restrict mock backend and chain clients to development.
- Retain widget-owned Celo/Base RPC and contract defaults.
SSR and packaging
- Make @goodwidget/ai-credits-widget/register defer browser-only element imports until registration.
- Make Custom Element class evaluation safe without DOM globals.
- Package typed ESM/CJS entries for React, element, and register consumers.
Consumer documentation
- Document React and Custom Element integration.
- Document required EIP-1193 methods:
  eth_accounts, eth_chainId, eth_requestAccounts,
  wallet_switchEthereumChain, personal_sign, and eth_sendTransaction.

import { AiCreditsWidget } from '@goodwidget/ai-credits-widget'

<AiCreditsWidget provider={provider} backendUrl={backendUrl} themeOverrides={themeOverrides} />

Fixes [PLAN] Prepare AI Credits Widget for GoodWallet integration #133

Co-authored-by: L03TJ3 <6606028+L03TJ3@users.noreply.github.com>

…widget structure - Simplified payment status checks in the adapter by removing redundant conditions for 'backend_unavailable'. - Updated the logic for setting derived statuses to improve clarity and maintainability. - Enhanced the AiCreditsWidgetElement structure by creating a base class for better property management. - Introduced bridged property handling in createMiniAppElement to improve component integration and state management.

Copilot

Pull request overview

Prepares the AI Credits widget package for GoodWallet integration by aligning provider/theming handoff across React and Custom Element hosts, tightening production safety around backend/chain client selection, and making register SSR-friendly.

Changes:

Forward Custom Element runtime inputs (provider/config/theme overrides/default theme + bridged properties like backendUrl) into AiCreditsWidget.
Fail closed when production/staging backend configuration is missing; restrict mock backend/chain clients to development.
Make @goodwidget/ai-credits-widget/register SSR-safe via guarded, deferred element import; add consumer-facing docs and package export/sideEffects metadata.

Reviewed changes

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

Show a summary per file

File	Description
packages/embed/src/createMiniAppElement.tsx	Adds SSR-safe HTMLElement base and improved property bridging into React + provider/theming propagation.
packages/ai-credits-widget/src/register.ts	Defers element import until runtime and guards `customElements` for SSR/Next.js safety.
packages/ai-credits-widget/src/index.ts	Exposes `UnavailableAiCreditsBackendClient` from the package root.
packages/ai-credits-widget/src/element.ts	Bridges `backendUrl` as a property and exports a typed element class.
packages/ai-credits-widget/src/chainClient.ts	Updates mock behavior gating by widget environment and adjusts mock method signatures.
packages/ai-credits-widget/src/backendClient.ts	Introduces an unavailable backend client and environment-aware backend client factory.
packages/ai-credits-widget/src/AiCreditsWidget.tsx	Removes an unused import (no behavioral change).
packages/ai-credits-widget/src/adapter.ts	Threads environment into backend/chain client creation and adds a backend-unavailable state on missing prod config.
packages/ai-credits-widget/README.md	Documents React + Custom Element usage and required EIP-1193 capabilities.
packages/ai-credits-widget/package.json	Ensures typed ESM/CJS exports for `.`, `./element`, `./register`, with correct publish files + sideEffects.

…niAppElement return type - Introduced new MiniAppElement and MiniAppElementConstructor interfaces to enhance type safety and clarity in the embed package. - Changed the return type of createMiniAppElement to MiniAppElementConstructor for better integration and usage consistency. - Updated index export to include the new types for improved accessibility in other modules.

blueogin · 2026-07-29T20:53:52Z

@copilot
review this pr

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

Copilot · 2026-07-29T21:00:57Z

@copilot
review this pr

Reviewed and addressed the outstanding PR feedback in commit 6d44cbd.

Changes included:

fixed custom-element pre-upgrade hydration for provider/themeOverrides/config
restored backward-compatible createChainClient call patterns
threaded environment into history backend-client creation
guarded register auto-call against unhandled import rejection

Validation run:

pnpm --filter @goodwidget/embed lint
pnpm --filter @goodwidget/ai-credits-widget lint
pnpm --filter @goodwidget/embed build
pnpm --filter @goodwidget/ai-credits-widget build
parallel validation (CodeQL clean)

sirpy

@blueogin there are many open issues, resolve and reply.

sirpy · 2026-07-30T08:56:48Z

-    _options: { txHashes?: string[]; previousBalance?: string } = {},
-  ): Promise<SettlementResult> {
+  async waitForSettlement(ref: AccountRef): Promise<SettlementResult> {
    await sleep(MOCK_DELAY_MS)


is it really waiting for settlement? what is the mock delay for

This is mock-only behavior. It waits for the shared 600ms mock delay to exercise the pending/loading UI, then marks pending mock transactions as funded. It does not poll real settlement; the production client handles actual settlement polling.

sirpy · 2026-07-30T08:57:31Z


-  async getWithdrawableUsd(_buyer: string): Promise<string> {
+  async getWithdrawableUsd(): Promise<string> {
    return '0'


what are these mock values?

The mock intentionally returns "0" because it does not model FundingVault principal balances

Initial plan

d7905c4

Copilot AI assigned Copilot and L03TJ3 Jul 29, 2026

Copilot AI requested review from Copilot and removed request for Copilot July 29, 2026 13:14

Copilot started work on behalf of L03TJ3 July 29, 2026 13:15 View session

Copilot AI linked an issue Jul 29, 2026 that may be closed by this pull request

[PLAN] Prepare AI Credits Widget for GoodWallet integration #133

Open

15 tasks

fix: prepare AI Credits widget host integration

8c9cb2a

Co-authored-by: L03TJ3 <6606028+L03TJ3@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot July 29, 2026 13:23

Copilot AI changed the title ~~[WIP] Prepare AI Credits Widget for GoodWallet integration~~ Prepare AI Credits Widget for GoodWallet integration Jul 29, 2026

Copilot finished work on behalf of L03TJ3 July 29, 2026 13:26

Copilot AI requested a review from L03TJ3 July 29, 2026 13:26

Copilot AI review requested due to automatic review settings July 29, 2026 18:54

Copilot started reviewing on behalf of blueogin July 29, 2026 18:54 View session

Copilot AI reviewed Jul 29, 2026

View reviewed changes

Comment thread packages/embed/src/createMiniAppElement.tsx

Comment thread packages/ai-credits-widget/src/chainClient.ts

Comment thread packages/ai-credits-widget/src/backendClient.ts

Comment thread packages/ai-credits-widget/src/register.ts Outdated

Copilot started work on behalf of blueogin July 29, 2026 20:54 View session

fix ai-credits review feedback for element upgrades and client compat…

6d44cbd

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

Copilot finished work on behalf of blueogin July 29, 2026 21:01

Copilot AI requested a review from blueogin July 29, 2026 21:01

blueogin approved these changes Jul 29, 2026

View reviewed changes

blueogin marked this pull request as ready for review July 29, 2026 21:10

blueogin requested a review from a team July 29, 2026 21:10

blueogin approved these changes Jul 29, 2026

View reviewed changes

sirpy requested changes Jul 30, 2026

View reviewed changes

blueogin requested a review from sirpy July 30, 2026 12:46

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Prepare AI Credits Widget for GoodWallet integration - #134

Prepare AI Credits Widget for GoodWallet integration#134
L03TJ3 with Copilot wants to merge 5 commits into
mainfrom
copilot/prep-ai-credits-widget-goodwallet-integration

Copilot AI commented Jul 29, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

blueogin commented Jul 29, 2026

Uh oh!

Copilot AI commented Jul 29, 2026

Uh oh!

sirpy left a comment

Uh oh!

sirpy Jul 30, 2026

Uh oh!

blueogin Jul 30, 2026

Uh oh!

sirpy Jul 30, 2026

Uh oh!

blueogin Jul 30, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

Copilot AI commented Jul 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

blueogin commented Jul 29, 2026

Uh oh!

Copilot AI commented Jul 29, 2026

Uh oh!

sirpy left a comment

Choose a reason for hiding this comment

Uh oh!

sirpy Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

blueogin Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

sirpy Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

blueogin Jul 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Copilot AI commented Jul 29, 2026 •

edited

Loading

blueogin Jul 30, 2026 •

edited

Loading