Skip to content

fix(core): downgrade baseline resolution logs to debug [NT-3840] - #407

Merged
Felipe Mamud (fmamud) merged 1 commit into
mainfrom
fix/optimized-entry-baseline-log-level-nt-3840
Aug 4, 2026
Merged

fix(core): downgrade baseline resolution logs to debug [NT-3840]#407
Felipe Mamud (fmamud) merged 1 commit into
mainfrom
fix/optimized-entry-baseline-log-level-nt-3840

Conversation

@fmamud

@fmamud Felipe Mamud (fmamud) commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • OptimizedEntryResolver.resolveWithContext was logging the baseline path — no selected optimizations, entry not optimized, and no matching optimization entry — at warn. That path is the majority case in any real personalization setup, not an anomaly. In React Native dev, LogBox promotes warn into a red-boxed modal, so it read as a fatal error on every screen mount.
  • Downgraded those three log calls to debug. Real anomalies (broken variant link, misconfigured entry) still log at warn.
  • Updated the matching tests to assert the new debug level and to verify warn is not called for the same messages.

Ticket: NT-3840

Test plan

  • pnpm test:unit — 343 core-sdk tests pass.
  • pnpm lint — clean.
  • pnpm typecheck (core-sdk) — clean.
  • RN travel-guide app: no warn-level log for baseline resolution when opening a personalized destination without matching interests. Variant selection still resolves normally.

🤖 Generated with Claude Code

The baseline path in `OptimizedEntryResolver.resolveWithContext` — no
selected optimizations, entry not optimized, and no matching optimization
entry for the current selections — is the majority case in any real
personalization setup, not an anomaly. Logging it at `warn` promoted
normal renders into red-boxed LogBox modals in React Native dev.

Downgrade those three log calls to `debug`. Real anomalies (broken
variant link, misconfigured entry) still log at `warn`.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@fmamud
Felipe Mamud (fmamud) merged commit ef3c543 into main Aug 4, 2026
46 checks passed
@fmamud
Felipe Mamud (fmamud) deleted the fix/optimized-entry-baseline-log-level-nt-3840 branch August 4, 2026 09:35
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.

3 participants