getItemSummary receives mapProp value instead of original list value

[mattlevine32]

Hi team, thanks for the great work on Puck - we're loving working with it. Quick one we ran into - mapProp seems to run before getItemSummary which means we can't use fields from the originally fetched object in the summary.

We want to only store an ID as a prop in JSON vs. more complex objects which is why we want to keep mapProp as just id.

Would it make sense to have getItemSummary always receive the raw item value?

  {
    label: 'Observation Panel',
    fields: {
      view: {
        type: 'select',
        options: [
          { value: 'Row', label: 'Row' },
          { value: 'Card', label: 'Card' },
        ],
      },
      observationPanelDefinitionId: {
        type: 'external',
        fetchList: fetchAvailablePanels,
        getItemSummary: (item: ObservationPanelDefinition) => item.title,
        mapProp: (item: ObservationPanelDefinition) => item.id,
        mapRow: (item: ObservationPanelDefinition) => ({title: item.title}),
        showSearch: true,
      },
    },
    defaultProps: {
      view: 'Row',
      observationPanelDefinitionId: null,
    },
    render: (props) => <BlockRenderer {...props} />,
  }