Skip to content

Extract shared smoke-test brevity prompt and import across 21 smoke workflows - #52565

Merged
pelikhan merged 3 commits into
mainfrom
copilot/refactor-extract-smoke-test-prompt
Aug 13, 2026
Merged

Extract shared smoke-test brevity prompt and import across 21 smoke workflows#52565
pelikhan merged 3 commits into
mainfrom
copilot/refactor-extract-smoke-test-prompt

Conversation

Copilot AI commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

This refactor removes duplicated “keep output short” prompt text repeated across smoke/conformance workflows and replaces it with a single shared prompt fragment. It standardizes brevity guidance for high-frequency CI smoke tests while reducing maintenance overhead.

  • What changed

    • Added a new shared fragment: .github/workflows/shared/smoke-test-brevity.md.
    • Updated 21 smoke workflows to import this fragment via imports.
    • Removed inline duplicated brevity/efficiency blocks from each workflow body.
  • Canonical shared prompt

    • Centralized the smoke-specific brevity rules (short outputs, minimal context, minimal file reads, targeted queries) into one reusable component.
  • Workflow updates

    • Existing workflows now reference the shared file instead of embedding near-identical text.
    • Generated .lock.yml files were updated to reflect the new import composition.
imports:
  - shared/smoke-test-brevity.md

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor to extract shared smoke-test brevity prompt block Extract shared smoke-test brevity prompt and import across 21 smoke workflows Aug 13, 2026
Copilot AI requested a review from pelikhan August 13, 2026 19:16
@pelikhan
pelikhan marked this pull request as ready for review August 13, 2026 19:27
Copilot AI balanced review requested due to automatic review settings August 13, 2026 19:27
@pelikhan

Copy link
Copy Markdown
Collaborator

@copilot resolve the merge conflicts on this branch.

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

Centralizes smoke-test brevity guidance in one reusable prompt fragment, reducing duplication across 21 workflows.

Changes:

  • Added the shared brevity prompt.
  • Replaced inline guidance with imports in 21 workflows.
  • Regenerated corresponding lock files.
Show a summary per file
File Description
.github/workflows/shared/smoke-test-brevity.md Defines canonical brevity guidance.
.github/workflows/smoke-aider.md Imports shared guidance.
.github/workflows/smoke-aider.lock.yml Regenerates Aider workflow.
.github/workflows/smoke-claude.md Imports shared guidance.
.github/workflows/smoke-claude.lock.yml Regenerates Claude workflow.
.github/workflows/smoke-codex.md Imports shared guidance.
.github/workflows/smoke-codex.lock.yml Regenerates Codex workflow.
.github/workflows/smoke-copilot-aoai-apikey.md Imports shared guidance.
.github/workflows/smoke-copilot-aoai-apikey.lock.yml Regenerates API-key workflow.
.github/workflows/smoke-copilot-aoai-entra.md Imports shared guidance.
.github/workflows/smoke-copilot-aoai-entra.lock.yml Regenerates Entra workflow.
.github/workflows/smoke-copilot-arm.md Imports shared guidance.
.github/workflows/smoke-copilot-arm.lock.yml Regenerates ARM workflow.
.github/workflows/smoke-copilot-auto.md Imports shared guidance.
.github/workflows/smoke-copilot-auto.lock.yml Regenerates auto-model workflow.
.github/workflows/smoke-copilot-mai.md Imports shared guidance.
.github/workflows/smoke-copilot-mai.lock.yml Regenerates MAI workflow.
.github/workflows/smoke-copilot-sdk.md Imports shared guidance.
.github/workflows/smoke-copilot-sdk.lock.yml Regenerates SDK workflow.
.github/workflows/smoke-copilot-small.md Imports shared guidance.
.github/workflows/smoke-copilot-small.lock.yml Regenerates small-model workflow.
.github/workflows/smoke-copilot.md Imports shared guidance.
.github/workflows/smoke-copilot.lock.yml Regenerates Copilot workflow.
.github/workflows/smoke-create-cross-repo-pr.md Imports shared guidance.
.github/workflows/smoke-create-cross-repo-pr.lock.yml Regenerates PR-creation workflow.
.github/workflows/smoke-crush.md Imports shared guidance.
.github/workflows/smoke-crush.lock.yml Regenerates Crush workflow.
.github/workflows/smoke-cursor.md Imports shared guidance.
.github/workflows/smoke-cursor.lock.yml Regenerates Cursor workflow.
.github/workflows/smoke-gemini.md Imports shared guidance.
.github/workflows/smoke-gemini.lock.yml Regenerates Gemini workflow.
.github/workflows/smoke-goose.md Imports shared guidance.
.github/workflows/smoke-goose.lock.yml Regenerates Goose workflow.
.github/workflows/smoke-kiro.md Imports shared guidance.
.github/workflows/smoke-kiro.lock.yml Regenerates Kiro workflow.
.github/workflows/smoke-opencode.md Imports shared guidance.
.github/workflows/smoke-opencode.lock.yml Regenerates OpenCode workflow.
.github/workflows/smoke-pi.md Imports shared guidance.
.github/workflows/smoke-pi.lock.yml Regenerates Pi workflow.
.github/workflows/smoke-pydantic.md Imports shared guidance.
.github/workflows/smoke-pydantic.lock.yml Regenerates Pydantic workflow.
.github/workflows/smoke-update-cross-repo-pr.md Imports shared guidance.
.github/workflows/smoke-update-cross-repo-pr.lock.yml Regenerates PR-update workflow.

Review details

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

  • Files reviewed: 43/43 changed files
  • Comments generated: 0
  • Review effort level: Balanced

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts on this branch.

Resolved and pushed in commit 9ab1cab542.

@pelikhan
pelikhan merged commit 0137e49 into main Aug 13, 2026
@pelikhan
pelikhan deleted the copilot/refactor-extract-smoke-test-prompt branch August 13, 2026 19:47
@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot Triage nudge from sous-chef. This branch is/was conflicting; please run make merge-main to resolve conflicts cleanly, then run the pr-finisher skill before handing this PR back to maintainers.

Context:

  • A maintainer asked to resolve merge conflicts, and a follow-up comment says commit 9ab1cab542 resolved them. Please verify the branch is now conflict-free and ready for maintainer review.

Run: https://github.com/github/gh-aw/actions/runs/31737307049

Generated by 👨🍳 PR Sous Chef
Comment /souschef to run again

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 6.96 AIC · ⌖ 5.48 AIC · ⊞ 8.5K ·
Comment /souschef to run again

github-actions Bot added a commit that referenced this pull request Aug 14, 2026
Extract a writeIfValid helper to remove repeated if(isValid...) writeEnvLine(...)
patterns for trace-id/span-id/parent-span-id output and env propagation.
Behavior unchanged; all log messages and file writes are identical.

Source: recent activity around #52565 (shared prompt extraction) and
#52542 (shared reporting import) motivated looking for similar
duplication-reduction opportunities in actions/setup/js.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

[refactoring] Extract shared component: smoke-test 'keep output short' prompt block

4 participants