Skip to content

Add clickable links for empty sessions in Diagnostic Report#165

Merged
rajbos merged 5 commits intomainfrom
copilot/add-link-to-empty-sessions
Feb 2, 2026
Merged

Add clickable links for empty sessions in Diagnostic Report#165
rajbos merged 5 commits intomainfrom
copilot/add-link-to-empty-sessions

Conversation

Copy link
Contributor

Copilot AI commented Jan 30, 2026

Empty sessions in the Diagnostic Report displayed as non-clickable "—" while sessions with interactions showed clickable title links. This prevented accessing debug details for empty sessions that may contain relevant metadata.

Changes

Modified src/webview/diagnostics/main.ts:

  • Session table rendering: Changed conditional to always render clickable links. Empty sessions now display "(Empty session)" instead of "—"
  • CSS styling: Added .empty-session-link class with normal (#999) and hover (#aaa) states for visual distinction from regular session links (#4FC3F7 / #81D4FA)
// Before: non-clickable for empty sessions
${sf.title ? `<a href="#" class="session-file-link"...>${sf.title}</a>` : '<span style="color: #666;">—</span>'}

// After: clickable for all sessions
${sf.title ? `<a href="#" class="session-file-link"...>${sf.title}</a>` : `<a href="#" class="session-file-link empty-session-link"...>(Empty session)</a>`}

The openSessionFile command now works consistently for all sessions regardless of interaction count.

Original prompt

This section details on the original issue you should resolve

<issue_title>[FEATURE] Add link to empty sessions</issue_title>
<issue_description>For Diagnostic Report we include empty sessions as well, however they contain some data, which might be in interest to access. Add the link to those sessions with details from the debug as it is done for sessions with interaction:

Image</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits January 31, 2026 21:46
Co-authored-by: rajbos <6085745+rajbos@users.noreply.github.com>
Co-authored-by: rajbos <6085745+rajbos@users.noreply.github.com>
Co-authored-by: rajbos <6085745+rajbos@users.noreply.github.com>
Copilot AI changed the title [WIP] Add link to access details of empty sessions Add clickable links for empty sessions in Diagnostic Report Jan 31, 2026
Copilot AI requested a review from rajbos January 31, 2026 21:50
@rajbos rajbos marked this pull request as ready for review February 2, 2026 15:39
Copy link
Owner

@rajbos rajbos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works, sweet!

@rajbos rajbos merged commit 1b26c00 into main Feb 2, 2026
16 checks passed
@rajbos rajbos deleted the copilot/add-link-to-empty-sessions branch February 2, 2026 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Add link to empty sessions

2 participants

Comments