Skip to content

daily-model-inventory: log raw responses in collect steps, drop copilot-api job - #48636

Merged
pelikhan merged 1 commit into
mainfrom
copilot/add-logging-to-collect-steps
Jul 28, 2026
Merged

daily-model-inventory: log raw responses in collect steps, drop copilot-api job#48636
pelikhan merged 1 commit into
mainfrom
copilot/add-logging-to-collect-steps

Conversation

Copilot AI commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Raw model API responses were not visible in action logs, requiring artifact downloads to inspect them. The collect_copilot_api_models job is redundant now that the SDK covers Copilot model data.

Changes

  • Log raw response in each collect step — adds a Log raw response / Log models step (if: always()) after each fetch step so the downloaded content appears directly in job logs:

    Job Logged file
    collect_openai_models openai/raw.json
    collect_anthropic_models anthropic/raw.json
    collect_gemini_models gemini/raw.json
    collect_copilot_billing_models copilot-billing/models.json
    collect_copilot_sdk_models copilot-sdk/models.json
    - name: Log raw response
      if: always()
      shell: bash
      run: cat /tmp/gh-aw/agent/model-inventory/openai/raw.json 2>/dev/null || echo "(no raw.json)"
  • Remove collect_copilot_api_models — drops the job, its artifact upload, and all agent-prompt references (copilot-api-models input bullet, validation source item 3, summary table row). Copilot SDK data (billing fields) is the primary source; the raw API endpoint is no longer needed.

…_models job

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Add raw response logging to collect steps; remove collect_copilot_api_models job daily-model-inventory: log raw responses in collect steps, drop copilot-api job Jul 28, 2026
@pelikhan
pelikhan marked this pull request as ready for review July 28, 2026 13:57
Copilot AI review requested due to automatic review settings July 28, 2026 13:57
Copilot AI requested a review from pelikhan July 28, 2026 13:57
@pelikhan
pelikhan merged commit 6c76795 into main Jul 28, 2026
@pelikhan
pelikhan deleted the copilot/add-logging-to-collect-steps branch July 28, 2026 13:57

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

Logs model inventories directly in workflow logs and removes the redundant Copilot API collector.

Changes:

  • Logs provider and Copilot SDK model responses.
  • Removes the Copilot API job and related prompt references.
  • Regenerates the compiled workflow.
Show a summary per file
File Description
.github/workflows/daily-model-inventory.md Updates collection jobs and analysis instructions.
.github/workflows/daily-model-inventory.lock.yml Reflects the recompiled workflow.

Review details

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Medium

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.83.5

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.

3 participants