Skip to content

feat: export CactusImage to make it available for consumers#12

Open
aryaminus wants to merge 2 commits intocactus-compute:mainfrom
aryaminus:feat/export-cactus-image
Open

feat: export CactusImage to make it available for consumers#12
aryaminus wants to merge 2 commits intocactus-compute:mainfrom
aryaminus:feat/export-cactus-image

Conversation

@aryaminus
Copy link

@aryaminus aryaminus commented Nov 27, 2025

Summary

  • Export CactusImage from src/index.tsx.

Motivation

CactusImage was previously internal or not exposed via the main entry point. Exporting it allows the example app and other consumers of the library to import and use CactusImage directly.

Copilot AI review requested due to automatic review settings November 27, 2025 21:26
Copy link

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 exports the CactusImage class from the library's main entry point (src/index.tsx), making it publicly available for consumers. Previously, CactusImage was only exported from the internal src/native/ module but not exposed through the top-level API.

Key Changes:

  • Added export statement for CactusImage from ./native/CactusImage in the main index file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -1,5 +1,7 @@
// Classes
export { CactusLM } from './classes/CactusLM';
export { CactusImage } from './native/CactusImage';
Copy link

Copilot AI Nov 27, 2025

Choose a reason for hiding this comment

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

The CactusImage export is placed under the "// Classes" comment, but it's actually a native module from the ./native/ directory. This should either be moved to a "// Native" section (similar to how there are separate sections for Classes, Hooks, Types, and Config) or placed under a more generic section name. This helps maintain clear organization of different module types in the public API.

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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