Skip to content

feat(cas): add anonymous chat steps to docs - #621

Merged
Raina451 merged 2 commits into
mainfrom
task/cas-update-auth-docs
Jul 29, 2026
Merged

feat(cas): add anonymous chat steps to docs#621
Raina451 merged 2 commits into
mainfrom
task/cas-update-auth-docs

Conversation

@scottcmg

Copy link
Copy Markdown
Contributor

Description

Conversational Agents provides support for anonymous chat using client credential token types from app-scoped confidential external apps, but the docs didn't have any information on how to do this.

This PR adds this functionality info and example in the conversational agent, and expands the auth docs to include information on how to set up an external app to use with conversational agents for anonymous chat.

@scottcmg
scottcmg requested a review from a team July 22, 2026 19:14
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-07-29 05:05 UTC

Comment thread docs/authentication.md Outdated
Comment thread docs/authentication.md Outdated
Comment thread src/models/conversational-agent/conversational-agent.models.ts
@claude

claude Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Review findings

Three issues found this run:

docs/authentication.md:66 — Missing section heading
The new confidential External App block flows directly into the PAT Token steps with no heading. Suggested a ### App-scoped External App (for anonymous Conversational Agents) subheading to match the document structure.

docs/authentication.md:74 — Redirect URI description is misleading
The client-credentials grant never performs a browser redirect, so describing the redirect URI as a normal configuration step implies it participates in the flow. Suggested a short clarification that it is a required portal field but not used in the token request.

conversational-agent.models.ts:116–117 — Code snippet missing import statement
The new App-scoped authentication code block omits the import { ConversationalAgent } statement, inconsistent with the ## Usage block above that includes it. Suggested adding the import to keep the snippet self-contained and copy-pasteable.

@sonarqubecloud

Copy link
Copy Markdown

Comment thread src/models/conversational-agent/conversational-agent.models.ts
Comment thread docs/authentication.md Outdated
@claude

claude Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

✅ No issues found. Checked for bugs and CLAUDE.md compliance.

@scottcmg
scottcmg requested a review from Raina451 July 28, 2026 06:14
@Raina451
Raina451 merged commit 2a51436 into main Jul 29, 2026
15 of 19 checks passed
@Raina451
Raina451 deleted the task/cas-update-auth-docs branch July 29, 2026 05:05
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