jsrepo v3 migration issue

[BuptStEve]

Hi~ieedan

My file structure is something like this.

src/components/notification/drawer
├── __snapshots__
│   └── drawer.test.ts.snap
├── demos
│   ├── drawer-body.meta.vue
│   ├── drawer-close.meta.vue
│   ├── drawer-content-swipeable.meta.vue
│   ├── drawer-content.meta.vue
│   ├── drawer-footer.meta.vue
│   ├── drawer-header.meta.vue
│   ├── drawer-overlay.meta.vue
│   ├── drawer-portal.meta.vue
│   ├── drawer-root.meta.vue
│   ├── drawer-trigger.meta.vue
│   ├── drawer.demo.01.vue
│   ├── drawer.demo.02.vue
│   ├── drawer.demo.03.vue
│   ├── drawer.demo.04.vue
│   ├── drawer.demo.05.vue
│   ├── drawer.demo.06.vue
│   ├── drawer.demo.07.vue
│   ├── index.meta.vue
│   └── index.vue
├── drawer-body.vue
├── drawer-close.vue
├── drawer-content-impl.vue
├── drawer-content-swipeable.vue
├── drawer-content.variants.ts
├── drawer-content.vue
├── drawer-footer.vue
├── drawer-header.vue
├── drawer-overlay.vue
├── drawer-portal.vue
├── drawer-root.vue
├── drawer-trigger.vue
├── drawer.test.ts
├── index.exports.ts
├── index.namespaced.ts
├── index.ts
└── useDrawerSwipe.ts

The migration tool works fine.

But I don't want to list all files. 😂

{
  name: 'drawer',
  add: 'when-added',
  type: 'notification',
  files: [
    {
      path: 'src/components/notification/drawer',
      files: [
        {
          path: 'drawer-body.vue',
        },
        {
          path: 'drawer-close.vue',
        },
        {
          path: 'drawer-content-impl.vue',
        },
        {
          path: 'drawer-content-swipeable.vue',
        },
        {
          path: 'drawer-content.variants.ts',
        },
        {
          path: 'drawer-content.vue',
        },
        {
          path: 'drawer-footer.vue',
        },
        {
          path: 'drawer-header.vue',
        },
        {
          path: 'drawer-overlay.vue',
        },
        {
          path: 'drawer-portal.vue',
        },
        {
          path: 'drawer-root.vue',
        },
        {
          path: 'drawer-trigger.vue',
        },
        {
          path: 'drawer.test.ts',
          role: 'test',
        },
        {
          path: 'index.exports.ts',
        },
        {
          path: 'index.namespaced.ts',
        },
        {
          path: 'index.ts',
        },
        {
          path: 'useDrawerSwipe.ts',
        },
      ],
    },
  ],
}

Maybe add some options like excludePath?
So I can exclude demos/ and __snapshots__/ folder.

[ieedan]

In this case I would probably recommend you write your own function to collect the files for the registry. It's a slippery slope to start adding options like that.

You can check out reactbits for a good example of this

[BuptStEve]

Yeah, I've seen the jsrepo.config of reactbits and your shadcn-svelte-extras 😂

[BuptStEve]

BTW how about resolving snapshots as test files as default?

┌   jsrepo  v3.0.8
│
▲  Couldn't find a language to resolve dependencies for src/components/base/button/__snapshots__/button.test.ts.snap.

[ieedan]

Just add the role to the folder then

[BuptStEve]

I don't get it, it's a nested folder?

Duplicate file reference: button/__snapshots__/button.test.ts.snap is in both ui/button and ui/button. Please ensure the file is not referenced multiple times.

{
  name: 'button',
  type: 'ui',
  files: [
    {
      path: 'src/components/base/button/__snapshots__',
      role: 'test',
    },
    {
      path: 'src/components/base/button',
    },
  ],
}

[ieedan]

Yeah you will have to provide the individual files since you're also referencing the same directory.

You could also just write a bit of code to accomplish that in the config file

[BuptStEve]

1. Jsrepo automatically mark files end with .snap as test files.

2. Path support glob syntax, for example:

{
  name: 'button',
  type: 'ui',
  files: [
    {
      path: 'src/components/base/button/*',
    },
  ],
}

3. Support add a function to return the role of the file. Maybe like this

{
  name: 'button',
  type: 'ui',
  files: [
    {
      path: 'src/components/base/button',
      getRegistryItemFile: (file: RegistryItemFile) => ({
        ...file,
        role: file.path.endsWith('.snap') ? 'test' : file.role,
      })
    },
  ],
}

[ieedan]

It should be possible to support glob imports but we wont be doing any auto detection for test files at least for now