make Video.metadata optional

Author: Brendonovich

apps/web/app/api/playlist/route.ts

@@ -70,17 +70,12 @@ const ApiLive = HttpApiBuilder.api(Api).pipe(
 						).pipe(Effect.provide(S3ProviderLayer));
 					}).pipe(
 						provideOptionalAuth,
+						Effect.tapErrorCause(Effect.logError),
 						Effect.catchTags({
 							VerifyVideoPasswordError: () => new HttpApiError.Forbidden(),
 							PolicyDenied: () => new HttpApiError.Unauthorized(),
-							DatabaseError: (e) =>
-								Effect.logError(e).pipe(
-									Effect.andThen(() => new HttpApiError.InternalServerError()),
-								),
-							S3Error: (e) =>
-								Effect.logError(e).pipe(
-									Effect.andThen(() => new HttpApiError.InternalServerError()),
-								),
+							DatabaseError: () => new HttpApiError.InternalServerError(),
+							S3Error: () => new HttpApiError.InternalServerError(),
 						}),
 					),
 				);

packages/web-domain/src/Video.ts

@@ -15,7 +15,9 @@ export class Video extends Schema.Class<Video>("Video")({
 	ownerId: Schema.String,
 	name: Schema.String,
 	public: Schema.Boolean,
-	metadata: Schema.Record({ key: Schema.String, value: Schema.Any }),
+	metadata: Schema.OptionFromNullOr(
+		Schema.Record({ key: Schema.String, value: Schema.Any }),
+	),
 	source: Schema.Struct({
 		type: Schema.Literal("MediaConvert", "local", "desktopMP4"),
 	}),