Skip to content

feat: accept --slave and --no-echo R-compatible CLI flags#109

Merged
eitsupi merged 1 commit intomainfrom
feat/accept-slave-no-echo-flags
Mar 7, 2026
Merged

feat: accept --slave and --no-echo R-compatible CLI flags#109
eitsupi merged 1 commit intomainfrom
feat/accept-slave-no-echo-flags

Conversation

@eitsupi
Copy link
Copy Markdown
Owner

@eitsupi eitsupi commented Mar 7, 2026

Summary

  • Add --slave and --no-echo as R-compatible no-op CLI flags
  • --slave is fully hidden (deprecated in R 4.0)
  • --no-echo is shown in long help (--help) only
  • Prevents errors when tools like R.nvim pass these flags to arf

Background

A user reported that arf fails when launched with --slave from R.nvim.
radian had the same issue, causing multiple bug reports.
arf already accepts several R-compatible no-op flags (--interactive, --no-readline, --no-restore-history), so this follows the same pattern.

Test plan

  • cargo clippy passes
  • CLI snapshot tests updated and passing
  • CLI integration tests passing

🤖 Generated with Claude Code

Add --slave (hidden) and --no-echo (shown in long help) as no-op
R-compatible flags. This prevents errors when tools like R.nvim
pass these flags to arf.

--slave was deprecated in R 4.0 and replaced by --no-echo.
Both are no-op in arf since it controls its own echo behavior.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@eitsupi eitsupi changed the title feat: accept --slave and --no-echo R-compatible CLI flags feat: accept --slave and --no-echo R-compatible CLI flags Mar 7, 2026
@eitsupi eitsupi merged commit cc4a384 into main Mar 7, 2026
10 checks passed
@eitsupi eitsupi deleted the feat/accept-slave-no-echo-flags branch March 7, 2026 13:51
@adamhsparks
Copy link
Copy Markdown

Thanks for the quick response! Looking forward to this being merged.

@eitsupi
Copy link
Copy Markdown
Owner Author

eitsupi commented Mar 7, 2026

It's already merged, so you should be able to install it using cargo.

It's such a small change that I'm hesitant to make a new release just for that.

@adamhsparks
Copy link
Copy Markdown

I typed it at the same time you merged it. :)

I've pulled it down via Cargo (my usual way of updating). I get another error. I'll open an issue.

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.

2 participants