@sidurijs/api 1.0.2 → 1.0.4

package/.turbo/turbo-build.log

@@ -1,4 +1,4 @@
 
-> @sidurijs/api@1.0.2 build /home/zagin/Projects/vxnus-studio/projects/siduri-x/apps/api
+> @sidurijs/api@1.0.4 build /home/zagin/Projects/vxnus-studio/projects/siduri-x/apps/api
 > tsc
 

package/.turbo/turbo-test.log

@@ -1,9 +1,13 @@
 
-> @sidurijs/api@1.0.1 test /home/zagin/Projects/vxnus-studio/projects/siduri-x/apps/api
+> @sidurijs/api@1.0.4 test /home/zagin/Projects/vxnus-studio/projects/siduri-x/apps/api
 > jest --config jest.config.json
 
-PASS src/context-mapper.test.ts (6.742 s)
-PASS src/runtime.test.ts (7.635 s)
+PASS src/context-mapper.test.ts (14.72 s)
+PASS src/knowledge.test.ts (18.588 s)
+PASS src/index.test.ts
+PASS src/t6-security.test.ts (17.383 s)
+PASS src/teach-mode.test.ts (18.646 s)
+PASS src/runtime.test.ts (17.892 s)
   ● Console
 
     console.error
@@ -25,20 +29,16 @@ PASS src/runtime.test.ts (7.635 s)
       at PerceptionPipeline.execute (../../packages/core/dist/perception-pipeline.js:21:34)
       at Object.<anonymous> (src/runtime.test.ts:74:22)
 
-PASS src/t7-release.test.ts
-PASS src/b0-b6.test.ts
-PASS src/index.test.ts
 PASS src/auth.test.ts
 PASS src/smoke.test.ts
-PASS src/knowledge.test.ts (9.111 s)
 PASS src/boot.test.ts
-PASS src/t4-gating.test.ts (9.041 s)
-PASS src/teach-mode.test.ts (9.014 s)
-PASS src/t6-security.test.ts (9.098 s)
-PASS src/t5-experience.test.ts (9.227 s)
+PASS src/b0-b6.test.ts
+PASS src/t7-release.test.ts
+PASS src/t4-gating.test.ts (18.564 s)
+PASS src/t5-experience.test.ts (18.36 s)
 
 Test Suites: 13 passed, 13 total
 Tests:       95 passed, 95 total
 Snapshots:   0 total
-Time:        10.494 s, estimated 14 s
+Time:        22.109 s, estimated 25 s
 Ran all test suites.

package/dist/app.js

@@ -150,8 +150,19 @@ function createApp(runtimes = new Map()) {
             const parsed = await (0, self_1.compilePersonaDocument)(content, {
                 brain: runtime?.brain,
                 companionId,
-                fallbackToParser: true,
+                fallbackToParser: false,
             });
+            if (!parsed.isValid) {
+                return res.json({
+                    detected: true,
+                    filename: node_path_1.default.basename(matchedFilePath),
+                    path: matchedFilePath,
+                    content,
+                    parsed,
+                    error: parsed.errors?.[0] || 'Brain compilation failed',
+                    alreadyInstalled: false,
+                });
+            }
             let alreadyInstalled = false;
             const repo = new self_1.SqliteSelfRepository({ dbPath: process.env.STORAGE_PATH || process.env.SQLITE_DB_PATH || 'siduri.sqlite' });
             try {
@@ -191,8 +202,14 @@ function createApp(runtimes = new Map()) {
             const parsed = await (0, self_1.compilePersonaDocument)(content, {
                 brain: runtime?.brain,
                 companionId: targetId,
-                fallbackToParser: true,
+                fallbackToParser: false,
             });
+            if (!parsed.isValid) {
+                return res.status(200).json({
+                    error: parsed.errors?.[0] || 'Brain compilation failed',
+                    ...parsed,
+                });
+            }
             res.json(parsed);
         }
         catch (e) {

package/dist/teach-mode.test.js

@@ -29,7 +29,29 @@ describe('Teach Mode API', () => {
     const mockAuthHeader = { 'Authorization': 'Bearer test-token' };
     beforeAll(() => {
         process.env.AUTH_TOKEN = 'test-token';
-        const instance = (0, app_1.createApp)(new Map());
+        const defaultMockBrain = {
+            generatePlan: jest.fn(),
+            compilePersona: jest.fn().mockImplementation(async (content) => {
+                if (content.includes('kind: "other"')) {
+                    return { isValid: false, manifest: null, errors: ['Invalid persona manifest'] };
+                }
+                return {
+                    isValid: true,
+                    greeting: 'Hello from Test Bot!',
+                    manifest: {
+                        id: 'test-bot',
+                        name: 'Test Bot',
+                        version: '1.0.0',
+                        identity: { name: 'Test Bot' },
+                        directives: [
+                            { id: 'dir-1', directive: 'Be helpful', category: 'behavioral' },
+                            { id: 'dir-2', directive: 'Execute system commands', category: 'behavioral' },
+                        ],
+                    },
+                };
+            }),
+        };
+        const instance = (0, app_1.createApp)(new Map([['default', { id: 'default', brain: defaultMockBrain }]]));
         app = instance.app;
     });
     afterAll(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sidurijs/api",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "publishConfig": {
     "access": "public"
   },
@@ -12,18 +12,18 @@
     "cors": "^2.8.6",
     "dotenv": "^17.4.2",
     "express": "^5.2.1",
-    "@sidurijs/body": "1.0.0",
-    "@sidurijs/brain": "1.0.1",
-    "@sidurijs/core": "1.0.2",
-    "@sidurijs/ear": "1.0.0",
-    "@sidurijs/hands": "1.0.0",
-    "@sidurijs/knowledge": "1.0.0",
-    "@sidurijs/archive": "1.0.1",
-    "@sidurijs/mouth": "1.0.0",
-    "@sidurijs/observation": "1.0.0",
-    "@sidurijs/self": "1.0.0",
-    "@sidurijs/vision": "1.0.0",
-    "@sidurijs/voice": "1.0.0"
+    "@sidurijs/body": "1.0.1",
+    "@sidurijs/brain": "1.0.3",
+    "@sidurijs/core": "1.0.4",
+    "@sidurijs/ear": "1.0.1",
+    "@sidurijs/hands": "1.0.1",
+    "@sidurijs/knowledge": "1.0.2",
+    "@sidurijs/archive": "1.0.2",
+    "@sidurijs/mouth": "1.0.1",
+    "@sidurijs/observation": "1.0.1",
+    "@sidurijs/self": "1.0.2",
+    "@sidurijs/vision": "1.0.1",
+    "@sidurijs/voice": "1.0.1"
   },
   "devDependencies": {
     "@types/cors": "^2.8.19",

package/src/app.ts

@@ -173,9 +173,21 @@ export function createApp(runtimes: Map<string, SiduriRuntime> = new Map()): App
       const parsed = await compilePersonaDocument(content, {
         brain: runtime?.brain,
         companionId,
-        fallbackToParser: true,
+        fallbackToParser: false,
       });
 
+      if (!parsed.isValid) {
+        return res.json({
+          detected: true,
+          filename: path.basename(matchedFilePath),
+          path: matchedFilePath,
+          content,
+          parsed,
+          error: parsed.errors?.[0] || 'Brain compilation failed',
+          alreadyInstalled: false,
+        });
+      }
+
       let alreadyInstalled = false;
       const repo = new SqliteSelfRepository({ dbPath: process.env.STORAGE_PATH || process.env.SQLITE_DB_PATH || 'siduri.sqlite' });
       try {
@@ -215,8 +227,14 @@ export function createApp(runtimes: Map<string, SiduriRuntime> = new Map()): App
       const parsed = await compilePersonaDocument(content, {
         brain: runtime?.brain,
         companionId: targetId,
-        fallbackToParser: true,
+        fallbackToParser: false,
       });
+      if (!parsed.isValid) {
+        return res.status(200).json({
+          error: parsed.errors?.[0] || 'Brain compilation failed',
+          ...parsed,
+        });
+      }
       res.json(parsed);
     } catch (e: any) {
       res.status(500).json({ error: e.message });

package/src/teach-mode.test.ts

@@ -27,7 +27,29 @@ describe('Teach Mode API', () => {
 
   beforeAll(() => {
     process.env.AUTH_TOKEN = 'test-token';
-    const instance = createApp(new Map());
+    const defaultMockBrain = {
+      generatePlan: jest.fn(),
+      compilePersona: jest.fn().mockImplementation(async (content: string) => {
+        if (content.includes('kind: "other"')) {
+          return { isValid: false, manifest: null, errors: ['Invalid persona manifest'] };
+        }
+        return {
+          isValid: true,
+          greeting: 'Hello from Test Bot!',
+          manifest: {
+            id: 'test-bot',
+            name: 'Test Bot',
+            version: '1.0.0',
+            identity: { name: 'Test Bot' },
+            directives: [
+              { id: 'dir-1', directive: 'Be helpful', category: 'behavioral' },
+              { id: 'dir-2', directive: 'Execute system commands', category: 'behavioral' },
+            ],
+          },
+        };
+      }),
+    };
+    const instance = createApp(new Map([['default', { id: 'default', brain: defaultMockBrain }]]) as any);
     app = instance.app;
   });