Skip to content

Commit

Permalink
chore: move multi-model into single sub module (#1443)
Browse files Browse the repository at this point in the history
  • Loading branch information
himself65 authored Nov 7, 2024
1 parent 396b1e1 commit 14cc9eb
Show file tree
Hide file tree
Showing 19 changed files with 83 additions and 30 deletions.
8 changes: 8 additions & 0 deletions .changeset/cool-cows-lick.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
---
"@llamaindex/env": patch
"llamaindex": patch
"@llamaindex/clip": patch
"@llamaindex/huggingface": patch
---

chore: move multi-model into single sub module
12 changes: 12 additions & 0 deletions packages/env/multi-model/package.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
{
"type": "module",
"main": "./dist/index.cjs",
"module": "./dist/index.js",
"types": "./dist/index.d.ts",
"exports": {
"browser": "./dist/index.browser.js",
"edge-light": "./dist/index.edge-light.js",
"workerd": "./dist/index.workerd.js"
},
"private": true
}
29 changes: 28 additions & 1 deletion packages/env/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
"types": "./dist/index.d.ts",
"import": "./dist/index.js",
"require": "./dist/index.cjs",
"default": "./dist/index.cjs"
"default": "./dist/index.js"
},
"workerd": {
"types": "./dist/index.workerd.d.ts",
Expand All @@ -50,9 +50,36 @@
"types": "./dist/index.d.cts",
"default": "./dist/index.cjs"
}
},
"./multi-model": {
"workerd": {
"types": "./multi-model/dist/multi-model.workerd.d.ts",
"default": "./multi-model/dist/multi-model.workerd.js"
},
"edge-light": {
"types": "./multi-model/dist/multi-model.edge-light.d.ts",
"default": "./multi-model/dist/multi-model.edge-light.js"
},
"browser": {
"types": "./multi-model/dist/multi-model.browser.d.ts",
"default": "./multi-model/dist/multi-model.browser.js"
},
"import": {
"types": "./multi-model/dist/multi-model.d.ts",
"default": "./multi-model/dist/multi-model.js"
},
"require": {
"types": "./multi-model/dist/multi-model.d.cts",
"default": "./multi-model/dist/multi-model.cjs"
},
"default": {
"types": "./multi-model/dist/multi-model.d.ts",
"default": "./multi-model/dist/multi-model.js"
}
}
},
"files": [
"multi-model",
"dist",
"CHANGELOG.md",
"!**/*.tsbuildinfo"
Expand Down
6 changes: 0 additions & 6 deletions packages/env/src/index.browser.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,6 @@
import "./global-check.js";

export { consoleLogger, emptyLogger, type Logger } from "./logger/index.js";
export {
loadTransformers,
setTransformers,
type LoadTransformerEvent,
type OnLoad,
} from "./multi-model/index.browser.js";
export { Tokenizers, tokenizers, type Tokenizer } from "./tokenizers/js.js";
export { NotSupportCurrentRuntimeClass } from "./utils/shared.js";
export * from "./web-polyfill.js";
Expand Down
6 changes: 0 additions & 6 deletions packages/env/src/index.edge-light.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,6 @@
*/
import "./global-check.js";
export { consoleLogger, emptyLogger, type Logger } from "./logger/index.js";
export {
loadTransformers,
setTransformers,
type LoadTransformerEvent,
type OnLoad,
} from "./multi-model/index.non-nodejs.js";
export * from "./node-polyfill.js";
export { Tokenizers, tokenizers, type Tokenizer } from "./tokenizers/js.js";
export { NotSupportCurrentRuntimeClass } from "./utils/shared.js";
6 changes: 0 additions & 6 deletions packages/env/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,12 +35,6 @@ export function createSHA256(): SHA256 {
}

export { consoleLogger, emptyLogger, type Logger } from "./logger/index.js";
export {
loadTransformers,
setTransformers,
type LoadTransformerEvent,
type OnLoad,
} from "./multi-model/index.js";
export { Tokenizers, tokenizers, type Tokenizer } from "./tokenizers/node.js";
export {
AsyncLocalStorage,
Expand Down
6 changes: 0 additions & 6 deletions packages/env/src/index.workerd.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,4 @@ export function getEnv(name: string): string | undefined {
}

export { consoleLogger, emptyLogger, type Logger } from "./logger/index.js";
export {
loadTransformers,
setTransformers,
type LoadTransformerEvent,
type OnLoad,
} from "./multi-model/index.non-nodejs.js";
export { Tokenizers, tokenizers, type Tokenizer } from "./tokenizers/js.js";
File renamed without changes.
File renamed without changes.
6 changes: 6 additions & 0 deletions packages/env/src/multi-model.browser.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
export {
loadTransformers,
setTransformers,
type LoadTransformerEvent,
type OnLoad,
} from "./internal/multi-model/browser.js";
6 changes: 6 additions & 0 deletions packages/env/src/multi-model.edge-light.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
export {
loadTransformers,
setTransformers,
type LoadTransformerEvent,
type OnLoad,
} from "./internal/multi-model/non-node.js";
6 changes: 6 additions & 0 deletions packages/env/src/multi-model.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
export {
loadTransformers,
setTransformers,
type LoadTransformerEvent,
type OnLoad,
} from "./internal/multi-model/node.js";
6 changes: 6 additions & 0 deletions packages/env/src/multi-model.workerd.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
export {
loadTransformers,
setTransformers,
type LoadTransformerEvent,
type OnLoad,
} from "./internal/multi-model/non-node.js";
4 changes: 2 additions & 2 deletions packages/llamaindex/e2e/node/embedding/clip.e2e.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import type { LoadTransformerEvent } from "@llamaindex/env";
import { setTransformers } from "@llamaindex/env";
import type { LoadTransformerEvent } from "@llamaindex/env/multi-model";
import { setTransformers } from "@llamaindex/env/multi-model";
import { ClipEmbedding, ImageNode, Settings } from "llamaindex";
import assert from "node:assert";
import { type Mock, test } from "node:test";
Expand Down
5 changes: 4 additions & 1 deletion packages/providers/clip/src/embedding.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,10 @@ import { MultiModalEmbedding } from "@llamaindex/core/embeddings";
import type { ImageType } from "@llamaindex/core/schema";
// only import type, to avoid bundling error
import { Settings } from "@llamaindex/core/global";
import { type LoadTransformerEvent, loadTransformers } from "@llamaindex/env";
import {
type LoadTransformerEvent,
loadTransformers,
} from "@llamaindex/env/multi-model";

Check failure on line 8 in packages/providers/clip/src/embedding.ts

View workflow job for this annotation

GitHub Actions / typecheck

Cannot find module '@llamaindex/env/multi-model' or its corresponding type declarations.
import type {
CLIPTextModelWithProjection,
CLIPVisionModelWithProjection,
Expand Down
5 changes: 4 additions & 1 deletion packages/providers/huggingface/src/embedding.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
import { BaseEmbedding } from "@llamaindex/core/embeddings";
import { Settings } from "@llamaindex/core/global";
import { type LoadTransformerEvent, loadTransformers } from "@llamaindex/env";
import {
type LoadTransformerEvent,
loadTransformers,
} from "@llamaindex/env/multi-model";
import type { pipeline } from "@xenova/transformers";
import { HuggingFaceEmbeddingModelType } from "./shared";

Expand Down
2 changes: 1 addition & 1 deletion packages/providers/huggingface/src/llm.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import {
type LLMChatParamsStreaming,
type LLMMetadata,
} from "@llamaindex/core/llms";
import { loadTransformers } from "@llamaindex/env";
import { loadTransformers } from "@llamaindex/env/multi-model";
import type {
PreTrainedModel,
PreTrainedTokenizer,
Expand Down

0 comments on commit 14cc9eb

Please sign in to comment.