Skip to content

Commit f21ebc6

Browse files
author
Jenkins
committed
refactor: move Node.js bindings from bindings/node to crates/llmtrace-nodejs
1 parent a68b1c6 commit f21ebc6

16 files changed

Lines changed: 8 additions & 8 deletions

File tree

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[workspace]
22
resolver = "2"
3-
exclude = ["bindings/node"]
3+
exclude = ["crates/llmtrace-nodejs"]
44

55
members = [
66
"crates/llmtrace-core",

RALPH_LOOPS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -811,7 +811,7 @@ Implemented in commit `dcf100d`. ClickHouseTraceRepository with MergeTree engine
811811

812812
**Tasks**:
813813

814-
1. **Define protobuf** schema in `proto/llmtrace.proto` for trace/span ingestion
814+
1. **Define protobuf** schema in `crates/llmtrace-proto/llmtrace.proto` for trace/span ingestion
815815
2. **Implement gRPC server** using `tonic` in the proxy (or separate binary)
816816
3. **Streaming ingestion**: support client-side streaming for batch trace upload
817817
4. **Config**: `grpc.enabled`, `grpc.listen_addr`
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
[package]
2-
name = "llmtrace-node"
2+
name = "llmtrace-nodejs"
33
version = "0.1.0"
44
edition = "2021"
55
license = "MIT"
@@ -11,8 +11,8 @@ description = "Native Node.js bindings for LLMTrace via NAPI-RS"
1111
crate-type = ["cdylib"]
1212

1313
[dependencies]
14-
llmtrace-core = { path = "../../crates/llmtrace-core" }
15-
llmtrace-security = { path = "../../crates/llmtrace-security" }
14+
llmtrace-core = { path = "../llmtrace-core" }
15+
llmtrace-security = { path = "../llmtrace-security" }
1616
napi = { version = "2", features = ["napi9", "serde-json"] }
1717
napi-derive = "2"
1818
serde = { version = "1.0", features = ["derive"] }
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ console.log(result.securityScore); // 80
121121
## Building from Source
122122

123123
```bash
124-
cd bindings/node
124+
cd crates/llmtrace-nodejs
125125
npm install
126126
npm run build
127127
npm test

bindings/node/__tests__/index.test.ts renamed to crates/llmtrace-nodejs/__tests__/index.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
* Jest test suite for @llmtrace/node native bindings.
33
*
44
* These tests require the native .node binary to be built first:
5-
* cd bindings/node && npm run build
5+
* cd crates/llmtrace-nodejs && npm run build
66
*
77
* Then run:
88
* npm test

0 commit comments

Comments
 (0)