Backup/SnowLuma
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix(onebot): resolve self nickname via user profile when friend list omits self

Browse Source 
The session warmup only set qqInfo.nickname when QQ's friend-list
response happened to echo self back, which it omits for some
accounts / versions / multi-account warmup races. The WebUI online
sessions card then rendered a blank nickname row above the UIN.
Fall back to a direct user-profile fetch when the friend-list scan
misses self, and seed qqInfo.nickname with the UIN at session
creation so the panel never paints empty while warmup is in flight.
This commit is contained in:
DrSmoothl
2026-05-13 21:11:26 +08:00
parent ea5a1d505a
commit e438f353af
1 changed files with 29 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/core/src/onebot/manager.ts
+29 -2
View File
@@ -60,6 +60,10 @@ export class OneBotManager {
instance.addPid(activePid);
}
// Seed nickname with the UIN so the WebUI never renders blank while
// warmup is in flight or if warmup never resolves a real nickname.
if (!qqInfo.nickname) qqInfo.nickname = uin;
this.instances.set(uin, instance);
log.info('session started: UIN=%s', uin);
@@ -81,12 +85,17 @@ export class OneBotManager {
}
async function warmUpBridgeState(uin: string, qqInfo: QQInfo, bridge: Bridge): Promise<void> {
// Step 1: Fetch friend list + derive self profile
const selfUin = parseInt(uin, 10) || 0;
let selfResolved = false;
// Step 1: Fetch friend list + derive self profile when QQ happens to
// include self in the response. Some accounts / versions omit self,
// which used to leave qqInfo.nickname empty — see step 1b for the
// explicit fallback.
try {
const friends = await bridge.fetchFriendList();
log.info('friends loaded: UIN=%s count=%d', uin, friends.length);
const selfUin = parseInt(uin, 10) || 0;
for (const f of friends) {
if (f.uin === selfUin) {
qqInfo.setSelfProfile({
@@ -96,6 +105,7 @@ async function warmUpBridgeState(uin: string, qqInfo: QQInfo, bridge: Bridge): P
});
qqInfo.nickname = f.nickname || uin;
log.debug('self info: UIN=%s uid=%s nickname=%s', uin, f.uid, f.nickname ?? '');
selfResolved = true;
break;
}
}
@@ -103,6 +113,23 @@ async function warmUpBridgeState(uin: string, qqInfo: QQInfo, bridge: Bridge): P
log.warn('failed to load friends for UIN %s: %s', uin, e instanceof Error ? e.message : String(e));
}
// Step 1b: friend-list path didn't resolve self → fetch user profile
// directly via OIDB 0xFE1_2 so multi-account WebUI shows a nickname
// for every injected session, not just the ones where QQ echoed self
// back in the friend list.
if (!selfResolved && selfUin > 0) {
try {
const profile = await bridge.fetchUserProfile(selfUin);
qqInfo.setSelfProfile(profile);
qqInfo.nickname = profile.nickname || uin;
log.debug('self info via profile: UIN=%s uid=%s nickname=%s',
uin, profile.uid, profile.nickname);
} catch (e) {
log.warn('failed to load self profile for UIN %s: %s',
uin, e instanceof Error ? e.message : String(e));
}
}
// Step 2: Fetch group list
let groups: { groupId: number }[] = [];
try {