Skip to content

[test] Verify PR #8 transient draft preview links#11

Open
harshita-boonlia-glean wants to merge 4 commits into
mainfrom
verify/pr-8-transient-preview
Open

[test] Verify PR #8 transient draft preview links#11
harshita-boonlia-glean wants to merge 4 commits into
mainfrom
verify/pr-8-transient-preview

Conversation

@harshita-boonlia-glean

Copy link
Copy Markdown
Contributor

Summary

Verification PR for #8 (transient workflows for draft previews).

Includes PR #8 changes (sync-agents.sh, comment-pr.sh) merged with main, plus a small edit to glean-sync-action-test-bench instructions to trigger draft preview sync.

What to verify

  • Draft preview POST goes to POST /rest/api/v1/agents (not /agents/{id})
  • Request body includes transient: true and parentWorkflowId: 302f462f7adb4675b111cc486f826bab
  • Response workflow.id captured as previewId
  • PR comment links to /chat/agents/{previewId}/preview (not /edit)

Test plan

  • glean-agent-sync workflow passes on this PR
  • Open preview link from PR comment and confirm transient preview loads

Made with Cursor

cursoragent and others added 3 commits June 8, 2026 13:47
Co-authored-by: rishabh-chawla-glean <rishabh-chawla-glean@users.noreply.github.com>
Touches agent instructions only to exercise transient workflow preview
sync and /preview link in PR comments.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown

Glean Agent Sync — Draft Preview

Agent Status Preview
Glean Sync Action Test Bench ✅ Draft Preview Preview in Glean

Updated by glean-io/agent-sync-action

@harshita-boonlia-glean

Copy link
Copy Markdown
Contributor Author

Tested on salessavvy-test

cc: @rishabh-chawla-glean @devansh-dwivedi-glean

@devansh-dwivedi-glean devansh-dwivedi-glean left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also there are merge conflicts.

with:
instance-url-fe: https://app.glean.com
instance-url-be: https://scio-prod-be.glean.com/
instance-url-be: https://salessavvy-test-be.glean.com/

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@harshita-boonlia-glean Can we update the instance urls to be mocked, rather than storing real instance urls here.

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.

3 participants