Revert "fix: reject required custom auth fields (#44)"

This reverts commit 9694d05.

Author: TheRealAgentK

.github/workflows/self-test.yml

@@ -54,9 +54,6 @@ jobs:
           fi
           echo "✅ Auto-discovery correctly skips tests/"
 
-      - name: Test validate_integration.py — focused pytest regressions
-        run: pytest tests/test_validate_integration.py
-
       - name: Test check_imports.py — valid imports
         run: |
           python scripts/check_imports.py tests/examples/submodule-imports/valid_submodules.py

scripts/docs/validate_integration.md

@@ -107,7 +107,7 @@ Validates based on `auth.type`:
 | Auth Type | Required Fields | Description |
 |-----------|----------------|-------------|
 | `platform` | `provider`, `scopes` (array) | OAuth2-based authentication |
-| `custom` | `fields.properties` | API key or token-based authentication. `auth.fields.required` must be absent or empty; credential presence is handled by the platform connection flow. |
+| `custom` | `fields.properties` | API key or token-based authentication |
 | _(omitted)_ | — | No authentication (public APIs) |
 
 #### Action Configuration (`_validate_actions_config`)

scripts/validate_integration.py

@@ -121,10 +121,7 @@ def _check_required_files(self):
         # use absolute imports like 'from <integration> import <instance>'.
         has_actions_dir = (self.path / 'actions').is_dir()
         if not (self.path / '__init__.py').exists() and not has_actions_dir:
-            self.add_warning(
-                "Missing __init__.py "
-                "(required for package-style integrations, optional for modular integrations with actions/)"
-            )
+            self.add_warning("Missing __init__.py (required for package-style integrations, optional for modular integrations with actions/)")
 
         # Check for forbidden files
         if (self.path / 'integration.py').exists():
@@ -247,14 +244,6 @@ def _validate_auth_config(self):
                 self.add_error("Custom auth requires 'fields' configuration")
             elif 'properties' not in auth.get('fields', {}):
                 self.add_error("Custom auth fields must have 'properties' defined")
-            else:
-                required_fields = auth['fields'].get('required')
-                if required_fields:
-                    self.add_error(
-                        "Custom auth fields must not define a non-empty auth.fields.required array. "
-                        "Credential collection and presence are handled by the platform connection flow; "
-                        "define auth.fields.properties only."
-                    )
 
         elif auth_type is not None:
             self.add_warning(f"Unknown auth type: '{auth_type}'. Expected 'platform' or 'custom'")

tests/examples/bad-icon/config.json

@@ -15,7 +15,8 @@
           "label": "API Key",
           "help_text": "Enter your API key"
         }
-      }
+      },
+      "required": ["api_key"]
     }
   },
   "actions": {

tests/examples/good-integration/config.json

@@ -15,7 +15,8 @@
           "label": "API Key",
           "help_text": "Enter your API key"
         }
-      }
+      },
+      "required": ["api_key"]
     }
   },
   "actions": {

tests/examples/long-lines-integration/config.json

@@ -15,7 +15,8 @@
           "label": "API Key",
           "help_text": "Enter your API key"
         }
-      }
+      },
+      "required": ["api_key"]
     }
   },
   "actions": {

tests/examples/modular-integration/config.json

@@ -15,7 +15,8 @@
           "label": "API Key",
           "help_text": "Enter your API key"
         }
-      }
+      },
+      "required": ["api_key"]
     }
   },
   "actions": {

tests/sample-api/config.json

@@ -15,7 +15,8 @@
           "label": "API Key",
           "help_text": "Enter your API key from the dashboard"
         }
-      }
+      },
+      "required": ["api_key"]
     }
   },
   "actions": {