feat(workbenches): implement PR follow-up feature - #789
Open
floreks wants to merge 5 commits into
Open
Conversation
- Add `CreateWorkbenchPRFollowup` method in console client mocks. - Implement `pr-followup` command to send follow-up prompts for workbench jobs. - Include `workbenches` command in CLI to manage Plural workbenches. - Rename `ListaStacks` to `ListStacks` in multiple files for consistency. - Update dependencies in `go.mod`. Add tests for workbench features, including PR follow-up client and resolver logic.
floreks
requested review from
maciaszczykm,
michaeljguarino and
zreigz
as code owners
July 28, 2026 12:40
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
Contributor
Greptile SummaryAdds workbench PR follow-up support to the CLI.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| cmd/command/workbenches/pr_followup_service.go | Adds prompt validation, pull request resolution, follow-up creation, and optional missing-job handling. |
| cmd/command/workbenches/pull_request_resolver.go | Adds provider-aware pull request URL inference from explicit inputs and Git metadata. |
| cmd/command/workbenches/workbenches.go | Defines and wires the new workbenches PR follow-up CLI command. |
| pkg/console/workbenches.go | Adds the Console GraphQL client wrapper for creating workbench PR follow-ups. |
| pkg/console/console.go | Extends the internal Console client contract and consistently applies the stack-listing method rename. |
| go.mod | Updates the Console client and associated transitive dependency selections. |
Reviews (2): Last reviewed commit: "refactor(workbenches): remove pr_followu..." | Re-trigger Greptile
…tallation - Fix typo in `ListStacks` API error response parameter in `stacks.go` - Update Plural CLI installation command with specific commit hash in GitHub Actions workflow
- Delete file `pr_followup_service_test.go` as it is no longer needed - Modify `pr_followup_service.go` to use `console.ConsoleClient` directly - Simplify client interface in `NewPRFollowupService` function
Member
Author
Member
|
@floreks made a few basic modifications to have this use the new queued prompt based mutations, feel free to modify it as you wish (think the gql client code generated weirdly for instance). |
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
2 participants
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.
CreateWorkbenchPRFollowupmethod in console client mocks.pr-followupcommand to send follow-up prompts for workbench jobs.workbenchescommand in CLI to manage Plural workbenches.ListaStackstoListStacksin multiple files for consistency.go.mod.Add tests for workbench features, including PR follow-up client and resolver logic.
Summary
Labels
Test Plan
Checklist