Skip to content

Suppress logs when running non-interactive mode#115

Merged
atilafassina merged 7 commits into
mainfrom
get-db/logs
Jan 8, 2026
Merged

Suppress logs when running non-interactive mode#115
atilafassina merged 7 commits into
mainfrom
get-db/logs

Conversation

@atilafassina

@atilafassina atilafassina commented Jan 8, 2026

Copy link
Copy Markdown
Collaborator

logs are too verbose when running non-interactive.
This reduces the information to the bare essentials. Offering better UX and perceived-performance.

detailed description

@atilafassina atilafassina requested a review from Copilot January 8, 2026 12:34
@atilafassina atilafassina self-assigned this Jan 8, 2026
@atilafassina atilafassina added the enhancement New feature or request label Jan 8, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR suppresses verbose logging in non-interactive mode to improve UX and perceived performance. The changes focus on removing intermediate log messages when running with the --yes flag, while preserving essential output.

Key changes:

  • Streamlined output for non-interactive mode by removing intermediate log messages
  • Consolidated final output to show only the connection string
  • Improved error message structure by using a dedicated error object

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
packages/get-db/src/lib/texts.ts Removed unused message functions and added structured error object for referrer validation
packages/get-db/src/lib/instant-postgres.ts Removed verbose logging statements for connection strings and env file updates
packages/get-db/src/cli.ts Conditionally suppressed intro art and non-interactive message; simplified final output
.changeset/four-guests-dig.md Added changeset documentation for the log suppression feature

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/get-db/src/cli.ts Outdated
atilafassina and others added 2 commits January 8, 2026 14:22
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@atilafassina atilafassina requested a review from Copilot January 8, 2026 13:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/get-db/src/cli.ts
@atilafassina atilafassina marked this pull request as ready for review January 8, 2026 13:39
@atilafassina atilafassina enabled auto-merge (squash) January 8, 2026 13:39
Comment thread packages/get-db/src/lib/instant-postgres.ts
Comment thread packages/get-db/src/lib/texts.ts Outdated

Copilot AI commented Jan 8, 2026

Copy link
Copy Markdown
Contributor

@atilafassina I've opened a new pull request, #117, to work on those changes. Once the pull request is ready, I'll request review from you.

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: atilafassina <2382552+atilafassina@users.noreply.github.com>

Copilot AI commented Jan 8, 2026

Copy link
Copy Markdown
Contributor

@atilafassina I've opened a new pull request, #118, to work on those changes. Once the pull request is ready, I'll request review from you.

@rodneyshibu rodneyshibu self-requested a review January 8, 2026 15:17
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: atilafassina <2382552+atilafassina@users.noreply.github.com>
@atilafassina atilafassina merged commit e215940 into main Jan 8, 2026
5 checks passed
@atilafassina atilafassina deleted the get-db/logs branch January 8, 2026 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants