Skip to content

Commit

Permalink
fix: config loading
Browse files Browse the repository at this point in the history
  • Loading branch information
segunadebayo committed May 17, 2024
1 parent 4e169b6 commit 2f63a4c
Show file tree
Hide file tree
Showing 4 changed files with 26 additions and 3 deletions.
5 changes: 5 additions & 0 deletions .changeset/hip-trains-protect.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@pandacss/config': patch
---

Fix issue where panda could load unrelated config files that look like a config e.g. `theming-panda.config.ts`
1 change: 1 addition & 0 deletions .npmrc
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,4 @@ auto-install-peers=true
#
link-workspace-packages=true
prefer-workspace-packages=true
package-manager-strict=false
9 changes: 9 additions & 0 deletions packages/config/__tests__/is-panda-config.test.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
import { isPandaConfig } from '../src/is-panda-config'

describe('is-panda-config', () => {
test('should work as expected', () => {
expect(isPandaConfig('panda.config.ts')).toBe(true)
expect(isPandaConfig('testing-panda.config.ts')).toBe(false)
expect(isPandaConfig('panda-config.ts')).toBe(false)
})
})
14 changes: 11 additions & 3 deletions packages/config/src/is-panda-config.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,12 @@
const configs = ['.ts', '.js', '.mts', '.mjs', '.cts', '.cjs']
const pandaConfigRegex = new RegExp(`panda.config(${configs.join('|')})$`)
const configName = 'panda'

export const isPandaConfig = (file: string) => pandaConfigRegex.test(file)
const pandaConfigFiles = new Set([
`${configName}.config.ts`,
`${configName}.config.js`,
`${configName}.config.mts`,
`${configName}.config.mjs`,
`${configName}.config.cts`,
`${configName}.config.cjs`,
])

export const isPandaConfig = (file: string) => pandaConfigFiles.has(file)

0 comments on commit 2f63a4c

Please sign in to comment.