Skip to content
This repository has been archived by the owner on Oct 30, 2024. It is now read-only.

expose agent system prompt + give it chat functionality #32

Merged
merged 5 commits into from
Apr 10, 2024

Conversation

AlephNotation
Copy link
Contributor

No description provided.

@AlephNotation AlephNotation self-assigned this Apr 10, 2024
@AlephNotation
Copy link
Contributor Author

Ok got a bit carried away here since we're introducing breaking changes.

This PR now includes an overhaul of the model's response typing. We no longer need to extend ModelResponseType and hope that it includes the extended optional type in the payload. I know this may seem overkill but I was running into this problem yesterday. Plus the more we can enforce type strictness the better these things seem to do.

We now perform a merge with ObjectiveComplete so that if a model wants to return an ObjectiveComplete response it must also include any additional fields we've defined.

Copy link
Contributor

@matildepark matildepark left a comment

Choose a reason for hiding this comment

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

I don't like it when things work, so let's just do it

@matildepark matildepark merged commit c8a5230 into main Apr 10, 2024
1 check passed
@matildepark matildepark deleted the agent-custom/expose-constructor branch April 10, 2024 02:30
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