Skip to content

Conversation

@Tokeneyez420
Copy link

Hi team, This pull request resolves the @xenova/transformers version error (as reported in #843) that was preventing claude-flow from initializing correctly. To fix it, I performed the following steps:
Removed pnpm-lock.yaml, node_modules, and package-lock.json to ensure a clean state.
Cleared the npm cache (npm cache clean --force).
Re-initialized claude-flow using npx claude-flow@alpha init --force.
I've confirmed locally that the initialization command now completes successfully without the previous error. This should address the issue described in #843. Please let me know if you have any questions or need further review! Thanks,

@BartDuwez
Copy link

Seems like there a bunch of other things happening in the PR too, care to explain?

@veraset-marketing
Copy link

veraset-marketing commented Oct 28, 2025

This didn't resolve the issue for me. Still getting the same error.

EDIT: initialized without scripts, and it worked.

💪

@Tokeneyez420
Copy link
Author

Hi @BartDuwez and @veraset-marketing,
Thanks for the feedback. @BartDuwez: You're absolutely right, there are a lot of changes in this PR. My intention was to resolve the @xenova/transformers error by ensuring a clean dependency state. The large number of deletions (27,414) comes from removing node_modules, package-lock.json, and pnpm-lock.yaml, and the insertions (331) are from the re-initialization process which recreated many project-specific files and re-installed dependencies. I understand this makes the PR difficult to review, and I apologize for that. @veraset-marketing: I'm very sorry to hear that the fix didn't resolve the issue for you. This indicates that my proposed solution is either incomplete or not universally applicable. Given this, I will:
Revert my current commit to clean up the PR and remove the large number of dependency changes. This will make it easier to focus on a more targeted fix.
Investigate further to understand why the @xenova/transformers error persists for others and find a more robust solution.
Open a new, more focused Pull Request once I have a confirmed, working, and cleaner fix.
Sorry for the confusion, learning

@veraset-marketing
Copy link

@Tokeneyez420 wanted to follow up to say that it DID work.

This is the first time I've ever applied a fix from GitHub anywhere to anything. When that Claude-flow MCP came back online, it was a satisfying moment.

Thanks for the contribution!

@Tokeneyez420
Copy link
Author

Hi @veraset-marketing,
Thank you so much for the follow-up and clarification! That's fantastic news that it worked for you when initialized "without scripts." I'm glad the steps helped resolve the issue. @BartDuwez: Based on @veraset-marketing's feedback, it seems the fix does work, but potentially with a nuance around "initializing without scripts." I've re-applied my original commit to this PR. The large number of changes in this PR is still due to the cleanup (removing node_modules, package-lock.json, pnpm-lock.yaml) and subsequent re-initialization. This process effectively resets the project's dependencies and generated files. My understanding now is that the core issue might be related to how claude-flow handles its scripts or existing environment during initialization, rather than a direct problem with the @xenova/transformers version itself, as a clean re-initialization seems to resolve it. I'm happy to discuss this further or make any adjustments needed to make this PR more reviewable or targeted. Thanks again for the help in debugging this! Best, Tokeneyez420

@BartDuwez
Copy link

@Tokeneyez420 perhaps I'm not understanding you correctly, but the only changes I see now is that you added the metrics folder into the .gitignore file, which triggers the removal of some of those metric files...

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.

3 participants