I'm not used to lua, but vanilla has things like 'if ... and false then' or 'if ... and not false' that seem terribly wrong.

Example:

--[[ s2anc_artefact_armour_helmet_trigger ]]--

events.CharacterRankUpNeedsAncillary[#events.CharacterRankUpNeedsAncillary+1] =
function (context)
if conditions.CharacterType("General", context) and false then
if conditions.DateInRange(1100, 1900, context) then
effect.ancillary("s2anc_artefact_armour_helmet", 50, context)
end
return true
end
return false
end

Incidentally, although there is absolutely no condition that would prevent that ancillary from spawning at every other general level up I have never seen it, and I've been rerolling a lot to test other changes.