fix: Comprehensive field compatibility fixes for Commander API

Field Type Definition Updates:
- Update field-types.json with missing elements for complex fields
- Fix phone field: add elements [region, number, ext, type]
- Fix name field: change elements from [firstName, lastName, fullName] to [first, middle, last]
- Fix bankAccount field: add missing otherType element
- Fix privateKey field: add elements [publicKey, privateKey]
- Fix schedule field: add elements [type, time, month]
- Add missing appFiller field type with elements [applicationTitle, contentFilter, macroSequence]
- Add missing pamResources field type with elements [controllerUid, folderUid, resourceRef]
- Update complex field processing logic to handle new field definitions

Field Value Formatting Fixes:
- Add automatic field validation and formatting in processFieldsForSDK()
- Fix date fields: auto-convert Unix seconds to milliseconds
- Fix card numbers: remove dashes, spaces, and dots from card numbers
- Fix expiration dates: convert MM/YY to MM/YYYY format with smart year detection
- Provide user warnings when field values are automatically reformatted

Testing & Infrastructure:
- Add GetFieldType() helper function for testing field type definitions
- Add comprehensive field loading tests in loader_test.go
- Add field processing tests for new complex field types (appFiller, schedule)
- Add field formatting tests with 25+ test cases covering edge cases
- Add field reconstruction tests for complex field processing pipeline

Resolves issues with:
- Field type mismatches between MCP and Commander API expectations
- Date fields sending Unix timestamps in wrong format
- Card numbers being created with dashes causing display issues
- Expiration dates using MM/YY when Vault expects MM/YYYY
- Missing field type definitions causing processing failures

All changes ensure full compatibility with Commander API field requirements
and provide comprehensive test coverage for field processing pipeline.

Author: maksimu

internal/ksm/complex_fields_test.go

@@ -1064,3 +1064,183 @@ func TestIsSensitiveFieldComprehensive(t *testing.T) {
 		})
 	}
 }
+
+func TestProcessAppFillerField(t *testing.T) {
+	client := &Client{}
+
+	tests := []struct {
+		name     string
+		value    interface{}
+		unmask   bool
+		expected interface{}
+		found    bool
+	}{
+		{
+			name: "app filler - masked",
+			value: []interface{}{
+				map[string]interface{}{
+					"applicationTitle": "Banking App",
+					"contentFilter":    "input[type=password]",
+					"macroSequence":    "username{TAB}password{ENTER}",
+				},
+			},
+			unmask: false,
+			expected: []map[string]interface{}{
+				{
+					"applicationTitle": "Banking App",
+					"contentFilter":    "input[type=password]",
+					"macroSequence":    "use***ER}",
+				},
+			},
+			found: true,
+		},
+		{
+			name: "app filler - unmasked",
+			value: []interface{}{
+				map[string]interface{}{
+					"applicationTitle": "Shopping Site",
+					"contentFilter":    ".login-form",
+					"macroSequence":    "[email protected]{TAB}SecurePass123{ENTER}",
+				},
+			},
+			unmask: true,
+			expected: []map[string]interface{}{
+				{
+					"applicationTitle": "Shopping Site",
+					"contentFilter":    ".login-form",
+					"macroSequence":    "[email protected]{TAB}SecurePass123{ENTER}",
+				},
+			},
+			found: true,
+		},
+		{
+			name: "minimal app filler",
+			value: []interface{}{
+				map[string]interface{}{
+					"applicationTitle": "Simple App",
+				},
+			},
+			unmask: false,
+			expected: []map[string]interface{}{
+				{
+					"applicationTitle": "Simple App",
+				},
+			},
+			found: true,
+		},
+		{
+			name:     "empty value",
+			value:    []interface{}{},
+			unmask:   false,
+			expected: nil,
+			found:    false,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result, found := client.processAppFillerField(tt.value, tt.unmask)
+			assert.Equal(t, tt.found, found, "Found status should match")
+			if tt.found {
+				assert.Equal(t, tt.expected, result, "App filler data should match")
+			}
+		})
+	}
+}
+
+func TestProcessScheduleField(t *testing.T) {
+	client := &Client{}
+
+	tests := []struct {
+		name     string
+		value    interface{}
+		unmask   bool
+		expected interface{}
+		found    bool
+	}{
+		{
+			name: "complete schedule",
+			value: []interface{}{
+				map[string]interface{}{
+					"type":          "Monthly",
+					"time":          "14:30",
+					"cron":          "0 14 1 * *",
+					"tz":            "America/New_York",
+					"weekday":       "1",
+					"intervalCount": float64(1),
+				},
+			},
+			unmask: false,
+			expected: []map[string]interface{}{
+				{
+					"type":          "Monthly",
+					"time":          "14:30",
+					"cron":          "0 14 1 * *",
+					"tz":            "America/New_York",
+					"weekday":       "1",
+					"intervalCount": float64(1),
+				},
+			},
+			found: true,
+		},
+		{
+			name: "weekly schedule",
+			value: []interface{}{
+				map[string]interface{}{
+					"type": "Weekly",
+					"time": "09:00",
+				},
+			},
+			unmask: false,
+			expected: []map[string]interface{}{
+				{
+					"type": "Weekly",
+					"time": "09:00",
+				},
+			},
+			found: true,
+		},
+		{
+			name: "multiple schedules",
+			value: []interface{}{
+				map[string]interface{}{
+					"type": "Daily",
+					"time": "06:00",
+				},
+				map[string]interface{}{
+					"type": "Weekly",
+					"time": "12:00",
+				},
+			},
+			unmask: false,
+			expected: []map[string]interface{}{
+				{
+					"type": "Daily",
+					"time": "06:00",
+				},
+				{
+					"type": "Weekly",
+					"time": "12:00",
+				},
+			},
+			found: true,
+		},
+		{
+			name:     "empty value",
+			value:    []interface{}{},
+			unmask:   false,
+			expected: nil,
+			found:    false,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result, found := client.processScheduleField(tt.value, tt.unmask)
+			assert.Equal(t, tt.found, found, "Found status should match")
+			if tt.found {
+				assert.Equal(t, tt.expected, result, "Schedule data should match")
+			}
+		})
+	}
+}