Skip to content

docs(agents): sync from rivet-dev/agents - #129

Merged
eersnington merged 1 commit into
mainfrom
docs-sync/agents
Oct 7, 2026
Merged

eersnington merged 1 commit into
mainfrom
docs-sync/agents

Conversation

@NathanFlurry

Copy link
Copy Markdown
Member

Automated docs sync from rivet-dev/agents@d08130d.

Do not edit vendor/agents/ here. Edit the docs in rivet-dev/agents and this PR updates itself.

@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-129 October 7, 2026 19:22 Destroyed
@railway-app

railway-app Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

🚅 Deployed to the website-pr-129 environment in rivet-website

Service Status Web Updated
website 😴 Sleeping (View Logs) Web Oct 7, 2026 at 7:49 pm UTC

@the-company-company the-company-company Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 3 medium-severity findings

Reviewed commit 4d886cd.

@@ -1,189 +0,0 @@
---

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 Medium · Preserve the removed Pi Durable route

Deleting this page leaves both links in src/content/posts/2026-10-07-introducing-pi-durable-for-rivet/page.mdx pointing at a missing /agents/docs/pi-durable/ route. The repository's built SEO check fails twice on those links, and deployed historical links would 404. Keep route compatibility by redirecting this path to /agents/docs/pi/ (or retain a forwarding page) when consolidating the content.

@@ -1,33 +1,23 @@
import { createRegistry } from "@earendil-works/pi-durable";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 Medium · Install the new direct dependency in connector quickstarts

This new import is not included in the GitHub connector's standalone npm add command, so the copied server relies on an undeclared transitive dependency and can fail to resolve under package managers that do not hoist it. The same mismatch exists in the updated Slack, Linear, and Discord server snippets. Add @earendil-works/pi-durable to all four connector install commands, as the main quickstart already does.

Comment on lines +30 to +31
await fetch(`https://api.github.com/repos/${task.input.repo}/issues/${task.input.pr}/comments`, {
method: "POST",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 Medium · Make the retried comment phase idempotent

The architecture documented in this change reruns the current task phase after a crash. If GitHub accepts this POST and the Actor stops before the terminal checkpoint is committed, comment runs again and creates a duplicate PR comment, contradicting the guidance that tasks are for work that must finish exactly once. Key the outbound operation by task.id and find/update an existing marked comment on retry, or use an API operation with an idempotency key before committing completion.

@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-129 October 7, 2026 19:39 Destroyed

@the-company-company the-company-company Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 3 medium-severity findings

Reviewed commit 04b814e.

@@ -1,189 +0,0 @@
---

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 Medium · Preserve the removed Pi Durable route

Deleting this page leaves both links in src/content/posts/2026-10-07-introducing-pi-durable-for-rivet/page.mdx pointing at a missing /agents/docs/pi-durable/ route. The repository's built SEO check fails twice on those links, and deployed historical links would 404. Keep route compatibility by redirecting this path to /agents/docs/pi/ (or retain a forwarding page) when consolidating the content.

@@ -1,33 +1,23 @@
import { createRegistry } from "@earendil-works/pi-durable";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 Medium · Install the new direct dependency in connector quickstarts

This new import is not included in the GitHub connector's standalone npm add command, so the copied server relies on an undeclared transitive dependency and can fail to resolve under package managers that do not hoist it. The same mismatch exists in the updated Slack, Linear, and Discord server snippets. Add @earendil-works/pi-durable to all four connector install commands, as the main quickstart already does.

Comment on lines +30 to +31
await fetch(`https://api.github.com/repos/${task.input.repo}/issues/${task.input.pr}/comments`, {
method: "POST",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 Medium · Make the retried comment phase idempotent

The architecture documented in this change reruns the current task phase after a crash. If GitHub accepts this POST and the Actor stops before the terminal checkpoint is committed, comment runs again and creates a duplicate PR comment, contradicting the guidance that tasks are for work that must finish exactly once. Key the outbound operation by task.id and find/update an existing marked comment on retry, or use an API operation with an idempotency key before committing completion.

@eersnington
eersnington merged commit 615beab into main Oct 7, 2026
3 of 4 checks passed
@eersnington
eersnington deleted the docs-sync/agents branch October 7, 2026 20:01

This branch was successfully deployed

No deployments
rivet-website / website-pr-129 — 04b814ec Deployed Oct 7, 2026 by railway-app[bot]
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.

2 participants