Skip to content

Add record_ecosystem_meta to smoke test comparison handler#586

Merged
pavera merged 2 commits intomainfrom
copilot/update-cli-smoke-tests
Mar 4, 2026
Merged

Add record_ecosystem_meta to smoke test comparison handler#586
pavera merged 2 commits intomainfrom
copilot/update-cli-smoke-tests

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 4, 2026

dependabot-core is removing the enable_record_ecosystem_meta feature flag, making record_ecosystem_meta unconditionally emitted after every create_pull_request, update_pull_request, or close_pull_request call. The CLI's compare function had no case for []model.RecordEcosystemMeta, causing smoke tests to fail with "unexpected type" errors.

  • Added []model.RecordEcosystemMeta case to the compare switch in internal/server/api.go
  • Added compareRecordEcosystemMeta comparison function
  • Added tests covering match, mismatch, and dispatch through compare

Companion to dependabot/smoke-tests#389.

Original prompt

This section details on the original issue you should resolve

<issue_title>Update CLI to handle the smoke tests</issue_title>
<issue_description>smoke tests are failing because the cli is getting an error on the output dependabot/smoke-tests#389

Update the CLI to handle this.</issue_description>

Comments on the Issue (you are @copilot in this section)

  • Fixes github/dependabot-updates#12652

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

The enable_record_ecosystem_meta feature flag is being removed from
dependabot-core, making record_ecosystem_meta unconditionally active.
The CLI's compare function was missing a case for this type, causing
smoke tests to fail with "unexpected type" errors.

Co-authored-by: pavera <660677+pavera@users.noreply.github.com>
Copilot AI changed the title [WIP] Update CLI to handle smoke test errors Add record_ecosystem_meta to smoke test comparison handler Mar 4, 2026
@pavera pavera marked this pull request as ready for review March 4, 2026 18:10
@pavera pavera requested review from a team as code owners March 4, 2026 18:10
@pavera pavera added this pull request to the merge queue Mar 4, 2026
Merged via the queue into main with commit 8f9702c Mar 4, 2026
23 of 89 checks passed
@pavera pavera deleted the copilot/update-cli-smoke-tests branch March 4, 2026 18:16
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