FEAT: Proof of Pizza - Agentic Dominos Ordering #1005

ropresearch · 2024-12-12T06:34:52Z

Relates to: PROOF OF PIZZA

Risks

Needing to use ozempic after a few days

Background

What does this PR do?

Modifies the pizza plugin made by Shaw and also provides an example of direct api integration in the Twitter client (this is how I did it)

What kind of change is this?

Code change to a plugin PR and example in the Twitter client.
Also because it's just cool.

Documentation changes needed?

N/A

Testing

The plugin has not been tested, the example for the direct Twitter integration is fully tested and what I did.

Where should a reviewer start?

packages/plugin-pizza and packages/client-twitter/pizza.ts

Gonna link the twitter post lol
https://x.com/ropirito/status/1867013533243769100

chandognft · 2024-12-12T06:39:39Z

wow

monilpat

Thanks for doing this honestly you are the GOAT for this!

monilpat · 2024-12-13T04:27:42Z

packages/client-twitter/src/interactions.ts

+            modelClass: ModelClass.LARGE,
+        });
+
+        console.log("[PIZZA-GEN][INTERACTIONS CLIENT] PIZZA check response: ", pizzaCheckResponse, " ", currentPost);


Thanks for CRUSHING this you are the GOAT for doing this! Can we elizaLogger everywhere instead

monilpat · 2024-12-13T04:28:20Z

packages/client-twitter/src/interactions.ts

+        ${currentPost}`
+        + pizzaDecisionFooter;
+
+        const pizzaCheckResponse = await generateText({


Highly recommend using generateObjectV2 which returns a typesafe JSON instead of having the parse the response and hope for the best

generateObjectV2 doesn't work with Groq, it throws "Error: Unknown model at getEncodingNameForModel". Is that a known thing?
in generateText it's hardcoded to gpt-4o

monilpat · 2024-12-13T04:29:27Z

packages/client-twitter/src/interactions.ts

@@ -282,6 +290,37 @@ export class TwitterInteractionClient {
            this.client.saveRequestMessage(message, state);
        }

+        const pizzaCheck = `


Also, should this directly be added to the twitter client and if so should it be configurable as it seems like overkill to always run

monilpat · 2024-12-13T04:30:43Z

packages/core/src/parsing.ts

@@ -31,6 +36,24 @@ export const parseShouldRespondFromText = (
              : null;
 };

+export const parsePizzaDecisionFromText = (


This can be removed if you use generateObjectV2

monilpat · 2024-12-13T04:35:10Z

packages/plugin-dominos/src/PizzaOrderManager.ts

+    private readonly menuConfig = {
+        defaultProductCode: "PIZZA",
+        basePrices: {
+            [PizzaSize.SMALL]: 9.99,


Not in this but would be good to make these fetched from the site re prices

monilpat · 2024-12-13T04:38:38Z

packages/plugin-dominos/src/PizzaOrderManager.ts

+            return order;
+
+        } catch (error) {
+            console.error('Error processing order:', error);


elizaLogger :)

monilpat · 2024-12-13T04:38:59Z

packages/plugin-dominos/src/actions/confirmOrder.ts

+export const confirmOrder: Action = {
+    name: "CONFIRM_ORDER",
+    similes: ["FINALIZE_ORDER", "FINISH_ORDER", "PLACE_ORDER"],
+    examples: [


Would recommend adding some

monilpat · 2024-12-13T04:39:37Z

packages/plugin-dominos/src/actions/confirmOrder.ts

+            order.status = OrderStatus.CONFIRMED;
+            await orderManager.saveOrder(userId, order);
+
+            return (


Also could use callback here instead

monilpat · 2024-12-13T04:40:01Z

packages/plugin-dominos/src/actions/endOrder.ts

+    description: "Ends the current pizza order and clears the order data.",
+    similes: ["END_ORDER", "FINISH_ORDER", "COMPLETE_ORDER", "STOP_ORDER"],
+    examples: [
+        [


Thanks for all the great examples

monilpat · 2024-12-13T04:43:58Z

packages/plugin-dominos/src/actions/startOrder.ts

+    console.log("Existing order: ", existingOrder);
+
+    // Extract order details from message using LLM
+    const extractionTemplate = `


Would consider moving the template and the schema definition to a new file template to templates.ts and schema to types.ts

monilpat · 2024-12-13T04:47:18Z

And merge conflicts :)

monilpat · 2024-12-13T04:55:07Z

We may want to test out the different plugins via a quick screengrab. Once again most of these aren't blocking just wanted to share to help! I know you have SO much going on so thanks for taking the time for this @ropresearch :)

ropresearch · 2024-12-13T05:03:33Z

Thanks guys! Yeah a bunch of this bc of how I have my agent set up, will adjust it for the main repo :)

Also agree its overkill in the actual twitter client haha, was more for demo purposes!

monilpat · 2024-12-13T05:05:29Z

Thanks guys! Yeah a bunch of this bc of how I have my agent set up, will adjust it for the main repo :)

Also agree its overkill in the actual twitter client haha, was more for demo purposes!

Yeah makes total sense!

Awesome glad we are aligned!

Great work :)

packages/core/src/parsing.ts

+    const match = text
+        .split('\n')[0]
+        .trim()
+        .replace("[", "")
+        .toUpperCase()
+        .replace("]", "")


To fix the problem, we need to ensure that all occurrences of the character ] are replaced in the input string. This can be achieved by using a regular expression with the global flag (g). This way, the replace method will replace all occurrences of the specified character in the string.

packages/core/src/parsing.ts

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

lalalune and others added 5 commits December 10, 2024 22:33

init commit of dominos, untested

e425d35

import types from dominos

0d33b02

order flow is kind of working but still bugs

2df768d

billions

a4900a9

pizzagate

d90b614

ropresearch marked this pull request as ready for review December 12, 2024 06:35

odilitime changed the base branch from main to develop December 12, 2024 07:19

Base automatically changed from develop to main December 13, 2024 02:37

monilpat changed the base branch from main to develop December 13, 2024 04:26

monilpat reviewed Dec 13, 2024

View reviewed changes

Base automatically changed from develop to main December 13, 2024 19:07

Merge branch 'develop' into ropirito/pizzabot

975c76a

wtfsayo changed the base branch from main to develop January 6, 2025 17:23

github-advanced-security bot found potential problems Jan 6, 2025

View reviewed changes

wtfsayo and others added 7 commits January 6, 2025 22:54

Fix code scanning alert no. 44: Incomplete string escaping or encoding

4f2a81a

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

Merge branch 'develop' into ropirito/pizzabot

4ed46f7

update/replace elizaos

76d9e61

test pr

f68c66f

enhance existing actions and Order Manager

2256335

test plugin build

bfe6504

add missing handler

630c4fe

wtfsayo added the testing label Jan 6, 2025

@@ -26,3 +26,3 @@
                     .toUpperCase()
-                    .replace("]", "")
+                    .replace(/\]/g, "")
                     .match(/^(RESPOND|IGNORE|STOP)$/i);
@@ -47,3 +47,3 @@
                     .toUpperCase()
-                    .replace("]", "")
+                    .replace(/\]/g, "")
                     .match(/^(YES|NO)$/i);

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

FEAT: Proof of Pizza - Agentic Dominos Ordering #1005

FEAT: Proof of Pizza - Agentic Dominos Ordering #1005

ropresearch commented Dec 12, 2024

chandognft commented Dec 12, 2024

monilpat left a comment

monilpat Dec 13, 2024

monilpat Dec 13, 2024

den0xR Dec 16, 2024 •

edited

Loading

monilpat Dec 13, 2024

monilpat Dec 13, 2024

monilpat Dec 13, 2024

monilpat Dec 13, 2024

monilpat Dec 13, 2024

monilpat Dec 13, 2024

monilpat Dec 13, 2024

monilpat Dec 13, 2024

monilpat commented Dec 13, 2024

monilpat commented Dec 13, 2024

ropresearch commented Dec 13, 2024

monilpat commented Dec 13, 2024

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

FEAT: Proof of Pizza - Agentic Dominos Ordering #1005

Are you sure you want to change the base?

FEAT: Proof of Pizza - Agentic Dominos Ordering #1005

Conversation

ropresearch commented Dec 12, 2024

Relates to: PROOF OF PIZZA

Risks

Background

What does this PR do?

What kind of change is this?

Documentation changes needed?

Testing

Where should a reviewer start?

chandognft commented Dec 12, 2024

monilpat left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

den0xR Dec 16, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

monilpat commented Dec 13, 2024

monilpat commented Dec 13, 2024

ropresearch commented Dec 13, 2024

monilpat commented Dec 13, 2024

den0xR Dec 16, 2024 •

edited

Loading