Skip to content

fix: field deserialize documents VSCODE-772#1303

Draft
tculig wants to merge 6 commits intomongodb-js:mainfrom
tculig:VSCODE-772-field-deserialize-documents
Draft

fix: field deserialize documents VSCODE-772#1303
tculig wants to merge 6 commits intomongodb-js:mainfrom
tculig:VSCODE-772-field-deserialize-documents

Conversation

@tculig
Copy link
Copy Markdown
Contributor

@tculig tculig commented Mar 31, 2026

Description

https://jira.mongodb.org/browse/VSCODE-772

To test insert some bad values:

use('test');
db.badDocs.insertMany([
  {
    name: "valid doc",
    count: Long("42"),
    ts: new Date(),
  },
  {
    name: "invalid doc",
    count: Long("42"),        // valid
    badLong: { $numberLong: "pineapple" },   // invalid
    badDate: { $date: "garbage" },           // invalid
    nested: {
      ok: Long("99"),
      broken: { $numberDecimal: "not-a-number" },
    }
  }
]);

Confirm that the view looks like this:
image

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

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