Skip to content

feat(screenshot-scenarios): drive the sandbox via the sandbox-org skill#12143

Open
JulianMaurin wants to merge 1 commit into
mainfrom
devs/JulianMaurin/julian/map-screenshot-scenarios-to-sandbox-org/drive-sandbox-via-sandbox-org-skill--d9aa6ece
Open

feat(screenshot-scenarios): drive the sandbox via the sandbox-org skill#12143
JulianMaurin wants to merge 1 commit into
mainfrom
devs/JulianMaurin/julian/map-screenshot-scenarios-to-sandbox-org/drive-sandbox-via-sandbox-org-skill--d9aa6ece

Conversation

@JulianMaurin

@JulianMaurin JulianMaurin commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Retire the bespoke Mergifyio/sandbox driver: staging now goes through the
sandbox-org skill (mergify-internal plugin) against the mergify-sandbox org,
which owns the repo/queue/freeze mechanics and confines every write to the org.
references/sandbox.md shrinks to the scenario repo choice and a recipe-step →
sandbox.sh command mapping.

Depends-On: https://github.com/Mergifyio/skills/pull/31

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

Copilot AI review requested due to automatic review settings July 17, 2026 18:38
@mergify
mergify Bot had a problem deploying to Mergify Merge Protections July 17, 2026 18:38 Failure
@mergify

mergify Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🔴 3 of 7 protections blocking · waiting on 👀 reviews and ⛓️ dependency

Protection Waiting on
🔴 ⛓️ Depends-On Requirements ⛓️ dependency
🔴 👀 Review Requirements 👀 reviews
🔴 🔎 Reviews 👀 reviews
🟢 🤖 Continuous Integration
🟢 Enforce conventional commit
🟢 📕 PR description
🟢 🚦 Auto-queue

🔴 ⛓️ Depends-On Requirements

Waiting for

  • depends-on = Mergifyio/skills#31 [⛓️ Mergifyio/skills#31]
This rule is failing.

Requirement based on the presence of Depends-On in the body of the pull request

  • depends-on = Mergifyio/skills#31 [⛓️ Mergifyio/skills#31]

🔴 👀 Review Requirements

Waiting for

  • #approved-reviews-by >= 2
This rule is failing.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = mergify-ci-bot

🔴 🔎 Reviews

Waiting for

  • #review-requested = 0
This rule is failing.
  • #review-requested = 0
  • #changes-requested-reviews-by = 0
  • #review-threads-unresolved = 0

Show 4 satisfied protections

🟢 🤖 Continuous Integration

  • all of:
    • check-success = build
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|internal|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?!?:

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@mergify
mergify Bot requested a review from a team July 17, 2026 18:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the internal screenshot-scenarios agent skill documentation to retire the bespoke Mergifyio/sandbox driver and instead stage dashboard states through the sandbox-org skill against the mergify-sandbox org, with sandbox.sh-based command mappings.

Changes:

  • Repoints the skill’s staging flow from Mergifyio/sandbox + raw CLI to sandbox-org (mergify-internal plugin) in the mergify-sandbox org.
  • Rewrites references/sandbox.md from a full driver guide to a scenario-repo selection + recipe-step → sandbox.sh command mapping.
  • Updates recipe/docs text to align (partially) with the new sandbox-org approach.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
.claude/skills/screenshot-scenarios/SKILL.md Switches the skill narrative/prereqs/workflow to sandbox-org-driven staging and sandbox.sh command mapping.
.claude/skills/screenshot-scenarios/references/scenario-recipes.md Updates recipe intro + some steps to reference mergify-sandbox and sandbox.sh (still has some legacy queue wording).
.claude/skills/screenshot-scenarios/references/sandbox.md Replaces bespoke driver docs with scenario repo choice + step→command table + cleanup guidance.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .claude/skills/screenshot-scenarios/references/scenario-recipes.md Outdated
Comment thread .claude/skills/screenshot-scenarios/references/sandbox.md Outdated
Comment thread .claude/skills/screenshot-scenarios/references/scenario-recipes.md
Comment thread .claude/skills/screenshot-scenarios/SKILL.md Outdated
@JulianMaurin
JulianMaurin force-pushed the devs/JulianMaurin/julian/map-screenshot-scenarios-to-sandbox-org/drive-sandbox-via-sandbox-org-skill--d9aa6ece branch from 2a0e4a5 to 12b9a06 Compare July 17, 2026 18:43
@JulianMaurin

JulianMaurin commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

Revision history

# Type Changes Reason Date
1 initial 2a0e4a5 2026-07-17 18:43 UTC
2 rebase 2a0e4a5 → 12b9a06 (rebase only) add Depends-On: skills#31 (screenshot-scenarios consumes sandbox-org) 2026-07-17 18:43 UTC
3 rebase 12b9a06 → 416c073 (rebase only) 2026-07-20 06:52 UTC
4 content 416c073 → 3c621b7 (raw) Address Copilot review: consistent sandbox.sh usage in recipes, explicit baseline-config restore in cleanup, sandbox.sh named in prerequisites 2026-07-20 06:54 UTC
5 rebase 3c621b7 → c9285de (rebase only) 2026-07-20 06:57 UTC

@mergify
mergify Bot had a problem deploying to Mergify Merge Protections July 17, 2026 18:43 Failure
@JulianMaurin
JulianMaurin force-pushed the devs/JulianMaurin/julian/map-screenshot-scenarios-to-sandbox-org/drive-sandbox-via-sandbox-org-skill--d9aa6ece branch from 12b9a06 to 416c073 Compare July 20, 2026 06:52
@mergify
mergify Bot had a problem deploying to Mergify Merge Protections July 20, 2026 06:53 Failure
@JulianMaurin
JulianMaurin force-pushed the devs/JulianMaurin/julian/map-screenshot-scenarios-to-sandbox-org/drive-sandbox-via-sandbox-org-skill--d9aa6ece branch from 416c073 to 3c621b7 Compare July 20, 2026 06:54
@mergify
mergify Bot had a problem deploying to Mergify Merge Protections July 20, 2026 06:55 Failure
Retire the bespoke Mergifyio/sandbox driver: staging now goes through the
sandbox-org skill (mergify-internal plugin) against the mergify-sandbox org,
which owns the repo/queue/freeze mechanics and confines every write to the org.
references/sandbox.md shrinks to the scenario repo choice and a recipe-step →
sandbox.sh command mapping.

Depends-On: Mergifyio/skills#31

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Change-Id: Id9aa6ece794ca228052e351d558da4f1abb655b0
@JulianMaurin
JulianMaurin force-pushed the devs/JulianMaurin/julian/map-screenshot-scenarios-to-sandbox-org/drive-sandbox-via-sandbox-org-skill--d9aa6ece branch from 3c621b7 to c9285de Compare July 20, 2026 06:57
@mergify
mergify Bot had a problem deploying to Mergify Merge Protections July 20, 2026 06:57 Failure
@JulianMaurin
JulianMaurin marked this pull request as ready for review July 20, 2026 11:38
@mergify
mergify Bot requested a review from a team July 20, 2026 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants