Skip to content

Conversation

@bbopen
Copy link
Owner

@bbopen bbopen commented Jan 15, 2026

Summary

  • auto-register Arrow decoder in Node bridges for frictionless Arrow defaults
  • expose helper for manual registration + tests for auto-register behavior
  • update living app + docs to rely on auto-registration

Testing

  • npm test -- test/runtime_codec.test.ts

Notes

@coderabbitai
Copy link

coderabbitai bot commented Jan 15, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ed1cf275e6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 551 to +554
'arrowAvailable': arrow_available(),
'scipyAvailable': module_available('scipy.sparse'),
'torchAvailable': module_available('torch'),
'sklearnAvailable': module_available('sklearn.base'),

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Avoid importing heavy SciPy/Sklearn during meta

module_available() uses importlib.util.find_spec, which imports the parent package for dotted names; calling it with 'scipy.sparse' and 'sklearn.base' therefore imports SciPy/Sklearn on every meta request (bridge startup). These imports are heavy and can take seconds or fail when native dependencies are missing, delaying or disrupting startup even when those codecs aren’t used. Consider checking only the top-level package (e.g., 'scipy', 'sklearn') or deferring submodule imports until actually needed.

Useful? React with 👍 / 👎.

@bbopen bbopen self-assigned this Jan 15, 2026
@bbopen bbopen added enhancement New feature or request area:codec Area: codecs and serialization priority:p2 Priority P2 (medium) labels Jan 15, 2026
@bbopen bbopen deleted the branch feat/codec-envelope-validation January 15, 2026 21:54
@bbopen bbopen closed this Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:codec Area: codecs and serialization enhancement New feature or request priority:p2 Priority P2 (medium)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants