build: say beside the install why yarn prints the workspaces warning (closes #595) #709

Merged
clawbot merged 1 commits from issue-595-workspaces-warning into next 2026-10-28 00:28:28 +01:00
Collaborator

Yarn 1 prints "Workspaces can only be enabled in private projects." for each dependency whose package.json has a workspaces field but no "private": true. Two dependencies publish the field:

  • eslint 10.8.1. Every release from 10.5.0 to the latest, 10.13.0, has the field.
  • babel-plugin-istanbul 7.0.1, which jest pulls in. Every 8.x release also has the field. jest 30.5 and later require 8.x.

No update removes the warning. Removing it would mean downgrading eslint to 10.4.1 and forcing babel-plugin-istanbul to 7.0.0, below the ^7.0.1 that jest requires. So the change is a one-line comment beside the install in script/bootstrap and in the Dockerfile. The comment names both packages and says why the warning stays.

Judgement call: the Dockerfile has three script/bootstrap installs, and the comment sits at the first one only.

Model: opus-5-5

Yarn 1 prints "Workspaces can only be enabled in private projects." for each dependency whose `package.json` has a `workspaces` field but no `"private": true`. Two dependencies publish the field: - `eslint` 10.8.1. Every release from 10.5.0 to the latest, 10.13.0, has the field. - `babel-plugin-istanbul` 7.0.1, which `jest` pulls in. Every 8.x release also has the field. `jest` 30.5 and later require 8.x. No update removes the warning. Removing it would mean downgrading `eslint` to 10.4.1 and forcing `babel-plugin-istanbul` to 7.0.0, below the `^7.0.1` that `jest` requires. So the change is a one-line comment beside the install in `script/bootstrap` and in the `Dockerfile`. The comment names both packages and says why the warning stays. Judgement call: the `Dockerfile` has three `script/bootstrap` installs, and the comment sits at the first one only. Model: opus-5-5
clawbot added the needs-review label 2026-10-28 00:18:50 +01:00
clawbot self-assigned this 2026-10-28 00:18:51 +01:00
clawbot added 1 commit 2026-10-28 00:18:51 +01:00
build: say beside the install why yarn prints the workspaces warning (closes #595)
check / check (push) Waiting to run
e2e / e2e-chrome (push) Waiting to run
e2e / e2e-firefox (push) Waiting to run
e1cc103353
Yarn 1 warns about every dependency that publishes a workspaces field
without "private": true. eslint 10.8.1 publishes one, and so does
babel-plugin-istanbul 7.0.1, which jest pulls in. No newer release of
either package drops the field. Older releases lack it, but taking one
means downgrading eslint or forcing babel-plugin-istanbul below the
^7.0.1 that jest requires. A one-line comment in script/bootstrap and
in the Dockerfile now names both packages.

Model: opus-5-5
Author
Collaborator

Review: PASS

The comment names the two packages that cause the warning and says why the warning stays, beside the install in script/bootstrap and in the Dockerfile.

Judgement call: one comment in the Dockerfile is enough, because all three installs run script/bootstrap, which carries the same comment beside its yarn install.

Follow-up: the new comment is 103 columns wide in both files, where every other comment there wraps at 80; two lines would match.

Model: opus-5-5

**Review: PASS** The comment names the two packages that cause the warning and says why the warning stays, beside the install in `script/bootstrap` and in the `Dockerfile`. Judgement call: one comment in the `Dockerfile` is enough, because all three installs run `script/bootstrap`, which carries the same comment beside its `yarn install`. Follow-up: the new comment is 103 columns wide in both files, where every other comment there wraps at 80; two lines would match. Model: opus-5-5
clawbot merged commit a33a40a6fa into next 2026-10-28 00:28:28 +01:00
clawbot deleted branch issue-595-workspaces-warning 2026-10-28 00:28:28 +01:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#709