Refactor InverseRules to use a slice instead of a string; update related context creation and tests for consistency

Author: pelikhan

cmd/generate/context.go

@@ -32,7 +32,7 @@ func (h *generateCommandHandler) CreateContextFromPrompt(promptFile string) (*Pr
 		// Infered intent of the prompt
 		Intent:       "",
 		Rules:        []string{},
-		InverseRules: "",
+		InverseRules: []string{},
 		InputSpec:    "",
 		Tests:        "",
 		TestData:     "",

cmd/generate/context_test.go

@@ -37,7 +37,7 @@ messages:
 			expectedFields: map[string]interface{}{
 				"intent":       "",
 				"rules":        []string{},
-				"inverseRules": "",
+				"inverseRules": []string{},
 			},
 		},
 		{
@@ -156,7 +156,7 @@ messages:
 						t.Errorf("Expected %s to be %q, got %q", field, expectedValue, context.Rules)
 					}
 				case "inverseRules":
-					if context.InverseRules != expectedValue.(string) {
+					if !reflect.DeepEqual(context.InverseRules, expectedValue.([]string)) {
 						t.Errorf("Expected %s to be %q, got %q", field, expectedValue, context.InverseRules)
 					}
 				}

cmd/generate/pipeline.go

@@ -212,7 +212,12 @@ Inverse Rules:`, context.Rules)
 	if err != nil {
 		return err
 	}
-	context.InverseRules = inverseRules
+
+	parsed := ParseRules(inverseRules)
+	if parsed == nil {
+		return fmt.Errorf("failed to parse inverse output rules: %s", inverseRules)
+	}
+	context.InverseRules = parsed
 
 	return nil
 }

cmd/generate/types.go

@@ -83,7 +83,7 @@ type PromptPexContext struct {
 	PromptHash     string            `json:"promptHash" yaml:"promptHash"`
 	Intent         string            `json:"intent" yaml:"intent"`
 	Rules          []string          `json:"rules" yaml:"rules"`
-	InverseRules   string            `json:"inverseRules" yaml:"inverseRules"`
+	InverseRules   []string          `json:"inverseRules" yaml:"inverseRules"`
 	InputSpec      string            `json:"inputSpec" yaml:"inputSpec"`
 	Tests          string            `json:"tests" yaml:"tests"`
 	PromptPexTests []PromptPexTest   `json:"promptPexTests" yaml:"promptPexTests"`