shiyue/upage-git
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: ensure nanostores listeners trigger on artifact updates

Browse Source 
Refactored artifact update logic to create new Map instances when modifying artifacts, ensuring nanostores listeners are properly triggered. This prevents issues where listeners do not react to in-place mutations of existing Map objects.
This commit is contained in:
LIlGG
2025-10-13 12:42:56 +08:00
parent 8c6f55dda8
commit 4e39d41362
1 changed files with 17 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
app/.client/stores/chat.ts
View File

@@ -120,15 +120,16 @@ export class ChatStore {
};
const artifactsByMessageId = this.artifacts.get();
let artifactsByPageName = artifactsByMessageId.get(messageId);
if (!artifactsByPageName) {
artifactsByPageName = new Map();
artifactsByMessageId.set(messageId, artifactsByPageName);
}
const existingArtifactsByPageName = artifactsByMessageId.get(messageId);
const artifactsByPageName = existingArtifactsByPageName ? new Map(existingArtifactsByPageName) : new Map();
artifactsByPageName.set(name, newArtifact);
this.artifacts.set(artifactsByMessageId);
// create new outer Map instance to trigger nanostores listener
const newArtifactsByMessageId = new Map(artifactsByMessageId);
newArtifactsByMessageId.set(messageId, artifactsByPageName);
this.artifacts.set(newArtifactsByMessageId);
const bridge = await editorBridge;
bridge.updatePageAttributes(name, { title });
}
@@ -140,14 +141,19 @@ export class ChatStore {
}
const artifactsByMessageId = this.artifacts.get();
const artifactsByPageName = artifactsByMessageId.get(messageId);
if (!artifactsByPageName) {
const existingArtifactsByPageName = artifactsByMessageId.get(messageId);
if (!existingArtifactsByPageName) {
return;
}
artifactsByPageName.set(name, { ...artifact, ...state });
artifactsByMessageId.set(messageId, artifactsByPageName);
this.artifacts.set(artifactsByMessageId);
const artifactsByPageName = new Map(existingArtifactsByPageName);
artifactsByPageName.set(name, { ...artifact, ...state });
// create new outer Map instance to trigger nanostores listener
const newArtifactsByMessageId = new Map(artifactsByMessageId);
newArtifactsByMessageId.set(messageId, artifactsByPageName);
this.artifacts.set(newArtifactsByMessageId);
}
private getArtifact(messageId: string, pageName: string) {