Commit f134fd6
authored
ci: use workspace protocol for internal
## Summary
Fixes the publish workflow failure that surfaced after the pnpm
migration (#616) merged.
### What broke
After lerna bumped versions in [run
25099449315](https://github.com/apify/apify-shared-js/actions/runs/25099449315/job/73544484277),
it called `pnpm install --lockfile-only --ignore-scripts` to refresh the
lockfile. pnpm then tried to fetch the newly bumped versions (e.g.
`@apify/consts@^2.52.2`) from the registry — which don't exist yet
because lerna hasn't published them. The publish step exited 1 with
`ERR_PNPM_NO_MATCHING_VERSION`.
### Fix
Switches every internal `@apify/<package>` dependency from a registry
range (e.g. `^2.52.1`) to the pnpm [`workspace:^`
protocol](https://pnpm.io/workspaces#workspace-protocol-workspace). pnpm
now resolves them as workspace links, and lerna replaces `workspace:^`
with the actual published version at publish time — same pattern
apify-storage-local-js and crawlee already use.
Lockfile diff confirms the switch is purely the resolution mechanism
(`version: 2.52.1` → `version: link:../consts`); no transitive dep
changes.
## Test plan
- [x] `pnpm install` — clean
- [x] `pnpm lint` — clean
- [x] `pnpm test` — 459/459 passing
- [x] `pnpm build` — all 15 packages built
- [ ] Next merge to master triggers a successful publish@apify/* deps (#625)1 parent d293765 commit f134fd6
9 files changed
Lines changed: 42 additions & 80 deletions
File tree
- packages
- actor-memory-expression
- input_schema
- input_secrets
- json_schemas
- log
- markdown
- pseudo_url
- utilities
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
52 | | - | |
| 51 | + | |
| 52 | + | |
53 | 53 | | |
54 | 54 | | |
55 | 55 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
52 | | - | |
53 | | - | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
47 | | - | |
48 | | - | |
| 47 | + | |
| 48 | + | |
49 | 49 | | |
50 | 50 | | |
51 | 51 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
58 | | - | |
| 58 | + | |
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
| 51 | + | |
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
52 | | - | |
| 51 | + | |
| 52 | + | |
53 | 53 | | |
54 | 54 | | |
55 | 55 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
47 | | - | |
| 47 | + | |
48 | 48 | | |
49 | 49 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
52 | | - | |
| 51 | + | |
| 52 | + | |
53 | 53 | | |
54 | 54 | | |
55 | 55 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
0 commit comments