Skip to content

[Fix #1188] Replace serverlessworkflow references with open-workflow-specification - #1189

Merged
cdavernas merged 2 commits into
open-workflow-specification:mainfrom
ricardozanini:issue/1188
Oct 9, 2026
Merged

cdavernas merged 2 commits into
open-workflow-specification:mainfrom
ricardozanini:issue/1188

Conversation

@ricardozanini

@ricardozanini ricardozanini commented Aug 27, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

This PR fixes #1188 by replacing all references to the old serverlessworkflow domain and organization with the new open-workflow-specification domain and ensuring event type namespaces follow proper reverse-domain notation.

Changes

  • Replaced serverlessworkflow.io with open-workflow-specification.org in event URIs
  • Updated github.com/serverlessworkflow/* to github.com/open-workflow-specification/*
  • Changed io.serverlessworkflow.* event types to org.open-workflow-specification.* (correct reverse-domain notation for DNS open-workflow-specification.org)
  • Updated catalog and raw.githubusercontent.com URLs to new organization
  • Updated GitHub badge URLs to reference new organization

Files Updated

  • Event type URIs in examples (YAML files)
  • Documentation (dsl.md, dsl-reference.md, README.md)
  • Contributing guidelines and security documentation
  • ADRs and CTK documentation

Notes

  • VS Code Marketplace URL kept as-is (to be reviewed separately)
  • Historical ADR reference preserved for accuracy
  • DNS from old domain redirects to new one, so this is not a breaking change

Test Plan

  • Verified all serverlessworkflow.io references replaced
  • Verified all GitHub organization URLs updated
  • Verified all event type URIs updated with correct reverse-domain notation
  • Only intentional references remain (Marketplace URL, historical ADR)

Copilot AI lite review requested due to automatic review settings August 27, 2026 14:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI lite review requested due to automatic review settings October 9, 2026 15:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Event identifier renaming may break existing consumers, and the README contains an invalid SDK repository link.

1 open finding

🧠 Review effort: Lite

Comment thread dsl.md
@ricardozanini
ricardozanini force-pushed the issue/1188 branch 2 times, most recently from 3a5b334 to 395bafe Compare October 9, 2026 15:53
Copilot AI lite review requested due to automatic review settings October 9, 2026 15:53
@ricardozanini

Copy link
Copy Markdown
Collaborator Author

@cdavernas I think we should preserve the events namespace or it might break implementations.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Resolve the script argument compatibility break, event namespace migration strategy, and inconsistent stdin/example documentation.

1 open finding
1 resolved since last review

🧠 Review effort: Lite

Comment thread schema/workflow.yaml Outdated
Copilot AI lite review requested due to automatic review settings October 9, 2026 15:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Compatibility, schema-breaking, and script-runtime concerns remain unresolved.

1 open finding

🧠 Review effort: Lite

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Remaining legacy references, schema/documentation inconsistencies, and event-contract compatibility concerns must be addressed.

4 open findings

🧠 Review effort: Lite

Comment thread dsl-reference.md Outdated
Comment thread dsl-reference.md
Comment thread dsl.md

@cdavernas cdavernas left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM! Cheers :heart

@ricardozanini

Copy link
Copy Markdown
Collaborator Author

@copilot resolve the merge conflicts in this pull request

@ricardozanini ricardozanini reopened this Oct 9, 2026
Copilot AI lite review requested due to automatic review settings October 9, 2026 19:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Align the Script documentation with the schema and update the remaining legacy lifecycle event namespaces.

1 open finding
4 resolved since last review

🧠 Review effort: Lite

Comment thread dsl.md Outdated
Copilot AI lite review requested due to automatic review settings October 9, 2026 19:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Correct the remaining lifecycle namespaces and migration-tool organization reference.

0 open findings

1 resolved since last review

🧠 Review effort: Lite

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Update the remaining old event namespaces in dsl.md and repository ownership reference in the migration-tool ADR.

1 open finding

🧠 Review effort: Lite

Comment thread dsl.md Outdated
Copilot AI lite review requested due to automatic review settings October 9, 2026 20:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

…erences with open-workflow-specification

This PR fixes open-workflow-specification#1188 by replacing all references to the old serverlessworkflow
domain and organization with the new open-workflow-specification domain:

- Replaced serverlessworkflow.io with open-workflow-specification.org
- Updated github.com/serverlessworkflow/* to github.com/open-workflow-specification/*
- Changed io.serverlessworkflow.* event types to org.open-workflow-specification.*
  (correct reverse-domain notation for DNS open-workflow-specification.org)
- Updated catalog and raw.githubusercontent.com URLs
- Updated GitHub badge URLs
- Updated documentation and ADRs

NOTE: Event type namespace change is breaking for current runtimes and brokers
listening to the old io.serverlessworkflow.* event types.

Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com>
…space

Following the project rebranding, update the canonical event type namespace
from io.serverlessworkflow.* to org.open-workflow-specification.* (correct
reverse-domain notation for open-workflow-specification.org).

Add deprecation notice: implementations MUST support both namespaces during
the 1.x lifecycle and MAY drop io.serverlessworkflow.* in 2.0.

Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com>
@cdavernas
cdavernas merged commit 0cd062d into open-workflow-specification:main Oct 9, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace serverlessworkflow.io with open-workflow-specification.org in events URIs

3 participants