docs: view port requirements and configuration

[MicBun]

compliment for: https://github.com/trufnetwork/truf-network/issues/1124

Summary by CodeRabbit

• Documentation

  • Added explicit Security Group/Firewall guidance with Required (SSH 22, P2P 6600, RPC 8484) and Optional (MCP 8000) ports.
  • Clarified port purposes and when each is needed, noting MCP as optional and P2P’s role in network health.
  • Updated AMI deployment and node operator guides with clearer, categorized port instructions.

• Style

  • Revised AMI runtime and console welcome messages to reflect the new port sections and optional MCP guidance, improving clarity and consistency.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Documentation and console message text updated to introduce explicit Security Group/port guidance: required ports 6600 (P2P) and 8484 (RPC), optional 8000 (MCP), plus SSH 22 in docs. AMI pipeline stack strings were revised to reflect these changes and mark MCP setup as optional. No code logic or APIs changed.

Changes

Cohort / File(s)	Summary
Infra: AMI pipeline messaging deployments/infra/stacks/ami_pipeline_stack.go	Replaced MCP setup block with a Security Group Ports section (6600 P2P, 8484 RPC); moved MCP to optional guidance; updated welcome/console output strings accordingly; no control-flow or API changes.
Docs: AMI deployment guide docs/ami-deployment-guide.md	Added explicit inbound rules tables and notes; defined required ports (22, 6600, 8484) and optional MCP (8000); clarified port purposes and wording.
Docs: Node operator guide docs/node-operator-guide.md	Reworded firewall/security group instructions; separated Required (6600, 8484) and Optional (8000 MCP) ports; added note on P2P connectivity behavior; commands unchanged.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• chore: enhance AMI with streamlined node configuration and setup #1182 — Also modifies AMI pipeline welcome/console messages and MCP/port instructions, overlapping the same guidance areas.

Suggested reviewers

• outerlook

Poem

I nudge the ports—6600, 8484—click!
SSH waves hello on 22, nice and quick.
MCP at 8000 waits, if you please,
Optional breezes on cloud-born seas.
A hop, a thump: docs align tight—
This rabbit sings, "Your nodes take flight!" 🐇✨

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch docsClarifyPorts

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3f76e92 and 8c5fc32.

📒 Files selected for processing (3)

• deployments/infra/stacks/ami_pipeline_stack.go (2 hunks)
• docs/ami-deployment-guide.md (1 hunks)
• docs/node-operator-guide.md (1 hunks)

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[holdex]

Time Submission Status

Member	Status	Time	Action	Last Update
MicBun	✅ Submitted	2h 30min	Update time	Oct 14, 2025, 5:39 AM