Skip to content

@W-23154741 - Crud commands for devops entities#515

Merged
ad-shreya merged 9 commits into
mainfrom
ashreya/crud-commands
Jul 23, 2026
Merged

@W-23154741 - Crud commands for devops entities#515
ad-shreya merged 9 commits into
mainfrom
ashreya/crud-commands

Conversation

@ad-shreya

@ad-shreya ad-shreya commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

@W-23154741@

What does this PR do?

New commands added-

  • sf devops pipeline get
  • sf devops pipeline list
  • sf devops pipeline project delete
  • sf devops pipeline stage delete
  • sf devops project update
  • sf devops stage environment delete

What issues does this PR fix or reference?

#, @@

Functionality Before

<insert gif and/or summary>

Functionality After

<insert gif and/or summary>

@ad-shreya
ad-shreya requested a review from a team as a code owner July 17, 2026 08:04
ad-shreya and others added 8 commits July 23, 2026 09:04
Add devops pipeline list (all pipelines) and devops pipeline get
(single pipeline with stages, repositories, and connected projects).
Regenerate schemas for all existing commands.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ad-shreya
ad-shreya force-pushed the ashreya/crud-commands branch from 14ae967 to 2a47fac Compare July 23, 2026 03:51
Added validateSalesforceId() calls to 5 additional utility functions
to prevent SOQL injection vulnerabilities:

- prepareWorkItem.ts: validate workItemId in resolveProjectIdFromWorkItem
- deleteStageEnvironment.ts: validate environmentId before query
- getPipeline.ts: validate pipelineId before query
- detachProject.ts: validate both projectId and pipelineId
- stage/environment/delete.ts: document existing validation via Flags.salesforceId()

All SOQL queries in the codebase now use either validateSalesforceId()
for IDs or escapeSOQL() for string values, preventing injection attacks.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@ad-shreya
ad-shreya merged commit 9bfbb8f into main Jul 23, 2026
13 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