Skip to content

Commit 0db900f

Browse files
committed
feat(cli): temporarily disable ONNX Runtime integration
To unblock release, temporarily disable ONNX/MiniLM integration: - Comment out extract-onnx-runtime and extract-minilm-model in build steps - Remove from build:js and build:watch scripts in package.json - Disable MiniLM inference in handle-ask.mts (falls back to pattern matching) The ask command continues to work using pattern matching only. ONNX integration will be re-enabled once build issues are resolved.
1 parent 1687348 commit 0db900f

File tree

3 files changed

+19
-15
lines changed

3 files changed

+19
-15
lines changed

packages/cli/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@
3535
"scripts": {
3636
"build": "node --import=./scripts/load.mjs scripts/build.mjs",
3737
"build:sea:internal:bootstrap": "node .config/esbuild.sea-bootstrap.build.mjs",
38-
"build:js": "node scripts/extract-yoga-wasm.mjs && node scripts/extract-onnx-runtime.mjs && node scripts/extract-minilm-model.mjs && node .config/esbuild.cli.build.mjs",
39-
"build:watch": "node scripts/extract-yoga-wasm.mjs && node scripts/extract-onnx-runtime.mjs && node scripts/extract-minilm-model.mjs && node .config/esbuild.cli.build.mjs --watch",
38+
"build:js": "node scripts/extract-yoga-wasm.mjs && node .config/esbuild.cli.build.mjs",
39+
"build:watch": "node scripts/extract-yoga-wasm.mjs && node .config/esbuild.cli.build.mjs --watch",
4040
"dev": "pnpm run build:watch",
4141
"publish:sea": "node --import=./scripts/load.mjs scripts/publish-sea.mjs",
4242
"check": "node --import=./scripts/load.mjs scripts/check.mjs",

packages/cli/scripts/build.mjs

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -84,16 +84,16 @@ async function main() {
8484
command: 'pnpm',
8585
args: ['run', 'clean:dist'],
8686
},
87-
{
88-
name: 'Extract MiniLM Model',
89-
command: 'node',
90-
args: ['scripts/extract-minilm-model.mjs'],
91-
},
92-
{
93-
name: 'Extract ONNX Runtime',
94-
command: 'node',
95-
args: ['scripts/extract-onnx-runtime.mjs'],
96-
},
87+
// {
88+
// name: 'Extract MiniLM Model',
89+
// command: 'node',
90+
// args: ['scripts/extract-minilm-model.mjs'],
91+
// },
92+
// {
93+
// name: 'Extract ONNX Runtime',
94+
// command: 'node',
95+
// args: ['scripts/extract-onnx-runtime.mjs'],
96+
// },
9797
{
9898
name: 'Extract Yoga WASM',
9999
command: 'node',

packages/cli/src/commands/ask/handle-ask.mts

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -271,13 +271,17 @@ async function getEmbeddingPipeline() {
271271
}
272272

273273
try {
274+
// TEMPORARILY DISABLED: ONNX Runtime build issues.
274275
// Load our custom MiniLM inference engine.
275276
// This uses direct ONNX Runtime + embedded WASM (no transformers.js).
276277
// Note: Model is optional - pattern matching works fine without it.
277-
const { MiniLMInference } = await import('../../utils/minilm-inference.mts')
278-
embeddingPipeline = await MiniLMInference.create()
278+
// const { MiniLMInference } = await import('../../utils/minilm-inference.mts')
279+
// embeddingPipeline = await MiniLMInference.create()
280+
// return embeddingPipeline
279281

280-
return embeddingPipeline
282+
// Temporarily fall back to pattern matching only.
283+
embeddingPipelineFailure = true
284+
return null
281285
} catch (_e) {
282286
// Model not available - silently fall back to pattern matching.
283287
embeddingPipelineFailure = true

0 commit comments

Comments
 (0)