bugs in React & Next template

Author: SFARPak

Roo-Code/apps/web-evals/src/app/runs/new/new-run.tsx

@@ -94,7 +94,7 @@ export function NewRun() {
 		async (values: CreateRun) => {
 			try {
 				if (mode === "openrouter") {
-					values.settings = { ...(values.settings || {}), openRouterModelId: model }
+					values.settings = { ...values.settings, openRouterModelId: model }
 				}
 
 				const { id } = await createRun(values)

Roo-Code/packages/cloud/src/TelemetryClient.ts

@@ -60,7 +60,7 @@ abstract class BaseTelemetryClient implements TelemetryClient {
 		// Event properties take precedence in case of conflicts.
 		const mergedProperties = {
 			...providerProperties,
-			...(event.properties || {}),
+			...event.properties,
 		}
 
 		// Filter out properties that shouldn't be captured by this client

Roo-Code/packages/telemetry/src/BaseTelemetryClient.ts

@@ -51,7 +51,7 @@ export abstract class BaseTelemetryClient implements TelemetryClient {
 
 		// Merge provider properties with event-specific properties.
 		// Event properties take precedence in case of conflicts.
-		const mergedProperties = { ...providerProperties, ...(event.properties || {}) }
+		const mergedProperties = { ...providerProperties, ...event.properties }
 
 		// Filter out properties that shouldn't be captured by this client
 		return Object.fromEntries(Object.entries(mergedProperties).filter(([key]) => this.isPropertyCapturable(key)))

Roo-Code/packages/telemetry/src/TelemetryService.ts

@@ -198,7 +198,7 @@ export class TelemetryService {
 			itemType,
 			itemName,
 			target,
-			...(properties || {}),
+			...properties,
 		})
 	}
 

Roo-Code/src/__tests__/command-mentions.spec.ts

@@ -218,7 +218,7 @@ npm install
 
 	describe("command mention regex patterns", () => {
 		it("should match valid command mention patterns anywhere", () => {
-			const commandRegex = /\/([a-zA-Z0-9_\.-]+)(?=\s|$)/g
+			const commandRegex = /\/([a-zA-Z0-9_.-]+)(?=\s|$)/g
 
 			const validPatterns = ["/setup", "/build-prod", "/test_suite", "/my-command", "/command123"]
 
@@ -230,19 +230,19 @@ npm install
 		})
 
 		it("should match command patterns in middle of text", () => {
-			const commandRegex = /\/([a-zA-Z0-9_\.-]+)(?=\s|$)/g
+			const commandRegex = /\/([a-zA-Z0-9_.-]+)(?=\s|$)/g
 
 			const validPatterns = ["Please /setup", "Run /build now", "Use /deploy here"]
 
 			validPatterns.forEach((pattern) => {
 				const match = pattern.match(commandRegex)
 				expect(match).toBeTruthy()
-				expect(match![0]).toMatch(/^\/[a-zA-Z0-9_\.-]+$/)
+				expect(match![0]).toMatch(/^\/[a-zA-Z0-9_.-]+$/)
 			})
 		})
 
 		it("should match commands at start of new lines", () => {
-			const commandRegex = /\/([a-zA-Z0-9_\.-]+)(?=\s|$)/g
+			const commandRegex = /\/([a-zA-Z0-9_.-]+)(?=\s|$)/g
 
 			const multilineText = "First line\n/setup the project\nAnother line\n/deploy when ready"
 			const matches = multilineText.match(commandRegex)
@@ -255,7 +255,7 @@ npm install
 		})
 
 		it("should match multiple commands in message", () => {
-			const commandRegex = /(?:^|\s)\/([a-zA-Z0-9_\.-]+)(?=\s|$)/g
+			const commandRegex = /(?:^|\s)\/([a-zA-Z0-9_.-]+)(?=\s|$)/g
 
 			const validText = "/setup the project\nThen /deploy later"
 			const matches = validText.match(commandRegex)
@@ -267,7 +267,7 @@ npm install
 		})
 
 		it("should not match invalid command patterns", () => {
-			const commandRegex = /\/([a-zA-Z0-9_\.-]+)(?=\s|$)/g
+			const commandRegex = /\/([a-zA-Z0-9_.-]+)(?=\s|$)/g
 
 			const invalidPatterns = ["/ space", "/with space", "/with/slash", "//double", "/with@symbol"]
 

Roo-Code/src/activate/registerCodeActions.ts

@@ -40,7 +40,7 @@ const registerCodeAction = (context: vscode.ExtensionContext, command: CodeActio
 			}
 
 			const params = {
-				...{ filePath, selectedText },
+				 filePath, selectedText,
 				...(startLine !== undefined ? { startLine: startLine.toString() } : {}),
 				...(endLine !== undefined ? { endLine: endLine.toString() } : {}),
 				...(diagnostics ? { diagnostics } : {}),

Roo-Code/src/api/providers/bedrock.ts

@@ -823,7 +823,7 @@ export class AwsBedrockHandler extends BaseProvider implements SingleCompletionH
 		 * match[4] - The resource ID (e.g., "anthropic.claude-3-sonnet-20240229-v1:0")
 		 */
 
-		const arnRegex = /^arn:aws:(?:bedrock|sagemaker):([^:]+):([^:]*):(?:([^\/]+)\/([\w\.\-:]+)|([^\/]+))$/
+		const arnRegex = /^arn:aws:(?:bedrock|sagemaker):([^:]+):([^:]*):(?:([^/]+)\/([\w.\-:]+)|([^/]+))$/
 		let match = arn.match(arnRegex)
 
 		if (match && match[1] && match[3] && match[4]) {

Roo-Code/src/api/providers/openai.ts

@@ -46,7 +46,7 @@ export class OpenAiHandler extends BaseProvider implements SingleCompletionHandl
 
 		const headers = {
 			...DEFAULT_HEADERS,
-			...(this.options.openAiHeaders || {}),
+			...this.options.openAiHeaders,
 		}
 
 		const timeout = getApiRequestTimeout()
@@ -459,7 +459,7 @@ export async function getOpenAiModels(baseUrl?: string, apiKey?: string, openAiH
 		const config: Record<string, any> = {}
 		const headers: Record<string, string> = {
 			...DEFAULT_HEADERS,
-			...(openAiHeaders || {}),
+			...openAiHeaders,
 		}
 
 		if (apiKey) {

Roo-Code/src/api/providers/router-provider.ts

@@ -50,7 +50,7 @@ export abstract class RouterProvider extends BaseProvider {
 			apiKey,
 			defaultHeaders: {
 				...DEFAULT_HEADERS,
-				...(options.openAiHeaders || {}),
+				...options.openAiHeaders,
 			},
 		})
 	}

Roo-Code/src/api/transform/cache-strategy/__tests__/cache-strategy.spec.ts

@@ -18,7 +18,7 @@ const defaultModelInfo: ModelInfo = {
 const createConfig = (overrides: Partial<CacheStrategyConfig> = {}): CacheStrategyConfig => ({
 	modelInfo: {
 		...defaultModelInfo,
-		...(overrides.modelInfo || {}),
+		...overrides.modelInfo,
 	},
 	systemPrompt: "You are a helpful assistant",
 	messages: [],