Skip to content

Add .quiet() method to suppress console output #137

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
konard wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add .quiet() method to suppress console output #137

konard wants to merge 3 commits into from

Conversation

@konard
Copy link
Member

@konard konard commented Nov 30, 2025 •
edited
Loading

Summary

This PR implements the .quiet() method on ProcessRunner, providing functionality similar to zx's quiet() method. This allows users to suppress console output while still capturing command results programmatically.

Changes

Core Implementation

  • Added .quiet() method to ProcessRunner class in src/$.mjs:4140-4144
    • Sets options.mirror = false to disable console output
    • Returns this for method chaining
    • Works seamlessly with existing await, pipe, and event patterns

Testing

  • Created comprehensive test suite in tests/quiet-method.test.mjs
    • Tests output suppression for stdout and stderr
    • Verifies method chaining capabilities
    • Confirms result capture still works
    • Validates compatibility with normal (non-quiet) mode

Examples

  • Added demonstration script in examples/quiet-method-demo.mjs
    • Shows usage comparison with and without .quiet()
    • Demonstrates the use case from the issue
    • Includes practical examples with JSON processing

Use Case (from Issue #136)

// Before (would fail with: .quiet is not a function)
await $`gh api gists/${gistId} --jq '{owner: .owner.login, files: .files, history: .history}'`.quiet();

// After (works perfectly!)
await $`gh api gists/${gistId} --jq '{owner: .owner.login, files: .files, history: .history}'`.quiet();

Test Results

All tests pass successfully:

  • ✅ 617 passing tests
  • ✅ 5 skipped tests (expected)
  • ⚠️ 2 pre-existing failures (unrelated to this change)
  • ✅ New .quiet() tests: 6/6 passing

Breaking Changes

None. This is a purely additive feature that doesn't modify any existing functionality.

Related

Fixes #136

🤖 Generated with Claude Code

@konard
Initial commit with task details for issue #136
4b016e1 
Adding CLAUDE.md with task information for AI processing.
This file will be removed when the task is complete.

Issue: #136
@konard konard self-assigned this Nov 30, 2025
@konard @claude
Add .quiet() method to suppress console output
6fbc051 
This commit implements the .quiet() method on ProcessRunner, similar
to zx's quiet() functionality. When called, it disables console output
(mirroring) while still capturing stdout/stderr for programmatic use.

Implementation details:
- Added quiet() method that sets options.mirror = false
- Returns this for method chaining
- Works with both stdout and stderr suppression
- Compatible with existing pipeline and await patterns

Testing:
- Created comprehensive test suite in tests/quiet-method.test.mjs
- Added usage example in examples/quiet-method-demo.mjs
- All existing tests pass (617 pass, 5 skip, 2 pre-existing fails)

This enables the use case from issue #136:
await $\`gh api gists/\${gistId} --jq '...'\`.quiet();

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@konard konard changed the title [WIP] Support .quite() function like in zx. Add .quiet() method to suppress console output Nov 30, 2025
@konard konard marked this pull request as ready for review November 30, 2025 18:26
@konard konard mentioned this pull request Nov 30, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@konard konard

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Support .quite() function like in zx.

2 participants

@konard