jcarr
/
gemini-cli
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

general rules should apply to whole project (#159) 

* general rules should apply to whole project

* Merge remote-tracking branch 'origin/main' into fix_eslint

* lint fixes under server package
This commit is contained in:
Olcan 2025-04-24 15:42:18 -07:00 committed by GitHub
parent 0510d06ecf
commit 133f39494e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
4 changed files with 6 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
eslint.config.js
View File

@ -89,7 +89,7 @@ export default tseslint.config(
}, },
{ {
// General overrides and rules for the project (TS/TSX files) // General overrides and rules for the project (TS/TSX files)
files: ['packages/cli/src/**/*.{ts,tsx}'], // Target only TS/TSX in the cli package files: ['packages/*/src/**/*.{ts,tsx}'], // Target only TS/TSX in the cli package
languageOptions: { languageOptions: {
globals: { globals: {
...globals.node, ...globals.node,

4
packages/server/src/core/client.ts
View File

@ -68,7 +68,7 @@ export class GeminiClient {
config: { config: {
systemInstruction: getCoreSystemPrompt(), systemInstruction: getCoreSystemPrompt(),
...this.generateContentConfig, ...this.generateContentConfig,
tools: tools, tools,
}, },
history: [ history: [
{ {
@ -111,7 +111,7 @@ export class GeminiClient {
// What do we do when we have both function responses and confirmations? // What do we do when we have both function responses and confirmations?
const fnResponses = turn.getFunctionResponses(); const fnResponses = turn.getFunctionResponses();
if (fnResponses.length == 0) { if (fnResponses.length === 0) {
break; // user's turn to respond break; // user's turn to respond
} }
request = fnResponses; request = fnResponses;

5
packages/server/src/tools/ls.ts
View File

@ -257,10 +257,7 @@ export class LSTool extends BaseTool<LSToolParams, ToolResult> {
// Create formatted content for LLM // Create formatted content for LLM
const directoryContent = entries const directoryContent = entries
.map((entry) => { .map((entry) => `${entry.isDirectory ? '[DIR] ' : ''}${entry.name}`)
// More concise format for LLM
return `${entry.isDirectory ? '[DIR] ' : ''}${entry.name}`;
})
.join('\n'); .join('\n');
return { return {

4
packages/server/src/tools/read-many-files.ts
View File

@ -104,7 +104,7 @@ const DEFAULT_EXCLUDES: string[] = [
// Default values for encoding and separator format // Default values for encoding and separator format
const DEFAULT_ENCODING: BufferEncoding = 'utf-8'; const DEFAULT_ENCODING: BufferEncoding = 'utf-8';
const DEFAULT_OUTPUT_SEPARATOR_FORMAT: string = '--- {filePath} ---'; const DEFAULT_OUTPUT_SEPARATOR_FORMAT = '--- {filePath} ---';
/** /**
* Tool implementation for finding and reading multiple text files from the local filesystem * Tool implementation for finding and reading multiple text files from the local filesystem
@ -257,7 +257,7 @@ Default excludes apply to common non-text files and large dependency directories
const toolBaseDir = this.targetDir; const toolBaseDir = this.targetDir;
const filesToConsider = new Set<string>(); const filesToConsider = new Set<string>();
const skippedFiles: { path: string; reason: string }[] = []; const skippedFiles: Array<{ path: string; reason: string }> = [];
const processedFilesRelativePaths: string[] = []; const processedFilesRelativePaths: string[] = [];
let concatenatedContent = ''; let concatenatedContent = '';