Skip to content

fix(completion): stop fish from writing debug logs#22

Closed
jeet-dekivadia wants to merge 1 commit into
openai:mainfrom
jeet-dekivadia:codex/suppress-fish-completion-stderr
Closed

fix(completion): stop fish from writing debug logs#22
jeet-dekivadia wants to merge 1 commit into
openai:mainfrom
jeet-dekivadia:codex/suppress-fish-completion-stderr

Conversation

@jeet-dekivadia

Copy link
Copy Markdown

Summary

  • discard stderr from fish's internal completion probe instead of appending it to /tmp/fish-debug.log

Why

The fish completion script currently writes diagnostics from every tab-completion invocation to a fixed temporary file. That leaves an unbounded, cross-session debug log behind during normal CLI usage and can retain command diagnostics unexpectedly. Bash already suppresses the same internal probe's stderr; fish should do the same.

Validation

  • git diff --check
  • go test ./internal/autocomplete -count=1
  • verified the embedded fish completion script no longer references fish-debug

Signed-off-by: Jeet Dekivadia <jeet.university@gmail.com>
@jeet-dekivadia jeet-dekivadia requested a review from a team as a code owner May 30, 2026 10:25
@jeet-dekivadia

Copy link
Copy Markdown
Author

Closing in favor of #21, which now includes this fish stderr fix together with focused regression coverage so reviewers have one cohesive completion-hygiene patch.

@jeet-dekivadia jeet-dekivadia deleted the codex/suppress-fish-completion-stderr branch May 30, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant