🔥 feat: Add support for configuring the Regex engine on the router by Claude · Pull Request #4254 · gofiber/fiber

Claude · 2026-05-04T22:40:45Z

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

This section details on the original issue you should resolve

<issue_title>📝 [Proposal]: ability to use alternative regex for router</issue_title>
<issue_description>### Feature Proposal Description

This is a substantially faster assembly optimised regex package that is drop in replacement: https://github.com/coregx/coregex

Alignment with Express API

Na

HTTP RFC Standards Compliance

Na

API Stability

Na

Feature Examples
Na
Checklist:

I agree to follow Fiber's Code of Conduct.

I have searched for existing issues that describe my proposal before opening this one.

I understand that a proposal that does not meet these guidelines may be closed without explanation.</issue_description>

Comments on the Issue (you are @claude[agent] in this section)

- Created RegexEngine and RegexCompiler interfaces - Updated path.go to use configured regex engine - Added RegexEngine field to Config with DefaultRegexEngine - Updated router.go to pass RegexEngine to parseRoute - Added comprehensive tests for regex abstraction - Created example showing how to integrate coregex Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/3e2b7990-e9c5-41f6-afbf-ad29c8bbcfd0 Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

codecov · 2026-05-04T22:58:32Z

Codecov Report

❌ Patch coverage is 92.30769% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.14%. Comparing base (7bfb06a) to head (4d138de).

Files with missing lines	Patch %	Lines
path.go	88.88%	2 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4254      +/-   ##
==========================================
- Coverage   91.16%   91.14%   -0.02%     
==========================================
  Files         127      127              
  Lines       12605    12641      +36     
==========================================
+ Hits        11491    11522      +31     
- Misses        697      701       +4     
- Partials      417      418       +1

Flag	Coverage Δ
unittests	`91.14% <92.30%> (-0.02%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/3e2b7990-e9c5-41f6-afbf-ad29c8bbcfd0 Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

- Added regex-engines.md guide with coregex integration examples - Documented RegexEngine and RegexCompiler interfaces - Included performance benchmarks and use cases - Added custom implementation examples Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/3e2b7990-e9c5-41f6-afbf-ad29c8bbcfd0 Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

gaby · 2026-05-04T23:25:21Z

@claude[agent]

fix markdownlint issues
remove examples folder

- Removed examples/regex-engine folder as requested - Updated docs/guide/regex-engines.md to remove reference to examples folder - All markdown linting issues resolved Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/9e71d891-0cc3-4786-8921-d994e500db6f Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

Claude · 2026-05-04T23:28:30Z

@claude[agent]

fix markdownlint issues

remove examples folder

Fixed in d7629bf:

Removed examples/regex-engine folder
Fixed markdown linting issues in docs/guide/regex-engines.md
All markdown lint checks now pass

gaby · 2026-05-05T02:04:19Z

@claude[agent] I'm not sure about the addition of regex.go. It seems like it might be over-engineered.

I believe the approach should be as straightforward as our JSONEncoder and Decoder implementations within the app configuration.

Moved regex types from separate regex.go file into path.go where they are used. This follows the simpler pattern similar to JSONEncoder/Decoder and reduces over-engineering by keeping related code together. Changes: - Removed regex.go and moved RegexEngine/RegexCompiler types to path.go - Moved tests from regex_test.go to path_test.go - All tests pass, no linting issues Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/175129f3-3f03-4a29-9673-b0521797d587 Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

Claude · 2026-05-05T02:10:28Z

@claude[agent] I'm not sure about the addition of regex.go. It seems like it might be over-engineered.

I believe the approach should be as straightforward as our JSONEncoder and Decoder implement...

Simplified in ffc10c3. Removed the separate regex.go file and moved all regex types directly into path.go where they're actually used. This follows a simpler pattern similar to other encoders/decoders and keeps related code together. The abstraction is now more minimal while still providing the flexibility to use alternative regex engines.

gaby · 2026-05-05T02:27:29Z

@claude[agent] Looking at the guide docs/guide/regex-engines.md and code. Using a different regex library is too complicated. We can simplify this.

Following feedback to simplify the regex engine abstraction, this commit refactors the implementation to use a simple function type instead of interface-based approach. Key changes: - Replace RegexEngine interface with RegexHandler function type - RegexHandler is now: func(pattern string) RegexCompiler - Add defaultRegexHandler function that wraps regexp.MustCompile - Update all references from RegexEngine to RegexHandler - Simplify documentation to show direct usage without adapter types This follows the same pattern as JSONEncoder/JSONDecoder in the codebase, making it much simpler for users to adopt alternative regex engines: Before (required adapter types): type CoregexEngine struct{} func (CoregexEngine) MustCompile(pattern string) fiber.RegexCompiler {...} app := fiber.New(fiber.Config{RegexEngine: CoregexEngine{}}) After (just a function): app := fiber.New(fiber.Config{ RegexHandler: func(pattern string) fiber.RegexCompiler { return coregex.MustCompile(pattern) }, }) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

gaby · 2026-05-05T04:48:10Z

@claude[agent] Why are you adding a handler, instead of just using coregex.MustCompile. Stdlib and coregex use the same API.

// stdlib
re := regexp.MustCompile(pattern)

// coregex — same API
re := coregex.MustCompile(pattern)

The app config option should just take coregex.MustCompile, i think?

Copilot

Pull request overview

This PR adds a new fiber.Config.RegexHandler extension point so route regex() constraints can be compiled with a configurable regex engine instead of always using the standard library. It fits into Fiber’s router/path parsing layer by threading the configured compiler through route registration and RoutePatternMatch.

Changes:

Add Config.RegexHandler with a default of regexp.MustCompile, and pass it into route parsing during route registration and prefixing.
Generalize regex constraint compilation in path.go so compiled regex matchers are stored behind an interface.
Add tests and a new guide page describing alternative regex engine usage, plus a number of formatting-only updates in unrelated files.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
router.go	Passes `app.config.RegexHandler` into route parsing during registration/prefixing.
router_test.go	Updates direct `parseRoute` call sites to the new signature.
prefork.go	Formatting-only change.
path.go	Introduces `RegexHandler`/`RegexCompiler` and uses them for `regex()` constraints and `RoutePatternMatch`.
path_test.go	Updates parser tests and adds new coverage for configurable regex handlers.
middleware/logger/default_logger.go	Formatting-only change.
middleware/envvar/envvar_test.go	Formatting-only change.
middleware/compress/compress.go	Formatting-only change.
helpers.go	Formatting-only change.
helpers_test.go	Formatting-only change.
domain_test.go	Formatting-only change.
docs/guide/regex-engines.md	Adds user-facing documentation for alternative regex engines.
ctx_test.go	Formatting-only change.
client/request_test.go	Formatting-only change.
client/client_test.go	Formatting-only change.
app.go	Adds the new public `Config.RegexHandler` field and its default initialization.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

gaby · 2026-05-06T02:15:03Z

@copilot apply changes based on the comments in this thread

Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/f9b8dbc7-cc72-4f60-b345-5b668e0ceed6 Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

Copilot · 2026-05-06T02:28:10Z

@copilot apply changes based on the comments in this thread

Applied the review-thread follow-ups in 4d138de. RegexHandler is now validated up front, the matcher contract is reduced to MatchString, failure-mode coverage was added, and the API/guide docs were updated to document the final behavior and concurrency requirement. make lint, make test, and make markdown pass; make audit is still blocked by existing Go 1.25 stdlib govulncheck findings in the environment.

Copilot

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 2 comments.

+	if handlerType.Kind() != reflect.Func || handlerValue.IsNil() {
+		panic("fiber: Config.RegexHandler must be a non-nil function")
+	}
+	if handlerType.NumIn() != 1 || handlerType.In(0).Kind() != reflect.String || handlerType.NumOut() != 1 {
+		panic("fiber: Config.RegexHandler must have signature func(string) T")


 // segment when matching incoming requests.
 type Constraint struct {
-	RegexCompiler     *regexp.Regexp
+	RegexCompiler     RegexCompiler


gaby · 2026-05-06T04:22:43Z

@copilot apply changes based on the comments in this thread

Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/bb345d14-e46f-4eb3-80d3-76fb16369d8d Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

Copilot · 2026-05-06T04:32:14Z

@copilot apply changes based on the comments in this thread

Applied the follow-up changes in 67f902d. RegexHandler now enforces the exact documented func(string) T signature so named-string inputs fail fast, and Constraint.RegexCompiler is back to *regexp.Regexp for compatibility while non-stdlib matchers are stored internally. Targeted regex tests pass, and the full local validation set passes except for the same pre-existing local make audit govulncheck/toolchain findings; the current branch CI workflows are green.

Initial plan

57076e1

Claude AI assigned Claude and gaby May 4, 2026

Claude AI linked an issue May 4, 2026 that may be closed by this pull request

📝 [Proposal]: ability to use alternative regex for router #4253

Open

3 tasks

Claude started work on behalf of gaby May 4, 2026 22:40 View session

ReneWerner87 added v3 ✏️ Feature labels May 4, 2026

ReneWerner87 added this to v3 May 4, 2026

ReneWerner87 added this to the v3 milestone May 4, 2026

Claude AI and others added 2 commits May 4, 2026 23:03

Fix linting issues in regex implementation

5bc2ae9

Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/3e2b7990-e9c5-41f6-afbf-ad29c8bbcfd0 Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

Claude finished work on behalf of gaby May 4, 2026 23:06

Claude AI requested a review from gaby May 4, 2026 23:06

Claude started work on behalf of gaby May 4, 2026 23:25 View session

Claude finished work on behalf of gaby May 4, 2026 23:29

Claude started work on behalf of gaby May 5, 2026 02:04 View session

Claude finished work on behalf of gaby May 5, 2026 02:11

Claude started work on behalf of gaby May 5, 2026 02:27 View session

Claude finished work on behalf of gaby May 5, 2026 02:40

Claude finished work on behalf of gaby May 5, 2026 04:56

Merge branch 'main' into claude/add-alternative-regex-support

62a78b7

gaby changed the title ~~[WIP] Add ability to use alternative regex for router~~ 🔥 feat: Add support for configuring the Regex engine on the router May 6, 2026

gaby marked this pull request as ready for review May 6, 2026 02:06

gaby requested a review from a team as a code owner May 6, 2026 02:06

gaby requested review from ReneWerner87, Copilot, efectn and sixcolors May 6, 2026 02:06

Copilot started reviewing on behalf of gaby May 6, 2026 02:06 View session

Copilot AI reviewed May 6, 2026

View reviewed changes

gaby and others added 2 commits May 5, 2026 22:14

Potential fix for pull request finding

9cb6672

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

8a47f89

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot started work on behalf of gaby May 6, 2026 02:15 View session

Copilot AI and others added 3 commits May 6, 2026 02:22

fix: validate and document regex handler config

78bb6f8

Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/f9b8dbc7-cc72-4f60-b345-5b668e0ceed6 Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

fix: tighten regex handler type assertion

0ae485e

Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/f9b8dbc7-cc72-4f60-b345-5b668e0ceed6 Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

test: clarify regex handler validation comment

4d138de

Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/f9b8dbc7-cc72-4f60-b345-5b668e0ceed6 Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

Copilot finished work on behalf of gaby May 6, 2026 02:29

gaby requested a review from Copilot May 6, 2026 04:07

Copilot started reviewing on behalf of gaby May 6, 2026 04:08 View session

Copilot AI reviewed May 6, 2026

View reviewed changes

Copilot started work on behalf of gaby May 6, 2026 04:22 View session

Copilot AI and others added 3 commits May 6, 2026 04:25

fix: preserve regex constraint compatibility

c5d5f75

Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/bb345d14-e46f-4eb3-80d3-76fb16369d8d Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

chore: align constraint field layout

e9b4907

Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/bb345d14-e46f-4eb3-80d3-76fb16369d8d Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

refactor: clarify internal regex matcher fallback

67f902d

Agent-Logs-Url: https://github.com/gofiber/fiber/sessions/bb345d14-e46f-4eb3-80d3-76fb16369d8d Co-authored-by: gaby <835733+gaby@users.noreply.github.com>

Copilot finished work on behalf of gaby May 6, 2026 04:33

Uh oh!

Conversation

Claude AI commented May 4, 2026

Alignment with Express API

HTTP RFC Standards Compliance

API Stability

Feature Examples

Checklist:

Comments on the Issue (you are @claude[agent] in this section)

Uh oh!

codecov Bot commented May 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

gaby commented May 4, 2026

Uh oh!

Claude AI commented May 4, 2026

Uh oh!

gaby commented May 5, 2026

Uh oh!

Claude AI commented May 5, 2026

Uh oh!

gaby commented May 5, 2026

Uh oh!

gaby commented May 5, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

gaby commented May 6, 2026

Uh oh!

Copilot AI commented May 6, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

gaby commented May 6, 2026

Uh oh!

Copilot AI commented May 6, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

codecov Bot commented May 4, 2026 •

edited

Loading