Normalize cachebust_origin in OBJ loader cache keys

[JYZ-LESLIE]

Summary

• add normalizeModelCacheUrl helper to strip cachebust_origin query params robustly
• use normalized URL as the cache key in useGlobalObjLoader
• add focused regression tests for different cachebust query shapes

This addresses duplicate cache entries and repeated model loads caused by cachebust variants of the same model URL.

Validation

• Could not run tests in this environment because bun is not installed.

/claim #93

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
3d-viewer	Ready	Preview, Comment	May 12, 2026 1:30pm

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 44c4b9d3d9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Strip repeated cachebust_origin params completely

The new normalizer only removes the first cachebust_origin when the parameter appears multiple times, because the regex optionally consumes the following & and the next occurrence is no longer preceded by ? or &. For example, ...?cachebust_origin=a&cachebust_origin=b&pn=C1 normalizes to a URL that still contains one cachebust_origin, so equivalent cache-busted URLs can still produce different cache keys and duplicate OBJ cache entries.

Useful? React with 👍 / 👎.