Skip to content

Conversation

@Seungwoo321
Copy link
Member

@Seungwoo321 Seungwoo321 commented Aug 14, 2025

Summary

  • Export all composables from main index.ts to enable custom renderer development
  • Resolves import errors in plotly-renderer and lazy-table-renderer packages
  • No circular dependency issues detected

Problem

External packages (@vue-pivottable/plotly-renderer, @vue-pivottable/lazy-table-renderer) were unable to import useProvidePivotData and providePivotData functions, causing build errors as reported in issue #270.

Solution

Added export * from './composables' to src/index.ts to expose all composables for external use.

Analysis

Conducted thorough circular dependency analysis:

  • ✅ Helper modules have no external dependencies
  • ✅ Composables only import from helper/types/utils (no component imports)
  • ✅ Components import composables in one direction only
  • ✅ Build completes successfully without warnings

Test plan

  • Build main package successfully
  • Verify no circular dependency warnings
  • Test plotly-renderer package can import required functions
  • Test lazy-table-renderer package can import required functions

🤖 Generated with Claude Code

- Export all composables to enable custom renderer development
- Allows plotly-renderer and lazy-table-renderer to import required functions
- No circular dependency issues detected after analysis
- Maintains backward compatibility

Resolves import errors for external packages that need useProvidePivotData and providePivotData

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@Seungwoo321 Seungwoo321 merged commit 73d42f4 into develop Aug 14, 2025
2 checks passed
Seungwoo321 added a commit that referenced this pull request Aug 14, 2025
- Add changeset documentation for PR #295
- Tracks the composables export feature for release notes

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.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