Recognize .hpp/.hxx/.h++ as C++ headers

[anshul-garg27]

Fixes #9387.

Description

language_by_filename in crates/languages/src/lib.rs recognizes .cpp, .cxx, .cc, .h, and .hh as C++, but not .hpp (or its rarer cousins .hxx and .h++). Modern C++ codebases — Boost, Qt, and most C++-only libraries — use .hpp precisely to distinguish C++ headers from plain C .h headers, so opening a Boost / .hpp header in Warp's editor today renders the "Language support is unavailable for this file type" footer instead of C++ syntax highlighting.

-        "cpp" | "cxx" | "cc" | "h" | "hh" => language_by_name("cpp"),
+        "cpp" | "cxx" | "cc" | "h" | "hh" | "hpp" | "hxx" | "h++" => language_by_name("cpp"),

Testing

• Added cpp_header_extensions_resolve_to_cpp in crates/languages/src/lib_tests.rs covering all three new extensions; fails on master, passes after the change.
• cargo fmt -p languages -- --check passes locally.
• Couldn't run `cargo nextest run -p languages` locally because the Metal toolchain isn't installed (same caveat as Expand ~ in warp://action/new_tab?path= URLs #9277, Recognize .command files as shell scripts #9345), so relying on CI for the full clippy / nextest pass.

Note on overlap with #9388

#9388 covers hpp | hxx | H (uppercase .H for case-sensitive filesystems); this PR covers hpp | hxx | h++. Happy to fold .H in here if the maintainers prefer landing a single combined PR — otherwise either landing first and a small follow-up for the missing extension works.

Changelog Entries for Stable

CHANGELOG-IMPROVEMENT: C++ headers using `.hpp`, `.hxx`, or `.h++` now open with C++ syntax highlighting in Warp's editor.

[oz-for-oss]

@anshul-garg27

I'm starting a first review of this pull request.

You can follow along in the session on Warp.

I approved this pull request. No matching stakeholder was found for the changed files, so no human reviewers were requested.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

I approved this pull request and requested human review from: @vorporeal, @alokedesai, @zachbai.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

[oz-for-oss]

Overview

This PR maps .hpp, .hxx, and .h++ file extensions to the existing C++ language support and adds a unit test covering the new extensions.

Concerns

• No blocking correctness, security, error handling, or performance concerns found in the changed lines.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

[oz-for-oss]

Overview

This PR maps .hpp, .hxx, and .h++ filename extensions to the existing C++ language support and adds a unit test covering those new header extensions.

Concerns

• No blocking correctness, security, error-handling, or performance concerns found in the changed lines.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

[anshul-garg27]

Heads-up — issue #9387 was filed earlier today for this exact bug, and #9388 covers a partly-overlapping set: hpp | hxx | H (uppercase .H for case-sensitive filesystems). This PR's set is hpp | hxx | h++. The two are easy to combine into hpp | hxx | h++ | H if helpful — happy to amend here, or fine to defer .H to a follow-up if the maintainers prefer keeping this PR as-is.

If this lands first it will close #9387 too (linked via the new "Fixes #9387" trailer once added). Pushed an updated body with the issue link.