fix(standalone): print server URL on startup#205
Conversation
There was a problem hiding this comment.
Code Review
This pull request modifies the standalone server startup logic to ensure the connection URL is always printed to the console using console.log, bypassing the logger's level restrictions. It also includes logic to display localhost when the host is set to 0.0.0.0. Feedback was provided regarding the handling of IPv6 addresses, noting that they require square brackets to form valid URLs and suggesting that the IPv6 'any' address (::) should also be mapped to localhost for consistency.
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
✅ Files skipped from review due to trivial changes (3)
📝 WalkthroughWalkthroughThe standalone entry point now prints the server URL to stdout after the HTTP server starts, mapping ChangesDirect Server URL Output
Documentation sync manifest and local paths
Suggested labels
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
logger.info() is silenced by default (WARN level) so the existing URL log never appeared. Bypass the logger with console.log and convert any-address hosts (0.0.0.0, ::) to localhost for user-friendly output. IPv6 addresses are wrapped in brackets per RFC 2732. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
f398f3b to
c1269ab
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@architecture-principles`:
- Line 1: Remove the committed absolute-path symlinks (architecture-principles
and workflows) and stop referencing them in the repo; instead update repository
setup to resolve docs via the existing documentation-sync-manifest.json and the
DOCS_REPO_PATH environment variable (default ../0-documents/markdown-documents).
Specifically: delete the symlink files from git and commit the removals, ensure
any code or scripts that previously accessed those symlinks now read
DOCS_REPO_PATH (or fallback to the manifest default), and add/update
README/setup instructions to document configuring DOCS_REPO_PATH (or convert the
external docs into a git submodule if you want the docs tracked inside the
project). Ensure no absolute /Users/... symlinks remain committed.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 4dee41e2-35ae-42e3-ac03-3d30421bb6fd
📒 Files selected for processing (3)
architecture-principlesdocumentation-sync-manifest.jsonworkflows
✅ Files skipped from review due to trivial changes (1)
- workflows
12867d3 to
1430062
Compare
1430062 to
b1192e5
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@design-docs`:
- Line 1: Remove the committed absolute-path symlinks (design-docs,
architecture-principles, workflows) from the repository and replace them with
portable alternatives: add those symlink targets or their containing directories
to .gitignore, create template placeholder files (e.g., design-docs.example,
architecture-principles.example, workflows.example) with placeholder paths,
update README to document how each developer should create local symlinks or
configure paths, and ensure references use the existing
documentation-sync-manifest.json approach (relative paths / environment
variables) rather than hard-coded absolute paths.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 391763e2-036f-4710-82c5-e32b37581b82
📒 Files selected for processing (4)
architecture-principlesdesign-docsdocumentation-sync-manifest.jsonworkflows
✅ Files skipped from review due to trivial changes (1)
- architecture-principles
🚧 Files skipped from review as they are similar to previous changes (2)
- workflows
- documentation-sync-manifest.json
b1192e5 to
c1269ab
Compare
Summary
%% Last Modified: 05/17/26 14:03:57 %%
logger.info()is silenced by default —Logger.leveldefaults toWARN(level 2), soinfocalls (level 1) never firestandalone.tsbut was never visible to userslogger.infowith a directconsole.logthat always prints0.0.0.0→localhostfor a user-friendly display URLBefore: starting
pnpm standalone/node dist-standalone/index.cjsshowed no URLAfter: prints
\n claude-devtools running at http://localhost:3456\nTest plan
%% Last Modified: 05/17/26 14:03:57 %%
pnpm standalone— confirm URL prints on startupPORT=8080— confirmhttp://localhost:8080printsHOST=192.168.1.1— confirm actual host prints (not rewritten to localhost)pnpm standalone:build && node dist-standalone/index.cjs— confirm built artifact also prints URL🤖 Generated with Claude Code
Summary by CodeRabbit
Bug Fixes
Documentation