Skip to content

Use automatic model selection in issue auto-triage workflow - #4659

Merged
paulmedynski merged 3 commits into
mainfrom
dev/automation/connection-encryption-info
Sep 10, 2026
Merged

Use automatic model selection in issue auto-triage workflow#4659
paulmedynski merged 3 commits into
mainfrom
dev/automation/connection-encryption-info

Conversation

@priyankatiwari08

@priyankatiwari08 priyankatiwari08 commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Description

Configures the SqlClient issue auto-triage workflow with model: auto, allowing the GitHub Copilot service to select a relevant available model server-side instead of pinning a specific model. This avoids failures caused by unavailable model defaults while allowing model selection to evolve without workflow updates.

Adds the narrow shell allowlist required by strict-mode compilation, regenerates the workflow lock file with gh-aw v0.88.2, and aligns the lock-verification workflow to the same compiler version.

Issues

Fixes #4640

Testing

  • gh aw compile issue-triage --strict --no-emit
  • gh aw compile issue-triage --strict
  • Recompiled a second time and confirmed the lock file was unchanged
  • GitHub Verify gh aw lock files check passed

Guidelines

  • Tests added or updated — not applicable; validated with the gh-aw strict compiler
  • Public API changes documented — not applicable
  • Verified against customer repro — replaced the unavailable model selection with the supported auto selector
  • Ensured no breaking changes were introduced

Please review the contribution guidelines:

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f7fb5003-4d42-42fa-9f97-7c0986adb093

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.

🔵 Needs a closer look

The regenerated lock workflow introduces a large set of runtime/behavioral changes beyond the model pin, and should get a final human validation pass.

Pull request overview

Pins the SqlClient issue auto-triage agentic workflow to a supported GitHub Copilot model (gpt-5-mini) to address failures caused by an unavailable default model, and regenerates the compiled workflow lockfile.

Changes:

  • Set model: gpt-5-mini in the workflow frontmatter.
  • Add a strict-mode shell tool allowlist entry in the workflow frontmatter.
  • Regenerate issue-triage.lock.yml with gh-aw v0.88.2 and update the gh-aw setup action lock; adjust .gitattributes handling for workflow lock files.
File summaries
File Description
.github/workflows/issue-triage.md Pins the Copilot model to gpt-5-mini and adds a bash tool allowlist in frontmatter.
.github/workflows/issue-triage.lock.yml Regenerated compiled workflow reflecting the pinned model and updated gh-aw runtime/compiler output.
.github/aw/actions-lock.json Updates the pinned github/gh-aw-actions/setup action to v0.88.2.
.gitattributes Keeps lock files marked as generated (linguist), adjusts merge behavior by removing merge=ours.
Review details
  • Files reviewed: 3/4 changed files
  • Comments generated: 0
  • Review effort level: Lite

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

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: fd89be7a-64f7-4d7a-bdd9-eb01183dd36f
Copilot AI review requested due to automatic review settings September 8, 2026 16:05
@priyankatiwari08 priyankatiwari08 changed the title Fix unsupported model in issue auto-triage workflow Use automatic model selection in issue auto-triage workflow Sep 8, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: fd89be7a-64f7-4d7a-bdd9-eb01183dd36f

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.

🔵 Needs a closer look

The regenerated workflow lock file introduces a large set of runtime changes that warrant final human review to ensure the GitHub Actions workflow still behaves as intended.

Review details

Suppressed comments (1)

Previously missed (1) — in code that hasn't changed since the last review.

.gitattributes:5

  • This PR also changes merge behavior for generated workflow lock files by removing merge=ours, but the PR description doesn’t mention or explain this. Since it affects how future lockfile updates are handled during merges, it’d be helpful to document the intent here (or re-add merge=ours if the change was accidental).
  • Files reviewed: 4/5 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

Copilot AI review requested due to automatic review settings September 8, 2026 16:08

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.

🟢 Approval recommended

The workflow frontmatter change is consistently reflected in the regenerated lock file and the verification workflow/compiler pins are aligned, with no detected configuration or syntax issues in the changed regions.

Review details
  • Files reviewed: 4/5 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@cheenamalhotra cheenamalhotra moved this from To triage to In review in SqlClient Board Sep 9, 2026
@paulmedynski
paulmedynski merged commit fde91ee into main Sep 10, 2026
10 of 12 checks passed
@paulmedynski
paulmedynski deleted the dev/automation/connection-encryption-info branch September 10, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[aw] SqlClient Issue Auto-Triage failed

6 participants