Skip to content

fix(fspy): track Bun file accesses on macOS#542

Merged
wan9chi merged 1 commit into
mainfrom
fix/fspy-bun-file-accesses
Jul 13, 2026
Merged

fix(fspy): track Bun file accesses on macOS#542
wan9chi merged 1 commit into
mainfrom
fix/fspy-bun-file-accesses

Conversation

@wan9chi

@wan9chi wan9chi commented Jul 13, 2026

Copy link
Copy Markdown
Member

Motivation

Bun uses private Darwin filesystem symbols that bypass the existing macOS interposers. Files read, written, or listed through those symbols were therefore absent from automatic input tracking, so affected tasks could reuse outdated cached output. Intercept those symbols and exercise Bun through the shared node_fs matrix.

Fixes #532.

wan9chi commented Jul 13, 2026

Copy link
Copy Markdown
Member Author

@wan9chi wan9chi force-pushed the fix/fspy-bun-file-accesses branch from a2a75a0 to 2fd76e0 Compare July 13, 2026 03:25
@wan9chi wan9chi force-pushed the test/fspy-deno-node-fs branch from 46553de to 14d3c1e Compare July 13, 2026 03:28
@wan9chi wan9chi force-pushed the fix/fspy-bun-file-accesses branch 2 times, most recently from f919e6b to 9358e9d Compare July 13, 2026 03:32
@socket-security

socket-security Bot commented Jul 13, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​bun@​1.3.14911008091100

View full report

@wan9chi wan9chi force-pushed the test/fspy-deno-node-fs branch from 14d3c1e to dd85cb5 Compare July 13, 2026 03:36
@wan9chi wan9chi force-pushed the fix/fspy-bun-file-accesses branch 5 times, most recently from 9a8d0e1 to c7728f5 Compare July 13, 2026 04:36
@wan9chi wan9chi marked this pull request as ready for review July 13, 2026 04:36

wan9chi commented Jul 13, 2026

Copy link
Copy Markdown
Member Author

Merge activity

  • Jul 13, 4:40 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jul 13, 4:45 AM UTC: Graphite couldn't merge this pull request because a downstack PR test(fspy): run node fs tests with Deno #541 failed to merge.
  • Jul 13, 4:54 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jul 13, 4:55 AM UTC: Graphite rebased this pull request as part of a merge.
  • Jul 13, 5:00 AM UTC: @wan9chi merged this pull request with Graphite.

@wan9chi wan9chi changed the base branch from test/fspy-deno-node-fs to graphite-base/542 July 13, 2026 04:45
@wan9chi wan9chi force-pushed the fix/fspy-bun-file-accesses branch from c7728f5 to f631238 Compare July 13, 2026 04:50
@wan9chi wan9chi force-pushed the graphite-base/542 branch from dd85cb5 to 9cbf275 Compare July 13, 2026 04:50
@wan9chi wan9chi changed the base branch from graphite-base/542 to test/fspy-deno-node-fs July 13, 2026 04:50
@wan9chi wan9chi changed the base branch from test/fspy-deno-node-fs to graphite-base/542 July 13, 2026 04:54
@wan9chi wan9chi changed the base branch from graphite-base/542 to main July 13, 2026 04:54
Co-authored-by: GPT-5 Codex <codex@openai.com>
@wan9chi wan9chi force-pushed the fix/fspy-bun-file-accesses branch 2 times, most recently from 341799a to 7d7ae9e Compare July 13, 2026 04:55
@wan9chi wan9chi merged commit 4003f65 into main Jul 13, 2026
16 checks passed
@wan9chi wan9chi deleted the fix/fspy-bun-file-accesses branch July 13, 2026 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fspy auto input-tracking misses Bun subprocess file reads — cached tasks replay stale results

1 participant