Restructure results for SalesforceResultState functions

[mtuchi]

Summary

Restructured response format for bulk, create, update and destroy functions:

Fixes #878

Details

Added standardized result structure:

{
    success: boolean,
    completed: number,
    errors: [
      {
        id: string,
        message: string
      }
    ]
}

For bulk function and for destroy() function when deleting multiple records. I have also added integration tests for bulk, update, destroy and create function, to run the test get credentials from LP: Salesforce Demo Org (API user)

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to know!):

• Code generation (copilot but not intellisense)
• Learning or fact checking
• Strategy / design
• Optimisation / refactoring
• Translation / spellchecking / doc gen
• Other
• I have not used AI

You can read more details in our Responsible AI Policy

Review Checklist

Before merging, the reviewer should check the following items:

• Does the PR do what it claims to do?
• If this is a new adaptor, added the adaptor on marketing website ?
• Are there any unit tests?
• Is there a changeset associated with this PR? Should there be? Note that
  dev only changes don't need a changeset.
• Have you ticked a box under AI Usage?

[josephjclark]

The integration tests are cool but I would value unit tests against this function way more!

I won't block the PR for tests on this though (especially as I've requested a change). If you have time for a few tests that's great. If you don't, don't worry.

What I'm really trying to say is: consider unit testing before integration testing. Also when you write a nice clean standalone util function like this, it's a great candidate for unit tests!

[mtuchi]

Thanks for the reminder I have added couple of unit test for formatResults