Refactor MCP sample to use class-level Scanner

[Copilot]

Addresses code review feedback to eliminate the @SuppressWarnings("resource") annotation by refactoring Scanner initialization from method-level to class-level scope.

Changes

• Moved Scanner instantiation from getUserApproval() to a static final field CONSOLE_SCANNER
• Removed resource leak suppression warning
• Added documentation clarifying that System.in Scanner should remain open for application lifetime

Before

private static boolean getUserApproval() {
    @SuppressWarnings("resource")
    Scanner scanner = new Scanner(System.in);
    while (true) {
        System.out.print("Approve MCP call? (y/n): ");
        String input = scanner.nextLine().trim().toLowerCase();
        // ...
    }
}

After

// Scanner for user input (reused across approval requests)
// This Scanner wraps System.in and should not be closed to allow continued console input
private static final Scanner CONSOLE_SCANNER = new Scanner(System.in);

private static boolean getUserApproval() {
    while (true) {
        System.out.print("Approve MCP call? (y/n): ");
        String input = CONSOLE_SCANNER.nextLine().trim().toLowerCase();
        // ...
    }
}

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.