Coding 2023-10-15
Okay, I got distracted today, and there was travel, so this is going to be a quick entry. I worked through the names in MOTR a little more, and I'm right now in the area of the Artifact protocol, along with some of the helper classes it relies on. The Artifact protocol is suffering from a lack of evocativeness, with method names like convert, forbid_narrowing, and exposed_selections.
- convert just needs a more evocative name
- forbid_narrowing should be better aligned with what the implementations ultimately do, which is focused around the "exclusive fields" of a Parametric.
- exposed_selections points towards the parametric.MappedSelection helper class, which stands out to me as the biggest problem here.
The point of MappedSelection is that it extracts a value from a Selection container, but it also offers a way to convert that value to a segment of a string-based path. It is made via the parametric.map_over decorator and the installer.adaptor helper, which is a name that extra needs to be updated, because it hasn't gotten to the current round of naming yet...
Anyway, I think that class isn't being used to its full potential yet, which complicates things, but it is being used. It's too late right now for me to determine more. I'll try to understand this code better in the morning.
Good night.