Skip to content

Conversation

@Bhavd13
Copy link
Contributor

@Bhavd13 Bhavd13 commented Jan 5, 2026

Commit Type

  • feature - New functionality
  • fix - Designer 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

Adds the A2A (Agent-to-Agent) Request operation and connector group to the Consumption search service, making it discoverable in the designer's action/trigger search menu.

Impact of Change

  • Users: Users can now find and add A2A Request triggers when building Consumption Logic Apps workflows
  • Developers: None
  • System: None - additive change only

Test Plan

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

Contributors

@Bhavd13

Screenshots/Videos

A2A.search.trigger.video.mp4

Copilot AI review requested due to automatic review settings January 5, 2026 09:44
@github-actions
Copy link

github-actions bot commented Jan 5, 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: fix(designer): Add A2A Request operation and connector group to Consumption search menu
  • Issue: None — title is concise and descriptive, follows conventional commit style and indicates scope and intent.
  • Recommendation: None required. (Optional: keep this exact style for future small/designer fixes.)

Commit Type

  • Properly selected (fix - Designer fix).
  • Only one commit type selected, which is correct.

Risk Level

  • The PR body selects Low and the repository labels include risk:low — these match.

What & Why

  • Current: Adds the A2A (Agent-to-Agent) Request operation and connector group to the Consumption search service, making it discoverable in the designer's action/trigger search menu.
  • Issue: None — concise and clear.
  • Recommendation: None.

⚠️ Impact of Change

  • The change is small and additive (2 additions across 1 file) — impact is minimal.
  • Recommendation:
    • Users: Users can now discover the A2A Request trigger in Consumption designer search (already stated). No change required.
    • Developers: Consider noting in the Impact section whether any other services or consumers (e.g., telemetry, search indexing jobs) need to be aware of the new connector group/operation (likely no, but calling it out helps reviewers).
    • System: Add a one-line confirmation that no schema or runtime contract changes were made, if that is indeed the case.

⚠️ Test Plan

  • The PR marks Manual testing completed and lists "Tested in: Local standalone designer".
  • Assessment: Acceptable for this small UI/discovery add, but automated coverage would be preferable.
  • Recommendation:
    • If feasible, add a small unit test (or an integration/E2E test) asserting that the built-in operations/connectors list includes the new A2A entries. This prevents regressions if the operations list is later refactored.
    • If automated tests are not practical here, update the Test Plan with a short justification explaining why (e.g., "cannot be unit-tested due to X; covered by manual test Y in local environment and validated with link to recording/screenshot").

Contributors

  • Current: @Bhavd13 is listed.
  • Assessment: Good — contributor credited.

⚠️ Screenshots/Videos

  • A link to an attachment is present in the body.
  • Assessment: Good to include visuals for UI changes.
  • Recommendation: Ensure the linked asset is accessible to maintainers (link works from GitHub PR UI). If possible, embed a small inline screenshot in the PR body for quick review.

Summary Table

Section Status Recommendation
Title Keep as-is.
Commit Type No change needed.
Risk Level Matches labels.
What & Why Clear and concise.
Impact of Change ⚠️ Add a brief note on downstream consumers or confirm none exist.
Test Plan ⚠️ Prefer automated test; otherwise justify manual-only testing.
Contributors Good.
Screenshots/Videos ⚠️ Ensure asset link is accessible and consider embedding.

Final Message:
This PR is small, additive, and low-risk — it passes the PR title/body checklist. The advised risk level (risk:low) matches the label on the PR. Please consider the following minimal follow-ups before merge:

  • Add or point to a small automated test that validates the new operation/group is included in built-in lists, or add a short justification in the Test Plan explaining why automated tests are not feasible.
  • Confirm the screenshot/video link is accessible to reviewers; if possible paste the image inline into the PR description.
  • Optionally, add one sentence in Impact stating that no other services (indexers, telemetry, runtime contract) are affected by this change.

After making any optional clarifications, this PR is ready to merge.


Last updated: Fri, 09 Jan 2026 17:03:35 GMT

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 adds support for the A2A (Agent-to-Agent) Request trigger to the Consumption Logic Apps search service, making it discoverable in the designer's operation search menu. This is a straightforward additive change that follows established patterns in the codebase.

Key Changes

  • Adds a2aRequestOperation to the consumption built-in operations list
  • Adds a2aRequestGroup to the consumption built-in connectors list
  • Both additions follow the existing pattern for similar trigger/action types (AS2, RosettaNet)

@Bhavd13 Bhavd13 changed the title Add A2A trigger in search menu Add A2A Request operation and connector group to Consumption search menu Jan 5, 2026
@Bhavd13 Bhavd13 changed the title Add A2A Request operation and connector group to Consumption search menu fix(designer): Add A2A Request operation and connector group to Consumption search menu Jan 9, 2026
@takyyon takyyon added the risk:low Low risk change with minimal impact label Jan 9, 2026
@takyyon takyyon merged commit 4d13be2 into Azure:main Jan 9, 2026
13 checks passed
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.

2 participants