Fixed _omb_cd_dirstack[0]: unbound variable. Fixes #716.
User description
Fixes #716
PR Type
Bug fix
Description
-
Fixed unbound variable error in directory stack handling
-
Added parameter expansion with default value to prevent bash error
Diagram Walkthrough
flowchart LR
A["Array Access"] -- "Add Default Value" --> B["Safe Parameter Expansion"]
B -- "Prevents" --> C["Unbound Variable Error"]
File Walkthrough
| Relevant files | |||
|---|---|---|---|
| Bug fix |
|
PR Reviewer Guide 🔍
Here are some key observations to aid the review process:
|
🎫 Ticket compliance analysis 🔶 716 - Partially compliant Compliant requirements:
Non-compliant requirements:
Requires further human verification:
|
| ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪ |
| 🧪 No relevant tests |
| 🔒 No security concerns identified |
⚡ Recommended focus areas for reviewBehavior Change
${_omb_cd_dirstack[0]-} preserves previous logic when the array is empty and does not inadvertently skip prepending $PWD or alter DIRSTACK semantics in edge cases. |
PR Code Suggestions ✨
No code suggestions found for the PR.
If we could get this reviewed that would be superduper so I can get back on master :)