Skip to content

refactor(designer): Rename "agentic loop" terminology to "agent" across UI#8792

Merged
ccastrotrejo merged 3 commits intomainfrom
ccastrotrejo/agentRename
Feb 4, 2026
Merged

refactor(designer): Rename "agentic loop" terminology to "agent" across UI#8792
ccastrotrejo merged 3 commits intomainfrom
ccastrotrejo/agentRename

Conversation

@ccastrotrejo
Copy link
Contributor

@ccastrotrejo ccastrotrejo commented Feb 4, 2026

Commit Type

  • feature - New functionality
  • fix - Bug fix
  • refactor - Code restructuring without behavior change
  • perf - Performance improvement
  • docs - Documentation update
  • test - Test-related changes
  • chore - Maintenance/tooling

Risk Level

  • Low - Minor changes, limited scope
  • Medium - Moderate changes, some user impact
  • High - Major changes, significant user/system impact

What & Why

This PR standardizes the agent-related terminology throughout the Logic Apps Designer UI by renaming "agentic loop" to "agent".

Key Changes:

  • UI Labels: Updated button text from "Add an agentic loop" to "Add an agent"
  • Delete Modal: Changed "Delete agentic loop" to "Delete agent" and "Delete agentic loop tool" to "Delete agent tool"
  • Operation Descriptions: Updated summary from "Agent Loop" to "Agent" and description from "Loop in which..." to "Action in which..."
  • Error Messages: Simplified messages like "Cannot add subsequent actions below agentic loops in agent to agent workflows" to "Cannot add subsequent actions below agents in agent to agent workflows"
  • Kind Change Dialog: Updated conversational workflow warning message to use consistent "agent" terminology

Files Modified:

  • libs/designer-ui/src/lib/modals/DeleteNodeModal.tsx
  • libs/designer/src/lib/ui/common/EdgeContextualMenu/EdgeContextualMenu.tsx
  • libs/designer-v2/src/lib/ui/common/EdgeContextualMenu/EdgeContextualMenu.tsx
  • libs/designer-v2/src/lib/ui/common/KindChangeDialog/KindChangeDialog.tsx
  • libs/logic-apps-shared/src/designer-client-services/lib/base/operations/agent.ts
  • libs/logic-apps-shared/src/designer-client-services/lib/consumption/manifests/agentloop.ts
  • libs/logic-apps-shared/src/designer-client-services/lib/consumption/operations/agent.ts
  • libs/logic-apps-shared/src/designer-client-services/lib/standard/manifest/agentloop.ts
  • Localize/lang/strings.json (localization strings updated)

Impact of Change

  • Users: Will see simplified and consistent "agent" terminology in the designer UI instead of "agentic loop"
  • Developers: No API changes; purely UI string updates
  • System: No performance or architecture impact

Test Plan

  • Unit tests added/updated
  • E2E tests added/updated
  • Manual testing completed
  • Tested in: Standalone designer

Contributors

@ccastrotrejo

Screenshots/Videos

CleanShot 2026-02-04 at 13 52 56@2x

- Change description from 'Loop in which...' to 'Action in which...'
- Update UI strings: 'agentic loop' → 'agent', 'agentic loops' → 'agents'
- Update delete modal titles and error messages
- Affects: operation definitions, EdgeContextualMenu, DeleteNodeModal, KindChangeDialog
@github-actions
Copy link

github-actions bot commented Feb 4, 2026

🤖 AI PR Validation Report

PR Review Results

Thank you for your submission! Here's detailed feedback on your PR title and body compliance:

PR Title

  • Current: refactor(designer): Rename "agentic loop" terminology to "agent" across UI
  • Issue: None — title is clear, scoped, and follows conventional commit style.
  • Recommendation: Keep as-is.

Commit Type Commit Type

  • Properly selected (refactor).
  • Only one commit type selected which is correct for these string/UX changes.

Risk Level Risk Level

  • The PR body selects Low and the PR has the risk:low label — these match.
  • Advised risk from diff: risk:low (this PR changes UI/localization strings and non-behavioral labels and descriptions; small text/manifest edits and one unit test added). No change recommended.

What & Why What & Why

  • Current: This PR standardizes the agent-related terminology throughout the Logic Apps Designer UI by renaming "agentic loop" to "agent". The PR lists key UI, modal, and description updates and files modified.
  • Issue: None — the summary is concise and explains intent.
  • Recommendation: Optionally call out any follow-up steps (e.g., informing localization/translation teams about key removals/additions) in this section.

⚠️ Impact of Change Impact of Change

  • Impact section is present and accurate for the most part, but please add a short note about potential consumers of changed manifest/operation descriptions.
  • Recommendation: Expand the Impact section slightly to mention:
    • Users: Minor visible text changes in UI (already included).
    • Developers: Note that operation manifests and summary/description strings changed (search/consumers that depend on exact text/IDs should be verified). Suggest: "Verify no external code or telemetry depends on the exact string values or older localization IDs."
    • System: No perf/architecture impact (already stated).

Test Plan Test Plan

  • Assessment: PR claims Unit tests added/updated and Manual testing completed. The diff includes a new unit test file (KindChangeDialog.spec.tsx) which validates the dialog rendering and close action. E2E tests not added and that is acceptable for a refactor of UI strings provided manual test coverage is documented.
  • Recommendation: If you expect any runtime behavior changes from manifest edits (operationType/summary/description), consider adding a small integration/E2E check or note why E2E isn't necessary. Also ensure CI coverage runs the new unit test.

⚠️ Contributors Contributors

  • Assessment: Contributors section lists @ccastrotrejo. Good.
  • Recommendation: Optionally mention PMs/designers or localization contacts if they helped with wording or translations.

⚠️ Screenshots/Videos Screenshots/Videos

  • Assessment: Screenshot included in PR body. Good to have visual confirmation for UI text changes.
  • Recommendation: None required.

Summary Table

Section Status Recommendation
Title Keep as-is.
Commit Type Keep as-is.
Risk Level Keep risk:low.
What & Why Optionally add translation/next-step note.
Impact of Change ⚠️ Add note to verify consumers of changed manifest strings.
Test Plan Unit test added; consider CI/E2E note if needed.
Contributors ⚠️ Optional: credit any PM/Designer/localization reviewers.
Screenshots/Videos Screenshot included.

Final guidance
Please update only if you want to add the optional clarifications mentioned: a short line in What & Why about notifying localization/translation teams and a small note in Impact of Change to verify consumers of changed manifest/operation strings (if any). Otherwise this PR is good to merge: title and body comply with the template, risk label matches, unit test added, and the changes are consistent with the stated intent.

Thank you for the clear PR — nice, focused refactor!


Last updated: Wed, 04 Feb 2026 20:51:48 GMT

@ccastrotrejo ccastrotrejo changed the title Ccastrotrejo/agent rename refactor: Rename "agentic loop" terminology to "agent" across UI Feb 4, 2026
@ccastrotrejo ccastrotrejo added the risk:low Low risk change with minimal impact label Feb 4, 2026
@github-actions
Copy link

github-actions bot commented Feb 4, 2026

📊 Coverage check completed. See workflow run for details.

@ccastrotrejo ccastrotrejo marked this pull request as ready for review February 4, 2026 20:11
Copilot AI review requested due to automatic review settings February 4, 2026 20:11
@ccastrotrejo ccastrotrejo changed the title refactor: Rename "agentic loop" terminology to "agent" across UI refactor(designer): Rename "agentic loop" terminology to "agent" across UI Feb 4, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR standardizes agent-related terminology across the Logic Apps Designer UI by replacing "agentic loop" with "agent" throughout user-facing text. The changes simplify the language used to describe agents and make it more consistent with the product's naming conventions.

Changes:

  • Updated UI button labels from "Add an agentic loop" to "Add an agent"
  • Changed operation summary from "Agent Loop" to "Agent" and description from "Loop in which..." to "Action in which..."
  • Updated error messages and warning dialogs to use "agent" instead of "agentic loop"
  • Modified localization strings across all affected files

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
libs/logic-apps-shared/src/designer-client-services/lib/standard/manifest/agentloop.ts Updated agent operation description from "Loop" to "Action"
libs/logic-apps-shared/src/designer-client-services/lib/consumption/operations/agent.ts Updated agent operation summary and description terminology
libs/logic-apps-shared/src/designer-client-services/lib/consumption/manifests/agentloop.ts Updated agent manifest description
libs/logic-apps-shared/src/designer-client-services/lib/base/operations/agent.ts Updated base agent operation summary and description
libs/designer/src/lib/ui/common/EdgeContextualMenu/EdgeContextualMenu.tsx Updated contextual menu button text and error messages
libs/designer-v2/src/lib/ui/common/KindChangeDialog/KindChangeDialog.tsx Updated conversational workflow warning message
libs/designer-v2/src/lib/ui/common/EdgeContextualMenu/EdgeContextualMenu.tsx Updated contextual menu button text and error messages in v2 designer
libs/designer-ui/src/lib/modals/DeleteNodeModal.tsx Updated delete modal titles for agent and agent tool
Localize/lang/strings.json Added new localization string IDs and removed obsolete ones

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

📊 Coverage check completed. See workflow run for details.

@ccastrotrejo ccastrotrejo merged commit 8fd7f76 into main Feb 4, 2026
14 checks passed
@ccastrotrejo ccastrotrejo deleted the ccastrotrejo/agentRename branch February 4, 2026 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-validated risk:low Low risk change with minimal impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants