Skip to content

Make setup safely refresh the runtime - #76

Merged
Kristof Roomp (mcroomp) merged 1 commit into
mainfrom
fix/setup-daemon-uv-sync
Sep 24, 2026
Merged

Kristof Roomp (mcroomp) merged 1 commit into
mainfrom
fix/setup-daemon-uv-sync

Conversation

@mcroomp

Copy link
Copy Markdown
Contributor

Summary

  • stop a running daemon and wait for its process lock before synchronizing .client-venv
  • remove incomplete legacy .dist-info metadata that lacks RECORD
  • use uv copy mode to avoid cross-filesystem hardlink warnings
  • clean extension build output and exclude local/stale files from the VSIX

Validation

  • ran node setup.mjs against an environment containing stale tantivy-0.26.0.dist-info; setup removed it and synchronized without warnings
  • started a real daemon, reran setup, observed Shutdown sent, and confirmed the daemon was offline afterward
  • extension compile succeeded
  • extension tests: 68 passed, 8 integration tests skipped because the daemon was not running
  • VSIX package succeeded with only the expected 14 files
  • node --check setup.mjs
  • git diff --check

Follow-up to #75.

Stop the daemon before synchronizing its environment, remove incomplete legacy package metadata, use uv copy mode across filesystems, and keep local or stale files out of packaged extensions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mcroomp
Kristof Roomp (mcroomp) merged commit f40b8c2 into main Sep 24, 2026
7 checks passed
@mcroomp
Kristof Roomp (mcroomp) deleted the fix/setup-daemon-uv-sync branch September 24, 2026 07:51
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.

1 participant