Decide Whether IsMediaAndToolsCurrent Has the Tool-Change Sense Inverted #1079

Question

SidecarFile.Read() marks the sidecar out of date when IsToolsCurrent() is false and SidecarUpdateOnToolChange is on. SidecarFile.IsMediaAndToolsCurrent(), which Update() uses to decide whether to rerun the tools, treats tools as current when IsToolsCurrent() is true or SidecarUpdateOnToolChange is on.

The two read as opposite senses:

• With the option on, Read() already forced an update, so the or makes Update() skip the tool check.
• With the option off, Update() reruns the tools whenever the versions differ, which is the case the option is meant to leave alone. Update() only runs when the state or media changed.

Reproduction shape

1. Set SidecarUpdateOnToolChange: false, process name.mkv, and then upgrade a media tool.
2. Process again with a change that alters the state.
3. Update() reruns all tools and rewrites the stored tool versions.

Decision needed

Is this intended, or should IsMediaAndToolsCurrent() use IsToolsCurrent() || !SidecarUpdateOnToolChange?

Found while drawing the process command diagrams, ARCHITECTURE.md "Process Command Logic", suspect branch S11.