Make StoryObj more strongly typed

[emillaine]

Is your feature request related to a problem? Please describe.

Case 1

I just wrote a typo in a story definition, and it took a while to figure out the issue:

type Story = StoryObj<typeof MyComponent>;
export const Default: Story = { /* ... */ };
export const StoryName: Story = {
  ...Default.args,
  args: {
    ...Default.args,
    // other args
  }
};

(the first ...Default.args should be ...Default)

Case 2

The args of StoryObj is typed as Partial<MyComponentProps> instead of simply MyComponentProps.

This has lead to forgetting to specify a required prop in args, multiple times.

Describe the solution you'd like

Case 1

Is it possible to make StoryObj catch this by using stronger typings?

Case 2

Is it possible to remove Partial from the args type, so that required props stay required?

Describe alternatives you've considered

No response

Are you able to assist to bring the feature to reality?

no

Additional context

No response

[valentinpalkovic]

I think @kasperpeulen can explain why Partial is required for args.

[kasperpeulen]

@kylegach @cdedreuille @jonniebigodes I completely agree, we should switch the defaults ASAP. This is getting confusing. TS4.9 is out for a long time already.

[jonniebigodes]

@emlai thanks for reaching out to us with your question. We appreciate it 🙏 ! To give you some context on it, we launched support for TS-4.9 a while ago and enabled it in the dropdown so that you and other users had the option and also gave you some time to upgrade. However, one thing to point out though is that we cannot guarantee that everyone has already migrated to said version. If we commit to this, we'll have to produce the snippets accordingly to prevent the site from reporting a missing snippet and potentially re-arrange the logic of how it's presented. I'll follow up with Kyle Gach when he's back so that we can have a path moving forward. Sounds good to y'all?

Let me know, and we'll take it from there.

cc @kasperpeulen

[emillaine]

Sounds great. Thank you! ❤️

[cdedreuille]

This should be solved with the new docs design we are working on. That said I agree with @kasperpeulen that we should put TS4.9 as default. Even I was confused about it sometimes.

[emillaine]

Btw now that I'm declaring stories using const meta = { ... } satisfies Meta<typeof MyComponent>, I'm encountering a new TypeScript error when using decorators in meta. Created a new issue for it: #24656