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

Typescript migration #105

Open
wants to merge 25 commits into
base: main
Choose a base branch
from

Conversation

Walchshofer
Copy link

No description provided.

- Remove old JavaScript implementations
- Remove backup type definitions
- Clean up legacy utility files
- Add core utilities (enums, errors, logger)
- Add factory implementations
- Add event handlers
- Add helper utilities
- Add managers
- Add parsers
- Add domain-specific type organization
- Add type guards and validations
- Implement comprehensive type definitions
- Set up central type exports
- Add new Agent class definitions
- Update task management system
- Add team management functionality
- Implement TypeScript conversions
- Convert stores to TypeScript
- Implement proper type safety
- Update state management
- Add proper type definitions
- Update subscribers
- Add index exports
- Clean up remaining conversions
@darielnoel
Copy link
Contributor

Hey @Walchshofer,

Thanks a ton for diving into the TypeScript PR! It’s awesome to see the effort you’re putting in.

Quick update on where we’re at: we’ve got another big PR from @alienkarma in the pipeline. It sets up ESLint and Prettier and touches a lot of files, so we’ve been planning to merge it after our initial launch. Given the scope of both PRs, it’d be super helpful to sync up and figure out the smoothest path forward.

Also, the TypeScript migration is exciting but could mean some big shifts for current JavaScript users, plus tweaks to our docs and onboarding flow. Let’s chat about the best way to handle these so everyone benefits. If you’re up for it, jump into our Discord so we can brainstorm with the team.

Thanks again for all your hard work, really looking forward to making this happen together!

Cheers,
Dariel

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