Skip to content

feat: unify source recovery and retain export results - #443

Merged
guan4tou2 merged 2 commits into
mainfrom
codex/post-merge-mac-acceptance
Oct 25, 2026
Merged

guan4tou2 merged 2 commits into
mainfrom
codex/post-merge-mac-acceptance

Conversation

@guan4tou2

@guan4tou2 guan4tou2 commented Oct 24, 2026 •

Copy link
Copy Markdown
Owner

Changes

Shell connection and repair in Settings now reuse the existing verified onboarding flow. Dependency preparation stays in its source page, bundled-source recovery preserves project permissions, and source status refreshes when returning to the app.

Successful exports retain the artifact path and completion time in the dialog. Reopening the dialog recovers the project's last successful export; missing files and failed metadata/reveal operations remain explicit. ExportPlan selection and attachment policy are unchanged.

Also clears a resolved project-specific settings error after retry, corrects manual shell selection versus detected shell, and consolidates duplicate UI checks and Electron launches. HUD remains enabled by default; non-HUD E2E launches suppress only the test overlay.

Validation

  • Ordered local typecheck/spec/architecture/i18n/unit gates passed: 4,250 passed, 28 skipped.
  • Settings 6 and export 8 Electron behavior cases passed; final source/HUD cases 2/2 passed.
  • Native Mac source/settings/export and smaller-window checks are recorded in Spec 062 verification.md.
  • Fresh Mac arm64 DMG built; package resources, optional AI checksums, app seal and DMG integrity passed. The refreshed DMG was not manually replayed again.

Spec 062 stays Draft. T008–T013, real Windows/WSL and broader installation/update acceptance remain open. This is an internal-test candidate, not a release or notarization claim.

Refs #434, #435, #436.

CI correction

The first run passed macOS/Ubuntu unit and installer smoke but failed Windows's existing late-discovery AI pause assertion; E2E was skipped. Commit ccf0347 reproduces the filesystem/Date.now clock mismatch locally and persists a filesystem-calibrated boundary. The original scenario now also checks restart and capture of a genuinely new post-resume conversation. Focused pause/durability tests: 31/31 passed. Current CI must pass before merge.

@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f522e6b1-33e9-461e-8c74-973739712c32
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

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