Merge branch 'develop' into docs/return-example

.github/ISSUE_TEMPLATE/bug.yml

@@ -2,7 +2,6 @@ name: Bug report
 description: Report a bug or unexpected behavior in the project
 
 labels: ["bug", "area/backend", "area/frontend"]
-type: Bug
 
 body:
   - type: markdown

.github/ISSUE_TEMPLATE/feature.yml

@@ -2,7 +2,6 @@ name: Feature request
 description: Suggest a new feature or improvement to enhance the project
 
 labels: ["enhancement", "area/backend", "area/frontend"]
-type: Feature
 
 body:
   - type: textarea

.github/dependabot.yml

@@ -26,7 +26,7 @@ updates:
     open-pull-requests-limit: 50
     labels: ["dependency-upgrade", "area/backend"]
     ignore:
-      # Ignore versions of Protobuf >= 4.0.0 because Orc still uses version 3
+      # Ignore versions of Protobuf that are equal to or greater than 4.0.0 as Orc still uses 3
       - dependency-name: "com.google.protobuf:*"
         versions: ["[4,)"]
 
@@ -44,75 +44,68 @@ updates:
       build:
         applies-to: version-updates
         patterns: ["@esbuild/*", "@rollup/*", "@swc/*"]
-
       types:
         applies-to: version-updates
         patterns: ["@types/*"]
-
       storybook:
         applies-to: version-updates
-        patterns: ["storybook*", "@storybook/*", "eslint-plugin-storybook"]
-
+        patterns: ["@storybook/*"]
       vitest:
         applies-to: version-updates
         patterns: ["vitest", "@vitest/*"]
-
-      major:
-        update-types: ["major"]
-        applies-to: version-updates
-        exclude-patterns: [
-            "@esbuild/*",
-            "@rollup/*",
-            "@swc/*",
-            "@types/*",
-            "storybook*",
-            "@storybook/*",
-            "eslint-plugin-storybook",
-            "vitest",
-            "@vitest/*",
-            # Temporary exclusion of these packages from major updates
-            "eslint-plugin-vue",
-          ]
-
-      minor:
-        update-types: ["minor"]
-        applies-to: version-updates
-        exclude-patterns: [
-            "@esbuild/*",
-            "@rollup/*",
-            "@swc/*",
-            "@types/*",
-            "storybook*",
-            "@storybook/*",
-            "eslint-plugin-storybook",
-            "vitest",
-            "@vitest/*",
-            # Temporary exclusion of these packages from minor updates
-            "moment-timezone",
-            "monaco-editor",
-          ]
-
       patch:
-        update-types: ["patch"]
         applies-to: version-updates
+        patterns: ["*"]
         exclude-patterns:
           [
             "@esbuild/*",
             "@rollup/*",
             "@swc/*",
             "@types/*",
-            "storybook*",
             "@storybook/*",
-            "eslint-plugin-storybook",
             "vitest",
             "@vitest/*",
           ]
-
+        update-types: ["patch"]
+      minor:
+        applies-to: version-updates
+        patterns: ["*"]
+        exclude-patterns: [
+            "@esbuild/*",
+            "@rollup/*",
+            "@swc/*",
+            "@types/*",
+            "@storybook/*",
+            "vitest",
+            "@vitest/*",
+            # Temporary exclusion of packages below from minor updates
+            "moment-timezone",
+            "monaco-editor",
+          ]
+        update-types: ["minor"]
+      major:
+        applies-to: version-updates
+        patterns: ["*"]
+        exclude-patterns: [
+            "@esbuild/*",
+            "@rollup/*",
+            "@swc/*",
+            "@types/*",
+            "@storybook/*",
+            "vitest",
+            "@vitest/*",
+            # Temporary exclusion of packages below from major updates
+            "eslint-plugin-storybook",
+            "eslint-plugin-vue",
+          ]
+        update-types: ["major"]
     ignore:
-      # Ignore updates to monaco-yaml; version is pinned to 5.3.1 due to patch-package script additions
+      # Ignore updates to monaco-yaml, version is pinned to 5.3.1 due to patch-package script additions
       - dependency-name: "monaco-yaml"
-        versions: [">=5.3.2"]
+        versions:
+          - ">=5.3.2"
 
-      # Ignore updates of version 1.x for vue-virtual-scroller, as the project uses the beta of 2.x
+      # Ignore updates of version 1.x, as we're using the beta of 2.x (still in beta)
       - dependency-name: "vue-virtual-scroller"
-        versions: ["1.x"]
+        versions:
+          - "1.x"

.github/pull_request_template.md

@@ -12,7 +12,7 @@ _Example: Replaces legacy scroll directive with the new API._
 ### 🔗 Related Issue
 
 Which issue does this PR resolve? Use [GitHub Keywords](https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests#linking-a-pull-request-to-an-issue) to automatically link the pull request to the issue.  
-_Example: Closes https://github.com/kestra-io/kestra/issues/ISSUE_NUMBER._
+_Example: Closes https://github.com/kestra-io/kestra/issues/12345._
 
 ### 🎨 Frontend Checklist
 

.github/workflows/auto-translate-ui-keys.yml

@@ -20,7 +20,7 @@ jobs:
     runs-on: ubuntu-latest
     timeout-minutes: 10
     steps:
-        - uses: actions/checkout@v6
+        - uses: actions/checkout@v5
           name: Checkout
           with:
             fetch-depth: 0

.github/workflows/codeql-analysis.yml

@@ -27,7 +27,7 @@ jobs:
 
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v6
+      uses: actions/checkout@v5
       with:
         # We must fetch at least the immediate parents so that if this is
         # a pull request then we can checkout the head.

.github/workflows/global-create-new-release-branch.yml

@@ -33,7 +33,7 @@ jobs:
             exit 1;
           fi
       # Checkout
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v5
         with:
           fetch-depth: 0
           path: kestra

.github/workflows/global-start-release.yml

@@ -39,7 +39,7 @@ jobs:
 
       # Checkout
       - name: Checkout
-        uses: actions/checkout@v6
+        uses: actions/checkout@v5
         with:
           fetch-depth: 0
           token: ${{ secrets.GH_PERSONAL_TOKEN }}

.github/workflows/main-build.yml

@@ -28,7 +28,7 @@ jobs:
     steps:
       # Targeting develop branch from develop
       - name: Trigger EE Workflow (develop push, no payload)
-        uses: peter-evans/repository-dispatch@28959ce8df70de7be546dd1250a005dd32156697
+        uses: peter-evans/repository-dispatch@5fc4efd1a4797ddb68ffd0714a238564e4cc0e6f
         if: ${{ github.event_name == 'push' && github.ref == 'refs/heads/develop' }}
         with:
           token: ${{ secrets.GH_PERSONAL_TOKEN }}
@@ -64,7 +64,6 @@ jobs:
       DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }}
       DOCKERHUB_PASSWORD: ${{ secrets.DOCKERHUB_PASSWORD }}
       SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
-      GH_PERSONAL_TOKEN: ${{ secrets.GH_PERSONAL_TOKEN }}
 
 
   publish-develop-maven:

.github/workflows/pull-request.yml

@@ -16,7 +16,7 @@ jobs:
     - name: Check EE repo for branch with same name
       if: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.repo.fork == false }}
       id: check-ee-branch
-      uses: actions/github-script@v8
+      uses: actions/github-script@v7
       with:
         github-token: ${{ secrets.GH_PERSONAL_TOKEN }}
         script: |
@@ -40,7 +40,7 @@ jobs:
 
       # Targeting pull request (only if not from a fork and EE has no branch with same name)
     - name: Trigger EE Workflow (pull request, with payload)
-      uses: peter-evans/repository-dispatch@28959ce8df70de7be546dd1250a005dd32156697
+      uses: peter-evans/repository-dispatch@5fc4efd1a4797ddb68ffd0714a238564e4cc0e6f
       if: ${{ github.event_name == 'pull_request'
         && github.event.pull_request.number != ''
         && github.event.pull_request.head.repo.fork == false
@@ -50,7 +50,7 @@ jobs:
         repository: kestra-io/kestra-ee
         event-type: "oss-updated"
         client-payload: >-
-          {"commit_sha":"${{ github.event.pull_request.head.sha }}","pr_repo":"${{ github.repository }}"}
+          {"commit_sha":"${{ github.sha }}","pr_repo":"${{ github.repository }}"}
 
   file-changes:
     if: ${{ github.event.pull_request.draft == false }}

.github/workflows/release-docker.yml

@@ -32,4 +32,3 @@ jobs:
       DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }}
       DOCKERHUB_PASSWORD: ${{ secrets.DOCKERHUB_PASSWORD }}
       SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
-      GH_PERSONAL_TOKEN: ${{ secrets.GH_PERSONAL_TOKEN }}

.github/workflows/vulnerabilities-check.yml

@@ -17,7 +17,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       # Checkout
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v5
         with:
           fetch-depth: 0
 
@@ -43,7 +43,7 @@ jobs:
 
         # Upload dependency check report
       - name: Upload dependency check report
-        uses: actions/upload-artifact@v6
+        uses: actions/upload-artifact@v5
         if: ${{ always() }}
         with:
           name: dependency-check-report
@@ -58,7 +58,7 @@ jobs:
       actions: read
     steps:
       # Checkout
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v5
         with:
           fetch-depth: 0
 
@@ -95,7 +95,7 @@ jobs:
       actions: read
     steps:
       # Checkout
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v5
         with:
           fetch-depth: 0
 

build-and-start-e2e-tests.sh

@@ -29,8 +29,8 @@ start_time2=$(date +%s)
 
 echo "cd ./ui"
 cd ./ui
-echo "npm ci"
-npm ci
+echo "npm i"
+npm i
 
 echo 'sh ./run-e2e-tests.sh --kestra-docker-image-to-test "kestra/kestra:$LOCAL_IMAGE_VERSION"'
 ./run-e2e-tests.sh --kestra-docker-image-to-test "kestra/kestra:$LOCAL_IMAGE_VERSION"

build.gradle

@@ -7,7 +7,7 @@ buildscript {
     }
 
     dependencies {
-        classpath "net.e175.klaus:zip-prefixer:0.4.0"
+        classpath "net.e175.klaus:zip-prefixer:0.3.1"
     }
 }
 
@@ -21,7 +21,7 @@ plugins {
 
     // test
     id "com.adarshr.test-logger" version "4.0.0"
-    id "org.sonarqube" version "7.2.1.6560"
+    id "org.sonarqube" version "7.0.1.6134"
     id 'jacoco-report-aggregation'
 
     // helper
@@ -32,7 +32,7 @@ plugins {
 
     // release
     id 'net.researchgate.release' version '3.1.0'
-    id "com.gorylenko.gradle-git-properties" version "2.5.4"
+    id "com.gorylenko.gradle-git-properties" version "2.5.3"
     id 'signing'
     id "com.vanniktech.maven.publish" version "0.35.0"
 
@@ -223,13 +223,13 @@ subprojects {subProj ->
             t.environment 'ENV_TEST2', "Pass by env"
 
 
-//            if (subProj.name == 'core' || subProj.name == 'jdbc-h2' || subProj.name == 'jdbc-mysql' || subProj.name == 'jdbc-postgres') {
-//                // JUnit 5 parallel settings
-//                t.systemProperty 'junit.jupiter.execution.parallel.enabled', 'true'
-//                t.systemProperty 'junit.jupiter.execution.parallel.mode.default', 'concurrent'
-//                t.systemProperty 'junit.jupiter.execution.parallel.mode.classes.default', 'same_thread'
-//                t.systemProperty 'junit.jupiter.execution.parallel.config.strategy', 'dynamic'
-//            }
+            if (subProj.name == 'core' || subProj.name == 'jdbc-h2' || subProj.name == 'jdbc-mysql' || subProj.name == 'jdbc-postgres') {
+                // JUnit 5 parallel settings
+                t.systemProperty 'junit.jupiter.execution.parallel.enabled', 'true'
+                t.systemProperty 'junit.jupiter.execution.parallel.mode.default', 'concurrent'
+                t.systemProperty 'junit.jupiter.execution.parallel.mode.classes.default', 'same_thread'
+                t.systemProperty 'junit.jupiter.execution.parallel.config.strategy', 'dynamic'
+            }
         }
 
         tasks.register('flakyTest', Test) { Test t ->
@@ -331,7 +331,7 @@ subprojects {
         }
 
         dependencies {
-            agent "org.aspectj:aspectjweaver:1.9.25.1"
+            agent "org.aspectj:aspectjweaver:1.9.25"
         }
 
         test {

cli/src/main/java/io/kestra/cli/AbstractCommand.java

@@ -42,7 +42,7 @@ import picocli.CommandLine.Option;
 @Introspected
 public abstract class AbstractCommand implements Callable<Integer> {
     @Inject
-    protected ApplicationContext applicationContext;
+    private ApplicationContext applicationContext;
 
     @Inject
     private EndpointDefaultConfiguration endpointConfiguration;

cli/src/main/java/io/kestra/cli/App.java

@@ -8,10 +8,11 @@ import io.kestra.cli.commands.plugins.PluginCommand;
 import io.kestra.cli.commands.servers.ServerCommand;
 import io.kestra.cli.commands.sys.SysCommand;
 import io.kestra.cli.commands.templates.TemplateCommand;
-import io.kestra.cli.services.EnvironmentProvider;
 import io.micronaut.configuration.picocli.MicronautFactory;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import io.micronaut.context.ApplicationContext;
 import io.micronaut.context.ApplicationContextBuilder;
+import io.micronaut.context.env.Environment;
 import io.micronaut.core.annotation.Introspected;
 import org.slf4j.bridge.SLF4JBridgeHandler;
 import picocli.CommandLine;
@@ -19,9 +20,11 @@ import picocli.CommandLine;
 import java.lang.reflect.Field;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
-import java.util.*;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.Objects;
 import java.util.concurrent.Callable;
-import java.util.stream.Stream;
 
 @CommandLine.Command(
     name = "kestra",
@@ -46,77 +49,35 @@ import java.util.stream.Stream;
 @Introspected
 public class App implements Callable<Integer> {
     public static void main(String[] args) {
-        System.exit(runCli(args));
-    }
-
-    public static int runCli(String[] args, String... extraEnvironments) {
-        return runCli(App.class, args, extraEnvironments);
-    }
-
-    public static int runCli(Class<?> cls, String[] args, String... extraEnvironments) {
-        ServiceLoader<EnvironmentProvider> environmentProviders = ServiceLoader.load(EnvironmentProvider.class);
-        String[] baseEnvironments = environmentProviders.findFirst().map(EnvironmentProvider::getCliEnvironments).orElseGet(() -> new String[0]);
-        return execute(
-            cls,
-            Stream.concat(
-                Arrays.stream(baseEnvironments),
-                Arrays.stream(extraEnvironments)
-            ).toArray(String[]::new),
-            args
-        );
+        execute(App.class, new String [] { Environment.CLI }, args);
     }
 
     @Override
     public Integer call() throws Exception {
-        return runCli(new String[0]);
+        return PicocliRunner.call(App.class, "--help");
     }
 
-    protected static int execute(Class<?> cls, String[] environments, String... args) {
+    protected static void execute(Class<?> cls, String[] environments, String... args) {
         // Log Bridge
         SLF4JBridgeHandler.removeHandlersForRootLogger();
         SLF4JBridgeHandler.install();
 
         // Init ApplicationContext
-        CommandLine commandLine = getCommandLine(cls, args);
-
-        ApplicationContext applicationContext = App.applicationContext(cls, commandLine, environments);
-
-        Class<?> targetCommand = commandLine.getCommandSpec().userObject().getClass();
-
-        if (!AbstractCommand.class.isAssignableFrom(targetCommand) && args.length == 0) {
-            // if no command provided, show help
-            args = new String[]{"--help"};
-        }
+        ApplicationContext applicationContext = App.applicationContext(cls, environments, args);
 
         // Call Picocli command
-        int exitCode;
+        int exitCode = 0;
         try {
              exitCode = new CommandLine(cls, new MicronautFactory(applicationContext)).execute(args);
         } catch (CommandLine.InitializationException e){
-            System.err.println("Could not initialize picocli CommandLine, err: " + e.getMessage());
+            System.err.println("Could not initialize picoli ComandLine, err: " + e.getMessage());
             e.printStackTrace();
             exitCode = 1;
         }
         applicationContext.close();
 
         // exit code
-        return exitCode;
-    }
-
-    private static CommandLine getCommandLine(Class<?> cls, String[] args) {
-        CommandLine cmd = new CommandLine(cls, CommandLine.defaultFactory());
-        continueOnParsingErrors(cmd);
-
-        CommandLine.ParseResult parseResult = cmd.parseArgs(args);
-        List<CommandLine> parsedCommands = parseResult.asCommandLineList();
-
-        return parsedCommands.getLast();
-    }
-
-    public static ApplicationContext applicationContext(Class<?> mainClass,
-                                                        String[] environments,
-                                                        String... args) {
-        return App.applicationContext(mainClass, getCommandLine(mainClass, args), environments);
+        System.exit(Objects.requireNonNullElse(exitCode, 0));
     }
 
 
@@ -124,17 +85,25 @@ public class App implements Callable<Integer> {
      * Create an {@link ApplicationContext} with additional properties based on configuration files (--config) and
      * forced Properties from current command.
      *
+     * @param args args passed to java app
      * @return the application context created
      */
     protected static ApplicationContext applicationContext(Class<?> mainClass,
-                                                           CommandLine commandLine,
-                                                           String[] environments) {
+                                                           String[] environments,
+                                                           String[] args) {
 
         ApplicationContextBuilder builder = ApplicationContext
             .builder()
             .mainClass(mainClass)
             .environments(environments);
 
+        CommandLine cmd = new CommandLine(mainClass, CommandLine.defaultFactory());
+        continueOnParsingErrors(cmd);
+
+        CommandLine.ParseResult parseResult = cmd.parseArgs(args);
+        List<CommandLine> parsedCommands = parseResult.asCommandLineList();
+
+        CommandLine commandLine = parsedCommands.getLast();
         Class<?> cls = commandLine.getCommandSpec().userObject().getClass();
 
         if (AbstractCommand.class.isAssignableFrom(cls)) {

cli/src/main/java/io/kestra/cli/commands/configs/sys/ConfigCommand.java

@@ -1,5 +1,6 @@
 package io.kestra.cli.commands.configs.sys;
 
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.extern.slf4j.Slf4j;
 import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
@@ -19,6 +20,8 @@ public class ConfigCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"configs",  "--help"});
+        PicocliRunner.call(App.class, "configs",  "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/flows/FlowCommand.java

@@ -1,5 +1,6 @@
 package io.kestra.cli.commands.flows;
 
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import lombok.extern.slf4j.Slf4j;
 import io.kestra.cli.AbstractCommand;
@@ -18,8 +19,7 @@ import picocli.CommandLine;
         FlowDotCommand.class,
         FlowExportCommand.class,
         FlowUpdateCommand.class,
-        FlowUpdatesCommand.class,
-        FlowsSyncFromSourceCommand.class
+        FlowUpdatesCommand.class
     }
 )
 @Slf4j
@@ -29,6 +29,8 @@ public class FlowCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"flow",  "--help"});
+        PicocliRunner.call(App.class, "flow",  "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/flows/FlowsSyncFromSourceCommand.java

@@ -1,55 +0,0 @@
-package io.kestra.cli.commands.flows;
-
-import io.kestra.cli.AbstractApiCommand;
-import io.kestra.cli.services.TenantIdSelectorService;
-import io.kestra.core.models.flows.FlowWithSource;
-import io.kestra.core.models.flows.GenericFlow;
-import io.kestra.core.repositories.FlowRepositoryInterface;
-import jakarta.inject.Inject;
-import java.util.List;
-import lombok.extern.slf4j.Slf4j;
-import picocli.CommandLine;
-
-@CommandLine.Command(
-    name = "syncFromSource",
-    description = "Update a single flow",
-    mixinStandardHelpOptions = true
-)
-@Slf4j
-public class FlowsSyncFromSourceCommand extends AbstractApiCommand {
-
-    @Inject
-    private TenantIdSelectorService tenantService;
-
-    @SuppressWarnings("deprecation")
-    @Override
-    public Integer call() throws Exception {
-        super.call();
-
-        FlowRepositoryInterface repository = applicationContext.getBean(FlowRepositoryInterface.class);
-        String tenant = tenantService.getTenantId(tenantId);
-
-        List<FlowWithSource> persistedFlows = repository.findAllWithSource(tenant);
-
-        int count = 0;
-        for (FlowWithSource persistedFlow : persistedFlows) {
-            // Ensure exactly one trailing newline. We need this new line
-            // because when we update a flow from its source,
-            // we don't update it if no change is detected.
-            // The goal here is to force an update from the source for every flows
-            GenericFlow flow = GenericFlow.fromYaml(tenant,persistedFlow.getSource() + System.lineSeparator());
-            repository.update(flow, persistedFlow);
-            stdOut("- %s.%s".formatted(flow.getNamespace(), flow.getId()));
-            count++;
-        }
-        stdOut("%s flow(s) successfully updated!".formatted(count));
-
-        return 0;
-    }
-
-    protected boolean loadExternalPlugins() {
-        return true;
-    }
-
-
-}

cli/src/main/java/io/kestra/cli/commands/flows/namespaces/FlowNamespaceCommand.java

@@ -1,6 +1,7 @@
 package io.kestra.cli.commands.flows.namespaces;
 
 import io.kestra.cli.App;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import lombok.extern.slf4j.Slf4j;
 import io.kestra.cli.AbstractCommand;
@@ -21,6 +22,8 @@ public class FlowNamespaceCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"flow", "namespace",  "--help"});
+        PicocliRunner.call(App.class, "flow", "namespace",  "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/migrations/MigrationCommand.java

@@ -3,6 +3,7 @@ package io.kestra.cli.commands.migrations;
 import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
 import io.kestra.cli.commands.migrations.metadata.MetadataMigrationCommand;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import lombok.extern.slf4j.Slf4j;
 import picocli.CommandLine;
@@ -23,6 +24,8 @@ public class MigrationCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"migrate",  "--help"});
+        PicocliRunner.call(App.class, "migrate",  "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/migrations/metadata/MetadataMigrationCommand.java

@@ -10,8 +10,7 @@ import picocli.CommandLine;
     description = "populate metadata for entities",
     subcommands = {
         KvMetadataMigrationCommand.class,
-        SecretsMetadataMigrationCommand.class,
-        NsFilesMetadataMigrationCommand.class
+        SecretsMetadataMigrationCommand.class
     }
 )
 @Slf4j

cli/src/main/java/io/kestra/cli/commands/migrations/metadata/MetadataMigrationService.java

@@ -1,51 +1,47 @@
 package io.kestra.cli.commands.migrations.metadata;
 
-import com.google.common.annotations.VisibleForTesting;
 import io.kestra.core.models.kv.PersistedKvMetadata;
-import io.kestra.core.models.namespaces.files.NamespaceFileMetadata;
 import io.kestra.core.repositories.FlowRepositoryInterface;
 import io.kestra.core.repositories.KvMetadataRepositoryInterface;
-import io.kestra.core.repositories.NamespaceFileMetadataRepositoryInterface;
 import io.kestra.core.storages.FileAttributes;
 import io.kestra.core.storages.StorageContext;
 import io.kestra.core.storages.StorageInterface;
 import io.kestra.core.storages.kv.InternalKVStore;
 import io.kestra.core.storages.kv.KVEntry;
 import io.kestra.core.tenant.TenantService;
-import io.kestra.core.utils.NamespaceUtils;
+import jakarta.inject.Inject;
 import jakarta.inject.Singleton;
-import lombok.AllArgsConstructor;
 
 import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.net.URI;
-import java.nio.file.NoSuchFileException;
 import java.time.Instant;
-import java.util.*;
-import java.util.function.Function;
+import java.util.Collections;
+import java.util.List;
+import java.util.Map;
+import java.util.Optional;
 import java.util.stream.Collectors;
-import java.util.stream.Stream;
 
 import static io.kestra.core.utils.Rethrow.throwConsumer;
 import static io.kestra.core.utils.Rethrow.throwFunction;
 
 @Singleton
-@AllArgsConstructor
 public class MetadataMigrationService {
-    protected FlowRepositoryInterface flowRepository;
-    protected TenantService tenantService;
-    protected KvMetadataRepositoryInterface kvMetadataRepository;
-    protected NamespaceFileMetadataRepositoryInterface namespaceFileMetadataRepository;
-    protected StorageInterface storageInterface;
-    protected NamespaceUtils namespaceUtils;
+    @Inject
+    private TenantService tenantService;
 
-    @VisibleForTesting
-    public Map<String, List<String>> namespacesPerTenant() {
+    @Inject
+    private FlowRepositoryInterface flowRepository;
+
+    @Inject
+    private KvMetadataRepositoryInterface kvMetadataRepository;
+
+    @Inject
+    private StorageInterface storageInterface;
+
+    protected Map<String, List<String>> namespacesPerTenant() {
         String tenantId = tenantService.resolveTenant();
-        return Map.of(tenantId, Stream.concat(
-            Stream.of(namespaceUtils.getSystemFlowNamespace()),
-            flowRepository.findDistinctNamespace(tenantId).stream()
-        ).map(NamespaceUtils::asTree).flatMap(Collection::stream).distinct().toList());
+        return Map.of(tenantId, flowRepository.findDistinctNamespace(tenantId));
     }
 
     public void kvMigration() throws IOException {
@@ -53,9 +49,7 @@ public class MetadataMigrationService {
             .flatMap(namespacesForTenant -> namespacesForTenant.getValue().stream().map(namespace -> Map.entry(namespacesForTenant.getKey(), namespace)))
             .flatMap(throwFunction(namespaceForTenant -> {
                 InternalKVStore kvStore = new InternalKVStore(namespaceForTenant.getKey(), namespaceForTenant.getValue(), storageInterface, kvMetadataRepository);
-                List<FileAttributes> list = listAllFromStorage(storageInterface, StorageContext::kvPrefix, namespaceForTenant.getKey(), namespaceForTenant.getValue()).stream()
-                    .map(PathAndAttributes::attributes)
-                    .toList();
+                List<FileAttributes> list = listAllFromStorage(storageInterface, namespaceForTenant.getKey(), namespaceForTenant.getValue());
                 Map<Boolean, List<KVEntry>> entriesByIsExpired = list.stream()
                     .map(throwFunction(fileAttributes -> KVEntry.from(namespaceForTenant.getValue(), fileAttributes)))
                     .collect(Collectors.partitioningBy(kvEntry -> Optional.ofNullable(kvEntry.expirationDate()).map(expirationDate -> Instant.now().isAfter(expirationDate)).orElse(false)));
@@ -81,39 +75,15 @@ public class MetadataMigrationService {
             }));
     }
 
-    public void nsFilesMigration() throws IOException {
-        this.namespacesPerTenant().entrySet().stream()
-            .flatMap(namespacesForTenant -> namespacesForTenant.getValue().stream().map(namespace -> Map.entry(namespacesForTenant.getKey(), namespace)))
-            .flatMap(throwFunction(namespaceForTenant -> {
-                List<PathAndAttributes> list = listAllFromStorage(storageInterface, StorageContext::namespaceFilePrefix, namespaceForTenant.getKey(), namespaceForTenant.getValue());
-                return list.stream()
-                    .map(pathAndAttributes -> NamespaceFileMetadata.of(namespaceForTenant.getKey(), namespaceForTenant.getValue(), pathAndAttributes.path(), pathAndAttributes.attributes()));
-            }))
-            .forEach(throwConsumer(nsFileMetadata -> {
-                if (namespaceFileMetadataRepository.findByPath(nsFileMetadata.getTenantId(), nsFileMetadata.getNamespace(), nsFileMetadata.getPath()).isEmpty()) {
-                    namespaceFileMetadataRepository.save(nsFileMetadata);
-                }
-            }));
-    }
-
     public void secretMigration() throws Exception {
         throw new UnsupportedOperationException("Secret migration is not needed in the OSS version");
     }
 
-    private static List<PathAndAttributes> listAllFromStorage(StorageInterface storage, Function<String, String> prefixFunction, String tenant, String namespace) throws IOException {
+    private static List<FileAttributes> listAllFromStorage(StorageInterface storage, String tenant, String namespace) throws IOException {
         try {
-            String prefix = prefixFunction.apply(namespace);
-            if (!storage.exists(tenant, namespace, URI.create(StorageContext.KESTRA_PROTOCOL + prefix))) {
-                return Collections.emptyList();
-            }
-
-            return storage.allByPrefix(tenant, namespace, URI.create(StorageContext.KESTRA_PROTOCOL + prefix + "/"), true).stream()
-                .map(throwFunction(uri -> new PathAndAttributes(uri.getPath().substring(prefix.length()), storage.getAttributes(tenant, namespace, uri))))
-                .toList();
-        } catch (FileNotFoundException | NoSuchFileException e) {
+            return storage.list(tenant, namespace, URI.create(StorageContext.KESTRA_PROTOCOL + StorageContext.kvPrefix(namespace)));
+        } catch (FileNotFoundException e) {
             return Collections.emptyList();
         }
     }
-
-    public record PathAndAttributes(String path, FileAttributes attributes) {}
 }

cli/src/main/java/io/kestra/cli/commands/migrations/metadata/NsFilesMetadataMigrationCommand.java

@@ -1,31 +0,0 @@
-package io.kestra.cli.commands.migrations.metadata;
-
-import io.kestra.cli.AbstractCommand;
-import jakarta.inject.Inject;
-import jakarta.inject.Provider;
-import lombok.extern.slf4j.Slf4j;
-import picocli.CommandLine;
-
-@CommandLine.Command(
-    name = "nsfiles",
-    description = "populate metadata for Namespace Files"
-)
-@Slf4j
-public class NsFilesMetadataMigrationCommand extends AbstractCommand {
-    @Inject
-    private Provider<MetadataMigrationService> metadataMigrationServiceProvider;
-
-    @Override
-    public Integer call() throws Exception {
-        super.call();
-        try {
-            metadataMigrationServiceProvider.get().nsFilesMigration();
-        } catch (Exception e) {
-            System.err.println("❌ Namespace Files Metadata migration failed: " + e.getMessage());
-            e.printStackTrace();
-            return 1;
-        }
-        System.out.println("✅ Namespace Files Metadata migration complete.");
-        return 0;
-    }
-}

cli/src/main/java/io/kestra/cli/commands/namespaces/NamespaceCommand.java

@@ -4,6 +4,7 @@ import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
 import io.kestra.cli.commands.namespaces.files.NamespaceFilesCommand;
 import io.kestra.cli.commands.namespaces.kv.KvCommand;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import lombok.extern.slf4j.Slf4j;
 import picocli.CommandLine;
@@ -24,6 +25,8 @@ public class NamespaceCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"namespace", "--help"});
+        PicocliRunner.call(App.class, "namespace", "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/namespaces/files/NamespaceFilesCommand.java

@@ -2,6 +2,7 @@ package io.kestra.cli.commands.namespaces.files;
 
 import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import lombok.extern.slf4j.Slf4j;
 import picocli.CommandLine;
@@ -21,6 +22,8 @@ public class NamespaceFilesCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"namespace", "files",  "--help"});
+        PicocliRunner.call(App.class, "namespace", "files",  "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/namespaces/kv/KvCommand.java

@@ -2,6 +2,7 @@ package io.kestra.cli.commands.namespaces.kv;
 
 import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import lombok.extern.slf4j.Slf4j;
 import picocli.CommandLine;
@@ -21,6 +22,8 @@ public class KvCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"namespace", "kv",  "--help"});
+        PicocliRunner.call(App.class, "namespace", "kv",  "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/plugins/PluginCommand.java

@@ -2,6 +2,7 @@ package io.kestra.cli.commands.plugins;
 
 import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import picocli.CommandLine.Command;
 
@@ -24,7 +25,9 @@ public class PluginCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"plugins", "--help"});
+        PicocliRunner.call(App.class, "plugins", "--help");
+
+        return 0;
     }
 
     @Override

cli/src/main/java/io/kestra/cli/commands/servers/ServerCommand.java

@@ -1,5 +1,6 @@
 package io.kestra.cli.commands.servers;
 
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import lombok.extern.slf4j.Slf4j;
 import io.kestra.cli.AbstractCommand;
@@ -27,6 +28,8 @@ public class ServerCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"server",  "--help"});
+        PicocliRunner.call(App.class, "server",  "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/sys/SysCommand.java

@@ -2,6 +2,7 @@ package io.kestra.cli.commands.sys;
 
 import io.kestra.cli.commands.sys.database.DatabaseCommand;
 import io.kestra.cli.commands.sys.statestore.StateStoreCommand;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.extern.slf4j.Slf4j;
 import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
@@ -24,6 +25,8 @@ public class SysCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"sys",  "--help"});
+        PicocliRunner.call(App.class, "sys",  "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/sys/database/DatabaseCommand.java

@@ -2,6 +2,7 @@ package io.kestra.cli.commands.sys.database;
 
 import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import picocli.CommandLine;
 
@@ -19,6 +20,8 @@ public class DatabaseCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"sys", "database", "--help"});
+        PicocliRunner.call(App.class, "sys", "database", "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/sys/statestore/StateStoreCommand.java

@@ -2,6 +2,7 @@ package io.kestra.cli.commands.sys.statestore;
 
 import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import picocli.CommandLine;
 
@@ -19,6 +20,8 @@ public class StateStoreCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"sys", "state-store", "--help"});
+        PicocliRunner.call(App.class, "sys", "state-store", "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/sys/statestore/StateStoreMigrateCommand.java

@@ -57,7 +57,7 @@ public class StateStoreMigrateCommand extends AbstractCommand {
             String taskRunValue = statesUriPart.length > 2 ? statesUriPart[1] : null;
             String stateSubName = statesUriPart[statesUriPart.length - 1];
             boolean flowScoped = flowQualifierWithStateQualifiers[0].endsWith("/" + flow.getId());
-            StateStore stateStore = new StateStore(runContextFactory.of(flow, Map.of()), false);
+            StateStore stateStore = new StateStore(runContext(runContextFactory, flow), false);
 
             try (InputStream is = storageInterface.get(flow.getTenantId(), flow.getNamespace(), stateStoreFileUri)) {
                 stateStore.putState(flowScoped, stateName, stateSubName, taskRunValue, is.readAllBytes());
@@ -70,4 +70,12 @@ public class StateStoreMigrateCommand extends AbstractCommand {
         stdOut("Successfully ran the state-store migration.");
         return 0;
     }
+
+    private RunContext runContext(RunContextFactory runContextFactory, Flow flow) {
+        Map<String, String> flowVariables = new HashMap<>();
+        flowVariables.put("tenantId", flow.getTenantId());
+        flowVariables.put("id", flow.getId());
+        flowVariables.put("namespace", flow.getNamespace());
+        return runContextFactory.of(flow, Map.of("flow", flowVariables));
+    }
 }

cli/src/main/java/io/kestra/cli/commands/templates/TemplateCommand.java

@@ -4,6 +4,7 @@ import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
 import io.kestra.cli.commands.templates.namespaces.TemplateNamespaceCommand;
 import io.kestra.core.models.templates.TemplateEnabled;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import lombok.extern.slf4j.Slf4j;
 import picocli.CommandLine;
@@ -26,6 +27,8 @@ public class TemplateCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"template",  "--help"});
+        PicocliRunner.call(App.class, "template",  "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/commands/templates/namespaces/TemplateNamespaceCommand.java

@@ -3,6 +3,7 @@ package io.kestra.cli.commands.templates.namespaces;
 import io.kestra.cli.AbstractCommand;
 import io.kestra.cli.App;
 import io.kestra.core.models.templates.TemplateEnabled;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import lombok.SneakyThrows;
 import lombok.extern.slf4j.Slf4j;
 import picocli.CommandLine;
@@ -23,6 +24,8 @@ public class TemplateNamespaceCommand extends AbstractCommand {
     public Integer call() throws Exception {
         super.call();
 
-        return App.runCli(new String[]{"template", "namespace", "--help"});
+        PicocliRunner.call(App.class, "template", "namespace", "--help");
+
+        return 0;
     }
 }

cli/src/main/java/io/kestra/cli/services/DefaultEnvironmentProvider.java

@@ -1,16 +0,0 @@
-package io.kestra.cli.services;
-
-import io.micronaut.context.env.Environment;
-
-import java.util.Arrays;
-import java.util.stream.Stream;
-
-public class DefaultEnvironmentProvider implements EnvironmentProvider {
-    @Override
-    public String[] getCliEnvironments(String... extraEnvironments) {
-        return Stream.concat(
-            Stream.of(Environment.CLI),
-            Arrays.stream(extraEnvironments)
-        ).toArray(String[]::new);
-    }
-}

cli/src/main/java/io/kestra/cli/services/EnvironmentProvider.java

@@ -1,5 +0,0 @@
-package io.kestra.cli.services;
-
-public interface EnvironmentProvider {
-    String[] getCliEnvironments(String... extraEnvironments);
-}

cli/src/main/resources/META-INF/services/io.kestra.cli.services.EnvironmentProvider

@@ -1 +0,0 @@
-io.kestra.cli.services.DefaultEnvironmentProvider

cli/src/test/java/io/kestra/cli/AppTest.java

@@ -1,11 +1,14 @@
 package io.kestra.cli;
 
 import io.kestra.core.models.ServerType;
+import io.micronaut.configuration.picocli.MicronautFactory;
+import io.micronaut.configuration.picocli.PicocliRunner;
 import io.micronaut.context.ApplicationContext;
 import io.micronaut.context.env.Environment;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.ValueSource;
+import picocli.CommandLine;
 
 import java.io.ByteArrayOutputStream;
 import java.io.PrintStream;
@@ -19,15 +22,11 @@ class AppTest {
         ByteArrayOutputStream out = new ByteArrayOutputStream();
         System.setOut(new PrintStream(out));
 
-        // No arg will print help
-        assertThat(App.runCli(new String[0])).isZero();
-        assertThat(out.toString()).contains("kestra");
+        try (ApplicationContext ctx = ApplicationContext.run(Environment.CLI, Environment.TEST)) {
+            PicocliRunner.call(App.class, ctx, "--help");
 
-        out.reset();
-
-        // Explicit help command
-        assertThat(App.runCli(new String[]{"--help"})).isZero();
-        assertThat(out.toString()).contains("kestra");
+            assertThat(out.toString()).contains("kestra");
+        }
     }
 
     @ParameterizedTest
@@ -39,12 +38,11 @@ class AppTest {
         final String[] args = new String[]{"server", serverType, "--help"};
 
         try (ApplicationContext ctx = App.applicationContext(App.class, new String [] { Environment.CLI }, args)) {
+            new CommandLine(App.class, new MicronautFactory(ctx)).execute(args);
+
             assertTrue(ctx.getProperty("kestra.server-type", ServerType.class).isEmpty());
+            assertThat(out.toString()).startsWith("Usage: kestra server " + serverType);
         }
-
-        assertThat(App.runCli(args)).isZero();
-
-        assertThat(out.toString()).startsWith("Usage: kestra server " + serverType);
     }
 
     @Test
@@ -54,10 +52,12 @@ class AppTest {
 
         final String[] argsWithMissingParams = new String[]{"flow", "namespace", "update"};
 
-        assertThat(App.runCli(argsWithMissingParams)).isEqualTo(2);
+        try (ApplicationContext ctx = App.applicationContext(App.class, new String [] { Environment.CLI }, argsWithMissingParams)) {
+            new CommandLine(App.class, new MicronautFactory(ctx)).execute(argsWithMissingParams);
 
-        assertThat(out.toString()).startsWith("Missing required parameters: ");
-        assertThat(out.toString()).contains("Usage: kestra flow namespace update ");
-        assertThat(out.toString()).doesNotContain("MissingParameterException: ");
+            assertThat(out.toString()).startsWith("Missing required parameters: ");
+            assertThat(out.toString()).contains("Usage: kestra flow namespace update ");
+            assertThat(out.toString()).doesNotContain("MissingParameterException: ");
+        }
     }
 }

cli/src/test/java/io/kestra/cli/commands/flows/FlowsSyncFromSourceCommandTest.java

@@ -1,73 +0,0 @@
-package io.kestra.cli.commands.flows;
-
-import static io.kestra.core.tenant.TenantService.MAIN_TENANT;
-import static org.assertj.core.api.Assertions.assertThat;
-
-import io.kestra.core.models.flows.Flow;
-import io.kestra.core.repositories.FlowRepositoryInterface;
-import io.micronaut.configuration.picocli.PicocliRunner;
-import io.micronaut.context.ApplicationContext;
-import io.micronaut.context.env.Environment;
-import io.micronaut.runtime.server.EmbeddedServer;
-import java.io.ByteArrayOutputStream;
-import java.io.PrintStream;
-import java.net.URL;
-import java.util.List;
-import org.junit.jupiter.api.Test;
-
-class FlowsSyncFromSourceCommandTest {
-    @Test
-    void updateAllFlowsFromSource()  {
-        URL directory = FlowUpdatesCommandTest.class.getClassLoader().getResource("flows");
-        ByteArrayOutputStream out = new ByteArrayOutputStream();
-        System.setOut(new PrintStream(out));
-        try (ApplicationContext ctx = ApplicationContext.run(Environment.CLI, Environment.TEST)) {
-
-            EmbeddedServer embeddedServer = ctx.getBean(EmbeddedServer.class);
-            embeddedServer.start();
-
-            String[] args = {
-                "--plugins",
-                "/tmp", // pass this arg because it can cause failure
-                "--server",
-                embeddedServer.getURL().toString(),
-                "--user",
-                "myuser:pass:word",
-                "--delete",
-                directory.getPath(),
-            };
-            PicocliRunner.call(FlowUpdatesCommand.class, ctx, args);
-
-            assertThat(out.toString()).contains("successfully updated !");
-            out.reset();
-
-            FlowRepositoryInterface repository = ctx.getBean(FlowRepositoryInterface.class);
-            List<Flow> flows = repository.findAll(MAIN_TENANT);
-            for (Flow flow : flows) {
-                assertThat(flow.getRevision()).isEqualTo(1);
-            }
-
-            args = new String[]{
-                "--plugins",
-                "/tmp", // pass this arg because it can cause failure
-                "--server",
-                embeddedServer.getURL().toString(),
-                "--user",
-                "myuser:pass:word"
-
-            };
-            PicocliRunner.call(FlowsSyncFromSourceCommand.class, ctx, args);
-
-            assertThat(out.toString()).contains("4 flow(s) successfully updated!");
-            assertThat(out.toString()).contains("- io.kestra.outsider.quattro");
-            assertThat(out.toString()).contains("- io.kestra.cli.second");
-            assertThat(out.toString()).contains("- io.kestra.cli.third");
-            assertThat(out.toString()).contains("- io.kestra.cli.first");
-
-            flows = repository.findAll(MAIN_TENANT);
-            for (Flow flow : flows) {
-                assertThat(flow.getRevision()).isEqualTo(2);
-            }
-        }
-    }
-}

cli/src/test/java/io/kestra/cli/commands/migrations/metadata/MetadataMigrationServiceTest.java

@@ -1,57 +0,0 @@
-package io.kestra.cli.commands.migrations.metadata;
-
-import io.kestra.core.repositories.FlowRepositoryInterface;
-import io.kestra.core.tenant.TenantService;
-import io.kestra.core.utils.NamespaceUtils;
-import io.kestra.core.utils.TestsUtils;
-import org.junit.jupiter.api.Test;
-import org.mockito.Mockito;
-
-import java.util.Collection;
-import java.util.List;
-import java.util.Map;
-import java.util.stream.Stream;
-
-import static org.assertj.core.api.Assertions.assertThat;
-
-public class MetadataMigrationServiceTest<T extends MetadataMigrationService> {
-    private static final String TENANT_ID = TestsUtils.randomTenant();
-
-    protected static final String SYSTEM_NAMESPACE = "my.system.namespace";
-
-    @Test
-    void namespacesPerTenant() {
-        Map<String, List<String>> expected = getNamespacesPerTenant();
-        Map<String, List<String>> result = metadataMigrationService(
-            expected
-        ).namespacesPerTenant();
-
-        assertThat(result).hasSize(expected.size());
-        expected.forEach((tenantId, namespaces) -> {
-            assertThat(result.get(tenantId)).containsExactlyInAnyOrderElementsOf(
-                Stream.concat(
-                    Stream.of(SYSTEM_NAMESPACE),
-                    namespaces.stream()
-                ).map(NamespaceUtils::asTree).flatMap(Collection::stream).distinct().toList()
-            );
-        });
-    }
-
-    protected Map<String, List<String>> getNamespacesPerTenant() {
-        return Map.of(TENANT_ID, List.of("my.first.namespace", "my.second.namespace", "another.namespace"));
-    }
-
-    protected T metadataMigrationService(Map<String, List<String>> namespacesPerTenant) {
-        FlowRepositoryInterface mockedFlowRepository = Mockito.mock(FlowRepositoryInterface.class);
-        Mockito.doAnswer((params) -> namespacesPerTenant.get(params.getArgument(0).toString())).when(mockedFlowRepository).findDistinctNamespace(Mockito.anyString());
-        NamespaceUtils namespaceUtils = Mockito.mock(NamespaceUtils.class);
-        Mockito.when(namespaceUtils.getSystemFlowNamespace()).thenReturn(SYSTEM_NAMESPACE);
-        //noinspection unchecked
-        return ((T) new MetadataMigrationService(mockedFlowRepository, new TenantService() {
-            @Override
-            public String resolveTenant() {
-                return TENANT_ID;
-            }
-        }, null, null, null, namespaceUtils));
-    }
-}

cli/src/test/java/io/kestra/cli/commands/migrations/metadata/NsFilesMetadataMigrationCommandTest.java

@@ -1,175 +0,0 @@
-package io.kestra.cli.commands.migrations.metadata;
-
-import io.kestra.cli.App;
-import io.kestra.core.exceptions.ResourceExpiredException;
-import io.kestra.core.models.flows.Flow;
-import io.kestra.core.models.flows.GenericFlow;
-import io.kestra.core.models.kv.PersistedKvMetadata;
-import io.kestra.core.models.namespaces.files.NamespaceFileMetadata;
-import io.kestra.core.repositories.FlowRepositoryInterface;
-import io.kestra.core.repositories.KvMetadataRepositoryInterface;
-import io.kestra.core.repositories.NamespaceFileMetadataRepositoryInterface;
-import io.kestra.core.serializers.JacksonMapper;
-import io.kestra.core.storages.*;
-import io.kestra.core.storages.kv.*;
-import io.kestra.core.tenant.TenantService;
-import io.kestra.core.utils.TestsUtils;
-import io.kestra.plugin.core.log.Log;
-import io.micronaut.configuration.picocli.PicocliRunner;
-import io.micronaut.context.ApplicationContext;
-import io.micronaut.context.env.Environment;
-import io.micronaut.core.annotation.NonNull;
-import org.junit.jupiter.api.Test;
-
-import java.io.*;
-import java.net.URI;
-import java.nio.charset.StandardCharsets;
-import java.nio.file.Path;
-import java.time.Duration;
-import java.time.Instant;
-import java.util.List;
-import java.util.Optional;
-
-import static org.assertj.core.api.Assertions.assertThat;
-import static org.assertj.core.api.Assertions.assertThatThrownBy;
-
-public class NsFilesMetadataMigrationCommandTest {
-    @Test
-    void run() throws IOException {
-        ByteArrayOutputStream out = new ByteArrayOutputStream();
-        System.setOut(new PrintStream(out));
-        ByteArrayOutputStream err = new ByteArrayOutputStream();
-        System.setErr(new PrintStream(err));
-
-        try (ApplicationContext ctx = ApplicationContext.run(Environment.CLI, Environment.TEST)) {
-            /* Initial setup:
-            * - namespace 1: my/path, value
-            * - namespace 1: another/path
-            * - namespace 2: yet/another/path
-            * - Nothing in database */
-            String namespace = TestsUtils.randomNamespace();
-            String path = "/my/path";
-            StorageInterface storage = ctx.getBean(StorageInterface.class);
-            String value = "someValue";
-            putOldNsFile(storage, namespace, path, value);
-
-            String anotherPath = "/another/path";
-            String anotherValue = "anotherValue";
-            putOldNsFile(storage, namespace, anotherPath, anotherValue);
-
-            String anotherNamespace = TestsUtils.randomNamespace();
-            String yetAnotherPath = "/yet/another/path";
-            String yetAnotherValue = "yetAnotherValue";
-            putOldNsFile(storage, anotherNamespace, yetAnotherPath, yetAnotherValue);
-
-            NamespaceFileMetadataRepositoryInterface namespaceFileMetadataRepository = ctx.getBean(NamespaceFileMetadataRepositoryInterface.class);
-            String tenantId = TenantService.MAIN_TENANT;
-            assertThat(namespaceFileMetadataRepository.findByPath(tenantId, namespace, path).isPresent()).isFalse();
-
-            /* Expected outcome from the migration command:
-            * - no namespace files has been migrated because no flow exist in the namespace so they are not picked up because we don't know they exist */
-            String[] nsFilesMetadataMigrationCommand = {
-                "migrate", "metadata", "nsfiles"
-            };
-            PicocliRunner.call(App.class, ctx, nsFilesMetadataMigrationCommand);
-
-
-            assertThat(out.toString()).contains("✅ Namespace Files Metadata migration complete.");
-            // Still it's not in the metadata repository because no flow exist to find that namespace file
-            assertThat(namespaceFileMetadataRepository.findByPath(tenantId, namespace, path).isPresent()).isFalse();
-            assertThat(namespaceFileMetadataRepository.findByPath(tenantId, namespace, anotherPath).isPresent()).isFalse();
-            assertThat(namespaceFileMetadataRepository.findByPath(tenantId, anotherNamespace, yetAnotherPath).isPresent()).isFalse();
-
-            // A flow is created from namespace 1, so the namespace files in this namespace should be migrated
-            FlowRepositoryInterface flowRepository = ctx.getBean(FlowRepositoryInterface.class);
-            flowRepository.create(GenericFlow.of(Flow.builder()
-                .tenantId(tenantId)
-                .id("a-flow")
-                .namespace(namespace)
-                .tasks(List.of(Log.builder().id("log").type(Log.class.getName()).message("logging").build()))
-                .build()));
-
-            /* We run the migration again:
-            * - namespace 1 my/path file is seen and metadata is migrated to database
-            * - namespace 1 another/path file is seen and metadata is migrated to database
-            * - namespace 2 yet/another/path is not seen because no flow exist in this namespace */
-            out.reset();
-            PicocliRunner.call(App.class, ctx, nsFilesMetadataMigrationCommand);
-
-            assertThat(out.toString()).contains("✅ Namespace Files Metadata migration complete.");
-            Optional<NamespaceFileMetadata> foundNsFile = namespaceFileMetadataRepository.findByPath(tenantId, namespace, path);
-            assertThat(foundNsFile.isPresent()).isTrue();
-            assertThat(foundNsFile.get().getVersion()).isEqualTo(1);
-            assertThat(foundNsFile.get().getSize()).isEqualTo(value.length());
-
-            Optional<NamespaceFileMetadata> anotherFoundNsFile = namespaceFileMetadataRepository.findByPath(tenantId, namespace, anotherPath);
-            assertThat(anotherFoundNsFile.isPresent()).isTrue();
-            assertThat(anotherFoundNsFile.get().getVersion()).isEqualTo(1);
-            assertThat(anotherFoundNsFile.get().getSize()).isEqualTo(anotherValue.length());
-
-            NamespaceFactory namespaceFactory = ctx.getBean(NamespaceFactory.class);
-            Namespace namespaceStorage = namespaceFactory.of(tenantId, namespace, storage);
-            FileAttributes nsFileRawMetadata = namespaceStorage.getFileMetadata(Path.of(path));
-            assertThat(nsFileRawMetadata.getSize()).isEqualTo(value.length());
-            assertThat(new String(namespaceStorage.getFileContent(Path.of(path)).readAllBytes())).isEqualTo(value);
-
-            FileAttributes anotherNsFileRawMetadata = namespaceStorage.getFileMetadata(Path.of(anotherPath));
-            assertThat(anotherNsFileRawMetadata.getSize()).isEqualTo(anotherValue.length());
-            assertThat(new String(namespaceStorage.getFileContent(Path.of(anotherPath)).readAllBytes())).isEqualTo(anotherValue);
-
-            assertThat(namespaceFileMetadataRepository.findByPath(tenantId, anotherNamespace, yetAnotherPath).isPresent()).isFalse();
-            assertThatThrownBy(() -> namespaceStorage.getFileMetadata(Path.of(yetAnotherPath))).isInstanceOf(FileNotFoundException.class);
-
-            /* We run one last time the migration without any change to verify that we don't resave an existing metadata.
-            * It covers the case where user didn't perform the migrate command yet but they played and added some KV from the UI (so those ones will already be in metadata database). */
-            out.reset();
-            PicocliRunner.call(App.class, ctx, nsFilesMetadataMigrationCommand);
-
-            assertThat(out.toString()).contains("✅ Namespace Files Metadata migration complete.");
-            foundNsFile = namespaceFileMetadataRepository.findByPath(tenantId, namespace, path);
-            assertThat(foundNsFile.get().getVersion()).isEqualTo(1);
-        }
-    }
-
-    @Test
-    void namespaceWithoutNsFile() {
-        ByteArrayOutputStream out = new ByteArrayOutputStream();
-        System.setOut(new PrintStream(out));
-        ByteArrayOutputStream err = new ByteArrayOutputStream();
-        System.setErr(new PrintStream(err));
-
-        try (ApplicationContext ctx = ApplicationContext.run(Environment.CLI, Environment.TEST)) {
-            String tenantId = TenantService.MAIN_TENANT;
-            String namespace = TestsUtils.randomNamespace();
-
-            // A flow is created from namespace 1, so the namespace files in this namespace should be migrated
-            FlowRepositoryInterface flowRepository = ctx.getBean(FlowRepositoryInterface.class);
-            flowRepository.create(GenericFlow.of(Flow.builder()
-                .tenantId(tenantId)
-                .id("a-flow")
-                .namespace(namespace)
-                .tasks(List.of(Log.builder().id("log").type(Log.class.getName()).message("logging").build()))
-                .build()));
-
-            String[] nsFilesMetadataMigrationCommand = {
-                "migrate", "metadata", "nsfiles"
-            };
-            PicocliRunner.call(App.class, ctx, nsFilesMetadataMigrationCommand);
-
-            assertThat(out.toString()).contains("✅ Namespace Files Metadata migration complete.");
-            assertThat(err.toString()).doesNotContain("java.nio.file.NoSuchFileException");
-        }
-    }
-
-    private static void putOldNsFile(StorageInterface storage, String namespace, String path, String value) throws IOException {
-        URI nsFileStorageUri = getNsFileStorageUri(namespace, path);
-        storage.put(TenantService.MAIN_TENANT, namespace, nsFileStorageUri, new StorageObject(
-            null,
-            new ByteArrayInputStream(value.getBytes(StandardCharsets.UTF_8))
-        ));
-    }
-
-    private static @NonNull URI getNsFileStorageUri(String namespace, String path) {
-        return URI.create(StorageContext.KESTRA_PROTOCOL + StorageContext.namespaceFilePrefix(namespace) + path);
-    }
-}

cli/src/test/java/io/kestra/cli/commands/sys/statestore/StateStoreMigrateCommandTest.java

@@ -55,7 +55,11 @@ class StateStoreMigrateCommandTest {
             );
             assertThat(storage.exists(tenantId, flow.getNamespace(), oldStateStoreUri)).isTrue();
 
-            RunContext runContext = ctx.getBean(RunContextFactory.class).of(flow, Map.of());
+            RunContext runContext = ctx.getBean(RunContextFactory.class).of(flow, Map.of("flow", Map.of(
+                "tenantId", tenantId,
+                "id", flow.getId(),
+                "namespace", flow.getNamespace()
+            )));
             StateStore stateStore = new StateStore(runContext, true);
             Assertions.assertThrows(MigrationRequiredException.class, () -> stateStore.getState(true, "my-state", "sub-name", "my-taskrun-value"));
 

cli/src/test/java/io/kestra/cli/services/FileChangedEventListenerTest.java

@@ -19,6 +19,7 @@ import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.TimeoutException;
 import java.util.concurrent.atomic.AtomicBoolean;
+import org.junitpioneer.jupiter.RetryingTest;
 
 import static io.kestra.core.utils.Rethrow.throwRunnable;
 import static org.assertj.core.api.Assertions.assertThat;
@@ -58,7 +59,7 @@ class FileChangedEventListenerTest {
     }
 
     @FlakyTest
-    @Test
+    @RetryingTest(2)
     void test() throws IOException, TimeoutException {
         var tenant = TestsUtils.randomTenant(FileChangedEventListenerTest.class.getSimpleName(), "test");
         // remove the flow if it already exists
@@ -97,7 +98,7 @@ class FileChangedEventListenerTest {
     }
 
     @FlakyTest
-    @Test
+    @RetryingTest(2)
     void testWithPluginDefault() throws IOException, TimeoutException {
         var tenant = TestsUtils.randomTenant(FileChangedEventListenerTest.class.getName(), "testWithPluginDefault");
         // remove the flow if it already exists
@@ -137,4 +138,4 @@ class FileChangedEventListenerTest {
             Duration.ofSeconds(10)
         );
     }
-}
+}

cli/src/test/resources/application-test.yml

@@ -21,7 +21,6 @@ kestra:
   server:
     liveness:
       enabled: false
-    termination-grace-period: 5s
 micronaut:
   http:
     services:

core/build.gradle

@@ -24,9 +24,6 @@ dependencies {
     // reactor
     api "io.projectreactor:reactor-core"
 
-    // awaitility
-    api 'org.awaitility:awaitility'
-
     // micronaut
     api "io.micronaut.data:micronaut-data-model"
     implementation "io.micronaut:micronaut-http-server-netty"
@@ -85,8 +82,8 @@ dependencies {
     testImplementation "io.micronaut:micronaut-http-server-netty"
     testImplementation "io.micronaut:micronaut-management"
 
-    testImplementation "org.testcontainers:testcontainers:1.21.4"
-    testImplementation "org.testcontainers:junit-jupiter:1.21.4"
+    testImplementation "org.testcontainers:testcontainers:1.21.3"
+    testImplementation "org.testcontainers:junit-jupiter:1.21.3"
     testImplementation "org.bouncycastle:bcpkix-jdk18on"
 
     testImplementation "org.wiremock:wiremock-jetty12"

core/src/main/java/io/kestra/core/contexts/KestraContext.java

@@ -15,7 +15,6 @@ import org.slf4j.LoggerFactory;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Optional;
-import java.util.Set;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.atomic.AtomicReference;
 
@@ -85,11 +84,6 @@ public abstract class KestraContext {
 
     public abstract StorageInterface getStorageInterface();
 
-    /**
-     * Returns the Micronaut active environments.
-     */
-    public abstract Set<String> getEnvironments();
-
     /**
      * Shutdowns the Kestra application.
      */
@@ -188,10 +182,5 @@ public abstract class KestraContext {
             // Lazy init of the PluginRegistry.
             return this.applicationContext.getBean(StorageInterface.class);
         }
-
-        @Override
-        public Set<String> getEnvironments() {
-            return this.applicationContext.getEnvironment().getActiveNames();
-        }
     }
 }

core/src/main/java/io/kestra/core/docs/JsonSchemaGenerator.java

@@ -42,12 +42,13 @@ import io.kestra.core.plugins.PluginRegistry;
 import io.kestra.core.plugins.RegisteredPlugin;
 import io.kestra.core.serializers.JacksonMapper;
 import io.micronaut.core.annotation.Nullable;
-import io.swagger.v3.oas.annotations.Hidden;
 import io.swagger.v3.oas.annotations.media.Content;
 import io.swagger.v3.oas.annotations.media.Schema;
 import jakarta.inject.Inject;
 import jakarta.inject.Singleton;
 import lombok.extern.slf4j.Slf4j;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.lang.reflect.*;
 import java.time.*;
@@ -298,9 +299,7 @@ public class JsonSchemaGenerator {
         }
 
         // default value
-        builder.forFields()
-            .withIgnoreCheck(fieldScope -> fieldScope.getAnnotation(Hidden.class) != null)
-            .withDefaultResolver(this::defaults);
+        builder.forFields().withDefaultResolver(this::defaults);
 
         // def name
         builder.forTypesInGeneral()
@@ -810,9 +809,9 @@ public class JsonSchemaGenerator {
         // we don't return base properties unless specified with @PluginProperty and hidden is false
         builder
             .forFields()
-            .withIgnoreCheck(fieldScope -> (base != null &&
+            .withIgnoreCheck(fieldScope -> base != null &&
                 (fieldScope.getAnnotation(PluginProperty.class) == null || fieldScope.getAnnotation(PluginProperty.class).hidden()) &&
-                fieldScope.getDeclaringType().getTypeName().equals(base.getName())) || fieldScope.getAnnotation(Hidden.class) != null
+                fieldScope.getDeclaringType().getTypeName().equals(base.getName())
             );
 
         SchemaGeneratorConfig schemaGeneratorConfig = builder.build();

core/src/main/java/io/kestra/core/docs/Plugin.java

@@ -3,7 +3,6 @@ package io.kestra.core.docs;
 import io.kestra.core.models.annotations.PluginSubGroup;
 import io.kestra.core.plugins.RegisteredPlugin;
 import io.micronaut.core.annotation.Nullable;
-import io.swagger.v3.oas.annotations.media.Schema;
 import lombok.Data;
 import lombok.NoArgsConstructor;
 
@@ -118,17 +117,10 @@ public class Plugin {
             .filter(not(io.kestra.core.models.Plugin::isInternal))
             .filter(clazzFilter)
             .filter(c -> !c.getName().startsWith("org.kestra."))
-            .map(c -> {
-                Schema schema = c.getAnnotation(Schema.class);
-
-                var title = Optional.ofNullable(schema).map(Schema::title).filter(t -> !t.isEmpty()).orElse(null);
-                var description = Optional.ofNullable(schema).map(Schema::description).filter(d -> !d.isEmpty()).orElse(null);
-                var deprecated = io.kestra.core.models.Plugin.isDeprecated(c) ? true : null;
-
-                return new PluginElementMetadata(c.getName(), deprecated, title, description);
-            })
+            .map(c -> new PluginElementMetadata(c.getName(), io.kestra.core.models.Plugin.isDeprecated(c) ? true : null))
             .toList();
     }
 
-    public record PluginElementMetadata(String cls, Boolean deprecated, String title, String description) {}
+    public record PluginElementMetadata(String cls, Boolean deprecated) {
+    }
 }

core/src/main/java/io/kestra/core/exceptions/FlowNotFoundException.java

@@ -1,23 +0,0 @@
-package io.kestra.core.exceptions;
-
-/**
- * Exception that can be thrown when a Flow is not found.
- */
-public class FlowNotFoundException extends NotFoundException {
-
-    /**
-     * Creates a new {@link FlowNotFoundException} instance.
-     */
-    public FlowNotFoundException() {
-        super();
-    }
-
-    /**
-     * Creates a new {@link NotFoundException} instance.
-     *
-     * @param message the error message.
-     */
-    public FlowNotFoundException(final String message) {
-        super(message);
-    }
-}

core/src/main/java/io/kestra/core/exceptions/InputOutputValidationException.java

@@ -1,37 +0,0 @@
-package io.kestra.core.exceptions;
-
-import io.kestra.core.models.flows.Data;
-import io.kestra.core.models.flows.Input;
-import io.kestra.core.models.flows.Output;
-
-import java.util.List;
-import java.util.Set;
-import java.util.stream.Collectors;
-
-/**
- * Exception that can be thrown when Inputs/Outputs have validation problems.
- */
-public class InputOutputValidationException extends KestraRuntimeException {
-    public InputOutputValidationException(String message) {
-        super(message);
-    }
-    public static InputOutputValidationException of( String message, Input<?> input){
-        String inputMessage = "Invalid value for input" + " `" + input.getId() + "`. Cause: " + message;
-        return new InputOutputValidationException(inputMessage);
-    }
-    public static InputOutputValidationException of( String message, Output output){
-        String outputMessage = "Invalid value for output" + " `" + output.getId() + "`. Cause: " + message;
-        return new InputOutputValidationException(outputMessage);
-    }
-    public static InputOutputValidationException of(String message){
-        return new InputOutputValidationException(message);
-    }
-
-    public static InputOutputValidationException merge(Set<InputOutputValidationException> exceptions){
-        String combinedMessage = exceptions.stream()
-                .map(InputOutputValidationException::getMessage)
-                .collect(Collectors.joining(System.lineSeparator()));
-        throw new InputOutputValidationException(combinedMessage);
-    }
-
-}

core/src/main/java/io/kestra/core/exceptions/KestraRuntimeException.java

@@ -1,8 +1,6 @@
 package io.kestra.core.exceptions;
 
 import java.io.Serial;
-import java.util.List;
-import java.util.stream.Collectors;
 
 /**
  * The top-level {@link KestraRuntimeException} for non-recoverable errors.

core/src/main/java/io/kestra/core/exceptions/ResourceAccessDeniedException.java

@@ -1,15 +0,0 @@
-package io.kestra.core.exceptions;
-
-import java.io.Serial;
-
-public class ResourceAccessDeniedException extends KestraRuntimeException {
-    @Serial
-    private static final long serialVersionUID = 1L;
-
-    public ResourceAccessDeniedException() {
-    }
-
-    public ResourceAccessDeniedException(String message) {
-        super(message);
-    }
-}

core/src/main/java/io/kestra/core/models/HasSource.java

@@ -7,6 +7,7 @@ import java.io.IOException;
 import java.io.InputStream;
 import java.util.List;
 import java.util.function.BiConsumer;
+import java.util.function.Consumer;
 import java.util.function.Function;
 import java.util.zip.ZipEntry;
 import java.util.zip.ZipInputStream;
@@ -64,7 +65,7 @@ public interface HasSource {
 
             if (isYAML(fileName)) {
                 byte[] bytes = inputStream.readAllBytes();
-                List<String> sources = List.of(new String(bytes).split("(?m)^---\\s*$"));
+                List<String> sources = List.of(new String(bytes).split("---"));
                 for (int i = 0; i < sources.size(); i++) {
                     String source = sources.get(i);
                     reader.accept(source, String.valueOf(i));

core/src/main/java/io/kestra/core/models/Label.java

@@ -4,16 +4,13 @@ import io.kestra.core.utils.MapUtils;
 import io.swagger.v3.oas.annotations.media.Schema;
 import jakarta.annotation.Nullable;
 import jakarta.validation.constraints.NotEmpty;
-import jakarta.validation.constraints.Pattern;
 
 import java.util.*;
 import java.util.function.Predicate;
 import java.util.stream.Collectors;
 
 @Schema(description = "A key/value pair that can be attached to a Flow or Execution. Labels are often used to organize and categorize objects.")
-public record Label(
-        @NotEmpty @Pattern(regexp = "^[\\p{Ll}][\\p{L}0-9._-]*$", message = "Invalid label key. A valid key contains only lowercase letters numbers hyphens (-) underscores (_) or periods (.) and must begin with a lowercase letter.") String key,
-        @NotEmpty String value) {
+public record Label(@NotEmpty String key, @NotEmpty String value) {
     public static final String SYSTEM_PREFIX = "system.";
 
     // system labels
@@ -26,7 +23,6 @@ public record Label(
     public static final String REPLAYED = SYSTEM_PREFIX + "replayed";
     public static final String SIMULATED_EXECUTION = SYSTEM_PREFIX + "simulatedExecution";
     public static final String TEST = SYSTEM_PREFIX + "test";
-    public static final String FROM = SYSTEM_PREFIX + "from";
 
     /**
      * Static helper method for converting a list of labels to a nested map.

core/src/main/java/io/kestra/core/models/QueryFilter.java

@@ -94,7 +94,7 @@ public record QueryFilter(
         KIND("kind") {
             @Override
             public List<Op> supportedOp() {
-                return List.of(Op.EQUALS,Op.NOT_EQUALS, Op.IN, Op.NOT_IN);
+                return List.of(Op.EQUALS,Op.NOT_EQUALS);
             }
         },
         LABELS("labels") {
@@ -106,7 +106,7 @@ public record QueryFilter(
         FLOW_ID("flowId") {
             @Override
             public List<Op> supportedOp() {
-                return List.of(Op.EQUALS, Op.NOT_EQUALS, Op.CONTAINS, Op.STARTS_WITH, Op.ENDS_WITH, Op.REGEX, Op.IN, Op.NOT_IN, Op.PREFIX);
+                return List.of(Op.EQUALS, Op.NOT_EQUALS, Op.CONTAINS, Op.STARTS_WITH, Op.ENDS_WITH, Op.REGEX);
             }
         },
         UPDATED("updated") {
@@ -180,24 +180,6 @@ public record QueryFilter(
             public List<Op> supportedOp() {
                 return List.of(Op.EQUALS, Op.NOT_EQUALS);
             }
-        },
-        PATH("path") {
-            @Override
-            public List<Op> supportedOp() {
-                return List.of(Op.EQUALS, Op.NOT_EQUALS, Op.IN);
-            }
-        },
-        PARENT_PATH("parentPath") {
-            @Override
-            public List<Op> supportedOp() {
-                return List.of(Op.EQUALS, Op.NOT_EQUALS, Op.STARTS_WITH);
-            }
-        },
-        VERSION("version") {
-            @Override
-            public List<Op> supportedOp() {
-                return List.of(Op.EQUALS, Op.NOT_EQUALS);
-            }
         };
 
         private static final Map<String, Field> BY_VALUE = Arrays.stream(values())
@@ -226,7 +208,7 @@ public record QueryFilter(
         FLOW {
             @Override
             public List<Field> supportedField() {
-                return List.of(Field.LABELS, Field.NAMESPACE, Field.QUERY, Field.SCOPE, Field.FLOW_ID);
+                return List.of(Field.LABELS, Field.NAMESPACE, Field.QUERY, Field.SCOPE);
             }
         },
         NAMESPACE {
@@ -241,7 +223,7 @@ public record QueryFilter(
                 return List.of(
                     Field.QUERY, Field.SCOPE, Field.FLOW_ID, Field.START_DATE, Field.END_DATE,
                     Field.STATE, Field.LABELS, Field.TRIGGER_EXECUTION_ID, Field.CHILD_FILTER,
-                    Field.NAMESPACE, Field.KIND
+                    Field.NAMESPACE,Field.KIND
                 );
             }
         },
@@ -293,19 +275,6 @@ public record QueryFilter(
                     Field.UPDATED
                 );
             }
-        },
-        NAMESPACE_FILE_METADATA {
-            @Override
-            public List<Field> supportedField() {
-                return List.of(
-                    Field.QUERY,
-                    Field.NAMESPACE,
-                    Field.PATH,
-                    Field.PARENT_PATH,
-                    Field.VERSION,
-                    Field.UPDATED
-                );
-            }
         };
 
         public abstract List<Field> supportedField();

core/src/main/java/io/kestra/core/models/Setting.java

@@ -16,7 +16,6 @@ import jakarta.validation.constraints.NotNull;
 public class Setting {
     public static final String INSTANCE_UUID = "instance.uuid";
     public static final String INSTANCE_VERSION = "instance.version";
-    public static final String INSTANCE_EDITION = "instance.edition";
 
     @NotNull
     private String key;

core/src/main/java/io/kestra/core/models/conditions/ConditionContext.java

@@ -3,6 +3,7 @@ package io.kestra.core.models.conditions;
 import io.kestra.core.models.flows.FlowInterface;
 import lombok.*;
 import io.kestra.core.models.executions.Execution;
+import io.kestra.core.models.flows.Flow;
 import io.kestra.core.models.triggers.multipleflows.MultipleConditionStorageInterface;
 import io.kestra.core.runners.RunContext;
 

core/src/main/java/io/kestra/core/models/executions/Execution.java

@@ -658,20 +658,18 @@ public class Execution implements DeletedInterface, TenantInterface {
     public boolean hasFailedNoRetry(List<ResolvedTask> resolvedTasks, TaskRun parentTaskRun) {
         return this.findTaskRunByTasks(resolvedTasks, parentTaskRun)
             .stream()
-            // NOTE: we check on isFailed first to avoid the costly shouldBeRetried() method
-            .anyMatch(taskRun -> taskRun.getState().isFailed() && shouldNotBeRetried(resolvedTasks, parentTaskRun, taskRun));
-    }
-
-    private static boolean shouldNotBeRetried(List<ResolvedTask> resolvedTasks, TaskRun parentTaskRun, TaskRun taskRun) {
-        ResolvedTask resolvedTask = resolvedTasks.stream()
-            .filter(t -> t.getTask().getId().equals(taskRun.getTaskId())).findFirst()
-            .orElse(null);
-        if (resolvedTask == null) {
-            log.warn("Can't find task for taskRun '{}' in parentTaskRun '{}'",
-                taskRun.getId(), parentTaskRun.getId());
-            return false;
-        }
-        return !taskRun.shouldBeRetried(resolvedTask.getTask().getRetry());
+            .anyMatch(taskRun -> {
+                ResolvedTask resolvedTask = resolvedTasks.stream()
+                    .filter(t -> t.getTask().getId().equals(taskRun.getTaskId())).findFirst()
+                    .orElse(null);
+                if (resolvedTask == null) {
+                    log.warn("Can't find task for taskRun '{}' in parentTaskRun '{}'",
+                        taskRun.getId(), parentTaskRun.getId());
+                    return false;
+                }
+                return !taskRun.shouldBeRetried(resolvedTask.getTask().getRetry())
+                    && taskRun.getState().isFailed();
+            });
     }
 
     public boolean hasCreated() {

core/src/main/java/io/kestra/core/models/executions/ExecutionTrigger.java

@@ -1,16 +1,15 @@
 package io.kestra.core.models.executions;
 
-import io.kestra.core.models.tasks.Output;
-import io.kestra.core.models.triggers.AbstractTrigger;
 import io.micronaut.core.annotation.Introspected;
-import io.swagger.v3.oas.annotations.media.Schema;
-import jakarta.validation.constraints.NotNull;
 import lombok.Builder;
 import lombok.Value;
+import io.kestra.core.models.tasks.Output;
+import io.kestra.core.models.triggers.AbstractTrigger;
 
 import java.net.URI;
 import java.util.Collections;
 import java.util.Map;
+import jakarta.validation.constraints.NotNull;
 
 @Value
 @Builder
@@ -22,7 +21,6 @@ public class ExecutionTrigger {
     @NotNull
     String type;
 
-    @Schema(type = "object", additionalProperties = Schema.AdditionalPropertiesValue.TRUE)
     Map<String, Object> variables;
 
     URI logFile;

core/src/main/java/io/kestra/core/models/executions/LogEntry.java

@@ -3,7 +3,7 @@ package io.kestra.core.models.executions;
 import com.fasterxml.jackson.annotation.JsonInclude;
 import io.kestra.core.models.DeletedInterface;
 import io.kestra.core.models.TenantInterface;
-import io.kestra.core.models.flows.FlowInterface;
+import io.kestra.core.models.flows.Flow;
 import io.kestra.core.models.triggers.AbstractTrigger;
 import io.kestra.core.models.triggers.TriggerContext;
 import io.swagger.v3.oas.annotations.Hidden;
@@ -97,7 +97,7 @@ public class LogEntry implements DeletedInterface, TenantInterface {
             .build();
     }
 
-    public static LogEntry of(FlowInterface flow, AbstractTrigger abstractTrigger) {
+    public static LogEntry of(Flow flow, AbstractTrigger abstractTrigger, ExecutionKind executionKind) {
         return LogEntry.builder()
             .tenantId(flow.getTenantId())
             .namespace(flow.getNamespace())
@@ -107,7 +107,7 @@ public class LogEntry implements DeletedInterface, TenantInterface {
             .build();
     }
 
-    public static LogEntry of(TriggerContext triggerContext, AbstractTrigger abstractTrigger) {
+    public static LogEntry of(TriggerContext triggerContext, AbstractTrigger abstractTrigger, ExecutionKind executionKind) {
         return LogEntry.builder()
             .tenantId(triggerContext.getTenantId())
             .namespace(triggerContext.getNamespace())

core/src/main/java/io/kestra/core/models/executions/TaskRun.java

@@ -3,7 +3,9 @@ package io.kestra.core.models.executions;
 import com.fasterxml.jackson.annotation.JsonInclude;
 import io.kestra.core.models.TenantInterface;
 import io.kestra.core.models.flows.State;
+import io.kestra.core.models.tasks.FlowableTask;
 import io.kestra.core.models.tasks.ResolvedTask;
+import io.kestra.core.models.tasks.Task;
 import io.kestra.core.models.tasks.retrys.AbstractRetry;
 import io.kestra.core.utils.IdUtils;
 import io.swagger.v3.oas.annotations.Hidden;
@@ -93,16 +95,8 @@ public class TaskRun implements TenantInterface {
             this.forceExecution
         );
     }
-    public TaskRun withStateAndAttempt(State.Type state) {
-        List<TaskRunAttempt> newAttempts = new ArrayList<>(this.attempts != null ? this.attempts : List.of());
-
-        if (newAttempts.isEmpty()) {
-            newAttempts.add(TaskRunAttempt.builder().state(new State(state)).build());
-        } else {
-            TaskRunAttempt updatedLast = newAttempts.getLast().withState(state);
-            newAttempts.set(newAttempts.size() - 1, updatedLast);
-        }
 
+    public TaskRun replaceState(State newState) {
         return new TaskRun(
             this.tenantId,
             this.id,
@@ -112,9 +106,9 @@ public class TaskRun implements TenantInterface {
             this.taskId,
             this.parentTaskRunId,
             this.value,
-            newAttempts,
+            this.attempts,
             this.outputs,
-            this.state.withState(state),
+            newState,
             this.iteration,
             this.dynamic,
             this.forceExecution
@@ -320,11 +314,4 @@ public class TaskRun implements TenantInterface {
             .build();
     }
 
-    public TaskRun addAttempt(TaskRunAttempt attempt) {
-        if (this.attempts == null) {
-            this.attempts = new ArrayList<>();
-        }
-        this.attempts.add(attempt);
-        return this;
-    }
 }

core/src/main/java/io/kestra/core/models/flows/Concurrency.java

@@ -24,8 +24,4 @@ public class Concurrency {
     public enum Behavior {
         QUEUE, CANCEL, FAIL;
     }
-
-    public static boolean possibleTransitions(State.Type type) {
-        return type.equals(State.Type.CANCELLED) || type.equals(State.Type.FAILED);
-    }
 }

core/src/main/java/io/kestra/core/models/flows/Data.java

@@ -1,5 +1,7 @@
 package io.kestra.core.models.flows;
 
+import io.kestra.core.models.validations.ManualConstraintViolation;
+import jakarta.validation.ConstraintViolationException;
 
 /**
  * Interface for defining an identifiable and typed data.
@@ -27,4 +29,16 @@ public interface Data {
      */
     String getDisplayName();
 
+    @SuppressWarnings("unchecked")
+    default ConstraintViolationException toConstraintViolationException(String message, Object value) {
+        Class<Data> cls = (Class<Data>) this.getClass();
+
+        return ManualConstraintViolation.toConstraintViolationException(
+            "Invalid " + (this instanceof Output ? "output" : "input") + " for `" + getId() + "`, " + message + ", but received `" + value + "`",
+            this,
+            cls,
+            this.getId(),
+            value
+        );
+    }
 }

core/src/main/java/io/kestra/core/models/flows/Flow.java

@@ -1,5 +1,6 @@
 package io.kestra.core.models.flows;
 
+import com.fasterxml.jackson.annotation.JsonIgnore;
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.core.JsonProcessingException;
@@ -10,7 +11,6 @@ import com.fasterxml.jackson.databind.introspect.JacksonAnnotationIntrospector;
 import io.kestra.core.exceptions.InternalException;
 import io.kestra.core.models.HasUID;
 import io.kestra.core.models.annotations.PluginProperty;
-import io.kestra.core.models.flows.check.Check;
 import io.kestra.core.models.flows.sla.SLA;
 import io.kestra.core.models.listeners.Listener;
 import io.kestra.core.models.tasks.FlowableTask;
@@ -130,14 +130,6 @@ public class Flow extends AbstractFlow implements HasUID {
     @PluginProperty
     List<SLA> sla;
 
-    @Schema(
-        title = "Conditions evaluated before the flow is executed.",
-        description = "A list of conditions that are evaluated before the flow is executed.  If no checks are defined, the flow executes normally."
-    )
-    @Valid
-    @PluginProperty
-    List<Check> checks;
-
     public Stream<String> allTypes() {
         return Stream.of(
                 Optional.ofNullable(triggers).orElse(Collections.emptyList()).stream().map(AbstractTrigger::getType),
@@ -354,7 +346,7 @@ public class Flow extends AbstractFlow implements HasUID {
      * To be conservative a flow MUST not return any source.
      */
     @Override
-    @Schema(hidden = true)
+    @JsonIgnore
     public String getSource() {
         return null;
     }

core/src/main/java/io/kestra/core/models/flows/FlowWithSource.java

@@ -1,12 +1,14 @@
 package io.kestra.core.models.flows;
 
+import com.fasterxml.jackson.annotation.JsonIgnore;
 import io.micronaut.core.annotation.Introspected;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.ToString;
 import lombok.experimental.SuperBuilder;
-import io.swagger.v3.oas.annotations.media.Schema;
 
+import java.util.Objects;
+import java.util.regex.Pattern;
 
 @SuperBuilder(toBuilder = true)
 @Getter
@@ -41,12 +43,11 @@ public class FlowWithSource extends Flow {
             .concurrency(this.concurrency)
             .retry(this.retry)
             .sla(this.sla)
-            .checks(this.checks)
             .build();
     }
 
     @Override
-    @Schema(hidden = false)
+    @JsonIgnore(value = false)
     public String getSource() {
         return this.source;
     }
@@ -84,7 +85,6 @@ public class FlowWithSource extends Flow {
             .concurrency(flow.concurrency)
             .retry(flow.retry)
             .sla(flow.sla)
-            .checks(flow.checks)
             .build();
     }
 }

core/src/main/java/io/kestra/core/models/flows/State.java

@@ -84,24 +84,12 @@ public class State {
         );
     }
 
-    /**
-     * non-terminated execution duration is hard to provide in SQL, so we set it to null when endDate is empty
-     */
     @JsonProperty(access = JsonProperty.Access.READ_ONLY)
-    @JsonInclude(JsonInclude.Include.NON_EMPTY)
-    public Optional<Duration> getDuration() {
-        if (this.getEndDate().isPresent()) {
-            return Optional.of(Duration.between(this.getStartDate(), this.getEndDate().get()));
-        } else {
-            return Optional.empty();
-        }
-    }
-
-    /**
-     * @return either the Duration persisted in database, or calculate it on the fly for non-terminated executions
-     */
-    public Duration getDurationOrComputeIt() {
-        return this.getDuration().orElseGet(() -> Duration.between(this.getStartDate(), Instant.now()));
+    public Duration getDuration() {
+        return Duration.between(
+            this.histories.getFirst().getDate(),
+            this.histories.size() > 1 ? this.histories.get(this.histories.size() - 1).getDate() : Instant.now()
+        );
     }
 
     @JsonProperty(access = JsonProperty.Access.READ_ONLY)
@@ -121,7 +109,7 @@ public class State {
 
     public String humanDuration() {
         try {
-            return DurationFormatUtils.formatDurationHMS(getDurationOrComputeIt().toMillis());
+            return DurationFormatUtils.formatDurationHMS(getDuration().toMillis());
         } catch (Throwable e) {
             return getDuration().toString();
         }
@@ -267,10 +255,6 @@ public class State {
             return this == Type.RUNNING || this == Type.KILLING;
         }
 
-        public boolean onlyRunning() {
-            return this == Type.RUNNING;
-        }
-
         public boolean isFailed() {
             return this == Type.FAILED;
         }

core/src/main/java/io/kestra/core/models/flows/check/Check.java

@@ -1,109 +0,0 @@
-package io.kestra.core.models.flows.check;
-
-import jakarta.validation.constraints.NotEmpty;
-import jakarta.validation.constraints.NotNull;
-import lombok.Getter;
-import lombok.NoArgsConstructor;
-import lombok.experimental.SuperBuilder;
-
-import java.util.Comparator;
-import java.util.List;
-import java.util.Objects;
-
-/**
- * Represents a check within a Kestra flow.
- * <p>
- * A {@code Check} defines a boolean condition that is evaluated when validating flow's inputs
- * and before triggering an execution.
- * <p>
- * If the condition evaluates to {@code false}, the configured {@link Behavior}
- * determines how the execution proceeds, and the {@link Style} determines how
- * the message is visually presented in the UI.
- * </p>
- */
-@SuperBuilder
-@Getter
-@NoArgsConstructor
-public class Check {
-    
-    /**
-     * The condition to evaluate.
-     */
-    @NotNull
-    @NotEmpty
-    String condition;
-    
-    /**
-     * The message associated with this check, will be displayed when the condition evaluates to {@code false}.
-     */
-    @NotEmpty
-    String message;
-    
-    /**
-     * Defines the style of the message displayed in the UI when the condition evaluates to {@code false}.
-     */
-    Style style = Style.INFO;
-    
-    /**
-     * The behavior to apply when the condition evaluates to {@code false}.
-     */
-    Behavior behavior = Behavior.BLOCK_EXECUTION;
-    
-    /**
-     * The visual style used to display the message when the check fails.
-     */
-    public enum Style {
-        /**
-         * Display the message as an error.
-         */
-        ERROR,
-        /**
-         * Display the message as a success indicator.
-         */
-        SUCCESS,
-        /**
-         * Display the message as a warning.
-         */
-        WARNING,
-        /**
-         * Display the message as informational content.
-         */
-        INFO;
-    }
-    
-    /**
-     * Defines how the flow should behave when the condition evaluates to {@code false}.
-     */
-    public enum Behavior {
-        /**
-         * Block the creation of the execution.
-         */
-        BLOCK_EXECUTION,
-        /**
-         * Create the execution as failed.
-         */
-        FAIL_EXECUTION,
-        /**
-         * Create a new execution as a result of the check failing.
-         */
-        CREATE_EXECUTION;
-    }
-    
-    /**
-     * Resolves the effective behavior for a list of {@link Check}s based on priority.
-     *
-     * @param checks the list of checks whose behaviors are to be evaluated
-     * @return the highest-priority behavior, or {@code CREATE_EXECUTION} if the list is empty or only contains nulls
-     */
-    public static Check.Behavior resolveBehavior(List<Check> checks) {
-        if (checks == null || checks.isEmpty()) {
-            return Behavior.CREATE_EXECUTION;
-        }
-        
-        return checks.stream()
-            .map(Check::getBehavior)
-            .filter(Objects::nonNull).min(Comparator.comparingInt(Enum::ordinal))
-            .orElse(Behavior.CREATE_EXECUTION);
-    }
-    
-}

core/src/main/java/io/kestra/core/models/flows/input/InputAndValue.java

@@ -1,12 +1,10 @@
 package io.kestra.core.models.flows.input;
 
-import io.kestra.core.exceptions.InputOutputValidationException;
 import io.kestra.core.models.flows.Input;
 import jakarta.annotation.Nullable;
+import jakarta.validation.ConstraintViolationException;
 import jakarta.validation.constraints.NotNull;
 
-import java.util.Set;
-
 /**
  * Represents an input along with its associated value and validation state.
  *
@@ -14,15 +12,15 @@ import java.util.Set;
  * @param value     The provided value for the input.
  * @param enabled   {@code true} if the input is enabled; {@code false} otherwise.
  * @param isDefault {@code true} if the provided value is the default; {@code false} otherwise.
- * @param exceptions The validation exceptions, if the input value is invalid; {@code null} otherwise.
+ * @param exception The validation exception, if the input value is invalid; {@code null} otherwise.
  */
 public record InputAndValue(
     Input<?> input,
     Object value,
     boolean enabled,
     boolean isDefault,
-    Set<InputOutputValidationException> exceptions) {
-
+    ConstraintViolationException exception) {
+    
     /**
      * Creates a new {@link InputAndValue} instance.
      *

core/src/main/java/io/kestra/core/models/flows/input/MultiselectInput.java

@@ -7,7 +7,6 @@ import io.kestra.core.models.property.Property;
 import io.kestra.core.models.validations.ManualConstraintViolation;
 import io.kestra.core.validations.Regex;
 import io.swagger.v3.oas.annotations.media.Schema;
-import jakarta.validation.ConstraintViolation;
 import jakarta.validation.ConstraintViolationException;
 import jakarta.validation.constraints.NotNull;
 import lombok.Builder;
@@ -15,7 +14,10 @@ import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.experimental.SuperBuilder;
 
-import java.util.*;
+import java.util.Collection;
+import java.util.List;
+import java.util.Objects;
+import java.util.Optional;
 import java.util.function.Function;
 
 @SuperBuilder
@@ -75,35 +77,30 @@ public class MultiselectInput extends Input<List<String>> implements ItemTypeInt
 
     @Override
     public void validate(List<String> inputs) throws ConstraintViolationException {
-        Set<ConstraintViolation<?>> violations = new HashSet<>();
-
         if (values != null && options != null) {
-            violations.add( ManualConstraintViolation.of(
+            throw ManualConstraintViolation.toConstraintViolationException(
                 "you can't define both `values` and `options`",
                 this,
                 MultiselectInput.class,
                 getId(),
                 ""
-            ));
+            );
         }
 
         if (!this.getAllowCustomValue()) {
             for (String input : inputs) {
                 List<@Regex String> finalValues = this.values != null ? this.values : this.options;
                 if (!finalValues.contains(input)) {
-                    violations.add(ManualConstraintViolation.of(
-                        "value `" + input + "` doesn't match the values `" + finalValues + "`",
+                    throw ManualConstraintViolation.toConstraintViolationException(
+                        "it must match the values `" + finalValues + "`",
                         this,
                         MultiselectInput.class,
                         getId(),
                         input
-                    ));
+                    );
                 }
             }
         }
-        if (!violations.isEmpty()) {
-            throw ManualConstraintViolation.toConstraintViolationException(violations);
-        }
     }
 
     /** {@inheritDoc} **/
@@ -148,7 +145,7 @@ public class MultiselectInput extends Input<List<String>> implements ItemTypeInt
 
         String type = Optional.ofNullable(result).map(Object::getClass).map(Class::getSimpleName).orElse("<null>");
         throw ManualConstraintViolation.toConstraintViolationException(
-            "Invalid expression result. Expected a list of strings",
+            "Invalid expression result. Expected a list of strings, but received " + type,
             this,
             MultiselectInput.class,
             getId(),

core/src/main/java/io/kestra/core/models/flows/input/SelectInput.java

@@ -125,7 +125,7 @@ public class SelectInput extends Input<String> implements RenderableInput {
 
         String type = Optional.ofNullable(result).map(Object::getClass).map(Class::getSimpleName).orElse("<null>");
         throw ManualConstraintViolation.toConstraintViolationException(
-            "Invalid expression result. Expected a list of strings",
+            "Invalid expression result. Expected a list of strings, but received " + type,
             this,
             SelectInput.class,
             getId(),

core/src/main/java/io/kestra/core/models/hierarchies/SubflowGraphTask.java

@@ -48,7 +48,7 @@ public class SubflowGraphTask extends AbstractGraphTask {
 
     public record SubflowTaskWrapper<T extends Output>(RunContext runContext, ExecutableTask<T> subflowTask) implements TaskInterface, ExecutableTask<T> {
         @Override
-        public List<SubflowExecution<?>> createSubflowExecutions(RunContext runContext, FlowMetaStoreInterface flowExecutorInterface, FlowInterface currentFlow, Execution currentExecution, TaskRun currentTaskRun) throws InternalException {
+        public List<SubflowExecution<?>> createSubflowExecutions(RunContext runContext, FlowMetaStoreInterface flowExecutorInterface, Flow currentFlow, Execution currentExecution, TaskRun currentTaskRun) throws InternalException {
             return subflowTask.createSubflowExecutions(runContext, flowExecutorInterface, currentFlow, currentExecution, currentTaskRun);
         }
 

core/src/main/java/io/kestra/core/models/kv/PersistedKvMetadata.java

@@ -20,6 +20,7 @@ import java.util.Optional;
 @Slf4j
 @Getter
 @FieldDefaults(makeFinal = true, level = AccessLevel.PRIVATE)
+@AllArgsConstructor
 @ToString
 @EqualsAndHashCode
 public class PersistedKvMetadata implements DeletedInterface, TenantInterface, HasUID {
@@ -53,19 +54,6 @@ public class PersistedKvMetadata implements DeletedInterface, TenantInterface, H
 
     private boolean deleted;
 
-    public PersistedKvMetadata(String tenantId, String namespace, String name, String description, Integer version, boolean last, @Nullable Instant expirationDate, @Nullable Instant created, @Nullable Instant updated, boolean deleted) {
-        this.tenantId = tenantId;
-        this.namespace = namespace;
-        this.name = name;
-        this.description = description;
-        this.version = version;
-        this.last = last;
-        this.expirationDate = expirationDate;
-        this.created = Optional.ofNullable(created).orElse(Instant.now());
-        this.updated = updated;
-        this.deleted = deleted;
-    }
-
     public static PersistedKvMetadata from(String tenantId, KVEntry kvEntry) {
         return PersistedKvMetadata.builder()
             .tenantId(tenantId)
@@ -80,15 +68,12 @@ public class PersistedKvMetadata implements DeletedInterface, TenantInterface, H
     }
 
     public PersistedKvMetadata asLast() {
-        return this.toBuilder().updated(Instant.now()).last(true).build();
-    }
-
-    public PersistedKvMetadata toDeleted() {
-        return this.toBuilder().updated(Instant.now()).deleted(true).build();
+        Instant saveDate = Instant.now();
+        return this.toBuilder().created(Optional.ofNullable(this.created).orElse(saveDate)).updated(saveDate).last(true).build();
     }
 
     @Override
     public String uid() {
-        return IdUtils.fromParts(getTenantId(), getNamespace(), getName(), String.valueOf(getVersion()));
+        return IdUtils.fromParts(getTenantId(), getNamespace(), getName(), getVersion().toString());
     }
 }

core/src/main/java/io/kestra/core/models/namespaces/files/NamespaceFileMetadata.java

@@ -1,132 +0,0 @@
-package io.kestra.core.models.namespaces.files;
-
-import com.fasterxml.jackson.annotation.JsonCreator;
-import com.fasterxml.jackson.annotation.JsonIgnore;
-import io.kestra.core.models.DeletedInterface;
-import io.kestra.core.models.HasUID;
-import io.kestra.core.models.TenantInterface;
-import io.kestra.core.storages.FileAttributes;
-import io.kestra.core.storages.NamespaceFile;
-import io.kestra.core.utils.IdUtils;
-import io.swagger.v3.oas.annotations.Hidden;
-import jakarta.annotation.Nullable;
-import jakarta.validation.constraints.NotNull;
-import jakarta.validation.constraints.Pattern;
-import lombok.*;
-import lombok.experimental.FieldDefaults;
-import lombok.extern.slf4j.Slf4j;
-
-import java.time.Instant;
-
-@Builder(toBuilder = true)
-@Slf4j
-@Getter
-@FieldDefaults(makeFinal = true, level = AccessLevel.PRIVATE)
-@ToString
-@EqualsAndHashCode
-public class NamespaceFileMetadata implements DeletedInterface, TenantInterface, HasUID {
-    @With
-    @Hidden
-    @Pattern(regexp = "^[a-z0-9][a-z0-9_-]*")
-    private String tenantId;
-
-    @NotNull
-    private String namespace;
-
-    @NotNull
-    private String path;
-
-    private String parentPath;
-
-    @NotNull
-    private Integer version;
-
-    @Builder.Default
-    private boolean last = true;
-
-    @NotNull
-    private Long size;
-
-    @Builder.Default
-    private Instant created = Instant.now();
-
-    @Nullable
-    private Instant updated;
-
-    private boolean deleted;
-
-    @JsonCreator
-    public NamespaceFileMetadata(String tenantId, String namespace, String path, String parentPath, Integer version, boolean last, Long size, Instant created, @Nullable Instant updated, boolean deleted) {
-        this.tenantId = tenantId;
-        this.namespace = namespace;
-        this.path = path;
-        this.parentPath = parentPath(path);
-        this.version = version;
-        this.last = last;
-        this.size = size;
-        this.created = created;
-        this.updated = updated;
-        this.deleted = deleted;
-    }
-
-    public static String path(String path, boolean trailingSlash) {
-        if (trailingSlash && !path.endsWith("/")) {
-            return path + "/";
-        } else if (!trailingSlash && path.endsWith("/")) {
-            return path.substring(0, path.length() - 1);
-        }
-        return path;
-    }
-
-    public String path(boolean trailingSlash) {
-        return path(this.path, trailingSlash);
-    }
-
-    public static String parentPath(String path) {
-        String withoutTrailingSlash = path.endsWith("/") ? path.substring(0, path.length() - 1) : path;
-        // The parent path can't be set, it's always computed
-        return withoutTrailingSlash.contains("/") ?
-            withoutTrailingSlash.substring(0, withoutTrailingSlash.lastIndexOf("/") + 1) :
-            null;
-    }
-
-    public static NamespaceFileMetadata of(String tenantId, NamespaceFile namespaceFile) {
-        return NamespaceFileMetadata.builder()
-            .tenantId(tenantId)
-            .namespace(namespaceFile.namespace())
-            .path(namespaceFile.path(true).toString())
-            .version(namespaceFile.version())
-            .build();
-    }
-
-    public static NamespaceFileMetadata of(String tenantId, String namespace, String path, FileAttributes fileAttributes) {
-        return NamespaceFileMetadata.builder()
-            .tenantId(tenantId)
-            .namespace(namespace)
-            .path(path)
-            .created(Instant.ofEpochMilli(fileAttributes.getCreationTime()))
-            .updated(Instant.ofEpochMilli(fileAttributes.getLastModifiedTime()))
-            .size(fileAttributes.getSize())
-            .version(1)
-            .build();
-    }
-
-    public NamespaceFileMetadata asLast() {
-        Instant saveDate = Instant.now();
-        return this.toBuilder().updated(saveDate).last(true).build();
-    }
-
-    public NamespaceFileMetadata toDeleted() {
-        return this.toBuilder().deleted(true).updated(Instant.now()).build();
-    }
-
-    @Override
-    public String uid() {
-        return IdUtils.fromParts(getTenantId(), getNamespace(), getPath(), String.valueOf(getVersion()));
-    }
-
-    @JsonIgnore
-    public boolean isDirectory() {
-        return this.path.endsWith("/");
-    }
-}

core/src/main/java/io/kestra/core/models/property/Property.java

@@ -11,7 +11,6 @@ import com.fasterxml.jackson.databind.ser.std.StdSerializer;
 import com.google.common.annotations.VisibleForTesting;
 import io.kestra.core.exceptions.IllegalVariableEvaluationException;
 import io.kestra.core.runners.RunContext;
-import io.kestra.core.runners.RunContextProperty;
 import io.kestra.core.serializers.JacksonMapper;
 import io.swagger.v3.oas.annotations.media.Schema;
 import jakarta.validation.constraints.NotNull;
@@ -36,6 +35,7 @@ import static io.kestra.core.utils.Rethrow.throwFunction;
 @JsonDeserialize(using = Property.PropertyDeserializer.class)
 @JsonSerialize(using = Property.PropertySerializer.class)
 @Builder
+@NoArgsConstructor
 @AllArgsConstructor(access = AccessLevel.PACKAGE)
 @Schema(
     oneOf = {
@@ -51,7 +51,6 @@ public class Property<T> {
         .copy()
         .configure(SerializationFeature.WRITE_DURATIONS_AS_TIMESTAMPS, false);
 
-    private final boolean skipCache;
     private String expression;
     private T value;
 
@@ -61,23 +60,13 @@ public class Property<T> {
     @Deprecated
     // Note: when not used, this constructor would not be deleted but made private so it can only be used by ofExpression(String) and the deserializer
     public Property(String expression) {
-        this(expression, false);
-    }
-
-    private Property(String expression, boolean skipCache) {
         this.expression = expression;
-        this.skipCache = skipCache;
     }
 
-    /**
-     * @deprecated use {@link #ofValue(Object)} instead.
-     */
     @VisibleForTesting
-    @Deprecated
     public Property(Map<?, ?> map) {
         try {
             expression = MAPPER.writeValueAsString(map);
-            this.skipCache = false;
         } catch (JsonProcessingException e) {
             throw new IllegalArgumentException(e);
         }
@@ -90,11 +79,14 @@ public class Property<T> {
     /**
      * Returns a new {@link Property} with no cached rendered value,
      * so that the next render will evaluate its original Pebble expression.
+     * <p>
+     * The returned property will still cache its rendered result.
+     * To re-evaluate on a subsequent render, call {@code skipCache()} again.
      *
      * @return a new {@link Property} without a pre-rendered value
      */
     public Property<T> skipCache() {
-        return new Property<>(expression, true);
+        return Property.ofExpression(expression);
     }
 
     /**
@@ -141,7 +133,6 @@ public class Property<T> {
 
     /**
      * Build a new Property object with a Pebble expression.<br>
-     * This property object will not cache its rendered value.
      * <p>
      * Use {@link #ofValue(Object)} to build a property with a value instead.
      */
@@ -151,15 +142,15 @@ public class Property<T> {
             throw new IllegalArgumentException("'expression' must be a valid Pebble expression");
         }
 
-        return new Property<>(expression, true);
+        return new Property<>(expression);
     }
 
     /**
-     * Render a property, then convert it to its target type.<br>
+     * Render a property then convert it to its target type.<br>
      * <p>
-     * This method is designed to be used only by the {@link RunContextProperty}.
+     * This method is designed to be used only by the {@link io.kestra.core.runners.RunContextProperty}.
      *
-     * @see RunContextProperty#as(Class)
+     * @see io.kestra.core.runners.RunContextProperty#as(Class)
      */
     public static <T> T as(Property<T> property, PropertyContext context, Class<T> clazz) throws IllegalVariableEvaluationException {
         return as(property, context, clazz, Map.of());
@@ -168,57 +159,25 @@ public class Property<T> {
     /**
      * Render a property with additional variables, then convert it to its target type.<br>
      * <p>
-     * This method is designed to be used only by the {@link RunContextProperty}.
+     * This method is designed to be used only by the {@link io.kestra.core.runners.RunContextProperty}.
      *
-     * @see RunContextProperty#as(Class, Map)
+     * @see io.kestra.core.runners.RunContextProperty#as(Class, Map)
      */
     public static <T> T as(Property<T> property, PropertyContext context, Class<T> clazz, Map<String, Object> variables) throws IllegalVariableEvaluationException {
-        if (property.skipCache || property.value == null) {
+        if (property.value == null) {
             String rendered = context.render(property.expression, variables);
-            property.value = deserialize(rendered, clazz);
+            property.value = MAPPER.convertValue(rendered, clazz);
         }
 
         return property.value;
     }
 
-    private static <T> T deserialize(Object rendered, Class<T> clazz) throws IllegalVariableEvaluationException {
-        try {
-            return MAPPER.convertValue(rendered, clazz);
-        } catch (IllegalArgumentException e) {
-            if (rendered instanceof String str) {
-                try {
-                    return MAPPER.readValue(str, clazz);
-                } catch (JsonProcessingException ex) {
-                    throw new IllegalVariableEvaluationException(ex);
-                }
-            }
-
-            throw new IllegalVariableEvaluationException(e);
-        }
-    }
-
-    private static <T> T deserialize(Object rendered, JavaType type) throws IllegalVariableEvaluationException {
-        try {
-            return MAPPER.convertValue(rendered, type);
-        } catch (IllegalArgumentException e) {
-            if (rendered instanceof String str) {
-                try {
-                    return MAPPER.readValue(str, type);
-                } catch (JsonProcessingException ex) {
-                    throw new IllegalVariableEvaluationException(ex);
-                }
-            }
-
-            throw new IllegalVariableEvaluationException(e);
-        }
-    }
-
     /**
      * Render a property then convert it as a list of target type.<br>
      * <p>
-     * This method is designed to be used only by the {@link RunContextProperty}.
+     * This method is designed to be used only by the {@link io.kestra.core.runners.RunContextProperty}.
      *
-     * @see RunContextProperty#asList(Class)
+     * @see io.kestra.core.runners.RunContextProperty#asList(Class)
      */
     public static <T, I> T asList(Property<T> property, PropertyContext context, Class<I> itemClazz) throws IllegalVariableEvaluationException {
         return asList(property, context, itemClazz, Map.of());
@@ -227,39 +186,37 @@ public class Property<T> {
     /**
      * Render a property with additional variables, then convert it as a list of target type.<br>
      * <p>
-     * This method is designed to be used only by the {@link RunContextProperty}.
+     * This method is designed to be used only by the {@link io.kestra.core.runners.RunContextProperty}.
      *
-     * @see RunContextProperty#asList(Class, Map)
+     * @see io.kestra.core.runners.RunContextProperty#asList(Class, Map)
      */
     @SuppressWarnings("unchecked")
     public static <T, I> T asList(Property<T> property, PropertyContext context, Class<I> itemClazz, Map<String, Object> variables) throws IllegalVariableEvaluationException {
-        if (property.skipCache || property.value == null) {
+        if (property.value == null) {
             JavaType type = MAPPER.getTypeFactory().constructCollectionLikeType(List.class, itemClazz);
-            String trimmedExpression = property.expression.trim();
-            // We need to detect if the expression is already a list or if it's a pebble expression (for eg. referencing a variable containing a list).
-            // Doing that allows us to, if it's an expression, first render then read it as a list.
-            if (trimmedExpression.startsWith("{{") && trimmedExpression.endsWith("}}")) {
-                property.value = deserialize(context.render(property.expression, variables), type);
-            }
-            // Otherwise, if it's already a list, we read it as a list first then render it from run context which handle list rendering by rendering each item of the list
-            else {
-                List<?> asRawList = deserialize(property.expression, List.class);
-                property.value = (T) asRawList.stream()
-                    .map(throwFunction(item -> {
-                        Object rendered = null;
-                        if (item instanceof String str) {
-                            rendered = context.render(str, variables);
-                        } else if (item instanceof Map map) {
-                            rendered = context.render(map, variables);
-                        }
-
-                        if (rendered != null) {
-                            return deserialize(rendered, itemClazz);
-                        }
-
-                        return item;
-                    }))
-                    .toList();
+            try {
+                String trimmedExpression = property.expression.trim();
+                // We need to detect if the expression is already a list or if it's a pebble expression (for eg. referencing a variable containing a list).
+                // Doing that allows us to, if it's an expression, first render then read it as a list.
+                if (trimmedExpression.startsWith("{{") && trimmedExpression.endsWith("}}")) {
+                    property.value = MAPPER.readValue(context.render(property.expression, variables), type);
+                }
+                // Otherwise, if it's already a list, we read it as a list first then render it from run context which handle list rendering by rendering each item of the list
+                else {
+                    List<?> asRawList = MAPPER.readValue(property.expression, List.class);
+                    property.value = (T) asRawList.stream()
+                        .map(throwFunction(item -> {
+                            if (item instanceof String str) {
+                                return MAPPER.convertValue(context.render(str, variables), itemClazz);
+                            } else if (item instanceof Map map) {
+                                return MAPPER.convertValue(context.render(map, variables), itemClazz);
+                            }
+                            return item;
+                        }))
+                        .toList();
+                }
+            } catch (JsonProcessingException e) {
+                throw new IllegalVariableEvaluationException(e);
             }
         }
 
@@ -269,9 +226,9 @@ public class Property<T> {
     /**
      * Render a property then convert it as a map of target types.<br>
      * <p>
-     * This method is designed to be used only by the {@link RunContextProperty}.
+     * This method is designed to be used only by the {@link io.kestra.core.runners.RunContextProperty}.
      *
-     * @see RunContextProperty#asMap(Class, Class)
+     * @see io.kestra.core.runners.RunContextProperty#asMap(Class, Class)
      */
     public static <T, K, V> T asMap(Property<T> property, RunContext runContext, Class<K> keyClass, Class<V> valueClass) throws IllegalVariableEvaluationException {
         return asMap(property, runContext, keyClass, valueClass, Map.of());
@@ -283,11 +240,11 @@ public class Property<T> {
      * This method is safe to be used as many times as you want as the rendering and conversion will be cached.
      * Warning, due to the caching mechanism, this method is not thread-safe.
      *
-     * @see RunContextProperty#asMap(Class, Class, Map)
+     * @see io.kestra.core.runners.RunContextProperty#asMap(Class, Class, Map)
      */
     @SuppressWarnings({"rawtypes", "unchecked"})
     public static <T, K, V> T asMap(Property<T> property, RunContext runContext, Class<K> keyClass, Class<V> valueClass, Map<String, Object> variables) throws IllegalVariableEvaluationException {
-        if (property.skipCache || property.value == null) {
+        if (property.value == null) {
             JavaType targetMapType = MAPPER.getTypeFactory().constructMapType(Map.class, keyClass, valueClass);
 
             try {
@@ -295,12 +252,12 @@ public class Property<T> {
                 // We need to detect if the expression is already a map or if it's a pebble expression (for eg. referencing a variable containing a map).
                 // Doing that allows us to, if it's an expression, first render then read it as a map.
                 if (trimmedExpression.startsWith("{{") && trimmedExpression.endsWith("}}")) {
-                    property.value = deserialize(runContext.render(property.expression, variables), targetMapType);
+                    property.value = MAPPER.readValue(runContext.render(property.expression, variables), targetMapType);
                 }
                 // Otherwise if it's already a map we read it as a map first then render it from run context which handle map rendering by rendering each entry of the map (otherwise it will fail with nested expressions in values for eg.)
                 else {
                     Map asRawMap = MAPPER.readValue(property.expression, Map.class);
-                    property.value = deserialize(runContext.render(asRawMap, variables), targetMapType);
+                    property.value = MAPPER.convertValue(runContext.render(asRawMap, variables), targetMapType);
                 }
             } catch (JsonProcessingException e) {
                 throw new IllegalVariableEvaluationException(e);

core/src/main/java/io/kestra/core/models/tasks/ExecutableTask.java

@@ -24,7 +24,7 @@ public interface ExecutableTask<T extends Output>{
      */
     List<SubflowExecution<?>> createSubflowExecutions(RunContext runContext,
                                                       FlowMetaStoreInterface flowExecutorInterface,
-                                                      FlowInterface currentFlow, Execution currentExecution,
+                                                      Flow currentFlow, Execution currentExecution,
                                                       TaskRun currentTaskRun) throws InternalException;
 
     /**

core/src/main/java/io/kestra/core/models/tasks/runners/PluginUtilsService.java

@@ -4,8 +4,10 @@ import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.core.type.TypeReference;
 import io.kestra.core.exceptions.IllegalVariableEvaluationException;
 import io.kestra.core.models.tasks.runners.TaskLogLineMatcher.TaskLogMatch;
+import io.kestra.core.runners.DefaultRunContext;
 import io.kestra.core.runners.RunContext;
 import io.kestra.core.serializers.JacksonMapper;
+import io.kestra.core.services.FlowService;
 import jakarta.validation.constraints.NotNull;
 import org.apache.commons.lang3.StringUtils;
 import org.slf4j.Logger;
@@ -36,7 +38,6 @@ import static io.kestra.core.utils.Rethrow.throwConsumer;
 abstract public class PluginUtilsService {
 
     private static final TypeReference<Map<String, String>> MAP_TYPE_REFERENCE = new TypeReference<>() {};
-    private static final TaskLogLineMatcher LOG_LINE_MATCHER = new TaskLogLineMatcher();
 
     public static Map<String, String> createOutputFiles(
         Path tempDirectory,
@@ -169,9 +170,12 @@ abstract public class PluginUtilsService {
     }
 
     public static Map<String, Object> parseOut(String line, Logger logger, RunContext runContext, boolean isStdErr, Instant customInstant) {
+
+        TaskLogLineMatcher logLineMatcher = ((DefaultRunContext) runContext).getApplicationContext().getBean(TaskLogLineMatcher.class);
+
         Map<String, Object> outputs = new HashMap<>();
         try {
-            Optional<TaskLogMatch> matches = LOG_LINE_MATCHER.matches(line, logger, runContext, customInstant);
+            Optional<TaskLogMatch> matches = logLineMatcher.matches(line, logger, runContext, customInstant);
             if (matches.isPresent()) {
                 TaskLogMatch taskLogMatch = matches.get();
                 outputs.putAll(taskLogMatch.outputs());
@@ -211,7 +215,8 @@ abstract public class PluginUtilsService {
                 realNamespace = runContext.render(namespace);
                 realFlowId = runContext.render(flowId);
                 // validate that the flow exists: a.k.a access is authorized by this namespace
-                runContext.acl().allowNamespace(realNamespace).check();
+                FlowService flowService = ((DefaultRunContext)runContext).getApplicationContext().getBean(FlowService.class);
+                flowService.checkAllowedNamespace(flowInfo.tenantId(), realNamespace, flowInfo.tenantId(), flowInfo.namespace());
             } else if (namespace != null || flowId != null) {
                 throw new IllegalArgumentException("Both `namespace` and `flowId` must be set when `executionId` is set.");
             } else {

core/src/main/java/io/kestra/core/models/tasks/runners/TaskLogLineMatcher.java

@@ -27,6 +27,7 @@ import static io.kestra.core.runners.RunContextLogger.ORIGINAL_TIMESTAMP_KEY;
  * ::{"outputs":{"key":"value"}}::
  * }</pre>
  */
+@Singleton
 public class TaskLogLineMatcher {
 
     protected static final Pattern LOG_DATA_SYNTAX = Pattern.compile("^::(\\{.*})::$");
@@ -107,4 +108,4 @@ public class TaskLogLineMatcher {
         String message
     ) {
     }
-}
+}

core/src/main/java/io/kestra/core/models/triggers/AbstractTrigger.java

@@ -82,12 +82,6 @@ abstract public class AbstractTrigger implements TriggerInterface {
     @PluginProperty(hidden = true, group = PluginProperty.CORE_GROUP)
     private boolean failOnTriggerError = false;
 
-    @PluginProperty(group = PluginProperty.CORE_GROUP)
-    @Schema(
-        title = "Specifies whether a trigger is allowed to start a new execution even if a previous run is still in progress."
-    )
-    private boolean allowConcurrent = false;
-
     /**
      * For backward compatibility: we rename minLogLevel to logLevel.
      * @deprecated use {@link #logLevel} instead

core/src/main/java/io/kestra/core/models/triggers/Schedulable.java

@@ -1,37 +1,22 @@
 package io.kestra.core.models.triggers;
 
 import io.kestra.core.exceptions.IllegalVariableEvaluationException;
-import io.kestra.core.models.annotations.PluginProperty;
 import io.kestra.core.models.conditions.ConditionContext;
 import io.kestra.core.runners.RunContext;
-import io.swagger.v3.oas.annotations.media.Schema;
 
 import java.time.ZonedDateTime;
-import java.util.Map;
 
 public interface Schedulable extends PollingTriggerInterface{
     String PLUGIN_PROPERTY_RECOVER_MISSED_SCHEDULES = "recoverMissedSchedules";
 
-    @Schema(
-        title = "The inputs to pass to the scheduled flow"
-    )
-    @PluginProperty(dynamic = true)
-    Map<String, Object> getInputs();
-
-    @Schema(
-        title = "Action to take in the case of missed schedules",
-        description = "`ALL` will recover all missed schedules, `LAST`  will only recovered the last missing one, `NONE` will not recover any missing schedule.\n" +
-            "The default is `ALL` unless a different value is configured using the global plugin configuration."
-    )
-    @PluginProperty
-    RecoverMissedSchedules getRecoverMissedSchedules();
-    
     /**
      * Compute the previous evaluation of a trigger.
      * This is used when a trigger misses some schedule to compute the next date to evaluate in the past.
      */
     ZonedDateTime previousEvaluationDate(ConditionContext conditionContext) throws IllegalVariableEvaluationException;
-    
+
+    RecoverMissedSchedules getRecoverMissedSchedules();
+
     /**
      * Load the default RecoverMissedSchedules from plugin property, or else ALL.
      */

core/src/main/java/io/kestra/core/models/triggers/Trigger.java

@@ -74,7 +74,7 @@ public class Trigger extends TriggerContext implements HasUID {
         );
     }
 
-    public static String uid(FlowInterface flow, AbstractTrigger abstractTrigger) {
+    public static String uid(Flow flow, AbstractTrigger abstractTrigger) {
         return IdUtils.fromParts(
             flow.getTenantId(),
             flow.getNamespace(),
@@ -172,7 +172,7 @@ public class Trigger extends TriggerContext implements HasUID {
 
         if (abstractTrigger instanceof PollingTriggerInterface pollingTriggerInterface) {
             try {
-                nextDate = pollingTriggerInterface.nextEvaluationDate(conditionContext, lastTrigger);
+                nextDate = pollingTriggerInterface.nextEvaluationDate(conditionContext, Optional.empty());
             } catch (InvalidTriggerConfigurationException e) {
                 disabled = true;
             }

core/src/main/java/io/kestra/core/models/triggers/TriggerService.java

@@ -6,9 +6,12 @@ import io.kestra.core.models.executions.Execution;
 import io.kestra.core.models.executions.ExecutionTrigger;
 import io.kestra.core.models.tasks.Output;
 import io.kestra.core.models.flows.State;
+import io.kestra.core.runners.DefaultRunContext;
+import io.kestra.core.runners.FlowInputOutput;
 import io.kestra.core.runners.RunContext;
 import io.kestra.core.utils.IdUtils;
 import io.kestra.core.utils.ListUtils;
+import java.time.ZonedDateTime;
 import java.util.*;
 
 public abstract class TriggerService {
@@ -48,6 +51,58 @@ public abstract class TriggerService {
         return generateExecution(IdUtils.create(), trigger, context, executionTrigger, conditionContext);
     }
 
+    public static Execution generateScheduledExecution(
+        AbstractTrigger trigger,
+        ConditionContext conditionContext,
+        TriggerContext context,
+        List<Label> labels,
+        Map<String, Object> inputs,
+        Map<String, Object> variables,
+        Optional<ZonedDateTime> scheduleDate
+    ) {
+        RunContext runContext = conditionContext.getRunContext();
+        ExecutionTrigger executionTrigger = ExecutionTrigger.of(trigger, variables);
+
+        List<Label> executionLabels = new ArrayList<>(ListUtils.emptyOnNull(labels));
+        if (executionLabels.stream().noneMatch(label -> Label.CORRELATION_ID.equals(label.key()))) {
+            // add a correlation ID if none exist
+            executionLabels.add(new Label(Label.CORRELATION_ID, runContext.getTriggerExecutionId()));
+        }
+        Execution execution = Execution.builder()
+            .id(runContext.getTriggerExecutionId())
+            .tenantId(context.getTenantId())
+            .namespace(context.getNamespace())
+            .flowId(context.getFlowId())
+            .flowRevision(conditionContext.getFlow().getRevision())
+            .variables(conditionContext.getFlow().getVariables())
+            .labels(executionLabels)
+            .state(new State())
+            .trigger(executionTrigger)
+            .scheduleDate(scheduleDate.map(date -> date.toInstant()).orElse(null))
+            .build();
+
+        Map<String, Object> allInputs = new HashMap<>();
+        // add flow inputs with default value
+        var flow = conditionContext.getFlow();
+        if (flow.getInputs() != null) {
+            flow.getInputs().stream()
+                .filter(input -> input.getDefaults() != null)
+                .forEach(input -> allInputs.put(input.getId(), input.getDefaults()));
+        }
+
+        if (inputs != null) {
+            allInputs.putAll(inputs);
+        }
+
+        // add inputs and inject defaults
+        if (!allInputs.isEmpty()) {
+            FlowInputOutput flowInputOutput = ((DefaultRunContext)runContext).getApplicationContext().getBean(FlowInputOutput.class);
+            execution = execution.withInputs(flowInputOutput.readExecutionInputs(conditionContext.getFlow(), execution, allInputs));
+        }
+
+        return execution;
+    }
+
     private static Execution generateExecution(
         String id,
         AbstractTrigger trigger,
@@ -56,7 +111,6 @@ public abstract class TriggerService {
         ConditionContext conditionContext
     ) {
         List<Label> executionLabels = new ArrayList<>(ListUtils.emptyOnNull(trigger.getLabels()));
-        executionLabels.add(new Label(Label.FROM, "trigger"));
         if (executionLabels.stream().noneMatch(label -> Label.CORRELATION_ID.equals(label.key()))) {
             // add a correlation ID if none exist
             executionLabels.add(new Label(Label.CORRELATION_ID, id));

core/src/main/java/io/kestra/core/models/triggers/multipleflows/MultipleConditionWindow.java

@@ -2,12 +2,14 @@ package io.kestra.core.models.triggers.multipleflows;
 
 import com.fasterxml.jackson.annotation.JsonIgnore;
 import io.kestra.core.models.HasUID;
+import io.kestra.core.models.flows.Flow;
 import io.kestra.core.models.flows.FlowId;
 import io.kestra.core.utils.IdUtils;
 import lombok.Builder;
 import lombok.Value;
 
 import java.time.ZonedDateTime;
+import java.util.Arrays;
 import java.util.HashMap;
 import java.util.Map;
 

core/src/main/java/io/kestra/core/models/validations/ManualConstraintViolation.java

@@ -67,11 +67,6 @@ public class ManualConstraintViolation<T> implements ConstraintViolation<T> {
             invalidValue
         )));
     }
-    public static <T> ConstraintViolationException toConstraintViolationException(
-        Set<? extends ConstraintViolation<?>> constraintViolations
-    ) {
-        return new ConstraintViolationException(constraintViolations);
-    }
 
     public String getMessageTemplate() {
         return "{messageTemplate}";

core/src/main/java/io/kestra/core/reporter/reports/FeatureUsageReport.java

@@ -23,12 +23,12 @@ import java.util.Objects;
 
 @Singleton
 public class FeatureUsageReport extends AbstractReportable<FeatureUsageReport.UsageEvent> {
-
+    
     private final FlowRepositoryInterface flowRepository;
     private final ExecutionRepositoryInterface executionRepository;
     private final DashboardRepositoryInterface dashboardRepository;
     private final boolean enabled;
-
+    
     @Inject
     public FeatureUsageReport(FlowRepositoryInterface flowRepository,
                               ExecutionRepositoryInterface executionRepository,
@@ -37,26 +37,26 @@ public class FeatureUsageReport extends AbstractReportable<FeatureUsageReport.Us
         this.flowRepository = flowRepository;
         this.executionRepository = executionRepository;
         this.dashboardRepository = dashboardRepository;
-
+        
         ServerType serverType = KestraContext.getContext().getServerType();
         this.enabled = ServerType.EXECUTOR.equals(serverType) || ServerType.STANDALONE.equals(serverType);
     }
-
+    
     @Override
     public UsageEvent report(final Instant now, TimeInterval interval) {
         return UsageEvent
             .builder()
             .flows(FlowUsage.of(flowRepository))
             .executions(ExecutionUsage.of(executionRepository, interval.from(), interval.to()))
-            .dashboards(new Count(dashboardRepository.countAllForAllTenants()))
+            .dashboards(new Count(dashboardRepository.count()))
             .build();
     }
-
+    
     @Override
     public boolean isEnabled() {
         return enabled;
     }
-
+    
     @Override
     public UsageEvent report(Instant now, TimeInterval interval, String tenant) {
         Objects.requireNonNull(tenant, "tenant is null");
@@ -67,7 +67,7 @@ public class FeatureUsageReport extends AbstractReportable<FeatureUsageReport.Us
             .executions(ExecutionUsage.of(tenant, executionRepository, interval.from(), interval.to()))
             .build();
     }
-
+    
     @SuperBuilder(toBuilder = true)
     @Getter
     @Jacksonized

core/src/main/java/io/kestra/core/repositories/DashboardRepositoryInterface.java

@@ -16,14 +16,14 @@ import java.util.Map;
 import java.util.Optional;
 
 public interface DashboardRepositoryInterface {
-
+    
     /**
      * Gets the total number of Dashboards.
      *
      * @return the total number.
      */
-    long countAllForAllTenants();
-
+    long count();
+    
     Boolean isEnabled();
 
     Optional<Dashboard> get(String tenantId, String id);

core/src/main/java/io/kestra/core/repositories/ExecutionRepositoryInterface.java

@@ -2,6 +2,7 @@ package io.kestra.core.repositories;
 
 import io.kestra.core.models.QueryFilter;
 import io.kestra.core.models.executions.Execution;
+import io.kestra.core.models.executions.TaskRun;
 import io.kestra.core.models.executions.statistics.DailyExecutionStatistics;
 import io.kestra.core.models.executions.statistics.ExecutionCount;
 import io.kestra.core.models.executions.statistics.Flow;
@@ -93,8 +94,6 @@ public interface ExecutionRepositoryInterface extends SaveRepositoryInterface<Ex
 
     Flux<Execution> findAllAsync(@Nullable String tenantId);
 
-    Flux<Execution> findAsync(String tenantId, List<QueryFilter> filters);
-
     Execution delete(Execution execution);
 
     Integer purge(Execution execution);

core/src/main/java/io/kestra/core/repositories/FlowRepositoryInterface.java

@@ -8,7 +8,6 @@ import io.kestra.plugin.core.dashboard.data.Flows;
 import io.micronaut.data.model.Pageable;
 import jakarta.annotation.Nullable;
 import jakarta.validation.ConstraintViolationException;
-import reactor.core.publisher.Flux;
 
 import java.util.List;
 import java.util.Optional;
@@ -159,8 +158,6 @@ public interface FlowRepositoryInterface extends QueryBuilderInterface<Flows.Fie
             .toList();
     }
 
-    Flux<Flow> findAsync(String tenantId, List<QueryFilter> filters);
-
     FlowWithSource create(GenericFlow flow);
 
     FlowWithSource update(GenericFlow flow, FlowInterface previous) throws ConstraintViolationException;

core/src/main/java/io/kestra/core/repositories/NamespaceFileMetadataRepositoryInterface.java

@@ -1,46 +0,0 @@
-package io.kestra.core.repositories;
-
-import io.kestra.core.models.FetchVersion;
-import io.kestra.core.models.QueryFilter;
-import io.kestra.core.models.namespaces.files.NamespaceFileMetadata;
-import io.micronaut.data.model.Pageable;
-
-import java.io.IOException;
-import java.util.List;
-import java.util.Optional;
-
-public interface NamespaceFileMetadataRepositoryInterface extends SaveRepositoryInterface<NamespaceFileMetadata> {
-    Optional<NamespaceFileMetadata> findByPath(
-        String tenantId,
-        String namespace,
-        String path
-    ) throws IOException;
-
-    default ArrayListTotal<NamespaceFileMetadata> find(
-        Pageable pageable,
-        String tenantId,
-        List<QueryFilter> filters,
-        boolean allowDeleted
-    ) {
-        return this.find(pageable, tenantId, filters, allowDeleted, FetchVersion.LATEST);
-    }
-
-    ArrayListTotal<NamespaceFileMetadata> find(
-        Pageable pageable,
-        String tenantId,
-        List<QueryFilter> filters,
-        boolean allowDeleted,
-        FetchVersion fetchBehavior
-    );
-
-    default NamespaceFileMetadata delete(NamespaceFileMetadata namespaceFileMetadata) throws IOException {
-        return this.save(namespaceFileMetadata.toBuilder().deleted(true).build());
-    }
-
-    /**
-     * Purge (hard delete) a list of namespace files metadata. If no version is specified, all versions are purged.
-     * @param namespaceFilesMetadata the list of namespace files metadata to purge
-     * @return the number of purged namespace files metadata
-     */
-    Integer purge(List<NamespaceFileMetadata> namespaceFilesMetadata);
-}

core/src/main/java/io/kestra/core/repositories/SettingRepositoryInterface.java

@@ -1,10 +1,10 @@
 package io.kestra.core.repositories;
 
 import io.kestra.core.models.Setting;
-import jakarta.validation.ConstraintViolationException;
 
 import java.util.List;
 import java.util.Optional;
+import jakarta.validation.ConstraintViolationException;
 
 public interface SettingRepositoryInterface {
     Optional<Setting> findByKey(String key);
@@ -13,7 +13,5 @@ public interface SettingRepositoryInterface {
 
     Setting save(Setting setting) throws ConstraintViolationException;
 
-    Setting internalSave(Setting setting) throws ConstraintViolationException;
-
     Setting delete(Setting setting);
 }

core/src/main/java/io/kestra/core/repositories/TriggerRepositoryInterface.java

@@ -16,8 +16,8 @@ import java.util.function.Function;
 public interface TriggerRepositoryInterface extends QueryBuilderInterface<Triggers.Fields> {
     Optional<Trigger> findLast(TriggerContext trigger);
 
-    Optional<Trigger> findByUid(String uid);
-    
+    Optional<Trigger> findByExecution(Execution execution);
+
     List<Trigger> findAll(String tenantId);
 
     List<Trigger> findAllForAllTenants();
@@ -39,13 +39,13 @@ public interface TriggerRepositoryInterface extends QueryBuilderInterface<Trigge
      * @param tenantId the tenant of the triggers
      * @return The count.
      */
-    long countAll(@Nullable String tenantId);
+    int count(@Nullable String tenantId);
 
     /**
      * Find all triggers that match the query, return a flux of triggers
+     * as the search is not paginated
      */
-    Flux<Trigger> findAsync(String tenantId, List<QueryFilter> filters);
-
+    Flux<Trigger> find(String tenantId, List<QueryFilter> filters);
 
     default Function<String, String> sortMapping() throws IllegalArgumentException {
         return Function.identity();

core/src/main/java/io/kestra/core/runners/AclChecker.java

@@ -1,50 +0,0 @@
-package io.kestra.core.runners;
-
-import javax.annotation.CheckReturnValue;
-import java.util.List;
-
-/**
- * Check if the current taskrun has access to the requested resources.
- *
- * <p>
- * IMPORTANT: remember to call the <code>check()</code> method to check the ACL.
- *
- * @see AllowedResources
- */
-public interface AclChecker {
-
-    /**Tasks that need to access resources outside their namespace should use this interface to check ACL (Allowed namespaces in EE).
-     * Allow all namespaces.
-     * <p>
-     * IMPORTANT: remember to call the <code>check()</code> method to check the ACL.
-     */
-    @CheckReturnValue
-    AllowedResources allowAllNamespaces();
-
-    /**
-     * Allow only the given namespace.
-     * <p>
-     * IMPORTANT: remember to call the <code>check()</code> method to check the ACL.
-     */
-    @CheckReturnValue
-    AllowedResources allowNamespace(String namespace);
-
-    /**
-     * Allow only the given namespaces.
-     * <p>
-     * IMPORTANT: remember to call the <code>check()</code> method to check the ACL.
-     */
-    @CheckReturnValue
-    AllowedResources allowNamespaces(List<String> namespaces);
-
-    /**
-     * Represents a set of allowed resources.
-     * Tasks that need to access resources outside their namespace should call the <code>check()</code> method to check the ACL (Allowed namespaces in EE).
-     */
-    interface AllowedResources {
-        /**
-         * Check if the current taskrun has access to the requested resources.
-         */
-        void check();
-    }
-}

core/src/main/java/io/kestra/core/runners/AclCheckerImpl.java

@@ -1,86 +0,0 @@
-package io.kestra.core.runners;
-
-import io.kestra.core.services.NamespaceService;
-import io.micronaut.context.ApplicationContext;
-
-import java.util.List;
-import java.util.Objects;
-
-class AclCheckerImpl implements AclChecker {
-    private final NamespaceService namespaceService;
-    private final RunContext.FlowInfo flowInfo;
-
-    AclCheckerImpl(ApplicationContext applicationContext, RunContext.FlowInfo flowInfo) {
-        this.namespaceService = applicationContext.getBean(NamespaceService.class);
-        this.flowInfo = flowInfo;
-    }
-
-    @Override
-    public AllowedResources allowAllNamespaces() {
-        return new AllowAllNamespaces(flowInfo, namespaceService);
-    }
-
-    @Override
-    public AllowedResources allowNamespace(String namespace) {
-        return new AllowNamespace(flowInfo, namespaceService, namespace);
-    }
-
-    @Override
-    public AllowedResources allowNamespaces(List<String> namespaces) {
-        return new AllowNamespaces(flowInfo, namespaceService, namespaces);
-    }
-
-
-    static class AllowAllNamespaces implements AllowedResources {
-        private final RunContext.FlowInfo flowInfo;
-        private final NamespaceService namespaceService;
-
-        AllowAllNamespaces(RunContext.FlowInfo flowInfo, NamespaceService namespaceService) {
-            this.flowInfo = Objects.requireNonNull(flowInfo);
-            this.namespaceService = Objects.requireNonNull(namespaceService);
-        }
-
-        @Override
-        public void check() {
-            this.namespaceService.checkAllowedAllNamespaces(flowInfo.tenantId(), flowInfo.tenantId(), flowInfo.namespace());
-        }
-    }
-
-    static class AllowNamespace implements AllowedResources {
-        private final RunContext.FlowInfo flowInfo;
-        private final NamespaceService namespaceService;
-        private final String namespace;
-
-        public AllowNamespace(RunContext.FlowInfo flowInfo, NamespaceService namespaceService, String namespace) {
-            this.flowInfo = Objects.requireNonNull(flowInfo);
-            this.namespaceService = Objects.requireNonNull(namespaceService);
-            this.namespace = Objects.requireNonNull(namespace);
-        }
-
-        @Override
-        public void check() {
-            namespaceService.checkAllowedNamespace(flowInfo.tenantId(), namespace, flowInfo.tenantId(), flowInfo.namespace());
-        }
-    }
-
-    static class AllowNamespaces implements AllowedResources {
-        private final RunContext.FlowInfo flowInfo;
-        private final NamespaceService namespaceService;
-        private final List<String> namespaces;
-
-        AllowNamespaces(RunContext.FlowInfo flowInfo, NamespaceService namespaceService, List<String> namespaces) {
-            this.flowInfo = Objects.requireNonNull(flowInfo);
-            this.namespaceService = Objects.requireNonNull(namespaceService);
-            this.namespaces = Objects.requireNonNull(namespaces);
-
-            if (namespaces.isEmpty()) {
-                throw new IllegalArgumentException("At least one namespace must be provided");
-            }
-        }
-
-        @Override
-        public void check() {
-            namespaces.forEach(namespace -> namespaceService.checkAllowedNamespace(flowInfo.tenantId(), namespace, flowInfo.tenantId(), flowInfo.namespace()));
-        }
-    }
-}

core/src/main/java/io/kestra/core/runners/DefaultRunContext.java

@@ -6,12 +6,10 @@ import com.google.common.base.CaseFormat;
 import com.google.common.collect.ImmutableMap;
 import io.kestra.core.exceptions.IllegalVariableEvaluationException;
 import io.kestra.core.metrics.MetricRegistry;
-import io.kestra.core.models.Plugin;
 import io.kestra.core.models.executions.AbstractMetricEntry;
 import io.kestra.core.models.property.Property;
 import io.kestra.core.models.tasks.Task;
 import io.kestra.core.models.triggers.AbstractTrigger;
-import io.kestra.core.plugins.PluginConfigurations;
 import io.kestra.core.services.KVStoreService;
 import io.kestra.core.storages.Storage;
 import io.kestra.core.storages.StorageInterface;
@@ -125,12 +123,7 @@ public class DefaultRunContext extends RunContext {
         this.traceParent = traceParent;
     }
 
-    /**
-     * @deprecated Plugin should not use the ApplicationContext anymore, and neither should they cast to this implementation.
-     *             Plugin should instead rely on supported API only.
-     */
     @JsonIgnore
-    @Deprecated(since = "1.2.0", forRemoval = true)
     public ApplicationContext getApplicationContext() {
         return applicationContext;
     }
@@ -237,14 +230,6 @@ public class DefaultRunContext extends RunContext {
         return runContext;
     }
 
-    @Override
-    public RunContext cloneForPlugin(Plugin plugin) {
-        PluginConfigurations pluginConfigurations = applicationContext.getBean(PluginConfigurations.class);
-        DefaultRunContext runContext = clone();
-        runContext.pluginConfiguration = pluginConfigurations.getConfigurationByPluginTypeOrAliases(plugin.getType(), plugin.getClass());
-        return runContext;
-    }
-
     /**
      * {@inheritDoc}
      */
@@ -589,21 +574,11 @@ public class DefaultRunContext extends RunContext {
         return isInitialized.get();
     }
 
-    @Override
-    public AclChecker acl() {
-        return new AclCheckerImpl(this.applicationContext, flowInfo());
-    }
-
     @Override
     public LocalPath localPath() {
         return localPath;
     }
 
-    @Override
-    public InputAndOutput inputAndOutput() {
-        return new InputAndOutputImpl(this.applicationContext, this);
-    }
-
     /**
      * Builder class for constructing new {@link DefaultRunContext} objects.
      */

core/src/main/java/io/kestra/core/runners/ExecutableUtils.java

@@ -26,6 +26,7 @@ import org.apache.commons.lang3.stream.Streams;
 import java.time.Instant;
 import java.time.ZonedDateTime;
 import java.util.*;
+import java.util.stream.Collectors;
 
 import static io.kestra.core.trace.Tracer.throwCallable;
 import static io.kestra.core.utils.Rethrow.throwConsumer;
@@ -53,10 +54,12 @@ public final class ExecutableUtils {
     }
 
     public static SubflowExecutionResult subflowExecutionResult(TaskRun parentTaskrun, Execution execution) {
+        List<TaskRunAttempt> attempts = parentTaskrun.getAttempts() == null ? new ArrayList<>() : new ArrayList<>(parentTaskrun.getAttempts());
+        attempts.add(TaskRunAttempt.builder().state(parentTaskrun.getState()).build());
         return SubflowExecutionResult.builder()
             .executionId(execution.getId())
             .state(parentTaskrun.getState().getCurrent())
-            .parentTaskRun(parentTaskrun.addAttempt(TaskRunAttempt.builder().state(parentTaskrun.getState()).build()))
+            .parentTaskRun(parentTaskrun.withAttempts(attempts))
             .build();
     }
 
@@ -64,7 +67,7 @@ public final class ExecutableUtils {
         RunContext runContext,
         FlowMetaStoreInterface flowExecutorInterface,
         Execution currentExecution,
-        FlowInterface currentFlow,
+        Flow currentFlow,
         T currentTask,
         TaskRun currentTaskRun,
         Map<String, Object> inputs,
@@ -189,11 +192,12 @@ public final class ExecutableUtils {
                 variables.put("taskRunIteration", currentTaskRun.getIteration());
             }
 
+            FlowInputOutput flowInputOutput = ((DefaultRunContext)runContext).getApplicationContext().getBean(FlowInputOutput.class);
             Instant scheduleOnDate = runContext.render(scheduleDate).as(ZonedDateTime.class).map(date -> date.toInstant()).orElse(null);
             Execution execution = Execution
                 .newExecution(
                     flow,
-                    (f, e) -> runContext.inputAndOutput().readInputs(f, e, inputs),
+                    (f, e) -> flowInputOutput.readExecutionInputs(f, e, inputs),
                     newLabels,
                     Optional.empty())
                 .withTrigger(ExecutionTrigger.builder()

core/src/main/java/io/kestra/core/runners/FlowInputOutput.java

@@ -3,13 +3,14 @@ package io.kestra.core.runners;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import io.kestra.core.encryption.EncryptionService;
 import io.kestra.core.exceptions.IllegalVariableEvaluationException;
-
-import io.kestra.core.exceptions.InputOutputValidationException;
+import io.kestra.core.exceptions.KestraRuntimeException;
 import io.kestra.core.models.executions.Execution;
 import io.kestra.core.models.flows.Data;
 import io.kestra.core.models.flows.DependsOn;
+import io.kestra.core.models.flows.Flow;
 import io.kestra.core.models.flows.FlowInterface;
 import io.kestra.core.models.flows.Input;
+import io.kestra.core.models.flows.Output;
 import io.kestra.core.models.flows.RenderableInput;
 import io.kestra.core.models.flows.Type;
 import io.kestra.core.models.flows.input.FileInput;
@@ -19,6 +20,7 @@ import io.kestra.core.models.property.Property;
 import io.kestra.core.models.property.PropertyContext;
 import io.kestra.core.models.property.URIFetcher;
 import io.kestra.core.models.tasks.common.EncryptedString;
+import io.kestra.core.models.validations.ManualConstraintViolation;
 import io.kestra.core.serializers.JacksonMapper;
 import io.kestra.core.storages.StorageContext;
 import io.kestra.core.storages.StorageInterface;
@@ -62,11 +64,11 @@ import static io.kestra.core.utils.Rethrow.throwFunction;
 public class FlowInputOutput {
     private static final Pattern URI_PATTERN = Pattern.compile("^[a-z]+:\\/\\/(?:www\\.)?[-a-zA-Z0-9@:%._\\+~#=]{1,256}\\.[a-zA-Z0-9()]{1,6}\\b(?:[-a-zA-Z0-9()@:%_\\+.~#?&\\/=]*)$");
     private static final ObjectMapper YAML_MAPPER = JacksonMapper.ofYaml();
-
+    
     private final StorageInterface storageInterface;
     private final Optional<String> secretKey;
     private final RunContextFactory runContextFactory;
-
+    
     @Inject
     public FlowInputOutput(
         StorageInterface storageInterface,
@@ -77,7 +79,7 @@ public class FlowInputOutput {
         this.runContextFactory = runContextFactory;
         this.secretKey = Optional.ofNullable(secretKey);
     }
-
+    
     /**
      * Validate all the inputs of a given execution of a flow.
      *
@@ -87,15 +89,15 @@ public class FlowInputOutput {
      * @return The list of {@link InputAndValue}.
      */
     public Mono<List<InputAndValue>> validateExecutionInputs(final List<Input<?>> inputs,
-                                                             final FlowInterface flow,
+                                                             final Flow flow,
                                                              final Execution execution,
                                                              final Publisher<CompletedPart> data) {
         if (ListUtils.isEmpty(inputs)) return Mono.just(Collections.emptyList());
-
+        
         return readData(inputs, execution, data, false)
             .map(inputData -> resolveInputs(inputs, flow, execution, inputData, false));
     }
-
+    
     /**
      * Reads all the inputs of a given execution of a flow.
      *
@@ -109,7 +111,7 @@ public class FlowInputOutput {
                                                          final Publisher<CompletedPart> data) {
         return this.readExecutionInputs(flow.getInputs(), flow, execution, data);
     }
-
+    
     /**
      * Reads all the inputs of a given execution of a flow.
      *
@@ -124,7 +126,7 @@ public class FlowInputOutput {
                                                          final Publisher<CompletedPart> data) {
         return readData(inputs, execution, data, true).map(inputData -> this.readExecutionInputs(inputs, flow, execution, inputData));
     }
-
+    
     private Mono<Map<String, Object>> readData(List<Input<?>> inputs, Execution execution, Publisher<CompletedPart> data, boolean uploadFiles) {
         return Flux.from(data)
             .publishOn(Schedulers.boundedElastic())
@@ -157,7 +159,11 @@ public class FlowInputOutput {
                             File tempFile = File.createTempFile(prefix, fileExtension);
                             try (var inputStream = fileUpload.getInputStream();
                                  var outputStream = new FileOutputStream(tempFile)) {
-                                inputStream.transferTo(outputStream);
+                                long transferredBytes = inputStream.transferTo(outputStream);
+                                if (transferredBytes == 0) {
+                                    sink.error(new KestraRuntimeException("Can't upload file: " + fileUpload.getFilename()));
+                                    return;
+                                }
                                 URI from = storageInterface.from(execution, inputId, fileName, tempFile);
                                 sink.next(Map.entry(inputId, from.toString()));
                             } finally {
@@ -208,8 +214,8 @@ public class FlowInputOutput {
             .filter(InputAndValue::enabled)
             .map(it -> {
                 //TODO check to return all exception at-once.
-                if (it.exceptions() != null && !it.exceptions().isEmpty()) {
-                    throw  InputOutputValidationException.merge(it.exceptions());
+                if (it.exception() != null) {
+                    throw it.exception();
                 }
                 return new AbstractMap.SimpleEntry<>(it.input().getId(), it.value());
             })
@@ -229,7 +235,7 @@ public class FlowInputOutput {
         }
         return MapUtils.flattenToNestedMap(resolved);
     }
-
+    
     /**
      * Utility method for retrieving types inputs.
      *
@@ -246,7 +252,7 @@ public class FlowInputOutput {
     ) {
         return resolveInputs(inputs, flow, execution, data, true);
     }
-
+    
     public List<InputAndValue> resolveInputs(
         final List<Input<?>> inputs,
         final FlowInterface flow,
@@ -293,9 +299,13 @@ public class FlowInputOutput {
                 try {
                     isInputEnabled = Boolean.TRUE.equals(runContext.renderTyped(dependsOnCondition.get()));
                 } catch (IllegalVariableEvaluationException e) {
-                    resolvable.resolveWithError(
-                        InputOutputValidationException.of("Invalid condition: " + e.getMessage())
-                    );
+                    resolvable.resolveWithError(ManualConstraintViolation.toConstraintViolationException(
+                        "Invalid condition: " + e.getMessage(),
+                        input,
+                        (Class<Input>)input.getClass(),
+                        input.getId(),
+                        this
+                    ));
                     isInputEnabled = false;
                 }
             }
@@ -315,7 +325,7 @@ public class FlowInputOutput {
                 }
             });
             resolvable.setInput(input);
-
+            
             Object value = resolvable.get().value();
 
             // resolve default if needed
@@ -328,7 +338,7 @@ public class FlowInputOutput {
             // validate and parse input value
             if (value == null) {
                 if (input.getRequired()) {
-                    resolvable.resolveWithError(InputOutputValidationException.of("Missing required input:"  + input.getId()));
+                    resolvable.resolveWithError(input.toConstraintViolationException("missing required input", null));
                 } else {
                     resolvable.resolveWithValue(null);
                 }
@@ -338,18 +348,17 @@ public class FlowInputOutput {
                     parsedInput.ifPresent(parsed -> ((Input) resolvable.get().input()).validate(parsed.getValue()));
                     parsedInput.ifPresent(typed -> resolvable.resolveWithValue(typed.getValue()));
                 } catch (ConstraintViolationException e) {
-                    Input<?> finalInput = input;
-                  Set<InputOutputValidationException> exceptions =  e.getConstraintViolations().stream()
-                      .map(c-> InputOutputValidationException.of(c.getMessage(), finalInput))
-                      .collect(Collectors.toSet());
-                    resolvable.resolveWithError(exceptions);
+                    ConstraintViolationException exception = e.getConstraintViolations().size() == 1 ?
+                        input.toConstraintViolationException(List.copyOf(e.getConstraintViolations()).getFirst().getMessage(), value) :
+                        input.toConstraintViolationException(e.getMessage(), value);
+                    resolvable.resolveWithError(exception);
                 }
             }
-        } catch (IllegalArgumentException e){
-            resolvable.resolveWithError(InputOutputValidationException.of(e.getMessage(), input));
-        }
-        catch (Exception e) {
-            resolvable.resolveWithError(InputOutputValidationException.of(e.getMessage()));
+        } catch (ConstraintViolationException e) {
+            resolvable.resolveWithError(e);
+        } catch (Exception e) {
+            ConstraintViolationException exception = input.toConstraintViolationException(e instanceof IllegalArgumentException ? e.getMessage() : e.toString(), resolvable.get().value());
+            resolvable.resolveWithError(exception);
         }
 
         return resolvable.get();
@@ -374,11 +383,11 @@ public class FlowInputOutput {
 
     @SuppressWarnings("unchecked")
     private static <T> Object resolveDefaultPropertyAs(Input<?> input, PropertyContext renderer, Class<T> clazz) throws IllegalVariableEvaluationException {
-        return Property.as((Property<T>) input.getDefaults().skipCache(), renderer, clazz);
+        return Property.as((Property<T>) input.getDefaults(), renderer, clazz);
     }
     @SuppressWarnings("unchecked")
     private static <T> Object resolveDefaultPropertyAsList(Input<?> input, PropertyContext renderer, Class<T> clazz) throws IllegalVariableEvaluationException {
-        return Property.asList((Property<List<T>>) input.getDefaults().skipCache(), renderer, clazz);
+        return Property.asList((Property<List<T>>) input.getDefaults(), renderer, clazz);
     }
 
     private RunContext buildRunContextForExecutionAndInputs(final FlowInterface flow, final Execution execution, Map<String, InputAndValue> dependencies, final boolean decryptSecrets) {
@@ -437,12 +446,8 @@ public class FlowInputOutput {
                             }
                             return entry;
                         });
-                }
-                catch (IllegalArgumentException e){
-                    throw InputOutputValidationException.of(e.getMessage(), output);
-                }
-                catch (Exception e) {
-                    throw InputOutputValidationException.of(e.getMessage());
+                } catch (Exception e) {
+                    throw output.toConstraintViolationException(e.getMessage(), current);
                 }
             })
             .filter(Optional::isPresent)
@@ -498,14 +503,14 @@ public class FlowInputOutput {
                         yield storageInterface.from(execution, id, current.toString().substring(current.toString().lastIndexOf("/") + 1), new File(current.toString()));
                     }
                 }
-                case JSON -> (current instanceof Map || current instanceof Collection<?>) ? current :  JacksonMapper.toObject(current.toString());
-                case YAML -> (current instanceof Map || current instanceof Collection<?>) ? current : YAML_MAPPER.readValue(current.toString(), JacksonMapper.OBJECT_TYPE_REFERENCE);
+                case JSON -> JacksonMapper.toObject(current.toString());
+                case YAML -> YAML_MAPPER.readValue(current.toString(), JacksonMapper.OBJECT_TYPE_REFERENCE);
                 case URI -> {
                     Matcher matcher = URI_PATTERN.matcher(current.toString());
                     if (matcher.matches()) {
                         yield current.toString();
                     } else {
-                        throw new IllegalArgumentException("Invalid URI format.");
+                        throw new IllegalArgumentException("Expected `URI` but received `" + current + "`");
                     }
                 }
                 case ARRAY, MULTISELECT -> {
@@ -535,10 +540,34 @@ public class FlowInputOutput {
         } catch (IllegalArgumentException e) {
             throw e;
         } catch (Throwable e) {
-            throw new Exception(" errors:\n```\n" + e.getMessage() + "\n```");
+            throw new Exception("Expected `" + type + "` but received `" + current + "` with errors:\n```\n" + e.getMessage() + "\n```");
         }
     }
 
+    public static Map<String, Object> renderFlowOutputs(List<Output> outputs, RunContext runContext) throws IllegalVariableEvaluationException {
+        if (outputs == null) return Map.of();
+
+        // render required outputs
+        Map<String, Object> outputsById = outputs
+            .stream()
+            .filter(output -> output.getRequired() == null || output.getRequired())
+            .collect(HashMap::new, (map, entry) -> map.put(entry.getId(), entry.getValue()), Map::putAll);
+        outputsById = runContext.render(outputsById);
+
+        // render optional outputs one by one to catch, log, and skip any error.
+        for (io.kestra.core.models.flows.Output output : outputs) {
+            if (Boolean.FALSE.equals(output.getRequired())) {
+                try {
+                    outputsById.putAll(runContext.render(Map.of(output.getId(), output.getValue())));
+                } catch (Exception e) {
+                    runContext.logger().warn("Failed to render optional flow output '{}'. Output is ignored.", output.getId(), e);
+                    outputsById.put(output.getId(), null);
+                }
+            }
+        }
+        return outputsById;
+    }
+
     /**
      * Mutable wrapper to hold a flow's input, and it's resolved value.
      */
@@ -567,30 +596,27 @@ public class FlowInputOutput {
         }
 
         public void isDefault(boolean isDefault) {
-            this.input = new InputAndValue(this.input.input(), this.input.value(), this.input.enabled(), isDefault, this.input.exceptions());
+            this.input = new InputAndValue(this.input.input(), this.input.value(), this.input.enabled(), isDefault, this.input.exception());
         }
 
         public void setInput(final Input<?> input) {
-            this.input = new InputAndValue(input, this.input.value(), this.input.enabled(), this.input.isDefault(), this.input.exceptions());
+            this.input = new InputAndValue(input, this.input.value(), this.input.enabled(), this.input.isDefault(), this.input.exception());
         }
 
         public void resolveWithEnabled(boolean enabled) {
-            this.input = new InputAndValue(this.input.input(), input.value(), enabled, this.input.isDefault(), this.input.exceptions());
+            this.input = new InputAndValue(this.input.input(), input.value(), enabled, this.input.isDefault(), this.input.exception());
             markAsResolved();
         }
 
         public void resolveWithValue(@Nullable Object value) {
-            this.input = new InputAndValue(this.input.input(), value,  this.input.enabled(), this.input.isDefault(), this.input.exceptions());
+            this.input = new InputAndValue(this.input.input(), value,  this.input.enabled(), this.input.isDefault(), this.input.exception());
             markAsResolved();
         }
 
-        public void resolveWithError(@Nullable Set<InputOutputValidationException> exception) {
+        public void resolveWithError(@Nullable ConstraintViolationException exception) {
             this.input = new InputAndValue(this.input.input(),  this.input.value(), this.input.enabled(), this.input.isDefault(), exception);
             markAsResolved();
         }
-        private void resolveWithError(@Nullable InputOutputValidationException exception){
-            resolveWithError(Collections.singleton(exception));
-        }
 
         private void markAsResolved() {
             this.isResolved = true;

core/src/main/java/io/kestra/core/runners/FlowListeners.java

@@ -5,8 +5,6 @@ import io.kestra.core.models.flows.FlowInterface;
 import io.kestra.core.models.flows.FlowWithException;
 import io.kestra.core.models.flows.FlowWithSource;
 import io.kestra.core.services.PluginDefaultService;
-import io.micronaut.context.annotation.Bean;
-import jakarta.annotation.PreDestroy;
 import lombok.SneakyThrows;
 import lombok.extern.slf4j.Slf4j;
 import io.kestra.core.queues.QueueFactoryInterface;
@@ -17,6 +15,7 @@ import io.kestra.core.services.FlowListenersInterface;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Optional;
+import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.function.BiConsumer;
 import java.util.function.Consumer;
 
@@ -27,13 +26,14 @@ import jakarta.inject.Singleton;
 @Singleton
 @Slf4j
 public class FlowListeners implements FlowListenersInterface {
+
+    private final AtomicBoolean isStarted = new AtomicBoolean(false);
     private final QueueInterface<FlowInterface> flowQueue;
     private final List<FlowWithSource> flows;
     private final List<Consumer<List<FlowWithSource>>> consumers = new ArrayList<>();
     private final List<BiConsumer<FlowWithSource, FlowWithSource>> consumersEach = new ArrayList<>();
-    private final PluginDefaultService pluginDefaultService;
 
-    private Runnable queueListenerCancellation;
+    private final PluginDefaultService pluginDefaultService;
 
     @Inject
     public FlowListeners(
@@ -49,9 +49,8 @@ public class FlowListeners implements FlowListenersInterface {
     @Override
     public void run() {
         synchronized (this) {
-            if (queueListenerCancellation == null) {
-                log.info("STARTING FLOW LISTENER: {}", this);
-                queueListenerCancellation = this.flowQueue.receive(either -> {
+            if (this.isStarted.compareAndSet(false, true)) {
+                this.flowQueue.receive(either -> {
                     FlowWithSource flow;
                     if (either.isRight()) {
                         flow = FlowWithException.from(either.getRight().getRecord(), either.getRight(), log).orElse(null);
@@ -155,18 +154,4 @@ public class FlowListeners implements FlowListenersInterface {
         // we forced a deep clone to avoid concurrency where instance are changed during iteration (especially scheduler).
         return new ArrayList<>(this.flows);
     }
-
-    @PreDestroy
-    @Override
-    public void close() throws Exception {
-        synchronized (this) {
-            boolean b = queueListenerCancellation != null;
-            log.info("THREAD STACKTRACE: {}", (Object) Thread.currentThread().getStackTrace());
-            log.info("LISTENER NOT NULL : {}", b);
-            log.info("LISTENER THIS : {}", this);
-            if (b) {
-                queueListenerCancellation.run();
-            }
-        }
-    }
 }

core/src/main/java/io/kestra/core/runners/FlowableUtils.java

@@ -11,7 +11,6 @@ import io.kestra.core.models.flows.State;
 import io.kestra.core.models.tasks.ResolvedTask;
 import io.kestra.core.models.tasks.Task;
 import io.kestra.core.serializers.JacksonMapper;
-import io.kestra.core.utils.ListUtils;
 import io.kestra.plugin.core.flow.Dag;
 
 import java.util.*;
@@ -144,13 +143,6 @@ public class FlowableUtils {
             return Collections.emptyList();
         }
 
-        // have submitted, leave
-        Optional<TaskRun> lastSubmitted = execution.findLastSubmitted(taskRuns);
-        if (lastSubmitted.isPresent()) {
-            return Collections.emptyList();
-        }
-
-
         // last success, find next
         Optional<TaskRun> lastTerminated = execution.findLastTerminated(taskRuns);
         if (lastTerminated.isPresent()) {
@@ -158,41 +150,14 @@ public class FlowableUtils {
 
             if (currentTasks.size() > lastIndex + 1) {
                 return Collections.singletonList(currentTasks.get(lastIndex + 1).toNextTaskRunIncrementIteration(execution, parentTaskRun.getIteration()));
+            } else {
+                return Collections.singletonList(currentTasks.getFirst().toNextTaskRunIncrementIteration(execution, parentTaskRun.getIteration()));
             }
         }
 
         return Collections.emptyList();
     }
 
-    public static Optional<State.Type> resolveSequentialState(
-        Execution execution,
-        List<ResolvedTask> tasks,
-        List<ResolvedTask> errors,
-        List<ResolvedTask> _finally,
-        TaskRun parentTaskRun,
-        RunContext runContext,
-        boolean allowFailure,
-        boolean allowWarning
-    ) {
-        if (ListUtils.emptyOnNull(tasks).stream()
-            .filter(resolvedTask -> !resolvedTask.getTask().getDisabled())
-            .findAny()
-            .isEmpty()) {
-            return Optional.of(State.Type.SUCCESS);
-        }
-
-        return resolveState(
-            execution,
-            tasks,
-            errors,
-            _finally,
-            parentTaskRun,
-            runContext,
-            allowFailure,
-            allowWarning
-        );
-    }
-
     public static Optional<State.Type> resolveState(
         Execution execution,
         List<ResolvedTask> tasks,
@@ -248,7 +213,7 @@ public class FlowableUtils {
             }
         } else {
             // first call, the error flow is not ready, we need to notify the parent task that can be failed to init error flows
-            if (execution.hasFailedNoRetry(tasks, parentTaskRun) || terminalState == State.Type.FAILED) {
+            if (execution.hasFailed(tasks, parentTaskRun) || terminalState == State.Type.FAILED) {
                 return Optional.of(execution.guessFinalState(tasks, parentTaskRun, allowFailure, allowWarning, terminalState));
             }
         }

core/src/main/java/io/kestra/core/runners/InputAndOutput.java

@@ -1,29 +0,0 @@
-package io.kestra.core.runners;
-
-import io.kestra.core.exceptions.IllegalVariableEvaluationException;
-import io.kestra.core.models.executions.Execution;
-import io.kestra.core.models.flows.FlowInterface;
-import io.kestra.core.models.flows.Output;
-
-import java.util.List;
-import java.util.Map;
-
-/**
- * InputAndOutput could be used to work with flow execution inputs and outputs.
- */
-public interface InputAndOutput {
-    /**
-     * Reads the inputs of a flow execution.
-     */
-    Map<String, Object> readInputs(FlowInterface flow, Execution execution, Map<String, Object> inputs);
-
-    /**
-     * Processes the outputs of a flow execution (parse them based on their types).
-     */
-    Map<String, Object> typedOutputs(FlowInterface flow, Execution execution, Map<String, Object> rOutputs);
-
-    /**
-     * Render flow execution outputs.
-     */
-    Map<String, Object> renderOutputs(List<Output> outputs) throws IllegalVariableEvaluationException;
-}

core/src/main/java/io/kestra/core/runners/InputAndOutputImpl.java

@@ -1,56 +0,0 @@
-package io.kestra.core.runners;
-
-import io.kestra.core.exceptions.IllegalVariableEvaluationException;
-import io.kestra.core.models.executions.Execution;
-import io.kestra.core.models.flows.FlowInterface;
-import io.kestra.core.models.flows.Output;
-import io.micronaut.context.ApplicationContext;
-
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-
-class InputAndOutputImpl implements InputAndOutput {
-    private final FlowInputOutput flowInputOutput;
-    private final RunContext runContext;
-
-    InputAndOutputImpl(ApplicationContext applicationContext, RunContext runContext) {
-        this.flowInputOutput = applicationContext.getBean(FlowInputOutput.class);
-        this.runContext = runContext;
-    }
-
-    @Override
-    public Map<String, Object> readInputs(FlowInterface flow, Execution execution, Map<String, Object> inputs) {
-        return flowInputOutput.readExecutionInputs(flow, execution, inputs);
-    }
-
-    @Override
-    public Map<String, Object> typedOutputs(FlowInterface flow, Execution execution, Map<String, Object> rOutputs) {
-        return flowInputOutput.typedOutputs(flow, execution, rOutputs);
-    }
-
-    @Override
-    public Map<String, Object> renderOutputs(List<Output> outputs) throws IllegalVariableEvaluationException {
-        if (outputs == null) return Map.of();
-
-        // render required outputs
-        Map<String, Object> outputsById = outputs
-            .stream()
-            .filter(output -> output.getRequired() == null || output.getRequired())
-            .collect(HashMap::new, (map, entry) -> map.put(entry.getId(), entry.getValue()), Map::putAll);
-        outputsById = runContext.render(outputsById);
-
-        // render optional outputs one by one to catch, log, and skip any error.
-        for (io.kestra.core.models.flows.Output output : outputs) {
-            if (Boolean.FALSE.equals(output.getRequired())) {
-                try {
-                    outputsById.putAll(runContext.render(Map.of(output.getId(), output.getValue())));
-                } catch (Exception e) {
-                    runContext.logger().warn("Failed to render optional flow output '{}'. Output is ignored.", output.getId(), e);
-                    outputsById.put(output.getId(), null);
-                }
-            }
-        }
-        return outputsById;
-    }
-}