Skip to content

bump prod#30

Merged
DuckySoLucky merged 4 commits intoprodfrom
dev
Feb 7, 2026
Merged

bump prod#30
DuckySoLucky merged 4 commits intoprodfrom
dev

Conversation

@DuckySoLucky
Copy link
Member

No description provided.

@DuckySoLucky DuckySoLucky requested a review from Copilot February 7, 2026 21:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR focuses on improving performance and reliability by introducing concurrency for independent API calls, standardizing HTTP requests through a shared client, and replacing custom helpers with standard library equivalents.

Changes:

  • Added a shared HTTP_CLIENT with timeouts/transport settings and migrated http.Get call sites to use it.
  • Parallelized several route and stats operations using errgroup/goroutines to reduce response latency.
  • Replaced custom slice/search and sorting helpers with slices.Contains and sort package utilities; added caching for regex compilation and image decoding.

Reviewed changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
src/utility/helper.go Caches DOMAIN once, replaces custom sort/regex usage with stdlib (sort, cached regexp).
src/stats/skills.go Switches from utility.Contains to slices.Contains.
src/stats/other.go Parallelizes member formatting via goroutines/channels.
src/stats/leveling/leveling.go Switches from utility.Contains to slices.Contains.
src/stats/items/helper.go Switches from utility.Contains to slices.Contains in item gem/category parsing.
src/stats/accessories.go Uses slices.Contains for accessory detection.
src/routes/stats.go Parallelizes profile/player and museum/members fetches via errgroup.
src/routes/skills.go Parallelizes profile/player fetch via errgroup.
src/routes/misc.go Parallelizes profile/player fetch via errgroup.
src/routes/item.go Updates RenderItem call signature.
src/routes.go Prefork-aware initialization flow; enables ETag middleware.
src/lib/renderer.go Adds singleflight for head rendering, in-memory image cache, avoids temp files, updates RenderItem signature and HTTP usage.
src/lib/custom_resources.go Adds compiled-regex cache to avoid repeated regexp.Compile.
src/api/mowojang.go Uses shared HTTP_CLIENT instead of http.Get.
src/api/hypixel_constants.go Uses shared HTTP_CLIENT instead of http.Get.
src/api/hypixel.go Uses shared HTTP_CLIENT instead of http.Get.
src/api/client.go Introduces shared HTTP client configuration.
main.go Adds Fiber read/write/idle timeouts.
NotEnoughUpdates-REPO Updates submodule commit.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

DuckySoLucky and others added 2 commits February 7, 2026 22:50
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@DuckySoLucky DuckySoLucky requested a review from Copilot February 7, 2026 21:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@DuckySoLucky DuckySoLucky merged commit 81a2320 into prod Feb 7, 2026
15 of 16 checks passed
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