Update agent notifications support for Auggie - #475
Conversation
|
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @tkossak on file. In order for us to review and merge your code, each contributor must visit https://cla.warp.dev to read and agree to our CLA. Once you have done so, please comment |
|
@tkossak is attempting to deploy a commit to the warpdotdev Team on Vercel. A member of the Team first needs to authorize it. |
|
Every PR must be linked to a same-repo issue before Oz can review it. Next step: open or find a same-repo issue describing this change, then link it to this PR by adding See the contribution guidelines for the full readiness model. Powered by Oz |
![oz-for-oss[bot]](https://avatars.githubusercontent.com/in/3450139?s=60&v=4){kind=small}
There was a problem hiding this comment.
Every PR must be linked to a same-repo issue before Oz can review it.
Next step: open or find a same-repo issue describing this change, then link it to this PR by adding Closes #123 to the PR description (or using the "Development" sidebar on GitHub). A maintainer will mark the issue ready-to-implement when it is ready. Once it is marked, comment /oz-review to re-trigger review.
See the contribution guidelines for the full readiness model.
Powered by Oz
|
@cla-bot check |
|
The cla-bot has been summoned, and re-checked this pull request! |
|
Thanks! |
|
/oz-review |
1 similar comment
|
/oz-review |
|
I'm re-reviewing this pull request in response to a review request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
There was a problem hiding this comment.
Overview
This docs PR updates the third-party CLI agents overview feature table to mark Auggie as supporting agent notifications. The change is small, but it leaves the published docs internally inconsistent about Auggie notification support.
Concerns
- The table now says Auggie supports agent notifications, while the unchanged note later on the same page still says Auggie does not support notifications yet. The dedicated Agent Notifications page also omits Auggie from the supported third-party agents and setup instructions.
Verdict
Found: 0 critical, 1 important, 0 suggestions
Request changes
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
| |---|---|---|---|---|---|---|---|---|---|---| | ||
| | Rich input editor (`Ctrl-G`) | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | ✓ | | ||
| | Agent notifications | ✓ | ✓ | ✓ | ✗ | ✗ | ✗ | ✗ | ✗ | ✗ | ✗ | | ||
| | Agent notifications | ✓ | ✓ | ✓ | ✗ | ✓ | ✗ | ✗ | ✗ | ✗ | ✗ | |
There was a problem hiding this comment.
/agents/capabilities/agent-notifications/ still omits Auggie from the supported agents/setup instructions. Update the supporting text/setup docs in the same PR or keep Auggie as ✗ here.
2 participants
Summary
Auggie does support agent notifications in warp - I marked it so in the feature table.
Related issues
Closes #576
Validation
docs only
Screenshots
Follow-ups