Skip to content

Add citation URL to streaming response metadata#558

Merged
rido-min merged 3 commits intomainfrom
fix/citation-url
Sep 8, 2025
Merged

Add citation URL to streaming response metadata#558
rido-min merged 3 commits intomainfrom
fix/citation-url

Conversation

@rido-min
Copy link
Copy Markdown
Member

@rido-min rido-min commented Sep 3, 2025

Include the citation URL in the metadata of the streaming response to enhance the document's information.

Copilot AI review requested due to automatic review settings September 3, 2025 23:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds citation URL to the streaming response metadata to provide more comprehensive document information. This enhancement allows consumers of the streaming response to access the source URL for each citation.

  • Includes citation URL in the DigitalDocument appearance metadata
  • Uses non-null assertion operator to access the URL property

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread packages/agents-hosting/src/app/streaming/streamingResponse.ts
@rido-min rido-min enabled auto-merge (squash) September 8, 2025 20:35
@rido-min rido-min merged commit 5703ae5 into main Sep 8, 2025
7 checks passed
@rido-min rido-min deleted the fix/citation-url branch September 8, 2025 20:39
rido-min added a commit that referenced this pull request Sep 8, 2025
* citation url

* add streamingResponse tests

---------

Co-authored-by: Rido <nouser@gh.com>
rido-min added a commit that referenced this pull request Sep 8, 2025
* Reorder fileDownloaders execution in agentApplication (#543)

* chore: update dependencies in package.json and package-lock.json (#549)

Co-authored-by: Rido <nouser@gh.com>

* fix: update npm install display name to npm ci and adjust branch filter for pull requests (#557)

* fix: update npm install display name to npm ci and adjust branch filter for pull requests

* fix: update branch filter in CI configuration to include release branches

---------

Co-authored-by: Rido <nouser@gh.com>

* Add citation URL to streaming response metadata (#558)

* citation url

* add streamingResponse tests

---------

Co-authored-by: Rido <nouser@gh.com>

* Restore teams-extensions (#548)

* restore teams-extensions. Pending Tests verification

* fixes @microsoft/agents-hosting-extensions-teams TeamsInfo.getTeamChannels wrongly typed
Fixes #507

* fix: correct typos in tenantId parameter and test descriptions

---------

Co-authored-by: Rido <nouser@gh.com>

* fix lock

* rolback deps

---------

Co-authored-by: Cecilia Avila <44245136+ceciliaavila@users.noreply.github.com>
Co-authored-by: Rido <nouser@gh.com>
@rido-min rido-min mentioned this pull request Sep 9, 2025
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.

3 participants