Skip to content

Conversation

@frontegg-david
Copy link
Contributor

@frontegg-david frontegg-david commented Feb 3, 2026

Summary by CodeRabbit

  • New Features

    • Added VM sanitization controls (maxSanitizeDepth, maxSanitizeProperties) to tune sanitized output limits.
  • Chores

    • Updated core dependencies: bumped vectoriadb and migrated enclave/AST packages to their scoped releases.
  • Documentation

    • Updated examples and docs to use the new package import paths.
  • Tests/Tooling

    • Adjusted test/tooling config to align with the updated package namespaces and transform rules.

@frontegg-david frontegg-david changed the base branch from main to release/0.8.x February 3, 2026 02:59
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

Walkthrough

Updated dependency names and versions and adjusted related imports/configs: vectoriadb → ^2.1.3, enclave-vm@enclave-vm/core ^2.10.1, ast-guard@enclave-vm/ast ^2.10.1; added VM sanitization options and updated Jest/build externals and docs accordingly.

Changes

Cohort / File(s) Summary
Vector DB version bumps
libs/sdk/package.json, package.json, plugins/plugin-codecall/package.json
Bumped vectoriadb from ^2.0.2^2.1.3.
Scoped Enclave migration (deps & build externals)
libs/uipack/package.json, package.json, plugins/plugin-codecall/package.json, plugins/plugin-codecall/project.json
Replaced enclave-vm with @enclave-vm/core (^2.10.1) and updated esbuild externals to use the scoped package.
AST package migration
libs/utils/package.json
Replaced ast-guard with @enclave-vm/ast (^2.10.1).
Imports & code updates
libs/uipack/src/bundler/sandbox/enclave-adapter.ts, plugins/plugin-codecall/src/services/enclave.service.ts, libs/utils/src/regex/safe-regex.ts, plugins/plugin-codecall/src/codecall.symbol.ts, plugins/plugin-codecall/src/codecall.types.ts, plugins/plugin-codecall/src/providers/code-call.config.ts
Updated import sources to scoped packages; added two new VM sanitization options (maxSanitizeDepth, maxSanitizeProperties) to types, schema, defaults, and Enclave construction wiring.
Jest transform rules
libs/plugins/jest.config.ts, plugins/plugin-codecall/jest.config.ts
Adjusted transformIgnorePatterns to reference @enclave-vm instead of enclave-vm; removed/changed ast-guard exemption.
Docs / examples
docs/frontmcp/plugins/codecall/api-reference.mdx, docs/frontmcp/plugins/codecall/security.mdx
Updated example import paths to @enclave-vm/core and @enclave-vm/ast; examples remain semantically unchanged.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐇 I hopped through package trees today,

swapped names and versions on the way,
scoped packages snug in every line,
new sanitizers set to keep things fine,
I twitched my nose and hopped away.

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 65.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'upgrade-enclave' is vague and generic, using a non-descriptive term that doesn't clearly convey the full scope of changes beyond the enclave package upgrade. Consider a more descriptive title that mentions the key dependency upgrades, such as 'Upgrade enclave packages and vectoriadb dependencies' or 'Update to @enclave-vm scoped packages and vectoriadb 2.1.3'.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch upgrade-enclave

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 3

🤖 Fix all issues with AI agents
In `@libs/uipack/package.json`:
- Line 65: The import in libs/uipack/src/bundler/sandbox/enclave-adapter.ts
currently pulls Enclave, CreateEnclaveOptions, and SecurityLevel from the old
package name; update the import statement to use the scoped package
'@enclave-vm/core' so the symbols Enclave, type CreateEnclaveOptions, and type
SecurityLevel are imported from '@enclave-vm/core' instead of 'enclave-vm'.

In `@package.json`:
- Line 39: Update source imports and Jest configs to match the package rename to
`@enclave-vm/core`: in libs/uipack/src/bundler/sandbox/enclave-adapter.ts change
the import line that reads "import { Enclave, type CreateEnclaveOptions, type
SecurityLevel } from 'enclave-vm'" to import those same symbols from
'@enclave-vm/core'; in plugins/plugin-codecall/src/services/enclave.service.ts
change "import { Enclave, type ExecutionResult, type ToolHandler, type
ReferenceSidecarOptions } from 'enclave-vm'" to import from '@enclave-vm/core';
and update any jest.config.ts transformIgnorePatterns entries that reference
'enclave-vm' to '@enclave-vm/core' so tests resolve the renamed package.

In `@plugins/plugin-codecall/package.json`:
- Around line 49-50: Update all import usages of the old package name to the new
one: in enclave.service.ts replace any import specifier string 'enclave-vm' with
'@enclave-vm/core' so the module resolution matches package.json, and in
jest.config.ts update the transformIgnorePatterns regex to reference
'@enclave-vm/core' instead of 'enclave-vm' so tests don’t ignore the new package
during transformation.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

Performance Test Results

Status: ✅ All tests passed

Summary

Project Tests Passed Warnings Failed Leaks
✅ demo-e2e-agents 4 4 0 0 0
✅ demo-e2e-cache 11 11 0 0 0
✅ demo-e2e-codecall 4 4 0 0 0
✅ demo-e2e-config 4 4 0 0 0
✅ demo-e2e-direct 3 3 0 0 0
✅ demo-e2e-elicitation 1 1 0 0 0
✅ demo-e2e-errors 4 4 0 0 0
✅ demo-e2e-hooks 3 3 0 0 0
✅ demo-e2e-multiapp 4 4 0 0 0
✅ demo-e2e-notifications 3 3 0 0 0
✅ demo-e2e-openapi 2 2 0 0 0
✅ demo-e2e-providers 4 4 0 0 0
✅ demo-e2e-public 4 4 0 0 0
✅ demo-e2e-redis 14 14 0 0 0
✅ demo-e2e-remember 4 4 0 0 0
✅ demo-e2e-remote 5 5 0 0 0
✅ demo-e2e-serverless 2 2 0 0 0
✅ demo-e2e-skills 15 15 0 0 0
✅ demo-e2e-standalone 2 2 0 0 0
✅ demo-e2e-transport-recreation 3 3 0 0 0
✅ demo-e2e-ui 4 4 0 0 0

Total: 100 tests across 21 projects

📊 View full report in workflow run


Generated at: 2026-02-03T06:12:35.330Z
Commit: ccabbfdf

@frontegg-david frontegg-david merged commit 763c495 into release/0.8.x Feb 3, 2026
76 of 77 checks passed
@frontegg-david frontegg-david deleted the upgrade-enclave branch February 3, 2026 06:47
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