Debt: Issue Quick Access cleanup

[justschen]

          Non-blocking. I feel like this function could be 2 different functions for actions or extensions... or you `if` setting a couple variables instead and then return a common object:

const buttons = [{
	iconClass: ThemeIcon.asClassName(Codicon.info),
	tooltip: localize('contributedIssuePage', "Open Extension Page")
}];

let label: string;
let trigger: THE_TRIGGER_TYPE;
let accept: THE_ACCEPT_TYPE;
if (action ....) {
    label = action.item.source?.title;
    trigger = () => {
        this.commandService.executeCommand('extension.open', action.item.source.id);
        return TriggerAction.CLOSE_PICKER;
    };
    accept = () => action.run();
} else if (extension) {
    label = extension.displayName ?? extension.name;
    trigger = () => {
        this.commandService.executeCommand('extension.open', extension.identifier.value);
 	return TriggerAction.CLOSE_PICKER;
    };
    accept = () => this.commandService.executeCommand('workbench.action.openIssueReporter', extension.identifier.value);
} else {
    // could this ever happen?
    return undefined;
}

return {
    label,
    highlights: { label: matchesFuzzy(filter, label, true) },
    buttons,
    trigger,
    accept
};

Originally posted by @TylerLeonhardt in #206303 (comment)