feat: add fish shell and PowerShell Core plugins

[arvindcr4]

Summary

Ports the ForgeCode ZSH shell plugin to fish shell and PowerShell Core (pwsh 7+), providing the same : command interface across all major shells.

Fish Shell Plugin

• Full feature parity: all 30+ commands with aliases
• fzf-powered tab completion for :commands and @file references
• Right prompt theme with zsh→fish color escape conversion
• Uses fish bind + commandline builtins, string collect for safe multi-line handling
• Auto-loads via conf.d/

PowerShell Core Plugin

• Full feature parity: all 30+ commands with aliases
• PSReadLine Enter/Tab key handlers (RevertLine+Insert+AcceptLine pattern)
• fzf-powered command and file completion
• Right prompt via ANSI cursor positioning
• Cross-platform: Windows, macOS, Linux (pwsh 7+)
• Background sync/update via Start-Job
• Cross-platform clipboard via Set-Clipboard with fallbacks

Files

File	Lines	Description
shell-plugin/fish/forge.fish	~1115	Fish: main plugin (47 functions, keybindings, dispatcher)
shell-plugin/fish/fish_right_prompt.fish	~98	Fish: right prompt with color escape conversion
shell-plugin/fish/README.md	~95	Fish: installation and usage docs
shell-plugin/powershell/ForgeCode.psm1	~1547	PowerShell: main module (all functions, PSReadLine handlers)
shell-plugin/powershell/ForgeCode.psd1	~72	PowerShell: module manifest
shell-plugin/powershell/README.md	~289	PowerShell: installation, usage, troubleshooting

Installation

Fish

cp shell-plugin/fish/forge.fish ~/.config/fish/conf.d/forge.fish
cp shell-plugin/fish/fish_right_prompt.fish ~/.config/fish/functions/fish_right_prompt.fish

PowerShell

# Copy to module path
Copy-Item -Recurse shell-plugin/powershell $HOME/.local/share/powershell/Modules/ForgeCode
# Add to $PROFILE
Add-Content $PROFILE 'Import-Module ForgeCode; Enable-ForgePlugin'

Notes

• Both plugins call `forge zsh doctor` and `forge zsh keyboard` since forge doesn't yet expose shell-agnostic subcommands — a follow-up could add generic alternatives
• The fish rprompt converter translates zsh `%F{...}`/`%f`/`%B`/`%b` escapes to fish `set_color` / PowerShell ANSI equivalents

Test plan

• Fish: verify `:` commands dispatch correctly
• Fish: verify tab completion for `:commands` and `@file`
• Fish: verify right prompt shows model/agent info
• Fish: test with vi-mode keybindings
• PowerShell: verify `Import-Module ForgeCode; Enable-ForgePlugin` loads cleanly
• PowerShell: verify `:` commands dispatch via PSReadLine
• PowerShell: verify tab completion with fzf
• PowerShell: test on Windows and Linux
• Both: test conversation switching with `:conversation -`

🤖 Generated with claude-flow

[CLAassistant]

All committers have signed the CLA.

[graphite-app]

Race condition in commands cache initialization. Multiple fish sessions for the same user could simultaneously detect the cache file as missing/empty and attempt to write to it concurrently, potentially corrupting the cache file.

function _forge_get_commands --description "Lazy-load command list from forge (outputs to stdout)"
    if not test -s "$_FORGE_COMMANDS_CACHE"
        set -l tmpfile (mktemp "$_FORGE_COMMANDS_CACHE.XXXXXX")
        CLICOLOR_FORCE=0 command $_FORGE_BIN list commands --porcelain 2>/dev/null >"$tmpfile"
        command mv -f "$tmpfile" "$_FORGE_COMMANDS_CACHE" 2>/dev/null
    end
    command cat "$_FORGE_COMMANDS_CACHE"
end

Use atomic write via temporary file + move to prevent corruption when multiple sessions initialize simultaneously.

Suggested change

	if not test -s "$_FORGE_COMMANDS_CACHE"
	CLICOLOR_FORCE=0 command $_FORGE_BIN list commands --porcelain 2>/dev/null >"$_FORGE_COMMANDS_CACHE"
	end
	command cat "$_FORGE_COMMANDS_CACHE"
	if not test -s "$_FORGE_COMMANDS_CACHE"
	set -l tmpfile (mktemp "$_FORGE_COMMANDS_CACHE.XXXXXX")
	CLICOLOR_FORCE=0 command $_FORGE_BIN list commands --porcelain 2>/dev/null >"$tmpfile"
	command mv -f "$tmpfile" "$_FORGE_COMMANDS_CACHE" 2>/dev/null
	end
	command cat "$_FORGE_COMMANDS_CACHE"

Spotted by Graphite



Is this helpful? React 👍 or 👎 to let us know.

[github-actions]

Action required: PR inactive for 5 days.
Status update or closure in 10 days.

[graphite-app]

The _forge_find_index function has a critical bug in its argument handling. When called with 2 arguments in pipe mode (e.g., cat file | _forge_find_index "value" 1), it incorrectly enters inline mode (line 136 checks if test (count $argv) -ge 2) but inline mode expects the first argument to be the output data, not the value to find.

This breaks multiple callers:

• Line 309: command cat $tmpfile | _forge_find_index "$current_provider" 1
• Line 341: command cat $tmpfile | _forge_find_index "$current_model" 1
• Line 609: command cat $tmpfile | _forge_find_index "$current_id" 1

Fix by checking if stdin is available before determining mode:

function _forge_find_index
    set -l value_to_find $argv[1]
    set -l field_number 1
    if test (count $argv) -ge 2
        set field_number $argv[2]
    end
    
    set -l index 1
    set -l line_num 0
    while read -l line
        set line_num (math $line_num + 1)
        if test $line_num -eq 1
            continue
        end
        set -l field_value (printf '%s' "$line" | awk "{print \$$field_number}")
        if test "$field_value" = "$value_to_find"
            echo $index
            return 0
        end
        set index (math $index + 1)
    end
    echo 1
    return 0
end

Suggested change

	function _forge_find_index --description "Find 1-based position of value in porcelain output (skipping header)"
	# Pipe mode: some_command | _forge_find_index <value> [field_number]
	# Inline mode: _forge_find_index <multiline_string> <value> [field_number]
	# Returns 1-based index after the header row. Defaults to 1 if not found.
	set -l value_to_find
	set -l field_number 1
	if test (count $argv) -ge 2
	# Inline: _forge_find_index "output" "value" [field]
	set -l output_data $argv[1]
	set value_to_find $argv[2]
	if test (count $argv) -ge 3
	set field_number $argv[3]
	end
	set -l index 1
	set -l line_num 0
	for line in (printf '%s\n' $output_data)
	set line_num (math $line_num + 1)
	if test $line_num -eq 1
	continue # skip header
	end
	set -l field_value (printf '%s' "$line" | awk "{print \$$field_number}")
	if test "$field_value" = "$value_to_find"
	echo $index
	return 0
	end
	set index (math $index + 1)
	end
	echo 1
	return 0
	else
	# Pipe mode: ... | _forge_find_index "value" [field]
	set value_to_find $argv[1]
	set -l index 1
	set -l line_num 0
	while read -l line
	set line_num (math $line_num + 1)
	if test $line_num -eq 1
	continue # skip header
	end
	set -l field_value (printf '%s' "$line" | awk "{print \$$field_number}")
	if test "$field_value" = "$value_to_find"
	echo $index
	return 0
	end
	set index (math $index + 1)
	end
	echo 1
	return 0
	end
	function _forge_find_index --description "Find 1-based position of value in porcelain output (skipping header)"
	# Pipe mode: some_command | _forge_find_index <value> [field_number]
	# Returns 1-based index after the header row. Defaults to 1 if not found.
	set -l value_to_find $argv[1]
	set -l field_number 1
	if test (count $argv) -ge 2
	set field_number $argv[2]
	end
	set -l index 1
	set -l line_num 0
	while read -l line
	set line_num (math $line_num + 1)
	if test $line_num -eq 1
	continue # skip header
	end
	set -l field_value (printf '%s' "$line" | awk "{print \$$field_number}")
	if test "$field_value" = "$value_to_find"
	echo $index
	return 0
	end
	set index (math $index + 1)
	end
	echo 1
	return 0
	end

Spotted by Graphite



Is this helpful? React 👍 or 👎 to let us know.