The langauges map is written so that it is lazy. We do not need to pre-populate it

Replace heapservice with lifecycle for code actions

Also changes processService to not modify its input actions, which seems like unexpected behavior

These arrays of actions should not be modified by the caller. Mark them as readonly to prevent this. This also let me eliminate a number of not null supressions that had been introduced during the strict null migration

One small code change was required in `showError`. However I believe this was actually a bug since modifying the input array was likely not what would be expected (in practice, we usually just passed in array literals so I don't think we hit this one)

These lists of actions should not be modified by callers

The actions being passed around are generally not expected to be modified. Use ReadonlyArray to prevent this

It prevents sending SIGQUIT

Fixes #75074

#74948