Skip to content

#350: Remove deprecated 'mcs sync --update' flag#351

Merged
bguidolim merged 1 commit into
mainfrom
bruno/350-remove-sync-update
May 28, 2026
Merged

#350: Remove deprecated 'mcs sync --update' flag#351
bguidolim merged 1 commit into
mainfrom
bruno/350-remove-sync-update

Conversation

@bguidolim
Copy link
Copy Markdown
Collaborator

@bguidolim bguidolim commented May 28, 2026

Closes #350

Summary

mcs sync --update was soft-deprecated one release ago (it warned and pointed at mcs update). It's fully superseded by mcs update, which fetches and re-applies across all configured scopes and — unlike --update — respects the opt-in generate-lockfile config instead of force-writing the lockfile. Now that the deprecation window has passed, this removes it so the CLI carries no deprecated surfaces and no divergent fetch-then-apply code path.

Changes

  • mcs sync --update is gone — invoking it now errors with Unknown option '--update'. Users who relied on it should switch to mcs update (pair with mcs config set generate-lockfile true if they want a lockfile written).

Test plan

  • swift test passes locally
  • swiftformat --lint . and swiftlint pass without violations
  • Run mcs sync --update → expect an Unknown option '--update' error
  • Run mcs update → fetches latest pack versions and re-applies across scopes
  • Run mcs sync with generate-lockfile: true → still writes mcs.lock.yaml
Checklist for engine changes
  • Docs updated if behavior changed (CLAUDE.md, docs/, techpack.yaml schema in ExternalPackManifest.swift)

- Drop the --update flag and its LockfileOperations.updatePacks() backing
- Delete update-path tests; scrub --update from CLAUDE.md and docs
@bguidolim bguidolim enabled auto-merge (squash) May 28, 2026 22:53
@bguidolim bguidolim merged commit 8aa6e12 into main May 28, 2026
4 checks passed
@bguidolim bguidolim deleted the bruno/350-remove-sync-update branch May 28, 2026 22:54
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.

Remove deprecated 'mcs sync --update' flag

1 participant