Skip to content

fix(standalone): print server URL on startup#205

Open
WesleyMFrederick wants to merge 1 commit into
matt1398:mainfrom
WesleyMFrederick:fix/standalone-print-url
Open

fix(standalone): print server URL on startup#205
WesleyMFrederick wants to merge 1 commit into
matt1398:mainfrom
WesleyMFrederick:fix/standalone-print-url

Conversation

@WesleyMFrederick
Copy link
Copy Markdown
Contributor

@WesleyMFrederick WesleyMFrederick commented May 17, 2026

Summary

%% Last Modified: 05/17/26 14:03:57 %%

  • logger.info() is silenced by default — Logger.level defaults to WARN (level 2), so info calls (level 1) never fire
  • The URL log line existed in standalone.ts but was never visible to users
  • Replaces the silenced logger.info with a direct console.log that always prints
  • Converts 0.0.0.0localhost for a user-friendly display URL

Before: starting pnpm standalone / node dist-standalone/index.cjs showed no URL
After: prints \n claude-devtools running at http://localhost:3456\n

Test plan

%% Last Modified: 05/17/26 14:03:57 %%

  • Run pnpm standalone — confirm URL prints on startup
  • Run with PORT=8080 — confirm http://localhost:8080 prints
  • Run with HOST=192.168.1.1 — confirm actual host prints (not rewritten to localhost)
  • Run pnpm standalone:build && node dist-standalone/index.cjs — confirm built artifact also prints URL

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Bug Fixes

    • Server startup now always prints the accessible HTTP URL to stdout and normalizes host display (local addresses shown as "localhost" and IPv6 formatted) for easy copying.
  • Documentation

    • Added a documentation sync manifest defining repo-to-docs mappings.
    • Added architecture principles, design docs, and workflow documentation entries.

Review Change Stack

@coderabbitai coderabbitai Bot added the bug Something isn't working label May 17, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

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.

Comment thread src/main/standalone.ts Outdated
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 17, 2026

Note

Reviews paused

It 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 reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7ca765d6-1e2b-4fa0-b839-0bfc51db0696

📥 Commits

Reviewing files that changed from the base of the PR and between 1430062 and b1192e5.

📒 Files selected for processing (4)
  • architecture-principles
  • design-docs
  • documentation-sync-manifest.json
  • workflows
✅ Files skipped from review due to trivial changes (3)
  • architecture-principles
  • workflows
  • design-docs

📝 Walkthrough

Walkthrough

The standalone entry point now prints the server URL to stdout after the HTTP server starts, mapping 0.0.0.0/:: to localhost and bracket-wrapping IPv6 hosts; additionally, a documentation-sync-manifest.json plus three local-path files (architecture-principles, design-docs, workflows) were added.

Changes

Direct Server URL Output

Layer / File(s) Summary
Server startup URL logging
src/main/standalone.ts
After httpServer.start, compute a display host (map 0.0.0.0/::localhost, bracket-wrap IPv6), then print http://{displayHost}:{port} via console.log, replacing the previous browser-guidance logger message.

Documentation sync manifest and local paths

Layer / File(s) Summary
Documentation manifest and local path entries
documentation-sync-manifest.json, architecture-principles, design-docs, workflows
Add documentation-sync-manifest.json with version, consumer_repo, docs_repo_env, docs_repo_default, and mappings for architecture-principles/, design-docs/, and workflows/. Add architecture-principles, design-docs, and workflows files containing local absolute path lines referenced by the manifest.

Suggested labels

documentation

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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>
@WesleyMFrederick WesleyMFrederick force-pushed the fix/standalone-print-url branch from f398f3b to c1269ab Compare May 17, 2026 21:09
@coderabbitai coderabbitai Bot added documentation Improvements or additions to documentation and removed bug Something isn't working labels May 17, 2026
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

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

📥 Commits

Reviewing files that changed from the base of the PR and between c1269ab and 12867d3.

📒 Files selected for processing (3)
  • architecture-principles
  • documentation-sync-manifest.json
  • workflows
✅ Files skipped from review due to trivial changes (1)
  • workflows

Comment thread architecture-principles Outdated
@WesleyMFrederick WesleyMFrederick force-pushed the fix/standalone-print-url branch from 12867d3 to 1430062 Compare May 17, 2026 22:31
@coderabbitai coderabbitai Bot added bug Something isn't working and removed documentation Improvements or additions to documentation labels May 17, 2026
@WesleyMFrederick WesleyMFrederick force-pushed the fix/standalone-print-url branch from 1430062 to b1192e5 Compare May 17, 2026 22:33
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

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

📥 Commits

Reviewing files that changed from the base of the PR and between 12867d3 and 1430062.

📒 Files selected for processing (4)
  • architecture-principles
  • design-docs
  • documentation-sync-manifest.json
  • workflows
✅ 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

Comment thread design-docs Outdated
@coderabbitai coderabbitai Bot added documentation Improvements or additions to documentation and removed bug Something isn't working labels May 17, 2026
@WesleyMFrederick WesleyMFrederick force-pushed the fix/standalone-print-url branch from b1192e5 to c1269ab Compare May 17, 2026 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant