Skip to content

add ask mode#38

Merged
sakihet merged 14 commits into
mainfrom
ai-ask
Nov 11, 2025
Merged

add ask mode#38
sakihet merged 14 commits into
mainfrom
ai-ask

Conversation

@sakihet
Copy link
Copy Markdown
Owner

@sakihet sakihet commented Nov 8, 2025

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 8, 2025

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

Project Deployment Preview Comments Updated (UTC)
edith Ready Ready Preview Comment Nov 11, 2025 1:05am

Copy link
Copy Markdown

Copilot AI left a comment

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 adds a new "ask" mode to the AI panel, enabling users to ask questions about their notes with context-aware responses. The changes introduce an interactive chat interface where users can submit questions and receive answers based on their note content.

Key changes:

  • Added new 'ask' mode as an AI interaction option
  • Implemented chat-style interface for question-and-answer interactions
  • Added support for providing note context to the language model

Reviewed Changes

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

File Description
src/types/ai.ts Added 'ask' to AiMode type and new expectedOutputs configuration option
src/composables/useBuiltInAi.ts Implemented promptNoteContext function and configured expected output formats
src/components/TheAiPanel.vue Added ask mode UI with chat interface and integrated note context handling

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

Comment thread src/types/ai.ts Outdated
temperature?: number
maxTemperature?: number
initialPrompts: Prompt[]
expectedOutputs?: Array<{ type: string, langueages: string[] }>
Copy link

Copilot AI Nov 8, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'langueages' to 'languages'.

Suggested change
expectedOutputs?: Array<{ type: string, langueages: string[] }>
expectedOutputs?: Array<{ type: string, languages: string[] }>

Copilot uses AI. Check for mistakes.
Comment thread src/composables/useBuiltInAi.ts Outdated
}
],
expectedOutputs: [{
type: 'text', langueages: ['en', 'ja']
Copy link

Copilot AI Nov 8, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'langueages' to 'languages'.

Suggested change
type: 'text', langueages: ['en', 'ja']
type: 'text', languages: ['en', 'ja']

Copilot uses AI. Check for mistakes.
Comment thread src/components/TheAiPanel.vue Outdated
// @ts-ignore
await promptNoteContext(props.editor?.getText() || '')
// @ts-ignore
const content = props.editor?.getText() || ''
Copy link

Copilot AI Nov 8, 2025

Choose a reason for hiding this comment

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

The variable 'content' is declared but never used. This should be removed to avoid confusion.

Suggested change
const content = props.editor?.getText() || ''

Copilot uses AI. Check for mistakes.
@sakihet sakihet linked an issue Nov 9, 2025 that may be closed by this pull request
@sakihet sakihet merged commit 3a32cba into main Nov 11, 2025
4 checks passed
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.

add ask mode

2 participants