Skip to content

Fix default namespace lookup, bump dependencies#22

Merged
austin-denoble merged 4 commits intomainfrom
adenoble/fix-namespace-lookup
Jan 5, 2026
Merged

Fix default namespace lookup, bump dependencies#22
austin-denoble merged 4 commits intomainfrom
adenoble/fix-namespace-lookup

Conversation

@austin-denoble
Copy link
Copy Markdown
Contributor

@austin-denoble austin-denoble commented Jan 5, 2026

Problem

Default namespace representation shifted in describe_index_stats for 2025-10. We need to check for both possibilities in the sample app.

There were also a number of NPM high severity vulnerabilities that needed to be bumped:

  • vitest
  • @vitest/coverage
  • nodemon

Solution

  • Check for either "" or "__default__" when looking up namespaces.
  • Run npm audit to fix high severity vulnerabilities: vitest, @vitest/coverage-v8, and nodemon.
    • Bumped to more recent version of node for this (v20).

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Infrastructure change (CI configs, etc)
  • Non-code change (docs, etc)
  • None of the above: (explain here)

Test Plan

CI - testing coverage

@austin-denoble austin-denoble merged commit 999f04d into main Jan 5, 2026
6 checks passed
This was referenced Jan 5, 2026
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