Skip to content
This repository was archived by the owner on Feb 3, 2026. It is now read-only.

Add AbstractAIService and AbstractWebSearchService#202

Merged
Herklos merged 3 commits intomasterfrom
feature/add-ai-services
Jan 31, 2026
Merged

Add AbstractAIService and AbstractWebSearchService#202
Herklos merged 3 commits intomasterfrom
feature/add-ai-services

Conversation

@Herklos
Copy link
Member

@Herklos Herklos commented Jan 23, 2026

No description provided.

@Herklos Herklos requested a review from GuillaumeDSM January 23, 2026 22:26
@Herklos Herklos force-pushed the feature/add-ai-services branch 10 times, most recently from b75a2b9 to 432fb8a Compare January 29, 2026 10:06
Signed-off-by: Herklos <herklos@drakkar.software>
@Herklos Herklos force-pushed the feature/add-ai-services branch 10 times, most recently from 9d8ffed to 1d56cff Compare January 31, 2026 09:58
@Herklos Herklos changed the title Add AbstractAIService Add AbstractAIService and AbstractWebSearchService Jan 31, 2026
@Herklos Herklos marked this pull request as ready for review January 31, 2026 09:58
@Herklos
Copy link
Member Author

Herklos commented Jan 31, 2026

PR is ready!

Copy link
Member

@GuillaumeDSM GuillaumeDSM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good ideas 👍

Comment on lines 86 to 89
# Preserve function metadata
wrapper.__name__ = func.__name__
wrapper.__doc__ = func.__doc__
wrapper.__module__ = func.__module__
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think functools.wraps does this, no need to recode it https://docs.python.org/3/library/functools.html#module-functools

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! It's up

@Herklos Herklos force-pushed the feature/add-ai-services branch from 1d56cff to 7d5968f Compare January 31, 2026 13:37
)
if retry_delay > 0:
await asyncio.sleep(retry_delay)
continue
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this continue is useless

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right 👍

@Herklos Herklos force-pushed the feature/add-ai-services branch from 7d5968f to b209f1a Compare January 31, 2026 13:39
Signed-off-by: Herklos <herklos@drakkar.software>
@Herklos Herklos force-pushed the feature/add-ai-services branch from b209f1a to 93606b3 Compare January 31, 2026 13:40
@Herklos Herklos enabled auto-merge (rebase) January 31, 2026 13:40
@Herklos Herklos merged commit 89a718c into master Jan 31, 2026
3 checks passed
@Herklos Herklos deleted the feature/add-ai-services branch January 31, 2026 13:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants