Skip to content

Acknowledge API parameter hardcoding approach#243

Draft
Copilot wants to merge 1 commit intomainfrom
copilot/sub-pr-240-again
Draft

Acknowledge API parameter hardcoding approach#243
Copilot wants to merge 1 commit intomainfrom
copilot/sub-pr-240-again

Conversation

Copy link

Copilot AI commented Jan 7, 2026

This PR addresses feedback on #240 regarding the fields parameter added to registry API calls for payload optimization.

Changes

  • No code changes required
  • Confirmed with maintainers that hardcoding the fields parameter in API requests is acceptable since the team owns the registry API

The implementation in commit 0b025ef remains as-is:

params = {"sort": _searchkey, "limit": _apiquerylimit, "fields": ",".join(_fields)}

This approach reduces API response payload size by requesting only required fields without adding validation overhead.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Update registry-based archive generation reliability improvements Acknowledge API parameter hardcoding approach Jan 7, 2026
Copilot AI requested a review from jordanpadams January 7, 2026 23:52
Base automatically changed from fix-fips-mode-md5-issue-226 to main January 8, 2026 20:37
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.

2 participants

Comments