@backstage/plugin-scaffolder-backend 1.22.0-next.2 → 1.22.0

package/CHANGELOG.md

@@ -1,5 +1,51 @@
 # @backstage/plugin-scaffolder-backend
 
+## 1.22.0
+
+### Minor Changes
+
+- e9663a9: Move away from using `ctx.logStream`
+- aa543c9: Migrate plugin to use the new auth services, add an optional service discovery to the router options and change the permissions type to be `PermissionsService`.
+- e9663a9: Enable the redaction of secrets using the redacting logger and the secrets from the `TaskSpec`
+- c6b132e: Introducing checkpoints for scaffolder task action idempotency
+
+### Patch Changes
+
+- 984abfa: Fixing the lost of the initial state after a task recovery.
+- 703ebc9: Fix support for unauthenticated requests to create scaffolder tasks
+- f44589d: Introduced `createMockActionContext` to unify the way of creating scaffolder mock context.
+
+  It will help to maintain tests in a long run during structural changes of action context.
+
+- 0fb419b: Updated dependency `uuid` to `^9.0.0`.
+  Updated dependency `@types/uuid` to `^9.0.0`.
+- bbd1fe1: Made "checkpoint" on scaffolder action context non-optional
+- Updated dependencies
+  - @backstage/plugin-scaffolder-node@0.4.0
+  - @backstage/plugin-scaffolder-backend-module-azure@0.1.6
+  - @backstage/plugin-scaffolder-backend-module-bitbucket-cloud@0.1.4
+  - @backstage/plugin-scaffolder-backend-module-bitbucket-server@0.1.4
+  - @backstage/plugin-scaffolder-backend-module-bitbucket@0.2.4
+  - @backstage/backend-common@0.21.4
+  - @backstage/integration@1.9.1
+  - @backstage/plugin-auth-node@0.4.9
+  - @backstage/config@1.2.0
+  - @backstage/errors@1.2.4
+  - @backstage/backend-plugin-api@0.6.14
+  - @backstage/plugin-scaffolder-backend-module-gerrit@0.1.6
+  - @backstage/plugin-scaffolder-backend-module-github@0.2.4
+  - @backstage/plugin-scaffolder-backend-module-gitlab@0.3.0
+  - @backstage/plugin-scaffolder-backend-module-gitea@0.1.4
+  - @backstage/plugin-permission-common@0.7.13
+  - @backstage/plugin-catalog-node@1.8.0
+  - @backstage/catalog-client@1.6.1
+  - @backstage/backend-tasks@0.5.19
+  - @backstage/plugin-permission-node@0.7.25
+  - @backstage/plugin-catalog-backend-module-scaffolder-entity-model@0.1.11
+  - @backstage/catalog-model@1.4.5
+  - @backstage/types@1.1.1
+  - @backstage/plugin-scaffolder-common@1.5.1
+
 ## 1.22.0-next.2
 
 ### Minor Changes

package/alpha/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@backstage/plugin-scaffolder-backend",
-  "version": "1.22.0-next.2",
+  "version": "1.22.0",
   "main": "../dist/alpha.cjs.js",
   "types": "../dist/alpha.d.ts"
 }

package/dist/alpha.cjs.js

@@ -4,7 +4,7 @@ Object.defineProperty(exports, '__esModule', { value: true });
 
 var alpha = require('@backstage/plugin-scaffolder-common/alpha');
 var pluginPermissionNode = require('@backstage/plugin-permission-node');
-var router = require('./cjs/router-52d79583.cjs.js');
+var router = require('./cjs/router-77345506.cjs.js');
 var backendPluginApi = require('@backstage/backend-plugin-api');
 var backendCommon = require('@backstage/backend-common');
 var integration = require('@backstage/integration');

package/dist/cjs/{router-52d79583.cjs.js → router-77345506.cjs.js}

@@ -1615,6 +1615,7 @@ class DatabaseTaskStore {
     try {
       const spec = JSON.parse(result.spec);
       const secrets = result.secrets ? JSON.parse(result.secrets) : void 0;
+      const state = result.state ? JSON.parse(result.state).state : void 0;
       return {
         id: result.id,
         spec,
@@ -1622,7 +1623,8 @@ class DatabaseTaskStore {
         lastHeartbeatAt: parseSqlDateToIsoString(result.last_heartbeat_at),
         createdAt: parseSqlDateToIsoString(result.created_at),
         createdBy: (_a = result.created_by) != null ? _a : void 0,
-        secrets
+        secrets,
+        state
       };
     } catch (error) {
       throw new Error(`Failed to parse spec of task '${taskId}', ${error}`);
@@ -1659,6 +1661,15 @@ class DatabaseTaskStore {
       if (updateCount < 1) {
         return void 0;
       }
+      const getState = () => {
+        try {
+          return task.state ? JSON.parse(task.state).state : void 0;
+        } catch (error) {
+          throw new Error(
+            `Failed to parse state of the task '${task.id}', ${error}`
+          );
+        }
+      };
       const secrets = this.parseTaskSecrets(task);
       return {
         id: task.id,
@@ -1667,7 +1678,8 @@ class DatabaseTaskStore {
         lastHeartbeatAt: task.last_heartbeat_at,
         createdAt: task.created_at,
         createdBy: (_a = task.created_by) != null ? _a : void 0,
-        secrets
+        secrets,
+        state: getState()
       };
     });
   }
@@ -1843,7 +1855,7 @@ class DatabaseTaskStore {
       taskIdsToRecover.push(...result.map((i) => i.id));
       for (const { id, spec } of result) {
         const taskSpec = JSON.parse(spec);
-        await this.db("task_events").insert({
+        await tx("task_events").insert({
           task_id: id,
           event_type: "recovered",
           body: JSON.stringify({
@@ -2077,7 +2089,8 @@ class StorageTaskBroker {
             taskId: pendingTask.id,
             spec: pendingTask.spec,
             secrets: pendingTask.secrets,
-            createdBy: pendingTask.createdBy
+            createdBy: pendingTask.createdBy,
+            state: pendingTask.state
           },
           this.storage,
           abortController.signal,
@@ -3632,4 +3645,4 @@ exports.createRouter = createRouter;
 exports.createWaitAction = createWaitAction;
 exports.scaffolderActionRules = scaffolderActionRules;
 exports.scaffolderTemplateRules = scaffolderTemplateRules;
-//# sourceMappingURL=router-52d79583.cjs.js.map
+//# sourceMappingURL=router-77345506.cjs.js.map