Fix #28 (#29)

* Fix #28

* Add test coverage

cortex.test.ts

@@ -48,19 +48,19 @@ test("can configure, get, and delete and Cortexes", async () => {
     overrides: {
       companyInfo: "a very good company that does AI stuff",
       companyName: "Cortex Click, Inc. --test",
-      inheritRules: false,
     },
   };
 
   let cortex = await testClient.configureCortex(cortexName, cortexConfig);
 
   cortex = await testClient.getCortex(cortexName);
   expect(cortex.config.catalogs).toStrictEqual(cortexConfig.catalogs);
+  expect(cortex.config.overrides?.inheritRules).toBe(true); // test input doesn't specify `inheritRules`, should be true by default
   // TODO - check all the properties
 
   // delete the cortex
   await cortex.delete();
-  // assert that the get failes
+  // assert that the get fails
   await expect(async () => {
     await testClient.getCortex(cortexName);
   }).rejects.toThrowError("Failed to get cortex: Not Found");

cortex.ts

@@ -134,7 +134,7 @@ export class Cortex {
       overrides: {
         companyInfo: body.companyInfo,
         companyName: body.companyName,
-        inheritRules: !!body.inheritRules,
+        inheritRules: body.inheritRules === false ? false : true,
       },
     };
     return new Cortex(config, apiClient, name);
@@ -156,7 +156,7 @@ export class Cortex {
       examples: config.chatConfig?.examples,
       greeting: config.chatConfig?.greeting,
       intro: config.chatConfig?.intro,
-      inheritRules: !!config.overrides?.inheritRules,
+      inheritRules: config.overrides?.inheritRules === false ? false : true,
       public: !!config.public,
       instructions: config.instructions,
       personality: config.customizations?.personality,