Skip to content

docs: reconcile Rename change-annotations (#70/#133) into canonical docs#142

Merged
clrudolphi merged 1 commit into
masterfrom
docs/issue-134-rename-annotations-reconciliation
Jul 13, 2026
Merged

docs: reconcile Rename change-annotations (#70/#133) into canonical docs#142
clrudolphi merged 1 commit into
masterfrom
docs/issue-134-rename-annotations-reconciliation

Conversation

@clrudolphi

Copy link
Copy Markdown
Collaborator

Summary

  • Adds an as-built "Rename change annotations" subsection under F16 in LSP-IDE-Support-Feature-Designs.md: negotiation, WorkspaceEditBuilder/RenameChangeAnnotations, the VS workspace/applyEdit push, closed-file cache invalidation, and where the as-built behavior deviates from the plan (NeedsConfirmation logic).
  • Registers WorkspaceEditBuilder, RenameChangeAnnotations, and RenamePostApplyCoordinator in LSP-IDE-Support-Architecture.md.
  • Promotes plan risk RA-1 (undo granularity) into the Open Questions register as Q23 in LSP-IDE-Support-Open-Questions.md.
  • Folds RA-2/RA-3/RA-4 into a new "Known limitations" subsection under the F16 as-built content, since each turned out to be moot or resolved differently than planned.

Closes #134

Test plan

  • Docs-only change; no build/test impact.
  • Cross-checked every as-built claim against the current Features/Rename/*.cs source (WorkspaceEditBuilder, RenameChangeAnnotations, StepRenameHandler, RenamePostApplyCoordinator).

🤖 Generated with Claude Code

Folds the as-built Rename Change Annotations implementation back into
LSP-IDE-Support-Feature-Designs.md (F16 as-built section), registers
WorkspaceEditBuilder/RenameChangeAnnotations/RenamePostApplyCoordinator
in Architecture.md, promotes RA-1 into the Open Questions register as
Q23, and folds RA-2/RA-3/RA-4 into a new known-limitations subsection.

Closes #134

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@clrudolphi clrudolphi merged commit 5a71d6a into master Jul 13, 2026
@clrudolphi clrudolphi deleted the docs/issue-134-rename-annotations-reconciliation branch July 13, 2026 00:29
clrudolphi added a commit that referenced this pull request Jul 13, 2026
Resolves the Q23/Q24 numbering collision: master (via #142) already
landed RA-1 as Q23, so this branch's IH-3 row is renumbered to Q24 and
kept after it.
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.

Reconcile Rename change-annotations (#70) into canonical design docs

1 participant