Bug fix for Set-PSResourceRepository dynamic parameter

[alerickson]

PR Summary

This PR moves CheckRepositoryStore() from BeginProcessing() to the start of GetDynamicParameters() because PowerShell calls them in this order:

• GetDynamicParameters()
• BeginProcessing()
• ProcessRecord()

Previously, dynamic parameter discovery tried to read the repository store before it had been created. On a fresh installation, that threw an exception, so execution never reached the initialization in BeginProcessing().

Initializing immediately before the read fixes that ordering problem using the existing initialization logic. The old BeginProcessing() override was removed because it would only repeat the same check.

PR Context

Resolves issue #2066

PR Checklist

• PR has a meaningful title
  • Use the present tense and imperative mood when describing your changes
• Summarized changes
• Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
• This PR is ready to merge and is not Work in Progress.
  • If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.
• Breaking changes
  • None
  • OR
  • Documentation needed
    • Issue filed:
• User-facing changes
  • Not Applicable
  • OR
  • Documentation needed
    • Issue filed:
• Testing - New and feature
  • N/A or can only be tested interactively
  • OR
  • Make sure you've added a new test if existing tests do not effectively test the code changed
• Tooling
  • I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
  • OR
  • I have considered the user experience from a tooling perspective and enumerated concerns in the summary.

[azure-pipelines]

Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

[alerickson]

/azp run

[azure-pipelines]

Azure Pipelines:
Successfully started running 1 pipeline(s).

[Copilot]

🟡 Changes recommended

The missing-store scenario needs an automated regression test because existing tests pre-create the store.

1 open finding

• Add regression test for missing-store initialization order · New

What changed in this PR

Fixes the fresh-install failure reported in #2066 by initializing the repository store before dynamic parameter discovery reads it.

Changes:

• Moves CheckRepositoryStore() into GetDynamicParameters().
• Removes the redundant BeginProcessing() override.

File	Description
src/​code/​SetPSResourceRepository.cs	Initializes the repository store before dynamic parameter discovery.

🧠 Review effort: Balanced

---

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

[alerickson]

/azp run

[azure-pipelines]

Azure Pipelines:
Successfully started running 1 pipeline(s).