Skip to content

Update Archivx to publish diagrams through pull requests - #55563

Merged
pelikhan merged 3 commits into
mainfrom
copilot/update-agentic-workflow-pr
Aug 25, 2026
Merged

Update Archivx to publish diagrams through pull requests#55563
pelikhan merged 3 commits into
mainfrom
copilot/update-agentic-workflow-pr

Conversation

Copilot AI commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

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

    • Replaced upload-artifact with create-pull-request
    • Limited generated changes to scratchpad/archivx/*.html
    • Kept the agent job read-only while routing repository writes through safe outputs
  • Workflow prompt

    • Renders the Glowmotion HTML output into the repository checkout
    • Instructs the agent to create one pull request containing only the generated diagram
    • Updates the discussion copy to reference the PR-backed diagram path
  • Generated workflow

    • Recompiled archivx-agentic-workflows-analyzer.lock.yml from the updated markdown source
safe-outputs:
  create-pull-request:
    title-prefix: "[archivx] "
    draft: true
    allowed-files:
      - "scratchpad/archivx/*.html"

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan
pelikhan marked this pull request as ready for review August 25, 2026 01:05
Copilot AI balanced review requested due to automatic review settings August 25, 2026 01:05
@pelikhan

Copy link
Copy Markdown
Collaborator

@copilot resolve the merge conflicts on this branch.

Copilot AI 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.

Pull request overview

Updates Archivx to publish generated HTML diagrams through scoped draft pull requests instead of artifacts.

Changes:

  • Configures constrained create-pull-request safe 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`.
@pelikhan

Copy link
Copy Markdown
Collaborator

@copilot generate svg diagrams that can be inserted as images in the docs (store somewhere in the docs)

@github-actions

Copy link
Copy Markdown
Contributor

👍 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:

  • Clear, focused change: only the Archivx workflow files are modified
  • Excellent description with implementation details and safe-output configuration
  • Proper scoping with scratchpad/archivx/*.html allowlist
  • Maintains read-only agent boundary by routing writes through safe outputs
  • Authored by Copilot agent, following the project's agentic development model

This is ready for review and merge. The workflow refactor addresses the original artifact staging issue while maintaining safety boundaries.

Generated by ✅ Contribution Check · copilot · auto · 46.5 AIC · ⌖ 6.86 AIC · ⊞ 9.3K ·

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts on this branch.

Resolved by merging origin/main and fixing the lockfile conflict in commit 27cd7ef.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

@copilot generate svg diagrams that can be inserted as images in the docs (store somewhere in the docs)

Implemented in 7859eac: Archivx now generates SVG diagrams and routes them to docs/src/assets/archivx/ so they can be embedded in docs images.

@pelikhan
pelikhan merged commit 9440560 into main Aug 25, 2026
2 checks passed
@pelikhan
pelikhan deleted the copilot/update-agentic-workflow-pr branch August 25, 2026 01:58
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.87.5

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