[stevevanhooser]

Summary

Mirrors VH-Lab/NDI-matlab#1039, a follow-up to #337. ndi.update.check counted every untracked file in a clone as one of the user's "modifications". So ndi.update.install, ndi update and the Update button asked "Update anyway?", with No as the default, just because someone had their own scripts in the clone. With Confirm=False it refused outright. Yet git stash and git pull never touch untracked files.

Untracked files no longer count as modifications. Only changed tracked files and local commits still need the user's OK.

There is one case where an untracked file does matter. If the update adds a file at the same path, git refuses to overwrite the untracked file, and the pull stops partway. For that case:

• check: finds these files by comparing git status --porcelain --untracked-files=all with git diff --name-only --diff-filter=A HEAD <upstream>, for NDI-python and for the ~/.ndi/tools clones. They are reported as Modifications[*]["BlockingFiles"], which replaces NewFiles.
• install: raises UpdateError("ndi:update:install:untrackedFilesInTheWay") before touching anything, and names the files. This happens even with AllowModified=True.
• Update button: update_dialog returns a warning alert listing those files, instead of offering an update that would fail.
• modificationSummary: lists them as "in the way" and says what happens to each kind of file.

Bridge contract: the check, install, modificationSummary and ndiPane entries are re-synced to NDI-matlab a24f19076, with decision-log notes.

Merge order

Merge VH-Lab/NDI-matlab#1039 first. a24f19076 is on that PR's branch, which CI checks this branch against because it has the same name. Once #1039 is merged with a merge commit, the hash is on NDI-matlab main. If it is squash-merged, the four hashes need re-recording.

Testing

Run locally with NDI-matlab checked out at a24f19076:

• black --check and ruff check are clean.
• pytest tests/ --ignore=tests/symmetry: 6102 passed, 609 skipped. This includes the bridge checks run in strict mode.
• New tests:
  • test_untracked_files_need_no_consent: untracked files, including one in an untracked folder, set no modifications, and the update goes ahead.
  • test_untracked_file_in_the_way_stops_the_update: covers a top-level file and one in an untracked folder. Both are reported, install refuses with or without AllowModified, and the file is left as it was.
  • test_dialog_with_untracked_files_in_the_way_only_explains: checks the dialog's warning for that case.
• Existing tests were updated for BlockingFiles and the new summary wording.

---

Generated by Claude Code