Skip to content

Remove example "you/your-pack" from README#247

Merged
bguidolim merged 1 commit intomainfrom
claude/remove-example-pack-SJM9S
Mar 6, 2026
Merged

Remove example "you/your-pack" from README#247
bguidolim merged 1 commit intomainfrom
claude/remove-example-pack-SJM9S

Conversation

@bguidolim
Copy link
Owner

https://claude.ai/code/session_01HwncrQvAgxapc9sGpoMwQV

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:42
@bguidolim bguidolim merged commit 95fe107 into main Mar 6, 2026
4 checks passed
@bguidolim bguidolim deleted the claude/remove-example-pack-SJM9S branch March 6, 2026 15:45
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.

2 participants