[ctawiah]

Context

Detach must remove the descriptor state of a linked resource without touching the repository source.

What changes

• Delete the descriptor when it defines the selected resource.
• Keep the managed-file detach behavior.
• Leave the repository source unchanged.

Design notes

• Deletion uses the existing staged rollback path.

Review focus

• Can detach delete or edit a repository source?

Verification

• make build
• go test ./...
• golangci-lint run ./...

Related changes

1. refactor(sync): remove Phase 1 source references #1005: Phase 1 reference removal
2. feat(sync): add projection and problem core #987: projection and problem core
3. feat(sync): add the adapter core and the Markdown adapter #988: adapter core and Markdown
4. feat(sync): add the structured document adapters #989: structured adapters
5. feat(sync): add the Python adapter #991: Python adapter
6. feat(sync): add the TypeScript and JavaScript adapters #992: TypeScript and JavaScript adapters
7. feat(sync): add source descriptor storage #993: descriptor storage
8. feat(sync): compile and publish repository sources #994: compile and publish
9. This PR: descriptor detach
10. feat(sync): prepare repository source links #997: link preparation
11. feat(sync): integrate repository sources with planning and link #995: planning and link
12. feat(sync): add the link commands #998: link commands

🤖 Generated with Claude Code

---

Note

Overview
Detach now treats descriptor-linked variations and attachments like first-class synced resources: it removes sync-owned descriptors and sync.lock entries while leaving repository source files untouched.

Confirmation output is richer. Instead of a single “local changes will be lost” flag, detach compares managed files to their lock fingerprint and linked resources to both reference digest and projection fingerprint, surfacing “unsynced changes” when mapping, relationships, or projected content no longer match the baseline.

Discovery and cleanup cover .ref.yaml / descriptor paths as well as managed files. Symlinks sitting on managed paths are ignored as definitions for change warnings and resource listing, but are still removed when detaching. SyncedAttachment gains a Fingerprint() helper aligned with how locks record linked vs managed attachments.

^{Reviewed by Cursor Bugbot for commit 2613913. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

Stale Bugbot comment from a previous run.

[cursor]

Stale Bugbot comment from a previous run.

[cursor]

Stale Bugbot comment from a previous run.

[cursor]

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit f1ff089. Configure here.}