fix: Sends plural data source list query params during pagination

[EspenAlbert]

Summary

• HandleDataSourceReadList kept only pageNum and dropped list filters already built by BuildQueryParamMap.
• Clone existing query params, set pageNum on the copy (WithPageNum).
• Unblocks integration_type on mongodbatlas_log_integrations and integration_type / provider_type on mongodbatlas_metric_integrations.
• Collection-restore plural filters stay optional; the schema hook that hid them is dropped on feat: Adds cloud backup collection restore job resource #4669.

Does not close CLOUDP-433211. Codegen getQueryParams still skips computed_optional (include_system_managed on service accounts).

Test plan

• go test ./internal/common/autogen/ -count=1
• Acc: TestAccCloudBackupCollectionRestoreJob_snapshotSameClusterDatabaseRename second step (source_namespace filter, results.# = 1) on test: Adds collection restore job acceptance tests #4671
• Optional local: TestAccCloudBackupCollectionRestoreJobCollections_envRead (skipped in CI)

[augmentcode]

🤖 Augment PR Summary

Summary: Fixes pagination for auto-generated plural data sources that use query filters.

Changes:

• Retains the query-parameter map built from data-source configuration on every paginated request.
• Introduces WithPageNum to clone that map and apply the paginator’s current pageNum without mutating the original call parameters.
• Adds unit coverage for nil, empty, filtered, and pre-paginated maps, including input immutability.
• This enables list filters such as integration and collection-restore filters to remain effective across pages.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

[Copilot]

Pull request overview

Fixes pagination for auto-generated plural data sources so that existing list filter query parameters are preserved across pages while still sending the correct pageNum. This aligns the autogen list read path with expected Terraform data source filtering behavior and unblocks filtering for several plural data sources.

Changes:

• Introduces WithPageNum to clone existing query params and set pageNum without mutating the original map.
• Updates HandleDataSourceReadList to use WithPageNum(req.CallParams.QueryParams, pageNum) so pagination retains previously built filters.
• Adds unit tests for WithPageNum behavior (nil/empty inputs, overwrites pageNum, preserves filters, and avoids input mutation) and includes release notes.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
internal/common/autogen/query_param.go	Adds WithPageNum helper to clone query params and set pageNum safely.
internal/common/autogen/handle_operations.go	Uses WithPageNum during pagination so list filters are not dropped.
internal/common/autogen/query_params_test.go	Adds unit coverage validating WithPageNum correctness and immutability.
.changelog/4680.txt	Documents the bugfix for plural data source pagination filters.

---

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

[AgustinBettati]

Nice, thanks for quickly triaging and driving this fix

[erabil-mdb]

LGTM!