@bedrockio/ai 0.8.3 → 0.8.4

package/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 0.8.4
+
+- Fixed issue with user roles parsed from template is injected into conversation
+  history.
+
 ## 0.8.3
 
 - Fixed issue with message history not being retained.

package/dist/cjs/BaseClient.js

@@ -177,13 +177,20 @@ class BaseClient {
         });
         let system = '';
         let { messages = [] } = options;
+        // Templates may contain multiple roles, ie SYSTEM or USER, making them
+        // useful for one-off prompting. However in a multi-turn conversation
+        // the entire chat history will be passed, so do not inject user messages
+        // when they already exist in the options.
+        const hasUserMessages = messages.some((message) => {
+            return message.role === 'user';
+        });
         for (let section of sections) {
             const { title = 'system', content } = section;
             const role = title.toLowerCase();
             if (role === 'system') {
                 system += [system, content].join('\n');
             }
-            else {
+            else if (!hasUserMessages) {
                 messages = [
                     ...messages,
                     {

package/dist/esm/BaseClient.js

@@ -175,13 +175,20 @@ export default class BaseClient {
         });
         let system = '';
         let { messages = [] } = options;
+        // Templates may contain multiple roles, ie SYSTEM or USER, making them
+        // useful for one-off prompting. However in a multi-turn conversation
+        // the entire chat history will be passed, so do not inject user messages
+        // when they already exist in the options.
+        const hasUserMessages = messages.some((message) => {
+            return message.role === 'user';
+        });
         for (let section of sections) {
             const { title = 'system', content } = section;
             const role = title.toLowerCase();
             if (role === 'system') {
                 system += [system, content].join('\n');
             }
-            else {
+            else if (!hasUserMessages) {
                 messages = [
                     ...messages,
                     {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bedrockio/ai",
-  "version": "0.8.3",
+  "version": "0.8.4",
   "description": "Bedrock wrapper for common AI chatbots.",
   "type": "module",
   "scripts": {

package/types/BaseClient.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"BaseClient.d.ts","sourceRoot":"","sources":["../src/BaseClient.js"],"names":[],"mappings":"AAKA;IACE,0BASC;IARC,aAIC;IACD,2BAEE;IAKJ;;;;;OAKG;IACH,gBAFW,aAAa,gBAgCvB;IAED;;;;;OAKG;IACH,gBAHW,aAAa,GAAG,aAAa,gCAsDvC;IAED;;;;OAIG;IACH,wBAFW,MAAM,OAIhB;IAID,8BAGC;IAED,8BAGC;IAED,qCAGC;IAED;;OAEG;IACH,0CAGC;IAED;;OAEG;IACH,oDAIC;IAED;;OAEG;IACH,oDAIC;IAID;;OAEG;IACH,oCAOC;IAED;;;MAeC;IAED,4CAyCC;IAED,uCAoBC;IAED;;;MA4BC;IAED,uDAWC;IAED,kDAMC;CACF;;;;;WAIa,MAAM,GAAC,aAAa,EAAE;;;;YACtB,MAAM;;;;YACN,OAAO;;;;;;;;aAEP,MAAM,GAAG,MAAM;;;;;;;;;;;sBAOf,MAAM;;;UAKN,QAAQ,GAAG,MAAM,GAAG,WAAW;aAC/B,MAAM;;iCA/Ua,sBAAsB"}
+{"version":3,"file":"BaseClient.d.ts","sourceRoot":"","sources":["../src/BaseClient.js"],"names":[],"mappings":"AAKA;IACE,0BASC;IARC,aAIC;IACD,2BAEE;IAKJ;;;;;OAKG;IACH,gBAFW,aAAa,gBAgCvB;IAED;;;;;OAKG;IACH,gBAHW,aAAa,GAAG,aAAa,gCAsDvC;IAED;;;;OAIG;IACH,wBAFW,MAAM,OAIhB;IAID,8BAGC;IAED,8BAGC;IAED,qCAGC;IAED;;OAEG;IACH,0CAGC;IAED;;OAEG;IACH,oDAIC;IAED;;OAEG;IACH,oDAIC;IAID;;OAEG;IACH,oCAOC;IAED;;;MAeC;IAED,4CAiDC;IAED,uCAoBC;IAED;;;MA4BC;IAED,uDAWC;IAED,kDAMC;CACF;;;;;WAIa,MAAM,GAAC,aAAa,EAAE;;;;YACtB,MAAM;;;;YACN,OAAO;;;;;;;;aAEP,MAAM,GAAG,MAAM;;;;;;;;;;;sBAOf,MAAM;;;UAKN,QAAQ,GAAG,MAAM,GAAG,WAAW;aAC/B,MAAM;;iCAvVa,sBAAsB"}