refactor: consolidate copy reference into shared helper and differentiate titles

Extracts shared logic into copyReferenceInternal helper. Renames command
titles to "Copy Database Reference" and "Copy Collection Reference" to
avoid duplicate labels in VS Code command/keybinding lists.

Author: bgaeddert

l10n/bundle.l10n.json

@@ -986,7 +986,7 @@
   "The name must be between {0} and {1} characters.": "The name must be between {0} and {1} characters.",
   "The output window may contain additional information.": "The output window may contain additional information.",
   "The process exited prematurely.": "The process exited prematurely.",
-  "The reference has been copied to the clipboard": "The reference has been copied to the clipboard",
+  "The reference has been copied to the clipboard.": "The reference has been copied to the clipboard.",
   "The selected authentication method is not supported.": "The selected authentication method is not supported.",
   "The selected connection has been removed.": "The selected connection has been removed.",
   "The selected folder has been removed.": "The selected folder has been removed.",

package.json

@@ -538,13 +538,13 @@
         "//": "Copy Database Reference",
         "category": "DocumentDB",
         "command": "vscode-documentdb.command.copyDatabaseReference",
-        "title": "Copy Reference"
+        "title": "Copy Database Reference"
       },
       {
         "//": "Copy Collection Reference",
         "category": "DocumentDB",
         "command": "vscode-documentdb.command.copyCollectionReference",
-        "title": "Copy Reference"
+        "title": "Copy Collection Reference"
       }
     ],
     "submenus": [

src/commands/copyReference/copyReference.ts

@@ -9,22 +9,23 @@ import * as vscode from 'vscode';
 import { type CollectionItem } from '../../tree/documentdb/CollectionItem';
 import { type DatabaseItem } from '../../tree/documentdb/DatabaseItem';
 
-export async function copyDatabaseReference(_context: IActionContext, node: DatabaseItem): Promise<void> {
+async function copyReferenceInternal<T extends DatabaseItem | CollectionItem>(
+    node: T | undefined,
+    getReference: (node: T) => string,
+): Promise<void> {
     if (!node) {
         throw new Error(l10n.t('No node selected.'));
     }
 
-    const reference = node.databaseInfo.name;
+    const reference = getReference(node);
     await vscode.env.clipboard.writeText(reference);
-    void vscode.window.showInformationMessage(l10n.t('The reference has been copied to the clipboard'));
+    void vscode.window.showInformationMessage(l10n.t('The reference has been copied to the clipboard.'));
 }
 
-export async function copyCollectionReference(_context: IActionContext, node: CollectionItem): Promise<void> {
-    if (!node) {
-        throw new Error(l10n.t('No node selected.'));
-    }
+export async function copyDatabaseReference(_context: IActionContext, node: DatabaseItem): Promise<void> {
+    await copyReferenceInternal(node, (n) => n.databaseInfo.name);
+}
 
-    const reference = `${node.databaseInfo.name}.${node.collectionInfo.name}`;
-    await vscode.env.clipboard.writeText(reference);
-    void vscode.window.showInformationMessage(l10n.t('The reference has been copied to the clipboard'));
+export async function copyCollectionReference(_context: IActionContext, node: CollectionItem): Promise<void> {
+    await copyReferenceInternal(node, (n) => `${n.databaseInfo.name}.${n.collectionInfo.name}`);
 }