fix(marketplace-auto-review): silence loader Pino logs in validate-schema#1644
Conversation
…hema parseWorkflow logs warnings via Pino at default stdout. validate-schema's stdout is consumed by the decide node via variable substitution into a TypeScript expression, so any non-JSON log line breaks the substituted script. Set log level to fatal before parseWorkflow runs to suppress warnings cleanly. Long-term, the platform logger should write to stderr instead of stdout for CLI tool ergonomics, but that's a broader cross-cutting change.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe marketplace schema validation script now suppresses loader output by importing and configuring the logger to ChangesLogger Configuration
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…hema (coleam00#1644) parseWorkflow logs warnings via Pino at default stdout. validate-schema's stdout is consumed by the decide node via variable substitution into a TypeScript expression, so any non-JSON log line breaks the substituted script. Set log level to fatal before parseWorkflow runs to suppress warnings cleanly. Long-term, the platform logger should write to stderr instead of stdout for CLI tool ergonomics, but that's a broader cross-cutting change.
validate-schema stdout was being polluted by parseWorkflow's Pino warnings (workflow_missing_description, etc.) which then broke the decide node's substitution. Set log level to fatal before any parseWorkflow call. Fifth iteration.
Summary by CodeRabbit