Skip to content

chore: add skills for updating native Android/iOS SDK dependencies - #1203

Closed
Robert-SD wants to merge 3 commits into
developfrom
devin/rn-sdk-update-skills
Closed

chore: add skills for updating native Android/iOS SDK dependencies#1203
Robert-SD wants to merge 3 commits into
developfrom
devin/rn-sdk-update-skills

Conversation

@Robert-SD

Copy link
Copy Markdown
Contributor

Adds two agent skills for bumping this plugin's native dependencies:

  • .agents/skills/react-native-update-android-sdk/SKILL.md
  • .agents/skills/react-native-update-ios-sdk/SKILL.md

Migrated from the meta-repo's .agents/workflows (react-native-android-sdk-update.md, react-native-ios-sdk-update.md), since the update procedure only concerns this repository. Mirrors the equivalent skills added to adyen-flutter in Adyen/adyen-flutter#717.

No ticket associated with this change.

Testing notes: reviewed skill steps for accuracy against current repo structure (dependencies.gradle, podspec, README badges, example app build commands).

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces two new agent skill definitions to automate updating the native Android and iOS SDK dependencies for a React Native plugin. The feedback recommends updating the iOS skill instructions to explicitly include staging and committing the example/ios/Podfile.lock file if it is modified during the verification process, ensuring that the example app's dependencies remain properly synchronized.

Comment thread .agents/skills/react-native-update-ios-sdk/SKILL.md
…s review

- Skills already live inside the adyen-react-native repo, so the platform
  prefix was redundant context.
- Explicitly call out staging example/ios/Podfile.lock if pod install
  modifies it, per gemini-code-assist review feedback.
@Robert-SD

Copy link
Copy Markdown
Contributor Author

Fixed in commit b90fcf4 — staging instructions now explicitly call out including example/ios/Podfile.lock if pod install modifies it.

Skills are also exposed from the meta-repo root via symlinks
(Adyen/adyen-checkout-sdk-meta#80), and the directory name is the
invocation identifier. Keeping the generic name here meant the same
skill was invoked as /update-android-sdk locally but
/react-native-update-android-sdk from the meta-repo — restoring the
react-native- prefix keeps the identifier consistent everywhere.
@sonarqubecloud

Copy link
Copy Markdown

@Robert-SD

Copy link
Copy Markdown
Contributor Author

Team decided to keep these skills in the meta-repo instead of the native SDK repo. Superseded by a consolidated skills migration in Adyen/adyen-checkout-sdk-meta.

@Robert-SD Robert-SD closed this Jul 30, 2026
@Robert-SD
Robert-SD deleted the devin/rn-sdk-update-skills branch July 30, 2026 11:25
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.

1 participant