Skip to content

Conversation

@betegon
Copy link
Member

@betegon betegon commented Dec 23, 2025

we got feedback that docs didn't explain available options clearly.

  • Rewrote setup page as "Getting Started" with Desktop App vs CLI vs MCP comparison
  • Added installation and context to CLI page
  • Removed deprecated "overlay" terminology across docs
  • Simplified architecture page to avoid duplicating setup instructions
  • Updated fixture paths and commands to use @spotlightjs/spotlight

@vercel
Copy link

vercel bot commented Dec 23, 2025

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

Project Deployment Review Updated (UTC)
spotlightjs Ready Ready Preview, Comment Dec 23, 2025 8:33pm

Copy link
Member

@BYK BYK left a comment

Choose a reason for hiding this comment

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

Great job! We should also eradicate any mentions of "sidecar" if it is not accompanied with "server".

Future work:

  1. Mention that the server always starts an HTTP-based MCP server too
  2. We can have an stdio-based MCP server that can connect to existing servers or start its own server
  3. spotlight run choosing a random port by default for project isolation

@betegon betegon enabled auto-merge (squash) December 23, 2025 20:33
@betegon betegon merged commit ac52336 into main Dec 23, 2025
20 checks passed
@betegon betegon deleted the bete/feat/remove-overlay-references branch December 23, 2025 20:38
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