codewhisperer: reduce token infilling suggestions (#3944)

Author: zixlin7

src/codewhisperer/service/keyStrokeHandler.ts

@@ -20,7 +20,6 @@ import { ClassifierTrigger } from './classifierTrigger'
 import { isIamConnection } from '../../auth/connection'
 import { session } from '../util/codeWhispererSession'
 import { extractContextForCodeWhisperer } from '../util/editorContext'
-import { CodeWhispererUserGroupSettings } from '../util/userGroupUtil'
 
 const performance = globalThis.performance ?? require('perf_hooks').performance
 
@@ -119,7 +118,6 @@ export class KeyStrokeHandler {
             // we do not want to trigger when there is immediate right context on the same line
             // with "}" being an exception because of IDE auto-complete
             if (
-                CodeWhispererUserGroupSettings.getUserGroup() === CodeWhispererConstants.UserGroup.RightContext &&
                 rightContextAtCurrentLine.length &&
                 !rightContextAtCurrentLine.startsWith(' ') &&
                 rightContextAtCurrentLine.trim() !== '}'

src/codewhisperer/util/userGroupUtil.ts

@@ -56,9 +56,7 @@ export class CodeWhispererUserGroupSettings {
     }
 
     private guessUserGroup(): UserGroup {
-        const randomNum = Math.random()
-        const result = randomNum <= 1 / 2 ? UserGroup.Control : UserGroup.RightContext
-        return result
+        return UserGroup.Control
     }
 
     static #instance: CodeWhispererUserGroupSettings | undefined

src/test/codewhisperer/service/keyStrokeHandler.test.ts

@@ -99,7 +99,7 @@ describe('keyStrokeHandler', function () {
             await testShouldInvoke('a', true)
         })
 
-        it('Should skip invoking if there is immediate right context on the same line and not a single } for the user group', async function () {
+        it('Should skip invoking if there is immediate right context on the same line and not a single }', async function () {
             const casesForSuppressTokenFilling = [
                 {
                     rightContext: 'add',
@@ -127,14 +127,10 @@ describe('keyStrokeHandler', function () {
                 },
             ]
             casesForSuppressTokenFilling.forEach(async ({ rightContext, shouldInvoke }) => {
-                await testShouldInvoke('{', shouldInvoke, rightContext, CodeWhispererConstants.UserGroup.RightContext)
+                await testShouldInvoke('{', shouldInvoke, rightContext)
             })
         })
 
-        it('Should not skip invoking based on right context for control group', async function () {
-            await testShouldInvoke('{', true, 'add')
-        })
-
         async function testShouldInvoke(
             input: string,
             shouldTrigger: boolean,