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 into
masterfrom
feature/add-ai-services
Jan 31, 2026
Merged

Herklos merged 3 commits into
masterfrom
feature/add-ai-services

Conversation

@Herklos

@Herklos Herklos commented Jan 23, 2026

Copy link
Copy Markdown
Contributor

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

Herklos commented Jan 31, 2026

Copy link
Copy Markdown
Contributor Author

PR is ready!

@GuillaumeDSM GuillaumeDSM left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
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
Copy Markdown
Contributor 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
Copy Markdown
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
Copy Markdown
Contributor 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