Skip to content

chore: enable smoke workflows to run on all PRs#144

Merged
Mossaka merged 2 commits intomainfrom
copilot/update-workflows-to-run-for-all-prs
Dec 19, 2025
Merged

chore: enable smoke workflows to run on all PRs#144
Mossaka merged 2 commits intomainfrom
copilot/update-workflows-to-run-for-all-prs

Conversation

Copy link
Contributor

Copilot AI commented Dec 19, 2025

Changed smoke-claude.lock.yml and smoke-copilot.lock.yml to trigger on all PRs instead of only when the "smoke" label is added.

Changes

  • Removed types: [labeled] constraint from pull_request trigger (now uses default events: opened, synchronize, reopened)
  • Removed label filtering condition from activation and pre_activation job conditions

Before:

"on":
  pull_request:
    types:
    - labeled

After:

"on":
  pull_request:

The fork protection condition github.event.pull_request.head.repo.id == github.repository_id is preserved.

Original prompt

change https://github.com/githubnext/gh-aw-firewall/blob/main/.github/workflows/smoke-claude.lock.yml and https://github.com/githubnext/gh-aw-firewall/blob/main/.github/workflows/smoke-copilot.lock.yml to always run for every PRs


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@github-actions
Copy link

💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...]

@github-actions
Copy link

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@Mossaka Mossaka marked this pull request as ready for review December 19, 2025 17:52
@github-actions
Copy link

Smoke Test Results

Last 2 merged PRs:

Test Results:

  • ✅ GitHub MCP: Retrieved PR data successfully
  • ✅ File Writing: Created /tmp/gh-aw/agent/smoke-test-copilot-20378114503.txt
  • ✅ Bash Tool: Verified file content

Overall Status: PASS

📰 BREAKING: Report filed by Smoke Copilot fer issue #144 🗺️

@github-actions
Copy link

Smoke Test Results - Claude Engine

Recent PRs:

Test Results:

  • ✅ GitHub MCP: Retrieved last 2 merged PRs
  • ✅ File Writing: Created test file successfully
  • ✅ Bash Tool: Verified file content
  • ❌ Playwright MCP: Page loaded but resources blocked (ERR_TUNNEL_CONNECTION_FAILED)

Overall Status: PARTIAL PASS

Note: Playwright test shows page title contains "GitHub" but asset loading failed due to network restrictions in test environment.

💥 [THE END] — Illustrated by Smoke Claude fer issue #144 🗺️

Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Copilot AI changed the title [WIP] Update workflows to run for every pull request chore: enable smoke workflows to run on all PRs Dec 19, 2025
Copilot AI requested a review from Mossaka December 19, 2025 17:58
@github-actions
Copy link

Test Coverage Report

Metric Coverage Covered/Total
Lines 66.1% 708/1071
Statements 66.27% 729/1100
Functions 70.73% 87/123
Branches 60.89% 232/381
Coverage Thresholds

The project has the following coverage thresholds configured:

  • Lines: 38%
  • Statements: 38%
  • Functions: 35%
  • Branches: 30%

Coverage report generated by `npm run test:coverage`

@Mossaka Mossaka merged commit 212c938 into main Dec 19, 2025
28 checks passed
@Mossaka Mossaka deleted the copilot/update-workflows-to-run-for-all-prs branch December 19, 2025 23:12
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.

2 participants