fix: resolve logic bug and mutable default argument in getUserDataForFid

[dajiaohuang]

Summary

修复了 getUserDataForFid 函数中的两个问题：

Bug 1: 逻辑错误

原代码中 if/else 的两个分支执行完全相同的操作：

if (foundValue && foundValue.timestamp < timestamp) {
  acc[type] = { value, timestamp };
} else {
  acc[type] = { value, timestamp }; // 两个分支完全相同
}

这会导致当找到的值更新时不会覆盖旧值，或者无论条件如何都覆盖。正确的逻辑应该是只在找到的值比当前值旧时才更新。

Bug 2: 可变默认参数

JavaScript 中使用可变对象作为默认参数是一个反模式，因为默认对象在所有调用中共享。原代码：

hubRequestOptions = {
  headers: {
    api_key: DEFAULT_HUB_API_KEY,
  },
},

会在所有调用中共享同一个对象，如果调用者修改了它，会影响后续调用。

Fix

• 移除了冗余的 else 分支
• 创建新的 requestOptions 对象，每次调用时都是fresh的
• 正确合并用户提供的 hubRequestOptions 和默认的 api_key header

[vercel]

@dajiaohuang is attempting to deploy a commit to the Discove Team on Vercel.

A member of the Team first needs to authorize it.