Implementing the preceding_pf_ids as a simple list[ProductFootprintId] is great as a starter point to satisfy the needs of the spec but it would be better off as a proper class, enabling organisation of the product_footprint_ids etc, however, this probably requires more context, which I dont have right now