Skip to content

Fix unsound regex fast path, output path collisions, and drop-sink validation#47

Merged
eitamring merged 1 commit into
mainfrom
fix/regex-assertions-and-path-collisions
Jul 11, 2026
Merged

Fix unsound regex fast path, output path collisions, and drop-sink validation#47
eitamring merged 1 commit into
mainfrom
fix/regex-assertions-and-path-collisions

Conversation

@eitamring

@eitamring eitamring commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Three fixes from a pre-release audit. No performance changes.

  • The regex fast path turned fully-literal patterns into substring checks, dropping anchors and word boundaries: ^PING$ matched any line containing PING and could shadow later rules. The fast path now skips any pattern with a zero-width assertion. Regression tests plus a parity fuzz against plain regexp (1.3M execs).
    vet, test, and race pass; the 1M-line demo output is byte-identical to main.

@eitamring
eitamring force-pushed the fix/regex-assertions-and-path-collisions branch from f6c88d7 to fefe2af Compare July 11, 2026 02:59
@eitamring
eitamring merged commit dafb84a into main Jul 11, 2026
4 checks passed
@eitamring
eitamring deleted the fix/regex-assertions-and-path-collisions branch July 11, 2026 03:26
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