diff options
| author | Taylor Mullen <[email protected]> | 2025-05-16 22:41:57 -0700 |
|---|---|---|
| committer | N. Taylor Mullen <[email protected]> | 2025-05-16 22:43:50 -0700 |
| commit | 58e02240612e0a0eddc1427a795f5003ee2b3d07 (patch) | |
| tree | ebaece1d404a1b1c73ee162f3e46494366231db4 /packages/server/src | |
| parent | e486d84d6a627ce2563ac2313d55c46a663d4bc0 (diff) | |
Refactor: Use String.prototype.replaceAll() and update TS lib
- Replaces the custom `replaceAll` implementation in `packages/server/src/tools/edit.ts` with the standard `String.prototype.replaceAll()`.
- Updates `packages/server/tsconfig.json` to include `ES2021` in the `lib` compiler options to ensure TypeScript recognizes this method. This aligns with the project's Node.js version requirements \(Node.js 16.x+\).
Fixes https://github.com/google-gemini/gemini-cli/issues/7
Diffstat (limited to 'packages/server/src')
| -rw-r--r-- | packages/server/src/tools/edit.ts | 19 |
1 files changed, 2 insertions, 17 deletions
diff --git a/packages/server/src/tools/edit.ts b/packages/server/src/tools/edit.ts index 296c31ed..f7c911ec 100644 --- a/packages/server/src/tools/edit.ts +++ b/packages/server/src/tools/edit.ts @@ -205,8 +205,7 @@ Expectation for parameters: }; } else { // Successful edit calculation - newContent = this.replaceAll( - currentContent, + newContent = currentContent.replaceAll( params.old_string, params.new_string, ); @@ -274,8 +273,7 @@ Expectation for parameters: if (correctedOccurrences === 0 || correctedOccurrences !== 1) { return false; } - newContent = this.replaceAll( - currentContent, + newContent = currentContent.replaceAll( params.old_string, params.new_string, ); @@ -393,19 +391,6 @@ Expectation for parameters: } /** - * Replaces all occurrences of a substring in a string - */ - private replaceAll(str: string, find: string, replace: string): string { - if (find === '') { - return str; - } - // Use RegExp with global flag for true replacement of all instances - // Escape special regex characters in the find string - const escapedFind = find.replace(/[.*+?^${}()|[\]\\]/g, '\\$&'); - return str.replace(new RegExp(escapedFind, 'g'), replace); - } - - /** * Creates parent directories if they don't exist */ private ensureParentDirectoriesExist(filePath: string): void { |
