Skip to content
Open
8 changes: 5 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
<!--
Copyright 2021-Present The Serverless Workflow Specification Authors

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
Expand All @@ -15,11 +15,13 @@
-->

# editor

CNCF Serverless Workflow Specification Visual Editor

## Prerequisites

To build and run the editor locally, you will need:

- **Node.js 22** (current LTS version; see https://nodejs.org/)
- **pnpm 10.31.0**

Expand Down
34 changes: 34 additions & 0 deletions netlify.toml
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
#
# Copyright 2021-Present The Serverless Workflow Specification Authors
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#

[build.environment]
NODE_VERSION = "24"
PNPM_VERSION = "10.31.0"
PREVIEW_PACKAGE_NAME = "@serverlessworkflow/diagram-editor"

[build]
command = """
set -e
pnpm -F "${PREVIEW_PACKAGE_NAME}..." build:dev
pnpm -F "$PREVIEW_PACKAGE_NAME" build:storybook
"""
publish = "packages/serverless-workflow-diagram-editor/dist-storybook"
ignore = """
set -e -o pipefail
git fetch --no-tags --depth=1 origin main:refs/remotes/origin/main
npm i -g "pnpm@$PNPM_VERSION"
! pnpm -F '...[origin/main]' -rc exec 'echo "$PNPM_PACKAGE_NAME"' | grep -Fqx "$PREVIEW_PACKAGE_NAME"
Comment on lines +31 to +33
Copy link

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

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

The ignore check hard-codes main (git fetch ... origin main and ...[origin/main]). This will behave incorrectly if the default branch changes or if a PR targets a non-main base branch. Consider deriving the base ref dynamically (e.g., from Netlify/GitHub-provided PR base branch env) so preview builds are skipped/run against the actual PR base.

Suggested change
git fetch --no-tags --depth=1 origin main:refs/remotes/origin/main
npm i -g "pnpm@$PNPM_VERSION"
! pnpm -F '...[origin/main]' -rc exec 'echo "$PNPM_PACKAGE_NAME"' | grep -Fqx "$PREVIEW_PACKAGE_NAME"
BASE_REF="${GITHUB_BASE_REF:-$(git symbolic-ref --quiet --short refs/remotes/origin/HEAD | sed 's#^origin/##')}"
git fetch --no-tags --depth=1 origin "$BASE_REF:refs/remotes/origin/$BASE_REF"
npm i -g "pnpm@$PNPM_VERSION"
! pnpm -F "...[origin/$BASE_REF]" -rc exec 'echo "$PNPM_PACKAGE_NAME"' | grep -Fqx "$PREVIEW_PACKAGE_NAME"

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I don't want to introduce too much logic, deriving the branch ref and risking to have bugs with:

BASE_REF="${GITHUB_BASE_REF:-$(git symbolic-ref --quiet --short refs/remotes/origin/HEAD | sed 's#^origin/##')}"

"""