Skip to content

fix: preserve preview state and retry missing inputs - #110

Merged
Timpan4 merged 1 commit into
mainfrom
fix/bootstrap-preview-retries
Sep 7, 2026
Merged

Timpan4 merged 1 commit into
mainfrom
fix/bootstrap-preview-retries

Conversation

@Timpan4

@Timpan4 Timpan4 commented Sep 7, 2026 •

Copy link
Copy Markdown
Owner

Missing app manifests could be marked completed, and missing optional-app shortcuts could remain permanently skipped after the input appeared. Required app inputs now fail visibly, absent optional registry and shortcut inputs remain skipped and retryable, and old incorrectly completed shortcut records recover on an ordinary rerun.

Dry runs now preserve saved state, progress, logs, markers, and reports while printing preview output to the console. Force does not reset completed state during preview, and the existing restart guard remains covered.

Validation: seven focused Pester cases passed. Includes production bootstrap execution in Windows PowerShell 5.1 for DryRun, DryRun with Force, and DryRun with PromptRestart, using disposable runtime files and guards against system operations. No live setup operations were run. Independent review found no actionable defects. git diff --check passed.

Closes #46
Closes #47

@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 43 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 43e6d531-dd7c-4181-96f3-4a5c12217133

📥 Commits

Reviewing files that changed from the base of the PR and between 7f1825d and f7439a6.

📒 Files selected for processing (4)
  • bootstrap.ps1
  • modules/BootstrapRun.ps1
  • modules/WinGetInstall.ps1
  • tests/BootstrapInputsPreview.Tests.ps1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Timpan4
Timpan4 merged commit fda4b5d into main Sep 7, 2026
3 checks passed
@Timpan4
Timpan4 deleted the fix/bootstrap-preview-retries branch September 7, 2026 22:25
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.

Keep bootstrap dry runs free of durable state and restart changes Keep missing configuration inputs retryable

1 participant