Skip to content

Commit 7fb0c46

Browse files
committed
fix: move @framers/sql-storage-adapter from peer to dependencies
It was a peer marked optional, but core/storage/SqlStorageAdapter.ts imports it eagerly at module load. Consumers got 'Cannot find package @framers/sql-storage-adapter' on bare `import { agent } from '@framers/agentos'` unless they manually installed it. Either the runtime should lazy-load it (so the peer-optional contract holds) or it belongs in dependencies. Making it a hard dep is the simpler fix — sql storage is on the default code path through ConversationManager and most consumers want it.
1 parent 5c1e68e commit 7fb0c46

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

package.json

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -675,6 +675,7 @@
675675
"vitest": "^1.6.0"
676676
},
677677
"dependencies": {
678+
"@framers/sql-storage-adapter": "^0.6.5",
678679
"@opentelemetry/api": "^1.9.0",
679680
"@opentelemetry/api-logs": "^0.201.1",
680681
"ajv": "^8.17.1",
@@ -696,7 +697,6 @@
696697
"zod-to-json-schema": "^3.25.2"
697698
},
698699
"peerDependencies": {
699-
"@framers/sql-storage-adapter": "^0.6.5",
700700
"graphology": ">=0.25.0",
701701
"graphology-communities-louvain": ">=2.0.0",
702702
"graphology-types": ">=0.24.0",
@@ -707,9 +707,6 @@
707707
"typescript": ">=5.0.0"
708708
},
709709
"peerDependenciesMeta": {
710-
"@framers/sql-storage-adapter": {
711-
"optional": true
712-
},
713710
"graphology": {
714711
"optional": true
715712
},

0 commit comments

Comments
 (0)