Skip to content

Conversation

@soridalac
Copy link
Contributor

@soridalac soridalac commented Nov 4, 2025

What does this PR do?

Add support ignoreConflicts param for project deploy/retrieve

What issues does this PR fix or reference?

@W-20002028@

GH discussion: forcedotcom/mcp#24

soridalac and others added 10 commits November 3, 2025 15:27
* Add telemetry to list_devops_projects tool

* Address PR feedback: Use throw pattern and remove PII

- Convert fetchProjects to throw errors instead of returning them
- Remove duplicate telemetry tracking (eliminates instanceof check)
- Remove username field from telemetry events (PII concern)
- Update tests to match throw pattern behavior
- Simplifies error handling from 3 paths to 2 paths (-19 lines)

---------

Co-authored-by: Manan Dey <manan.dey@salesforce.com>
* feat: add e2e test for AFV

* fix: update e2e and reinstall deps

* fix: update AFV e2e

* chore: Add GH_TOKEN to e2e workflow (#316)

* chore: update main/merge

* chore: skip test for win

---------

Co-authored-by: Cristian Dominguez <6853656+cristiand391@users.noreply.github.com>
Co-authored-by: Willhoit <iowillhoit@users.noreply.github.com>
Auto-update dependency after provider package publish.

Related release: mcp-provider-dx-core@0.4.0
Auto-update dependency after provider package publish.

Related release: mcp-provider-devops@0.1.9
@soridalac soridalac requested a review from a team as a code owner November 6, 2025 22:55
@soridalac soridalac requested a review from a team as a code owner November 7, 2025 17:35
@cristiand391
Copy link
Member

QA notes:

🔴 re-deploying with ignoreConflicts: true still fails with conflicts detected
copilot:
Screenshot 2025-11-18 at 10 42 31

mcp-inspector:

  1. deploy without ignoreConflicts, the code defaults to false internally:
Screenshot 2025-11-18 at 10 48 25
  1. re-deploy with ignoreConflicts: true, same error
Screenshot 2025-11-18 at 10 49 12

it does work If I restart the server and the first deploy sets ignoreConflicts: true, that might be why the E2E test works 🤔
Screenshot 2025-11-18 at 10 51 01

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.

5 participants