Skip to content

fix(cli): clarify prompt mode login guidance#38

Merged
liruifengv merged 1 commit into
MoonshotAI:mainfrom
Fengzdadi:codex/friendly-prompt-login-message
May 26, 2026
Merged

fix(cli): clarify prompt mode login guidance#38
liruifengv merged 1 commit into
MoonshotAI:mainfrom
Fengzdadi:codex/friendly-prompt-login-message

Conversation

@Fengzdadi
Copy link
Copy Markdown
Contributor

Related Issue

No related issue. This came from trying kimi -p hello in a fresh configuration and seeing a prompt-mode error that only mentioned editing config.toml.

Problem

When prompt mode runs without a configured model, the current error tells users to set default_model in config.toml. That is technically correct, but it misses the normal first-run path for Kimi Code users: start the interactive CLI and run /login to provision a model before retrying prompt mode.

What changed

Updated the prompt-mode missing-model error to point users to kimi + /login first, while still mentioning default_model for manual configuration. The existing prompt-mode test now asserts the friendlier guidance, and a patch changeset records the CLI-visible wording change.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works.
  • Ran gen-changesets skill, or this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update.

Validation:

  • pnpm --filter @moonshot-ai/kimi-code exec vitest run test/cli/run-prompt.test.ts
  • pnpm -C apps/kimi-code run build
  • KIMI_CODE_HOME=/private/tmp/kimi-code-prompt-message-check node apps/kimi-code/dist/main.mjs -p hello

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 25, 2026

🦋 Changeset detected

Latest commit: 27c9eb7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@moonshot-ai/kimi-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Fengzdadi Fengzdadi marked this pull request as ready for review May 25, 2026 21:48
Copy link
Copy Markdown
Collaborator

@liruifengv liruifengv left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@liruifengv liruifengv merged commit e9e4a48 into MoonshotAI:main May 26, 2026
5 checks passed
@github-actions github-actions Bot mentioned this pull request May 26, 2026
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