Feature: use set({ property: null }) to remove optional properties

[zirkelc]

Hi,

ElectroDB allows to remove optional properties with .remove(['property]) method. I would like to suggest to make it possible to use .set{ property: null }) to remove properties as well.

Use Case

I often find myself encapsulating entity updates in a single function that I can easily mock for unit tests, for example:

const updateLocation = async (key, properties) => {
  const result = await StoreLocations.patch({ cityId, mallId, storeId, buildingId })
    .set({ category: "food/meal" })
    .where((attr, op) => op.eq(attr.category, "food/coffee"))
    .go();

  return result.data;
}

If I were to remove the discount field, currently, I would have to either create a second function which uses remove() or use conditionals to apply remove() if properties.discount === undefined:

// dedicated update function for remove
const updateLocationRemoveDiscount = async (key, properties) => {
  const result = await StoreLocations.patch({ cityId, mallId, storeId, buildingId })
  .remove(["discount"])
  .where((attr, op) => op.eq(attr.category, "food/coffee"))
  .go();

  return result.data;
}

// apply remove conditionally
const updateLocation = async (key, properties) => {
  let query = StoreLocations.patch({ cityId, mallId, storeId, buildingId })
    .set({ category: "food/meal" })
    .where((attr, op) => op.eq(attr.category, "food/coffee"));

  if('discount' in properties && properties.discount === undefined) 
    query =  query.remove(["discount"])

  const result = await query.go();

  return result.data;
}

In the second case I have to be careful to distinguish between properties.discount is undefined because it is not available on properties or because it was set to undefined explicitly. Allowing for null values on set() would solve this nicely I think.