:fmt doesn't used configured formatter

[emilyyyylime]

Summary

I tried to ask on Matrix, but got no answers. Basically I use clangd and clang-format. Running clang-format manually or formatting a range with = formats correctly. :fmt or the auto-format on save format incorrectly. Also see clangd/clangd#1926

Reproduction Steps

$ touch .clang-format
$ printf '#include "a.hpp"\n#include "b.hpp"\n' > b.cpp
$ clang-format b.cpp
#include "b.hpp"
#include "a.hpp"

However, in Helix, using :fmt formats with a.hpp first.

Relevant configs:

[language-server.clangd]
command = 'clangd'
args = ['--function-arg-placeholders', '--completion-style=detailed', '--clang-tidy', '--log=verbose']
[[language]]
name = 'cpp'
formatter = { command = 'clang-format', args = ['--style=file'] }
auto-format = true
language-servers = ['clangd', 'typos']

Helix log

~/.cache/helix/helix.log

``` 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] clangd version 16.0.6\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] Features: linux\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] PID: 199243\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] Working directory: /home/emilylime/hi\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] argv[0]: /usr/bin/clangd\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] argv[1]: --function-arg-placeholders\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] argv[2]: --completion-style=detailed\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] argv[3]: --clang-tidy\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] argv[4]: --log=verbose\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.616] User config file is /home/emilylime/.config/clangd/config.yaml\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] Starting LSP over stdin/stdout\n" 2024-02-06T09:31:10.616 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.616] <<< {\"id\":0,\"jsonrpc\":\"2.0\",\"method\":\"initialize\",\"params\":{\"capabilities\":{\"general\":{\"positionEncodings\":[\"utf-8\",\"utf-32\",\"utf-16\"]},\"textDocument\":{\"codeAction\":{\"codeActionLiteralSupport\":{\"codeActionKind\":{\"valueSet\":[\"\",\"quickfix\",\"refactor\",\"refactor.extract\",\"refactor.inline\",\"refactor.rewrite\",\"source\",\"source.organizeImports\"]}},\"dataSupport\":true,\"disabledSupport\":true,\"isPreferredSupport\":true,\"resolveSupport\":{\"properties\":[\"edit\",\"command\"]}},\"completion\":{\"completionItem\":{\"deprecatedSupport\":true,\"insertReplaceSupport\":true,\"resolveSupport\":{\"properties\":[\"documentation\",\"detail\",\"additionalTextEdits\"]},\"snippetSupport\":true,\"tagSupport\":{\"valueSet\":[1]}},\"completionItemKind\":{}},\"hover\":{\"contentFormat\":[\"markdown\"]},\"inlayHint\":{\"dynamicRegistration\":false},\"publishDiagnostics\":{\"versionSupport\":true},\"rename\":{\"dynamicRegistration\":false,\"honorsChangeAnnotations\":false,\"prepareSupport\":true},\"signatureHelp\":{\"signatureInformation\":{\"activeParameterSupport\":true,\"documentationFormat\":[\"markdown\"],\"parameterInformation\":{\"labelOffsetSupport\":true}}}},\"window\":{\"workDoneProgress\":true},\"workspace\":{\"applyEdit\":true,\"configuration\":true,\"didChangeConfiguration\":{\"dynamicRegistration\":false},\"didChangeWatchedFiles\":{\"dynamicRegistration\":true,\"relativePatternSupport\":false},\"executeCommand\":{\"dynamicRegistration\":false},\"fileOperations\":{\"didRename\":true,\"willRename\":true},\"inlayHint\":{\"refreshSupport\":false},\"symbol\":{\"dynamicRegistration\":false},\"workspaceEdit\":{\"documentChanges\":true,\"failureHandling\":\"abort\",\"normalizesLineEndings\":false,\"resourceOperations\":[\"create\",\"rename\",\"delete\"]},\"workspaceFolders\":true}},\"clientInfo\":{\"name\":\"helix\",\"version\":\"23.10 (dbac78b)\"},\"processId\":199232,\"rootPath\":\"/home/emilylime\",\"rootUri\":\"file:///home/emilylime\",\"workspaceFolders\":[{\"name\":\"emilylime\",\"uri\":\"file:///home/emilylime\"}]}}\n" 2024-02-06T09:31:10.617 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:10.617 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.616] <-- initialize(0)\n" 2024-02-06T09:31:10.617 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.617] --> reply:initialize(0) 0 ms\n" 2024-02-06T09:31:10.617 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.617] >>> {\"id\":0,\"jsonrpc\":\"2.0\",\"result\":{\"capabilities\":{\"astProvider\":true,\"callHierarchyProvider\":true,\"clangdInlayHintsProvider\":true,\"codeActionProvider\":{\"codeActionKinds\":[\"quickfix\",\"refactor\",\"info\"]},\"compilationDatabase\":{\"automaticReload\":true},\"completionProvider\":{\"resolveProvider\":false,\"triggerCharacters\":[\".\",\"<\",\">\",\":\",\"\\\"\",\"/\",\"*\"]},\"declarationProvider\":true,\"definitionProvider\":true,\"documentFormattingProvider\":true,\"documentHighlightProvider\":true,\"documentLinkProvider\":{\"resolveProvider\":false},\"documentOnTypeFormattingProvider\":{\"firstTriggerCharacter\":\"\\n\",\"moreTriggerCharacter\":[]},\"documentRangeFormattingProvider\":true,\"documentSymbolProvider\":true,\"executeCommandProvider\":{\"commands\":[\"clangd.applyFix\",\"clangd.applyTweak\"]},\"foldingRangeProvider\":true,\"hoverProvider\":true,\"implementationProvider\":true,\"inlayHintProvider\":true,\"memoryUsageProvider\":true,\"referencesProvider\":true,\"renameProvider\":{\"prepareProvider\":true},\"selectionRangeProvider\":true,\"semanticTokensProvider\":{\"full\":{\"delta\":true},\"legend\":{\"tokenModifiers\":[\"declaration\",\"definition\",\"deprecated\",\"deduced\",\"readonly\",\"static\",\"abstract\",\"virtual\",\"dependentName\",\"defaultLibrary\",\"usedAsMutableReference\",\"usedAsMutablePointer\",\"constructorOrDestructor\",\"userDefined\",\"functionScope\",\"classScope\",\"fileScope\",\"globalScope\"],\"tokenTypes\":[\"variable\",\"variable\",\"parameter\",\"function\",\"method\",\"function\",\"property\",\"variable\",\"class\",\"interface\",\"enum\",\"enumMember\",\"type\",\"type\",\"unknown\",\"namespace\",\"typeParameter\",\"concept\",\"type\",\"macro\",\"modifier\",\"operator\",\"comment\"]},\"range\":false},\"signatureHelpProvider\":{\"triggerCharacters\":[\"(\",\")\",\"{\",\"}\",\"<\",\">\",\",\"]},\"standardTypeHierarchyProvider\":true,\"textDocumentSync\":{\"change\":2,\"openClose\":true,\"save\":true},\"typeDefinitionProvider\":true,\"typeHierarchyProvider\":true,\"workspaceSymbolProvider\":true},\"serverInfo\":{\"name\":\"clangd\",\"version\":\"clangd version 16.0.6 linux x86_64-pc-linux-gnu\"}}}\n" 2024-02-06T09:31:10.617 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:10.617 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.617] <<< {\"jsonrpc\":\"2.0\",\"method\":\"initialized\",\"params\":{}}\n" 2024-02-06T09:31:10.618 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:10.618 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.617] <-- initialized\n" 2024-02-06T09:31:10.618 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.618] <<< {\"jsonrpc\":\"2.0\",\"method\":\"textDocument/didOpen\",\"params\":{\"textDocument\":{\"languageId\":\"cpp\",\"text\":\"#include \\\"b.hpp\\\"\\n#include \\\"a.hpp\\\"\\n\",\"uri\":\"file:///home/emilylime/hi/b.cpp\",\"version\":0}}}\n" 2024-02-06T09:31:10.618 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:10.618 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.618] <-- textDocument/didOpen\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.619] config note at /home/emilylime/.config/clangd/config.yaml:1:0: Parsing config fragment\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.619] config warning at /home/emilylime/.config/clangd/config.yaml:9:2: Unknown InlayHints key 'BlockEnd'\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.619] config note at /home/emilylime/.config/clangd/config.yaml:1:0: Parsed 1 fragments from file\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.619] Config fragment: compiling /home/emilylime/.config/clangd/config.yaml:1 -> 0x0000726588002910 (trusted=true)\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.619] --> textDocument/publishDiagnostics\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.619] >>> {\"jsonrpc\":\"2.0\",\"method\":\"textDocument/publishDiagnostics\",\"params\":{\"diagnostics\":[{\"message\":\"Unknown InlayHints key 'BlockEnd'\",\"range\":{\"end\":{\"character\":10,\"line\":8},\"start\":{\"character\":2,\"line\":8}},\"severity\":2,\"source\":\"clangd-config\"}],\"uri\":\"file:///home/emilylime/.config/clangd/config.yaml\"}}\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.619] Failed to find compilation database for /home/emilylime/hi/b.cpp\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.619] ASTWorker building file /home/emilylime/hi/b.cpp version 0 with command clangd fallback\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "[/home/emilylime/hi]\n" 2024-02-06T09:31:10.619 helix_lsp::transport [ERROR] clangd err <- "/usr/bin/clang -std=c++20 -Wall -Wextra -Wpedantic -Weverything -Wno-c++98-compat -Wno-c++98-compat-pedantic -resource-dir=/usr/lib/clang/16 -- /home/emilylime/hi/b.cpp\n" 2024-02-06T09:31:10.620 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.620] Driver produced command: cc1 -cc1 -triple x86_64-pc-linux-gnu -fsyntax-only -disable-free -clear-ast-before-backend -disable-llvm-verifier -discard-value-names -main-file-name b.cpp -mrelocation-model pic -pic-level 2 -pic-is-pie -mframe-pointer=all -fmath-errno -ffp-contract=on -fno-rounding-math -mconstructor-aliases -funwind-tables=2 -target-cpu x86-64 -tune-cpu generic -mllvm -treat-scalable-fixed-error-as-warning -debugger-tuning=gdb -fcoverage-compilation-dir=/home/emilylime/hi -resource-dir /usr/lib/clang/16 -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/13.2.1/../../../../include/c++/13.2.1 -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/13.2.1/../../../../include/c++/13.2.1/x86_64-pc-linux-gnu -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/13.2.1/../../../../include/c++/13.2.1/backward -internal-isystem /usr/lib/clang/16/include -internal-isystem /usr/local/include -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/13.2.1/../../../../x86_64-pc-linux-gnu/include -internal-externc-isystem /include -internal-externc-isystem /usr/include -Wall -Wextra -Wpedantic -Weverything -Wno-c++98-compat -Wno-c++98-compat-pedantic -std=c++20 -fdeprecated-macro -fdebug-compilation-dir=/home/emilylime/hi -ferror-limit 19 -stack-protector 2 -fgnuc-version=4.2.1 -fno-implicit-modules -fcxx-exceptions -fexceptions -no-round-trip-args -faddrsig -D__GCC_HAVE_DWARF2_CFI_ASM=1 -x c++ /home/emilylime/hi/b.cpp\n" 2024-02-06T09:31:10.620 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.620] Building first preamble for /home/emilylime/hi/b.cpp version 0\n" 2024-02-06T09:31:10.624 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.624] indexed preamble AST for /home/emilylime/hi/b.cpp version 0:\n" 2024-02-06T09:31:10.624 helix_lsp::transport [ERROR] clangd err <- " symbol slab: 0 symbols, 120 bytes\n" 2024-02-06T09:31:10.624 helix_lsp::transport [ERROR] clangd err <- " ref slab: 0 symbols, 0 refs, 128 bytes\n" 2024-02-06T09:31:10.624 helix_lsp::transport [ERROR] clangd err <- " relations slab: 0 relations, 24 bytes\n" 2024-02-06T09:31:10.624 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.624] Built preamble of size 217552 for file /home/emilylime/hi/b.cpp version 0 in 0.00 seconds\n" 2024-02-06T09:31:10.625 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.625] Indexing c++20 standard library in the context of /home/emilylime/hi/b.cpp\n" 2024-02-06T09:31:10.629 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.629] indexed file AST for /home/emilylime/hi/b.cpp version 0:\n" 2024-02-06T09:31:10.629 helix_lsp::transport [ERROR] clangd err <- " symbol slab: 0 symbols, 120 bytes\n" 2024-02-06T09:31:10.629 helix_lsp::transport [ERROR] clangd err <- " ref slab: 0 symbols, 0 refs, 128 bytes\n" 2024-02-06T09:31:10.629 helix_lsp::transport [ERROR] clangd err <- " relations slab: 0 relations, 24 bytes\n" 2024-02-06T09:31:10.629 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.629] Build dynamic index for main-file symbols with estimated memory usage of 248 bytes\n" 2024-02-06T09:31:10.629 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.629] --> textDocument/publishDiagnostics\n" 2024-02-06T09:31:10.629 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.629] >>> {\"jsonrpc\":\"2.0\",\"method\":\"textDocument/publishDiagnostics\",\"params\":{\"diagnostics\":[{\"code\":\"pp_file_not_found\",\"message\":\"'b.hpp' file not found\",\"range\":{\"end\":{\"character\":16,\"line\":0},\"start\":{\"character\":9,\"line\":0}},\"severity\":1,\"source\":\"clang\"}],\"uri\":\"file:///home/emilylime/hi/b.cpp\",\"version\":0}}\n" 2024-02-06T09:31:10.629 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:10.826 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.826] <<< {\"id\":1,\"jsonrpc\":\"2.0\",\"method\":\"textDocument/inlayHint\",\"params\":{\"range\":{\"end\":{\"character\":0,\"line\":2},\"start\":{\"character\":0,\"line\":0}},\"textDocument\":{\"uri\":\"file:///home/emilylime/hi/b.cpp\"}}}\n" 2024-02-06T09:31:10.826 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:10.826 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.826] <-- textDocument/inlayHint(1)\n" 2024-02-06T09:31:10.826 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.826] ASTWorker running InlayHints on version 0 of /home/emilylime/hi/b.cpp\n" 2024-02-06T09:31:10.826 helix_lsp::transport [ERROR] clangd err <- "I[09:31:10.826] --> reply:textDocument/inlayHint(1) 0 ms\n" 2024-02-06T09:31:10.827 helix_lsp::transport [ERROR] clangd err <- "V[09:31:10.826] >>> {\"id\":1,\"jsonrpc\":\"2.0\",\"result\":[]}\n" 2024-02-06T09:31:10.827 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:12.102 helix_lsp::transport [ERROR] clangd err <- "I[09:31:12.102] Indexed c++20 standard library: 14543 symbols, 3000 filtered\n" 2024-02-06T09:31:12.130 helix_lsp::transport [ERROR] clangd err <- "V[09:31:12.130] Build dynamic index for header symbols with estimated memory usage of 8542740 bytes\n" 2024-02-06T09:31:12.622 helix_lsp::transport [ERROR] clangd err <- "V[09:31:12.622] <<< {\"jsonrpc\":\"2.0\",\"method\":\"textDocument/didChange\",\"params\":{\"contentChanges\":[{\"range\":{\"end\":{\"character\":0,\"line\":1},\"start\":{\"character\":0,\"line\":0}},\"text\":\"\"},{\"range\":{\"end\":{\"character\":0,\"line\":1},\"start\":{\"character\":0,\"line\":1}},\"text\":\"#include \\\"b.hpp\\\"\\n\"}],\"textDocument\":{\"uri\":\"file:///home/emilylime/hi/b.cpp\",\"version\":1}}}\n" 2024-02-06T09:31:12.622 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:12.622 helix_lsp::transport [ERROR] clangd err <- "I[09:31:12.622] <-- textDocument/didChange\n" 2024-02-06T09:31:12.673 helix_lsp::transport [ERROR] clangd err <- "I[09:31:12.673] Failed to find compilation database for /home/emilylime/hi/b.cpp\n" 2024-02-06T09:31:12.673 helix_lsp::transport [ERROR] clangd err <- "I[09:31:12.673] ASTWorker building file /home/emilylime/hi/b.cpp version 1 with command clangd fallback\n" 2024-02-06T09:31:12.673 helix_lsp::transport [ERROR] clangd err <- "[/home/emilylime/hi]\n" 2024-02-06T09:31:12.673 helix_lsp::transport [ERROR] clangd err <- "/usr/bin/clang -std=c++20 -Wall -Wextra -Wpedantic -Weverything -Wno-c++98-compat -Wno-c++98-compat-pedantic -resource-dir=/usr/lib/clang/16 -- /home/emilylime/hi/b.cpp\n" 2024-02-06T09:31:12.674 helix_lsp::transport [ERROR] clangd err <- "V[09:31:12.674] Driver produced command: cc1 -cc1 -triple x86_64-pc-linux-gnu -fsyntax-only -disable-free -clear-ast-before-backend -disable-llvm-verifier -discard-value-names -main-file-name b.cpp -mrelocation-model pic -pic-level 2 -pic-is-pie -mframe-pointer=all -fmath-errno -ffp-contract=on -fno-rounding-math -mconstructor-aliases -funwind-tables=2 -target-cpu x86-64 -tune-cpu generic -mllvm -treat-scalable-fixed-error-as-warning -debugger-tuning=gdb -fcoverage-compilation-dir=/home/emilylime/hi -resource-dir /usr/lib/clang/16 -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/13.2.1/../../../../include/c++/13.2.1 -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/13.2.1/../../../../include/c++/13.2.1/x86_64-pc-linux-gnu -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/13.2.1/../../../../include/c++/13.2.1/backward -internal-isystem /usr/lib/clang/16/include -internal-isystem /usr/local/include -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/13.2.1/../../../../x86_64-pc-linux-gnu/include -internal-externc-isystem /include -internal-externc-isystem /usr/include -Wall -Wextra -Wpedantic -Weverything -Wno-c++98-compat -Wno-c++98-compat-pedantic -std=c++20 -fdeprecated-macro -fdebug-compilation-dir=/home/emilylime/hi -ferror-limit 19 -stack-protector 2 -fgnuc-version=4.2.1 -fno-implicit-modules -fcxx-exceptions -fexceptions -no-round-trip-args -faddrsig -D__GCC_HAVE_DWARF2_CFI_ASM=1 -x c++ /home/emilylime/hi/b.cpp\n" 2024-02-06T09:31:12.675 helix_lsp::transport [ERROR] clangd err <- "V[09:31:12.674] Rebuilding invalidated preamble for /home/emilylime/hi/b.cpp version 1 (previous was version 0)\n" 2024-02-06T09:31:12.686 helix_lsp::transport [ERROR] clangd err <- "V[09:31:12.686] indexed preamble AST for /home/emilylime/hi/b.cpp version 1:\n" 2024-02-06T09:31:12.686 helix_lsp::transport [ERROR] clangd err <- " symbol slab: 0 symbols, 120 bytes\n" 2024-02-06T09:31:12.686 helix_lsp::transport [ERROR] clangd err <- " ref slab: 0 symbols, 0 refs, 128 bytes\n" 2024-02-06T09:31:12.686 helix_lsp::transport [ERROR] clangd err <- " relations slab: 0 relations, 24 bytes\n" 2024-02-06T09:31:12.710 helix_lsp::transport [ERROR] clangd err <- "I[09:31:12.710] Built preamble of size 217552 for file /home/emilylime/hi/b.cpp version 1 in 0.04 seconds\n" 2024-02-06T09:31:12.713 helix_lsp::transport [ERROR] clangd err <- "V[09:31:12.713] indexed file AST for /home/emilylime/hi/b.cpp version 1:\n" 2024-02-06T09:31:12.713 helix_lsp::transport [ERROR] clangd err <- " symbol slab: 0 symbols, 120 bytes\n" 2024-02-06T09:31:12.713 helix_lsp::transport [ERROR] clangd err <- " ref slab: 0 symbols, 0 refs, 128 bytes\n" 2024-02-06T09:31:12.713 helix_lsp::transport [ERROR] clangd err <- " relations slab: 0 relations, 24 bytes\n" 2024-02-06T09:31:12.713 helix_lsp::transport [ERROR] clangd err <- "V[09:31:12.713] Build dynamic index for main-file symbols with estimated memory usage of 248 bytes\n" 2024-02-06T09:31:12.713 helix_lsp::transport [ERROR] clangd err <- "I[09:31:12.713] --> textDocument/publishDiagnostics\n" 2024-02-06T09:31:12.713 helix_lsp::transport [ERROR] clangd err <- "V[09:31:12.713] >>> {\"jsonrpc\":\"2.0\",\"method\":\"textDocument/publishDiagnostics\",\"params\":{\"diagnostics\":[{\"code\":\"pp_file_not_found\",\"message\":\"'a.hpp' file not found\",\"range\":{\"end\":{\"character\":16,\"line\":0},\"start\":{\"character\":9,\"line\":0}},\"severity\":1,\"source\":\"clang\"}],\"uri\":\"file:///home/emilylime/hi/b.cpp\",\"version\":1}}\n" 2024-02-06T09:31:12.713 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:15.582 helix_lsp::transport [ERROR] clangd err <- "V[09:31:15.582] <<< {\"id\":2,\"jsonrpc\":\"2.0\",\"method\":\"textDocument/inlayHint\",\"params\":{\"range\":{\"end\":{\"character\":0,\"line\":2},\"start\":{\"character\":0,\"line\":0}},\"textDocument\":{\"uri\":\"file:///home/emilylime/hi/b.cpp\"}}}\n" 2024-02-06T09:31:15.582 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:15.582 helix_lsp::transport [ERROR] clangd err <- "I[09:31:15.582] <-- textDocument/inlayHint(2)\n" 2024-02-06T09:31:15.583 helix_lsp::transport [ERROR] clangd err <- "V[09:31:15.582] ASTWorker running InlayHints on version 1 of /home/emilylime/hi/b.cpp\n" 2024-02-06T09:31:15.583 helix_lsp::transport [ERROR] clangd err <- "I[09:31:15.582] --> reply:textDocument/inlayHint(2) 0 ms\n" 2024-02-06T09:31:15.583 helix_lsp::transport [ERROR] clangd err <- "V[09:31:15.583] >>> {\"id\":2,\"jsonrpc\":\"2.0\",\"result\":[]}\n" 2024-02-06T09:31:15.583 helix_lsp::transport [ERROR] clangd err <- "\n" 2024-02-06T09:31:17.521 helix_core::syntax [ERROR] TS parser failed, disabling TS for the current buffer: Err(Cancelled) ```

Notably, logs don't include any communication between Helix and clangd regarding formatting of the file, so I don't know what Helix actually does to format the file.

Also: hx --health cpp:

Configured language servers:
  ✓ clangd: /usr/bin/clangd
  ✓ typos-lsp: /home/emilylime/.cargo/bin/typos-lsp
Configured debug adapter: lldb-vscode
Binary for debug adapter: /usr/bin/lldb-vscode
Configured formatter: clang-format
Binary for formatter: /usr/bin/clang-format
Highlight queries: ✓
Textobject queries: ✓
Indent queries: ✓

Platform

Arch Linux

Terminal Emulator

Kitty

Installation Method

source

Helix Version

helix 23.10 (dbac78b)

[the-mikedavis]

This is happening because we currently prefer external formatters (the formatter key in your configuration) to language server formatting (also see #9543). So we're essentially running:

$ printf '#include "a.hpp"\n#include "b.hpp"\n' | clang-format --style=file
#include "a.hpp"
#include "b.hpp"

for formatting on save or :fmt. = only uses language server formatting so it behaves as you expect.

Oddly, clang-format --style=file b.cpp behaves differently than cat b.cpp | clang-format --style=file.

You can remove your configuration for the external formatter and the auto-formatting should work as you expect.

[emilyyyylime]

Thanks for this. I opened llvm/llvm-project#81118 to try and resolve the underlying issue. I hadn't realised Helix was piping into the formatter command, but it does make much more sense.

[emilyyyylime]

On further exploration, I found that clang-format's behaviour makes perfect sense; the formatting is dependant on the name of the file being formatted, and clang-format can't know that if it's only being piped into. I think generally external formatters can format your code more intelligently if they can read the code in-context of your project, so giving a full path is perhaps something we would want to pursue, though I understand it complicates universal compatibility with formatters.
Shall I open an issue?

[the-mikedavis]

See #3134, #8834