feat(semaphore): add Get Pipeline action component#3108
Open
nenadilic84 wants to merge 6 commits intosuperplanehq:mainfrom
Open
feat(semaphore): add Get Pipeline action component#3108nenadilic84 wants to merge 6 commits intosuperplanehq:mainfrom
nenadilic84 wants to merge 6 commits intosuperplanehq:mainfrom
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
nenadilic84
force-pushed
the
feat/semaphore-get-pipeline
branch
3 times, most recently
from
65445b8 to
cae5eca
Compare
nenadilic84
force-pushed
the
feat/semaphore-get-pipeline
branch
from
cae5eca to
367cefd
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Author
|
Demo video coming shortly — recording the integration setup and fetching pipeline details from Semaphore. |
nenadilic84
force-pushed
the
feat/semaphore-get-pipeline
branch
from
5337965 to
648f6a0
Compare
Add a new GetPipeline component for the Semaphore integration that retrieves a pipeline by ID from the Semaphore API. This enables users to inspect pipeline state and result within SuperPlane workflows. Includes backend component, tests, example output JSON, and frontend mapper for displaying pipeline details. Closes superplanehq#2827 Signed-off-by: Nenad Ilic <nenadilic84@gmail.com>
Use the default state registry to correctly reflect execution status (success, error, running, etc.) instead of hardcoding neutral state. Signed-off-by: Nenad Ilic <nenadilic84@gmail.com>
Signed-off-by: Nenad Ilic <nenadilic84@gmail.com>
Removes the unused ComponentBaseSpec import that causes a TypeScript compilation error under strict mode (noUnusedLocals). Signed-off-by: Nenad Ilic <nenadilic84@gmail.com>
Signed-off-by: Nenad Ilic <nenadilic84@gmail.com>
- Use JSON marshal/unmarshal in test to robustly inspect emitted Pipeline payload instead of direct type assertion - Remove duplicate local stringOrDash helper and import the shared version from mappers/utils.ts Signed-off-by: Nenad Ilic <nenadilic84@gmail.com>
nenadilic84
force-pushed
the
feat/semaphore-get-pipeline
branch
from
1a241a4 to
00808e6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
GetPipelinecomponent to the Semaphore integration that retrieves a pipeline by ID via the Semaphore APITest plan
GetPipelineSetup rejects emptypipelineIdGetPipelineSetup accepts validpipelineIdGetPipelineExecute emits correct payload on default channelgo test ./pkg/integrations/semaphore/...Closes #2827