Ensure CLI flags override function-style server config

[bluwy]

Changes

Fix #5102

Revive the attached CLI flags props from

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

Lines 118 to 127 in c0db581

	if (typeof flags.port === 'number') {
	// @ts-expect-error astroConfig.server may be a function, but TS doesn't like attaching properties to a function.
	// TODO: Come back here and refactor to remove this expected error.
	astroConfig.server.port = flags.port;
	}
	if (typeof flags.host === 'string' || typeof flags.host === 'boolean') {
	// @ts-expect-error astroConfig.server may be a function, but TS doesn't like attaching properties to a function.
	// TODO: Come back here and refactor to remove this expected error.
	astroConfig.server.host = flags.host;
	}

Testing

Updated the astro-basic config that would cause cli.test.js to fail without this PR

Docs

n/a

[changeset-bot]

🦋 Changeset detected

Latest commit: c0db581

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

[natemoo-re]

Nice fix!