What edge cases are covered in the tests, especially around hyphens?

Question

RepoPilot · Accepted Answer

The tests cover two important edge cases around hyphens:

1. **Basic hyphen escaping** [test.js:L10-L14]: The test verifies that hyphens are escaped as `\x2d` rather than a simple backslash escape:
   ```
   escapeStringRegexp('foo - bar') === 'foo \x2d bar'
   ```

2. **Unicode flag compatibility** [test.js:L16-L20]: A second test ensures that the escaped hyphen works correctly when used with the Unicode flag:
   ```
   new RegExp(escapeStringRegexp('-'), 'u')
   ```
   This regex should match the hyphen character.

These tests exist because hyphens have special meaning in character classes and require careful handling. The implementation uses the `\x2d` hexadecimal escape form [index.js:L11] rather than a simple backslash escape. According to the code comments [index.js:L7-L8], this is done because "a `\xnn` escape [is needed] when the simpler form would be disallowed by Unicode patterns' stricter grammar."

The README further notes [readme.md:L24] that this approach "escapes `-` in a way compatible with PCRE" and "compatible with the Unicode flag," which aligns with what the tests validate.

FOLLOW_UPS:
- What other RegExp special characters are escaped?
- Why is type checking important for this library?
- How does this library differ from native RegExp.escape()?