Refactor duplicated location-based LSP response handlers

[Copilot]

Addresses code duplication feedback: send_definition_result(), send_references_result(), and send_implementation_result() shared ~300 identical lines differing only in debug logging.

Changes

• Consolidated into single parameterized function: Created send_location_result(params, debug_name) that handles all location-based LSP responses
• Preserved debug logging: Optional debug_name parameter enables debug output for implementation handler while keeping definition/references handlers silent
• Reduced surface area: 168 net lines removed (201 deletions, 33 additions)

Implementation

// Generic handler for definition, references, and implementation
static void send_location_result(const char *params, const char *debug_name) {
    // Single implementation of location parsing and LSP response formatting
    // Conditional debug logging based on debug_name parameter
}

static void send_definition_result(const char *params) {
    send_location_result(params, NULL);  // No debug logging
}

static void send_implementation_result(const char *params) {
    send_location_result(params, "send_implementation_result");  // With debug logging
}

This eliminates the risk of inconsistent bug fixes and simplifies maintenance of LSP location handling.

---

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