[No QA] Docs updates for E/A#94602 auto-add new agent as full copilot#95675
Conversation
… creator Co-authored-by: Yuwen Memon <yuwenmemon@users.noreply.github.com>
Concierge reviewer checklist:
For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member? cc @zanyrenney |
|
@zanyrenney Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button] |
|
@MelvinBot please resolve the conflicts here. |
| 4. Review the auto-generated profile image and display name. If needed, you can edit these later. | ||
| 5. Click **Write custom instructions** and describe what you want the agent to do. | ||
| 6. Click **Create agent**. | ||
| 6. Review the note above the **Create agent** button, which explains that the agent will be added as a full copilot of your account so it can act on your behalf. |
There was a problem hiding this comment.
Naming/terminology inconsistency: "full copilot" should be "full-access Copilot" to match the exact feature name and the phrasing used everywhere else in this article (lines 14, 52, 53, 57, 177). Copilot is a product feature name and must be capitalized, and the "full-access" qualifier should be used consistently. Suggested: "...added as a full-access Copilot on your account so it can act on your behalf."
|
HelpDot Documentation Review Overall Assessment Scores Summary
Key Findings
Recommendations
Files Reviewed
Note: Detailed line-by-line feedback has been provided as inline comments. |
…ent-copilot Co-authored-by: Yuwen Memon <yuwenmemon@users.noreply.github.com> # Conflicts: # docs/articles/new-expensify/ai-agents/Create-and-Use-Custom-Agents.md
|
🤖 Resolved the merge conflicts and pushed the merge commit — the PR is now mergeable. The only conflict was in I resolved all four conflict hunks by keeping this PR's more complete wording, since it's the more accurate and internally consistent version:
I also kept the one genuinely useful clarification that only existed on
|
|
A preview of your ExpensifyHelp changes have been deployed to https://0ded5be0.helpdot.pages.dev ⚡️ Updated articles: |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a679eb486c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| Every agent has its own Expensify account. You can Copilot into the agent's account to review its activity, update its instructions, and see the actions it has taken. | ||
|
|
||
| When you create an agent, it's automatically added as a full Copilot on your own account. This gives the agent delegated access to your account so it can use your personal context and manage expenses and reports on your behalf, while continuing to operate from its own account. | ||
| When you create an agent, it's automatically added as a full-access Copilot on your own account. This gives the agent delegated access to your account so it can use your personal context and manage expenses and reports on your behalf, while continuing to operate from its own account. You can review or remove this access at any time in the **Copilot: Delegated Access** section under **Account > Security**. |
There was a problem hiding this comment.
Point users to Account > Copilot
The Copilot delegated-access section is not under Account > Security in the current app/help flow: the route is settings/copilot (src/ROUTES.ts) and the existing Copilot article directs add/remove actions through Account > Copilot. With this path, users trying to review or remove the automatically added agent access will be sent to the wrong settings page; the same incorrect breadcrumb is repeated later in this article.
Useful? React with 👍 / 👎.
Explanation of Change
App #94602 changes agent creation so that a newly created agent is now automatically sent as a full-access Copilot of its creator's account, letting it act on the creator's behalf without the user adding it as a Copilot by hand. The create screen also now shows a note explaining this before the agent is created.
The existing help article
Create and Use Agentsdescribed this as an optional / manual action ("You can also add an agent as a Copilot on your own account"). This PR updates the article to reflect the new automatic behavior:descriptionto match the new behavior.All UI labels (Copilot: Delegated Access, Account > Security, Create agent, New agent) were verified against the existing Copilot help articles and the PR's copy for exact capitalization.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/651811
Tests
// TODO: The human co-author must fill out the tests you ran before marking this PR as "ready for review"
// Please describe what tests you performed that validates your changed worked.
QA
// TODO: The human co-author must fill out the QA tests you ran before marking this PR as "ready for review".
// Please describe what QA needs to do to validate your changes and what areas do they need to test for regressions.
@yuwenmemon, please review the files changed and confirm they reflect the current behavior. Then mark this PR
Ready for review.