[BUG]: Notification badge (red dot) on sidebar icon does not clear after viewing session or page refresh

[cyberprophet]

Description

The notification badge (red/blue dot) displayed on the top-right of sidebar project icons does not disappear after the user views the session or refreshes the page.

Steps to Reproduce

1. Open opencode web UI
2. Have an active session that receives a response (AI turn completes)
3. A red/blue dot appears on the sidebar project icon
4. Click into the session to view it
5. Expected: Badge disappears
6. Actual: Badge remains visible
7. Refresh the page
8. Expected: Badge disappears (state was viewed)
9. Actual: Badge still remains visible

Root Cause (Technical)

markViewed() is only called inside syncSessionRoute() when the session changes (line 1658 in layout.tsx):

if (session !== activeRoute.session) {
  activeRoute.session = session
  activeRoute.sessionProject = syncSessionRoute(directory, id, root) // markViewed called here
  return
}

This means:

• If the user is already on the session when a notification arrives → markViewed() is never called
• After page refresh, the same session URL is restored → session !== activeRoute.session is false on first render → markViewed() is never called
• Notification state is persisted to localStorage with viewed: false, so it survives refresh

Environment

• Platform: Web UI
• Component: packages/app/src/pages/layout.tsx, packages/app/src/context/notification.tsx

Suggested Fix

1. Call markViewed() on initial page load if the current URL matches a session
2. Or add a reactive effect that calls markViewed() whenever the current session has unseen notifications

[cyberprophet]

[cyberprophet]

Regression Analysis

This bug was introduced in commit 8176bafc5 — chore(app): solidjs refactoring (#13399) (2026-03-02).

Before the refactor (working correctly)

// Called unconditionally whenever session URL changed
notification.session.markViewed(id)

After the refactor (bug introduced)

const session = `${dir}/${id}`
if (session !== activeRoute.session) {  // ← condition added for perf optimization
  activeRoute.session = session
  activeRoute.sessionProject = syncSessionRoute(directory, id, root) // markViewed only called here
  return
}
// If same session → markViewed never called ← BUG

The SolidJS refactoring added a session !== activeRoute.session guard to avoid redundant work, but this inadvertently prevents markViewed() from being called on:

1. Page refresh — same session URL is restored, condition is false on first render
2. Notification arriving while already on the session — session hasn't "changed" so condition is false

Commit Timeline

Date	Commit	Description
2026-01-21	64c80f1b5	fix(app): don't show notification on session if active — correctly sets viewed: true when notification arrives on active session
2026-02-14	85b5f5b70	feat(app): clear notifications action — adds manual clear button
2026-03-02	8176bafc5	chore(app): solidjs refactoring — introduces the regression

Fix

Call markViewed() on initial load if the current session has unseen notifications, in addition to when the session changes. A createEffect that reacts to the current session having unseen count > 0 would handle both cases.