Repository navigation
docs(agents): E2B/Daytona sandboxes, fleshed-out examples, design patterns first - #115
Conversation
…terns first Co-authored-by: Amp <amp@ampcode.com> Amp-Thread-ID: https://ampcode.com/threads/T-af28ce42-6c01-4358-8608-8c5c7f7e3150
|
This PR was not deployed automatically as @eersnington does not have access to the Railway project. In order to get automatic PR deploys, please add @eersnington to your workspace on Railway. |
| c.log.error({ msg: "reply failed", error }); | ||
| } finally { |
There was a problem hiding this comment.
🟠 Medium · Failed replies are permanently discarded
Any failure from prompt, getLastAssistantText, or postReply is caught, after which finally removes the comment from durable state and the action returns successfully. A transient model/provider/GitHub failure therefore produces no reply and can never be retried. The same drain implementation appears in the Discord, Linear, and Slack examples. Only dequeue after postReply succeeds; on failure, retain the head item and rethrow or schedule an explicit bounded retry/backoff.
| </div> | ||
|
|
||
| <CodeSnippet file="examples/design-patterns/agent-per-key.ts" title="keys.ts" /> | ||
| <CodeGroup> |
There was a problem hiding this comment.
🔵 Low · Multi-file examples bypass the workspace contract
This new group spans related server.ts and client.ts modules but omits workspace, contrary to the repository's required multi-file-example convention. The other groups added on this page, plus the new/expanded groups in agent-to-agent, quickstart, schedules, subagents, user-subscriptions, and workflows, do the same. Mark these as <CodeGroup workspace> so the files are treated and checked as one compilable example.
| "adHocServices": { | ||
| "rivet-website": { | ||
| "name": "rivet-website", | ||
| "command": "cd /home/user/workspace/repos/website && ./node_modules/.bin/tsx scripts/assemble.ts && ./node_modules/.bin/tsx scripts/generate-markdown.ts && ./node_modules/.bin/astro dev --host 0.0.0.0 --port 4321", |
There was a problem hiding this comment.
🔵 Low · Local portal state is committed as documentation
This generated editor/runtime metadata hard-codes one developer's /home/user/workspace/repos/website checkout and transient port 31227; it is not portable project configuration and sits inside the published docs source tree. Remove both .amp/portals files and ignore this local state so another portal run cannot reintroduce it.
Polishes the Agents docs (
agents/only).Sandboxes
e2b.tsanddaytona.tsside by side.Structure
Examples
apiKeysempty-string fallback, the reviewer's missing repo, and the auth placeholder comment.Writing