Skip to content

Stream Glyphs imports in bounded batches - #173

Merged
kostyafarber merged 1 commit into
mainfrom
agent/stream-glyphs-import
Aug 2, 2026
Merged

Stream Glyphs imports in bounded batches#173
kostyafarber merged 1 commit into
mainfrom
agent/stream-glyphs-import

Conversation

@kostyafarber

@kostyafarber kostyafarber commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • route .glyphs and .glyphspackage through the existing FontImport pipeline
  • parse each Glyphs source once, publish its header/directory, and convert Shift glyph geometry in bounded Rayon batches
  • preassign glyph identities so components resolve independently of batch order
  • publish directory-only workspaces and acquire component closures lazily from SQLite

Verification

  • cargo test -p shift-backends
  • cargo test -p shift-workspace
  • cargo clippy -p shift-backends -p shift-workspace --all-targets -- -D warnings
  • full pre-commit suite

Profiling

The release profile_streaming_import control imports the repository's 327-glyph Homenaje fixture in 22.4 ms end-to-end. The reported large Glyphs corpus still needs to be rerun on the originating Mac; the upstream glyphs-reader parse remains eager, while Shift conversion, packing, compression, and SQLite publication are now bounded.

@kostyafarber
kostyafarber merged commit af89e3f into main Aug 2, 2026
13 checks passed
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.

1 participant