Skip to content

test: drop stale smoke-copilot-pat references - #6587

Merged
lpcox merged 1 commit into
mainfrom
fix-smoke-copilot-pat-test-refs
Jul 25, 2026
Merged

test: drop stale smoke-copilot-pat references#6587
lpcox merged 1 commit into
mainfrom
fix-smoke-copilot-pat-test-refs

Conversation

@lpcox

@lpcox lpcox commented Jul 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

npm test on main fails with 2 failing suites because they still reference the smoke-copilot-pat workflow, which was deleted in #6570.

Root cause

Commit 5e5ae7bf (#6570) deleted .github/workflows/smoke-copilot-pat.md and .github/workflows/smoke-copilot-pat.lock.yml, but left references in two CI test files, causing ENOENT failures:

  • scripts/ci/smoke-copilot-workflow.test.ts
  • scripts/ci/ready-for-aw-workflows.test.ts

Fix

Remove the stale smoke-copilot-pat entries from both test lists.

Verification

Test Suites: 252 passed, 252 total
Tests:       4016 passed, 4016 total

The smoke-copilot-pat workflow was deleted in #6570 but two CI test
suites still referenced its .md and .lock.yml files, causing ENOENT
failures. Remove the stale entries.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 25, 2026 14:30

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

Removes test references to the deleted smoke-copilot-pat workflow, restoring CI test reliability.

Changes:

  • Removes the obsolete source workflow entry.
  • Removes the obsolete compiled lock-file entry.
Show a summary per file
File Description
scripts/ci/smoke-copilot-workflow.test.ts Drops the deleted workflow from smoke tests.
scripts/ci/ready-for-aw-workflows.test.ts Drops the deleted lock file from readiness tests.

Review details

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Medium

@github-actions

Copy link
Copy Markdown
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 98.96% 98.99% 📈 +0.03%
Statements 98.88% 98.91% 📈 +0.03%
Functions 99.41% 99.41% ➡️ +0.00%
Branches 95.30% 95.30% ➡️ +0.00%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/log-directory-setup.ts 96.2% → 100.0% (+3.78%) 96.3% → 100.0% (+3.71%)

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

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.

2 participants