Refactor: Migrate interfaces.js to new registration mechanism#1239
Merged
KernelDeimos merged 9 commits intomainfrom Mar 31, 2025
Merged
Refactor: Migrate interfaces.js to new registration mechanism#1239KernelDeimos merged 9 commits intomainfrom
KernelDeimos merged 9 commits intomainfrom
Conversation
- Created EntityStoreInterfaceService for crud-q interface\n- Created AnalyticsInterfaceService for puter-analytics interface\n- Added InterfacesModule to load these services\n- Removed interfaces.js\n\nCloses #1131 ai: true
ai: true
This comment is flagged as AI-generated, but Claude rate-limited before it could actually make the commit so this commit was made by hand. Well, while I'm writing this commit message I may as well mention that Claud's rate limits are relentless and it has become impossible to use Claude for some purposes as a result. ai: true
… entitystore and analytics - Removed interfaces module\n- Added EntityStoreModule.js to entitystore module\n- Added AnalyticsModule.js to analytics module\n- Updated main index.js to use the new modules directly\n\nai: true ai: true
KernelDeimos
force-pushed
the
cleanup/remove-interfaces-js
branch
from
0d5656e to
d401226
Compare
- Created KVStoreModule.js\n- Created KVStoreInterfaceService.js to register the puter-kvstore interface\n- Updated exports.js to include the new module\n\nai: true
- Removed unnecessary index.js file from kvstore module\n\nai: true
- Removed unnecessary index.js files from analytics and entitystore modules\n\nai: true
...because it actually threw out my previous commit where I already did this.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[ai]\n\nThis PR migrates the file to the new registration mechanism, as described in issue #1131.\n\n### Changes:\n\n- Created for registering the interface\n- Created for registering the interface\n- Added to load these services\n- Removed \n\nThe new implementation follows the pattern used in where interfaces are registered using the event handler.\n\nCloses #1131