Skip to content

fix(wordpress): preserve local work during wp-env pulls - #33

Merged
masoudgolchin merged 1 commit into
fix/1217817153513085-safe-env-syncfrom
fix/1218021765137250-safe-wp-env-pulls
Sep 7, 2026
Merged

fix(wordpress): preserve local work during wp-env pulls#33
masoudgolchin merged 1 commit into
fix/1217817153513085-safe-env-syncfrom
fix/1218021765137250-safe-wp-env-pulls

Conversation

@masoudgolchin

Copy link
Copy Markdown
Member

Summary

WordPress pulls could delete local-only plugins, overwrite local work, and skip needed files because one plugin directory existed. The file pull now previews first and copies missing files by default. Overwrites and deletions are separate opt-ins, each backed by a full local snapshot before transfer.

A required-file checker compares the captured DB and mu-plugin inventory with local paths, including network plugins. The skill recognizes natural requests such as bringing profiles to local. Project exclusions use current dependency evidence; removing a file from Git alone no longer means it must be excluded from local runtime.

Test plan

  • All 34 tests passed, including 8 real filesystem/rsync and inventory cases.
  • An independent forward test detected and copied the missing profiles plugin while preserving local edits and local-only files.
  • All 9 packages, generated metadata, script syntax, and diff checks passed.

Notes

  • Refs asana#1218021765137250
  • Depends on fix(base): preview and scope environment sync writes #32; this PR contains only the WordPress change.
  • WordPress pack 1.2.1; marketplace 2.13.4.
  • Local snapshots avoid macOS openrsync backup/delete behavior found during testing. They need enough disk space and a fresh directory outside the synced tree.
  • The helper checks file presence, not version or PHP compatibility. A real staging SSH connection, WordPress server, and profile flow were not exercised.
  • A preview is not a lock; avoid competing writers and inspect partial failures before retrying.

@masoudgolchin
masoudgolchin merged commit 227e84a into fix/1217817153513085-safe-env-sync Sep 7, 2026
1 check passed
@masoudgolchin
masoudgolchin deleted the fix/1218021765137250-safe-wp-env-pulls branch September 7, 2026 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant