Skip to content

Conversation

@wnghdcjfe
Copy link
Collaborator

@wnghdcjfe wnghdcjfe commented Oct 10, 2025

related #215

It works pretty well
en vercion output code

describeAPI(
    HttpMethod.POST,
    "/signup",
    {
        summary: "Signup API",
        tag: "Auth",
        description: "Registers a user account with username and password.",
    },
    targetApp,
    (apiDoc) => {
        itDoc("Returns 201 when signup succeeds", async () => {
            await apiDoc
                .test()
                .prettyPrint()
                .req()
                .body({
                    username: field("Username", "newuser"),
                    password: field("Password", "P@ssw0rd123"),
                })
                .res()
                .status(HttpStatus.CREATED)
        })
  ..

console
1

edited docs
3
2

Summary by CodeRabbit

  • New Features

    • Generate command now supports language selection (-l/--lang): default output is Korean, with an option for English; the chosen language is shown during generation.
  • Documentation

    • Guides updated with usage examples and tips for both Korean (default) and English outputs.
  • Chores

    • Editor config simplified: removed global formatter/lint-on-save defaults and added per-language handling for Markdown/MDX to avoid undesired auto-fixes.

@coderabbitai
Copy link

coderabbitai bot commented Oct 10, 2025

Walkthrough

Adds a CLI language option (-l, --lang) to select output language (en or ko; default ko) and threads the choice through the generation flow by changing generateByLLM(appPath, envPath) → generateByLLM(appPath, envPath, isEn). Refactors prompt assembly in script/llm/prompt/index.ts to use a promptTemplate with explicit route-analysis and example sections for both languages. Adjusts route grouping initialization to explicit checks. Updates documentation (docs and ko i18n) and examples to describe the language option. Updates .vscode/settings.json to remove global Prettier/ESLint defaults and add per-language Markdown/MDX settings.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested labels

Documentation

Suggested reviewers

  • PENEKhun
  • json-choi

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title succinctly highlights the addition of an isEn flag to the LLM feature, directly reflecting the primary change to propagate language selection in the generate command. It is concise, avoids unnecessary detail or noise, and clearly informs a reviewer of the main enhancement introduced.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch issue/215

📜 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 586b3a6 and 4d9531c.

📒 Files selected for processing (1)
  • .vscode/settings.json (0 hunks)
💤 Files with no reviewable changes (1)
  • .vscode/settings.json

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.

@coderabbitai coderabbitai bot added the Documentation Something isn't working label Oct 10, 2025
@wnghdcjfe wnghdcjfe merged commit 71087ff into develop Oct 13, 2025
2 checks passed
PENEKhun pushed a commit that referenced this pull request Oct 26, 2025
* feat: LLM - Support for isEn Flag
related #215
* fix(vscode): restore original .settings.json after accidental change
@PENEKhun PENEKhun deleted the issue/215 branch November 3, 2025 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants