If a task is manually completed ala cylc set, any task modifiers (held, retry, etc) are obsolete and should be cleared (both internally and in the data store) for clarity.
Leaving these modifiers in place is harmless, but confusing, and will make a mess of modifier filtering when that arrives.