Skip to content

Conversation

treblereel
Copy link
Collaborator

No description provided.

@treblereel treblereel marked this pull request as draft September 29, 2025 22:24
@treblereel treblereel force-pushed the agentic_helper branch 2 times, most recently from 2401aa7 to 61bd518 Compare October 1, 2025 19:20
.outputName("story")
.build());
spy(
dev.langchain4j.agentic.AgenticServices.agentBuilder(CreativeWriter.class)
Copy link
Member

Choose a reason for hiding this comment

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

We can drop the FQCN

}
}

private static class AgentInvocationHandler implements InvocationHandler {
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 we already have something pretty similar on the agentic-langchain4j module. Is it doable to share the same class?

@treblereel treblereel force-pushed the agentic_helper branch 2 times, most recently from 21fa658 to bfec4eb Compare October 6, 2025 22:03
Comment on lines 36 to 38
if (model instanceof HasAgenticScope<?> hasAgenticScope && inputClass.isEmpty()) {
return (T) hasAgenticScope.getAgenticScope();
}
Copy link
Collaborator

@fjtirado fjtirado Oct 7, 2025

Choose a reason for hiding this comment

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

You should not do this change here, but in as (when inputclass is not empty) and asJavaObject (when inputclass is empty) methods of AgenticModel

Copy link
Collaborator

@fjtirado fjtirado Oct 7, 2025

Choose a reason for hiding this comment

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

This class should not be added, the change in convert should be done in asJavaObject

Signed-off-by: Dmitrii Tikhomirov <[email protected]>
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.

3 participants