Skip to content
Merged
Show file tree
Hide file tree
Changes from 10 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/prerelease.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:
generate_artifact_toolkit_standalone:
strategy:
matrix:
supported_versions: [ '2024.1', '2024.2', '2024.3' ]
supported_versions: [ '2024.1', '2024.2', '2024.3', '2025.1' ]
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
// Copyright 2025 Amazon.com, Inc. or its affiliates. All Rights Reserved.
// SPDX-License-Identifier: Apache-2.0

package compat.com.intellij.lang.javascript

// probably not necessary, but inline to avoid loading this through core classpath
inline val JavascriptLanguage

Check notice on line 7 in plugins/core/jetbrains-ultimate/src-241-243/compat/com/intellij/lang/javascript/JavascriptLanguage.kt

View workflow job for this annotation

GitHub Actions / Qodana Community for JVM

Function or property has platform type

Declaration has type inferred from a platform call, which can lead to unchecked nullability issues. Specify type explicitly as nullable or non-nullable.

Check notice

Code scanning / QDJVMC

Function or property has platform type Note

Declaration has type inferred from a platform call, which can lead to unchecked nullability issues. Specify type explicitly as nullable or non-nullable.
get() = com.intellij.lang.javascript.JavascriptLanguage.INSTANCE
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,7 @@
import software.aws.toolkits.jetbrains.core.region.AwsRegionProvider
import software.aws.toolkits.resources.message
import software.aws.toolkits.telemetry.AwsTelemetry
import software.aws.toolkits.telemetry.Result.Failed
import software.aws.toolkits.telemetry.Result.Succeeded
import software.aws.toolkits.telemetry.MetricResult

class AwsConnectionRunConfigurationExtension<T : RunConfigurationBase<*>> {
fun addToTargetEnvironment(configuration: T, environment: MutableMap<String, TargetEnvironmentFunction<String>>, runtimeString: () -> String? = { null }) {
Expand Down Expand Up @@ -60,9 +59,9 @@

val connection = getConnection(configuration, credentialConfiguration)
environmentMutator(connection)
AwsTelemetry.injectCredentials(configuration.project, result = Succeeded, runtimeString = tryOrNull { runtimeString() })
AwsTelemetry.injectCredentials(configuration.project, result = MetricResult.Succeeded, runtimeString = tryOrNull { runtimeString() })
} catch (e: Exception) {
AwsTelemetry.injectCredentials(configuration.project, result = Failed, runtimeString = tryOrNull { runtimeString() })
AwsTelemetry.injectCredentials(configuration.project, result = MetricResult.Failed, runtimeString = tryOrNull { runtimeString() })

Check warning on line 64 in plugins/toolkit/jetbrains-core/src/software/aws/toolkits/jetbrains/core/execution/AwsConnectionExtension.kt

View check run for this annotation

Codecov / codecov/patch

plugins/toolkit/jetbrains-core/src/software/aws/toolkits/jetbrains/core/execution/AwsConnectionExtension.kt#L64

Added line #L64 was not covered by tests
LOG.error(e) { message("run_configuration_extension.inject_aws_connection_exception") }
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,13 @@ import com.intellij.openapi.vfs.VirtualFile
import com.intellij.ui.MutableCollectionComboBoxModel
import com.intellij.ui.SimpleListCellRenderer
import com.intellij.ui.components.JBTextField
import com.intellij.ui.dsl.builder.AlignX
import com.intellij.ui.dsl.builder.actionListener
import com.intellij.ui.dsl.builder.bind
import com.intellij.ui.dsl.builder.bindSelected
import com.intellij.ui.dsl.builder.bindText
import com.intellij.ui.dsl.builder.panel
import com.intellij.ui.dsl.builder.toMutableProperty
import com.intellij.ui.dsl.gridLayout.HorizontalAlign
import com.intellij.ui.layout.selected
import com.intellij.util.text.nullize
import org.jetbrains.annotations.TestOnly
Expand Down Expand Up @@ -182,7 +182,7 @@ class SyncServerlessApplicationDialog(
}
}
cell(stackNameField)
.horizontalAlign(HorizontalAlign.FILL)
.align(AlignX.FILL)
.enabledIf(createStackButton.component.selected)
.bindText(::newStackName)
.validationOnApply { field ->
Expand Down Expand Up @@ -215,7 +215,7 @@ class SyncServerlessApplicationDialog(
}
stackSelector.reload(forceFetch = true)
cell(stackSelector)
.horizontalAlign(HorizontalAlign.FILL)
.align(AlignX.FILL)
.enabledIf(updateStackButton.component.selected)
.errorOnApply(message("serverless.application.sync.validation.stack.missing")) {
it.isEnabled && (it.isLoading || it.selected() == null)
Expand All @@ -228,19 +228,19 @@ class SyncServerlessApplicationDialog(
.withBinding(::templateParameters.toMutableProperty())
.validationOnApply {
validateParameters(it, templateFileParameters)
}.horizontalAlign(HorizontalAlign.FILL)
}.align(AlignX.FILL)
.component.toolTipText = message("serverless.application.sync.tooltip.template.parameters")
}
val tagsString = message("tags.title")
row(tagsString) {
cell(tagsField)
.horizontalAlign(HorizontalAlign.FILL)
.align(AlignX.FILL)
.withBinding(::tags.toMutableProperty())
}

row(message("serverless.application.sync.label.bucket")) {
cell(s3BucketSelector)
.horizontalAlign(HorizontalAlign.FILL)
.align(AlignX.FILL)
.errorOnApply(message("serverless.application.sync.validation.s3.bucket.empty")) { it.isLoading || it.selected() == null }
.component.toolTipText = message("serverless.application.sync.tooltip.s3Bucket")

Expand All @@ -262,7 +262,7 @@ class SyncServerlessApplicationDialog(

row(message("serverless.application.sync.label.repo")) {
cell(ecrRepoSelector)
.horizontalAlign(HorizontalAlign.FILL)
.align(AlignX.FILL)
.errorOnApply(message("serverless.application.sync.validation.ecr.repo.empty")) {
it.isVisible && (it.isLoading || it.selected() == null)
}.component.toolTipText = message("serverless.application.sync.tooltip.ecrRepo")
Expand Down
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this file deleted?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unused

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@
import com.intellij.remoteDev.downloader.CodeWithMeClientDownloader
import com.intellij.ui.components.JBTabbedPane
import com.intellij.ui.dsl.builder.Align
import com.intellij.ui.dsl.builder.AlignX
import com.intellij.ui.dsl.builder.AlignY
import com.intellij.ui.dsl.builder.panel
import com.intellij.ui.dsl.gridLayout.HorizontalAlign
import com.intellij.ui.dsl.gridLayout.VerticalAlign
import com.intellij.util.ui.JBFont
import com.jetbrains.gateway.api.ConnectionRequestor
import com.jetbrains.gateway.api.GatewayConnectionHandle
Expand Down Expand Up @@ -325,7 +325,7 @@
setCenterPanel(
panel {
row {
icon(AllIcons.General.ErrorDialog).verticalAlign(VerticalAlign.TOP)
icon(AllIcons.General.ErrorDialog).align(AlignY.TOP)

Check warning on line 328 in plugins/toolkit/jetbrains-gateway/src/software/aws/toolkits/jetbrains/gateway/CawsConnectionProvider.kt

View check run for this annotation

Codecov / codecov/patch

plugins/toolkit/jetbrains-gateway/src/software/aws/toolkits/jetbrains/gateway/CawsConnectionProvider.kt#L328

Added line #L328 was not covered by tests

panel {
row {
Expand All @@ -344,7 +344,7 @@
collapsibleGroup(message("general.logs"), false) {
row {
cell(view)
.horizontalAlign(HorizontalAlign.FILL)
.align(AlignX.FILL)

Check warning on line 347 in plugins/toolkit/jetbrains-gateway/src/software/aws/toolkits/jetbrains/gateway/CawsConnectionProvider.kt

View check run for this annotation

Codecov / codecov/patch

plugins/toolkit/jetbrains-gateway/src/software/aws/toolkits/jetbrains/gateway/CawsConnectionProvider.kt#L347

Added line #L347 was not covered by tests
}
}.expanded = false
// TODO: can't seem to reliably force a terminal redraw on initial expand
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,14 @@
import com.intellij.execution.runners.ExecutionEnvironment
import com.intellij.javascript.debugger.LocalFileSystemFileFinder
import com.intellij.javascript.debugger.RemoteDebuggingFileFinder
import com.intellij.lang.javascript.JavascriptLanguage
import com.intellij.openapi.util.io.FileUtil
import com.intellij.openapi.vfs.VirtualFile
import com.intellij.xdebugger.XDebugProcess
import com.intellij.xdebugger.XDebugProcessStarter
import com.intellij.xdebugger.XDebugSession
import com.jetbrains.debugger.wip.WipLocalVmConnection
import com.jetbrains.nodeJs.NodeChromeDebugProcess
import compat.com.intellij.lang.javascript.JavascriptLanguage
import org.jetbrains.io.LocalFileFinder
import software.aws.toolkits.core.lambda.LambdaRuntime
import software.aws.toolkits.jetbrains.services.PathMapping
Expand All @@ -39,7 +39,7 @@

abstract class NodeJsImageDebugSupport : ImageDebugSupport {
override fun supportsPathMappings(): Boolean = true
override val languageId = JavascriptLanguage.INSTANCE.id
override val languageId = JavascriptLanguage.id

Check warning on line 42 in plugins/toolkit/jetbrains-ultimate/src/software/aws/toolkits/jetbrains/services/lambda/nodejs/NodeJsDebugSupport.kt

View check run for this annotation

Codecov / codecov/patch

plugins/toolkit/jetbrains-ultimate/src/software/aws/toolkits/jetbrains/services/lambda/nodejs/NodeJsDebugSupport.kt#L42

Added line #L42 was not covered by tests
override suspend fun createDebugProcess(
context: Context,
environment: ExecutionEnvironment,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,20 +5,20 @@ package software.aws.toolkits.jetbrains.services.lambda.nodejs

import com.intellij.javascript.nodejs.interpreter.NodeJsInterpreterManager
import com.intellij.lang.javascript.JavaScriptSupportLoader
import com.intellij.lang.javascript.JavascriptLanguage
import com.intellij.openapi.module.Module
import com.intellij.openapi.module.ModuleType
import com.intellij.openapi.module.WebModuleTypeBase
import com.intellij.openapi.project.Project
import com.intellij.openapi.projectRoots.Sdk
import compat.com.intellij.lang.javascript.JavascriptLanguage
import software.aws.toolkits.core.lambda.LambdaRuntime
import software.aws.toolkits.jetbrains.services.lambda.BuiltInRuntimeGroups
import software.aws.toolkits.jetbrains.services.lambda.SdkBasedRuntimeGroup

class NodeJsRuntimeGroup : SdkBasedRuntimeGroup() {
override val id: String = BuiltInRuntimeGroups.NodeJs
override val languageIds: Set<String> = setOf(
JavascriptLanguage.INSTANCE.id,
JavascriptLanguage.id,
JavaScriptSupportLoader.ECMA_SCRIPT_6.id
)
override val supportsPathMappings: Boolean = true
Expand Down
17 changes: 13 additions & 4 deletions ui-tests-starter/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ plugins {
id("toolkit-kotlin-conventions")
id("toolkit-intellij-plugin")

id("org.jetbrains.intellij.platform")
id("org.jetbrains.intellij.platform.base")
}

val ideProfile = IdeVersions.ideProfile(project)
Expand All @@ -27,15 +27,24 @@ intellijPlatform {
instrumentCode = false
}

tasks.initializeIntellijPlatformPlugin {
enabled = false
}

tasks.verifyPluginProjectConfiguration {
runtimeDirectory.set(null as File?)
enabled = false
}

val testPlugins by configurations.registering

dependencies {
testImplementation(platform("com.jetbrains.intellij.tools:ide-starter-squashed"))
// should really be set by the BOM, but too much work to figure out right now
testImplementation("org.kodein.di:kodein-di-jvm:7.20.2")
intellijPlatform {
intellijIdeaCommunity(IdeVersions.ideProfile(providers).map { it.name })

// shouldn't be needed? but IsolationException
val version = ideProfile.community.sdkVersion
intellijIdeaCommunity(version, !version.contains("SNAPSHOT"))
testFramework(TestFrameworkType.Starter)
}

Expand Down
Loading