Skip to content

refactor(fspy): introduce fspy_shm facade - #520

Merged
wan9chi merged 1 commit into
mainfrom
fspy-shm-facade
Jul 9, 2026
Merged

refactor(fspy): introduce fspy_shm facade#520
wan9chi merged 1 commit into
mainfrom
fspy-shm-facade

Conversation

@wan9chi

@wan9chiwan9chi commented Jul 9, 2026

Copy link
Copy Markdown
Member

Motivation

Isolate fspy shared-memory ownership behind a small internal facade so platform backends can be replaced incrementally without changing the channel layout or synchronization protocol.

Background

fspy_shared previously depended directly on shared_memory::Shmem and ShmemConf. That coupling made the Linux /dev/shm fix inseparable from channel code and forced every platform migration to touch the same low-level data path.

Changes

  • Adds the private fspy_shm crate with create, open, identity, length, pointer, and slice APIs.
  • Delegates to the existing shared_memory implementation on all platforms, preserving behavior in this PR.
  • Migrates fspy_shared production code and real shared-memory tests to the facade.
  • Adds create/open, cross-process visibility, owner cleanup, and opened-view lifetime coverage.
  • Refreshes vulnerable lockfile entries for anyhow, crossbeam-epoch, and memmap2.

@wan9chiGraphite App

wan9chi commented Jul 9, 2026

Copy link
Copy Markdown
MemberAuthor

@wan9chiGraphite App

wan9chi commented Jul 9, 2026

Copy link
Copy Markdown
MemberAuthor

Merge activity

  • Jul 9, 2:10 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jul 9, 2:11 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jul 9, 2:15 PM UTC: @wan9chi merged this pull request with Graphite.

Co-authored-by: GPT-5.6 <gpt-5.6@openai.com>
@wan9chi
wan9chi merged commit cb580c2 into mainJul 9, 2026
16 checks passed
@wan9chi
wan9chi deleted the fspy-shm-facade branch July 9, 2026 14:15
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@wan9chi