camel icon indicating copy to clipboard operation
camel copied to clipboard

[feat] Enhance WeChatToolkit

Open Ol1ver0413 opened this issue 3 months ago • 12 comments

Description

Enhance WeChatToolkit #3112 Integrate wechaty https://github.com/wechaty/wechaty

Checklist

Go over all the following points, and put an x in all the boxes that apply.

  • [ ] I have read the CONTRIBUTION guide (required)
  • [ ] I have linked this PR to an issue using the Development section on the right sidebar or by adding Fixes #issue-number in the PR description (required)
  • [ ] I have checked if any dependencies need to be added or updated in pyproject.toml and uv lock
  • [ ] I have updated the tests accordingly (required for a bug fix or a new feature)
  • [ ] I have updated the documentation if needed:
  • [ ] I have added examples if this is a new feature

If you are unsure about any of these, don't hesitate to ask. We are here to help!

Ol1ver0413 avatar Sep 28 '25 16:09 Ol1ver0413

[!IMPORTANT]

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • [ ] Create PR with unit tests
  • [ ] Post copyable unit tests in a comment
  • [ ] Commit unit tests in branch wechaty

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

ā¤ļø Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot] avatar Sep 28 '25 16:09 coderabbitai[bot]

I'm still trying to apply the token, which is a little bit hard to apply.

Ol1ver0413 avatar Sep 28 '25 16:09 Ol1ver0413

@Ol1ver0413 thanks !is this pr ready for review?Let us know if there's anything we can support!

fengju0213 avatar Sep 30 '25 08:09 fengju0213

@Ol1ver0413 thanks !is this pr ready for review?Let us know if there's anything we can support!

Thanks @fengju0213 ! I applied for a token on this website in order to do some tests, but I haven’t received any reply.

Ol1ver0413 avatar Oct 01 '25 02:10 Ol1ver0413

Hey @Ol1ver0413,

Just checking in to see whether you've had any luck getting the necessary tokens! Also would love to hear any updates you have about progress of the project.

waleedalzarooni avatar Nov 04 '25 04:11 waleedalzarooni

Hey @Ol1ver0413,

Just checking in to see whether you've had any luck getting the necessary tokens! Also would love to hear any updates you have about progress of the project.

Thanks @waleedalzarooni. Sorry for the delay — I’ve been quite busy with work lately. I’ll review this feature asap~

Ol1ver0413 avatar Nov 10 '25 08:11 Ol1ver0413

Hey @Ol1ver0413, Just checking in to see whether you've had any luck getting the necessary tokens! Also would love to hear any updates you have about progress of the project.

Thanks @waleedalzarooni. Sorry for the delay — I’ve been quite busy with work lately. I’ll review this feature asap~

No problem, thanks for the heads up

waleedalzarooni avatar Nov 10 '25 08:11 waleedalzarooni

Hey @Ol1ver0413, Just checking in to see whether you've had any luck getting the necessary tokens! Also would love to hear any updates you have about progress of the project.

Thanks @waleedalzarooni. Sorry for the delay — I’ve been quite busy with work lately. I’ll review this feature asap~

No problem, thanks for the heads up

Hey @waleedalzarooni @fengju0213 , I’ve noticed that using this library often leads to a lot of issues. Since it hasn’t been updated for many years, I encounter various errors with both Python 3.11 and 3.12. Also, I haven’t been able to get my token for a long time. Maybe you could give it a try and then we can compare notes?

Logs are as follow: raise ValueError(f'mutable default {type(f.default)} for field ' ValueError: mutable default <class 'wechaty_puppet.schemas.puppet.PuppetOptions'> for field puppet_options is not allowed: use default_factory

Ol1ver0413 avatar Nov 10 '25 09:11 Ol1ver0413

Hey @Ol1ver0413, Just checking in to see whether you've had any luck getting the necessary tokens! Also would love to hear any updates you have about progress of the project.

Thanks @waleedalzarooni. Sorry for the delay — I’ve been quite busy with work lately. I’ll review this feature asap~

No problem, thanks for the heads up

Hey @waleedalzarooni @fengju0213 , I’ve noticed that using this library often leads to a lot of issues. Since it hasn’t been updated for many years, I encounter various errors with both Python 3.11 and 3.12. Also, I haven’t been able to get my token for a long time. Maybe you could give it a try and then we can compare notes?

Logs are as follow: raise ValueError(f'mutable default {type(f.default)} for field ' ValueError: mutable default <class 'wechaty_puppet.schemas.puppet.PuppetOptions'> for field puppet_options is not allowed: use default_factory

It looks like you're right wechaty is not being correctly maintained so probably best to not go ahead with the integration. I will discuss with our internal team and let you know as soon as possible!

waleedalzarooni avatar Nov 13 '25 05:11 waleedalzarooni

Hey @Ol1ver0413, Just checking in to see whether you've had any luck getting the necessary tokens! Also would love to hear any updates you have about progress of the project.

Thanks @waleedalzarooni. Sorry for the delay — I’ve been quite busy with work lately. I’ll review this feature asap~

No problem, thanks for the heads up

Hey @waleedalzarooni @fengju0213 , I’ve noticed that using this library often leads to a lot of issues. Since it hasn’t been updated for many years, I encounter various errors with both Python 3.11 and 3.12. Also, I haven’t been able to get my token for a long time. Maybe you could give it a try and then we can compare notes?

Logs are as follow: raise ValueError(f'mutable default {type(f.default)} for field ' ValueError: mutable default <class 'wechaty_puppet.schemas.puppet.PuppetOptions'> for field puppet_options is not allowed: use default_factory

There may be some legal and regulatory issues regarding the way WeChat messages are received. We have already conducted some research, so we can postpone this PR for now. cc @waleedalzarooni @Ol1ver0413

fengju0213 avatar Nov 13 '25 06:11 fengju0213

Hey @Ol1ver0413, Just checking in to see whether you've had any luck getting the necessary tokens! Also would love to hear any updates you have about progress of the project.

Thanks @waleedalzarooni. Sorry for the delay — I’ve been quite busy with work lately. I’ll review this feature asap~

No problem, thanks for the heads up

Hey @waleedalzarooni @fengju0213 , I’ve noticed that using this library often leads to a lot of issues. Since it hasn’t been updated for many years, I encounter various errors with both Python 3.11 and 3.12. Also, I haven’t been able to get my token for a long time. Maybe you could give it a try and then we can compare notes? Logs are as follow: raise ValueError(f'mutable default {type(f.default)} for field ' ValueError: mutable default <class 'wechaty_puppet.schemas.puppet.PuppetOptions'> for field puppet_options is not allowed: use default_factory

There may be some legal and regulatory issues regarding the way WeChat messages are received. We have already conducted some research, so we can postpone this PR for now. cc @waleedalzarooni @Ol1ver0413

Thanks @fengju0213 @waleedalzarooni!If needed later on, I’d also like to be involved in the integration and development work.

Ol1ver0413 avatar Nov 13 '25 09:11 Ol1ver0413

Hey @Ol1ver0413, Just checking in to see whether you've had any luck getting the necessary tokens! Also would love to hear any updates you have about progress of the project.

Thanks @waleedalzarooni. Sorry for the delay — I’ve been quite busy with work lately. I’ll review this feature asap~

No problem, thanks for the heads up

Hey @waleedalzarooni @fengju0213 , I’ve noticed that using this library often leads to a lot of issues. Since it hasn’t been updated for many years, I encounter various errors with both Python 3.11 and 3.12. Also, I haven’t been able to get my token for a long time. Maybe you could give it a try and then we can compare notes? Logs are as follow: raise ValueError(f'mutable default {type(f.default)} for field ' ValueError: mutable default <class 'wechaty_puppet.schemas.puppet.PuppetOptions'> for field puppet_options is not allowed: use default_factory

There may be some legal and regulatory issues regarding the way WeChat messages are received. We have already conducted some research, so we can postpone this PR for now. cc @waleedalzarooni

Hey @Ol1ver0413, Just checking in to see whether you've had any luck getting the necessary tokens! Also would love to hear any updates you have about progress of the project.

Thanks @waleedalzarooni. Sorry for the delay — I’ve been quite busy with work lately. I’ll review this feature asap~

No problem, thanks for the heads up

Hey @waleedalzarooni @fengju0213 , I’ve noticed that using this library often leads to a lot of issues. Since it hasn’t been updated for many years, I encounter various errors with both Python 3.11 and 3.12. Also, I haven’t been able to get my token for a long time. Maybe you could give it a try and then we can compare notes? Logs are as follow: raise ValueError(f'mutable default {type(f.default)} for field ' ValueError: mutable default <class 'wechaty_puppet.schemas.puppet.PuppetOptions'> for field puppet_options is not allowed: use default_factory

There may be some legal and regulatory issues regarding the way WeChat messages are received. We have already conducted some research, so we can postpone this PR for now. cc @waleedalzarooni @Ol1ver0413

Thanks @fengju0213 @waleedalzarooni!If needed later on, I’d also like to be involved in the integration and development work.

thank you very much! @Ol1ver0413

fengju0213 avatar Nov 13 '25 12:11 fengju0213