Skip to content

Add an example of an agent cog with multiple prompts#851

Merged
juniper-shopify merged 1 commit intomainfrom
juni/Add_an_example_of_an_agent_cog_with_multiple_prompts
Mar 27, 2026
Merged

Add an example of an agent cog with multiple prompts#851
juniper-shopify merged 1 commit intomainfrom
juni/Add_an_example_of_an_agent_cog_with_multiple_prompts

Conversation

@juniper-shopify
Copy link
Copy Markdown
Contributor

No description provided.

@juniper-shopify juniper-shopify marked this pull request as ready for review March 26, 2026 19:25
@juniper-shopify juniper-shopify force-pushed the juni/Add_an_example_of_an_agent_cog_with_multiple_prompts branch from 4516d66 to f40fe07 Compare March 26, 2026 19:32
@juniper-shopify juniper-shopify force-pushed the juni/Make_command_runner_fixtures_helper_support_multiple_invocations branch from 5f4dfe6 to 1c4ffb5 Compare March 26, 2026 19:32
@juniper-shopify juniper-shopify force-pushed the juni/Make_command_runner_fixtures_helper_support_multiple_invocations branch from 1c4ffb5 to 2ce38b8 Compare March 26, 2026 19:40
@juniper-shopify juniper-shopify force-pushed the juni/Add_an_example_of_an_agent_cog_with_multiple_prompts branch from f40fe07 to 3e9c993 Compare March 26, 2026 19:40
@juniper-shopify juniper-shopify force-pushed the juni/Make_command_runner_fixtures_helper_support_multiple_invocations branch from 2ce38b8 to b355fbf Compare March 26, 2026 20:02
@juniper-shopify juniper-shopify force-pushed the juni/Add_an_example_of_an_agent_cog_with_multiple_prompts branch 2 times, most recently from 2401004 to d37f6ec Compare March 26, 2026 20:18
@juniper-shopify juniper-shopify force-pushed the juni/Make_command_runner_fixtures_helper_support_multiple_invocations branch from b355fbf to 2e0e832 Compare March 26, 2026 20:18
dump_raw_agent_messages_to "tmp/claude-messages2.log"
end
end

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is the 2 in the filename intentional or left over from dev?

Session ID: 51c68f29-7210-4c12-852f-0c169f621488
((2 + 2) * 3) - 5 = 12
STDOUT
assert_equal expected_stdout, logged_stdout
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this tripped me up at first — the math works out to 7 but integer! grabs the first number from the response which is 12. might be worth a quick comment so future readers don't think it's a bug?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

oh, no this is dumb. it should grab the last number maybe. I don't like this

Copy link
Copy Markdown
Contributor

@dersam dersam left a comment

Choose a reason for hiding this comment

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

There's a lot of MCP junk in the transcripts, can you pare that down?

@juniper-shopify juniper-shopify force-pushed the juni/Add_an_example_of_an_agent_cog_with_multiple_prompts branch from d37f6ec to 171ba41 Compare March 27, 2026 18:42
@juniper-shopify juniper-shopify changed the base branch from juni/Make_command_runner_fixtures_helper_support_multiple_invocations to graphite-base/851 March 27, 2026 19:07
@juniper-shopify juniper-shopify force-pushed the juni/Add_an_example_of_an_agent_cog_with_multiple_prompts branch from 171ba41 to 3415945 Compare March 27, 2026 19:07
@juniper-shopify juniper-shopify changed the base branch from graphite-base/851 to juni/Fix_number_parsing_order March 27, 2026 19:07
@juniper-shopify juniper-shopify requested a review from dersam March 27, 2026 19:08
Copy link
Copy Markdown
Contributor Author

juniper-shopify commented Mar 27, 2026

Merge activity

  • Mar 27, 7:16 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 27, 7:29 PM UTC: Graphite rebased this pull request as part of a merge.
  • Mar 27, 7:30 PM UTC: @juniper-shopify merged this pull request with Graphite.

@juniper-shopify juniper-shopify changed the base branch from juni/Fix_number_parsing_order to graphite-base/851 March 27, 2026 19:27
@juniper-shopify juniper-shopify changed the base branch from graphite-base/851 to main March 27, 2026 19:28
@juniper-shopify juniper-shopify force-pushed the juni/Add_an_example_of_an_agent_cog_with_multiple_prompts branch from 3415945 to 947de52 Compare March 27, 2026 19:29
@juniper-shopify juniper-shopify merged commit 2346677 into main Mar 27, 2026
3 checks passed
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