Fix clang-format violations in node_based_edge.hpp

[Claude]

The CanCombineWith and operator< methods in NodeBasedEdgeAnnotation struct were not formatted according to clang-format-18 rules, causing CI failures.

Changes

• Applied clang-format-18 to include/extractor/node_based_edge.hpp
• Reformatted multi-line std::tie statements with proper indentation and line breaks

The formatting adjustments ensure the code follows the project's style conventions:

bool CanCombineWith(const NodeBasedEdgeAnnotation &other) const
{
    return (std::tie(string_view_id, classes, travel_mode, is_left_hand_driving) ==
            std::tie(other.string_view_id,
                     other.classes,
                     other.travel_mode,
                     other.is_left_hand_driving));
}

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Rename name_table class and name_id property</issue_title>
<issue_description>### Why rename
At this point the NameTable class stores a lot more than just name strings; it holds also refs, exit numbers, destinations, and pronunciations. Because it's then semantically confusing to do data.name_id != EMPTY_NAMEID checks (this checks that a name id's entry in the name table is 0, rather than checks that a way has no name data) (see #4642) name table and all associated classes, members, files (possibly) should be renamed.

To rename

• util::NameTable -> util::PreDatafacadeStringViewer
• NodeBasedEdgeAnnotation::NameID -> NodeBasedEdgeAnnotation::StringViewID
• name_id -> string_view_id
• EMPTY_NAMEID -> EMPTY_STRINGVIEWID

cc @daniel-j-h </issue_description>

Comments on the Issue (you are @claude[agent] in this section)

@danpat Yep

• Fixes Rename name_table class and name_id property #4709