Skip to content

Extract tercen SDK into standalone tercen-rs crate#4

Merged
tsmonteiro merged 3 commits intomainfrom
extract-tercen-rs
Mar 2, 2026
Merged

Extract tercen SDK into standalone tercen-rs crate#4
tsmonteiro merged 3 commits intomainfrom
extract-tercen-rs

Conversation

@amaurel
Copy link
Member

@amaurel amaurel commented Mar 2, 2026

Summary

  • Extract all Tercen SDK modules (gRPC client, data streaming, context, color handling, facets, pages, etc.) into a standalone tercen-rs crate, removing ~7,200 lines from the operator
  • Replace all crate::tercen:: imports with tercen_rs:: and add tercen-rs as a path dependency
  • Remove build.rs and tercen_grpc_api submodule (proto compilation now lives in tercen-rs)
  • Clean up Dockerfile to remove references to the removed build artifacts

Test plan

  • Verify cargo build --profile dev-release succeeds
  • Run cargo fmt && cargo clippy -- -D warnings && cargo test
  • Run ./test_local.sh cpu against a Tercen instance to confirm end-to-end rendering still works
  • Verify Docker image builds successfully

🤖 Generated with Claude Code

amaurel and others added 3 commits March 1, 2026 11:12
Move all Tercen SDK modules (gRPC client, data streaming, context,
color handling, facets, pages, etc.) into the separate tercen-rs crate
and depend on it via path. Keep operator_properties locally as it
uses include_str!("../../operator.json") which is operator-specific.

- Replace all crate::tercen:: imports with tercen_rs::
- Remove build.rs and tercen_grpc_api submodule (now in tercen-rs)
- Handle DataFrame wrapping at boundaries (tercen-rs returns polars
  DataFrame, wrapped to ggrs_core DataFrame where needed)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
These are no longer needed since proto compilation moved to the
tercen-rs crate.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The operator.json container field must reference the branch-specific
image tag so Tercen pulls the correct Docker image when installing
from this branch.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@tsmonteiro tsmonteiro merged commit 2c779b5 into main Mar 2, 2026
2 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.

2 participants