feat(ai-sdk)!: drop ORM-backed store subpaths, decouple from @rudderjs/orm (#38)#44
Merged
Merged
Conversation
…derjs/orm (#38) Third child of the @rudderjs decoupling epic (#35), the gemstack side of a coordinated two-repo move (rudder side: rudderjs/rudder#1452). The subpaths @gemstack/ai-sdk/{conversation-orm,memory-orm,budget-orm, memory-embedding} imported @rudderjs/orm's Model, coupling the agnostic engine to the Rudder ORM. They have moved to the Rudder binding @rudderjs/ai under the same subpath names. memory-embedding moved too because its EmbeddingUserMemory hard-depends on OrmUserMemory. - Removed the 4 src dirs + their 4 tests. - Removed the 4 entries from package.json "exports". - Dropped @rudderjs/orm from peerDependencies + peerDependenciesMeta + devDependencies (no @rudderjs/orm import remains in src). - README: rewrote Status + Subpath exports to point ORM stores at @rudderjs/ai and lead with the neutral contracts (ConversationStore / UserMemory / BudgetStorage) + the CacheAdapter/StorageAdapter seams. - clean script also clears dist-test. Breaking (0.x minor): update @gemstack/ai-sdk/<orm-subpath> imports to @rudderjs/ai/<orm-subpath>. The neutral contracts remain exported here. Build + repo typecheck green; 911 tests pass (was 982; 71 moved to rudder).
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.
Closes #38. Third child of the
@rudderjsdecoupling epic #35. Gemstack side of a coordinated two-repo move.What
The subpaths
@gemstack/ai-sdk/{conversation-orm, memory-orm, budget-orm, memory-embedding}imported@rudderjs/orm'sModel- coupling the agnostic engine to the Rudder ORM. They've moved to the Rudder binding@rudderjs/aiunder the same subpath names.memory-embeddingmoved too (itsEmbeddingUserMemoryhard-depends onOrmUserMemory).exportsentries frompackage.json.@rudderjs/ormfrom peer + peerMeta + dev deps (no@rudderjs/ormimport remains insrc).@rudderjs/ai; lead with the neutral contracts +CacheAdapter/StorageAdapterseams.Breaking (0.x -> minor)
Update
@gemstack/ai-sdk/<orm-subpath>imports to@rudderjs/ai/<orm-subpath>. The neutralConversationStore/UserMemory/BudgetStoragecontracts remain exported from@gemstack/ai-sdk; non-Rudder apps implement them against their own persistence or use the in-memory defaults.Verification
Build + repo-wide typecheck green; 911 tests pass (was 982; 71 moved to
@rudderjs/ai). Minor changeset included.After this lands, the only remaining
@rudderjs/*couplings in@gemstack/ai-sdkare the/serverprovider (#39) and the doctor/CLI integrations (#40).