Skip to content

Conversation

@beinan
Copy link
Collaborator

@beinan beinan commented Jan 18, 2026

Summary

  • re-export context types from the crate root after splitting modules (rust/lance-context/src/lib.rs:3)
  • keep the in-memory context API in its own module for clarity (rust/lance-context/src/context.rs:3)
  • isolate record structs and Lance-backed store into dedicated files without changing behavior (rust/lance-context/src/record.rs:3, rust/lance-context/src/store.rs:18)
  • update the CI helper skill to invoke gh pr create when available (.codex/skills/ci-pr-helper/SKILL.md:11)

Testing

  • ./.codex/skills/ci-pr-helper/scripts/run_ci_checks.sh (fails: uv not found)
  • PATH=$HOME/.cargo/bin:$PATH cargo fmt --manifest-path rust/lance-context/Cargo.toml

@beinan beinan force-pushed the refactor/context-store-cleanup branch 4 times, most recently from 8e12874 to 98d248e Compare January 18, 2026 10:34
@beinan beinan force-pushed the refactor/context-store-cleanup branch from 98d248e to 605d017 Compare January 18, 2026 10:36
@beinan beinan merged commit a539c46 into lance-format:main Jan 18, 2026
8 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