Repository navigation
docs: add Quick Start section with npm installation and AI instructions - #20
Conversation
- Add Quick Start section at top of README with: - npm package installation (@eaglebyte/skylight-mcp) as recommended option - From source installation as secondary option - Both mcp.json and Claude Code formats - Add copy-pasteable AI instructions snippet for system prompts - Update package name to @eaglebyte/skylight-mcp - Add files array for npm publishing - Bump version to 1.1.0 - Remove outdated Known Limitations section - Remove redundant Usage with Claude Desktop section (now in Quick Start) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
📝 WalkthroughWalkthroughDocumentation is restructured from prerequisites to a comprehensive quick-start guide with installation options, environment configuration, and AI interaction instructions. Package metadata is updated: name changes to Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
Pre-merge checks✅ Passed checks (3 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
README.mdpackage.json
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-12-29T21:36:49.327Z
Learnt from: CR
Repo: TheEagleByte/skylight-mcp PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-29T21:36:49.327Z
Learning: Set timezone to America/New_York by default, configurable via SKYLIGHT_TIMEZONE environment variable
Applied to files:
README.md
🔇 Additional comments (7)
README.md (6)
45-67: Clear from-source installation instructions.The from-source installation option is well-documented with proper build steps and configuration examples. The placeholder path clearly indicates where users need to substitute their actual installation path.
120-130: Configuration table accurately documents defaults and requirements.The configuration table is well-structured and correctly documents
SKYLIGHT_TIMEZONEdefaulting toAmerica/New_York, which aligns with the project's established configuration. The distinction between required and optional variables is clear.Based on learnings, the timezone default is correctly documented.
21-35: JSON configuration examples are syntactically correct.The mcp.json examples for both installation options are properly formatted with valid JSON syntax and appropriate structure for MCP server configuration.
Also applies to: 53-66
14-80: Excellent Quick Start documentation with clear installation paths.The Quick Start section effectively serves its purpose by providing:
- Multiple installation options (npm and from-source)
- Copy-pasteable configuration for different environments
- Clear AI integration instructions
- Proper progression from setup to usage
This is a significant improvement in user experience for getting started with the MCP server.
18-43: Package is already published on npm—no action needed.The @eaglebyte/skylight-mcp package is published on npm (version 1.0.0 as of 2025-12-29) and users can use
npx @eaglebyte/skylight-mcpimmediately as shown in the README.Likely an incorrect or invalid review comment.
37-43: Fix Claude Code MCP command syntax: use--envnot-eand add--separator.The
claude mcp addcommand uses--env KEY=valueflags (not-e) and requires--to separate CLI flags from the MCP server command. The correct syntax is:claude mcp add skylight --env SKYLIGHT_EMAIL=your_email@example.com \ --env SKYLIGHT_PASSWORD=your_password \ --env SKYLIGHT_FRAME_ID=your_frame_id \ -- npx @eaglebyte/skylight-mcp⛔ Skipped due to learnings
Learnt from: CR Repo: TheEagleByte/skylight-mcp PR: 0 File: CLAUDE.md:0-0 Timestamp: 2025-12-29T21:36:49.327Z Learning: Require SKYLIGHT_FRAME_ID environment variable as household identifier from API URLspackage.json (1)
10-14: The LICENSE file exists in the repository root, so no action is needed.
Summary
@eaglebyte/skylight-mcpand bump to v1.1.0Test plan
npx @eaglebyte/skylight-mcpafter publishing🤖 Generated with Claude Code
Summary by CodeRabbit
Documentation
Chores
✏️ Tip: You can customize this high-level summary in your review settings.