mirror of
https://github.com/stablyai/orca.git
synced 2026-10-22 08:03:37 +00:00
* fix(worktree): retry local main refresh through git lock contention and skip false alarms * fix(worktree): overlap the local main refresh with the checkout and run one refresh per repo at a time * fix(worktree): skip the local base refresh when the create makes that branch itself Creating a workspace named feature-x from origin/feature-x runs `worktree add -b feature-x`, which now overlaps the refresh. The refresh's drift probe could see refs/heads/feature-x missing and its presence probe then see it (the add just wrote it), which reported "not fast-forward" and showed a sticky "Local feature-x was not refreshed" warning. `-b` refuses an existing branch, so there is nothing to refresh in that case: skip it on the local, prepared-checkout and SSH create paths. The SSH overlap tests move to their own file so the existing suite stays under the line limit. * fix(worktree): say plainly what happens after the local base refresh queue wait expires * test(worktree): prove SSH local base refreshes of one repo run one at a time * test(worktree): drop type assertions from the SSH refresh overlap test mocks * fix(worktree): fast-forward local main with one host-owned merge --ff-only per branch Moves the whole local base refresh into one shared routine that runs on the execution host (main process for local and WSL repos, the relay for SSH), so the app no longer keeps a second copy of the checks, queue and retry. A checked-out branch now moves with merge --ff-only (hooks, auto-gc and autostash off) instead of status-then-reset --hard, which silently overwrote an untracked file the new commit adds and could discard an edit or a commit made after the check. A free branch moves with a compare-and-swap update-ref that writes a reflog message. Status reads no longer take index.lock. Creates of one branch share one run plus at most one trailing run; a create waits at most 30 s and never starts a competing mutation. The failure toast is keyed by repo and branch because every create that joined a run reports the same fact. * fix(worktree): fast-forward local main even when the repo requires signed merges With merge.verifySignatures=true, the owner-checkout fast-forward refused an unsigned origin/main tip, so every create warned "Local main was not refreshed" where the old reset moved main. The new workspace is already created from that same unsigned commit, and a branch that is not checked out moves without a signature check, so the refusal protected nothing. Turn the setting off for this one merge, like the hooks, gc and autostash overrides. * fix(worktree): clear git read caches when a shared local main update lands late The update of local main can finish after a create stopped waiting for it, so the shared run now invalidates git read caches itself. The index.lock real-git test also no longer reads the developer's global git config. * fix(worktree): never overwrite an ignored file when fast-forwarding local main A plain `git merge --ff-only` silently replaces an ignored file (for example a local `.env`) at a path the new commit starts tracking. Pass `--no-overwrite-ignore` so git refuses instead and the create reports the checkout as having local changes. Supported on the fast-forward path since well before Git 2.25. Also make the relay test for one-refresh-per-branch hold the first merge until the second request has reached the relay, so it fails without the coalescing. * fix(worktree): keep the local main update a plain fast-forward whatever the user's merge settings say A per-branch mergeOptions such as '-s ours' or '--squash', or pull.twohead=ours, made the update create a merge commit that dropped upstream, or stage upstream without moving main, while reporting success. The command now clears the branch's mergeOptions and passes the strategy and signature choice on the command line, which beats any config. After the move Orca confirms local main is exactly the target before reporting it updated. The exact command also runs in the Git 2.25 compatibility suite. * fix(worktree): make the Git 2.25 fast-forward contract pass in CI and rerun on every change to it The new real-Git contract for the local main fast-forward wrote a post-merge hook into .git/hooks, which does not exist when the repo is created by the uninstalled Git 2.25.5 build CI uses (no templates), so the Git compatibility check failed. Create the directory first. The Git compatibility check also did not run when only the fast-forward module changed, so a later edit to its merge arguments (for example a flag Git 2.25 lacks) would skip the one check that tests them. Add the module to the check's paths. * fix(worktree): answer every create from a local main update toward its own base Creates from different remotes' main (origin/main and upstream/main) shared one queued update per repo and branch, which ran only the latest caller's target: a create could get no result for its own base, or a false "not refreshed" warning computed for another remote's main. The per-branch runner now queues one run per distinct target, still one at a time per branch, and only callers toward the same target share a queued run. Applied in the app and the relay. * test(worktree): record the third create's result in the mixed-remote burst tests and update the toast id rationale * chore(worktree): correct the toast id rationale
268 lines
10 KiB
TypeScript
268 lines
10 KiB
TypeScript
import { resolveWorktreeAddBaseRef } from '../../shared/worktree/base-ref'
|
|
import type {
|
|
LocalBaseRefRefreshResult,
|
|
LocalBaseRefUpdateSuggestion
|
|
} from '../../shared/worktree/base-ref-drift-types'
|
|
import { windowsLongPathGitArgs } from '../../shared/windows-long-path-git-args'
|
|
import { withRepoRefMaintenancePaused } from './local-repo-ref-maintenance'
|
|
import { gitExecFileAsync } from './runner'
|
|
import { runWithGitReadCacheInvalidation } from './status'
|
|
import { invalidateWslLinkedWorktreeGitRouting } from './wsl-linked-worktree-git-routing'
|
|
import {
|
|
getLocalBaseRefUpdateSuggestionForWorktreeCreate,
|
|
parseRemoteTrackingLocalBaseRef,
|
|
refreshLocalBaseRefForWorktreeCreate
|
|
} from './worktree-base-refresh'
|
|
import { resolveWorktreeBaseCommitOid } from './worktree-base-ref-probe'
|
|
import type {
|
|
AddWorktreeOptions,
|
|
AddWorktreeResult,
|
|
GitWorktreeExecOptions
|
|
} from './worktree-operation-options'
|
|
import { gitExecOptions, resolveWorktreeAddTimeoutMs } from './worktree-operation-options'
|
|
import { bumpWorktreeScanGeneration } from './worktree-scan-cache'
|
|
import { assertNoPendingWorktreeRemovalConflict } from '../worktree-background-removal'
|
|
|
|
export type WorktreeAddBaseContext = Pick<AddWorktreeResult, 'localBaseRefUpdateSuggestion'> & {
|
|
effectiveBase: string
|
|
effectiveBaseOid?: string
|
|
/** Started, not awaited: the worktree is created from the remote-tracking commit, so the refresh can overlap the checkout. Never rejects. */
|
|
pendingLocalBaseRefRefresh?: Promise<LocalBaseRefRefreshResult | undefined>
|
|
}
|
|
|
|
export async function resolveWorktreeAddBaseContext(
|
|
repoPath: string,
|
|
baseBranch: string,
|
|
refreshLocalBaseRef: boolean,
|
|
options: AddWorktreeOptions,
|
|
createdBranch: string
|
|
): Promise<WorktreeAddBaseContext> {
|
|
let effectiveBaseOid: string | null = null
|
|
const effectiveBase = await resolveWorktreeAddBaseRef(baseBranch, async (qualifiedRef) => {
|
|
effectiveBaseOid = await resolveWorktreeBaseCommitOid(repoPath, qualifiedRef, options)
|
|
return effectiveBaseOid !== null
|
|
})
|
|
// Why: `-b` refuses an existing branch, so creating the base's own local branch leaves nothing to refresh; probing it would race the overlapped add's branch write into a false "not fast-forward" warning.
|
|
const createsLocalBaseBranch =
|
|
parseRemoteTrackingLocalBaseRef(baseBranch, effectiveBase, options.remoteTrackingBase)
|
|
?.localBranch === createdBranch
|
|
const pendingLocalBaseRefRefresh =
|
|
refreshLocalBaseRef && !createsLocalBaseBranch
|
|
? refreshLocalBaseRefForWorktreeCreate(
|
|
repoPath,
|
|
baseBranch,
|
|
effectiveBase,
|
|
options.remoteTrackingBase,
|
|
options
|
|
).catch((error: unknown) => {
|
|
// Why: the create may already have succeeded by the time this settles; a refresh bug must not fail it.
|
|
console.warn('addWorktree: local base ref refresh failed unexpectedly', error)
|
|
return undefined
|
|
})
|
|
: undefined
|
|
const localBaseRefUpdateSuggestion =
|
|
!refreshLocalBaseRef && options.suggestLocalBaseRefUpdate
|
|
? await getLocalBaseRefUpdateSuggestionForWorktreeCreate(
|
|
repoPath,
|
|
baseBranch,
|
|
effectiveBase,
|
|
options.remoteTrackingBase,
|
|
options
|
|
)
|
|
: undefined
|
|
return {
|
|
effectiveBase,
|
|
// Refresh/suggestion work can span ref changes; only reuse the immediate resolution probe.
|
|
...(!refreshLocalBaseRef && !options.suggestLocalBaseRefUpdate && effectiveBaseOid
|
|
? { effectiveBaseOid }
|
|
: {}),
|
|
...(pendingLocalBaseRefRefresh ? { pendingLocalBaseRefRefresh } : {}),
|
|
...(localBaseRefUpdateSuggestion ? { localBaseRefUpdateSuggestion } : {})
|
|
}
|
|
}
|
|
|
|
export async function persistWorktreeCreationBase(
|
|
worktreePath: string,
|
|
branch: string,
|
|
effectiveBase: string,
|
|
options: GitWorktreeExecOptions = {}
|
|
): Promise<void> {
|
|
const configKey = `branch.${branch}.base`
|
|
try {
|
|
await gitExecFileAsync(['config', '--local', '--replace-all', configKey, effectiveBase], {
|
|
...gitExecOptions(worktreePath, options)
|
|
})
|
|
} catch (error) {
|
|
console.warn(`addWorktree: failed to set ${configKey} for ${worktreePath}`, error)
|
|
try {
|
|
// Why: reused branch names may carry stale base metadata; if replacement fails, unset it so consumers don't trust stale lineage.
|
|
await gitExecFileAsync(['config', '--local', '--unset-all', configKey], {
|
|
...gitExecOptions(worktreePath, options)
|
|
})
|
|
} catch (unsetError) {
|
|
console.warn(
|
|
`addWorktree: failed to unset stale ${configKey} for ${worktreePath}`,
|
|
unsetError
|
|
)
|
|
}
|
|
}
|
|
}
|
|
|
|
export async function configurePushAutoSetupRemote(
|
|
worktreePath: string,
|
|
options: GitWorktreeExecOptions
|
|
): Promise<void> {
|
|
try {
|
|
// Why: `--get` (not `--local --get`) treats a value at any scope as an explicit user choice.
|
|
let alreadySet = false
|
|
try {
|
|
await gitExecFileAsync(['config', '--get', 'push.autoSetupRemote'], {
|
|
...gitExecOptions(worktreePath, options)
|
|
})
|
|
alreadySet = true
|
|
} catch (readError) {
|
|
// Why: exit 1 means unset; other codes are real read failures and must not overwrite config.
|
|
const code = (readError as { code?: unknown })?.code
|
|
if (code !== 1) {
|
|
throw readError
|
|
}
|
|
}
|
|
if (!alreadySet) {
|
|
await gitExecFileAsync(['config', '--local', 'push.autoSetupRemote', 'true'], {
|
|
...gitExecOptions(worktreePath, options)
|
|
})
|
|
}
|
|
} catch (error) {
|
|
console.warn(`addWorktree: failed to set push.autoSetupRemote for ${worktreePath}`, error)
|
|
}
|
|
}
|
|
|
|
export async function unsetWorktreeCreationBase(
|
|
worktreePath: string,
|
|
branch: string,
|
|
options: GitWorktreeExecOptions = {}
|
|
): Promise<void> {
|
|
try {
|
|
await gitExecFileAsync(['config', '--local', '--unset-all', `branch.${branch}.base`], {
|
|
...gitExecOptions(worktreePath, options)
|
|
})
|
|
} catch {
|
|
// Best-effort cleanup; leave the original sparse-setup error as the actionable failure.
|
|
}
|
|
}
|
|
|
|
/**
|
|
* Create a new worktree.
|
|
* @param repoPath - Path to the main repo (or bare repo)
|
|
* @param worktreePath - Absolute path where the worktree will be created
|
|
* @param branch - Branch name for the new worktree
|
|
* @param baseBranch - Optional base branch to create from (defaults to HEAD)
|
|
* @remarks Side effects (best-effort, warn-only): passes `--no-track`, writes
|
|
* `branch.<branch>.base` for new-branch worktrees with a base ref, and may
|
|
* write `push.autoSetupRemote=true` to the repo's shared config.
|
|
*/
|
|
export async function addWorktree(
|
|
repoPath: string,
|
|
worktreePath: string,
|
|
branch: string,
|
|
baseBranch?: string,
|
|
refreshLocalBaseRef = false,
|
|
noCheckout = false,
|
|
options: AddWorktreeOptions = {}
|
|
): Promise<AddWorktreeResult> {
|
|
try {
|
|
return await withRepoRefMaintenancePaused('worktree-add', () =>
|
|
runWithGitReadCacheInvalidation(() =>
|
|
performAddWorktree(
|
|
repoPath,
|
|
worktreePath,
|
|
branch,
|
|
baseBranch,
|
|
refreshLocalBaseRef,
|
|
noCheckout,
|
|
options
|
|
)
|
|
)
|
|
)
|
|
} finally {
|
|
bumpWorktreeScanGeneration(repoPath)
|
|
}
|
|
}
|
|
|
|
async function performAddWorktree(
|
|
repoPath: string,
|
|
worktreePath: string,
|
|
branch: string,
|
|
baseBranch?: string,
|
|
refreshLocalBaseRef = false,
|
|
noCheckout = false,
|
|
options: AddWorktreeOptions = {}
|
|
): Promise<AddWorktreeResult> {
|
|
// Why: Git still owns that path and branch until the background delete finishes; a create now
|
|
// would race it, and the branch cleanup that follows would find the branch checked out again.
|
|
assertNoPendingWorktreeRemovalConflict(repoPath, { worktreePath, branch })
|
|
let pendingLocalBaseRefRefresh: Promise<LocalBaseRefRefreshResult | undefined> | undefined
|
|
let localBaseRefUpdateSuggestion: LocalBaseRefUpdateSuggestion | undefined
|
|
// Why: enable long paths for this Windows checkout without changing user Git config.
|
|
const args = [...windowsLongPathGitArgs(repoPath), 'worktree', 'add']
|
|
let effectiveBase: string | undefined
|
|
if (noCheckout) {
|
|
args.push('--no-checkout')
|
|
}
|
|
if (options.checkoutExistingBranch) {
|
|
// Why: -b would create a new branch instead of checking out the selected one.
|
|
args.push(worktreePath, branch)
|
|
} else {
|
|
// Why: --no-track avoids inheriting the base's upstream so `git status` won't misreport "behind by N" pre-publish; first push sets it (see push.autoSetupRemote below).
|
|
args.push('--no-track', '-b', branch, worktreePath)
|
|
if (baseBranch) {
|
|
const baseContext = await resolveWorktreeAddBaseContext(
|
|
repoPath,
|
|
baseBranch,
|
|
refreshLocalBaseRef,
|
|
options,
|
|
branch
|
|
)
|
|
effectiveBase = baseContext.effectiveBase
|
|
pendingLocalBaseRefRefresh = baseContext.pendingLocalBaseRefRefresh
|
|
localBaseRefUpdateSuggestion = baseContext.localBaseRefUpdateSuggestion
|
|
args.push(effectiveBase)
|
|
}
|
|
}
|
|
try {
|
|
await gitExecFileAsync(args, {
|
|
...gitExecOptions(repoPath, options),
|
|
// Why: resolve per call — hoisting this to a module const would freeze the override at import.
|
|
timeout: resolveWorktreeAddTimeoutMs()
|
|
})
|
|
} catch (error) {
|
|
// Why: settle the overlapped refresh inside the caller's ref-maintenance pause before reporting the failure.
|
|
await pendingLocalBaseRefRefresh
|
|
throw error
|
|
} finally {
|
|
// Git may have written the target's `.git` marker even when it reports a late
|
|
// failure, so drop any pre-create route before the follow-up commands route.
|
|
invalidateWslLinkedWorktreeGitRouting(worktreePath)
|
|
}
|
|
|
|
if (options.checkoutExistingBranch) {
|
|
return {}
|
|
}
|
|
|
|
if (effectiveBase) {
|
|
await persistWorktreeCreationBase(worktreePath, branch, effectiveBase, options)
|
|
}
|
|
|
|
// SSH parity: relay's addWorktreeOp (src/relay/git-handler-worktree-ops.ts) mirrors this — change both in lockstep.
|
|
// Why: --no-track leaves no upstream until first push; push.autoSetupRemote=true lets a plain
|
|
// `git push` create+set origin/<branch> (git >=2.37; older clients ignore it). `--local` on a
|
|
// linked worktree writes the shared common-dir config (whole repo) — intentional and idempotent,
|
|
// so it's warn-only and not rolled back on failure.
|
|
await configurePushAutoSetupRemote(worktreePath, options)
|
|
const localBaseRefRefresh = await pendingLocalBaseRefRefresh
|
|
return {
|
|
...(localBaseRefRefresh ? { localBaseRefRefresh } : {}),
|
|
...(localBaseRefUpdateSuggestion ? { localBaseRefUpdateSuggestion } : {})
|
|
}
|
|
}
|