Revert "fix: project snapshot import exclude list" (#691)

This reverts commit e7f2a2e.

Author: ruvenzx

README.md

@@ -1647,12 +1647,7 @@ if !res.Ok {
 inputSecrets := ...
 
 // Import the previously exported snapshot into the current project
-// Optionally specify entity types to exclude from the import (e.g., "connectors", "flows", "roles")
-importReq := &descope.ImportSnapshotRequest{
-	Files: files,
-	InputSecrets: inputSecrets,
-	Excludes: []string{"connectors", "flows"}, // Optional: exclude specific entity types from import
-}
+importReq := &descope.ImportSnapshotRequest{Files: files, InputSecrets: inputSecrets}
 err := descopeClient.Management.Project().ImportSnapshot(context.Background(), importReq)
 if err != nil {
 	// handle import failure

descope/internal/mgmt/audit_test.go

@@ -288,7 +288,7 @@ func TestCreateAuditWebhookConnector(t *testing.T) {
 		filter1 := filters[0].(map[string]any)
 		assert.EqualValues(t, "actions", filter1["filterType"])
 		assert.EqualValues(t, "includes", filter1["operator"])
-		assert.EqualValues(t, []any{"AccessKeyExchange", "AuthzNamespaceCreated"}, filter1["values"])
+		assert.EqualValues(t, []any{"action1", "action2"}, filter1["values"])
 		filter2 := filters[1].(map[string]any)
 		assert.EqualValues(t, "tenants", filter2["filterType"])
 		assert.EqualValues(t, "excludes", filter2["operator"])
@@ -300,7 +300,7 @@ func TestCreateAuditWebhookConnector(t *testing.T) {
 			{
 				FilterType: descope.FilterTypeActions,
 				Operator:   descope.OperatorIncludes,
-				Values:     []string{"AccessKeyExchange", "AuthzNamespaceCreated"},
+				Values:     []string{"action1", "action2"},
 			},
 			{
 				FilterType: descope.FilterTypeTenants,

descope/internal/mgmt/project_test.go

@@ -31,56 +31,8 @@ func TestProjectImport(t *testing.T) {
 		files, ok := req["files"].(map[string]any)
 		require.True(t, ok)
 		require.Equal(t, "bar", files["foo"])
-		excludes, ok := req["excludes"].([]any)
-		require.True(t, ok)
-		require.Len(t, excludes, 2)
-		require.Equal(t, "connectors", excludes[0])
-		require.Equal(t, "flows", excludes[1])
-	}))
-	req := &descope.ImportSnapshotRequest{
-		Files:    map[string]any{"foo": "bar"},
-		Excludes: []string{"connectors", "flows"},
-	}
-	err := mgmt.Project().ImportSnapshot(context.Background(), req)
-	require.NoError(t, err)
-}
-
-func TestProjectImportWithoutExcludes(t *testing.T) {
-	mgmt := newTestMgmt(nil, helpers.DoOk(func(r *http.Request) {
-		require.Equal(t, r.Header.Get("Authorization"), "Bearer a:key")
-		req := map[string]any{}
-		require.NoError(t, helpers.ReadBody(r, &req))
-		files, ok := req["files"].(map[string]any)
-		require.True(t, ok)
-		require.Equal(t, "bar", files["foo"])
-		// Verify excludes field is not present in the JSON when nil
-		_, excludesPresent := req["excludes"]
-		require.False(t, excludesPresent, "excludes field should not be present when nil")
-	}))
-	req := &descope.ImportSnapshotRequest{
-		Files: map[string]any{"foo": "bar"},
-		// Excludes is intentionally nil to test backward compatibility
-	}
-	err := mgmt.Project().ImportSnapshot(context.Background(), req)
-	require.NoError(t, err)
-}
-
-func TestProjectImportWithEmptyExcludes(t *testing.T) {
-	mgmt := newTestMgmt(nil, helpers.DoOk(func(r *http.Request) {
-		require.Equal(t, r.Header.Get("Authorization"), "Bearer a:key")
-		req := map[string]any{}
-		require.NoError(t, helpers.ReadBody(r, &req))
-		files, ok := req["files"].(map[string]any)
-		require.True(t, ok)
-		require.Equal(t, "bar", files["foo"])
-		// Verify excludes field is not present in the JSON when empty (due to omitempty)
-		_, excludesPresent := req["excludes"]
-		require.False(t, excludesPresent, "excludes field should not be present when empty due to omitempty")
 	}))
-	req := &descope.ImportSnapshotRequest{
-		Files:    map[string]any{"foo": "bar"},
-		Excludes: []string{}, // Empty slice to test backward compatibility
-	}
+	req := &descope.ImportSnapshotRequest{Files: map[string]any{"foo": "bar"}}
 	err := mgmt.Project().ImportSnapshot(context.Background(), req)
 	require.NoError(t, err)
 }
@@ -100,7 +52,6 @@ func TestValidateProjectImport(t *testing.T) {
 		files, ok := req["files"].(map[string]any)
 		require.True(t, ok)
 		require.Equal(t, "bar", files["foo"])
-
 		secrets, ok := req["inputSecrets"].(map[string]any)
 		require.True(t, ok)
 		list, ok := secrets["connectors"].([]any)

descope/types.go

@@ -1126,8 +1126,6 @@ type ExportSnapshotResponse struct {
 type ImportSnapshotRequest struct {
 	// All project settings and configurations represented as JSON files
 	Files map[string]any `json:"files"`
-	// An optional list of entity types to exclude from the import
-	Excludes []string `json:"excludes,omitempty"`
 	// An optional map of project entities and their secrets that will be
 	// injected into the snapshot before import (see below)
 	InputSecrets *SnapshotSecrets `json:"inputSecrets,omitempty"`