Right now if you replace foo_bar by spam_eggs, the underscore won't get colored when printing the diff.

I've tried and fail to fix the algorithm, so maybe we should use an other diff library instead.

Maybe this one: https://docs.rs/diff/0.1.11/diff/index.html ?

Or we port diff-so-fancy from Perl to Rust :P

Instead of just using string.replace or re.replace_all, actually do the work ourselves and compute a list of input and output fragments

Each fragment is a struct containing some text (either added or removed) and some position

This allows use to display the exact diff.

Implementation notes:

• Replace line_patcher module by just replacer

• Compute the list of patters and replacements much earlier (and just one) when using --subvert

• Also, implement Train-Case (also known as Http-Case) handling

• Display the patch while the source files are read. This means less shuffling of data around

• Tweak output

Old:

Patching foo/bar.js --- old is old +++ new is new

New:

foo/bar.js 3: old is old foo/bar.js 3: new is new

• We no longer allocate n*n bytes when patching a line of size 'n', so the fix for issue #52 is no longer needed

Fix #15

While trying to apply "--ignored" option I discovered that my version is 0.4.1 while newest is 0.6.4. I can't install if from assets (macOS warns me about it), but will try to install from source. It would be much easier to me if you will update your brew package version. Thanks.

Steps to reproduce:

echo 'stuffStuff stuffstuff STUFFSTUFF' > f
ruplacer stuffStuff fooFoo --subvert

Obtained result: only the first "stuffStuff" is replaced Expected result: all three should be replaced

Hi,

Is there a way to have a pattern or replacement starting with dashes? Right now ruplacer think of them as invalid options. Would it be possible to add "explicit" options? Something like ruplacer <-p|--pattern> --my-pattern-starting-with-double-dashes <-r|--replacement> -my-replacement-with-only-one-dash-at-the-beginning ?

Hi ! First, thank for this amazing tool ! I use it almost on a daily basis :) Sometime ruplacer over-deliver replacement candidate, and I wold like to select only a subset of them.
Would it be possible to pipe the candidate into a selector like fzf to achieve this purpose ?

If not, could you give me some pointer on how to achieve it ?

I find it very helpful to have a tool tell me it's version so I can consider upgrading to a newer version or track down bugs.

This change adds the common --version option/switch (along with -V). Fortunately, Clap makes this very easy by using a magic command attribute: https://github.com/clap-rs/clap/blob/v3.1.15/examples/derive_ref/README.md#command-attributes

Output of ruplacer --version:

ruplacer 0.6.4

Additional output of ruplacer --help:

ruplacer 0.6.4

[…]

    -V, --version
            Print version information

[…]

Previously, FilePatcher could not differentiate between files with and without trailing newlines, and always wrote files with trailing newlines. As a result, files without trailing newlines that were otherwise unchanged would be modified.

This commit introduces a new LineIterator that preserves the trailing delimiter, if present, and ensures that files are written with the same newline structure they had when read.

Workaround for https://github.com/dmerejkowsky/ruplacer/issues/56. Instead of regex, I think glob pattern would be more suited because the ignore crate already supported it.

Hello, I'm Hayashi-Yudai.

I use your repository and learn from the codes. This time, I found very tiny typo in help message. I'm grad if you merge my pull request.

Sincerely.

Thanks for creating ruplacer! It's been great for renaming variables across source files.

Is there a way to replace matches with the empty string? For example,

ruplacer "string" ""

fails with

error: The following required arguments were not provided:
    <replacement>

I've been working around this by just adding an empty capture group () and then replacing with '$1'.

$ echo "hello world" > a.txt
$ echo "foo bar baz" > b.txt
$ chmod 400 b.txt
$ ruplacer 'hello' 'goodbye'
./a.txt:1 - hello world
./a.txt:1 + goodbye world

Would perform 1 replacement on 1 matching file
Re-run ruplacer with --go to write these changes to the filesystem
$ ruplacer 'hello' 'goodbye' --go
Error: Could not write ./b.txt

Caused by:
    Permission denied (os error 13)

Proposed tag: question.

I want to make all class {Name}Test to be internal class {Name}Test, so I try to run:

ruplacer 'class (\w+)Test' 'internal class $1Test'

But it doesn't work as expected. I avoided it with

ruplacer 'class (\w+)(Test)' 'internal class $1$2'

But how can solve my issue without adding second capturing group?

This (currently WIP) PR proposes switching to ignore's parallel walker API. It's not complete yet (threads are still only set to 1), but I figured that I should start discussion with y'all here upstream before I got too carried away, since there's actually quite a few design considerations here, and I haven't even validated that this is something y'all want in the first place! :)

See individual commits for more details on changes. Note that this is based on #93, and commits specifically for this PR start with the commit titled refactor: make `Stats` multithread-friendly.

Intended to resolve #51, pending discussion (there's some trade-offs!). See individual commits for more details on changes.

Open questions:

• [ ] What new tests should be added? Cases that I see:
  • [ ] The case mentioned in the first commit (refactor: skip duplicate canonicalized paths) is a good candidate.

Problem

I want to patch code of ignored files (dependencies of my python project). But there is no difference with or without --ignored - Error: nothing found to replace. At the same time ripgrep correctly finds provided pattern.

To reproduce

Libs used:

• python==3.10.2
• ripgrep==13.0.0 (or rg)
• ruplacer==0.6.4

It might be overcomplicated, but it is my real situation. Steps to reproduce:

mkdir reproduce
cd reproduce

git init
echo venv/ > .gitignore

python3 -m venv venv
source venv/bin/activate
pip install "Django==1.11.29"

# no any output since all files ignored by gitignore:
rg 'from collections import Iterable'
ruplacer 'from collections import Iterable' 'from collections.abc import Iterable'

# shows expected
rg 'from collections import Iterable' --no-ignore

# doesn't shows expected
ruplacer 'from collections import Iterable' 'from collections.abc import Iterable' --ignored