Fix property naming conflict with parent class

- Rename thinkingOpenTag to _thinkingOpenTag (private)
- Rename thinkingCloseTag to _thinkingCloseTag (private)
- Avoids conflicts with properties that may exist in parent classes

Co-authored-by: AyRickk <[email protected]>

Author: Copilot

core/llm/llms/Vllm.ts

@@ -66,8 +66,8 @@ class Vllm extends OpenAI {
   static providerName = "vllm";
 
   // vLLM-specific options for thinking tag extraction
-  private thinkingOpenTag?: string;
-  private thinkingCloseTag?: string;
+  private _thinkingOpenTag?: string;
+  private _thinkingCloseTag?: string;
 
   // Override useOpenAIAdapterFor to NOT include "streamChat".
   // vLLM uses the reasoning_content field for thinking output (via vLLM's reasoning parser),
@@ -97,8 +97,8 @@ class Vllm extends OpenAI {
     }
 
     // Store vLLM-specific options
-    this.thinkingOpenTag = options.thinkingOpenTag;
-    this.thinkingCloseTag = options.thinkingCloseTag;
+    this._thinkingOpenTag = options.thinkingOpenTag;
+    this._thinkingCloseTag = options.thinkingCloseTag;
 
     if (options.isFromAutoDetect) {
       this._setupCompletionOptions();
@@ -116,7 +116,7 @@ class Vllm extends OpenAI {
     options: CompletionOptions,
   ): AsyncGenerator<ChatMessage> {
     // If no custom thinking tags configured, use parent implementation
-    if (!this.thinkingOpenTag || !this.thinkingCloseTag) {
+    if (!this._thinkingOpenTag || !this._thinkingCloseTag) {
       for await (const chunk of super._streamChat(messages, signal, options)) {
         yield chunk;
       }
@@ -125,8 +125,8 @@ class Vllm extends OpenAI {
 
     // Use thinking tag extractor for custom tag formats
     const extractor = new ThinkingTagExtractor(
-      this.thinkingOpenTag,
-      this.thinkingCloseTag,
+      this._thinkingOpenTag,
+      this._thinkingCloseTag,
     );
 
     for await (const chunk of super._streamChat(messages, signal, options)) {