Skip to content

Exclude private workspaces from Changesets publishing#1

Closed
erossignon with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-github-actions-job
Closed

Exclude private workspaces from Changesets publishing#1
erossignon with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-github-actions-job

Conversation

Copilot AI commented Jun 10, 2026

Copy link
Copy Markdown

The Version & Publish workflow was failing because Changesets attempted to publish internal workspaces that should never go to npm. This change aligns the release configuration with the repository’s intended publish surface.

  • Release scope

    • remove @node-i3x/app and @node-i3x/demo-embedded from the Changesets fixed-version publish group
    • add both workspaces to the Changesets ignore list so release jobs only publish the four public packages
  • Workspace publish metadata

    • mark packages/app and packages/demo-embedded as private: true
    • make the workspace manifests reflect their non-publishable role directly, instead of relying only on workflow behavior
  • Regression coverage

    • add a focused test that locks the release contract:
      • fixed-version publishing applies only to core, opcua-connector, pseudo-session-connector, and rest-server
      • app and demo-embedded remain private
{
  "fixed": [[
    "@node-i3x/core",
    "@node-i3x/opcua-connector",
    "@node-i3x/pseudo-session-connector",
    "@node-i3x/rest-server"
  ]],
  "ignore": [
    "@node-i3x/app",
    "@node-i3x/demo-embedded"
  ]
}

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'Version & Publish' Exclude private workspaces from Changesets publishing Jun 10, 2026
Copilot AI requested a review from erossignon June 10, 2026 09:01
Comment thread packages/app/package.json
@@ -1,5 +1,6 @@
{
"name": "@node-i3x/app",
"private": true,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

App should now be public

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

- "private": true,
+ "private": false,

@@ -1,5 +1,6 @@
{
"name": "@node-i3x/demo-embedded",
"private": true,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

demo-embeddedshould now be public

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

- "private": true,
+ "private": false,

Comment thread packages/app/package.json
@@ -1,5 +1,6 @@
{
"name": "@node-i3x/app",
"private": true,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

- "private": true,
+ "private": false,

@@ -1,5 +1,6 @@
{
"name": "@node-i3x/demo-embedded",
"private": true,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

- "private": true,
+ "private": false,

@erossignon erossignon marked this pull request as ready for review June 10, 2026 09:19
Copilot AI requested a review from erossignon June 10, 2026 09:19
@erossignon erossignon closed this Jun 13, 2026
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.

3 participants