Skip to content

docs: document dynamic port assignment and remove hardcoded 5272 references#447

Merged
MaanavD merged 1 commit intomicrosoft:mainfrom
MaanavD:docs/document-dynamic-port
Feb 19, 2026
Merged

docs: document dynamic port assignment and remove hardcoded 5272 references#447
MaanavD merged 1 commit intomicrosoft:mainfrom
MaanavD:docs/document-dynamic-port

Conversation

@MaanavD
Copy link
Copy Markdown
Collaborator

@MaanavD MaanavD commented Feb 19, 2026

Summary

Foundry Local assigns a dynamic port each time the service starts, but several docs and examples hardcode localhost:5272, which misleads users into thinking the port is fixed.

This PR:

  • docs/copilot-sdk-integration.md: Replaces hardcoded localhost:5272 in the architecture diagram, Quick Start code comment, and BYOK reference table with dynamic port guidance
  • samples/js/copilot-sdk-foundry-local/README.md: Updates BYOK code comment to note the port is dynamically assigned
  • README.md: Adds an [!IMPORTANT] callout in the SDK integration section explaining that the port is dynamic and how to discover it (manager.endpoint or foundry service status)

Related to #424

…rences

Foundry Local assigns a dynamic port on each service start. This commit:
- Replaces hardcoded localhost:5272 in the architecture diagram and BYOK table
- Updates code comments to note the port is dynamically assigned
- Adds an IMPORTANT callout in the README SDK section about dynamic ports
- Documents 'foundry service status' as the CLI discovery method

Related to microsoft#424
@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 19, 2026

@MaanavD is attempting to deploy a commit to the MSFT-AIP Team on Vercel.

A member of the Team first needs to authorize it.

@MaanavD MaanavD merged commit 9515b22 into microsoft:main Feb 19, 2026
1 of 2 checks passed
@MaanavD MaanavD deleted the docs/document-dynamic-port branch February 19, 2026 19:56
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.

2 participants