Skip to content

chore(ci): remove create-issue safe output from smoke tests#188

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/remove-safe-output-github-issues
Closed

chore(ci): remove create-issue safe output from smoke tests#188
Copilot wants to merge 2 commits intomainfrom
copilot/remove-safe-output-github-issues

Conversation

Copy link
Contributor

Copilot AI commented Jan 6, 2026

Remove the create-issue safe output capability from smoke tests (smoke-copilot and smoke-claude) to prevent these workflows from creating GitHub issues.

Changes

Both smoke-copilot.lock.yml and smoke-claude.lock.yml:

  • Removed create-issue: from safe-outputs frontmatter
  • Deleted entire create_issue job (~590 lines each)
  • Removed create_issue from needs arrays in add_comment and conclusion jobs
  • Removed GH_AW_CREATED_ISSUE_* env vars from add_comment job
  • Removed create_issue from config.json, tools.json, and validation schema
  • Updated mermaid dependency diagrams

smoke-claude.lock.yml only:

  • Removed mcp__safeoutputs__create_issue from allowed-tools list
Original prompt

remove safe output of github issues from all the smoke tests (smoke copilot, smoke claude etc.)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove safe output of GitHub issues from smoke tests chore(ci): remove create-issue safe output from smoke tests Jan 6, 2026
Copilot AI requested a review from Mossaka January 6, 2026 01:07
@Mossaka Mossaka marked this pull request as ready for review January 6, 2026 19:01
@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Test Coverage Report

Metric Coverage Covered/Total
Lines 81.02% 1230/1518
Statements 80.98% 1261/1557
Functions 78.16% 136/174
Branches 72.56% 410/565
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`

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