Skip to content

Conversation

@vucat12
Copy link

@vucat12 vucat12 commented Dec 20, 2025

🎯 Changes

Update the example URL in testing/panel/README.md to use port 3010 instead of 3001.

βœ… Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested this code locally with pnpm run test:pr.

πŸš€ Release Impact

  • This change affects published code, and I have generated a changeset.
  • This change is docs/CI/dev-only (no release).

πŸ“‹ Details & Motivation

The documentation previously referenced port 3001, but the application default has changed to 3010. This update ensures new users can copy-paste the example command and successfully connect to the panel without connection errors.

Summary by CodeRabbit

  • Documentation
    • Updated example usage documentation with the correct local development server port.

✏️ Tip: You can customize this high-level summary in your review settings.

Copilot AI review requested due to automatic review settings December 20, 2025 18:22
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 20, 2025

Walkthrough

A port number in an example URL is updated from 3001 to 3010 in the testing/panel README documentation.

Changes

Cohort / File(s) Summary
Documentation
testing/panel/README.md
Updated example usage URL port from 3001 to 3010

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

No functional changes; straightforward documentation update.

Poem

🐰 A port change hops along the way,
From 3001 to 3010's play,
The docs now point where tests should go,
A little tweak, a gentle flow! 🎯

Pre-merge checks and finishing touches

βœ… Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check βœ… Passed The PR description follows the required template with all sections included: Changes, Checklist, Release Impact, and additional Details & Motivation explaining the update rationale.
Title check βœ… Passed The title clearly and accurately summarizes the main change: updating an example URL port from localhost:3001 to localhost:3010 in the testing/panel/README.md file.
✨ Finishing touches
  • πŸ“ Generate docstrings
πŸ§ͺ Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects the port number in the README documentation to match the actual configuration used by the development server. The README was incorrectly referencing port 3001, while the actual dev script in package.json uses port 3010.

  • Updated localhost port reference from 3001 to 3010 in the usage instructions

πŸ’‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@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

πŸ“œ Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between aaafd69 and 18d98e7.

πŸ“’ Files selected for processing (1)
  • testing/panel/README.md (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
πŸ“š Learning: 2025-12-13T17:09:09.794Z
Learnt from: CR
Repo: TanStack/ai PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-13T17:09:09.794Z
Learning: Applies to examples/** : Examples are not built by Nx and should be run independently from their directories with `pnpm dev` or `pnpm install && pnpm dev`

Applied to files:

  • testing/panel/README.md
πŸͺ› markdownlint-cli2 (0.18.1)
testing/panel/README.md

21-21: Bare URL used

(MD034, no-bare-urls)

```

Then open http://localhost:3001
Then open http://localhost:3010
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟑 Minor

🧩 Analysis chain

🏁 Script executed:

# Search for port configuration in testing/panel directory
find testing/panel -type f \( -name "*.json" -o -name "*.js" -o -name "*.ts" -o -name "*.config.*" -o -name "vite.config.*" -o -name "next.config.*" \) 2>/dev/null | head -20

Repository: TanStack/ai

Length of output: 820


🏁 Script executed:

# Check package.json for dev script and port configuration
cat testing/panel/package.json

Repository: TanStack/ai

Length of output: 1378


🏁 Script executed:

# Check app.config.ts for any additional port configuration
cat testing/panel/app.config.ts

Repository: TanStack/ai

Length of output: 318


Wrap the bare URL in angle brackets to comply with markdown linting standards.

The bare URL violates the MD034 (no-bare-urls) markdown linting rule. Without angle brackets, the URL isn't converted into a link in many markdown parsers. Wrap it in angle brackets:

-Then open http://localhost:3010
+Then open <http://localhost:3010>
πŸ“ Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
Then open http://localhost:3010
Then open <http://localhost:3010>
🧰 Tools
πŸͺ› markdownlint-cli2 (0.18.1)

21-21: Bare URL used

(MD034, no-bare-urls)

πŸ€– Prompt for AI Agents
In testing/panel/README.md around line 21 the bare URL "http://localhost:3010"
violates MD034 (no-bare-urls); update the line to wrap the URL in angle brackets
so the markdown linter treats it as a link (replace the bare URL with
<http://localhost:3010>).

@vucat12 vucat12 changed the title Update README.md Update example URL port in testing/panel/README.md Dec 21, 2025
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.

1 participant