Skip to content

fix: Platform-aware shell handling and AgentforceSDK upgrade#150

Merged
haifeng-li-at-salesforce merged 2 commits intoforcedotcom:mainfrom
haifeng-li-at-salesforce:dpCut
Jan 29, 2026
Merged

fix: Platform-aware shell handling and AgentforceSDK upgrade#150
haifeng-li-at-salesforce merged 2 commits intoforcedotcom:mainfrom
haifeng-li-at-salesforce:dpCut

Conversation

@haifeng-li-at-salesforce
Copy link
Copy Markdown
Contributor

What does this PR do?

  • Fix cross-platform command execution by using platform-aware shell option (shell: true on Windows, shell: false on macOS/Linux) to properly handle arguments with spaces
  • Upgrade AgentforceSDK to version 14.13.1 and update related dependencies and type definitions

@haifeng-li-at-salesforce haifeng-li-at-salesforce requested a review from a team as a code owner January 28, 2026 23:42
const childProcess = spawn(command, args, {
env,
shell: true,
shell: isWindows,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

For iOS, setting shell: true causes a single parameter that contains spaces to be interpreted as two separate parameters.

pod 'AgentforceSDK', '14.7.6'
pod 'Messaging-InApp-Core', '1.9.3-Experimental'
pod 'AgentforceSDK', '14.13.1'
pod 'Messaging-InApp-Core', '1.10.2'
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Resolve the build and launch issue

Copy link
Copy Markdown
Contributor

@ben-zhang-at-salesforce ben-zhang-at-salesforce left a comment

Choose a reason for hiding this comment

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

jointed coded it and manually verified that it fixed the build and deploy issue for ios app generation.

@haifeng-li-at-salesforce haifeng-li-at-salesforce merged commit 7bf4125 into forcedotcom:main Jan 29, 2026
7 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.

2 participants