Update Archivx to publish diagrams through pull requests - #55563
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot resolve the merge conflicts on this branch. |
There was a problem hiding this comment.
Pull request overview
Updates Archivx to publish generated HTML diagrams through scoped draft pull requests instead of artifacts.
Changes:
- Configures constrained
create-pull-requestsafe output. - Renders diagrams into
scratchpad/archivx/. - Regenerates the compiled workflow.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/archivx-agentic-workflows-analyzer.md |
Updates publication flow and prompt. |
.github/workflows/archivx-agentic-workflows-analyzer.lock.yml |
Applies generated PR-safe-output jobs and permissions. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 1
- Review effort level: Balanced
| ``` | ||
|
|
||
| Then call `upload_artifact` **exactly once** with `path: "archivx-animated-diagram.html"` (the path is relative to the staging directory, not an absolute path). Only one upload is allowed per run. | ||
| Then call `create_pull_request` **exactly once** with a title like `[archivx] Update animated workflow diagram — YYYY-MM-DD` and a body summarizing the data range, main findings, and validation result. The pull request must include only `scratchpad/archivx/agentic-workflows-archivx.html`. |
|
@copilot generate svg diagrams that can be inserted as images in the docs (store somewhere in the docs) |
|
👍 Great work on the Archivx workflow update! This PR cleanly refactors the diagram publishing mechanism to use PR-backed storage instead of artifacts, which is a solid architectural improvement. What looks good:
This is ready for review and merge. The workflow refactor addresses the original artifact staging issue while maintaining safety boundaries.
|
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Resolved by merging |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Implemented in |
|
🎉 This pull request is included in a new release. Release: |
Archivx was generating animated workflow diagrams as uploaded artifacts, which failed when the artifact file was not present in the safe-output staging directory. This changes the workflow to commit the generated HTML diagram through a scoped draft pull request instead.
Safe output
upload-artifactwithcreate-pull-requestscratchpad/archivx/*.htmlWorkflow prompt
Generated workflow
archivx-agentic-workflows-analyzer.lock.ymlfrom the updated markdown source