Skip to content

Fix BTP CLI syntax errors in subaccount-roles-import skill#13

Open
tsvetanmadanski-sap wants to merge 2 commits into
mainfrom
fix-subaccount-import-roles
Open

Fix BTP CLI syntax errors in subaccount-roles-import skill#13
tsvetanmadanski-sap wants to merge 2 commits into
mainfrom
fix-subaccount-import-roles

Conversation

@tsvetanmadanski-sap

Copy link
Copy Markdown
Contributor

Summary

  • Fix CF_SUBACCOUNT_ID assignment with os.path.exists() guard to prevent silent failure when config file is absent
  • Fix btp list security/app field names: .name/.appId.xsappname/.appid (lowercase)
  • Fix btp add security/role syntax: replace non-existent flags with correct positional args and flags
  • Add handling for read-only role collections created by authentication-xsuaa
  • Fix inconsistent appId camelCase in jq verification commands
  • Update btp-cli-reference SKILL.md and reference files to match corrected syntax

Test plan

  • Verify btp add security/role command syntax against BTP CLI v2.106.1
  • Verify btp list security/app returns .xsappname and .appid fields
  • Test skill execution when .migration/cf-migration-config.json is absent

- Fix CF_SUBACCOUNT_ID assignment with os.path.exists() guard
- Fix btp list security/app field names: .name/.appId → .xsappname/.appid
- Fix btp add security/role syntax: correct positional args and flags
- Add handling for read-only role collections
- Fix jq verification command casing inconsistency
- Fix btp add/remove security/role command syntax throughout
- Correct jq field references: .name/.appId → .xsappname/.appid
- Update Common Issues table to reflect correct flag names
- Fix JSON schema field names for security/app responses
@cla-assistant

cla-assistant Bot commented Jul 13, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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