Skip to content

Update listExtensions.m#780

Merged
bendichter merged 1 commit intomainfrom
fix-extension-metadata-retrieval
Jan 13, 2026
Merged

Update listExtensions.m#780
bendichter merged 1 commit intomainfrom
fix-extension-metadata-retrieval

Conversation

@ehennestad
Copy link
Copy Markdown
Collaborator

@ehennestad ehennestad commented Jan 8, 2026

Motivation

Add filter to ignore extensions without a name when retrieving metadata for all published ndx-extensions.

Background:
An empty repository was created for a staged extension in the nwb-extensions organization. This added the following entry to the nwb-extension records:

{
    "ndx_fscv_record": {
        "ref": "ndx-fscv-record",
        "record_url": "https://github.com/nwb-extensions/ndx-fscv-record",
        "last_updated": "2026-01-07T16:55:57Z"
     }
}

This repository uses that records file to create a list of extensions, but the code failed when an extension without required metadata is present. This PR fixes that by ignoring entries in the records file which does not have a name.

How to test the behavior?

Running this command should work

matnwb.extension.listExtensions()

Currently (2026-01-13) this command fails on main due to the state of the ndx-extensions organization.

Checklist

  • Have you ensured the PR description clearly describes the problem and solutions?
  • Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
  • If this PR fixes an issue, is the first line of the PR description fix #XX where XX is the issue number?

@ehennestad ehennestad force-pushed the fix-extension-metadata-retrieval branch from f1a0789 to 33aa74d Compare January 13, 2026 10:47
@ehennestad ehennestad requested a review from bendichter January 13, 2026 11:12
@bendichter bendichter enabled auto-merge (squash) January 13, 2026 15:16
@bendichter bendichter disabled auto-merge January 13, 2026 15:20
@bendichter bendichter merged commit 3230248 into main Jan 13, 2026
24 of 43 checks passed
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.

2 participants