fix: Run dev command only for core packages (no-changelog)

[alexgrozav]

Summary

Provide details about your pull request and what it adds, fixes, or changes. Photos and videos are recommended.

• Filters out @n8n/chat and n8n-design-system from pnpm run dev
• Fixes dev/build error appearing due to code string import ... from '@n8n/chat'

How to test the change:

1. pnpm run dev should work as expected
2. The command should not run for the two packages above
3. The error regarding @n8n/chat should no longer appear

Issues fixed

Include links to Github issue or Community forum post or Linear ticket:

Important in order to close automatically and provide context to reviewers

https://linear.app/n8n/issue/PAY-1089/run-dev-command-only-for-core-packages

Review / Merge checklist

• PR title and summary are descriptive. Remember, the title automatically goes into the changelog. Use (no-changelog) otherwise. (conventions)
• Docs updated or follow-up ticket created.
• Tests included. N/A

  A bug is not considered fixed, unless a test is added to prevent it from happening again. A feature is not complete without tests.

  (internal) You can use Slack commands to trigger e2e tests or deploy test instance or deploy early access version on Cloud.

[ivov]

Adi is OOO - quick question, why don't we want dev for design-system? We'll have to rebuild for every change when making a DS component?

[alexgrozav]

Adi is OOO - quick question, why don't we want dev for design-system? We'll have to rebuild for every change when making a DS component?

Design system's npm run dev only starts storybook. The design system's source files are referenced directly in editor-ui so you'll get the latest version of the code at any time.

[cypress]

1 flaky test on run #3200 ↗︎

0

297

5

0

1

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 alexgrozav 🗃️ e2e/*
Project: n8n	Commit: 77fea106e2
Status: Passed	Duration: 06:37 💡
Started: Dec 5, 2023 4:49 PM	Ended: Dec 5, 2023 4:56 PM

  cypress/e2e/26-resource-locator.cy.ts • 1 flaky test

View Output Video

Test		Artifacts
Resource Locator > should retrieve list options when other params throw errors		Screenshots Video

Review all test suite changes for PR #7928 ↗︎

[github-actions]

✅ All Cypress E2E specs passed

[janober]

Got released with [email protected]