Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

♻️ Refactor and add retries #857

Merged
merged 24 commits into from Jun 26, 2023
Merged

♻️ Refactor and add retries #857

merged 24 commits into from Jun 26, 2023

Conversation

asim-shrestha
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Jun 25, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
agent-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2023 1:34am
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2023 1:34am

@vercel vercel bot temporarily deployed to Preview – docs June 25, 2023 21:59 Inactive
@@ -63,7 +63,7 @@ const getMessagePrefix = (message: Message) => {
} else if (getTaskStatus(message) === TASK_STATUS_COMPLETED) {
return `Executing: ${message.value}`;
} else if (getTaskStatus(message) === TASK_STATUS_FINAL) {
return `Finished: ${message.value}`;
return `Finished:`;
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
return `Finished:`;
return `Finished.`;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Some jank with task coupling. It will display the task value after this 🙃

useAgentStore.getState().setIsAgentThinking(false);

this.workLog.shift();
if (this.isRunning) {
Copy link
Contributor

Choose a reason for hiding this comment

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

invert this condition

next/src/services/agent/autonomous-agent.ts Outdated Show resolved Hide resolved
next/src/services/agent/autonomous-agent.ts Outdated Show resolved Hide resolved
next/src/services/agent/message-service.ts Outdated Show resolved Hide resolved
@vercel vercel bot temporarily deployed to Preview – docs June 26, 2023 01:34 Inactive
@asim-shrestha asim-shrestha merged commit bd50267 into main Jun 26, 2023
8 of 9 checks passed
@asim-shrestha asim-shrestha deleted the pausing branch July 4, 2023 02:47
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.

None yet

2 participants