add a new flag with --open for the dev and the preview

[JerryWu1234]

Changes

fix bug : #6578
I have added a new flag with --open for the dev and the preview.
And have extended the config of the schema with a property of open.

I didn't find any good method to test this property of open.

Testing

Docs

[changeset-bot]

🦋 Changeset detected

Latest commit: 5feafad

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[bluwy]

I think we also need to document the CLI flags for --help:

astro/packages/astro/src/core/dev/dev.ts

Lines 44 to 47 in da0cfeb

	['--port', `Specify which port to run on. Defaults to 3000.`],
	['--host', `Listen on all addresses, including LAN and public addresses.`],
	['--host <custom-address>', `Expose on a network IP address at <custom-address>`],
	['--help (-h)', 'See all available flags.'],

astro/packages/astro/src/core/preview/index.ts

Line 29 in da0cfeb

Flags: [['--help (-h)', 'See all available flags.']],

I tried running this locally, but I'm not sure it's working. I needed to handle the open option here to work:

astro/packages/astro/src/core/config/config.ts

Lines 94 to 105 in da0cfeb

	return {
	root: typeof flags.root === 'string' ? flags.root : undefined,
	site: typeof flags.site === 'string' ? flags.site : undefined,
	base: typeof flags.base === 'string' ? flags.base : undefined,
	port: typeof flags.port === 'number' ? flags.port : undefined,
	config: typeof flags.config === 'string' ? flags.config : undefined,
	host:
	typeof flags.host === 'string' || typeof flags.host === 'boolean' ? flags.host : undefined,
	drafts: typeof flags.drafts === 'boolean' ? flags.drafts : undefined,
	experimentalAssets:
	typeof flags.experimentalAssets === 'boolean' ? flags.experimentalAssets : undefined,
	};

[ematipico]

We need to add this new option to the help command

[ematipico]

@withastro/maintainers-docs could you please double check the description of the new argument?

[ematipico]

@wulinsheng123 I could suggest an option for testing:

• share the link to the vite option that we leverage
• share a video/gif that shows the new argument in action

[JerryWu1234]

@wulinsheng123 I could suggest an option for testing:

• share the link to the vite option that we leverage
• share a video/gif that shows the new argument in action

What can I do without Astro including a new feature with the property of open?

[JerryWu1234]

@wulinsheng123 I could suggest an option for testing:

• share the link to the vite option that we leverage
• share a video/gif that shows the new argument in action

Let me try to record a video

[JerryWu1234]

• share the link to the vite option that we leverage

I apologized.
I don't totally get what you said.
And don't know how to achieve it
could you give me a tips ?
@ematipico

[JerryWu1234]

https://cdn.discordapp.com/attachments/984757097726107661/1087992177856094308/VID20230322144505.mp4

@ematipico
This video help to check

[bluwy]

Updated the flag name slightly. LGTM. Thanks for implementing this!