Skip to content

fix(glsp-client-theia): declare the @theia/process GLSP requires - #12

Merged
martin-fleck-at merged 1 commit into
mainfrom
fix/theia-process-peer
Sep 9, 2026
Merged

fix(glsp-client-theia): declare the @theia/process GLSP requires#12
martin-fleck-at merged 1 commit into
mainfrom
fix/theia-process-peer

Conversation

@martin-fleck-at

Copy link
Copy Markdown
Contributor

@eclipse-glsp/theia-integration requires it at runtime and declares it in no block of its own, so the edge resolves only by hoisting luck.

  • Declare @theia/process as a peer with a devDependency mirror, matching how every other runtime requirement of this package is declared
  • Add its row to the README peer table, which the readme gate requires for every declared peer
  • The suite is green with and without this, so it is no evidence: the break appears only under an install that re-resolves and moves the hoist, which is where it was found

@eclipse-glsp/theia-integration requires it at runtime and declares it
in no block of its own, so the edge resolves only by hoisting luck.

- Declare @theia/process as a peer with a devDependency mirror, matching
  how every other runtime requirement of this package is declared
- Add its row to the README peer table, which the readme gate requires
  for every declared peer
- The suite is green with and without this, so it is no evidence: the
  break appears only under an install that re-resolves and moves the
  hoist, which is where it was found
@martin-fleck-at
martin-fleck-at merged commit 9a14d28 into main Sep 9, 2026
4 checks passed
@martin-fleck-at
martin-fleck-at deleted the fix/theia-process-peer branch September 9, 2026 11:26
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.

1 participant