Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 8, 2025

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original prompt

This section details on the original issue you should resolve

<issue_title>[BUG] Coinbase Notification Type Detection, Spend Permission Grant, and Passkey Sign-In Failure</issue_title>
<issue_description>## Bug Description
Coinbase Notification Type Detection Logic, Spend Permission Grant, and Hardcoded Passkey Credential sign-in are not functioning as expected in Onchain Test Kit (OTK-30).

Steps to Reproduce

  1. Prepare Onchain Test Kit and configure for Coinbase Wallet usage
  2. Initiate a workflow that triggers notification type detection
  3. Attempt to grant spend permission using hardcoded passkey
  4. Try to sign in with hardcoded passkey credential

Expected Behavior

Notification type should be detected correctly, spend permission should be granted, and sign in with hardcoded passkey should succeed without errors.

Actual Behavior

Notification type is not detected; spend permission cannot be granted; sign in with hardcoded passkey credential fails. Observed error during workflow execution.

Environment Information

  • Onchain Test Kit Version: [please supply]
  • Node.js Version: [please supply]
  • Operating System: [please supply]
  • Browser: [please supply]
  • Wallet: Coinbase Wallet

Configuration Used

// Paste your configuration code here
// Example:
const config = configure()
  .withCoinbaseWallet()
  // ... other configuration for notification type and passkey
  .build();

Error Logs

Error encountered during notification detection and passkey sign-in (see linked PR review).
_Originally posted by @joe10832 in https://github.com/MunyayLLC/https-github.com-joe10832-onchaintestkit/pull/197#pullrequestreview-3307872186_

Code Sample

import { createOnchainTest, configure } from '@coinbase/onchaintestkit';

const test = createOnchainTest(/* your config */);

test('coinbase notification test', async ({ page, coinbase }) => {
  // Steps that cause the bug
});

Screenshots

N/A

Additional Context

Issue discovered while implementing OTK-30. Test failure relates to Coinbase-specific logic for notification, spend permission, and passkey credential sign-in.

Possible Solution

Review and correct the Coinbase notification detection logic, permission grant flow, and passkey credential handling in OTK.

Checklist

  • I have searched for existing issues that describe this bug
  • I have included all the required environment information
  • I have provided a minimal code example that reproduces the issue
  • I have included relevant error messages and logs
    </issue_description>

<agent_instructions>import { createOnchainTest, configure } from '@coinbase/onchaintestkit';

const test = createOnchainTest(/* your config */);

test('coinbase notification test', async ({ page, coinbase }) => {
// Steps that cause the bug
});</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes #198

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copy link
Member

@joe10832 joe10832 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot <issue_title>copilot-setup-steps</issue_title>
<issue_description>/.github/copilot-instructions.md
/.github/instructions/**/*.instructions.md
**/AGENTS.md
/CLAUDE.md
/GEMINI.md</issue_description>import { createOnchainTest, configure } from '@coinbase/onchaintestkit';

const test = createOnchainTest(/* your config */);

test('coinbase notification test', async ({ page, coinbase }) => {
// Steps that cause the bug
});@copilot

@joe10832 joe10832 removed their assignment Oct 8, 2025
Copy link
Member

@joe10832 joe10832 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot // Paste your configuration code here
// Example:
const config = configure()
.withCoinbaseWallet()
// ... other configuration for notification type and passkey
.build();Error encountered during notification detection and passkey sign-in (see linked PR review).
_Originally posted by @joe10832 in #197 (review) { createOnchainTest, configure } from '@coinbase/onchaintestkit';

const test = createOnchainTest(/* your config */);

test('coinbase notification test', async ({ page, coinbase }) => {
// Steps that cause the bug
}); @copilot

@joe10832 joe10832 marked this pull request as ready for review October 8, 2025 11:28
@joe10832 joe10832 merged commit ebf8ed3 into main Oct 8, 2025
Copy link
Member

@joe10832 joe10832 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot import { createOnchainTest, configure } from '@coinbase/onchaintestkit';

const test = createOnchainTest(/* your config */);

test('coinbase notification test', async ({ page, coinbase }) => {
// Steps that cause the bug
});

Copy link
Member

@joe10832 joe10832 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

@joe10832 joe10832 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

@joe10832 joe10832 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2802684@copilot

@joe10832
Copy link
Member

joe10832 commented Oct 8, 2025 via email

Copy link
Member

@joe10832 joe10832 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joe10832
Copy link
Member

joe10832 commented Oct 23, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment