Skip to content

Add repair baseline install support#514

Merged
PrzemyslawKlys merged 9 commits into
mainfrom
codex/repair-required-resource
Jul 7, 2026
Merged

Add repair baseline install support#514
PrzemyslawKlys merged 9 commits into
mainfrom
codex/repair-required-resource

Conversation

@PrzemyslawKlys

@PrzemyslawKlys PrzemyslawKlys commented Jul 6, 2026

Copy link
Copy Markdown
Member

Summary- add Repair-ManagedModule -InstallMissing so named baselines can install missing modules while preserving installed-only behavior by default- let Repair-ManagedModule consume -RequiredResource / -RequiredResourceFile as desired-state repair inputs, including .psd1 / .json files and JSON strings, per-resource delivery options and opt-outs, explicit scope, wildcard version selectors, repository identity/source preservation, empty-baseline behavior, and -Name filtering- preserve private/auto repair plan commands for missing installs, license acceptance, and profile-routed required-resource repositories- preserve Install-ManagedModule -MinimumVersion / -MaximumVersion on install plan/result requests instead of flattening them into only VersionPolicy- share required-resource parsing with Install-ManagedModule and document the baseline repair flow## ExamplepowershellRepair-ManagedModule ` -Name $m365Modules ` -InstallMissing ` -Latest ` -Repository PSGallery ` -Scope CurrentUser ` -Plan ` -ShowSummary## Validation- dotnet build .\PSPublishModule\PSPublishModule.csproj -c Release -f net472 --nologo- focused PowerForge.Tests repair/install/module-state suite: 162 passed- git diff --check

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5a040e5f2f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs Outdated
Comment thread PSPublishModule/Cmdlets/InstallManagedModuleCommand.cs Outdated
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 85969dd24c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs Outdated
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs
Comment thread PowerForge/Services/ModuleState/ModuleStatePlanner.cs
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs
Comment thread PSPublishModule/Cmdlets/ManagedModuleRequiredResourceSupport.cs
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4941c04c5d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs Outdated
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs Outdated
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs Outdated
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs Outdated
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5b5f6a6db6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs
Comment thread PSPublishModule/Cmdlets/ManagedModuleRequiredResourceSupport.cs

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

if (transport != ModuleStateDeliveryTransport.ManagedModule)
return "Repair-ManagedModule";

P2 Badge Include InstallMissing in private install commands

When an Install action comes from -InstallMissing or a required-resource baseline and the requested transport is PrivateModule/Auto, this chooses Repair-ManagedModule for the prepared command but the argument builder never adds -InstallMissing. Re-running the emitted command with only -Name for a module that is still absent builds an empty desired state and performs no install, so private/auto plan output is not reproducible for the new missing-module repair path.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread PowerForge/Services/ModuleState/ModuleStateApplyService.cs
Comment thread PSPublishModule/Cmdlets/ManagedModuleRequiredResourceSupport.cs Outdated
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0f79e46ec4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread PowerForge/Services/ModuleState/ModuleStatePlanner.cs
Comment thread PSPublishModule/Cmdlets/RepairManagedModuleCommand.cs Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3d3b512294

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread PowerForge/Services/ModuleState/ModuleStateRepairPlanner.cs
Comment thread PowerForge/Services/ModuleState/ModuleStateApplyService.cs
@PrzemyslawKlys PrzemyslawKlys merged commit 96ba8d8 into main Jul 7, 2026
6 checks passed
@PrzemyslawKlys PrzemyslawKlys deleted the codex/repair-required-resource branch July 7, 2026 06:33
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.

1 participant