anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-08-16 01:19:19 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

[PR #7176] feat(bash): load user shell RC files for alias support #12286

New Issue
Open
opened 2026-02-16 18:17:12 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 18:17:12 -05:00
Owner
Copy Link
Copy Source

Original Pull Request: https://github.com/anomalyco/opencode/pull/7176

State: open
Merged: No

Summary

  • Adds Shell.execute() unified function for shell command execution with RC file loading support
  • Bash tool now loads user shell RC files (.bashrc, .zshrc, etc.) enabling access to aliases
  • Refactors shell escape (! prefix) to use the same shared execution logic

Changes

src/shell/shell.ts

  • Added Shell.execute() function with options for loadRcFiles, timeout, abort, onOutput
  • Added getInvocationArgs() helper for shell-specific invocation patterns (zsh, bash, fish, nu, cmd, powershell)

src/tool/bash.ts

  • Replaced inline spawn logic with Shell.execute({ loadRcFiles: true })
  • Bash tool now has access to user aliases and shell functions
  • Added TERM: "dumb" for cleaner output

src/session/prompt.ts

  • Refactored shell() function to use Shell.execute()
  • Removed duplicated spawn logic

Behavior Changes

Feature Before After
Bash tool RC files Not loaded Loaded (aliases work)
Bash tool TERM Inherited "dumb"
Code duplication Separate implementations Shared Shell.execute()
**Original Pull Request:** https://github.com/anomalyco/opencode/pull/7176 **State:** open **Merged:** No --- ## Summary - Adds `Shell.execute()` unified function for shell command execution with RC file loading support - Bash tool now loads user shell RC files (`.bashrc`, `.zshrc`, etc.) enabling access to aliases - Refactors shell escape (`!` prefix) to use the same shared execution logic ## Changes **`src/shell/shell.ts`** - Added `Shell.execute()` function with options for `loadRcFiles`, `timeout`, `abort`, `onOutput` - Added `getInvocationArgs()` helper for shell-specific invocation patterns (zsh, bash, fish, nu, cmd, powershell) **`src/tool/bash.ts`** - Replaced inline spawn logic with `Shell.execute({ loadRcFiles: true })` - Bash tool now has access to user aliases and shell functions - Added `TERM: "dumb"` for cleaner output **`src/session/prompt.ts`** - Refactored `shell()` function to use `Shell.execute()` - Removed duplicated spawn logic ## Behavior Changes | Feature | Before | After | |---------|--------|-------| | Bash tool RC files | Not loaded | Loaded (aliases work) | | Bash tool TERM | Inherited | `"dumb"` | | Code duplication | Separate implementations | Shared `Shell.execute()` |
yindo added the pull-request label 2026-02-16 18:17:12 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
acp
bad
bug
discussion
docs
documentation
enhancement
extension
good first issue
growth
help-wanted
mcp
model-problem
needs:compliance
nix
opentui
perf
pull-request
Mirrored from GitHub Pull Request
 tui
web
windows
zen
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#12286
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?