Skip to content

Conversation

elijah-potter
Copy link
Collaborator

Description

I noticed that, when Harper was used in Reddit, text would get duplicated when suggestions were applied. I've gone ahead and added a test case that replicates that environment and fixed the problem.

How Has This Been Tested?

Additional integration test + manual testing.

Checklist

  • I have performed a self-review of my own code
  • I have added tests to cover my changes

@elijah-potter elijah-potter changed the title fix(chrome-ext): duplicated test on Reddit fix(chrome-ext): duplication test on Reddit Oct 7, 2025
@elijah-potter elijah-potter merged commit 649d81c into master Oct 8, 2025
16 checks passed
@elijah-potter elijah-potter deleted the fix-reddit branch October 8, 2025 17:52
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Oct 17, 2025
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [Automattic/harper/harper-ls](https://github.com/Automattic/harper) | minor | `v0.67.0` -> `v0.68.0` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>Automattic/harper (Automattic/harper/harper-ls)</summary>

### [`v0.68.0`](https://github.com/Automattic/harper/releases/tag/v0.68.0)

[Compare Source](Automattic/harper@v0.67.0...v0.68.0)

#### What's Changed

- feat(core): rule to condense `miss spell` and friends by [@&#8203;elijah-potter](https://github.com/elijah-potter) in [#&#8203;2066](Automattic/harper#2066)
- feat(chrome-ext): enable by default on WordPress pages by [@&#8203;elijah-potter](https://github.com/elijah-potter) in [#&#8203;2065](Automattic/harper#2065)
- fix(chrome-ext): duplication test on Reddit by [@&#8203;elijah-potter](https://github.com/elijah-potter) in [#&#8203;2068](Automattic/harper#2068)
- build(deps): bump unicode-width from 0.2.1 to 0.2.2 by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2076](Automattic/harper#2076)
- fix(core): detect acronyms in `AnA` by [@&#8203;86xsk](https://github.com/86xsk) in [#&#8203;2072](Automattic/harper#2072)
- 2075 -- Harper Auto-submits Reddit Comments by [@&#8203;LoganBrinsmead](https://github.com/LoganBrinsmead) in [#&#8203;2079](Automattic/harper#2079)
- fix(typst): fix paragraph break spans by [@&#8203;86xsk](https://github.com/86xsk) in [#&#8203;2080](Automattic/harper#2080)
- feat(core): Add various new rules by [@&#8203;elijah-potter](https://github.com/elijah-potter) in [#&#8203;2084](Automattic/harper#2084)

#### New Contributors

- [@&#8203;LoganBrinsmead](https://github.com/LoganBrinsmead) made their first contribution in [#&#8203;2079](Automattic/harper#2079)

**Full Changelog**: <Automattic/harper@v0.67.0...v0.68.0>

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MS4xNDguNiIsInVwZGF0ZWRJblZlciI6IjQxLjE0OC42IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiXX0=-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant