@convertcom/mcp-server 0.0.1-beta.16 → 0.0.1-beta.17

package/build/operator.js

@@ -152,7 +152,7 @@ const WRITE_ACTIONS = [
     {
         action: 'create_split_url_test',
         mode: 'write',
-        description: 'Create a draft split URL experience and verify the persisted redirect. Required body: original_pattern (or url) and variation_pattern (variation_url and redirect_url are accepted aliases). Optional: name, variation_name, case_sensitive, description, objective, primary_goal, goals, audiences, locations, settings, split_regex_support, or canonical variations with at most 100 changes each for bounded verification. Payload verification is independent from attached-goal lifecycle health; goal_configuration reports archived or unresolved goals without encouraging a duplicate create. Returns one authoritative experience plus a compact create receipt. Unknown fields are rejected.',
+        description: 'Create a draft split URL experience with an Original no-redirect control and a redirect treatment, allocated 50/50, and verify both arms, baseline, allocation and redirects. Required body: original_pattern (or url) and variation_pattern (variation_url and redirect_url are accepted aliases). Optional: name, variation_name, case_sensitive, description, objective, primary_goal, goals, audiences, locations, settings, split_regex_support. Custom variations require unique names, exactly one explicit is_baseline: true control with no changes, one defaultRedirect per treatment, and positive numeric traffic_distribution per arm totaling 100; at most 100 changes per variation. goal_configuration separately reports archived or unresolved goals. Returns one authoritative experience plus a create receipt. If verified is false, inspect that existing draft; never repeat create or activate it automatically. Unknown fields are rejected.',
         path_parameters: ['account_id', 'project_id'],
         accepts_body: true,
     },
@@ -519,16 +519,24 @@ function validateOperatorWriteInput(args) {
     if (args.action === 'create_split_url_test') {
         const original = args.body.original_pattern ?? args.body.url;
         const variation = args.body.variation_pattern ?? args.body.variation_url ?? args.body.redirect_url;
-        const variationsIssue = validateCanonicalOperatorVariations(args.body.variations);
+        const variationsIssue = validateCanonicalOperatorVariations(args.body.variations, true);
         if (variationsIssue)
             return variationsIssue;
         if (typeof original !== 'string' || !original.trim()) {
             return 'operator.create_split_url_test requires body.original_pattern or body.url.';
         }
+        if (Array.isArray(args.body.variations)) {
+            const splitIssue = validateSplitUrlArms(args.body.variations);
+            if (splitIssue)
+                return splitIssue;
+        }
         if (!Array.isArray(args.body.variations)) {
             if (typeof variation !== 'string' || !variation.trim()) {
                 return 'operator.create_split_url_test requires body.variation_pattern; body.variation_url and body.redirect_url are accepted aliases.';
             }
+            const splitIssue = validateSplitUrlArms(buildSplitUrlExperienceBody(args.body).variations);
+            if (splitIssue)
+                return splitIssue;
         }
     }
     if (args.action === 'enable_split_url_regex'
@@ -639,13 +647,15 @@ function feedbackActionConstraints(actions) {
         });
     return constraints;
 }
-function validateCanonicalOperatorVariations(value) {
+function validateCanonicalOperatorVariations(value, allowBaseline = false) {
     if (typeof value === 'undefined')
         return null;
     if (!Array.isArray(value) || value.length === 0) {
         return 'body.variations must be a non-empty array when provided.';
     }
     const allowedVariationKeys = new Set(['name', 'status', 'traffic_distribution', 'traffic_percentage', 'changes']);
+    if (allowBaseline)
+        allowedVariationKeys.add('is_baseline');
     for (const [index, variation] of value.entries()) {
         if (!isObject(variation))
             return `body.variations[${index}] must be an object.`;
@@ -665,6 +675,28 @@ function validateCanonicalOperatorVariations(value) {
     }
     return null;
 }
+function validateSplitUrlArms(variations) {
+    const hint = 'Split URL tests require at least two uniquely named arms, exactly one explicit is_baseline: true control with no changes, and a defaultRedirect change in each treatment. Supply numeric traffic_distribution for every arm (positive, totaling 100).';
+    if (variations.some((v) => !isObject(v) || typeof v.name !== 'string' || !v.name.trim()
+        || !Array.isArray(v.changes) || v.changes.some((change) => !isObject(change))))
+        return hint;
+    if (variations.length < 2 || new Set(variations.map((v) => v.name)).size !== variations.length)
+        return hint;
+    const controls = variations.filter((v) => v.is_baseline === true);
+    if (controls.length !== 1 || controls[0].changes.length !== 0)
+        return hint;
+    for (const variation of variations) {
+        if (typeof variation.traffic_distribution !== 'number' || !Number.isFinite(variation.traffic_distribution)
+            || variation.traffic_distribution <= 0 || variation.traffic_distribution > 100)
+            return hint;
+        if (variation.is_baseline !== true
+            && variation.changes.filter((change) => change.type === 'defaultRedirect').length !== 1)
+            return hint;
+    }
+    if (Math.abs(variations.reduce((sum, v) => sum + v.traffic_distribution, 0) - 100) > 0.00005)
+        return hint;
+    return null;
+}
 function operatorVariationsContainCode(value) {
     if (!Array.isArray(value))
         return false;
@@ -1804,6 +1836,10 @@ async function createSplitUrlTest(args, context) {
             readback_status: readback?.status ?? null,
         },
         verified,
+        ...(verified ? {} : {
+            remediation_required: true,
+            next_steps: ['Do not repeat create or activate this experience. Inspect the existing resource using create_receipt.experience_id; if the ID is unavailable, locate it before any further write. Correct the draft only after reviewing the persisted arms and allocation.'],
+        }),
         goal_configuration: goalConfigurationSummary(goalStatus),
         ...(verified && verification.normalizedCodeWhitespace ? {
             verification_normalizations: [
@@ -1816,7 +1852,9 @@ async function createSplitUrlTest(args, context) {
                     ? 'Readback confirmed the draft split URL experience and exact redirect patterns after Convert canonicalized only surrounding code whitespace.'
                     : 'Readback confirmed the draft split URL experience and exact redirect patterns.'
                 : 'Readback succeeded, but one or more requested split URL fields did not persist exactly.'
-            : 'Create call succeeded but no readback ID was available.',
+            : experienceId
+                ? 'Create succeeded, but readback failed. The existing draft is unverified; do not repeat create.'
+                : 'Create succeeded but returned no usable experience ID. Locate the existing draft before any further write; do not repeat create.',
     };
 }
 async function enableSplitUrlRegex(args, context) {
@@ -2095,8 +2133,11 @@ export function buildSplitUrlExperienceBody(input) {
             }
             : input.settings,
         variations: Array.isArray(input.variations) ? input.variations : [
+            { name: 'Original', is_baseline: true, traffic_distribution: 50, changes: [] },
             {
                 name: stringFrom(input.variation_name, 'Variation URL'),
+                is_baseline: false,
+                traffic_distribution: 50,
                 changes: [
                     {
                         type: 'defaultRedirect',
@@ -3719,6 +3760,21 @@ function splitUrlExperienceVerification(data, expected) {
     if (!experienceIdentityMatches(actual, expected)) {
         return { matches: false, normalizedCodeWhitespace: false };
     }
+    const arms = collectionFrom(actual.variations);
+    const expectedArms = collectionFrom(expected.variations);
+    if (arms.length !== expectedArms.length || validateSplitUrlArms(arms) !== null
+        || new Set(arms.map((arm) => Number(arm.id))).size !== arms.length
+        || arms.some((arm) => !Number.isSafeInteger(Number(arm.id)) || Number(arm.id) <= 0)
+        || arms.some((arm) => (arm.is_baseline === true)
+            !== (expectedArms.find((item) => item.name === arm.name)?.is_baseline === true))) {
+        return { matches: false, normalizedCodeWhitespace: false };
+    }
+    for (const arm of arms) {
+        const expectedArm = expectedArms.find((item) => item.name === arm.name);
+        if (!expectedArm || arm.changes.length !== collectionFrom(expectedArm.changes).length) {
+            return { matches: false, normalizedCodeWhitespace: false };
+        }
+    }
     return expectedVariationsVerification(actual.variations, expected.variations, false);
 }
 function experienceIdentityMatches(actual, expected) {