Skip to content

Update README.md#248

Merged
bguidolim merged 1 commit intomainfrom
bguidolim-patch-1
Mar 6, 2026
Merged

Update README.md#248
bguidolim merged 1 commit intomainfrom
bguidolim-patch-1

Conversation

@bguidolim
Copy link
Owner

Summary

Changes

Test plan

  • swift test passes locally
  • swiftformat --lint . and swiftlint pass without violations
  • Affected commands verified with a real pack (e.g. mcs sync, mcs doctor)
Checklist for engine changes

Only check items that apply to this PR. Delete irrelevant ones.

  • .shellCommand components have supplementaryDoctorChecks defined (deriveDoctorCheck() returns nil for shell actions)
  • Any fix() implementation does cleanup/migration only — never installs or registers resources
  • State migrations are guarded by isNeeded() to stay idempotent with mcs sync
  • New file write/copy/delete paths use PathContainment.safePath() and handle the nil (escape) case
  • Docs updated if behavior changed (CLAUDE.md, docs/, techpack.yaml schema in ExternalPackManifest.swift)

@bguidolim bguidolim enabled auto-merge (squash) March 6, 2026 15:58
@bguidolim bguidolim merged commit 5a928a3 into main Mar 6, 2026
4 checks passed
@bguidolim bguidolim deleted the bguidolim-patch-1 branch March 6, 2026 16:01
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