@promptbook/browser 0.104.0-18 → 0.104.0-19

package/esm/index.es.js

@@ -19,7 +19,7 @@ const BOOK_LANGUAGE_VERSION = '2.0.0';
  * @generated
  * @see https://github.com/webgptorg/promptbook
  */
-const PROMPTBOOK_ENGINE_VERSION = '0.104.0-18';
+const PROMPTBOOK_ENGINE_VERSION = '0.104.0-19';
 /**
  * TODO: string_promptbook_version should be constrained to the all versions of Promptbook engine
  * Note: [💞] Ignore a discrepancy between file name and entity name
@@ -2208,9 +2208,10 @@ function isValidFilePath(filename) {
  *
  * Note: [🔂] This function is idempotent.
  * Note: Dataurl are considered perfectly valid.
- * Note: There are two similar functions:
- * - `isValidUrl` which tests any URL
- * - `isValidPipelineUrl` *(this one)* which tests just promptbook URL
+ * Note: There are few similar functions:
+ * - `isValidUrl` *(this one)* which tests any URL
+ * - `isValidAgentUrl` which tests just agent URL
+ * - `isValidPipelineUrl` which tests just pipeline URL
  *
  * @public exported from `@promptbook/utils`
  */
@@ -2620,6 +2621,39 @@ function titleToName(value) {
  * Note: [💞] Ignore a discrepancy between file name and entity name
  */
 
+/**
+ * Tests if given string is valid agent URL
+ *
+ * Note: There are few similar functions:
+ * - `isValidUrl` which tests any URL
+ * - `isValidAgentUrl` *(this one)* which tests just agent URL
+ * - `isValidPipelineUrl` which tests just pipeline URL
+ *
+ * @public exported from `@promptbook/utils`
+ */
+function isValidAgentUrl(url) {
+    if (!isValidUrl(url)) {
+        return false;
+    }
+    if (!url.startsWith('https://') && !url.startsWith('http://') /* <- Note: [👣] */) {
+        return false;
+    }
+    if (url.includes('#')) {
+        // TODO: [🐠]
+        return false;
+    }
+    /*
+    Note: [👣][🧠] Is it secure to allow pipeline URLs on private and unsecured networks?
+    if (isUrlOnPrivateNetwork(url)) {
+        return false;
+    }
+    */
+    return true;
+}
+/**
+ * TODO: [🐠] Maybe more info why the URL is invalid
+ */
+
 /**
  * Normalizes agent name from arbitrary string to valid agent name
  *
@@ -3336,21 +3370,36 @@ class FromCommitmentDefinition extends BaseCommitmentDefinition {
     applyToAgentModelRequirements(requirements, content) {
         const trimmedContent = content.trim();
         if (!trimmedContent) {
-            return requirements;
+            return {
+                ...requirements,
+                parentAgentUrl: undefined,
+            };
         }
-        // Validate URL
-        try {
-            // eslint-disable-next-line @typescript-eslint/no-unused-vars
-            const url = new URL(trimmedContent);
-            // TODO: Add more validation if needed (e.g. check for valid protocol)
+        if (trimmedContent.toUpperCase() === 'VOID' ||
+            trimmedContent.toUpperCase() === 'NULL' ||
+            trimmedContent.toUpperCase() === 'NONE' ||
+            trimmedContent.toUpperCase() === 'NIL') {
+            return {
+                ...requirements,
+                parentAgentUrl: null,
+            };
         }
-        catch (error) {
-            console.warn(`Invalid URL in FROM commitment: ${trimmedContent}`);
-            return requirements;
+        if (!isValidAgentUrl(trimmedContent)) {
+            throw new Error(spaceTrim$1((block) => `
+                        Invalid agent URL in FROM commitment: "${trimmedContent}"
+
+                        \`\`\`book
+                        ${block(content)}
+                        \`\`\`
+
+
+                `));
         }
+        const parentAgentUrl = trimmedContent;
+        console.log('!!!! parentAgentUrl', parentAgentUrl);
         return {
             ...requirements,
-            parentAgentUrl: trimmedContent,
+            parentAgentUrl,
         };
     }
 }