Feature/amazon q /doc: upload diagram files during doc generation #1

vikshe · 2025-01-24T17:49:06Z

Upload architecture diagram image files during documentation generation

Types of changes

Bug fix (non-breaking change which fixes an issue)
[ X ] New feature (non-breaking change which adds functionality)

Description

Upload architecture diagram image files during documentation generation

Checklist

[ X ] My code follows the code style of this project
I have added tests to cover my changes
[ X] A short description of the change has been added to the CHANGELOG if the change is customer-facing in the IDE.
I have added metrics for my changes (if required)

License

I confirm that my contribution is made under the terms of the Apache 2.0 license.

avi-alpert · 2025-01-24T20:29:14Z

...s-community/src/software/aws/toolkits/jetbrains/services/amazonq/FeatureDevSessionContext.kt

    // TODO: Need to correct this class location in the modules going further to support both amazonq and codescan.

+    // well known source files that do not have extensions
+    private val wellKnownSourceFiles = setOf(


this change just moves wellKnownSourceFiles further up right? just wanted to make sure there's no actual change here to /dev's functionality

for this variable - yes, only move it up.

the whole file has same functionality as before, the only changes is to add support to extend this class so that our class can include diagrams in the zip archive

…lemetry and ad… (aws#5345) * Filter out frequent client error from codegeneration telemetry and add errMsg to reasonDesc when desc is null * refactor error messages into constnt --------- Co-authored-by: Yuxin Lin <[email protected]>

…ted unit tests (aws#5337) * enable syntax highlighting when viewing diff for generated unit tests changelog remove unused import * lint * virtualFile always created and will not be null * use session.testFileName instead of extension * lint * create temporary virtual file * comment * Update bugfix-3b7843b4-9673-4afa-90e8-7c4278f0d0fe.json

) * Adding truncation logic to testGen user prompt

…s#5356) * Show the source file path instead of test file path in short answer if user does /test on test file.

….addConnection (aws#5338) * Updated AddConnection telemetry event in LoginBrowser.kt * Updated AddConnection telemetry event * Fixed issues using gradlew detekt

This reverts commit 9a60e50.

…cies (aws#5329) ui-tests-starter attempts to resolve the IDE distribution and fails, but it should not actually be required to build/execute

…nd folders to ignore list. (aws#5361)

…on (aws#5347) config(amazonq): customization override should respect users' selection --------- Co-authored-by: Richard Li <[email protected]>

* Correcting auth checks for /test workflow

…s#5326) * fix(amazonq): single click or select through arrow keys on review code issues does not focus code editor to line nor show issue details * add change report * add back modelzonemarket.cs file * deleted irrelevant comment --------- Co-authored-by: Tai Lai <[email protected]>

aws#5368) This reverts commit 805cb39.

It is easy to forget to update all artifacts on a new MV, so consolidate all artifacts under same matrix

Attempt to enforce more consistency on commit messages

github-actions · 2025-02-14T16:51:22Z

Qodana Community for JVM

12 new problems were found

Inspection name	Severity	Problems
`Usage of redundant or deprecated syntax or deprecated symbols`	🔶 Warning	9
`Unused symbol`	🔶 Warning	1
`Function or property has platform type`	◽️ Notice	1
`Join declaration and assignment`	◽️ Notice	1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked

View the detailed Qodana report

To be able to view the detailed Qodana report, you can either:

Register at Qodana Cloud and configure the action
Use GitHub Code Scanning with Qodana
Host Qodana report at GitHub Pages
Inspect and use qodana.sarif.json (see the Qodana SARIF format for details)

To get *.log files or any other Qodana artifacts, run the action with upload-result option set to true,
so that the action will upload the files as the job artifacts:

      - name: 'Qodana Scan'
        uses: JetBrains/[email protected]
        with:
          upload-result: true

Contact Qodana team

Contact us at [email protected]

Or via our issue tracker: https://jb.gg/qodana-issue
Or share your feedback: https://jb.gg/qodana-discussions

* improve error handling of workspace index

… client from AwsClientManager (aws#5331) * No need to store a client as a state within this class, AwsClientManager already handle the connection change resolving comments aws#5290 (comment)

Co-authored-by: David Hasani <[email protected]>

…o user-selected region (aws#5365) WARN - software.aws.toolkits.jetbrains.core.credentials.AwsConnectionManager - Using activeRegion when region is null, calling code needs to be migrated to handle null java.lang.IllegalStateException at software.aws.toolkits.jetbrains.core.credentials.AwsConnectionManager.getActiveRegion(AwsConnectionManager.kt:224) at software.aws.toolkits.jetbrains.core.credentials.AwsConnectionManagerKt.activeRegion(AwsConnectionManager.kt:370) at migration.software.aws.toolkits.jetbrains.services.telemetry.TelemetryService.record(TelemetryService.kt:59) at software.aws.toolkits.telemetry.SessionTelemetry.start(SessionTelemetry.kt:287) at software.aws.toolkits.telemetry.SessionTelemetry.start$default(SessionTelemetry.kt:271) at software.aws.toolkits.jetbrains.services.telemetry.AwsToolkitStartupMetrics.execute$lambda$0(AwsToolkitStartupMetrics.kt:14) at com.intellij.ide.util.RunOnceUtilKt.doRunOnce(RunOnceUtil.kt:68) at com.intellij.ide.util.RunOnceUtilKt.access$doRunOnce(RunOnceUtil.kt:1)

) Since 2.0, logs are isolated by platform/version pair

This reverts commit 4971993.

…s#5336) Previously, UTG did not have telemetry to collect user feedback on generated unit tests. This PR introduces telemetry support to address that gap. Additionally, it enables users to submit comments, which will be routed to a private SIM folder.

…mework (aws#5385) Force resolve 243.21565.192 -> 243.21565.193 ``` org.gradle.internal.resolve.ModuleVersionNotFoundException: Could not find any version that matches com.jetbrains.intellij.java:java-test-framework:{strictly [243, 243.21565.192]; prefer 243.21565.192}. Versions that do not match: - 243.24978.46 - 243.23654.189 - 243.23654.153 - 243.23654.117 - 243.22562.218 - + 124 more ```

* Adding truncation logic to testGen user prompt * Adding change log * Updating telemetry error message to align VSC and JB * Removing unwanted files

…ws#5360) * fix(amazonq): update logic for converting gitignore pattern to regex * fix(amazonq): update logic for converting gitignore pattern to regex --------- Co-authored-by: Gaurav Gandhi <[email protected]> Co-authored-by: Richard Li <[email protected]>

… up (aws#5375)

* fix tab acceptAction not working in JB remote * use isRunningOnRemoteBackend

…ws#5357) Co-authored-by: Viktor Shesternyak <[email protected]>

…ase ignore list. (aws#5389)

...rc/software/aws/toolkits/jetbrains/services/amazonqCodeTest/controller/CodeTestChatHelper.kt

@@ -15,6 +16,8 @@
 import software.aws.toolkits.jetbrains.services.amazonqCodeTest.session.Session
 import software.aws.toolkits.jetbrains.services.amazonqCodeTest.storage.ChatSessionStorage
 import software.aws.toolkits.jetbrains.services.cwc.messages.ChatMessageType
+import software.aws.toolkits.jetbrains.services.cwc.messages.FollowUp
+import software.aws.toolkits.resources.message


...rc/software/aws/toolkits/jetbrains/services/amazonqCodeTest/controller/CodeTestChatHelper.kt

+
+    suspend fun sendChatInputEnabledMessage(isEnabled: Boolean) {
+        if (isInvalidSession()) return
+        messagePublisher.publish(ChatInputEnabledMessage(activeCodeTestTabId as String, enabled = isEnabled))


...rc/software/aws/toolkits/jetbrains/services/amazonqCodeTest/controller/CodeTestChatHelper.kt

+                tabId = tabId,
+                messageId = messageId ?: UUID.randomUUID().toString(),
+                messageType = ChatMessageType.Answer,
+                message = message("amazonqFeatureDev.follow_instructions_for_authentication"),


...jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonqDoc/ui/UiContants.kt

+import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.FollowUp
+import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.FollowUpStatusType
+import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.FollowUpTypes
+import software.aws.toolkits.resources.message


...jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonqDoc/ui/UiContants.kt

+
+val NEW_SESSION_FOLLOWUPS: List<FollowUp> = listOf(
+    FollowUp(
+        pillText = message("amazonqDoc.prompt.reject.new_task"),


...jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonqDoc/ui/UiContants.kt

+        status = FollowUpStatusType.Info
+    ),
+    FollowUp(
+        pillText = message("amazonqDoc.prompt.reject.close_session"),


...ore/jetbrains-ultimate/src-241-243/compat/com/intellij/lang/javascript/JavascriptLanguage.kt

+package compat.com.intellij.lang.javascript
+
+// probably not necessary, but inline to avoid loading this through core classpath
+inline val JavascriptLanguage


vikshe force-pushed the bugfix/amazon-q-doc-upload-diagram branch from b339819 to 231c426 Compare January 24, 2025 17:56

aggagen approved these changes Jan 24, 2025

View reviewed changes

avi-alpert reviewed Jan 24, 2025

View reviewed changes

vikshe force-pushed the bugfix/amazon-q-doc-upload-diagram branch from 231c426 to 47cc7e5 Compare January 24, 2025 22:27

vikshe changed the title ~~Bugfix/amazon q /doc: upload diagram files during doc generation~~ Feature/amazon q /doc: upload diagram files during doc generation Jan 24, 2025

vikshe force-pushed the bugfix/amazon-q-doc-upload-diagram branch from 47cc7e5 to d5fa9d8 Compare January 27, 2025 19:30

vikshe force-pushed the bugfix/amazon-q-doc-upload-diagram branch from d5fa9d8 to 9e5289b Compare February 11, 2025 16:52

vikshe force-pushed the bugfix/amazon-q-doc-upload-diagram branch from 9e5289b to 9308cd1 Compare February 11, 2025 17:17

chungjac and others added 6 commits February 11, 2025 10:39

fix(amazonq): Truncating user prompt for unit test generation. (aws#5359

0c88e67

) * Adding truncation logic to testGen user prompt

telemetry(amazonq): refactor doc telemetry to v2 version (aws#5316)

eb19cbc

fix(amazonq): Q infers source file from active test file for UTG. (aw…

991abe2

…s#5356) * Show the source file path instead of test file path in short answer if user does /test on test file.

Updated AuthTelemetry.addConnection telemetry event to Telemetry.auth…

8c69af2

….addConnection (aws#5338) * Updated AddConnection telemetry event in LoginBrowser.kt * Updated AddConnection telemetry event * Fixed issues using gradlew detekt

Revert "revert(amazonq): hide java 21 option (aws#5354)" (aws#5358)

0761f14

This reverts commit 9a60e50.

vikshe force-pushed the bugfix/amazon-q-doc-upload-diagram branch from 9308cd1 to 0a24b17 Compare February 11, 2025 22:53

rli and others added 11 commits February 11, 2025 15:20

build: fix 251 build failure from resolving ui-tests-starter dependen…

f401e08

…cies (aws#5329) ui-tests-starter attempts to resolve the IDE distribution and fails, but it should not actually be required to build/execute

build: include 2025.1 in prerelease automation (aws#5330)

82cbd6c

Adding gitignore usecase for Test Generation to add more extensions a…

7087f28

…nd folders to ignore list. (aws#5361)

config(amazonq): customization override should respect users' selecti…

951d519

…on (aws#5347) config(amazonq): customization override should respect users' selection --------- Co-authored-by: Richard Li <[email protected]>

fix(amazonq): fix for /test authentication check (aws#5355)

b440147

* Correcting auth checks for /test workflow

Revert "feat(amazonq): Prefetch next inline recommendation (aws#5290)" (

68a179a

aws#5368) This reverts commit 805cb39.

ci: use single matrix for all prerelease artifacts (aws#5362)

39eacec

It is easy to forget to update all artifacts on a new MV, so consolidate all artifacts under same matrix

ci: steal commit message linter from VSC toolkit (aws#5351)

19931e5

Attempt to enforce more consistency on commit messages

Updating version to 3.55

3b6cdc3

Updating SNAPSHOT version to 3.56-SNAPSHOT

0972fcc

leigaol and others added 2 commits February 14, 2025 10:19

refactor(amazonq): improve error handling of workspace index (aws#5373)

3621194

* improve error handling of workspace index

refactor(Q tech debt): make cwsprClientAdapr stateless and obtain sdk…

4971993

… client from AwsClientManager (aws#5331) * No need to store a client as a state within this class, AwsClientManager already handle the connection change resolving comments aws#5290 (comment)

dhasani23 and others added 9 commits February 14, 2025 21:32

fix(amazonq): only search for sql strings in java files (aws#5376)

3d9586d

Co-authored-by: David Hasani <[email protected]>

fix: Fix idea.log path referenced in plugin run configurations (aws#5369

9426764

) Since 2.0, logs are isolated by platform/version pair

revert(amazonq): pr aws#5331 broke tests by throwing NotAMock (aws#5383)

fd7eb31

This reverts commit 4971993.

telemetry(amazonq): Aligning telemetry strings in VSC and JB. (aws#5382)

2eca5a7

* Adding truncation logic to testGen user prompt * Adding change log * Updating telemetry error message to align VSC and JB * Removing unwanted files

log(amazonq): update log string when workspace process is not spinned…

f5460c7

… up (aws#5375)

vikshe force-pushed the bugfix/amazon-q-doc-upload-diagram branch from 77d5577 to fec3b27 Compare February 18, 2025 20:19

andrewyuq and others added 4 commits February 18, 2025 22:55

fix(amazonq): fix tab acceptAction not working in JB remote (aws#5379)

e9c40fa

* fix tab acceptAction not working in JB remote * use isRunningOnRemoteBackend

fix(amazonq): /doc add suppoort for uploading architecture diagrams (a…

93b3b5a

…ws#5357) Co-authored-by: Viktor Shesternyak <[email protected]>

Adding more extensions and no extension files to test generation usec…

f0911c4

…ase ignore list. (aws#5389)

config(amazonq): update polling config for codefix (aws#5390)

beabdf1

vikshe force-pushed the bugfix/amazon-q-doc-upload-diagram branch from 16f8768 to beabdf1 Compare February 19, 2025 14:14

github-advanced-security bot found potential problems Feb 19, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/amazon q /doc: upload diagram files during doc generation #1

Feature/amazon q /doc: upload diagram files during doc generation #1

vikshe commented Jan 24, 2025

avi-alpert Jan 24, 2025

vikshe Jan 24, 2025

github-actions bot commented Feb 14, 2025 •

edited

Loading

Feature/amazon q /doc: upload diagram files during doc generation #1

Are you sure you want to change the base?

Feature/amazon q /doc: upload diagram files during doc generation #1

Conversation

vikshe commented Jan 24, 2025

Types of changes

Description

Checklist

License

avi-alpert Jan 24, 2025

Choose a reason for hiding this comment

vikshe Jan 24, 2025

Choose a reason for hiding this comment

github-actions bot commented Feb 14, 2025 • edited Loading

Qodana Community for JVM

github-actions bot commented Feb 14, 2025 •

edited

Loading