Skip to content

Commit a5424e2

Browse files
committed
src/cli(refactor[add]): remove automatic ./ to tilde migration
why: Path-first adds should not auto-migrate existing ./ sections; explicit --workspace ./ is the only way to create/target ./ labels. what: - Remove migration logic from _ensure_workspace_label_for_merge - Remove migration logic from _prepare_no_merge_items - Existing ./ sections now preserved when they match the target workspace refs: Simplify legacy heuristics
1 parent 18653d5 commit a5424e2

File tree

1 file changed

+0
-18
lines changed

1 file changed

+0
-18
lines changed

src/vcspull/cli/add.py

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -461,10 +461,6 @@ def _ensure_workspace_label_for_merge(
461461
if existing_label is None:
462462
workspace_label = preferred_label
463463
config_data.setdefault(workspace_label, {})
464-
elif existing_label == "./":
465-
workspace_label = preferred_label
466-
config_data[workspace_label] = config_data.pop("./")
467-
relabelled = True
468464
else:
469465
workspace_label = existing_label
470466

@@ -509,20 +505,6 @@ def _prepare_no_merge_items(
509505
target_index = len(items) - 1
510506
return workspace_label, target_index, relabelled
511507

512-
found_label: str | None = None
513-
for idx in reversed(matching_indexes):
514-
current_label = items[idx]["label"]
515-
if current_label != "./":
516-
found_label = current_label
517-
break
518-
519-
workspace_label = preferred_label if found_label is None else found_label
520-
521-
for idx in matching_indexes:
522-
if items[idx]["label"] == "./" and workspace_label != "./":
523-
items[idx]["label"] = workspace_label
524-
relabelled = True
525-
526508
target_index = matching_indexes[-1]
527509
workspace_label = items[target_index]["label"]
528510
return workspace_label, target_index, relabelled

0 commit comments

Comments
 (0)