DeepSeek/deepseek-harness
1
0
Fork 0
mirror of https://github.com/deepseek-ai/deepseek-harness synced 2026-08-15 21:04:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

cleanup(install): drop the master.path record

Browse Source 
Nothing read it. The launcher, dsh-upgrade, and the installer's own re-run
all ignored the file, and the diagnostic it was meant to feed was never
built, so it was write-only state.

Git already owns the fact it recorded: a staging worktree's .git file names
the repository path, and `git worktree list` in that clone enumerates every
worktree depending on it. An installer-written copy only adds state that can
go stale while nothing validates it.

The containment caveat it documented is real and stays in the script header
and the Agent Note, now pointing at git's own records.
This commit is contained in:
Turtle
2026-07-31 21:17:28 +08:00
parent 00c5f2abd1
commit f3a1ff41b7
4 changed files with 12 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
scripts/install.sh
View File

@@ -32,8 +32,8 @@
#
# Adopting an arbitrary clone leaves the container not self-contained: its
# staging worktrees hold an absolute gitdir pointer into that clone, so deleting
# it breaks them. $DSH_SOURCE/master.path records the resolved clone so the
# breakage is diagnosable.
# it breaks them. `git worktree list` in that clone is the record of which
# worktrees depend on it.
#
# When run through `curl | sh` the script text arrives on stdin, so every
# prompt and the final launch read the controlling terminal (/dev/tty) directly;
@@ -304,16 +304,6 @@ if [ -f "$_exclude" ] && ! grep -qxF '.agents/merge.lock' "$_exclude" 2>/dev/nul
fi
mkdir -p "$DSH_STAGING/.agents"
: >"$DSH_STAGING/.agents/merge.lock"
# A staging worktree holds an absolute gitdir pointer into the repository, so
# a container whose repository lives OUTSIDE it is not self-contained: deleting
# that repository breaks every worktree here. Record it only in that case, so
# the file's presence itself means "this container depends on an outside path".
_src_resolved=$(resolve_dir "$DSH_SOURCE")
case "$REPO_ROOT/" in
"$_src_resolved"/*) ;;
*) printf '%s\n' "$REPO_ROOT" >"$DSH_SOURCE/master.path"
info "recorded external repository in $DSH_SOURCE/master.path" ;;
esac
# --- 3. install dependencies (no build; the launcher runs from source) --------
step "Installing dependencies with pnpm (this can take a while)"