chore: remove person workflow elements (#28974)

mobile/openapi/lib/model/workflow_trigger.dart

@@ -25,13 +25,11 @@ class WorkflowTrigger {
 
   static const assetCreate = WorkflowTrigger._(r'AssetCreate');
   static const assetMetadataExtraction = WorkflowTrigger._(r'AssetMetadataExtraction');
-  static const personRecognized = WorkflowTrigger._(r'PersonRecognized');
 
   /// List of all possible values in this [enum][WorkflowTrigger].
   static const values = <WorkflowTrigger>[
     assetCreate,
     assetMetadataExtraction,
-    personRecognized,
   ];
 
   static WorkflowTrigger? fromJson(dynamic value) => WorkflowTriggerTypeTransformer().decode(value);
@@ -72,7 +70,6 @@ class WorkflowTriggerTypeTransformer {
       switch (data) {
         case r'AssetCreate': return WorkflowTrigger.assetCreate;
         case r'AssetMetadataExtraction': return WorkflowTrigger.assetMetadataExtraction;
-        case r'PersonRecognized': return WorkflowTrigger.personRecognized;
         default:
           if (!allowNull) {
             throw ArgumentError('Unknown enum value to decode: $data');

mobile/openapi/lib/model/workflow_type.dart

@@ -24,12 +24,10 @@ class WorkflowType {
   String toJson() => value;
 
   static const assetV1 = WorkflowType._(r'AssetV1');
-  static const assetPersonV1 = WorkflowType._(r'AssetPersonV1');
 
   /// List of all possible values in this [enum][WorkflowType].
   static const values = <WorkflowType>[
     assetV1,
-    assetPersonV1,
   ];
 
   static WorkflowType? fromJson(dynamic value) => WorkflowTypeTypeTransformer().decode(value);
@@ -69,7 +67,6 @@ class WorkflowTypeTypeTransformer {
     if (data != null) {
       switch (data) {
         case r'AssetV1': return WorkflowType.assetV1;
-        case r'AssetPersonV1': return WorkflowType.assetPersonV1;
         default:
           if (!allowNull) {
             throw ArgumentError('Unknown enum value to decode: $data');

open-api/immich-openapi-specs.json

@@ -27273,8 +27273,7 @@
         "description": "Plugin trigger type",
         "enum": [
           "AssetCreate",
-          "AssetMetadataExtraction",
+          "AssetMetadataExtraction"
-          "PersonRecognized"
         ],
         "type": "string"
       },
@@ -27301,8 +27300,7 @@
       "WorkflowType": {
         "description": "Workflow type",
         "enum": [
-          "AssetV1",
+          "AssetV1"
-          "AssetPersonV1"
         ],
         "type": "string"
       },

packages/plugin-core/manifest.json

@@ -203,31 +203,6 @@
       },
       "uiHints": ["Filter"]
     },
-    {
-      "name": "filterPerson",
-      "title": "Filter by person",
-      "description": "Filter by detected person",
-      "types": ["AssetV1"],
-      "schema": {
-        "properties": {
-          "personIds": {
-            "type": "string",
-            "array": true,
-            "title": "Person IDs",
-            "description": "List of person to match",
-            "uiHint": "personId"
-          },
-          "matchAny": {
-            "type": "boolean",
-            "title": "Match any",
-            "default": true,
-            "description": "Match any name (true) or require all names (false)"
-          }
-        },
-        "required": ["personIds"]
-      },
-      "uiHints": ["Filter"]
-    },
     {
       "name": "assetArchive",
       "title": "Archive asset",

packages/plugin-sdk/src/types.ts

@@ -10,7 +10,7 @@ type DeepPartial<T> = T extends Date
 
 export type WorkflowEventMap = {
   [WorkflowType.AssetV1]: AssetV1;
-  [WorkflowType.AssetPersonV1]: AssetPersonV1;
+  // [WorkflowType.AssetPersonV1]: AssetPersonV1;
 };
 
 export type WorkflowEventData<T extends WorkflowType> = WorkflowEventMap[T];
@@ -18,7 +18,7 @@ export type WorkflowEventData<T extends WorkflowType> = WorkflowEventMap[T];
 export enum WorkflowTrigger {
   AssetCreate = 'AssetCreate',
   AssetMetadataExtraction = 'AssetMetadataExtraction',
-  PersonRecognized = 'PersonRecognized',
+  // PersonRecognized = 'PersonRecognized',
 }
 
 export type WorkflowEventPayload<
@@ -122,9 +122,9 @@ export type AssetV1 = {
   };
 };
 
-export type AssetPersonV1 = AssetV1 & {
+// export type AssetPersonV1 = AssetV1 & {
-  person: {
+//   person: {
-    id: string;
+//     id: string;
-    name: string;
+//     name: string;
-  };
+//   };
-};
+// };

packages/sdk/src/fetch-client.ts

@@ -7271,13 +7271,11 @@ export enum PartnerDirection {
     SharedWith = "shared-with"
 }
 export enum WorkflowType {
-    AssetV1 = "AssetV1",
+    AssetV1 = "AssetV1"
-    AssetPersonV1 = "AssetPersonV1"
 }
 export enum WorkflowTrigger {
     AssetCreate = "AssetCreate",
-    AssetMetadataExtraction = "AssetMetadataExtraction",
+    AssetMetadataExtraction = "AssetMetadataExtraction"
-    PersonRecognized = "PersonRecognized"
 }
 export enum QueueJobStatus {
     Active = "active",

server/src/enum.ts

@@ -1174,7 +1174,7 @@ export const WorkflowTriggerSchema = z
 
 export enum WorkflowType {
   AssetV1 = 'AssetV1',
-  AssetPersonV1 = 'AssetPersonV1',
+  // AssetPersonV1 = 'AssetPersonV1',
 }
 
 export const WorkflowTypeSchema = z.enum(WorkflowType).describe('Workflow type').meta({ id: 'WorkflowType' });

server/src/utils/workflow.spec.ts

@@ -8,26 +8,26 @@ const tests: Array<{ trigger: WorkflowTrigger; types: WorkflowType[]; expected:
     types: [WorkflowType.AssetV1],
     expected: true,
   },
-  {
+  // {
-    trigger: WorkflowTrigger.AssetCreate,
+  //   trigger: WorkflowTrigger.AssetCreate,
-    types: [WorkflowType.AssetPersonV1],
+  //   types: [WorkflowType.AssetPersonV1],
-    expected: true,
+  //   expected: true,
-  },
+  // },
-  {
+  // {
-    trigger: WorkflowTrigger.PersonRecognized,
+  //   trigger: WorkflowTrigger.PersonRecognized,
-    types: [WorkflowType.AssetPersonV1],
+  //   types: [WorkflowType.AssetPersonV1],
-    expected: true,
+  //   expected: true,
-  },
+  // },
-  {
+  // {
-    trigger: WorkflowTrigger.PersonRecognized,
+  //   trigger: WorkflowTrigger.PersonRecognized,
-    types: [WorkflowType.AssetV1],
+  //   types: [WorkflowType.AssetV1],
-    expected: false,
+  //   expected: false,
-  },
+  // },
-  {
+  // {
-    trigger: WorkflowTrigger.PersonRecognized,
+  //   trigger: WorkflowTrigger.PersonRecognized,
-    types: [WorkflowType.AssetV1, WorkflowType.AssetPersonV1],
+  //   types: [WorkflowType.AssetV1, WorkflowType.AssetPersonV1],
-    expected: true,
+  //   expected: true,
-  },
+  // },
 ];
 
 describe(isMethodCompatible.name, () => {

server/src/utils/workflow.ts

@@ -4,7 +4,7 @@ import { PluginMethodSearchResponse } from 'src/repositories/plugin.repository';
 
 export const triggerMap: Record<WorkflowTrigger, WorkflowType[]> = {
   [WorkflowTrigger.AssetCreate]: [WorkflowType.AssetV1],
-  [WorkflowTrigger.PersonRecognized]: [WorkflowType.AssetPersonV1],
+  // [WorkflowTrigger.PersonRecognized]: [WorkflowType.AssetPersonV1],
   [WorkflowTrigger.AssetMetadataExtraction]: [WorkflowType.AssetV1],
 };
 
@@ -14,7 +14,7 @@ export const getWorkflowTriggers = () =>
 /** some types extend other types and have implied compatibility */
 const inferredMap: Record<WorkflowType, WorkflowType[]> = {
   [WorkflowType.AssetV1]: [],
-  [WorkflowType.AssetPersonV1]: [WorkflowType.AssetV1],
+  // [WorkflowType.AssetPersonV1]: [WorkflowType.AssetV1],
 };
 
 const withImpliedItems = (type: WorkflowType): WorkflowType[] => {

web/src/lib/utils/workflow.ts

@@ -6,9 +6,9 @@ export const getTriggerName = ($t: MessageFormatter, type: WorkflowTrigger) => {
     case WorkflowTrigger.AssetCreate: {
       return $t('trigger_asset_uploaded');
     }
-    case WorkflowTrigger.PersonRecognized: {
+    // case WorkflowTrigger.PersonRecognized: {
-      return $t('trigger_person_recognized');
+    //   return $t('trigger_person_recognized');
-    }
+    // }
     default: {
       return type;
     }
@@ -20,9 +20,9 @@ export const getTriggerDescription = ($t: MessageFormatter, type: WorkflowTrigge
     case WorkflowTrigger.AssetCreate: {
       return $t('trigger_asset_uploaded_description');
     }
-    case WorkflowTrigger.PersonRecognized: {
+    // case WorkflowTrigger.PersonRecognized: {
-      return $t('trigger_person_recognized_description');
+    //   return $t('trigger_person_recognized_description');
-    }
+    // }
     default: {
       return type;
     }