jless is a command-line JSON viewer

Hi,

As you already know I've imported a package into pkgsrc. During the night I got an e-mail from Japan pointing out that the command name conficts with the existing Japanease version of less.

I need to rename the resulting binary during installation, which is no big deal but, thought I'd ask if you have any preferences regarding the name.

Just tried jless after seeing on hacker news -- would it be possible to add a way to output the path to a given element?

e.g.

This would be super helpful when inspecting through highly nested json packets.

This tool is glorious!

But, I found a problem. If you pipe info to it, say from an aws cli: aws iam list-instance-profiles | jless for example, I cannot search.

Hitting / immediately exits, which was surprising.

This is obviously a personal preference, but I strongly prefer using the same keyboard shortcut to both collapse and expand. It's much easier for me than splitting them into two separate shortcuts.

There's a hackernews comment asking for musl versions of this binary: https://news.ycombinator.com/item?id=30310461

Humble request: musl version

Keeping a glibc chroot directory takes up a fair bit of space, leaving less space available > for other stuff.

Whenever I try compiling Rust programs it exhausts the available space I have.

Hence I am resorting to begging, starting now (because this program sounds potentially > very useful.) Please folks, if possible, include musl versions in addition to glibc. :)

In the spirit of overdoing everything, this workflows file allows you to build a rust version of your binary for common tier 1 and tier targets, using rust-cross if necessary.

Feel free to tweak the file as necessary.

I forked the repo and ran the action on my fork, so you can see what passed and what didn't pass before using it:

You can see how the run fared here: https://github.com/Takashiidobe/jless/actions/runs/1833862676/attempts/1

And what versions of jless were created here (32 builds passed): https://github.com/Takashiidobe/jless/releases/tag/v0.7.1-test

Steps to reproduce

1. Open a terminal
2. Pass some data to jless
   • Enough that more than the screen height is needed to display
3. Resize the height of the window
4. Scroll to the bottom to observe the active line go off-screen

Info

• Reproduceable on current main (9ed0cbf1b702ba25107b6690c25e742c85e5cc10)
• $ uname -a: Linux ai 5.16.11-arch1-2 #1 SMP PREEMPT Mon, 28 Feb 2022 10:17:30 +0000 x86_64 GNU/Linux

Thanks for the great tool, and let me know if you want any more info! I use i3-gaps so resizing windows is pretty common (tiling WM and all)

Thank you for your great work! It is really enjoyable.

I found that the movement commands of vim below are not working. Is it an omission or by design?

[Control][b] - Move back one full screen
[Control][f] - Move forward one full screen
[Control][d] - Move forward 1/2 screen
[Control][u] - Move back (up) 1/2 screen

p.s. I know I can use PageUp and PageDown to achieve it but my keyboard doesn't have these keys😅

This is a regression from v0.7.1. jless seemingly no longer responds to SIGWINCH signals. Likely introduced in https://github.com/PaulJuliusMartinez/jless/pull/32.

After drilling down to a specific value, is it possible to copy the full path of that value? I can see it on the status line, but I can't see how you can extract the full path.

The installation command in the README for Arch linux is pacman -U jless, but jless is not (yet!) in the main arch repositories. The link to the AUR is correct, but the pacman command cannot install from the AUR.

Since using tools like yay/aurman etc. which do install from the AUR are not officially supported by arch, and arch users presumably know how to install from the AUR anyway, I'd recommend just keeping the link, and not giving an installation command, just a message like "Available in the AUR".

Love the tool!

I got a error on cargo build:

% sudo cargo install jless
... snip ...
error[E0658]: or-patterns syntax is experimental
   --> /Users/user1/.cargo/registry/src/github.com-1ecc6299db9ec823/jless-0.7.1/src/app.rs:101:26
    |
101 |                 KeyEvent(Key::Ctrl('c') | Key::Char('q')) => break,
    |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: see issue #54883 <https://github.com/rust-lang/rust/issues/54883> for more information
error[E0658]: or-patterns syntax is experimental
  --> /Users/user1/.cargo/registry/src/github.com-1ecc6299db9ec823/jless-0.7.1/src/jsonparser.rs:67:24
   |
67 |         while let Some(JsonToken::Whitespace | JsonToken::Newline) = self.peek_token_or_eof() {
   |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: see issue #54883 <https://github.com/rust-lang/rust/issues/54883> for more information
error: aborting due to 2 previous errors
For more information about this error, try `rustc --explain E0658`.
error: failed to compile `jless v0.7.1`, intermediate artifacts can be found at `/tmp/cargo-installoCXkLq`
Caused by:
  could not compile `jless`
To learn more, run the command again with --verbose.

How can I build this? Thank you for your help.

Hi,

thanks for jless, quite interesting tool, as a matter of endorsement i share the vulnerabiliities found in the jless ecosystem using osv-scanner https://github.com/google/osv-scanner

https://paste.rs/Zfa.rhtml

Mostly solves #84 No automatic detection of anything, but at least now (or starting next release) I'll be able to build it on servers :) Let me know what you think @PaulJuliusMartinez

For users with a non-standard keyboard (e.g. dvorak), the default vim key bindings are not great.

It would be awesome to be able to remap them, ideally in a config file.

Ideas:

• Either as letter-to-letter, e.g. "remap t j" makes "t" go down (like vim nnoremap)
• Or as letter to command name, e.g. "map t moveDown": overwrites the current (if any) shortcut for "t" to do the action "moveDown". There would be a default mapping for "j" to "moveDown".

Hey!

It would be nice to transparently handle compressed files, like json.gz or json.zstd. Either by guessing file type (with magic bytes?) or just by relying on the extension.

Thanks!

I'm back 😊

Clap 4 drops the indexmap dependency, which is presumably good for the compile time.

I also reformatted the longer help descriptions so they are cut off in short help mode jless -h, while also are printed in a more redable form for jless --help

This PR corrects misspellings identified by the check-spelling action.

The misspellings have been reported at https://github.com/jsoref/jless/commit/188d3fe6ea983390f86dea3f168a0199ddbae7db#commitcomment-88023872

The action reports that the changes in this PR would make it happy: https://github.com/jsoref/jless/commit/23b5c00f31f89e498921c8eec8c32d6190addbb1

Note: this PR does not include the action. If you're interested in running a spell check on every PR and push, that can be offered separately.