Skip to content

Conversation

@anmolsinghbhatia
Copy link
Collaborator

Summary

  • Fixes Storybook failing to load in the propel package with a 500 Internal Server Error
  • Renames postcss.config.jspostcss.config.cjs to resolve ES module conflict

Problem

The @plane/propel package has "type": "module" in package.json, which makes Node.js treat all .js files as ES modules. However, postcss.config.js uses CommonJS syntax (module.exports), causing Vite/Storybook to fail with:

"module is not defined in ES module scope"

Solution

Rename the file to .cjs extension, which explicitly tells Node.js to treat it as CommonJS regardless of the package type setting.

Type of Change

  • Bug fix

@anmolsinghbhatia anmolsinghbhatia self-assigned this Dec 3, 2025
Copilot AI review requested due to automatic review settings December 3, 2025 12:57
@makeplane
Copy link

makeplane bot commented Dec 3, 2025

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 3, 2025

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

💤 Files selected but had no reviewable changes (1)
  • packages/propel/postcss.config.cjs

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-propel-storybook-postcss-config

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sriramveeraghanta sriramveeraghanta merged commit 9699a01 into preview Dec 4, 2025
11 of 13 checks passed
@sriramveeraghanta sriramveeraghanta deleted the fix-propel-storybook-postcss-config branch December 4, 2025 10:07
ClarenceChen0627 pushed a commit to ClarenceChen0627/plane that referenced this pull request Dec 5, 2025
ClarenceChen0627 pushed a commit to ClarenceChen0627/plane that referenced this pull request Dec 5, 2025
ClarenceChen0627 pushed a commit to ClarenceChen0627/plane that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants