navi --tldr not working even with rust client tlrc

[TwiggyWan]

hi
Describe the bug

navi --tldr says :

    
    Note:
    Please make sure you're using a version that supports the --markdown flag.
    If you are already using a supported version you can ignore this message. 
    The tldr client written in C (the default one in Homebrew) doesn't support markdown files, so navi can't use it.
    The client written in Rust is recommended. The one available in npm works, too.

So I installed tlrc, expecting it to work. but it doesnt have a --markdown flag.

To Reproduce
Steps to reproduce the behavior:

brew install navi
brew install tlrc
navi --tldr docker

Expected behavior
According to the hint message navi --tldr should be supported but it is not. Please consider to change the hint text.

Versions:

• OS: [e.g. macOS, WSL ubuntu, ubuntu] : debian bookworm, navi 2.23.0 & tlrc (v1.9.2 (implementing the tldr client specification v2.2)) installed through homebrew
• Shell Version [replace this text with the output of sh --version]: GNU bash, version 5.2.15(1)-release (x86_64-pc-linux-gnu)

Additional context
thanks

[welcome]

Thanks for opening your first issue here! In case you're facing a bug, please update navi to the latest version first. Maybe the bug is already solved! :)

[OlaoluwaM]

Same issue with v2.23.0

[denisidoro]

The rust client I tested years ago was tealdeer. I haven't tested tlrc yet.

[vakili]

The issue also occurs on my system with tealdeer (latest version, 1.6.1). I believe the reason is that navi attempts to pass the --markdown flag to the tldr client, and this flag is not supported. The working flag is --raw, as per the man page of tlrc. This issue is present even when the tealdeer option is set to true in the navi config, which as I understand is supposed to enforce use the --raw flag.

[miyl]

Regarding tealdeer it was supposedly fixed in #873, where the config option was added, however this commit was added in January this year, while the latest release of navi was December last year, so the fix isn't out yet, sadly.

Well, unless you build your version of navi from the master branch.

Hopefully a new version is soon released.

[TwiggyWan]

Hi, in that case i'm happy to consider this issue as resolved if the docs make it a bit clearer which tldr client to install, which is what I tried to do with pr #916 .

thanks