Skip to content

Consolidate security-guard into bot-detection workflow#15053

Merged
pelikhan merged 2 commits intomainfrom
copilot/upgrade-bot-detection-agent
Feb 12, 2026
Merged

Consolidate security-guard into bot-detection workflow#15053
pelikhan merged 2 commits intomainfrom
copilot/upgrade-bot-detection-agent

Conversation

Copy link
Contributor

Copilot AI commented Feb 12, 2026

Removes redundant security-guard workflow by merging its features into bot-detection.

Changes

  • Added to bot-detection: Import of shared/mood.md (adds "focus on quality, security, documentation" guidance)
  • Deleted: security-guard.md and security-guard.lock.yml (1,294 lines removed)

Both workflows had equivalent safe-outputs configuration (noop, messages) and decision frameworks. The only functional difference was the mood import.

# bot-detection.md frontmatter
timeout-minutes: 10
imports:
  - shared/mood.md  # ← Added
---

The compiled lock file now includes the runtime-import directive before the main workflow content.

Original prompt

Review security guard and bot detection agent. Upgrade bot detection prompt with features in security guard (if missing).
Delete security guard.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…ecurity-guard

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review February 12, 2026 01:11
Copilot AI review requested due to automatic review settings February 12, 2026 01:11
Copilot AI changed the title [WIP] Update bot detection agent with security guard features Consolidate security-guard into bot-detection workflow Feb 12, 2026
@pelikhan pelikhan merged commit e5339f6 into main Feb 12, 2026
@pelikhan pelikhan deleted the copilot/upgrade-bot-detection-agent branch February 12, 2026 01:11
Copilot AI requested a review from pelikhan February 12, 2026 01:11
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

This PR consolidates agentic workflow guidance by removing the dedicated security-guard workflow and aligning bot-detection with the shared “mood” guidance that’s used across many other workflows.

Changes:

  • Deleted .github/workflows/security-guard.md and its compiled .github/workflows/security-guard.lock.yml.
  • Added imports: - shared/mood.md to .github/workflows/bot-detection.md.
  • Recompiled .github/workflows/bot-detection.lock.yml to include the resolved manifest and runtime import for shared/mood.md.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/security-guard.md Removed the security guard agent workflow definition.
.github/workflows/security-guard.lock.yml Removed the compiled lock workflow for security guard.
.github/workflows/bot-detection.md Added shared/mood.md import to align prompt guidance with other workflows.
.github/workflows/bot-detection.lock.yml Updated compiled workflow to include resolved manifest + runtime import for shared/mood.md.

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

Comment on lines +41 to +42
imports:
- shared/mood.md
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

This PR deletes the security-guard workflow, but there are still repo docs/specs that reference security-guard.md / security-guard.lock.yml (e.g. specs/security-architecture-spec-validation.md). Please update or remove those references in the same PR to avoid stale/broken documentation after the workflow deletion.

Suggested change
imports:
- shared/mood.md

Copilot uses AI. Check for mistakes.
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