[deps] upgrade fiberfx to skip loggind metrics and health

[capcom6]

Summary by CodeRabbit

• Chores
  • Updated project dependencies to newer versions, delivering stability, compatibility and performance improvements.
  • Includes indirect dependency refreshes and a few dependency additions/removals to align transitive requirements.
  • No changes to public/exported APIs or user-facing functionality.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Updated go.mod with several dependency version changes: direct bumps for fiberfx and fiber/v2, multiple indirect dependency upgrades, two new indirect additions, and removal of some indirect entries. No API or exported symbols were changed.

Changes

Cohort / File(s)

Summary

Go module dependencies go.mod

Updated direct dependencies: fiberfx → v0.0.1, github.com/gofiber/fiber/v2 v2.52.9 → v2.52.10. Updated indirect dependencies (examples): github.com/andybalholm/brotli v1.1.0→v1.2.0, github.com/klauspost/compress v1.17.9→v1.18.1, github.com/mattn/go-colorable v0.1.13→v0.1.14, github.com/mattn/go-runewidth v0.0.16→v0.0.19, github.com/valyala/fasthttp v1.56.0→v1.68.0. Added indirects: github.com/clipperhouse/stringish v0.1.1, github.com/clipperhouse/uax29/v2 v2.3.0. Removed/adjusted indirects: uniseg (rivo) and valyala/tcplisten. Other golang.org/x/* transitive updates reflected.

Sequence Diagram(s)

(Skipped: changes are confined to dependency manifest and do not modify runtime control flow.)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5–10 minutes

• Focus review on compatibility of github.com/valyala/fasthttp jump and github.com/gofiber/fiber/v2 patch bump.
• Verify transitive additions (clipperhouse/*) and removals (rivo/uniseg, valyala/tcplisten) do not affect build tags or platform-specific code.

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title mentions upgrading fiberfx but contains a typo ('loggind' instead of 'logging') and is somewhat unclear about the actual scope of changes, which include multiple dependency updates beyond just fiberfx.	Correct the typo ('loggind' to 'logging') and consider clarifying whether the title should focus on fiberfx specifically or reflect the broader dependency upgrade scope shown in the changeset.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch deps/upgrade-fiberfx

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e9fddfa and 794d350.

⛔ Files ignored due to path filters (1)

• go.sum is excluded by !**/*.sum

📒 Files selected for processing (1)

• go.mod (3 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)

• GitHub Check: Benchmark
• GitHub Check: Test
• GitHub Check: Lint
• GitHub Check: E2E
• GitHub Check: Analyze (go)

🔇 Additional comments (3)

go.mod (3)

45-45: Verify Go version requirement and HTTP header compliance for fasthttp v1.68.0.

The upgrade contains a notable backwards incompatibility and Go version constraint that require manual verification:

1. Breaking change (v1.65.0): Headers delimited by only \n (instead of \r\n) are no longer supported. Verify your application doesn't rely on this non-compliant parsing.

2. Go version requirement: fasthttp v1.63.0+ requires Go 1.20+. Confirm your project's minimum supported Go version meets this constraint.

3. URL validation (v1.67.0): Stricter validation for usernames, IPv6, schemes, and host formats. Review any custom URL handling to ensure it still works with these rules.

4. compress v1.18.1: Included as a fasthttp dependency; no breaking changes identified in the intermediate releases (v1.17.9 → v1.18.1).

---

11-11: PR title does not align with fiberfx v0.0.1 release notes and visible changes.

The PR title claims to "skip logging metrics and health," but fiberfx v0.0.1 release notes document "[dto] simplify error response" and "[http] improve logging"—with no mention of a skip metrics/health feature. Additionally, codebase search found no configuration or code changes implementing this feature.

Clarify whether:

1. The PR title is inaccurate and this is merely a dependency update
2. The skip feature is expected to be auto-enabled by fiberfx v0.0.1 (requires confirmation from maintainers)
3. Other code changes outside go.mod implement this feature (if so, they should be reviewed separately)

Without clear evidence of the feature implementation or breaking change requirements, the fiberfx and fiber/v2 upgrades cannot be fully validated.

---

48-49: Dependency changes verified as intentional and safe.

The new clipperhouse dependencies are confirmed as transitive imports from the fiberfx v0.0.1 upgrade (a direct dependency). clipperhouse/uax29/v2 is a Unicode text-segmentation tokenizer implementing UAX #29, which directly replaces the removed rivo/uniseg library. clipperhouse/stringish provides a generic string-like type constraint and UTF-8 helper. The clipperhouse org appears to be an active source publishing Go modules with tagged releases, confirming these are from a maintained source.

The substitution of uniseg with uax29/v2 is intentional and safe—both libraries handle Unicode text segmentation, and the swap appears coordinated within the fiberfx dependency chain.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}