refactor: move branchDataProvider declaration above tryFindRevealedResource closure

[Copilot]

tryFindRevealedResource closed over branchDataProvider which was declared after it, creating a TDZ hazard that was safe only due to call order — easy to break silently on refactor.

Changes

• src/vscodeUriHandler.ts (revealAzureResourceInExplorer):
  • Moved branchDataProvider declaration above tryFindRevealedResource so the captured variable is initialized before the closure is defined
  • Simplified resource to a single declaration + assignment: let resource = await tryFindRevealedResource()

// Before
const tryFindRevealedResource = async () => {
    return await branchDataProvider.findNodeById(fulId); // closes over var declared below ⚠️
};

let resource: TreeElement | undefined;
const branchDataProvider = ...;
resource = await tryFindRevealedResource();

// After
const branchDataProvider = ...;

const tryFindRevealedResource = async () => {
    return await branchDataProvider.findNodeById(fulId); // declared above ✓
};

let resource: TreeElement | undefined = await tryFindRevealedResource();

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.