Skip to content

fix: suppress error message in non-development environments - #69

Merged
Harshi-Shah-CS merged 1 commit into
developmentfrom
cl-2079
Nov 5, 2025
Merged

fix: suppress error message in non-development environments#69
Harshi-Shah-CS merged 1 commit into
developmentfrom
cl-2079

Conversation

@Harshi-Shah-CS

Copy link
Copy Markdown
Contributor

fix: suppress error message in non-development environments

CopilotAI left a comment

Copy link
Copy Markdown

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 upgrades Apollo Client from v3.11.8 to v3.14.0 and implements console suppression for Apollo-related errors and warnings in non-development environments. The changes help reduce noise in production logs by filtering out Apollo Client internal warnings while preserving application-level logging.

  • Upgraded @apollo/client dependency to version 3.14.0
  • Added console suppression logic for Apollo-related messages in non-development environments
  • Introduced withDeprecationsDisabled helper method to handle deprecation warnings during watchQuery operations

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
FileDescription
package.jsonUpdates @apollo/client dependency from ^3.11.8 to ^3.14.0
package-lock.jsonReflects the updated @apollo/client version and integrity hash
src/util/apollo-client.tsAdds console suppression logic and exports isNotDevelopment flag
src/util/logs-polling-utilities.tsImplements withDeprecationsDisabled wrapper for watchQuery calls to suppress deprecation warnings
test/unit/util/apollo-client.test.tsAdds comprehensive tests for console suppression behavior

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

Comment threadsrc/util/apollo-client.ts Outdated
Comment threadsrc/util/logs-polling-utilities.ts
dhruvparekh12
dhruvparekh12 previously approved these changes Nov 5, 2025

@dhruvparekh12dhruvparekh12 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.

👍

@Harshi-Shah-CS
Harshi-Shah-CS merged commit 1125814 into developmentNov 5, 2025
6 of 7 checks passed
@Harshi-Shah-CS
Harshi-Shah-CS deleted the cl-2079 branch November 5, 2025 13:30
Sign up for freeto 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.

4 participants

@Harshi-Shah-CS@dhruvparekh12@Chhavi-Mandowara