Skip to content

Add Claude engine examples to reference documentation - #54099

Closed
pelikhan with Copilot wants to merge 4 commits into
mainfrom
copilot/deep-report-add-engine-claude-examples
Closed

Add Claude engine examples to reference documentation#54099
pelikhan with Copilot wants to merge 4 commits into
mainfrom
copilot/deep-report-add-engine-claude-examples

Conversation

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Engine-agnostic reference pages presented Copilot-only examples, leaving Claude users without copyable workflows and exposing them to silent engine fallback.

  • Claude equivalents
    • Added engine: claude variants for imports and Serena examples.
    • Added Claude configurations for threat detection and feature flags.
  • Example
    ---
    on: issues
    engine: claude
    
    imports:
      - shared/common-tools.md
    ---

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 29.9 AIC · ⌖ 8.24 AIC · ⊞ 9.3K ·
Comment /souschef to run again

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add engine: claude examples to reference/imports.md, serena.md, threat-detection.md, feature-flags.md Add Claude engine examples to reference documentation Aug 20, 2026
Copilot AI requested a review from pelikhan August 20, 2026 00:35
@pelikhan
pelikhan marked this pull request as ready for review August 20, 2026 00:48
Copilot AI balanced review requested due to automatic review settings August 20, 2026 00:48

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

Adds Claude variants to engine-agnostic reference documentation to prevent unintended Copilot fallback.

Changes:

  • Adds Claude examples for imports and Serena.
  • Documents Claude-compatible threat detection and feature flags.
Show a summary per file
File Description
reference/imports.md Adds Claude import examples.
reference/serena.md Adds Claude Serena workflows.
reference/threat-detection.md Adds Claude threat-detection configurations.
reference/feature-flags.md Adds Claude feature-flag examples.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Suppressed comments (1)

docs/src/content/docs/reference/threat-detection.md:404

  • The nested engine config changes only the detector. Without a top-level engine: claude, the copied workflow's main agent defaults to Copilot, so this Claude equivalent still creates a mixed Copilot/Claude workflow. Include the main engine selection in the snippet.
safe-outputs:
  create-pull-request:
  threat-detection:
    steps:
      - name: Connect to AI Gateway
  • Files reviewed: 4/4 changed files
  • Comments generated: 2
  • Review effort level: Balanced

Comment thread docs/src/content/docs/reference/threat-detection.md
Comment on lines +417 to +418
- .github/agents/code-reviewer.md # local
- githubnext/shared-agents/.github/agents/security-reviewer.md@v1.0.0 # remote, pinned
@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot This PR still needs a maintainer-facing finish pass.

Current blockers:

  • The latest review summary says one suppressed issue still needs fixing: the Claude example in reference/threat-detection.md should also set the top-level engine: claude, otherwise the workflow remains mixed-engine.
  • Please refresh the branch if GitHub offers it.
  • Then run the pr-finisher skill and leave a concise maintainer-facing handoff with validation and any remaining risks.

Generated by 👨🍳 PR Sous Chef

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 29.9 AIC · ⌖ 8.24 AIC · ⊞ 9.3K ·
Comment /souschef to run again

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

@copilot This PR still needs a maintainer-facing finish pass.

Current blockers:...

Fixed the Claude threat-detection examples to set top-level engine: claude and made the alternate agent import commented so the example remains valid. Changes are in commit 48a2423. Local formatting, lint, build, schema checks, and impacted tests pass; the docs build could not run because docs/node_modules is unavailable.

Copilot AI requested a review from gh-aw-bot August 20, 2026 01:37
@pelikhan pelikhan closed this Aug 20, 2026
@github-actions
github-actions Bot deleted the copilot/deep-report-add-engine-claude-examples branch August 27, 2026 03:38
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.

[deep-report] Add engine: claude examples to reference/imports.md, serena.md, threat-detection.md, feature-flags.md

4 participants