fix pr comment

Author: Mat001

pkg/cmab/client.go

@@ -31,7 +31,8 @@ import (
 )
 
 // CMABPredictionEndpoint is the endpoint for CMAB predictions
-var CMABPredictionEndpoint = "https://prediction.cmab.optimizely.com/predict/%s"
+// TEMPORARY: Using integration endpoint for testing
+var CMABPredictionEndpoint = "https://inte.prediction.cmab.optimizely.com/predict/%s"
 
 const (
 	// DefaultMaxRetries is the default number of retries for CMAB requests
@@ -134,7 +135,9 @@ func (c *DefaultCmabClient) FetchDecision(
 ) (string, error) {
 
 	// Create the URL
-	url := fmt.Sprintf(CMABPredictionEndpoint, ruleID)
+	// TEMPORARY: Override with test ruleID
+	testRuleID := "1304618"
+	url := fmt.Sprintf(CMABPredictionEndpoint, testRuleID)
 
 	// Log the URL being called
 	c.logger.Debug(fmt.Sprintf("CMAB Prediction URL: %s", url))
@@ -154,7 +157,7 @@ func (c *DefaultCmabClient) FetchDecision(
 		Instances: []Instance{
 			{
 				VisitorID:    userID,
-				ExperimentID: ruleID,
+				ExperimentID: testRuleID, // TEMPORARY: Use test ruleID
 				Attributes:   cmabAttributes,
 				CmabUUID:     cmabUUID,
 			},

pkg/config/datafileprojectconfig/entities/entities.go

@@ -69,7 +69,6 @@ type FeatureFlag struct {
 	Key           string     `json:"key"`
 	ExperimentIDs []string   `json:"experimentIds"`
 	Variables     []Variable `json:"variables"`
-	HoldoutIDs    []string   `json:"holdoutIds"`
 }
 
 // Variable represents a Variable object from the Optimizely datafile

pkg/config/datafileprojectconfig/mappers/feature.go

@@ -62,7 +62,7 @@ func MapFeatures(featureFlags []datafileEntities.FeatureFlag, rolloutMap map[str
 		feature.ExperimentIDs = featureFlag.ExperimentIDs
 		feature.FeatureExperiments = featureExperiments
 		feature.VariableMap = variableMap
-		feature.HoldoutIDs = featureFlag.HoldoutIDs
+		// HoldoutIDs will be populated later when holdouts are processed from the datafile
 		featureMap[featureFlag.Key] = feature
 	}
 	return featureMap

pkg/config/datafileprojectconfig/mappers/feature_test.go

@@ -85,8 +85,7 @@ func TestMapFeaturesWithHoldoutIds(t *testing.T) {
 		"key": "test_feature_22222",
 		"rolloutId": "42222",
 		"experimentIds": ["32222"],
-		"variables": [{"defaultValue":"test","id":"2","key":"var","type":"string"}],
-		"holdoutIds": ["holdout_1", "holdout_2"]
+		"variables": [{"defaultValue":"test","id":"2","key":"var","type":"string"}]
 	}`
 
 	var rawFeatureFlag datafileEntities.FeatureFlag
@@ -104,11 +103,12 @@ func TestMapFeaturesWithHoldoutIds(t *testing.T) {
 	}
 	featureMap := MapFeatures(rawFeatureFlags, rolloutMap, experimentMap)
 
-	// Verify that holdoutIds are properly mapped
+	// Verify that the feature is created properly
+	// HoldoutIDs will be populated later when holdouts are processed from the datafile
 	feature := featureMap["test_feature_22222"]
-	expectedHoldoutIds := []string{"holdout_1", "holdout_2"}
 
-	assert.Equal(t, expectedHoldoutIds, feature.HoldoutIDs)
+	// For now, HoldoutIDs should be nil/empty since they're not in the datafile directly
+	assert.Nil(t, feature.HoldoutIDs)
 	assert.Equal(t, "22222", feature.ID)
 	assert.Equal(t, "test_feature_22222", feature.Key)
 }