feat: accept workspace publish requests - #9339
Conversation
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
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.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 52ef600. Configure here.
BYK
left a comment
BYK
left a comment
There was a problem hiding this comment.
I was hoping that we could replace all regexp based parsing with peggy at this point.
| owner: getsentry # create token with access to all getsentry repos | ||
|
|
||
| - name: Check out publish controller | ||
| uses: actions/checkout@v5 |
There was a problem hiding this comment.
| uses: actions/checkout@v5 | |
| uses: actions/checkout@v7 |
Let's go with latest version
| requires_workspace_discovery="$(node -e ' | ||
| const { needsWorkspaceDiscovery } = require("./.__publish__/src/modules/publish-location"); | ||
| const input = JSON.parse(process.env.PUBLISH_ARGS || ""); | ||
| process.stdout.write(String(needsWorkspaceDiscovery(input))); | ||
| ')" |
There was a problem hiding this comment.
Why not put this into a file too? Even better, make it a runnable node script to avoid passing JS through YAML and bash?
| @@ -0,0 +1,45 @@ | |||
| { | |||
| function join(characters) { | |||
There was a problem hiding this comment.
I wish this file was generated from the code block in docs/publish-issue-format.md (or vice versa, the code block was generated from this) so there's only one source of truth.
| @@ -0,0 +1,29 @@ | |||
| const CHECK_RUNS_LINK = | |||
There was a problem hiding this comment.
I wonder if this can be part of the peggy parser/language too?
1 participant
Summary
Validation
Deployment
Deploy this controller before Craft PR #872. It adds the title parser and secure state contract required for workspace-qualified requests.