Add HTTP transport capability validation and time MCP server for agentic engines

pkg/workflow/codex_engine.go

@@ -15,9 +15,9 @@ func NewCodexEngine() *CodexEngine {
 		BaseEngine: BaseEngine{
 			id:                     "codex",
 			displayName:            "Codex",
-			description:            "Uses OpenAI Codex CLI (experimental)",
+			description:            "Uses OpenAI Codex CLI with MCP server support",
 			experimental:           true,
-			supportsToolsWhitelist: false,
+			supportsToolsWhitelist: true,
 		},
 	}
 }
@@ -74,6 +74,10 @@ codex exec \
 func (e *CodexEngine) RenderMCPConfig(yaml *strings.Builder, tools map[string]any, mcpTools []string) {
 	yaml.WriteString("          cat > /tmp/mcp-config/config.toml << EOF\n")
 
+	// Add history configuration to disable persistence
+	yaml.WriteString("          [history]\n")
+	yaml.WriteString("          persistence = \"none\"\n")
+
 	// Generate [mcp_servers] section
 	for _, toolName := range mcpTools {
 		switch toolName {

pkg/workflow/codex_engine_test.go

@@ -21,8 +21,8 @@ func TestCodexEngine(t *testing.T) {
 		t.Error("Codex engine should be experimental")
 	}
 
-	if engine.SupportsToolsWhitelist() {
-		t.Error("Codex engine should not support MCP tools")
+	if !engine.SupportsToolsWhitelist() {
+		t.Error("Codex engine should support MCP tools")
 	}
 
 	// Test installation steps

pkg/workflow/codex_test.go

@@ -139,6 +139,13 @@ This is a test workflow.
 				if !strings.Contains(lockContent, "[mcp_servers.github]") {
 					t.Errorf("Expected lock file to contain '[mcp_servers.github]' section in config.toml but it didn't.\nContent:\n%s", lockContent)
 				}
+				// Check that history configuration is present
+				if !strings.Contains(lockContent, "[history]") {
+					t.Errorf("Expected lock file to contain '[history]' section in config.toml but it didn't.\nContent:\n%s", lockContent)
+				}
+				if !strings.Contains(lockContent, "persistence = \"none\"") {
+					t.Errorf("Expected lock file to contain 'persistence = \"none\"' in config.toml but it didn't.\nContent:\n%s", lockContent)
+				}
 				// Ensure it does NOT contain mcp-servers.json
 				if strings.Contains(lockContent, "mcp-servers.json") {
 					t.Errorf("Expected lock file to NOT contain 'mcp-servers.json' when using codex.\nContent:\n%s", lockContent)
@@ -289,6 +296,27 @@ tools:
 			expectMcpServersJson: true,
 			expectCodexHome:      false,
 		},
+		{
+			name: "codex with custom MCP tools generates config.toml",
+			frontmatter: `---
+engine: codex
+tools:
+  github:
+    allowed: [get_issue, create_issue]
+  custom-server:
+    mcp:
+      type: stdio
+      command: "python"
+      args: ["-m", "my_server"]
+      env:
+        API_KEY: "{{ secrets.API_KEY }}"
+    allowed: ["*"]
+---`,
+			expectedAI:           "codex",
+			expectConfigToml:     true,
+			expectMcpServersJson: false,
+			expectCodexHome:      true,
+		},
 	}
 
 	for _, tt := range tests {
@@ -332,6 +360,18 @@ This is a test workflow for MCP configuration with different AI engines.
 				if !strings.Contains(lockContent, "command = \"docker\"") {
 					t.Errorf("Expected docker command in config.toml but didn't find it in:\n%s", lockContent)
 				}
+				// Check for custom MCP server if test includes it
+				if strings.Contains(tt.name, "custom MCP") {
+					if !strings.Contains(lockContent, "[mcp_servers.custom-server]") {
+						t.Errorf("Expected [mcp_servers.custom-server] section but didn't find it in:\n%s", lockContent)
+					}
+					if !strings.Contains(lockContent, "command = \"python\"") {
+						t.Errorf("Expected python command for custom server but didn't find it in:\n%s", lockContent)
+					}
+					if !strings.Contains(lockContent, "\"API_KEY\" = \"{{ secrets.API_KEY }}\"") {
+						t.Errorf("Expected API_KEY env var for custom server but didn't find it in:\n%s", lockContent)
+					}
+				}
 				// Should NOT have services section (services mode removed)
 				if strings.Contains(lockContent, "services:") {
 					t.Errorf("Expected NO services section in workflow but found it in:\n%s", lockContent)