Skip to content

Conversation

@Justkant
Copy link
Collaborator

This pull request introduces a new debug option to the simulator, allowing conditional logging of transport messages for improved debugging flexibility. The main changes add a debug property to the SimulatorProfile type and update the transport layer to respect this flag before logging messages.

Enhancements for debugging:

  • Added an optional debug property to the SimulatorProfile type in types.ts, enabling toggling of debug logging.
  • Updated the getSimulatorTransport function in transport.ts to conditionally log transport messages (wallet -> app and app -> wallet) only when the debug flag is set in the profile. [1] [2]

Documentation and release notes:

  • Added a changeset file documenting the new debug option and its effect on conditional logging in the simulator.

@Justkant Justkant requested review from a team November 19, 2025 17:33
@Justkant Justkant self-assigned this Nov 19, 2025
@changeset-bot
Copy link

changeset-bot bot commented Nov 19, 2025

🦋 Changeset detected

Latest commit: a83fda2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@ledgerhq/wallet-api-simulator Minor
@ledgerhq/wallet-api-tools Patch
@ledgerhq/client-nextjs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Nov 19, 2025

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

Project Deployment Preview Comments Updated (UTC)
wallet-api-wallet-api-tools Ready Ready Preview Comment Nov 19, 2025 5:33pm

@Justkant Justkant merged commit e6054db into main Nov 19, 2025
11 checks passed
@Justkant Justkant deleted the feat/simulator-debug branch November 19, 2025 17:41
@github-actions github-actions bot mentioned this pull request Nov 19, 2025
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