Skip to content

fix: skip safe dependency update PR when existing PR is open#1335

Merged
Mossaka merged 1 commit intomainfrom
copilot/update-safe-dependency-updates
Mar 17, 2026
Merged

fix: skip safe dependency update PR when existing PR is open#1335
Mossaka merged 1 commit intomainfrom
copilot/update-safe-dependency-updates

Conversation

Copy link
Contributor

Copilot AI commented Mar 17, 2026

The dependency-security-monitor workflow creates a new [Deps] PR on every run, even when one is already open.

  • Added Phase 3.3 to check for existing open PRs with the [Deps] title prefix before creating a new one
  • If found, skips PR creation and notes it in the summary
  • Mirrors the existing duplicate-check pattern already used for security issues (Phase 1.3)
  • Recompiled lock file

Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
@Mossaka Mossaka marked this pull request as ready for review March 17, 2026 18:14
@Mossaka Mossaka self-requested a review as a code owner March 17, 2026 18:14
Copilot AI review requested due to automatic review settings March 17, 2026 18:14
@github-actions
Copy link
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 86.05% 86.16% 📈 +0.11%
Statements 85.97% 86.09% 📈 +0.12%
Functions 86.13% 86.13% ➡️ +0.00%
Branches 79.21% 79.28% 📈 +0.07%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/docker-manager.ts 87.4% → 87.9% (+0.49%) 86.8% → 87.2% (+0.46%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

Copy link
Contributor

Copilot AI left a comment

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 Dependency Security Monitor automation by refining the runbook instructions and regenerating the compiled workflow to a newer gh-aw compiler output, including changes to activation artifacts, safe-outputs handling, and inline threat-detection execution.

Changes:

  • Add a new runbook step to detect and skip creating duplicate open [Deps] dependency update PRs.
  • Regenerate dependency-security-monitor.lock.yml with gh-aw v0.60.0, updating prompts/safe-outputs wiring and refactoring threat detection to run inline.
  • Adjust artifact handling and several workflow/job outputs to support the new activation + agent flow.

Reviewed changes

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

File Description
.github/workflows/dependency-security-monitor.md Adds guidance to skip PR creation when an existing [Deps] dependency update PR is already open.
.github/workflows/dependency-security-monitor.lock.yml Regenerated workflow with gh-aw v0.60.0; updates setup action reference, prompt construction, artifacts, safe-outputs config, and inlines threat detection.

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

steps:
- name: Setup Scripts
uses: github/gh-aw/actions/setup@v0.47.0
uses: github/gh-aw-actions/setup@v0.60.0
Comment on lines 92 to +97
- name: Checkout .github and .agents folders
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
with:
fetch-depth: 1
persist-credentials: false
sparse-checkout-cone-mode: true
fetch-depth: 1
@github-actions
Copy link
Contributor

🔮 Oracle smoke test ledger for PR #1335
PR titles: "fix: route GHEC Copilot proxy to copilot-api subdomain", "fix: fix awf-runner timeout detection and no-docker test timeouts"

  1. GitHub MCP merged PR review: ✅
  2. safeinputs-gh PR query: ❌ (tool unavailable in this runtime)
  3. Playwright github.com title contains "GitHub": ✅
  4. Tavily web search results: ❌ (Tavily tool unavailable; fallback search returned 0)
  5. File write test: ✅
  6. Bash cat verification: ✅
  7. Discussion query+comment flow: ❌ (required discussion tool unavailable)
  8. npm ci && npm run build: ✅
    Overall status: FAIL

🔮 The oracle has spoken through Smoke Codex

Warning

⚠️ Firewall blocked 2 domains

The following domains were blocked by the firewall during workflow execution:

  • ab.chatgpt.com
  • registry.npmjs.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "ab.chatgpt.com"
    - "registry.npmjs.org"

See Network Configuration for more information.

@github-actions
Copy link
Contributor

🏗️ Build Test Suite Results

Ecosystem Project Build/Install Tests Status
Bun elysia 1/1 passed ✅ PASS
Bun hono 1/1 passed ✅ PASS
C++ fmt N/A ✅ PASS
C++ json N/A ✅ PASS
Deno oak N/A 1/1 passed ✅ PASS
Deno std N/A 1/1 passed ✅ PASS
.NET hello-world N/A ✅ PASS
.NET json-parse N/A ✅ PASS
Go color 1/1 passed ✅ PASS
Go env 1/1 passed ✅ PASS
Go uuid 1/1 passed ✅ PASS
Java gson 1/1 passed ✅ PASS
Java caffeine 1/1 passed ✅ PASS
Node.js clsx All passed ✅ PASS
Node.js execa All passed ✅ PASS
Node.js p-limit All passed ✅ PASS
Rust fd 1/1 passed ✅ PASS
Rust zoxide 1/1 passed ✅ PASS

Overall: 8/8 ecosystems passed — ✅ PASS

Generated by Build Test Suite for issue #1335 ·

@Mossaka Mossaka merged commit 35c116f into main Mar 17, 2026
57 checks passed
@Mossaka Mossaka deleted the copilot/update-safe-dependency-updates branch March 17, 2026 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants