failing on the following providepath:

(sp/declarepath map-walker)
(sp/providepath map-walker
                 [(sp/walker map?)
                  (sp/continue-then-stay sp/MAP-VALS map-walker)])

@caio what version of specter?

are you doing aot compilation?

[com.rpl/specter “0.13.1”]`

yep, I’m doing aot compilation

@caio ok

could you open an issue for this? don't have time to look at it now

slightly gross:

(sp/transform [:parent sp/ALL #(= :children (first %)) sp/LAST] identity {:parent {:something {}}})
=> {:parent {:something {}}}
(sp/transform [:parent sp/ALL #(= :children (first %)) sp/LAST] identity {:parent {:something {} :children 1}})
=> {:parent {:something {}, :children 1}}

👍 gives me the correct result so not too gross. looks like i can drop the sp/LAST aswell.

(sp/transform [:parent sp/ALL #(= :children (first %))] identity {:parent {:something {}}})
=> {:parent {:something {}}}
(sp/transform [:parent sp/ALL #(= :children (first %))] identity {:parent {:something {} :children 1}})
=> {:parent {:something {}, :children 1}}

i'm still thinking there must be a cleaner way out there

yeah

must be a common enough scenario

I guess it defaults to this way because that's what update-in does

i could have sworn i already unsuccessfully tried what i'm about to post..

(sp/transform [:parent sp/ALL (sp/selected? :children)] identity {:parent {:something {} }})
=> {:parent {:something {}}}
(sp/transform [:parent sp/ALL (sp/selected? :children)] identity {:parent {:something {},:children 1 }})
=> {:parent {:something {}, :children 1}}

but it works!

I think the difference is the sp/ALL bit

hmm.

i'm looking at transform now and thinking maybe i should be wrapping everything in sp/selected?

(sp/transform [(sp/selected? :parent) (sp/selected? :children)] identity {:parent {:something {},:children 1 }})
=> {:parent {:something {}, :children 1}}

so if it doesn't exist, it's not created. not sure if that's sane or not?!

(sp/transform [(sp/selected? :parent) (sp/selected? :children)] identity {})
=> {}

not sure it's doing what you expect

(sp/transform [(sp/selected? :parent) (sp/selected? :children)] identity {})
=> {}
(sp/transform [(sp/selected? :parent) (sp/selected? :children)] identity {:parent nil})
=> {:parent nil}
(sp/transform [(sp/selected? :parent) (sp/selected? :children)] identity {:parent {:something nil}})
=> {:parent {:something nil}}
(sp/transform [(sp/selected? :parent) (sp/selected? :children)] identity {:parent {:something nil :children nil}})
=> {:parent {:something nil, :children nil}}
(sp/transform [(sp/selected? :parent) (sp/selected? :children)] (constantly 'foo) {:parent {:something nil :children nil}})
=> foo

you are correct...damn.!

found it

undocumented function, sp/must

(sp/transform [(sp/must :parent)] (constantly 'foo) {})
=> {}
(sp/transform [(sp/must :parent)] (constantly 'foo) {:parent {}})
=> {:parent foo}
(sp/transform [(sp/must :parent)] (constantly 'foo) {:parent {:children {}}})
=> {:parent foo}
(sp/transform [(sp/must :parent)] (constantly 'foo) {:parent {:children {} :foo nil
                                                              }})
=> {:parent foo}
(sp/transform [(sp/must :parent) (sp/must :children)] (constantly 'foo) {:parent {:children {} :foo nil}})
=> {:parent {:children foo, :foo nil}}
(sp/transform [(sp/must :parent) (sp/must :children)] (constantly 'foo) {:parent { :foo nil}})
=> {:parent {:foo nil}}

actually that's not true, it has a docstring, just not visible from core

https://github.com/nathanmarz/specter/blob/master/src/clj/com/rpl/specter/navs.cljc#L421

super. that's the one.

thanks a lot

np