Skip to content

Conversation

himanshusinghs
Copy link
Collaborator

Proposed changes

Adds isError annotation for handled errors that were missed.

Checklist

@himanshusinghs himanshusinghs requested a review from a team as a code owner October 8, 2025 10:54
@Copilot Copilot AI review requested due to automatic review settings October 8, 2025 10:54
@himanshusinghs himanshusinghs added the no-title-validation Add this label to disable the title check for this PR. label Oct 8, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

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 adds missing isError: true annotations to error response objects in MongoDB tools to ensure proper error handling consistency across the codebase.

Key changes:

  • Adds isError: true annotation to error responses in three MongoDB tools
  • Ensures consistent error handling patterns across collection-related operations

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/tools/mongodb/update/renameCollection.ts Added isError: true to "NamespaceNotFound" and "NamespaceExists" error cases
src/tools/mongodb/read/collectionIndexes.ts Added isError: true to collection not found error case
src/tools/mongodb/metadata/collectionStorageSize.ts Added isError: true to collection not found error case

@coveralls
Copy link
Collaborator

coveralls commented Oct 8, 2025

Pull Request Test Coverage Report for Build 18342327970

Details

  • 4 of 4 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.007%) to 82.531%

Totals Coverage Status
Change from base Build 18341606912: 0.007%
Covered Lines: 5311
Relevant Lines: 6323

💛 - Coveralls

@himanshusinghs himanshusinghs merged commit 46bf661 into main Oct 8, 2025
25 of 26 checks passed
@himanshusinghs himanshusinghs deleted the chore/consistent-error-handling branch October 8, 2025 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-title-validation Add this label to disable the title check for this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants