Skip to content

Fix/2125#322

Open
conradmugabe wants to merge 3 commits into
mainfrom
fix/2125
Open

Fix/2125#322
conradmugabe wants to merge 3 commits into
mainfrom
fix/2125

Conversation

@conradmugabe

@conradmugabe conradmugabe commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Checklist

  • Tests were added/updated according to the feature/bugfix/change made
  • Version was rolled according to semver requirements
  • API endpoints openapi schema was updated if applicable

Changes

conradmugabe and others added 3 commits July 10, 2026 18:10
The pre-commit hook runs 'prettier --write .' across the whole repo,
so pre-existing formatting drift (CHANGELOG.md line wrapping, missing
trailing newline in an e2e spec) is normalized here in an isolated
commit to keep the fix commit clean.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Data sources in the chat DocumentSidebar were wrapped in a next/link
that opened the source URL in a new tab on click. Render the document
title as plain, non-navigating text so clicking a data source does
nothing, and drop the blue hover affordance that signalled a link.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@conradmugabe
conradmugabe requested a review from mamigot as a code owner July 10, 2026 17:13
@conradmugabe conradmugabe self-assigned this Jul 10, 2026
@conradmugabe
conradmugabe requested a review from sonegillis1 July 10, 2026 17:13
@mamigot

mamigot commented Jul 10, 2026

Copy link
Copy Markdown
Member

Claude finished @conradmugabe's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews
  • Identify changed files
  • Run mechanical coverage check
  • Apply decision logic and post review

Result: ✅ E2E Coverage — No Issues

No new app/**/page.tsx routes were introduced (only components/document-sidebar.tsx and its test were modified). The mechanical coverage check passed with 545 checkpoints and 100% route coverage (25/25 files mapped).

@conradmugabe conradmugabe added the run-tests Trigger PR validation pipeline label Jul 10, 2026

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-tests Trigger PR validation pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants