Skip to content

[migrate] upgrade to StoryBook 8 with Parcel 2#53

Closed
TechQuery wants to merge 10 commits intomasterfrom
StoryBook-8
Closed

[migrate] upgrade to StoryBook 8 with Parcel 2#53
TechQuery wants to merge 10 commits intomasterfrom
StoryBook-8

Conversation

@TechQuery
Copy link
Copy Markdown
Member

@TechQuery TechQuery commented Apr 9, 2024

PR-53 PR-53 PR-53 Powered by Pull Request Badge

  1. [migrate] replace Vite with Parcel to simplify StoryBook configuration
  2. [optimize] use TS for ESLint configuration
  3. [optimize] upgrade to Node.js 22, PNPM 10, React 19, StoryBook 8 & other latest Upstream packages
  4. [fix] React base class of MobX components
  5. [fix] Timeline component layout
  6. [fix] PrismJS CSS compatible bug

[optimize] upgrade to PNPM v9 & other latest Upstream packages
[fix] ESM path extension for Vite
[optimize] update Upstream packages
Comment thread vite.config.ts Outdated
find: 'react/jsx-runtime',
replacement: 'react/jsx-runtime.js'
},
{ find: /^(react-dom\/.+)/, replacement: '$1.js' },
Copy link
Copy Markdown
Member Author

@TechQuery TechQuery Nov 16, 2024

Choose a reason for hiding this comment

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

This line has a similar bug in Development environment: storybookjs/storybook#24903 (comment)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The vite.config.ts file (and its dev environment alias bug) was resolved by the subsequent commit migrating from Vite to Parcel (980874e). The Parcel-based setup doesn't require those .js extension aliases.

[optimize] upgrade to Node.js 22 & other latest Upstream packages
@TechQuery TechQuery changed the title [optimize] upgrade to StoryBook 8 & other latest Upstream packages [migrate] upgrade to StoryBook 8 & Parcel 2 Jan 22, 2025
@TechQuery TechQuery changed the title [migrate] upgrade to StoryBook 8 & Parcel 2 [migrate] upgrade to StoryBook 8 with Parcel 2 Jan 22, 2025
[optimize] use TS for ESLint configuration
[optimize] upgrade to PNPM 10, React 19 & other latest Upstream packages
@TechQuery
Copy link
Copy Markdown
Member Author

@copilot 将本分支的依赖升级到最新后,解决构建失败的 bug。

- Override .storybook/.parcelrc to use @parcel/transformer-typescript-tsc
  for TypeScript files (fixes Stage 3 decorator support)
- Patch @parcel/resolver-storybook@0.0.0 for React 19 compatibility
  (react-dom/client.js → react-dom/client for React 19+)
- Patch @mdx-js/react@3.1.0 to add back mdx compatibility shim
  (needed by @parcel/transformer-mdx@2.13.3)

Agent-Logs-Url: https://github.com/idea2app/Idea-React/sessions/faacf000-63b8-4a64-b539-9ab698c7375b
Copilot AI review requested due to automatic review settings April 28, 2026 22:40
@TechQuery TechQuery review requested due to automatic review settings April 28, 2026 22:40
Copilot AI review requested due to automatic review settings April 28, 2026 22:43
@TechQuery TechQuery review requested due to automatic review settings April 28, 2026 22:43
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 28, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • storybook.js.org
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/Idea-React/Idea-React/node_modules/.bin/../storybook/bin/index.cjs build -c .storybook (dns block)
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/Idea-React/Idea-React/node_modules/.bin/../storybook/bin/index.cjs build -c .storybook ules/@parcel/config-storybook/ (dns block)
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/Idea-React/Idea-React/node_modules/.bin/../storybook/bin/index.cjs build -c .storybook --local pull.rebase /usr/local/bin/bash (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copy link
Copy Markdown
Member Author

@TechQuery TechQuery left a comment

Choose a reason for hiding this comment

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

@copilot 修改我的评论的同时,在 Read Me 中注明打补丁的原因。

Comment thread patches/@parcel__resolver-storybook@0.0.0.patch Outdated
Copilot AI review requested due to automatic review settings April 28, 2026 22:55
@TechQuery TechQuery review requested due to automatic review settings April 28, 2026 22:55
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 28, 2026

@copilot 修改我的评论的同时,在 Read Me 中注明打补丁的原因。

Added a "Patches" section to the README in d4f3326 explaining the reason for each pnpm patch.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • storybook.js.org
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/Idea-React/Idea-React/node_modules/.bin/../storybook/bin/index.cjs build -c .storybook (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copy link
Copy Markdown
Member Author

@TechQuery TechQuery left a comment

Choose a reason for hiding this comment

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

@copilot 严格按照我的要求改

Comment thread package.json
Comment thread ReadMe.md Outdated
Copilot AI review requested due to automatic review settings April 28, 2026 23:08
@TechQuery TechQuery review requested due to automatic review settings April 28, 2026 23:08
@TechQuery
Copy link
Copy Markdown
Member Author

Replaced with #63

@TechQuery TechQuery closed this Apr 29, 2026
@github-project-automation github-project-automation Bot moved this from In progress to Done in 开源项目 Apr 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Some improvements

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants