Skip to content

refactor: move standard library to yamllibrary #1573

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
maxy-shpfy wants to merge 1 commit into
base: 12-17-refactor_move_favorite_libs_to_indexdb_library
Choose a base branch
from
Draft

refactor: move standard library to yamllibrary #1573

maxy-shpfy wants to merge 1 commit into from

Conversation

@maxy-shpfy
Copy link
Collaborator

@maxy-shpfy maxy-shpfy commented Dec 17, 2025 •
edited
Loading

Description

Refactored component library management to use a registry-based approach. This PR:

  • Replaced the monolithic component library with a registry of library objects
  • Implemented a YamlFileLibrary class for standard components
  • Removed the deprecated fetchAndStoreComponentLibrary function
  • Updated FavoriteComponentToggle to use the new library structure
  • Simplified component lookup by using the library's hasComponent method
  • Removed unused EmptyState component from FlowSidebar
  • Updated GraphComponents to use the new library structure

Type of Change

  • Bug fix
  • New feature
  • Improvement
  • Cleanup/Refactor
  • Breaking change
  • Documentation update

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Test Instructions

  1. Verify that the component library loads correctly in the sidebar
  2. Check that favorite component toggling works as expected
  3. Ensure component search functionality works properly

This was referenced Dec 17, 2025
Merged
Merged
Merged
Merged
Merged
Merged
Draft
@maxy-shpfy Graphite App
Copy link
Collaborator Author

maxy-shpfy commented Dec 17, 2025 •
edited
Loading

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Dec 17, 2025
Draft
Open
Draft
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_standard_library_to_yamllibrary branch from 2b9306f to 512cabc Compare December 17, 2025 19:05
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_favorite_libs_to_indexdb_library branch from 8fde55e to 0f7cb8c Compare December 17, 2025 19:05
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_standard_library_to_yamllibrary branch 2 times, most recently from e0e86fa to c1bd3e3 Compare December 17, 2025 19:21
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_favorite_libs_to_indexdb_library branch from 64430f6 to 0566d1b Compare December 17, 2025 19:21
@maxy-shpfy maxy-shpfy mentioned this pull request Dec 18, 2025
Draft
4 tasks
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_favorite_libs_to_indexdb_library branch from 0566d1b to 472a8dc Compare December 18, 2025 16:59
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_standard_library_to_yamllibrary branch from c1bd3e3 to 2704b28 Compare December 18, 2025 16:59
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_favorite_libs_to_indexdb_library branch from 472a8dc to 03340dc Compare December 18, 2025 17:56
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_standard_library_to_yamllibrary branch from 2704b28 to f5fa742 Compare December 18, 2025 17:57
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_favorite_libs_to_indexdb_library branch from 03340dc to 8cd6373 Compare December 18, 2025 18:03
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_standard_library_to_yamllibrary branch from f5fa742 to 0024d70 Compare December 18, 2025 18:03
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_favorite_libs_to_indexdb_library branch from 8cd6373 to 568e99b Compare December 18, 2025 18:10
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_standard_library_to_yamllibrary branch from 0024d70 to d1617a3 Compare December 18, 2025 18:11
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_favorite_libs_to_indexdb_library branch from 568e99b to f3a8c70 Compare December 18, 2025 18:29
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_standard_library_to_yamllibrary branch from d1617a3 to 7f7bcf0 Compare December 18, 2025 18:30
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_favorite_libs_to_indexdb_library branch from f3a8c70 to 2b465c1 Compare December 18, 2025 18:35
@maxy-shpfy maxy-shpfy force-pushed the 12-17-refactor_move_standard_library_to_yamllibrary branch from 7f7bcf0 to f65f5c4 Compare December 18, 2025 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@maxy-shpfy