chore(amazonq): nep cherrypick codewhispererService.ts

[Will-ShaoHua]

Problem

Solution

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[floralph]

NOTE: This comment is for reference and not actionable.

Based on my git blame analysis of both the origin/floralph/nep-flare branch (for additions) and origin/main branch (for
deletions), here are the original authors of the code lines being changed:

ADDED LINES (from origin/floralph/nep-flare branch):

1. Import Statement Reorganization (Lines 25-34) - ADDED

• Lines 25, 33:
• Author: ege0zcan ([email protected])
• Timestamp: 2023-09-26 10:39:39 +0200
• Commit: ba0d1f8
• Message: "Implement CodeWhisperer LSP server using server interface (#21)"
• Description: Initial implementation of CodeWhisperer LSP server with credential logic updates

• Lines 27-32:
• Author: Will Lo ([email protected])
• Timestamp: 2025-06-11 14:55:10 -0700
• Commit: 6e974d1
• Message: "chore: sync nep branch with main (#1623)"
• Description: Synchronization commit

• Line 34:
• Author: Dhanasekar Karuppasamy ([email protected])
• Timestamp: 2025-04-29 08:46:00 -0700
• Commit: c9486ca
• Message: "feat: flare mvp implementation"
• Description: Initial FLARE MVP implementation

2. SuggestionType Enum (Lines 54-57) - ADDED

• Author: Dhanasekar Karuppasamy ([email protected])
• Timestamp: 2025-04-29 08:46:00 -0700
• Commit: c9486ca
• Message: "feat: flare mvp implementation"
• Description: Initial FLARE MVP implementation

3. suggestionType Property in Interface (Line 61) - ADDED

• Author: Dhanasekar Karuppasamy ([email protected])
• Timestamp: 2025-04-29 08:46:00 -0700
• Commit: c9486ca
• Message: "feat: flare mvp implementation"
• Description: Initial FLARE MVP implementation

4. Enhanced Request Handlers (Lines 253-268) - ADDED

• Line 253:
• Author: Dhanasekar Karuppasamy ([email protected])
• Timestamp: 2025-04-29 08:46:00 -0700
• Commit: c9486ca
• Message: "feat: flare mvp implementation"

• Lines 254-257:
• Author: Dhanasekar Karuppasamy ([email protected])
• Timestamp: 2025-06-02 09:09:09 -0700
• Commit: 32edd8d
• Message: "feat: log request responses to tmp file"

• Lines 258-259:
• Author: Dhanasekar Karuppasamy ([email protected])
• Timestamp: 2025-06-04 00:27:47 -0700
• Commit: d7c8e5f
• Message: "feat: simplify logging. remove graphql"

• Line 260:
• Author: pras0131 ([email protected])
• Timestamp: 2025-03-20 14:14:36 +0100
• Commit: 681889b
• Message: "feat: abort all inflight requests when resetCodewhispererService is invoked (#848)"

• Lines 261-268 (error handler):
• Author: Dhanasekar Karuppasamy ([email protected])
• Timestamps: 2025-06-02 09:09:09 -0700 / 2025-06-04 00:27:47 -0700
• Commits: 32edd8d / d7c8e5f
• Messages: "feat: log request responses to tmp file" / "feat: simplify logging. remove graphql"

5. mapCodeWhispererApiResponseToSuggestion Method (Lines 533-555) - ADDED

• Primary Author: Dhanasekar Karuppasamy ([email protected])
• Timestamps: 2025-04-29 08:46:00 -0700 / 2025-06-02 09:09:09 -0700
• Commits: c9486ca / 32edd8d
• Messages: "feat: flare mvp implementation" / "feat: log request responses to tmp file"
• Line 550: Viktor Shcherba ([email protected]) - 2023-11-02 09:28:53 +0100

6. suggestionType in Return Objects (Line 196) - ADDED

• Author: Dhanasekar Karuppasamy ([email protected])
• Timestamp: 2025-04-29 08:46:00 -0700
• Commit: c9486ca
• Message: "feat: flare mvp implementation"

DELETED LINES (from origin/main branch):

1. Comment Line (Line 26) - DELETED

• Author: ege0zcan ([email protected])
• Timestamp: 2023-10-05 12:10:22 +0300
• Commit: 833dc4d
• Message: "Move LSP Server logic into the Server and use Service as thin wrapper (#33)"
• Description: Architectural refactoring to move LSP logic
• Deleted Text: "// Define our own Suggestion interface to wrap the differences between Token and IAM Client"

2. Import Statements (Lines 50-52) - DELETED

• Lines 50-51:
• Author: Shailaja Yadav ([email protected])
• Timestamp: 2024-02-28 08:40:30 -0800
• Commit: 9833d58
• Message: "Added Script to create the CWSPR client (#144)"
• Deleted Text: Import statements for CodeWhispererSigv4Client and CodeWhispererTokenClient

• Line 52:
• Author: Justin M. Keyes ([email protected])
• Timestamp: 2025-06-09 18:04:24 -0400
• Commit: d25bcb6
• Message: "feat(q): builderid "paid tier" #1197"
• Description: Paid tier feature implementation with service model updates
• Deleted Text: import { getErrorId } from './utils'

3. Original req.on('complete') Handler (Lines 196-198) - DELETED

• Author: pras0131 ([email protected])
• Timestamp: 2025-03-20 14:14:36 +0100
• Commit: 681889b
• Message: "feat: abort all inflight requests when resetCodewhispererService is invoked (#848)"
• Description: Feature to abort inflight requests with comprehensive state management
• Deleted Text: Simple req.on('complete', () => { this.completeRequest(req) }) handler

4. Comment Line (Line 54) - DELETED

• Author: ege0zcan ([email protected])
• Timestamp: 2023-10-05 12:10:22 +0300
• Commit: 833dc4d
• Message: "Move LSP Server logic into the Server and use Service as thin wrapper (#33)"
• Deleted Text: "// Right now the only difference between the token client and the IAM client for codewhsiperer is the
difference in function name"

Summary of Key Contributors by Timeline:

2023:
• ege0zcan (Sep-Oct 2023) - Original CodeWhisperer LSP server implementation and architectural refactoring
• Viktor Shcherba (Nov 2023) - Telemetry and session management scaffolding

2024:
• Shailaja Yadav (Feb 2024) - Client creation scripts

2025:
• pras0131 (Mar 2025) - Request lifecycle management features
• Dhanasekar Karuppasamy (Apr-Jun 2025) - Primary contributor for FLARE MVP implementation (most additions)
• Justin M. Keyes (Jun 2025) - Paid tier features and service model updates
• Will Lo (Jun 2025) - Branch synchronization and maintenance

Change Pattern Analysis:

• Most Recent Changes (2025): Primarily FLARE MVP implementation by Dhanasekar Karuppasamy
• Oldest Affected Code (2023): Original architectural decisions by ege0zcan being modified/removed
• Code Movement: Import statements and comments being reorganized rather than fundamentally changed
• Feature Evolution: Simple request handlers being replaced with enhanced logging and telemetry

The majority of the ADDED functionality represents new FLARE MVP features, while the DELETED code represents cleanup of
older architectural patterns and import organization.

[dk19y]

Looks good to me