Is This Constructor Idiomatic for `effect/Schema`-Based Domain Types?

  make: (
    timestamp: DateTime.Utc,
    blockNumber: number,
    events: Chunk.Chunk<Event>,
    previousHash: string,
    hash: string
  ) =>
    // TODO `decodeUnknownSync()` is likely not necessary
    // since data 'should' be well-typed already
    // just for the sake of fully using `effect/Schema` feature set
    Schema.decodeUnknownSync(BlockSchema)({
      timestamp,
      blockNumber,
      events,
      previousHash,
      hash,
    }),