Skip to content

[Ask AI 01] feat: switch docs assistant to Luna - #228

Open
raviships wants to merge 1 commit into
nextfrom
feat/ask-ai-luna
Open

[Ask AI 01] feat: switch docs assistant to Luna#228
raviships wants to merge 1 commit into
nextfrom
feat/ask-ai-luna

Conversation

@raviships

@raviships raviships commented Aug 22, 2026

Copy link
Copy Markdown
Collaborator

Change

  • Switch Ask AI from GPT-4.1 mini to GPT-5.6 Luna with reasoning disabled.
  • Tell the model to call provideLinks before it writes the final answer.
  • Execute provideLinks so the model can continue after the tool call.

Without the executor, Luna could stop at the link call and return no answer text.

Benchmark

The benchmark asked five documentation questions twice for each configuration. Every run used the same EdgeStore pages, tool schemas, and prompt. Quality is the percentage of question-specific checks passed. Cost uses estimated OpenAI token pricing.

Model Complete answers One provideLinks call Quality Median first event Median total time Estimated cost per request
GPT-4.1 mini 10/10 10/10 90% 1.24 s 9.48 s $0.006283
Luna, no reasoning 10/10 10/10 87.5% 1.33 s 6.36 s $0.001417
Luna, low reasoning 10/10 10/10 85% 1.44 s 7.49 s $0.001910

Luna with no reasoning cut estimated cost by 77% and median total time by 33%. Its quality score was 2.5 percentage points lower, and its first event arrived 92 ms later. Low reasoning cost more, took longer, and scored lower in this run.

Checks

  • Targeted Prettier and ESLint checks pass.
  • pnpm build passes.
  • pnpm --filter docs exec tsc --noEmit --pretty false passes after rebuilding the workspace packages.

@vercel

vercel Bot commented Aug 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
edgestore-docs Ready Ready Preview Aug 22, 2026 4:00am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
example-access-control Skipped Skipped Aug 22, 2026 4:00am
example-components Skipped Skipped Aug 22, 2026 4:00am

Request Review

@vercel
vercel Bot temporarily deployed to Preview – example-components August 22, 2026 03:47 Inactive
@vercel
vercel Bot temporarily deployed to Preview – example-access-control August 22, 2026 03:47 Inactive
@raviships raviships changed the title feat: switch docs assistant to Luna [Ask AI 01] feat: switch docs assistant to Luna Aug 22, 2026
@vercel
vercel Bot temporarily deployed to Preview – example-components August 22, 2026 03:59 Inactive
@vercel
vercel Bot temporarily deployed to Preview – example-access-control August 22, 2026 03:59 Inactive
@raviships
raviships marked this pull request as ready for review August 22, 2026 04:05

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6dfec000d9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

- If a feature doesn't exist in EdgeStore, say so clearly.

After providing your answer, use the "provideLinks" tool to share relevant documentation links with the user. Include links to the documentation pages you referenced in your answer.`;
Before writing your final answer, call the "provideLinks" tool exactly once with the documentation pages you referenced. After the tool returns, write a complete, non-empty answer to the user. Do not call "provideLinks" again, and do not treat the tool call as the answer.`;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Leave room for the required post-tool answer

When a question requires four sequential getDocs retrieval rounds, provideLinks becomes the fifth step; its new executor produces a tool result, after which stepCountIs(5) stops generation before Luna can write the complete answer now required by this prompt. Increase the step allowance or use a completion condition that reserves a step after provideLinks, otherwise sufficiently complex documentation questions can still return links with no answer text.

Useful? React with 👍 / 👎.

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.

1 participant