Skip to content

docs: add AgentOS dev server previews - #299

Open
RivetAgent wants to merge 1 commit into
mainfrom
docs/agentos-dev-server-previews
Open

RivetAgent wants to merge 1 commit into
mainfrom
docs/agentos-dev-server-previews

Conversation

@RivetAgent

Copy link
Copy Markdown

Summary

  • add a Dev Servers workload guide for the code-generation loop
  • show how to configure AgentOS preview link lifetimes
  • show how to start a generated app's dev server, create a signed preview link, and revoke it
  • clarify when to use private httpRequest() access and when to deploy the finished app with Dynamic Apps

Validation

  • git diff --check
  • parsed docs/sidebar.json with JSON.parse

Note: Biome ignores the docs MDX and sidebar JSON paths, so there is no repo formatter check for these files.

@railway-app

railway-app Bot commented Sep 23, 2026

Copy link
Copy Markdown

This PR was not deployed automatically as @RivetAgent does not have access to the Railway project.

In order to get automatic PR deploys, please add @RivetAgent to your workspace on Railway.

1 similar comment
@railway-app

railway-app Bot commented Sep 23, 2026

Copy link
Copy Markdown

This PR was not deployed automatically as @RivetAgent does not have access to the Railway project.

In order to get automatic PR deploys, please add @RivetAgent to your workspace on Railway.

@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.

🟠 2 medium-severity findings

Reviewed commit d46a348.

Comment on lines +55 to +56
await agent.process.exec("npm install --prefix /home/agentos/app");
await agent.process.spawn("npm", [

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 · Configure the signed path as the dev server's public base

The server is started before the preview path is minted, so a conventional frontend cannot be configured for the /fetch/<token> prefix. For example, Vite's default HTML requests /@vite/client and /src/...; those requests omit the token prefix and hit the actor's non-preview route, which returns 404. Mint the preview first and pass preview.path as the framework's base/asset prefix when spawning the server, or explicitly limit this flow to applications whose asset URLs are relative.

Comment on lines +7 to +9
Dynamic Apps deploys finished applications as immutable releases. During the
code-generation loop, use agentOS when an agent needs to run a conventional
development server with hot reload. A signed preview link forwards HTTP traffic

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 · Do not promise hot reload over an HTTP-only preview

The preview handler forwards each request through the buffered vm.network.httpRequest() API and has no WebSocket/Upgrade path. WebSocket-based HMR clients such as Vite's therefore cannot connect through this link, even if the page and assets are otherwise configured for the preview prefix. Scope this to HTTP previews and document manual refresh, or add WebSocket forwarding before advertising hot reload.

This branch has not been deployed

No deployments
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.

1 participant