Kyle Lippincott

If we wanted to be smarter about this by default, would the recommendation be to add a way in the template language to get the environment (to check `$TERM` for...

I wouldn't typically call it a customization point, so I'm fine with moving it into Rust, but it being in the template language and overridable has helped for users that...

> So `watch` cannot interpret hyperlink escape sequence? That's bad. I don't think the builtin hyperlink() function would peek terminal capability bypassing the formatter API. I assumed it would, actually....

> I'd prefer if hyperlink() stayed dumb and we just document such failures in the respectivre documentation. What's the motivation for that preference? For context, we've deployed it in our...

Sounds like we should add a config setting for osc8 hyperlinks. It should support three settings: always / never / auto. "Auto" will use the `is_stdout_terminal` func from #7688 to...

> Should that config (or a separate config) also decide how to render the links when osc8 is disabled? I was just going to display the URL. Markdown is an...

So I think it sounds like we're proposing something like this: ``` 'hyperlink_osc8(url, text)' = ''' concat( raw_escape_sequence("\e]8;;" ++ url ++ "\e\\"), text, raw_escape_sequence("\e]8;;\e\\"), ) ''' 'hyperlink_showurl(url, text)' = 'url'...

I should have mentioned that I'd also tried this on 1.38.0 wip, built from source at commit 1ca40481bb2992b22df02ec208fc70d563436407 (Thu May 29 23:44:49 2025 +0200). Just now, I just pulled and...

We just had a user encounter this with `jj file list`, and we didn't get a nice error message when we encountered it, just: ``` $ jj file list foo~bar...

I agree with the desire to rename `rebase` to something else, for the reasons stated in the original proposal and also because in my experience most git users use rebase...