Adds a new telemetry event for a rare token to refresh.

1. Sends a telemetry event for missing expirity dates.
2. Sends a telemetry events when we skip refreshing of a non-cloud
sessions.

We skip refreshing a non-cloud sessions, because we expect it to be a
PAT. But now it's rather unexpected situation, so we send an event to be
aware about it.

(#4315)

Author: sergeibbb

docs/telemetry-events.md

@@ -521,6 +521,19 @@ or
 }
 ```
 
+### cloudIntegrations/refreshConnection/skippedUnusualToken
+
+> Sent when a connection session has a missing expiry date
+or when connection refresh is skipped due to being a non-cloud session
+
+```typescript
+{
+  'cloud': boolean,
+  'integration.id': string,
+  'reason': 'skip-non-cloud' | 'missing-expiry'
+}
+```
+
 ### cloudIntegrations/settingsOpened
 
 > Sent when a user chooses to manage the cloud integrations

src/constants.telemetry.ts

@@ -84,6 +84,10 @@ export interface TelemetryEvents extends WebviewShowAbortedEvents, WebviewShownE
 	/** Sent when refreshing a provider token from the api fails */
 	'cloudIntegrations/refreshConnection/failed': CloudIntegrationsRefreshConnectionFailedEvent;
 
+	/** Sent when a connection session has a missing expiry date
+	 * or when connection refresh is skipped due to being a non-cloud session */
+	'cloudIntegrations/refreshConnection/skippedUnusualToken': CloudIntegrationsRefreshConnectionSkipUnusualTokenEvent;
+
 	/** Sent when a cloud-based hosting provider is connected */
 	'cloudIntegrations/hosting/connected': CloudIntegrationsHostingConnectedEvent;
 	/** Sent when a cloud-based hosting provider is disconnected */
@@ -407,6 +411,12 @@ interface CloudIntegrationsRefreshConnectionFailedEvent {
 	'integration.id': string | undefined;
 }
 
+interface CloudIntegrationsRefreshConnectionSkipUnusualTokenEvent {
+	'integration.id': string;
+	reason: 'skip-non-cloud' | 'missing-expiry';
+	cloud: boolean | undefined;
+}
+
 interface CloudIntegrationsHostingConnectedEvent {
 	'hostingProvider.provider': IntegrationId;
 	'hostingProvider.key': string;

src/plus/integrations/integration.ts

@@ -282,7 +282,16 @@ export abstract class IntegrationBase<
 
 	@log()
 	async syncCloudConnection(state: 'connected' | 'disconnected', forceSync: boolean): Promise<void> {
-		if (this._session?.cloud === false) return;
+		if (this._session?.cloud === false) {
+			if (this.id !== HostingIntegrationId.GitHub) {
+				this.container.telemetry.sendEvent('cloudIntegrations/refreshConnection/skippedUnusualToken', {
+					'integration.id': this.id,
+					reason: 'skip-non-cloud',
+					cloud: false,
+				});
+			}
+			return;
+		}
 
 		switch (state) {
 			case 'connected':
@@ -336,6 +345,12 @@ export abstract class IntegrationBase<
 			} catch (ex) {
 				Logger.error(ex, scope);
 			}
+		} else if (this._session?.expiresAt == null && this.id !== HostingIntegrationId.GitHub) {
+			this.container.telemetry.sendEvent('cloudIntegrations/refreshConnection/skippedUnusualToken', {
+				'integration.id': this.id,
+				reason: 'missing-expiry',
+				cloud: this._session?.cloud,
+			});
 		}
 	}