Skip to content

fix(artifacts): Prevent double initialization of artifacts service#115

Merged
edenreich merged 3 commits intomainfrom
claude/issue-114-20251018-1634
Oct 18, 2025
Merged

fix(artifacts): Prevent double initialization of artifacts service#115
edenreich merged 3 commits intomainfrom
claude/issue-114-20251018-1634

Conversation

@edenreich
Copy link
Copy Markdown
Contributor

Add WithArtifactService method to ArtifactsServerBuilder to allow sharing
of artifact service instances between A2A server and artifacts server.
This eliminates the duplicate initialization log messages and improves
efficiency.

Changes:

  • Add WithArtifactService() to ArtifactsServerBuilder interface
  • Modify Build() to use provided service if available
  • Update all example applications to use shared artifact service

Fixes #114

Generated with Claude Code

Add WithArtifactService method to ArtifactsServerBuilder to allow sharing
of artifact service instances between A2A server and artifacts server.
This eliminates the duplicate initialization log messages and improves
efficiency.

Changes:
- Add WithArtifactService() to ArtifactsServerBuilder interface
- Modify Build() to use provided service if available
- Update all example applications to use shared artifact service

Fixes #114

Co-authored-by: Eden Reich <edenreich@users.noreply.github.com>
@edenreich edenreich merged commit 5f79ef4 into main Oct 18, 2025
1 check passed
@edenreich edenreich deleted the claude/issue-114-20251018-1634 branch October 18, 2025 17:10
ig-semantic-release-bot bot added a commit that referenced this pull request Oct 18, 2025
## [0.15.2](v0.15.1...v0.15.2) (2025-10-18)

### 🐛 Bug Fixes

* **artifacts:** Prevent double initialization of artifacts service ([#115](#115)) ([5f79ef4](5f79ef4)), closes [#114](#114)
@ig-semantic-release-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 0.15.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Initialization of artifacts service twice is unnecessary

1 participant