Avoids hover registration on untrackable files

Improves logging for hovers

Author: eamodio

src/hovers/lineHoverController.ts

@@ -2,6 +2,7 @@ import type { CancellationToken, ConfigurationChangeEvent, Position, TextDocumen
 import { Disposable, Hover, languages, Range, window } from 'vscode';
 import type { Container } from '../container';
 import { configuration } from '../system/-webview/configuration';
+import { isTrackableTextEditor } from '../system/-webview/vscode/editors';
 import { debug } from '../system/decorators/log';
 import { once } from '../system/event';
 import { Logger } from '../system/logger';
@@ -81,6 +82,8 @@ export class LineHoverController implements Disposable {
 			1: position => `${position.line}:${position.character}`,
 			2: false,
 		},
+		exit: r => (r != null ? 'provided' : 'skipped'),
+		singleLine: true,
 	})
 	async provideDetailsHover(
 		document: TextDocument,
@@ -141,6 +144,8 @@ export class LineHoverController implements Disposable {
 			1: position => `${position.line}:${position.character}`,
 			2: false,
 		},
+		exit: r => (r != null ? 'provided' : 'skipped'),
+		singleLine: true,
 	})
 	async provideChangesHover(
 		document: TextDocument,
@@ -197,7 +202,7 @@ export class LineHoverController implements Disposable {
 	private register(editor: TextEditor | undefined) {
 		this.unregister();
 
-		if (editor == null) return;
+		if (editor == null || !isTrackableTextEditor(editor)) return;
 
 		const cfg = configuration.get('hovers');
 		if (!cfg.enabled || !cfg.currentLine.enabled || (!cfg.currentLine.details && !cfg.currentLine.changes)) return;

src/trackers/lineTracker.ts

@@ -169,7 +169,7 @@ export class LineTracker {
 			return isIncluded(lineOrSelections, this._selections);
 		}
 
-		if (this._selections == null || this._selections.length === 0) return false;
+		if (!this._selections?.length) return false;
 
 		const line = lineOrSelections;
 		const activeOnly = options?.activeOnly ?? true;