search

IlanCosman / tide

🌊 The ultimate Fish prompt.
MIT License
2.74k stars 103 forks source link

Using `tide configure --auto xxx` in `config.fish` causes tide prompt not showing immediately in new shell #461

Closed wrvsrx closed 7 months ago

wrvsrx commented 7 months ago

Describe the bug

Using tide configure --auto xxx in config.fish causes tide prompt not showing immediately in new shell. Prompt appears normally if I open fish by terminal (wezterm), this problem only occurs when open fish in another shell such as bash or fish.

Steps to reproduce

  1. I create a minimal example: tide-prompt.tar.gz
  2. Download the example file, extract to ~/.config/fish
  3. bash --norc, then execute fish in bash.
  4. Prompt doesn't appear until I press enter.

Screenshots

image

Environment

Output of tide bug-report: 

fish version: 3.6.1
tide version: 6.0.1
term: xterm-256color
os: NixOS
terminal emulator: wezterm
fish startup:
fisher plugins:

Additional context

IlanCosman commented 7 months ago

tide configure --auto is not intended to be used in config.fish. It sets universal variables, so you just need to run it once and it will persist across shell instances.

wrvsrx commented 7 months ago

I thought the tide configure --auto command was meant for declaring configuration (#304). Since it is not used for declarative configuration, do we still need to wait for #304 to be resolved?

IlanCosman commented 7 months ago

I thought the tide configure --auto command was meant for declaring configuration (#304). Since it is not used for declarative configuration, do we still need to wait for #304 to be resolved?

This just comes down to a semantics question over what 'declarative configuration' is. Perhaps I have the definition wrong 🤷‍♂️ . But the idea is that you run the command once, and there's no plan to change that.