Merge pull request #909 from mikesmithgh/fix-man-pager

fix(man): Use cat as man pager

Author: skovhus

server/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Bash Language Server
 
+## 4.10.3
+
+- Use cat as man pager https://github.com/bash-lsp/bash-language-server/pull/909
+
+## 4.10.2
+
+- Bump semver development dependency causing false positive distributions security warnings https://github.com/bash-lsp/bash-language-server/pull/905
+
+
 ## 4.10.1
 
 - Handle tree-sitter-bash parse errors gracefully

server/package.json

@@ -3,7 +3,7 @@
   "description": "A language server for Bash",
   "author": "Mads Hartmann",
   "license": "MIT",
-  "version": "4.10.2",
+  "version": "4.10.3",
   "main": "./out/server.js",
   "typings": "./out/server.d.ts",
   "bin": {

server/src/util/sh.ts

@@ -53,7 +53,7 @@ const WORDS_WITHOUT_DOCUMENTATION = new Set([
 ])
 
 /**
- * Get documentation for the given word by usingZZ help and man.
+ * Get documentation for the given word by using help and man.
  */
 export async function getShellDocumentationWithoutCache({
   word,
@@ -73,7 +73,7 @@ export async function getShellDocumentationWithoutCache({
     // We have experimented with setting MANWIDTH to different values for reformatting.
     // The default line width of the terminal works fine for hover, but could be better
     // for completions.
-    { type: 'man', command: `man ${word} | col -bx` },
+    { type: 'man', command: `man -P cat ${word} | col -bx` },
   ]
 
   for (const { type, command } of DOCUMENTATION_COMMANDS) {