Skip to content

docs(nodes): add co-located doc.md for llm_kimi#1232

Closed
meetp06 wants to merge 1 commit into
rocketride-org:developfrom
meetp06:docs/RR-1107-kimi-node-doc
Closed

docs(nodes): add co-located doc.md for llm_kimi#1232
meetp06 wants to merge 1 commit into
rocketride-org:developfrom
meetp06:docs/RR-1107-kimi-node-doc

Conversation

@meetp06

@meetp06 meetp06 commented Jun 11, 2026

Copy link
Copy Markdown

Summary

The Kimi (Moonshot) node (llm_kimi) landed in #1112, but it was the only LLM node (1 of 20) missing its co-located doc.md — the docs-site page gathered into the site by builder docs:build. Every sibling (llm_deepseek, llm_minimax, llm_qwen, …) ships one; Kimi only had an in-node README.md. This adds the missing page so Kimi appears in the published docs.

What's included

  • New nodes/src/nodes/llm_kimi/doc.md following the llm_deepseek / llm_minimax pattern:
    • Hand-written prose: What it does, Lanes, Configuration, Profiles, Upstream docs — sourced from the node's README.md and services.json, plus a note that temperature is model-aware (K2 reasoning models require temperature=1, per feat(nodes): add Kimi (Moonshot) LLM node + model-sync registration #1112).
    • The ## Reference block produced by nodes:docs-generate from services.json (not hand-edited, per the co-located-docs rule).

Verification

  • nodes:docs-generate is idempotent on the file (re-run reports updated 0); the generated block matches services.json.
  • Section structure is identical to the shipping llm_deepseek/doc.md.
  • MDX-safe: no stray < / { outside inline code.

Closes #1107. Completes the documentation requirement from that issue, whose node implementation merged in #1112.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Added comprehensive guide for the Moonshot AI Kimi integration node, including configuration setup with API credentials and server base URL, supported model profiles, context window specifications, and temperature constraints for Kimi K2 reasoning models. Includes full reference documentation for pipeline integration.

The Kimi (Moonshot) node shipped in rocketride-org#1112 but was the only LLM node
(1 of 20) missing its co-located doc.md — the docs-site page gathered
by `builder docs:build`. This adds it, matching the llm_deepseek /
llm_minimax pattern: hand-written prose plus the machine-generated
`## Reference` block produced by `nodes:docs-generate` from services.json.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions github-actions Bot added docs Documentation module:nodes Python pipeline nodes labels Jun 11, 2026
@coderabbitai

coderabbitai Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 0c6bdad6-bf62-40b8-952b-327fd652f081

📥 Commits

Reviewing files that changed from the base of the PR and between 1995c64 and 3105f45.

📒 Files selected for processing (1)
  • nodes/src/nodes/llm_kimi/doc.md

📝 Walkthrough

Walkthrough

Added documentation for the llm_kimi:// node that describes its integration with Moonshot AI's Kimi models, including configuration fields, supported model profiles, context windows, and temperature constraints specific to Kimi K2 reasoning models.

Changes

Kimi Node Documentation

Layer / File(s) Summary
Kimi node overview and reference documentation
nodes/src/nodes/llm_kimi/doc.md
Introduces the llm_kimi:// node with metadata, purpose, questions → answers lane, configuration details (model, API key, base URL), supported profiles (Kimi K1, K1.5, K2 with context sizes), temperature constraint rule for K2 reasoning (forcing temperature=1), upstream Moonshot API docs link, and a reference section documenting protocol type, capabilities, data lanes, profiles, and configuration sections.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

docs, module:nodes

Suggested reviewers

  • jmaionchi
  • stepmikhaylov
  • Rod-Christensen

Poem

🐰 A Kimi doc hops into the night,
Moonshot's reasoning models shining bright,
K2's warmth stays fixed at one,
Questions meet answers—the deed is done!
No images here, just thoughtful text,
What feature shall we add next?

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'docs(nodes): add co-located doc.md for llm_kimi' is clear, specific, and directly reflects the main change—adding documentation for the Kimi node.
Linked Issues check ✅ Passed The PR adds documentation for the Kimi node as required by issue #1107, completing the documentation objectives specified for the node implementation.
Out of Scope Changes check ✅ Passed The PR contains only the co-located doc.md file for llm_kimi, which is directly in scope for issue #1107's documentation requirement.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@github-actions

Copy link
Copy Markdown
🤖 Internal: Discord sync marker

Auto-managed by the Discord notification workflow. Stores the linked Discord message ID. Do not edit or delete.

@meetp06

meetp06 commented Jun 11, 2026

Copy link
Copy Markdown
Author

Superseded by #1242, which folds in the llm_kimi doc.md migration as part of the broader #1239 work (enriched generated block + per-node README retirement). Closing in favor of that PR.

@meetp06 meetp06 closed this Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation module:nodes Python pipeline nodes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Kimi Moonshot node

1 participant