Updated API discussion points

src/vscode-dts/vscode.proposed.debugVisualization.d.ts

@@ -37,6 +37,7 @@ declare module 'vscode' {
 		 *   the data provider, it will replace the variable in its tree.
 		 *   Otherwise, the items will be shown as children of the variable.
 		 */
+		// @API don't return TreeDataProvider but a reference to it, like its ids
 		visualization?: Command | TreeDataProvider<unknown>;
 
 		/**

src/vscode-dts/vscode.proposed.quickPickSortByLabel.d.ts

@@ -11,6 +11,9 @@ declare module 'vscode' {
 		/**
 		 * An optional flag to sort the final results by index of first query match in label. Defaults to true.
 		 */
+		// @API is a bug that we need this API at all. why do we change the sort order
+		// when extensions give us a (sorted) array of items?
+		// @API sortByLabel isn't a great name
 		sortByLabel: boolean;
 	}
 }

src/vscode-dts/vscode.proposed.testCoverage.d.ts

@@ -26,6 +26,9 @@ declare module 'vscode' {
 		 * @param token A cancellation token.
 		 * @return Coverage metadata for all files involved in the test.
 		 */
+		// @API - pass something into the provide method:
+		// (1) have TestController#coverageProvider: TestCoverageProvider
+		// (2) pass TestRun into this method
 		provideFileCoverage(token: CancellationToken): ProviderResult<T[]>;
 
 		/**
@@ -118,6 +121,9 @@ declare module 'vscode' {
 		);
 	}
 
+	// @API are StatementCoverage and BranchCoverage etc really needed
+	// or is a generic type with a kind-property enough
+
 	/**
 	 * Contains coverage information for a single statement or line.
 	 */