**refactor(core): remove redundant dimensions property in BrainyData initialization**

- **Core**:
  - Removed the `dimensions` property from `BrainyData` initialization in `server.ts` as it was unused and redundant.
  - Simplified code structure and reduced potential confusion by avoiding unnecessary arguments in object initialization.

**Purpose**: Improve code clarity and maintainability by removing unused properties from the `BrainyData` setup, aligning with standardized dimension handling approaches.
This commit is contained in:
David Snelling 2025-07-25 13:53:19 -07:00
parent cf64900ef3
commit f2cc57ee3a
6 changed files with 10 additions and 14 deletions

View file

@ -1,16 +1,16 @@
{ {
"name": "@soulcraft/brainy-cli", "name": "@soulcraft/brainy-cli",
"version": "0.18.0", "version": "0.19.0",
"lockfileVersion": 3, "lockfileVersion": 3,
"requires": true, "requires": true,
"packages": { "packages": {
"": { "": {
"name": "@soulcraft/brainy-cli", "name": "@soulcraft/brainy-cli",
"version": "0.18.0", "version": "0.19.0",
"hasInstallScript": true, "hasInstallScript": true,
"license": "MIT", "license": "MIT",
"dependencies": { "dependencies": {
"@soulcraft/brainy": "^0.17.0", "@soulcraft/brainy": "^0.24.0",
"commander": "^14.0.0", "commander": "^14.0.0",
"omelette": "^0.4.17" "omelette": "^0.4.17"
}, },
@ -2087,9 +2087,9 @@
} }
}, },
"node_modules/@soulcraft/brainy": { "node_modules/@soulcraft/brainy": {
"version": "0.17.0", "version": "0.24.0",
"resolved": "https://registry.npmjs.org/@soulcraft/brainy/-/brainy-0.17.0.tgz", "resolved": "https://registry.npmjs.org/@soulcraft/brainy/-/brainy-0.24.0.tgz",
"integrity": "sha512-QoTiNnyK7fBuNRDsGOcP89UrFfwxoqkS6lRh6VpIXAWVTtxUBy256PWxQAqtZgpslOZRAA58GlXIQf3IYHiM5g==", "integrity": "sha512-9ZkzNQpqVx4N17i7KPU2pDufrmiXPO/pAonftpwWuHL8sIGI8xJhWzxUIG063jBNOzmMItF2/UEggJS1SpozgA==",
"license": "MIT", "license": "MIT",
"dependencies": { "dependencies": {
"@aws-sdk/client-s3": "^3.540.0", "@aws-sdk/client-s3": "^3.540.0",

View file

@ -1,6 +1,6 @@
{ {
"name": "@soulcraft/brainy-cli", "name": "@soulcraft/brainy-cli",
"version": "0.18.0", "version": "0.19.0",
"description": "Command-line interface for the Brainy vector graph database", "description": "Command-line interface for the Brainy vector graph database",
"type": "module", "type": "module",
"bin": { "bin": {
@ -46,7 +46,7 @@
"url": "git+https://github.com/soulcraft-research/brainy.git" "url": "git+https://github.com/soulcraft-research/brainy.git"
}, },
"dependencies": { "dependencies": {
"@soulcraft/brainy": "^0.17.0", "@soulcraft/brainy": "^0.24.0",
"commander": "^14.0.0", "commander": "^14.0.0",
"omelette": "^0.4.17" "omelette": "^0.4.17"
}, },

View file

@ -118,7 +118,6 @@ async function initializeBrainy() {
// Initialize the storage adapter before using it // Initialize the storage adapter before using it
await storage.init(); await storage.init();
brainyInstance = new BrainyData({ brainyInstance = new BrainyData({
dimensions: 384, // Default dimensions, can be overridden
storageAdapter: storage, storageAdapter: storage,
distanceFunction: cosineDistance distanceFunction: cosineDistance
}); });
@ -128,7 +127,6 @@ async function initializeBrainy() {
if (!brainyInstance) { if (!brainyInstance) {
const storage = await createStorage(storageOptions); const storage = await createStorage(storageOptions);
brainyInstance = new BrainyData({ brainyInstance = new BrainyData({
dimensions: 384, // Default dimensions, can be overridden
storageAdapter: storage, storageAdapter: storage,
distanceFunction: cosineDistance distanceFunction: cosineDistance
}); });

File diff suppressed because one or more lines are too long

View file

@ -53,7 +53,7 @@
"url": "git+https://github.com/soulcraft-research/brainy.git" "url": "git+https://github.com/soulcraft-research/brainy.git"
}, },
"dependencies": { "dependencies": {
"@soulcraft/brainy": "^0.15.0", "@soulcraft/brainy": "^0.24.0",
"express": "^4.18.2", "express": "^4.18.2",
"cors": "^2.8.5", "cors": "^2.8.5",
"helmet": "^7.1.0", "helmet": "^7.1.0",

View file

@ -138,7 +138,6 @@ async function initializeBrainy(): Promise<BrainyData> {
await storage.init() await storage.init()
brainyInstance = new BrainyData({ brainyInstance = new BrainyData({
dimensions: 384, // Default dimensions, can be overridden
storageAdapter: storage, storageAdapter: storage,
distanceFunction: cosineDistance distanceFunction: cosineDistance
}) })
@ -150,7 +149,6 @@ async function initializeBrainy(): Promise<BrainyData> {
const storage = await createStorage(storageOptions) const storage = await createStorage(storageOptions)
brainyInstance = new BrainyData({ brainyInstance = new BrainyData({
dimensions: 384, // Default dimensions, can be overridden
storageAdapter: storage, storageAdapter: storage,
distanceFunction: cosineDistance distanceFunction: cosineDistance
}) })