Add Copilot suggestions

Signed-off-by: Lorna-Kelly <lorna.kelly@ibm.com>

Author: Lorna-Kelly

adr/v1.0-adr-shared-workflow-editor.md

@@ -6,10 +6,10 @@ Proposed
 
 ## Context
 
-There is a need for a shared editor for the CNCF Workflow Specification that
+There is a need for a shared editor for the CNCF Serverless Workflow Specification that
 can be used consistently by multiple implementations (e.g. Quarkus Flow,
 SonataFlow, ZigFlow, Synapse, Lemline), as different tools provide inconsistent
-authoring and visualisation experiences, leading to duplicated effort and fragmented tooling.
+authoring and visualization experiences, leading to duplicated effort and fragmented tooling.
 
 Today we have:
 
@@ -24,16 +24,16 @@ We want to converge on a single core editor stack driven by the CNCF
 Serverless Workflow Specification, with a collaboration model that allows
 multiple vendors/maintainers to contribute and embed it.
 
-This ADR formalises the decision to build a shared editor, with a strictly
+This ADR formalizes the decision to build a shared editor, with a strictly
 scoped MVP, aligned with the proposal outlined in
-[specification#1131](https://github.com/serverlessworkflow/specification/issues/1131) and further refined through MVP
+[serverlessworkflow/specification#1131](https://github.com/serverlessworkflow/specification/issues/1131) and further refined through MVP
 scoping discussions.
 
 ## Decision
 
 After evaluating existing solutions, potential approaches, and their
 associated dependencies, we decided to build the editor from scratch to
-retain full ownership of the architecture and release process, prioritise
+retain full ownership of the architecture and release process, prioritize
 architectural simplicity, enable CNCF hosting and long-term sustainability,
 at the cost of slower initial delivery.
 
@@ -51,14 +51,14 @@ vendor.
 ### Proposed model
 
 - Repository ownership
-   - New repo under CNCF Workflow org, e.g. `workflow-spec-editor` (name TBD).
+   - New repo under the CNCF Serverless Workflow Specification org, e.g. `workflow-spec-editor` (name TBD).
 
 - Maintainers
 
    - Initial maintainers: representatives from at least:
-     - CNCF Workflow Spec maintainers
+     - CNCF Serverless Workflow Specification maintainers
      - Quarkus Flow / SonataFlow
-     - Other interested engine maintainers (e.g. Zigflow / Synapse / Lemline etc.).
+     - Other interested engine maintainers (e.g. ZigFlow / Synapse / Lemline etc.).
 
 - Decision process
 
@@ -90,8 +90,8 @@ vendor.
 ### 1. In Scope
 
 - Read-only visual representation of YAML / JSON workflow definitions
-- Visualise all task types and transitions
-- Fully expanded nested task visualisation
+- Visualize all task types and transitions
+- Fully expanded nested task visualization
 - Indication of basic validation issues
 - Editor available via NPM package
 - A simple demo app showing how to embed the editor as a component
@@ -113,10 +113,11 @@ vendor.
 
 ### 4. Validation
 
-- The editor assumes workflows provided by the backend are valid.
-   - Edge cases to handle: validation discrepancies between the TypeScript SDK and the backend (runtime implementation).
-- If rendering is possible, display warnings as needed.
-- If rendering is not possible, provide clear error feedback to user.
+- The editor performs lightweight client-side schema/structural validation to determine whether the workflow can be rendered.
+- The backend (runtime implementation) remains the source of truth for full specification conformance and advanced validation rules.
+  - Edge cases to handle: validation discrepancies between the TypeScript SDK and the backend (runtime implementation).
+- If structural validation passes and rendering is possible, render the workflow.
+- If structural validation fails and rendering is not possible, do not attempt to render a partial/invalid diagram; provide clear, actionable error feedback to the user instead.
 
 ### 5. Nested Tasks & Layout Strategy
 
@@ -126,8 +127,8 @@ vendor.
 
 ### 6. Development & Tooling Decisions Overview (MVP)
 
-- Web based implementation
-- React/TypeScript based architecture
+- Web-based implementation
+- React/TypeScript-based architecture
 - Diagram rendering via React Flow
 - Webpack for application bundling
 - Jest for unit testing
@@ -141,8 +142,8 @@ vendor.
 
 ### Positive
 
-- CNCF aligned ownership
-- Lowers the entry barrier to the CNCF Serverless Workflow spec
+- CNCF-aligned ownership
+- Lowers the entry barrier to the CNCF Serverless Workflow Specification
 - Encourages understanding and usage of workflow semantics
 - Reduces duplicated tooling effort across runtimes
 - Provides a shared, consistent user experience