Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Jan 29, 2026

Important

Update preview environment documentation to include detached commands, tool version management, and environment variable usage.

  • Documentation Updates:
    • Add keywords: Detached Commands, Tool Versions, mise to environments.mdx.
    • Update command execution to use detached: true instead of & in environments.mdx.
    • Add logfile option for detached commands in environments.mdx.
  • Environment Variables:
    • Recommend using ROO_<NAME>_HOST variables for service communication.
    • Provide examples for setting API URLs at build time for Vite, Next.js, CRA.
    • Suggest writing ROO_API_HOST to .env files for frameworks using dotenv.
  • Tool Management:
    • Document automatic tool installation using .tool-versions with mise in environments.mdx.

This description was created by Ellipsis for bbebc1e. You can customize this summary. It will automatically update as commits are pushed.

@vercel
Copy link

vercel bot commented Jan 29, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
roo-code-docs Ready Ready Preview, Comment Jan 29, 2026 6:16pm

Request Review

@roomote
Copy link
Contributor

roomote bot commented Jan 29, 2026

Rooviewer Clock   See task on Roo Cloud

Reviewed the preview environment documentation updates. Found 1 issue:

  • Fastify CORS example uses an inline dynamic import pattern that may not work as expected

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@cte cte merged commit 919fb86 into main Jan 29, 2026
4 checks passed
@cte cte deleted the cte/update-preview-env-docs branch January 29, 2026 18:16
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jan 29, 2026
Comment on lines +353 to 356
app.register(import('@fastify/cors'), {
origin: process.env.ROO_WEB_HOST || 'http://localhost:3000'
});
```
Copy link
Contributor

Choose a reason for hiding this comment

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

The inline dynamic import pattern import('@fastify/cors') returns a Promise resolving to a module namespace object ({ default: ... }), not the plugin function directly. This may not work as expected with Fastify's register method. The standard pattern from Fastify's documentation is:

Suggested change
app.register(import('@fastify/cors'), {
origin: process.env.ROO_WEB_HOST || 'http://localhost:3000'
});
```
import cors from '@fastify/cors';
app.register(cors, {
origin: process.env.ROO_WEB_HOST || 'http://localhost:3000'
});

Fix it with Roo Code or mention @roomote and request a fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants