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 <[email protected]>
* 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 <[email protected]>
Co-authored-by: Willhoit <[email protected]>
Auto-update dependency after provider package publish.

Related release: [email protected]
Auto-update dependency after provider package publish.

Related release: [email protected]
@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

soridalac and others added 2 commits January 23, 2026 15:14
* chore: update nut

* chore: bump deps + fix deploy/retrieve

* test: better e2e test coverage

* chore: bump STL lib to latest

* chore: bump sdr lib to latest

* fix: update copyright header

* fix: update more copyright header

* fix: update copyright header

* chore: bump deps

* chore(ci): use stable sf

* Revert "chore(ci): use stable sf"

This reverts commit 86aa6ac.

---------

Co-authored-by: soridalac <[email protected]>
Co-authored-by: Sorida Lac <[email protected]>
@soridalac soridalac requested review from a team as code owners January 28, 2026 23:04
@soridalac soridalac merged commit 63c764b into main Jan 29, 2026
16 checks passed
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