Skip to content

fix(shared): recheck debug logging flag#77

Merged
re-marked merged 1 commit into
mainfrom
codex/fix-debug-logging-runtime-toggle
May 6, 2026
Merged

fix(shared): recheck debug logging flag#77
re-marked merged 1 commit into
mainfrom
codex/fix-debug-logging-runtime-toggle

Conversation

@re-marked

Copy link
Copy Markdown
Owner

Summary

  • re-read DEBUG each time logForDebugging runs instead of snapshotting at module load
  • support trimmed truthy DEBUG values through the shared env helper
  • add disableDebugLogging() and export it for runtime/test cleanup
  • document the runtime debug logging contract

Closes #70.

Verification

  • pnpm test -- packages/shared/src/debug.test.ts
  • pnpm --filter @yokai-tui/shared typecheck
  • pnpm --filter @yokai-tui/shared lint
  • pnpm --filter @yokai-tui/shared build

Co-Authored-By: Mark <psyhik17@gmail.com>

Co-Authored-By: Codex <noreply@anthropic.com>
@re-marked
re-marked merged commit 3c111b0 into main May 6, 2026
1 check passed
@re-marked
re-marked deleted the codex/fix-debug-logging-runtime-toggle branch May 6, 2026 20:16
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.

E17: logForDebugging only checks process.env.DEBUG at module load

1 participant