fix: support tilde expansion

[HasanAlrimawi]

This PR suggests possible solution for #26676

Changes

• Expand tilde to home directory whenever applicable (being mentioned as home directory).

[CLAassistant]

All committers have signed the CLA.

[dsherret]

Can you add this as a test case? I think it will fail: echo ~/test/~/test (should only expand ~/)

[dsherret]

Thanks! Let's shift this to happen when executing and not when parsing.

[dsherret]

I don't think we should do this while parsing, but instead during execution. The home directory in the shell might change while running. For example, the following test case should pass:

$ export HOME=test ; echo ~/test/~/test
test/test/~/test

[HasanAlrimawi]

@dsherret
I was able to solve the testcase you mentioned. However, in src/shell/execute.rs fn execute_sequential_list
When I try to destructure the item.sequence before execution, in order to check tildes occurrences for expansion, some values get partially moved, which leaves me forced to re-generate the sequence again and that may be overhead.
Any suggestions/better approaches in order to update the WordParts with tildes expanded? (e.g. another place of the code that may help with that, etc.)

[HasanAlrimawi]

@dsherret I was able to solve the testcase you mentioned. However, in src/shell/execute.rs fn execute_sequential_list When I try to destructure the item.sequence before execution, in order to check tildes occurrences for expansion, some values get partially moved, which leaves me forced to re-generate the sequence again and that may be overhead. Any suggestions/better approaches in order to update the WordParts with tildes expanded? (e.g. another place of the code that may help with that, etc.)

@dsherret
Any possible insights?

[dsherret]

Sorry, I wasn't sure what the solution was here until I went to go work on it myself. Basically, we needed to parse the tildes and then expand them during execution. See #146