Skip to content

Conversation

@tiye
Copy link
Member

@tiye tiye commented Jun 23, 2025

No description provided.

@tiye tiye requested review from a team and Copilot June 23, 2025 19:07
Copy link

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 refines the command-line assistant’s prompt, reorganizes file read/write tools, updates the Google Search model, and adds verbose/thinking configurations in the main entry.

  • Expanded and restructured guideSteps into a prompt generator with environment context.
  • Renamed and refactored files-read and files-write tools for clearer batch operations.
  • Updated main.mts imports, switched model to gemini-2.5-flash, and introduced verbose and thinkingConfig options.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/tools/guide-steps.mts Added toolContextPrompt, imported execSync,os, and expanded instructions.
src/tools/google-search.mts Changed AI model to "gemini-2.5-flash".
src/tools/files-write.mts Converted read tool to a write tool, updated declaration and parameters.
src/tools/files-read.mts Recreated read tool, corrected order of import/export, added toolFn.
src/main.mts Fixed imports, toggled verbose logging, added thinkingConfig.
Comments suppressed due to low confidence (3)

src/tools/files-write.mts:66

  • [nitpick] The tool name uses camelCase while other tools use snake_case (e.g., read_files). Consider renaming to write_files or create_multiple_files for consistency.
    name: "createMultipleFiles",

src/tools/files-write.mts:68

  • [nitpick] The description could be clearer. For example: "Create multiple files from the entries array in the current working directory."
      "create multiple files called entries, on the current working directory",

src/main.mts:134

  • chalk is used here but not imported at the top of this file, which will cause a runtime error. Add import chalk from 'chalk';.
        console.log(chalk.gray("\n" + nextQuestion + "\n"));

@NoEgAm NoEgAm merged commit 3edda01 into main Jul 29, 2025
1 check passed
@NoEgAm NoEgAm deleted the thinking branch July 29, 2025 11:33
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.

3 participants