Skip to content

Conversation

@chrisayl
Copy link

Summary

This PR fixes two critical dependency issues preventing clean installation and ensuring npm package compatibility:

1. agentdb Dependency Issue (Already fixed locally in v2.7.16, but not in npm package)

  • ✅ Updated to agentdb@^1.6.1 to resolve @xenova/transformers conflict
  • ✅ Kept in optionalDependencies to prevent build failures on systems without native compilation

2. typescript-eslint Compatibility Issue (New fix)

  • ✅ Updated typescript-eslint from ^8.37.0 to ^8.46.0
  • ✅ Version 8.46.0 supports TypeScript >=4.8.4 <6.0.0 (includes 5.9.2)
  • ✅ Resolves peer dependency conflict with TypeScript 5.9.2

Problem

Current NPM Package Issue:

  • claude-flow@alpha v2.7.26 on npm still has agentdb@^1.3.9 (old version)
  • Users installing via npx claude-flow@alpha mcp start get dependency conflicts
  • Local v2.7.16 has the fix but it was never published to npm

TypeScript Issue:

  • [email protected] doesn't support TypeScript 5.9.x
  • Causes npm install to fail with ERESOLVE errors

Solution

This PR ensures that when the next version is published to npm, it will include both fixes:

  1. The agentdb fix from v2.7.16
  2. The typescript-eslint compatibility fix

Testing

npm install  # ✅ Installs cleanly without conflicts
node -e "require('agentdb')"  # ✅ Loads successfully
npm run typecheck  # ✅ TypeScript compilation works

Test Results:

  • ✅ Local installation works without conflicts
  • [email protected] loads successfully
  • ✅ TypeScript 5.9.2 compilation works
  • ✅ All peer dependencies resolved
  • ✅ No ERESOLVE errors

Files Changed

  • package.json: Updated typescript-eslint to ^8.46.0
  • package-lock.json: Regenerated with compatible dependencies

Deployment Note

Important: After merging, please publish to npm:

npm version patch  # or minor/major
npm publish --tag alpha

This will ensure users installing via npx claude-flow@alpha get the fixes.

Related Issues

Fixes: Dependency conflicts preventing clean npm install
Closes: #[issue-number-if-exists]

Checklist

  • Tested locally with clean install
  • agentdb loads without errors
  • TypeScript compilation works
  • No peer dependency warnings
  • Ready for npm publish

This commit fixes two critical dependency issues:

1. **agentdb dependency**: Already fixed in v2.7.16 locally but not published
   - Updated to agentdb@^1.6.1 to resolve @xenova/transformers conflict
   - Moved to optionalDependencies to prevent build failures

2. **typescript-eslint compatibility**: New fix for TypeScript 5.9 support
   - Updated typescript-eslint from ^8.37.0 to ^8.46.0
   - Version 8.46.0 supports TypeScript >=4.8.4 <6.0.0 (includes 5.9.2)

**Issue**: NPM package claude-flow@alpha v2.7.26 has agentdb@^1.3.9
**Solution**: This PR ensures future npm publishes include the correct dependencies

**Testing**:
- ✅ Local installation works without conflicts
- ✅ agentdb loads successfully
- ✅ TypeScript compilation works
- ✅ All peer dependencies resolved

Closes: Dependency conflict preventing clean npm install
@chrisayl
Copy link
Author

This PR fixes the issues described in #848

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