Use published modules on Github npm by jayvdb · Pull Request #132 · edge-toolkit/core

jayvdb · 2026-09-22T21:52:21Z

No description provided.

coderabbitai · 2026-09-22T21:52:28Z

Important

Review skipped

Too many files!

This PR contains 148 files, which is 48 over the limit of 100.

To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch.

Upgrade to a paid plan to raise the limit.

This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: c1493052-96ce-49f1-9789-cdd9a3ab68cf

📥 Commits

Reviewing files that changed from the base of the PR and between 42901fe and d81578a.

⛔ Files ignored due to path filters (9)

Cargo.lock is excluded by !**/*.lock
generated/dart-rest/lib/clients/modules.dart is excluded by !**/generated/**
generated/python-rest/et_rest_client/api/modules/get_module_file.py is excluded by !**/generated/**
generated/python-rest/pkg/et_rest_client.js is excluded by !**/generated/**
generated/python-ws/pkg/et_ws.js is excluded by !**/generated/**
generated/rust-rest/src/lib.rs is excluded by !**/generated/**
generated/specs/rest.yaml is excluded by !**/generated/**
generated/zig-rest/src/et_rest_client.zig is excluded by !**/generated/**
pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (148)

.dockerignore
.gitignore
.mise/config.coverage.toml
.mise/config.maint.toml
.mise/config.rust.toml
.mise/config.toml
.mise/cov-branch-assert.jq
CLAUDE.md
Cargo.toml
Dockerfile
Dockerfile.nanoserver
README.md
config/ast-grep/rules/new-fn-returns-option.yaml
config/ast-grep/rules/org-identity-lives-in-et-org.yaml
config/jscpd-baseline.json
data/model-modules/model-eye1/pkg/package.json
data/model-modules/model-face1/pkg/package.json
data/model-modules/model-har-motion1/pkg/package.json
data/model-modules/model-llm1/pkg/package.json
data/model-modules/model-speech1/pkg/package.json
libs/edge-toolkit/src/config.rs
libs/edge-toolkit/tests/staged_module_dirs.rs
libs/org/Cargo.toml
libs/org/src/lib.rs
libs/org/tests/identity.rs
libs/ws-runner-common/src/lib.rs
libs/ws-runner-common/tests/fetch_main_field.rs
libs/ws-runner-common/tests/module_file_missing.rs
pnpm-workspace.yaml
services/modules/src/lib.rs
services/modules/src/routes.rs
services/modules/tests/api_modules.rs
services/modules/tests/configure_missing_root.rs
services/modules/tests/declared_names.rs
services/ws-modules/dart-comm1/pkg/et_ws_dart_comm1.js
services/ws-modules/dart-comm1/pkg/package.json
services/ws-modules/dart-data1/pkg/et_ws_dart_data1.js
services/ws-modules/dart-data1/pkg/package.json
services/ws-modules/dart-math1/pkg/et_ws_dart_math1.js
services/ws-modules/dart-math1/pkg/package.json
services/ws-modules/dotnet-data1/pkg/package.json
services/ws-modules/dotnet-math1/pkg/package.json
services/ws-modules/face-detection/Cargo.toml
services/ws-modules/face-detection/src/lib.rs
services/ws-modules/har1/Cargo.toml
services/ws-modules/har1/src/lib.rs
services/ws-modules/java-data1/pkg/package.json
services/ws-modules/java-math1/pkg/package.json
services/ws-modules/js-data1/package.json
services/ws-modules/js-data1/pkg/package.json
services/ws-modules/js-math1/pkg/et_ws_js_math1.js
services/ws-modules/js-math1/pkg/package.json
services/ws-modules/kotlin-data1/pkg/package.json
services/ws-modules/kotlin-math1/pkg/package.json
services/ws-modules/llm1/Cargo.toml
services/ws-modules/llm1/src/lib.rs
services/ws-modules/math1-sender/Cargo.toml
services/ws-modules/pydata1/pkg/et_ws_pydata1.js
services/ws-modules/pydata1/pkg/package.json
services/ws-modules/pydemo1/pkg/et_ws_pydemo1.js
services/ws-modules/pydemo1/pkg/package.json
services/ws-modules/pyeye1/pkg/et_ws_pyeye1.js
services/ws-modules/pyface1/pkg/et_ws_pyface1.js
services/ws-modules/pymath1/pkg/et_ws_pymath1.js
services/ws-modules/pyo3-math1/pkg/package.json
services/ws-modules/pyspeech1/pkg/et_ws_pyspeech1.js
services/ws-modules/pyspeech1/pkg/package.json
services/ws-modules/rcomm1/pkg/et_ws_rcomm1.js
services/ws-modules/rcomm1/pkg/package.json
services/ws-modules/rdata1/pkg/et_ws_rdata1.js
services/ws-modules/rdata1/pkg/package.json
services/ws-modules/rmath1/pkg/et_ws_rmath1.js
services/ws-modules/rmath1/pkg/package.json
services/ws-modules/wasi-graphics-info/wasi_graphics_info/__init__.py
services/ws-modules/wasi-math1-sender/pkg/package.json
services/ws-modules/zig-data1/build.zig
services/ws-modules/zig-data1/build.zig.zon
services/ws-modules/zig-except1/build.zig
services/ws-modules/zig-except1/build.zig.zon
services/ws-modules/zig-math1/build.zig
services/ws-modules/zig-math1/build.zig.zon
services/ws-modules/zig-shared/build.zig
services/ws-modules/zig-shared/build.zig.zon
services/ws-pyo3-runner/Cargo.toml
services/ws-pyo3-runner/src/hub_module.rs
services/ws-pyo3-runner/tests/modules.rs
services/ws-server/Dockerfile
services/ws-server/static/app.js
services/ws-server/static/package.json
services/ws-test-server/src/lib.rs
services/ws-wasi-runner/Cargo.toml
services/ws-wasi-runner/src/host/wasi_keyvalue.rs
services/ws-wasi-runner/src/lib.rs
services/ws-wasi-runner/tests/modules.rs
services/ws-wasi-runner/tests/otel_propagation.rs
services/ws-web-runner/Cargo.toml
services/ws-web-runner/tests/modules.rs
utilities/cli/Cargo.toml
utilities/cli/src/deployment_types/docker_compose.rs
utilities/cli/src/deployment_types/k3s.rs
utilities/cli/src/deployment_types/mise.rs
utilities/cli/src/deployment_types/mod.rs
utilities/cli/src/deployment_types/scenario_image.rs
utilities/cli/src/input.rs
utilities/cli/src/lib.rs
utilities/cli/src/module_package_json/mod.rs
utilities/cli/tests/module_package_json.rs
utilities/cli/tests/scenario_generation.rs
verification/local/output/default/Dockerfile.dockerignore
verification/local/output/default/compose.yaml
verification/local/output/default/k3s.yaml
verification/local/output/default/mise.toml
verification/local/output/facility-security-scenario/Dockerfile.dockerignore
verification/local/output/facility-security-scenario/compose.yaml
verification/local/output/facility-security-scenario/k3s.yaml
verification/local/output/facility-security-scenario/mise.toml
verification/local/output/math1/Dockerfile.dockerignore
verification/local/output/math1/compose.yaml
verification/local/output/math1/k3s.yaml
verification/local/output/math1/mise.toml
verification/local/output/pyo3-math1/Dockerfile.dockerignore
verification/local/output/pyo3-math1/compose.yaml
verification/local/output/pyo3-math1/k3s.yaml
verification/local/output/pyo3-math1/mise.toml
verification/local/output/wasi-math1/Dockerfile.dockerignore
verification/local/output/wasi-math1/compose.yaml
verification/local/output/wasi-math1/k3s.yaml
verification/local/output/wasi-math1/mise.toml
verification/published/output/default/Dockerfile.dockerignore
verification/published/output/default/compose.yaml
verification/published/output/default/k3s.yaml
verification/published/output/default/mise.toml
verification/published/output/default/npmrc
verification/published/output/math1/Dockerfile.dockerignore
verification/published/output/math1/compose.yaml
verification/published/output/math1/k3s.yaml
verification/published/output/math1/mise.toml
verification/published/output/math1/npmrc
verification/published/output/pyo3-math1/Dockerfile.dockerignore
verification/published/output/pyo3-math1/compose.yaml
verification/published/output/pyo3-math1/k3s.yaml
verification/published/output/pyo3-math1/mise.toml
verification/published/output/pyo3-math1/npmrc
verification/published/output/wasi-math1/Dockerfile.dockerignore
verification/published/output/wasi-math1/compose.yaml
verification/published/output/wasi-math1/k3s.yaml
verification/published/output/wasi-math1/mise.toml
verification/published/output/wasi-math1/npmrc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

deepsource-io · 2026-09-22T21:53:31Z

DeepSource Code Review

We reviewed changes in 42901fe...d81578a on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade	Security Reliability Complexity Hygiene Coverage

Code Review Summary

Analyzer	Updated (UTC)	Details
C#	Sep 23, 2026 7:42a.m.	Review ↗
C & C++	Sep 23, 2026 7:42a.m.	Review ↗
Docker	Sep 23, 2026 7:42a.m.	Review ↗
Java	Sep 23, 2026 7:42a.m.	Review ↗
JavaScript	Sep 23, 2026 7:42a.m.	Review ↗
Python	Sep 23, 2026 7:42a.m.	Review ↗
Rust	Sep 23, 2026 7:42a.m.	Review ↗
Secrets	Sep 23, 2026 7:42a.m.	Review ↗
Code coverage	Sep 23, 2026 8:28a.m.	Review ↗

Code Coverage Summary

Language	Line Coverage (New Code)	Line Coverage (Overall)
Aggregate	98.2%	70.8% [▲ up 0.2% from `main`]
Python	-	89.6%
Rust	98.2%	69.2% [▲ up 0.3% from `main`]

➟ Additional coverage metrics may have been reported. See full coverage report ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

codacy-production · 2026-09-22T21:54:54Z

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 41 complexity · 1 duplication

Metric Results

Complexity 41

Duplication 1

View in Codacy

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer}
_{TIP This summary will be updated as you push new changes.}

codecov · 2026-09-22T22:40:51Z

Codecov Report

❌ Patch coverage is 97.34513% with 9 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
services/ws-wasi-runner/src/host/wasi_keyvalue.rs	50.00%	6 Missing and 3 partials ⚠️

📢 Thoughts on this report? Let us know!

deepsource-io Bot reviewed Sep 22, 2026

View reviewed changes

Comment thread generated/python-rest/pkg/et_rest_client.js

Comment thread generated/python-ws/pkg/et_ws.js

jayvdb force-pushed the rm-relative-paths branch 4 times, most recently from 2df5e81 to b080ee9 Compare September 23, 2026 05:41

Use published modules on Github npm

d81578a

jayvdb force-pushed the rm-relative-paths branch from b080ee9 to d81578a Compare September 23, 2026 07:42

jayvdb marked this pull request as ready for review September 23, 2026 09:15

jayvdb requested a review from pierre-tenedero September 23, 2026 13:40

pierre-tenedero approved these changes Sep 23, 2026

View reviewed changes

jayvdb merged commit a065741 into main Sep 23, 2026
71 of 75 checks passed

jayvdb deleted the rm-relative-paths branch September 23, 2026 18:00

Navigation Menu

Use published modules on Github npm - #132

Use published modules on Github npm#132
jayvdb merged 1 commit into
mainfrom
rm-relative-paths

jayvdb commented Sep 22, 2026

Uh oh!

coderabbitai Bot commented Sep 22, 2026 •

edited

Loading

Review skipped

Uh oh!

deepsource-io Bot commented Sep 22, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

codacy-production Bot commented Sep 22, 2026 •

edited

Loading

Uh oh!

codecov Bot commented Sep 22, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

jayvdb commented Sep 22, 2026

Uh oh!

coderabbitai Bot commented Sep 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

deepsource-io Bot commented Sep 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

DeepSource Code Review

PR Report Card

Code Review Summary

Code Coverage Summary

Uh oh!

Uh oh!

Uh oh!

codacy-production Bot commented Sep 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Up to standards ✅

Uh oh!

codecov Bot commented Sep 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

coderabbitai Bot commented Sep 22, 2026 •

edited

Loading

deepsource-io Bot commented Sep 22, 2026 •

edited

Loading

codacy-production Bot commented Sep 22, 2026 •

edited

Loading

codecov Bot commented Sep 22, 2026 •

edited

Loading