Skip to content

fix: stabilize Ai Credits AppKit Storybook/Playwright fixtures after #83#101

Merged
sirpy merged 1 commit into
mainfrom
fix/reown-appkit-provider
Jul 21, 2026
Merged

fix: stabilize Ai Credits AppKit Storybook/Playwright fixtures after #83#101
sirpy merged 1 commit into
mainfrom
fix/reown-appkit-provider

Conversation

@blueogin

Copy link
Copy Markdown
Contributor

Description

  • Fix QA fixture wrappers so data-testids render outside Tamagui (plain div instead of YStack without a provider), which was breaking disconnected/purchase Playwright lookups after the global Storybook provider was removed.
  • Harden AppKit connect coverage: drop the nonexistent app-kit-provider-default story/test; if w3m-modal.open is already present (Storybook play / AppKit), assert it instead of re-clicking Connect Wallet.
  • Refresh Ai Credits AppKit connect screenshots (acw-14-*); remove orphaned acw-13-appkit-provider-default.png.

About # (link your issue here)
After #83

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

Checklist:

  • PR title matches follow: (Feature|Bug|Chore) Task Name
  • My code follows the style guidelines of this project
  • I have followed all the instructions described in the initial task (check Definitions of Done)
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have added reference to a related issue in the repository
  • I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
  • I have added screenshots related to my pull request (for frontend tasks)
  • I have pasted a gif showing the feature.
  • @mentions of the person or team responsible for reviewing proposed changes

…eliability

- Update AiCreditsWidget story to check for modal visibility before clicking the connect button, preventing unnecessary clicks.
- Change story structure from YStack to div for better compatibility and rendering.
- Remove obsolete appKitProviderDefault test case and update related test logic for appKitConnectWallet.
- Regenerate test result images to reflect recent changes in widget behavior and UI states.
@blueogin
blueogin requested review from a team and L03TJ3 July 20, 2026 17:55
@sirpy
sirpy merged commit de6ecd9 into main Jul 21, 2026
1 check passed
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