Skip to content

fix(factory): route stderr writes through configureOutput - #502

Merged
margaretjgu merged 2 commits into
mainfrom
fix/factory-stderr-configure-output
Aug 6, 2026
Merged

fix(factory): route stderr writes through configureOutput#502
margaretjgu merged 2 commits into
mainfrom
fix/factory-stderr-configure-output

Conversation

@margaretjgu

@margaretjgu margaretjgu commented Aug 4, 2026

Copy link
Copy Markdown
Member

Relates to #455.

Migrates the error-path writes in factory.ts's action handler from process.stderr.write to cmd.configureOutput().writeErr, the channel cmd.error() and existing tests already use. Left stdout writes alone: configureHelpWithSchema already claims writeOut on every command for --help output, so routing regular handler output through it would silently change write semantics. Confirmed by migrating it first and watching status/sanitize/scroll-search tests break, then reverting that part.

Also bumps fast-uri 3.1.4 to 3.1.5 to clear a Trivy-flagged CVE that was failing CI, unrelated to the factory.ts change.

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ COPYPASTE jscpd yes no no 1.03s
✅ REPOSITORY gitleaks yes no no 56.91s
✅ REPOSITORY git_diff yes no no 0.7s
✅ REPOSITORY secretlint yes no no 29.05s
✅ REPOSITORY trivy yes no no 17.87s
✅ TYPESCRIPT eslint 1 0 0 3.77s

Notices

📣 MegaLinter 9.5.0 is out! Discover the new features and security recommendations in the release announcement. (Skip this info by defining SECURITY_SUGGESTIONS: false)

See detailed reports in MegaLinter artifacts
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@margaretjgu
margaretjgu merged commit ea9a7df into main Aug 6, 2026
46 of 49 checks passed
@margaretjgu
margaretjgu deleted the fix/factory-stderr-configure-output branch August 6, 2026 19:22
This was referenced Aug 7, 2026
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