Skip to content

Commit bda0a56

Browse files
Natallia HarshunovaOrKoN
authored andcommitted
Comments fix
1 parent 57467f1 commit bda0a56

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/PageCollector.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,6 @@ export class PageCollector<T> {
106106
const storedLists: Array<Array<WithSymbolId<T>>> = [[]];
107107
this.storage.set(page, storedLists);
108108

109-
// This is the single function responsible for adding items to storage.
110109
const collector = (value: T) => {
111110
const withId = value as WithSymbolId<T>;
112111
// Assign an ID only if it's a new item.
@@ -117,8 +116,8 @@ export class PageCollector<T> {
117116
const navigations = this.storage.get(page) ?? [[]];
118117
const currentNavigation = navigations[0];
119118

120-
// The aggregator sends the same object instance for updates, so we just
121-
// need to ensure it's in the list.
119+
// The issues aggregator sends the same object instance for updates, so we just
120+
// need to ensure it's not in the list.
122121
if (!currentNavigation.includes(withId)) {
123122
currentNavigation.push(withId);
124123
}
@@ -259,7 +258,6 @@ export class ConsoleCollector extends PageCollector<ConsoleMessage | Error | Agg
259258
if (seenKeys.has(primaryKey)) return;
260259
seenKeys.add(primaryKey);
261260

262-
// Trigger the aggregator via our mock manager. Do NOT call collector() here.
263261
const mockManager = this.#mockIssuesManagers.get(page);
264262
if (mockManager) {
265263
// @ts-expect-error We don't care that issues model is null

0 commit comments

Comments
 (0)