Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update React, Monaco, and replace rescripts with rewired #3

Merged
merged 5 commits into from
Nov 28, 2024

Conversation

shanejonas
Copy link
Member

@shanejonas shanejonas commented Nov 28, 2024

Update logs-react, playground, and inspector packages to use the latest React, React-DOM, and Monaco Editor versions, and replace rescripts with react-app-rewired.

  • Update dependencies:

    • Update react and react-dom to the latest versions in packages/playground/package.json.
    • Update monaco-editor to the latest version in packages/playground/package.json.
  • Replace rescripts with react-app-rewired:

    • Replace @rescripts/cli and @rescripts/utilities with react-app-rewired in devDependencies of packages/inspector/package.json, packages/logs-react/package.json, and packages/playground/package.json.
    • Update start, build, and test scripts to use react-app-rewired in packages/inspector/package.json, packages/logs-react/package.json, and packages/playground/package.json.
  • Rename and update configuration files:

    • Rename packages/inspector/monaco-rescript.js to packages/inspector/config-overrides.js and update content to use MonacoWebpackPlugin.
    • Rename packages/logs-react/rescript-monaco.js to packages/logs-react/config-overrides.js and update content to use MonacoWebpackPlugin.
    • Rename packages/playground/rescript-monaco.js to packages/playground/config-overrides.js and update content to use MonacoWebpackPlugin.
  • Remove unnecessary scripts:

    • Remove turbo run commands from scripts in packages/playground/package.json.

For more details, open the Copilot Workspace session.

Update `logs-react`, `playground`, and `inspector` packages to use the latest React, React-DOM, and Monaco Editor versions, and replace `rescripts` with `react-app-rewired`.

- **Update dependencies:**
  - Update `react` and `react-dom` to the latest versions in `packages/playground/package.json`.
  - Update `monaco-editor` to the latest version in `packages/playground/package.json`.

- **Replace `rescripts` with `react-app-rewired`:**
  - Replace `@rescripts/cli` and `@rescripts/utilities` with `react-app-rewired` in `devDependencies` of `packages/inspector/package.json`, `packages/logs-react/package.json`, and `packages/playground/package.json`.
  - Update `start`, `build`, and `test` scripts to use `react-app-rewired` in `packages/inspector/package.json`, `packages/logs-react/package.json`, and `packages/playground/package.json`.

- **Rename and update configuration files:**
  - Rename `packages/inspector/monaco-rescript.js` to `packages/inspector/config-overrides.js` and update content to use `MonacoWebpackPlugin`.
  - Rename `packages/logs-react/rescript-monaco.js` to `packages/logs-react/config-overrides.js` and update content to use `MonacoWebpackPlugin`.
  - Rename `packages/playground/rescript-monaco.js` to `packages/playground/config-overrides.js` and update content to use `MonacoWebpackPlugin`.

- **Remove unnecessary scripts:**
  - Remove `turbo run` commands from `scripts` in `packages/playground/package.json`.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/open-rpc/tools?shareId=XXXX-XXXX-XXXX-XXXX).
@@ -78,7 +72,7 @@
"typescript": "^3.7.3"
},
"rescripts": [
Copy link
Member Author

Choose a reason for hiding this comment

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

need to remove this

shanejonas and others added 4 commits November 28, 2024 18:18
…kage.json` to use `react-app-rewired`

* **`packages/inspector/package.json`**
  - Replace `@rescripts/cli` and `@rescripts/utilities` with `react-app-rewired` in `devDependencies`
  - Update `test` script to use `react-app-rewired`
  - Remove `rescripts` entry

* **`packages/playground/package.json`**
  - Replace `@rescripts/cli` and `@rescripts/utilities` with `react-app-rewired` in `devDependencies`
  - Remove `rescripts` entry
Update `logs-react`, `playground`, and `inspector` to the latest `react`, `react-dom`, and `monaco`, and replace `rescripts` with `react-app-rewired`.

* Update `react` and `react-dom` to version `18.2.0` in `packages/logs-react/package.json`, `packages/playground/package.json`, and `packages/inspector/package.json`.
* Update `monaco-editor` to version `0.34.1` in `packages/logs-react/package.json`, `packages/playground/package.json`, and `packages/inspector/package.json`.
* Replace `rescripts` with `react-app-rewired` in `packages/logs-react/package.json`, `packages/playground/package.json`, and `packages/inspector/package.json`.
* Rename `rescript-monaco.js` to `config-overrides.js` and ensure the `MonacoWebpackPlugin` configuration is equivalent to the old `rescript-monaco.js` in `packages/playground/config-overrides.js`.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/open-rpc/tools?shareId=XXXX-XXXX-XXXX-XXXX).
@shanejonas shanejonas merged commit c7699c9 into master Nov 28, 2024
0 of 3 checks passed
@BelfordZ BelfordZ deleted the shanejonas/update-react-monaco-rewired branch November 28, 2024 23:47
zcstarr added a commit that referenced this pull request Feb 26, 2025
# This is the 1st commit message:

feat: major update to use esm, material 6.xx and react 18

BREAKING CHANGE: this change will break packages that relied on
the older build system as there are multiple major package revision
for React, and material since the last release.

# This is the commit message #2:

fix: vite configuration and monaco

# This is the commit message #3:

fix: logs react to standarize exports

# This is the commit message #4:

fix: inspector works with vite

# This is the commit message #5:

fix: update json-schema-ref-parser

# This is the commit message #6:

fix: refactor editor to support v5/v6

# This is the commit message #7:

fix: rename default conversion to section

# This is the commit message #8:

fix: add updated packages for monaco-editor

# This is the commit message #9:

fix: cardListItem to use new Accordion

# This is the commit message #10:

fix: update to v6 styling

# This is the commit message #11:

chore: update to support v6

# This is the commit message #12:

chore: update to v6

# This is the commit message #13:

chore: update to v6

# This is the commit message #14:

chore: convert to v6

# This is the commit message #15:

chore: update v6

# This is the commit message #16:

chore: update to support react 19 and v6

# This is the commit message #17:

fix: nested button issue with mui refactor

# This is the commit message #18:

fix: wip module resolution tmp.

# This is the commit message #19:

feat: wip working build and editor

# This is the commit message #20:

feat: wip working build and editor

# This is the commit message #21:

fix: add missing package-lock.json

# This is the commit message #22:

chore: rm old fixtures

# This is the commit message #23:

fix: add support for theme color changes

# This is the commit message #24:

feat: wip working vite build with autocomplete

# This is the commit message #25:

fix: docs-react to build properly with vite

# This is the commit message #26:

fix: correct model not updating on document change

# This is the commit message #27:

feat: wip working playground
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.

2 participants