@bpasero There have been some unforeseen circumstances, I resubmitted it.

I would suggest to look closer at an existing example of a context key that is updated within a group and also for the context menu and just monkey-see-monkey-doo.

vscode/src/vs/workbench/common/contextkeys.ts

Line 40 in 7c1c530

export const ActiveEditorPinnedContext = new RawContextKey<boolean>('activeEditorIsNotPreview', false, localize('activeEditorIsNotPreview', "Whether the active editor is not in preview mode"));

The context key should not be specific for the context menu, because that is just a corner case.

I misunderstood the previous meaning，I'll make some adjustments, thanks！

@bpasero I did some modifications and testing, hopefully, all possible scenarios are covered, and It looks like

Besides the 💄 feedback, I wonder if the context key fails to update properly when you move a tab, because in that case onDidActiveEditorChange will not fire.

You are right, we need to handle move events, active events don't handle this well, It was an oversight on my part, switch to deal here:

Thanks!