Skip to content

fix: Move config to wiretap-api and launcher related api to wiretap-launcher#55

Merged
skymansandy merged 4 commits into
mainfrom
fix/noopconfig
May 23, 2026
Merged

fix: Move config to wiretap-api and launcher related api to wiretap-launcher#55
skymansandy merged 4 commits into
mainfrom
fix/noopconfig

Conversation

@skymansandy
Copy link
Copy Markdown
Owner

@skymansandy skymansandy commented May 23, 2026

Fixes #53 — config classes and launcher helpers are now available in noop artifacts, allowing consumers to use install(WiretapKtorHttpPlugin) { … } and call enableWiretapLauncher() from commonMain without expect/actual workarounds when switching between real and noop.

What changed

New wiretap-api module — contains:

WiretapHttpConfig
LogRetention
HeaderAction
WiretapSseConfig
WiretapWsConfig
ExperimentalWiretapSseApi

Shared across real and noop artifacts via api(...).

New wiretap-launcher / wiretap-launcher-noop modules — launcher API extracted from wiretap-core.
Noop now exposes:

launchWiretapConsole()
enableWiretapLauncher()

Test plan

  • Build all modules: ./gradlew build
  • Dev variant: ./gradlew :composeApp:build
  • Prod (noop) variant: ./gradlew :composeApp:build -PwiretapVariant=prod
  • Android debug + release assemble
  • iOS framework build for both variants
  • Shake-to-launch still works in debug

@skymansandy skymansandy marked this pull request as ready for review May 23, 2026 13:35
@skymansandy skymansandy merged commit 76940de into main May 23, 2026
6 checks passed
@skymansandy skymansandy deleted the fix/noopconfig branch May 23, 2026 13:52
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.

[Bug] noop artifact is missing configuration classes and helper functions, breaking compilation in shared KMP modules Description:

1 participant