Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Discord client version not in sync with 0.1.9 #132

Closed
jtalmi opened this issue Feb 6, 2025 · 5 comments
Closed

Discord client version not in sync with 0.1.9 #132

jtalmi opened this issue Feb 6, 2025 · 5 comments

Comments

@jtalmi
Copy link

jtalmi commented Feb 6, 2025

Describe the bug

pnpm build on eliza starter repo fails due to discord plugin version conflict:

$ pnpm build && pnpm start   

> @elizaos/[email protected] build /Users/jonathantalmi/dev/eliza-starter
> tsup src/index.ts --format esm --dts

CLI Building entry: src/index.ts
CLI Using tsconfig: tsconfig.json
CLI tsup v8.3.5
CLI Target: esnext
ESM Build start
DTS Build start
ESM dist/index.js 27.48 KB
ESM ⚡️ Build success in 235ms
DTS ⚡️ Build success in 807ms
DTS dist/index.d.ts 269.00 B

> @elizaos/[email protected] start /Users/jonathantalmi/dev/eliza-starter
> tsc && node --loader ts-node/esm src/index.ts

src/clients/index.ts:20:53 - error TS2345: Argument of type 'import("/Users/jonathantalmi/dev/eliza-starter/node_modules/.pnpm/@[email protected]_@[email protected]_@[email protected][email protected]_w_400c998ea5b04952faad4360fe701faf/node_modules/@elizaos/core/dist/index").IAgentRuntime' is not assignable to parameter of type 'import("/Users/jonathantalmi/dev/eliza-starter/node_modules/.pnpm/@[email protected]_@[email protected]_@[email protected][email protected]_w_fd2bcd6812fd1e3b800382dc0dd7a646/node_modules/@elizaos/core/dist/index").IAgentRuntime'.
  Types of property 'modelProvider' are incompatible.
    Type 'import("/Users/jonathantalmi/dev/eliza-starter/node_modules/.pnpm/@[email protected]_@[email protected]_@[email protected][email protected]_w_400c998ea5b04952faad4360fe701faf/node_modules/@elizaos/core/dist/index").ModelProviderName' is not assignable to type 'import("/Users/jonathantalmi/dev/eliza-starter/node_modules/.pnpm/@[email protected]_@[email protected]_@[email protected][email protected]_w_fd2bcd6812fd1e3b800382dc0dd7a646/node_modules/@elizaos/core/dist/index").ModelProviderName'.
      Property 'LMSTUDIO' is missing in type 'import("/Users/jonathantalmi/dev/eliza-starter/node_modules/.pnpm/@[email protected]_@[email protected]_@[email protected][email protected]_w_fd2bcd6812fd1e3b800382dc0dd7a646/node_modules/@elizaos/core/dist/index").ModelProviderName'.

20     clients.push(await DiscordClientInterface.start(runtime));
                                                       ~~~~~~~


Found 1 error in src/clients/index.ts:20

To Reproduce
Set up the starter project as in the README

Expected behavior

Successful build
Screenshots

Additional context

I fixed the bug by manually installing the 0.1.9 version of "@elizaos/client-discord": "0.1.9",

@jtalmi jtalmi changed the title Discord client version not in sync with 0.1.9 bug Something isn't working Discord client version not in sync with 0.1.9 Feb 6, 2025
@MeepoTu
Copy link

MeepoTu commented Feb 6, 2025

same issue

@MeepoTu
Copy link

MeepoTu commented Feb 6, 2025

i change package.json specifier version fixed this

"pnpm": {
    "overrides": {
      "onnxruntime-node": "1.20.0",
      "@elizaos/client-discord": "0.1.9"
    }
  },

@YoleYu
Copy link
Contributor

YoleYu commented Feb 6, 2025

#128

@positonic
Copy link

This is close by this PR

@madjin
Copy link
Contributor

madjin commented Feb 8, 2025

duplicate, should be fixed, ty

@madjin madjin closed this as completed Feb 8, 2025
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

No branches or pull requests

5 participants