-
-
Notifications
You must be signed in to change notification settings - Fork 30
Feat(docs): Improved docs #1082
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
BYK
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed until the MCP section
| Share debugging sessions: | ||
|
|
||
| 1. One team member runs the desktop app | ||
| 2. Others can view the UI via browser at `http://localhost:8969` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This requires tunneling with a tool like Ngrok
|
Merging this. We'll follow up with more improvements soon. |
| # Service 2 (frontend - connects to same sidecar) | ||
| spotlight run -p 8969 npm run dev:frontend |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This may not work
| spotlight run npm test | ||
|
|
||
| # Capture traces during integration tests | ||
| spotlight run -p 0 pytest tests/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| spotlight run -p 0 pytest tests/ | |
| spotlight run pytest tests/ |
| ```go | ||
| // Sentry will automatically use the SENTRY_SPOTLIGHT env var | ||
| sentry.Init(sentry.ClientOptions{ | ||
| Dsn: "your-dsn", // Optional | ||
| }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will it? - getsentry/sentry-go#1085
| ```bash | ||
| npm install -g @spotlightjs/sidecar | ||
| # or use npx | ||
| npx @spotlightjs/sidecar run node server.js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We may also mention this script: http://spotlightjs.com/install.sh
Not ideal but:
curl http://spotlightjs.com/install.sh | sh
| **Example output:** | ||
| ``` | ||
| 🔴 ERROR | 2025-10-31 14:23:45 | ||
| TypeError: Cannot read property 'id' of undefined | ||
| at getUserData (api/users.js:42) | ||
| at processRequest (api/middleware.js:18) | ||
| 📝 LOG | 2025-10-31 14:23:46 | ||
| [INFO] Request processed successfully | ||
| user_id: 12345 | ||
| duration: 234ms | ||
| ``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like hallucination
|
|
||
| **Perfect for:** | ||
| - Log aggregation tools | ||
| - Piping to other CLI tools |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| **Perfect for:** | ||
| - Creating incident reports | ||
| - Sharing errors in documentation | ||
| - GitHub issues or PRs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And feeding into LLMs?
| ### Debug Mode | ||
|
|
||
| Enable verbose logging to troubleshoot issues: | ||
|
|
||
| ```bash | ||
| spotlight tail -d | ||
| # or | ||
| spotlight tail --debug | ||
| ``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not really useful?
|
|
||
| ```bash | ||
| # Start sidecar and run tests, output as JSON | ||
| spotlight tail -f json > spotlight-events.json & |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be ndjson too
|
|
||
| ``` | ||
| 🔴 ERROR | 2025-10-31 14:23:45.123 | ||
| TypeError: Cannot read property 'id' of undefined | ||
| Location: api/users.js:42:10 | ||
| Stack Trace: | ||
| at getUserData (api/users.js:42:10) | ||
| at processRequest (api/middleware.js:18:5) | ||
| at Layer.handle (express/lib/router/layer.js:95:5) | ||
| Context: | ||
| request_id: req_abc123 | ||
| user_id: 12345 | ||
| method: GET | ||
| path: /api/users/12345 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like hallucination again
| - Headless or mobile development | ||
| - When you want debugging in a separate window | ||
| - Team members who prefer desktop apps | ||
| - Quick setup without browser configuration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Browser configuration?
| :::note[Other Platforms] | ||
| Currently, the desktop app is available for macOS. Windows and Linux support is coming soon. | ||
|
|
||
| In the meantime, you can use the [CLI](/docs/cli/) or [browser-based UI](/docs/sidecar/npx/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| In the meantime, you can use the [CLI](/docs/cli/) or [browser-based UI](/docs/sidecar/npx/). | |
| In the meantime, you can use the [CLI](/docs/cli/) with the browser-based UI. |
|
|
||
| The app automatically starts a Spotlight sidecar server: | ||
|
|
||
| - Default port: 8969 (configurable in settings) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it really?
| ### Changing the Port | ||
|
|
||
| If port 8969 is already in use: | ||
|
|
||
| 1. Open Spotlight **Settings** (via menu or gear icon) | ||
| 2. Change the **Sidecar Port** setting | ||
| 3. Restart the app | ||
| 4. Update your app's SDK configuration to match |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sounds like BS to me. We don't have any settings?
| ### UI Preferences | ||
|
|
||
| Customize the interface: | ||
|
|
||
| - **Theme**: Choose light or dark mode | ||
| - **Font size**: Adjust for readability | ||
| - **Panel layout**: Configure which panels are visible |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seriously?
| - ✅ You want machine-readable output | ||
|
|
||
| ### Use Browser UI When: | ||
| - ✅ You want overlay on your application |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overlay is long-dead. Feel like this is coming from agent rules file or something.
| ### Use Browser UI When: | ||
| - ✅ You want overlay on your application | ||
| - ✅ You're debugging frontend web apps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This entire section is no longer applicable.
| ### Use Browser UI When: | |
| - ✅ You want overlay on your application | |
| - ✅ You're debugging frontend web apps |
| **Via UI Config:** | ||
| See [Configuration reference](/docs/reference/configuration/#sidecar) for UI-side configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BS
| <CardGrid> | ||
| <LinkCard title="CLI Commands" href="/docs/cli/" description="Run with spotlight-sidecar command" /> | ||
| <LinkCard title="NPX" href="./npx/" description="Quick start with npx" /> | ||
| <LinkCard title="Webpack Plugin" href="./webpack/" description="Auto-start with webpack" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moar BS
| **Standalone Server:** | ||
| ```bash | ||
| # Using CLI | ||
| spotlight-sidecar server |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Death to spotlight-sidecar
|
|
||
| [Learn more about Desktop App →](/docs/desktop-app/) | ||
|
|
||
| ## Next Steps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Half of this file seems redundant as it covers the same things with spotlight CLI. Those parts should be merged.
BYK
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a huge improvement, thanks so much!
improved docs with Electron app + CLI + MCP