Skip to content

Fix chat document metadata and desktop notification alignment - #1757

Merged
Paul Lizer (paullizer) merged 1 commit into
microsoft:paullizer-react-v2-uifrom
paullizer:paullizer-chat-document-metadata
Oct 11, 2026
Merged

Paul Lizer (paullizer) merged 1 commit into
microsoft:paullizer-react-v2-uifrom
paullizer:paullizer-chat-document-metadata

Conversation

@paullizer

Copy link
Copy Markdown
Contributor

Summary

  • New Orchestrate answers could show GUIDs in the V2 Documents pane because lineage-only references lacked display metadata. Copy title, filename and file type from already-authorized manifests before publication, preserving citation identity and the final authorization check.
  • Render recognizable document cards with shared file-family icons, distinct filenames and personal/group/public workspace labels. Merge complementary existing metadata without new browser lookups or historical backfill.
  • Restore the desktop notification bell beside the collapse arrow while preserving mobile navigation behavior. Bump the application to 0.261.327.

Linked issue

N/A. The user chose to keep this work in the session without creating an issue.

Release Notes & Latest Features

  • New Feature
  • Bug Fix
  • UI Enhancement
  • Breaking Change
  • Internal only

Is this visible to end users?

  • Yes
  • No

Is this admin-facing (Admin Settings, governance, deployment, config)?

  • Yes
  • No

Should this become a Latest Feature card?

  • Yes
  • No
  • Already added

Screenshot needed for the card?

  • Yes
  • No
  • Attached

Version bump

  • application/single_app/config.py VERSION third segment bumped, or not needed because this is docs-only
  • deployers/version.txt bumped, or not needed because deployers/ was not changed

Testing / validation

  • npm --prefix .\application\v2_ui run typecheck - passed.
  • node .\functional_tests\test_v2_conversation_documents.mjs - passed.
  • npm --prefix .\application\v2_ui run build -- --outDir ..\..\ui_tests\artifacts\orchestration-plan-editor - passed; existing large-chunk warning remains.
  • python -m pytest .\functional_tests\test_chat_cited_source_tracking.py .\functional_tests\test_orchestration_citation_persistence.py -q - 21 passed.
  • python -m pytest .\functional_tests\test_orchestration_harness_execution.py -k "selected_result_citations_reach_headless_message_cache_and_delivery or citation_delivery_rechecks_exact_sources" -q --disable-warnings - 16 passed, 635 deselected.
  • python -m pytest .\functional_tests\test_orchestration_cosmos_response_contract.py -q - 28 passed.
  • python .\scripts\check_cosmos_response_boundaries.py --run-contracts - 693 passed and 231 subtests passed; 0 inventory review gaps.
  • python -m pytest .\ui_tests\test_v2_chat_document_metadata.py .\ui_tests\test_v2_drawer_generated_files.py .\ui_tests\test_v2_mobile_navigation.py .\ui_tests\test_v2_mobile_chat_layout.py -q - 29 passed, including actual file icons, desktop/mobile cards and bell geometry.
  • python .\functional_tests\test_docs_app_surface_coverage.py - 7/7 checks passed.
  • python .\functional_tests\test_docs_site_quality.py - 6/6 checks passed.
  • git diff --check - passed.

Backend integration and offline contract checks used an isolated environment to match repository pins after shared-machine dependency mismatches. No application dependency manifests changed; the temporary environment was removed. Existing dependency deprecation warnings remain.

Documentation

  • Release notes updated, or not needed
  • Feature documentation updated, or not needed
  • Fix documentation updated, or not needed

Updated the Documents drawer reference and added docs/explanation/fixes/CHAT_DOCUMENT_METADATA_AND_NOTIFICATION_ALIGNMENT_FIX.md. Release notes remain unchanged pending user approval. Existing conversations are intentionally not backfilled.

Security checklist

  • New Flask routes include @swagger_route(security=get_auth_security())
  • Settings sent to non-admin frontends use sanitize_settings_for_user()
  • Browser JavaScript is served from local SimpleChat static assets only; no CDN-hosted JS
  • No secrets, keys, connection strings, or local-only artifacts are included

No routes or settings-response contracts changed. Metadata enrichment reuses authorized source manifests; existing pre-publication authorization remains in place.

Retain authorized document display metadata on new orchestration answers, restore file icons and workspace labels, and preserve mobile navigation spacing.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Comment thread ui_tests/test_v2_chat_document_metadata.py
@paullizer
Paul Lizer (paullizer) merged commit 90d9596 into microsoft:paullizer-react-v2-ui Oct 11, 2026
12 checks passed
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.

2 participants