Skip to content

Conversation

@shetzel
Copy link
Contributor

@shetzel shetzel commented Jul 30, 2025

What does this PR do?

Adds agent activate and agent deactivate commands.

NOTE: Requires forcedotcom/agents#133. For testing, link the branch from that PR and build.
NOTE: The tsconfig.json edits will be removed before merging.

What issues does this PR fix or reference?

@W-19127491@

@shetzel shetzel marked this pull request as ready for review July 31, 2025 19:22
@shetzel shetzel requested a review from a team as a code owner July 31, 2025 19:22
W-19176724: edit messages for "agent activate|deactivate"
@WillieRuemmele
Copy link
Contributor

WillieRuemmele commented Aug 4, 2025

QA Notes

for both library + plugin

✅ : correct error message when activating Copilot_for_Salesforce
✅ : correct error message when deactivating Copilot_for_Salesforce
✅ : no error when activating already active agent
✅ : no error when deactivating agent
✅ : no error when deactivating already inactive agent
✅ : able to activate inactive agent
🆗 : no --json output
🔬 : 2 extra lines printed on invalid api name error

➜  afdx-pro-code-testdrive git:(main) ✗  hub:(GLOBAL - DevHub) scratch:(testdrive)
 ➜  ../../oss/plugin-agent/bin/run.js agent deactivate -n My_Agent1
Error (MissingAgentInOrgError): Agent My_Agent1 not found in org willieruemmele594@agentforce.com.


➜  afdx-pro-code-testdrive git:(main) ✗  hub:(GLOBAL - DevHub) scratch:(testdrive)

✅ : nice error for invalid api name for activate/deactivate
✅ : json response in error conditions with --json

@shetzel shetzel merged commit 394525e into main Aug 4, 2025
11 of 13 checks passed
@shetzel shetzel deleted the sh/activate-agent branch August 4, 2025 21:19
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.

4 participants