Skip to content

fix(desktop): ignore installed dependencies in pack watchers - #1767

Merged
donjor merged 1 commit into
hyprwsfrom
fix/1614-dev-watch-node-modules
Oct 16, 2026
Merged

donjor merged 1 commit into
hyprwsfrom
fix/1614-dev-watch-node-modules

Conversation

@donjor-agent

@donjor-agent donjor-agent Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

🤖 claude/anthropic claude-opus-5-5 (high) · host
with peer-worker pi/openai gpt-6.1-sol (medium)

Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

ignoreWatch: /[\\/]node_modules[\\/]/

Workspace imports resolve through their symlinks to source paths outside node_modules, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked contracts and shared sources remain watchable.

Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

Verification

  • Focused bundle tests: 3 passed
  • Targeted lint and desktop typecheck
  • Desktop production pack
  • vp run fork:ci: 762 tests passed
  • Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

Grounding

Claim: under vp run dev:app --desktop, dependency changes keep Electron running; source edits relaunch it.

  • Build: PR head f227ae6fed
  • Client: Electron dev app, checkout-local state
  • Dependency: bundled effect Pipeable.js
Step Windows created Electron
launch 1 running
dependency chmod and touch 1 same PID
dependency hardlink (links 14 → 15) 1 same PID
touch apps/desktop/src/main.ts 2 relaunched
touch packages/contracts/src/background.ts 3 relaunched
touch packages/shared/src/schemaJson.ts 4 relaunched

Not grounded: the pre-fix relaunch; no baseline run.

Model: GPT Sol (gpt-6.1-sol). Harness: Pi.

Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes

@rsi-software-agent rsi-software-agent Bot added the size:S 10-29 effective changed lines (test files excluded in mixed PRs). label Oct 16, 2026
Refs #1614

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
@donjor
donjor force-pushed the fix/1614-dev-watch-node-modules branch from 4a5c2fe to f227ae6 Compare October 16, 2026 11:30
@donjor
donjor merged commit 17cc12b into hyprws Oct 16, 2026
@donjor
donjor deleted the fix/1614-dev-watch-node-modules branch October 16, 2026 20:58
donjor added a commit that referenced this pull request Oct 18, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 18, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 18, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 19, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 19, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 19, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 20, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 20, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 20, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 20, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 20, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 21, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 21, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 21, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 21, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 21, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 22, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 22, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 23, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 24, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 27, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Oct 27, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Oct 29, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Oct 29, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Oct 30, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Oct 30, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Oct 30, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Oct 31, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Nov 6, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 6, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 6, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 6, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 6, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 6, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 6, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 7, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 7, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 7, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 8, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 8, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 9, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- d27acaf fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- fcd9ce3 fixup! fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
donjor added a commit that referenced this pull request Nov 9, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 9, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 9, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 9, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 9, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 10, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 10, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 10, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 10, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 10, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
rsi-software-agent Bot pushed a commit that referenced this pull request Nov 11, 2026
## Problem

pnpm hardlink metadata changes can rebuild desktop bundles and relaunch Electron without a source edit.

## Change

Wrap the pack array once to apply the existing watch exclusion to every desktop bundle:

```ts
ignoreWatch: /[\\/]node_modules[\\/]/
```

Workspace imports resolve through their symlinks to source paths outside `node_modules`, so they remain watchable.
One bounded fixture build checks that installed dependencies are excluded while linked `contracts` and `shared` sources remain watchable.

## Scope and approval

Refs #1614.
This PR covers only the Watch condition; the historical 14:10:20 hardlink-source condition stays open.
The host approved this scope and owns runtime grounding; no watcher or Electron instance was started by the worker.

## Verification

- [x] Focused bundle tests: 3 passed
- [x] Targeted lint and desktop typecheck
- [x] Desktop production pack
- [x] `vp run fork:ci`: 762 tests passed
- [x] Host-owned Electron runtime grounding: below

The regression test checks actual bundler-resolved paths against the configured filter, not runtime filesystem events.

## Grounding

**Claim:** under `vp run dev:app --desktop`, dependency changes keep Electron running; source edits relaunch it.

- **Build:** PR head `f227ae6fed`
- **Client:** Electron dev app, checkout-local state
- **Dependency:** bundled `effect` `Pipeable.js`

| Step | Windows created | Electron |
| --- | --- | --- |
| launch | 1 | running |
| dependency `chmod` and `touch` | 1 | same PID |
| dependency hardlink (links 14 → 15) | 1 | same PID |
| touch `apps/desktop/src/main.ts` | 2 | relaunched |
| touch `packages/contracts/src/background.ts` | 3 | relaunched |
| touch `packages/shared/src/schemaJson.ts` | 4 | relaunched |

**Not grounded:** the pre-fix relaunch; no baseline run.

Model: GPT Sol (`gpt-6.1-sol`). Harness: Pi.

## Fork trailers

Squashes:

- 9b5efb01736 fix(desktop): ignore installed dependencies in pack watchers (#1767) (#1767)
- 5b8cea0c896 fix(dev): worktree dev servers keep their own port, process tree, and Electron profile (#1911) (#1911)

Fork-Domain: upstream-fixes
Fork-Tier: bugfix
Fork-Upstreamable: yes
Co-authored-by: donjor <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 effective changed lines (test files excluded in mixed PRs).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant