2001526 - Chat Assistant @mentioned rendering

Status: VERIFIED FIXED

(Core :: Machine Learning: Frontend, task, P1)

Description

[:omarg]

PRD section: https://docs.google.com/document/d/1dbbcvNhI7WRjsmWqHUlO_OZhu7xwYm3pG0KjimNGvbI/edit?tab=t.pynoo4vz6wdz#bookmark=id.naa8t794f0ql

PRD section describes all of @mention functionality, this task is for a component that renders a mention the chat assistant conversation display

Figma: https://www.figma.com/design/5KuePTGmOEUFyCHBHCsGim/AI-Mode-%E2%80%94%C2%A0MVP-Scope-Design?node-id=38-43098&m=dev

Comment 1

[Nick]

Attachment: Bug 2001526 - Chat Assistant @mentioned rendering r=mardak,jlewis,flozia

Creating the contract between the smart input bar mentions and the ai-chat-message mentions. Adding the ability to spot the mentions in the payload and render them out in the DOM.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/a05b5fb1ada2
https://hg.mozilla.org/integration/autoland/rev/7d4b3df21a89
Chat Assistant @mentioned rendering r=flozia,ai-frontend-reviewers,Gijs,tschuster

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/454b854baa83
https://hg.mozilla.org/integration/autoland/rev/e225d09d402c
Revert "Bug 2001526 - Chat Assistant @mentioned rendering r=flozia,ai-frontend-reviewers,Gijs,tschuster" for causing mochitests failures.

Comment 4

[Serban Stanca [:SerbanS]]

Reverted this because it was causing mochitests failures.

• Revert link
• Push with failures
• Failure Log
• Failure line: Hit MOZ_CRASH(Page about:aichatcontent must not contain a CSP with the directive img-src that includes page-icon:) at /builds/worker/checkouts/gecko/dom/security/nsContentSecurityUtils.cpp:X

Comment 5

[Tom Schuster (OoO)]

Attachment: Bug 2001526 - Allow the page-icon: protocol by default as a CSP img-src in chrome documents. r?gijs!,simonf!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/0c4b7bfdf18b
https://hg.mozilla.org/integration/autoland/rev/28fa8cd965da
Allow the page-icon: protocol by default as a CSP img-src in chrome documents. r=Gijs,simonf

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/6102b3fcbab7
https://hg.mozilla.org/integration/autoland/rev/f61ddd1b4a5d
Chat Assistant @mentioned rendering r=flozia,ai-frontend-reviewers,Gijs,tschuster

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/d475653aa1f6
https://hg.mozilla.org/integration/autoland/rev/6163aac76c38
Revert "Bug 2001526 - Chat Assistant @mentioned rendering r=flozia,ai-frontend-reviewers,Gijs,tschuster" for causing bc failures @ browser_all_files_referenced.js

Comment 9

[amarc]

Backed out for causing bc failures

• Backout link
• Push with failures
• Failure Log

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/5cc190f37a14
https://hg.mozilla.org/integration/autoland/rev/f8db0c92eb3b
Chat Assistant @mentioned rendering r=flozia,ai-frontend-reviewers,Gijs,tschuster

Comment 11

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/f8db0c92eb3b

Comment 12

[Rares Doghi, Desktop QA]

Verified as fixed in our latest Beta 149.0b1