Skip to content

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

Open
margaretjgu wants to merge 2 commits into
mainfrom
fix/factory-stderr-configure-output
Open

fix(factory): route stderr writes through configureOutput#502
margaretjgu wants to merge 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

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.

1 participant