Conversation
There was a problem hiding this comment.
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
guideStepsinto a prompt generator with environment context. - Renamed and refactored
files-readandfiles-writetools for clearer batch operations. - Updated
main.mtsimports, switched model togemini-2.5-flash, and introducedverboseandthinkingConfigoptions.
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 towrite_filesorcreate_multiple_filesfor 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
chalkis used here but not imported at the top of this file, which will cause a runtime error. Addimport chalk from 'chalk';.
console.log(chalk.gray("\n" + nextQuestion + "\n"));
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.