Skip to content

docs: fix typos, stale content, and port inconsistencies#2144

Open
mesutoezdil wants to merge 21 commits into
kagent-dev:mainfrom
mesutoezdil:docs/fix-typos-and-stale-content
Open

docs: fix typos, stale content, and port inconsistencies#2144
mesutoezdil wants to merge 21 commits into
kagent-dev:mainfrom
mesutoezdil:docs/fix-typos-and-stale-content

Conversation

@mesutoezdil

@mesutoezdil mesutoezdil commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Fix several doc issues found across the repo.

  • DEVELOPMENT.md: Go version v1.26.1+ to v1.26.3+ (matches go.mod), fix duplicate step numbers 1-5, fix UI port 8001 to 3000
  • Makefile: wrong Azure env var AZUREOPENAI_API_KEY to AZURE_OPENAI_API_KEY in check-api-key and helm-install-provider
  • helm/README.md: wrong Azure env var names (AZUREOPENAI_API_KEY, AZURE_API_KEY) to AZURE_OPENAI_API_KEY, fix copy-paste placeholder in Azure helm install example
  • examples/substrate-openclaw/README.md: remove stale runtime: substrate field (not in CRD), fix UI port 8001 to 3000
  • docs/substrate-agentharness-lifecycle.md: remove draft PR note "This branch should use...", update step 5 to reflect that actors are created on demand by the HTTP gateway, not the controller
  • ui/README.md: kagents ui to kagent ui
  • python/README.md: replace hardcoded OpenAI API key prerequisite with provider-agnostic wording

Copilot AI review requested due to automatic review settings July 3, 2026 15:47
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jul 3, 2026
@mesutoezdil
mesutoezdil force-pushed the docs/fix-typos-and-stale-content branch 2 times, most recently from 2854a56 to 107ec3c Compare July 3, 2026 15:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates repository documentation to correct typos and stale instructions, and to align examples/commands (notably UI port-forwarding and provider configuration) across docs.

Changes:

  • Standardizes UI access instructions (port-forward to localhost:3000) and removes stale CRD fields in examples.
  • Updates provider/key documentation (e.g., Azure OpenAI variable naming, provider-agnostic Python prereqs).
  • Cleans up and modernizes lifecycle/development docs (step numbering, removed draft notes, corrected behavior description).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
ui/README.md Fixes CLI command name in the UI README header.
python/README.md Makes prerequisites provider-agnostic (not OpenAI-only).
helm/README.md Updates Azure OpenAI examples/env var naming and placeholders.
examples/substrate-openclaw/README.md Removes stale runtime: substrate field and aligns UI port-forward/gateway URL examples.
docs/substrate-agentharness-lifecycle.md Removes draft note and updates reconcile step to reflect actor creation behavior.
DEVELOPMENT.md Updates Go version, fixes step numbering, and aligns UI port-forward instructions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread helm/README.md Outdated
Comment thread helm/README.md Outdated
Comment thread docs/substrate-agentharness-lifecycle.md Outdated
@mesutoezdil
mesutoezdil force-pushed the docs/fix-typos-and-stale-content branch from be3344c to 93f7d07 Compare July 3, 2026 15:54
Comment thread examples/substrate-openclaw/README.md Outdated
Comment thread examples/substrate-openclaw/README.md
Comment thread helm/README.md Outdated
@mesutoezdil
mesutoezdil requested a review from a team as a code owner July 7, 2026 16:03
EItanya
EItanya previously approved these changes Jul 7, 2026
@EItanya
EItanya enabled auto-merge (squash) July 7, 2026 18:43
auto-merge was automatically disabled July 9, 2026 09:36

Head branch was pushed to by a user without write access

@mesutoezdil
mesutoezdil force-pushed the docs/fix-typos-and-stale-content branch 9 times, most recently from aa303ee to 9f9f762 Compare July 13, 2026 10:20
@mesutoezdil
mesutoezdil force-pushed the docs/fix-typos-and-stale-content branch 3 times, most recently from f82363b to 0e1d7e6 Compare July 15, 2026 19:02
Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
@mesutoezdil
mesutoezdil force-pushed the docs/fix-typos-and-stale-content branch from 0e1d7e6 to 8352358 Compare July 15, 2026 20:10
@github-actions github-actions Bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Jul 21, 2026
peterj pushed a commit that referenced this pull request Jul 22, 2026
The Makefile checked and passed AZUREOPENAI_API_KEY while the Go and
Python code read AZURE_OPENAI_API_KEY, so the helm-install targets
silently passed an empty Azure key. This renames the Makefile variable
and the two helm README export examples to match the code. Split out of
#2144 since it changes Makefile behavior rather than docs.

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
…ale-content

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
…ale-content

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
…ale-content

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
…ale-content

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
…ale-content

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
…ale-content

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
…ale-content

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
…ale-content

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
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.

3 participants